It seems that the key "--net darkcoin" is not needed.
Port 8999 is required to connect local pool to the global pool (transport).


More comments plox.


close #8


so, can we use `is_file` ?


@ddfreyne how do you do the @nanoc/contributors thing?


I think we should also log a warning, it doesn't seem right to not have it. WDYT?


s/Context CM/Context Root CM


Added a proxy function for popd #8


Debug enabled option in MConf?


According to Ray Wenderlich style it is preferred not to have empty lines between method declarations.


Whitespace errors in Git vs Eclipse, weird.


tu peux simplement écrire : `public Entry() {}`, ça fait pareil :)


```---------------------------------------------------------------
marker             time index            ex time         perct
---------------------------------------------------------------
Start              1422440473.88987800   -                0.00%
---------------------------------------------------------------
filemtime mark     1422440473.91307400   0.023196        18.12%
---------------------------------------------------------------
file_exists mark   1422440473.95806700   0.044993        35.14%
---------------------------------------------------------------
is_file mark       1422440473.97341300   0.015346        11.99%
---------------------------------------------------------------
is_readable mark   1422440474.01790700   0.044494        34.75%
---------------------------------------------------------------
Stop               1422440474.01791500   0.000008         0.01%
---------------------------------------------------------------
total              -                     0.128037       100.00%
---------------------------------------------------------------


Перевода строки не будет? Согласно стандарту перед доктайпом не должно быть вообще ничего, ни пробела ни перевода строки.


You're right, this is not intended. I have pushed a fix to develop.

Cheers


if you are about to add the RVM files - please make sure they have new lines at the end - it is required to properly read shell files


idem qu'au-dessus, avec une initialisation sur les champs pour tout ce qui est non null


idem que premier commentaire


Small check to this existing horrible mess.
``` ruby
if res.nil? || res.headers['Location'].include?('action=Login') || res.get_cookies.empty?
```


TeamCity Ybackend :: Feodorov/ybackend [Build 87](http://192.168.30.225:8111/viewLog.html?buildId=679&buildTypeId=Ybackend_FeodorovYbackend&guest=1) is now running


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 158](http://192.168.30.225:8111/viewLog.html?buildId=680&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 158](http://192.168.30.225:8111/viewLog.html?buildId=680&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 127 Build time: 00:00:01


Your blog does not allow comments, so I am commenting here:
You might like Scrivener
http://www.literatureandlatte.com/scrivener.php
And thanks for your blog post.


TeamCity Ybackend :: Feodorov/ybackend [Build 87](http://192.168.30.225:8111/viewLog.html?buildId=679&buildTypeId=Ybackend_FeodorovYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:16


@jeromenadaud thank you. `is_file` is much faster than `filemtime`


Hi Wifs,
Your site datagamer.fr looks so cool.
I also want to build someting. But where did you find the VPS for your website? Are there any services free for us?


Note: Json parsing is optional not required. Currently we don't handle escaping strings such as "


Проверял, перевода строки нет.


Шаблон девелопер будет адаптивным?


`/` in the beginning means only at the root of the repository, why the removal?


I bet you believe it's you? kappa


Ага, уже в процессе


Hello my name is Hendrik Hintze and i am an informatics student an i am using InteliJ, but the free version ends in 6 days. It would be verry kind of you if u could add my school to the universitylist so that i can continue my work. Thank you and have a good day


Should fix #120


It took much longer to code than I expected. I had to learn lots of new things to map those "buttons", they do not work like normal buttons. Only those I used in the first script are normal buttons.


@ericmj @devinus @fishcakez feedback on this is very welcome.

PS: I will move the connection start to a supervision tree soon.


Clean understandable translation.


The order of properties should be storage then atomicity, which is consistent with automatically generated code when connecting UI elements from Interface Builder. If you made a separation line before the first comment, you should  make a separation like before any other property comment as well


Enter was moved to :arrow_forward:.


Did you fix this?


Может синьйо тогда не верстать? Пусть будет из коробки один, но хороший. И проблему совместимости и корректной работы плагинов решим.


Fixed, thanks!


Это по идее название компонента, но согласен название довольно частоупотребляемое, мы вроде как префиксы планируем добавить классам компонентов, так что проблем не должно быть, если что переименуем компонент.


Unfortunately reality intervened :/
(#2021)


Do you want to add 1 or change to value to 1? I think what is your doing is replacing the value by '1'.


weak should go first


Please add translation.


Please add translation


why you have space after void ?


You're creating a new Vote here, and then another one below (build and save). That may be why you have a total of 2.


Can you elaborate?


Haha :)


Need to use try and catch, because if no key is found in PyTest config, Attribute exception is raised.
Also, when using runner script, pytest config can be called before pytest is initialized = therefore we need to handle AttributeException too.


You should use `save!` so that if Rails cannot save the instance it raises an exception. With `save` it will return true or false, and because you're not using the return value of your method you will miss the problem.


Тоже думал над этим, синио можно попробовать сделать как тему для девелопера.


Неееет, неееет :disappointed: . Мы же договорились выкурить этот анархоизм, не использовать oConfig, cfg...
Только ```Config::Get()```


`cfg[0]` must be used = this referes to JSON. `cfg` itself is touple containing JSON and list name.
Also, if key exists but it's value is empty, we don't want to return this empty value = hence updated if clause.


Forgot about this. Done.


order was wrong which caused variables being incorrectly mapped = test execution failed because Selenium driver could not be started.


Поправлю :smile:


Browserstack API expects touples and its methods failed when list was supplied


@jeljaik the problem is that the dimensions are hardcoded! We have the assumption of robot parts.
So the change `23 -> ROBOT_DOF` is quite useless.  IMHO.


>мы вроде как префиксы планируем добавить классам компонентов, так что проблем не должно быть

так: ```.ls-item``` ?


we need always to return JSON response object, `.json` property does not return it


Sometimes you have the first line of the method empty and sometimes don't.
Please delete those empty lines.


Oh my god, indeed. Thanks


yes, fixed with f05dffb525f96e8a02ff9ca6ace17cc0c808e91c


This wasn't actually calling `return` (ie it wasn't exiting early), so `fail_with` is a change of behaviour here.


Thanks for the tweaks, Robert.


Great !!

Just a remark : all commits should be linked to issues. This commit should be at least linked to issues #528 and #641 .


Только за. _Одна рука вверх, две, ноги поднять? Считается?_


Да


That's correct now.


WAT? from 0.2.2 to 3.0.0?


Thanks, Phil - that completely slipped my mind.


>синио можно попробовать сделать как тему для девелопера.

Плюс за это ещё и потому, что при этом можно проверить насколько хорошо проработана новая архитектура шаблона и найти её уязвимые и тонкие места, т.к. создание такой темы потребует творческих усилий.


Ok, I didn't know this.


The ENSIGHT10_READER variable is not set anywhere so there is no need to unset it.


Тут больше руки и ноги Максима считаются :laughing: Надо его мнение услышать.


Should be moved to above the parent if statement to remove redundant code


The handling of CGAL is now in CGAL.sh/csh
I have added 1>&2 to CGAL.sh


This changes fails as the $(WM_THIRD_PARTY_DIR)/platforms/include directory does not exist even after compiling libccmio-2.6.1


I rm -rf'd the dir and repulled the whole project, just to be sure. Still happening. I'm cursed. nn+ still working though. Wonder if something in my config could be messing things up.


Merged into OpenFOAM-dev


testtest


Would love to see a new release cut for pypi! :+1:


А фото с его руками подойдут в счет? :laughing:
Ждем


Interesting. I have not heard of `operator-surround`, it looks interesting. I'm not sure I understand completely, though: I like the `surround.vim` idea with mappings such as `ds...` and `cs...`. Is it possible to use `operator-surround` to replace this functionality?


this line


Если получится одним css из developer получить synio - отлично. Но я сомневаюсь, что это возможно.
Цель synio - привлекать новых пользователей/клиентов при первом знакомстве с LS, цель developer - облегчить жизнь разработчикам.


GetItemPtr now guards this in upstream


Yeah, it's not a big issue. It can also be useful for collecting diagnostic information from customers.


and this line define the same mapping "gt". Are you sure it will work as expected?


Oh, and is it possible to use the script local mappings to get the desired functionality without using either of these plugins?


> If you made this work, you did a hell of a job ;)

It seems that the way it is.
Btw, a lot of things are messed here :( starting from not mounting sdcards and ends to not working deepsleep. A lot work will be needed to made it bugfree...


Must be add to trinity too


With this line app works fine, I didn't check if it is there, wasn't sure how to.


You *totally* rock. Thank you! <3


Would this not be better as a class? Keep styles separate and all that.


Actually tried out it locally - it works. But I don't quite understand why. Does duplicated mappings override previous?


/c @krisajenkins


Oops :smile:


@jeromenadaud What environment did you use for testing?

When I run your test with php-cli I get same results as you — filemtime is faster than file_exists

But calling script through web server gave opposite results and file_exists worked faster. (Tested on CentOS 6.3 PHP 5.4.36 apache+mod_php and Ubuntu 14.04.1 LTS PHP 5.5.20 nginx+php-fpm)

```
---------------------------------------------------------------
marker             time index            ex time         perct
---------------------------------------------------------------
Start              1422441930.19970600   -                0.00%
---------------------------------------------------------------
filemtime mark     1422441930.30069100   0.100985        59.11%
---------------------------------------------------------------
file_exists mark   1422441930.34061400   0.039923        23.37%
---------------------------------------------------------------
is_file mark       1422441930.37054700   0.029933        17.52%
---------------------------------------------------------------
Stop               1422441930.37055000   0.000003         0.00%
---------------------------------------------------------------
total              -                     0.170844       100.00%
---------------------------------------------------------------
```


Sublime Text z nastavitvami (ki so default):

    // Set to true to insert spaces when tab is pressed
    "translate_tabs_to_spaces": false,
    // If translate_tabs_to_spaces is true, use_tab_stops will make tab and
    // backspace insert/delete up to the next tabstop
    "use_tab_stops": true,


Quite right ;)


>Если получится одним css из developer получить synio - отлично.

а почему только одним ксс? имя темы можно получить через конфиг, можно в шаблоне прописывать что-то вроде:
```smarty
{include 'list'|cat:Config::Get('view.theme')}
```
об этом я ещё говорил когда добавлял в админку переключение тем и их предпросмотр. Но я говори о внесении этого (темы) прямо в функцию смарти ```include```. Что позволит делать разные шаблоны для разных тем, просто положив новый шаблон для темы рядом с файлом:
```
list.tpl
list.theme_name.tpl
```

>Цель synio - привлекать новых пользователей/клиентов при первом знакомстве с LS

с этим я согласен.

P.S. Какая же все-таки классная подсветка синтаксиса у гитхаба. Нужно такую же тему на prettyfy сделать.


Added other file types


Можно попросить Лешу, чтобы он немного подогнал синио под структуру девелопера :wink:


The difference is likely the use of `arguments`, not apply vs call.


@pavelnovitsky but `is_file` is still faster even in your environment. Why not using `is_file` everywhere as possible ? It solve all problems: is the fastest, is code-readable, do extactly what it means: check if a file is a file.


I am a nice comment


Added other file types to be ignored


> The bind mounting of docker is not going to work everywhere

Yes indeed, as noted in the commit message. I don't think we can do much better unless we include the client (at 14MB, it more than doubles the image size), or rewrite the script (which would probably end up being quite big as well, as a Go executable).


@bradenmacdonald this is not exactly a correct way to fix this. In previous version it was fixed like this: https://github.com/open-craft/xblock-mentoring/commit/1a6e2092a361f0befff18d07607517d437046e3b#diff-23a9d1f1d9cda06ac8fa67c8c2f4589dR130-131

Now the code that deserializes XML into mentoring XBlocks are in main XBlock repository: https://github.com/edx/XBlock/blob/9c634481dfc85a17dcb3351ca232d7098a38e10e/xblock/runtime.py#L685

This is probably a good idea to teach upstream XBlock to play nicely with XML comments


Освежил его, так сказать? :+1:


@bradenmacdonald not a generator anymore :)


naming sounds a bit weird (maybe `classloader.delegate` true/false - it's named that way in libraries such as Tomcat), or considering there's a growing hierarchy of class-loaders these days (wanted?) it should be possible to switch them with a name specified as a string option e.g. `classloader.type`. alternatively delegate more work to the existing loader (I personally would welcome a `delegate` flag on `JRubyClassLoader` instead but I do not know the consequences) ...


Networking part is over! Now just gather 1 million users.


:-1: If that happens, I'll fork co and republish it.


This should just grab the data from `/govuk-info/info-statistics` and pass it straight through. Unfortunately it's currently timing out.


Fucking tard...


This mostly fixes https://github.com/schemaorg/schemaorg/issues/294


Why all this promise stuff? Devastating.


Excellent, thanks ^_^


~~yashoon~~


@jason


You have this license?


yeah, ideally we'd not generate errors here at all


> Освежил его, так сказать?

Да, он его, вроде как, в любом случае переделывать под современный лад (флэт дизайн итд итп) собирался.

> Нужно такую же тему на prettyfy сделать.

У нас уже используется highlight.js и по умолчанию включены именно гитхабовские стили (хотя не уверен что один в один с текущей подсветкой гитхаба)


Hi @jocel1,

I know this is not common but for the moment PS code norm stands that braces are prohibited when they only define one instruction and if there are brackets for a block, a line feed is needed.

```
if (count($counts))
	foreach ($manufacturers as $key => $manufacturer)
		if (isset($counts[(int)$manufacturer['id_manufacturer']]))
			$manufacturers[$key]['nb_products'] = $counts[(int)$manufacturer['id_manufacturer']];
		else
			$manufacturers[$key]['nb_products'] = 0;
```
Best regards


update the firmware, not just the module


This seems a bit surprising to me, I feel like `&Foo`, `Box<Foo>`, or `Rc<Foo>` where `Foo` is local may want to be considered? Put another way, I wonder if we want to check whether any of the deref steps are local, and not the receiver type as a whole?


奶奶环境保护价格


Shouldn't these be `handle_call(:link_me, ...)`?


I agree. @jocel1 what do you recommend ?

Regards


Sort.onClose 로직 보고 예외 케이스 잘 처리하기


없어져도 되지 않을까?
그냥 onPush(Row)하게 내비두자.


삭제


Tell me more sweety. :dancer:


I don't think "was done" is a good.


딴 클래스 보니까, 인터럽트 익셉션도 처리 하던데 어떨지 보자.


yes, it was long string with loads of dots. 'classloader.delegate' sounds
good. self.first comes from plexus (maven library) where I bored the code
from.

delegate switch on the classloader - let's see once travis is green again.


투두 삭제.


Hi, we must make more benchmark on other FS and check performance. If is_file still have the best mark we will make some changes in PrestaShop files.

Best regards.


Yeah, fixed in a later commit. :(


@gboulay Thanks for changing it. But I meant to use the existing code if the condition (node != self->first) is not met. We already have similar code in lru_remove_node etc and is better to reuse.


Is the default value for accept_ra voluntary absent or is it missed ?

```
  #ignore RAs on Ipv6
  sysctl { 'net.ipv6.conf.all.accept_ra': value => '0' }
  #??? sysctl { 'net.ipv6.conf.default.accept_ra': value => '0' } ???

```


@guestisp yes, `is_file()` is a little bit faster and is already used in autoload class. So why not to use it in this case.

BUT
1) I'm talking about different test results for one and the same script
2) We are discussing a small part of code that is not a bottleneck. 0.01 sec difference is not the thing  should be optimized now (ex. sql queries in loops and creating objects inside static methods are much slower)


Build failed with:
"No version specified. Downloading latest stable.
The remote server returned an error: (403) Forbidden."
I just changed XamarinHelper.fs file, is there a problem with a nuget server ?


Arse!


I had to remove `..` from QIcon parameter in order to have the icon set.  Looking at the generated `resources.cpp` suggests that correct path is indeed `:/icons/32x32/cool-retro-term.png`).

This is under FreeBSD which packages pretty vanilla Qt5; YMMV.


@pavelnovitsky  Totally agree.
We are discussing for 0.01 sec and how to use a *wrong* function to check for file existance to gain 0.01 seconds for the *first* request (as subsequential request are cached)

Please use the proper function `file_exists` or `is_file` and not the wrong function like silenced `filemtime`.
There are many more optimization that must be done in PS.

Having a 0.01 seconds gain in FS check when there are tons of queries in a loop to be fixed is absolutely non sense.


reack from fabian


Мне кажется synio надо делать как отдельный шаблон именно для той цели, чтобы посмотреть насколько developer удобен для того, чтобы на его базе что-то делать.

synio как тема это, на мой взгляд, какой-то костыльный вариант - все таки какая-то "тема" должна отличаться от исходного шаблона не очень, да может быть просто сделать для developer'a тему с какими-то очень незначительными изменениями, чтобы посмотреть/показать как это можно делать в рамках одного шаблона, но вот synio в качестве темы к developer'у это не айс. Девелопер должен быть минимально нагружен для лучшего понимания и изготовления других шаблонов на его основе.


Okay, I understand now, thanks for the help~


+ cp conf/init/webvirtmgr-redhat.conf /builddir/build/BUILDROOT/webvirtmgr-4.8.9-1.el6.x86_64/etc/init/webvirtmgr.conf
cp: cannot stat `conf/init/webvirtmgr-redhat.conf': No such file or directory


It is, the only difference is that operator-surround targets text-objects whereas vim-surrounds targets surroundings. For example, dsab in operator-surround translates to dsb in vim-surround.

For example, I set up the following maps that also replicate those of vim-surround
```
" operator mappings
nmap <silent>gs     <Plug>(operator-surround-append)
nmap <silent>gS     <Plug>(operator-surround-delete)
nmap <silent>g<c-s> <Plug>(operator-surround-replace)
xmap <silent>gs     <Plug>(operator-surround-append)
xmap <silent>gS     <Plug>(operator-surround-delete)
xmap <silent>g<c-s> <Plug>(operator-surround-replace)

nmap <silent>gss         V<Plug>(operator-surround-append)
nmap <silent>gSS         V<Plug>(operator-surround-delete)
nmap <silent>g<c-s><c-s> V<Plug>(operator-surround-replace)

" compatibility with original vim-surround by tpope
nmap ys                  gs
nmap ds                  gS
nmap cs                  g<c-s>
nmap yss                 gss
nmap dss                 gSS
nmap css                 g<c-s><c-s>
```
Without neither plugin, how about replacing `dsB` by
```
vaBm`o<esc>x``x
```
?


looking at your kernel PR you used this wlan module with 4.3 kernel i guess ? then it's not going to work. use older wlan modules


It's only been like 7 years since 9.5 came out... :)

On Wed, Jan 28, 2015 at 3:21 AM, NicklausW <notifications@github.com> wrote:

> 9.6? About time. XD
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vhelin/wla-dx/commit/1e1c09c3246e6c36b1ff98591dd8146a77fed0ed#commitcomment-9470990>
> .
>


,xm


,zsx


```js
Object.prototype.hasOwnProperty.call(obj, key)
```
 너무 길어...

```js
var hasOwnProperty = Object.prototype.hasOwnProperty;
hasOwnProperty.call(obj, key);
```
hasOwnProperty는 읽기 쉽게하려고 고른 이름.

만약 나혼자 쓸라하면 그냥...
```js
var OP = Object.prototype.hasOwnProperty;
OP.call(obj, key)
```
그럼 간단하게;
```js
for(var key in obj){
  if(OP.call(obj, key)){
    console.log(key + ', 내가 진짜 obj 프로퍼티야!');
  }
}
```
:notebook: 그저 간단하게 쓸라는 이유보다는 빠른 Loading

```js
Object.prototype.hasOwnProperty.call(obj, key)
```
Object (1)에가서 prototype프로퍼티 (2) 찾아서 hasOwnProperty 메서드 (3) 찾아서 call메서드(4) 찾아서 ...
4번이나 포인터를 따라 움직인다... 이걸 `for loop`에서 사용하면 4 * loop 숫자 만큼 완전 똑같은 짓을 한다...


Thank you for your positive comments ! This was entitled // small change....


Should this example have some sort of `MyComponent.prototype = Object.create(React.component)` ?


Sorry I missed this one, fixed !


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1098 of 1098 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-68.93%25-green.svg)


>Да, он его, вроде как, в любом случае переделывать под современный лад (флэт дизайн итд итп) собирался.

Ну и девелопер весь флетовый уже.

@kerbylav тоже, в общем-то, прав.


it's good that finally we're writing here about optimization :+1:


ㅁㄴㅇㄹ


Could we maybe add another simpler test for just the decodeBase64 method too? i.e., generic base64 string and test that that is decoded correctly?


independence* typo! ¬¬


Nice.


Не хочу придираться, но почему не в обычной венгерке? почему не aComponents, oUser?
от жс не отошел? :wink:


푸쉬 파이프를 복붙 하지 말고, Join의 pushPipe를 콜백으로 등록하고 콜백을 호출 하도록 변경.


update_sqlite_coverage (mbadams5) - i386-Windows-7-VisualStudio-12: OK (1573 of 1610 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-97.7%25-yellow.svg)


Yes, fixed.


Because I localize subsystems and their research unlocks separately. And because you seem to hate me touching research files to fix it, because you apparently don't know how to merge my changes in. :P


That's a pity actually...


Whoops. Try now.


I guess this issue should be connected with https://github.com/PrestaShop/PrestaShop/pull/2417 reduce disk IO commit 8160150


```js
if(nativeForEach && obj.forEach === nativeForEach){
  obj.forEach(iterator, context)
}
```
ECMA 5 의 `Array.prototype.forEach` 메서드가 있고 `forEach`의 이름으로된 프로퍼티가 obj에 없다면 native JS써라.


Yeah, but until we find a practical solution these things just rot.
On Jan 28, 2015 12:35 PM, "Tomasz Heimowski" <notifications@github.com>
wrote:

> That's a pity actually...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fsprojects/Paket/commit/27d51a67c6376e90c59550febce2cbaecbec5b35#commitcomment-9477021>
> .
>


You had me 100% convinced it was me :p Working like a charm now!


agreed


I'm benchmarking https://github.com/xon/xen_BBM_v2 with the old process of parsing the compiled content to build tabs/accordions/sliders/tables to my branch https://github.com/xon/xen_BBM_v2/tree/XF_Parser_Integration which tweaks the XF bbcode parser and implements parent/child bbcodes in the core parse itself and thus implementing the tabs/accordions/sliders/tables as native bbcodes with no post-processing of rendered post data.

I've done a simple benchmark of fetching every post on a forum with got 2.6 million posts. Once with the the old bbcode parsing, once without.

Haven't actually started analysing the results, as I need to finish loading ~5 million rows into MySQL and then identify outliers (busy VM, random stalls from background stuff, etc) .


> Does a multiple line print statement start with three """ and also end with three """. Also can this method be used with other methods and functions I am guessing so. Not just the print function.

It's not just print. This is part of the Python language syntax. A string can begin/end with `"""` or `'''`, and it is said to include every character within it, including newline characters. You can use these strings in functions like `print()` or assign variables to them.

> And what type of arguments can be used in the exit() method. Using 0 I guess means the program exited with no error. And 1?

You're learning C, right? You know how the `main()` function is supposed to return `int` in C? That's what I'm doing by passing a number to `exit()`. It tells the caller that the program failed, even when scripting. In BASH, this is stored in a variable called `$?`. Do a bad command like `git gnjfdskglnldf` that doesn't exist, then do `echo $?` to see the return code. I'm making the Python script do this same thing. This allows someone to put your program into a shell script and check for error codes before continuing.

> And what is the meaning and use of line 97 and line 98.

These lines make it so that `init()` will not be called if someone `import`s this source file as a module, because it will cause the program to exit if they do (`argv` probably doesn't contain what that function is looking for). However, `init()` will still be called if someone directly executes the file.


Is it possible to use `FMD_ENABLE_FAILURE_NOTIFICATION_TAG` here, so it stays in sync with findmydevice_launcher.js?


@gRoussac For this one, for code readability, I would keep using file_exists() or is_file(). It's indeed strange to use filemtime to check file existence :)

@pavelnovitsky : in my commit, filemtime is what we want to do, so the file_exists() check is not really needed, and the code is simplified.


Actually Ray code style guide doesn't contain any word about that (if is it wrong or not). As you said it can be written in column. Personally I use only this formatting if logic of that code has secondary meaning. Like here or for example as follows:
```CGFloat x; CGFloat y; CGFloat z; CGFloat foo; CGFloat bar;```
If you want I can change it :smile:


Why do you need to `requireApp` here, instead of `require` as before?


`obj.length`는 obj이 Object인지 Array인지를 확인하는데 자주 사용한다. Object은 length가 없다. (`Object.length === undefined`)

그러나 `Array.length`가 0일수도 있다.
:toilet:


This should have been `FMDInit()` instead of `start()`, right?


Neither Sparsity::getRow, nor Sparsity::row are marked internal, so why would such a warning get omitted?


おお、確かにそうですね！
修正しておきました。


Thanks for your suggested change, you will see this issue is now closed, there was an improved pull request by @bratkartoffel which was accepted.


I hope our translators will update their languages.
Maybe you want to write to our localization mailing list.


I have used it during debugging and then I thought it could be useful in general to disable this feature.  I'll drop this bit and have it always enabled in a transaction.


But isn't the DB cleaned up with https://github.com/owtf/owtf/commit/33b82c3ab62804301b708bf668002cf0210aa545#diff-f085c5ab450c5ed2ad02e2fb73c7fbcfR27 ?


你可以把这个放在一个叫做 ccloud 的package，这样你 go install github.com/xxx/casper-cloud/ccloud 就可以自动生成一个叫 ccloud 的可执行程序


Ok, I'll try with an older module, and i am using the 4.3 kernel because only it worked normally.


I think the problem is that internal.i has not been updated.


i'm happy to make that a "proper" `if` clause, btw.  in this case i feel 1 line is nicer than 5, though, especially with the already present nesting here…


why add 1 before port?


So expect the next pull request will be to make this code respect the bbcode parse tree caching setting.


package casper-cloud


fix #1


no hard code, pass from command line


you can use tmpl of template as var name


Now if they are null it will go ahead and call methods on them. This is will cause an NPE.


TeamCity Nimbus :: 0. Feature Branches [Build 0.0.0.0](http://twcbuild.cloudapp.net/viewLog.html?buildId=1115&buildTypeId=bt8) is now running


Не фанат венгерки если честно, но да, надо бы переделать под общий стиль лс.


The same null error here.


Very minor issue, but just so you know, to just interpolate an array element in PHP, the `{$...}` syntax is not needed. Instead of `"/{$_ENV['ORASVC']}"`, you could use `"/$_ENV[ORASVC]"`.


ws is off.


Maybe we could also compare maxPower?


Fixes #172.


cmd := parseCMD(line)
if cmd.Action == "Info", then blabla...


Thank you.


Do you know if "s" conflicts?
Isn't three aliases a but much?


In other locations, will not occur :ok_hand:
(see new commit, I add slash at the beginning of directories)


Thank you =)


Kinda long argument name. Maybe short it down a bit "rank/power/time" maybe?


r+


CentOS Bug Tracker


Da er det vel bedre å sjekke BlockFromTo-eventet og kansellere det om blokken den treffer er rails? :)


Standard accounts will not use BIP44, because multiple accounts are too complex to handle. This should not affect the Trezor plugin; BIP44 is still supported in this plugin. (let me know if it is not)

BIP39 was never supported; therefore it is better to use a different derivation, because complete incompatibility is better than partial compatibility; we don't want users to wrongly believe their seed is supported, and to find out later that they cannot retrieve all their funds.


Do we really need an MPerm? All of the information is public anyway using /seen, or /f p,


Remember @Override annotation


TeamCity Nimbus :: 0. Feature Branches [Build 0.0.0.0](http://twcbuild.cloudapp.net/viewLog.html?buildId=1115&buildTypeId=bt8) outcome was **SUCCESS**
Summary: Tests passed: 1454 Build time: 00:06:20


Recent distributions have /run as the main status directory with /var/run a symlink. All RA use "/var/run" in defaults. So, this test may fail in case HA_VARRUN is set to "/run".


TeamCity Nimbus :: 0. Feature Branches [Build 0.0.0.0](http://twcbuild.cloudapp.net/viewLog.html?buildId=1116&buildTypeId=bt8) is now running


写完就忘。。。。。。


S doesn't conflict, I checked that before adding it. Surely it could only be  "s", "status" :)


This is not working for me.

![image](https://cloud.githubusercontent.com/assets/55217/5937189/c7cddb1a-a6ec-11e4-8ac5-21ed025dc3e9.png)


saw approval from Manishearth
at https://github.com/Ms2ger/servo/commit/f9304a2af55ba94ff6909ea839953116965be04f


merging Ms2ger/servo/update-layers = f9304a2a into auto


Ms2ger/servo/update-layers = f9304a2a merged ok, testing candidate = cc2523ee


Cool idea. But the width of characters vary too much for us to be able to use it.


Also why don't we update the value in the database with the one given by the user? Something like ``UPDATE`` instead of ``INSERT``? Isn't that possible?


ART - https://github.com/temasek/android_art/commit/86e78b1bf4a39805c4801ab284b8a447c75294b2


Txt.repeat()


Pull request cleared, no code style suggestions found.


Not sure if you've tested this, but I was trying something similar today and it didn't seem to work. :(

I have another idea for how to fix it, I'll try to work up a PR soon.

Unless this does in fact work?


Medium: it must be final.


I am not sure wheter it would be nice to be able to sort for max Power. Surely possible, but in my original thinking it wasn't necessary.


You might want to do sortedBy.toLowerCase() just once instead of sveral times.


Heh :xd:


Medium: hide the condition complexity in a method, like isUserNameContained(user)


@vnctdj what do you mean with "void" here?


As russian guy, I can confirm that all changes are correct


@jocel1 I agree.


TeamCity Nimbus :: 0. Feature Branches [Build 0.0.0.0](http://twcbuild.cloudapp.net/viewLog.html?buildId=1116&buildTypeId=bt8) outcome was **SUCCESS**
Summary: Tests passed: 1454 Build time: 00:04:57


I still can't repro but, based on the info that you guys have provided it
seems to me that it should work.

2015-01-28 12:57 GMT+01:00 Tony Miller <notifications@github.com>:

> Not sure if you've tested this, but I was trying something similar today
> and it didn't seem to work. :(
>
> I have another idea for how to fix it, I'll try to work up a PR soon.
>
> Unless this does in fact work?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/drfill/homebrew-liquidsoap/commit/65d2c267d5fb9c28c543855ed408664a74a83a36#commitcomment-9477302>
> .
>


@MaTriXy
There is an edge-case that causes the hosting application to crash: when the device has no internet connectivity.
In this case, a url is loaded to the webview upon creation, but immediately afterwards, onReceivedError is called, causing the Activity to finish before it has finished starting.
FYI @yuval-netanel @grozen


Low: use JodaTime Library.


Как же интуитивно хочется для универсальности чтобы это было вынесено отсюда и заполнялось где-то по хуку какому-то. Ведь выше для этого и АПИ уже сделано.

Но это так, нотка перфекционизма. Просто эта строка каждый раз как её вижу меня волнует.


En fait techniquement parlant le CSS vient du Source Filmmaker, j'ai pas réussi a trouver celui de Hammer. La différence c'est que le SFM n'utilise pas les QDockWidget ni les QToolbar donc il n'y a pas de style pour ces éléments.

Mais en fouillant dans les dossiers de Dota 2, j'ai quand même trouvé un fichier "source1_import.txt" qui nomme explicitement le moteur "source2"

Half-Life 3 confirmed !


Medium: mark all the fields as final.
Final is used to mark all the dependencies of the class that are needed to construct the object.


Medium: document the methods using javadoc.


Ah, yes. Line 27 will avoid the duplicates. If we allow user to add/edit resources from the interface then the dirty column will be helpful. To use update, we need to know the primary key, so we have to fetch that resource first, know the primary key and then update.


this realy hurts my eyes, how can you even work like this? this is code is barely readable you should use better parm names as it makes this code almost unreadable


lol :)


What for did you code for Luna? It has been dead for years.


There used to be a question to that line but it was later deleted


High: return a proxy implementation to proxy the client implementation.

new XXXProxy(new XXXClient())


Medium: final


LibreOffice overrides the compatibility decomposition unicode function (and uses hb-icu integration otherwise), because the text fallback works after HarfBuzz finished shaping the text not before it, and relies on HarfBuzz returning zero glyph indices for characters not supported by the font. Having HarfBuzz do compatibility decomposition means things like math bold fraktur ‘𝕬’ will be shaped as plain ‘A’ by HarfBuzz if the font does not support it, which is undesirable.


High: apply singleton here.


I thought it is worth a try :) Maximum characters are 16 (plus two roleprefix max). Doing nothing isn't better ;)


Why not place this within the initial `Project.new` constructor?


Medium: use camel case for param names.


Why are we reusing the `uniqueurl`? Shouldn't that be different for different forks?


Medium: use camel case for DBusers.


This is theoretically right even though it should check that value is actually a tuple because Manager class can be instantiated outside `container.py` (See: `session.py`):

```
if (expiretime is None) and (type(value) is tuple):
    expiretime = value[1]
```

Apart from this minor issue which can be easily solved, it seems to me that there is a major architectural pitfall in the way it would work.

Before getting values, beaker checks if they are available with `Value.has_value()` then if has_value confirmed that the value is there it will run `Value. _get_value()`. So if the key disappears between the check and the get (due to memcache side expiration) it will lead to an error.

This error is explicitly handled for memcached in `Value. _get_value()` but it might break any time memcached client changes exception format or will lead to broken backends if they do not rise the right exception. Also this has a double cost for each key retrieved by a remote system (like memcache) which should be checked twice for each cached key.

I suppose that before actually considering the memcached expiration reliable `Value.get_value()` should be changed to actually retrieve the value without checking its existence.

In the meanwhile I'll check if the corner case it still caught for memcached (I only use redis for caching, so not sure for memcached) and if everything still works without leading to strange behaviours when keys expire between `has_value` and `_get_value` we can merge this.


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


High: throw an exception instead a message. The easy way is: throw new RuntimeExeption("bla");


Maybe we could change the primary key with something we can now right away. Would it be possible to have something like a ``dict``? That way the user update a configuration, let's say ``TOOL_BLEH=MEH`` then we know that ``TOOL_BLEH`` is primary?

Also, is the commit OK? I mean does it introduce a problem or not?


looks ok. we'll improve on this later


Isn't this a typo?  "...need TO direct your results.." ?


update_sqlite_coverage (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1573 of 1610 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-97.7%25-yellow.svg)


Will move things around eventually. I just wanted to get something working first.. In fact, what exists here isn't completely right, the remotes aren't proper. You can fork atm, but can't upload files therefore.


I man is power is equal. Then check for maxPower difference.


I realized that after posting. Mutable comments makes for confusing discussions.


good question - well I felt if someone forks my private repo, it'll likely be someone I gave access to, therefore it would make sense for me to be able to check on his/her work.

i'm opening to hearing arguments though.


Going through the tests these seemed not needed as AFAIK the an StringIO object is never the argument.


Well, it is public on massivecraft. Not all servers might have a /seen command or allow to use /f p. I still think it would be good. I, as a former faction leader, wouldn't want someone to be able to get all informtion of my faction on one sight.


I'll do some tests about this, but yeah, it is pretty long


I see no reason why not?


you may want to read

http://docs.travis-ci.com/user/how-to-skip-a-build/


 intrinsische Motivation entwickeln:
 Precht bezieht sich hierbei auf die Vorstellung von Maria Montessori, die das Kind als "Baumeister seines Selbst" sieht. Als Lehrperson ist es Aufgabe die Schülerinnen und Schüler beizustehen und ihnen zu helfen sich selbst etwas beizubringen, anstatt diese wie im heutigen Modell  ständig zu belehren. Dabei ist es wichtig ein richtiges Maß zu finden, die Kinder mit Angeboten zu füttern und nicht zu überfordern, um diese intrinsische Motivation auch am Leben zu erhalten.
Kinder müssen auch Phasen der Langeweile durchleben um danach ihr gesamtes Potenzial auszuschöpfen, wenn es das Kind zum jeweiligen Zeitpunkt für richtig hält.
 individuelles Lernen
 keine Fächer sondern Zusammenhänge der Welt verstehen
 Bindung in Gemeinschaften und Lernhäusern
 Beziehung- und Verantwortungskultur entwickeln
 Werte und Wertschätzung förden (zB. Schuluniformen)
 lernfreundliche Schularchitektur
 Konzentrationsfähigkeit trainieren
 Notenabschaffung und persönliche Bewertung von Eigenleistungen
 Ganztagsschule von 8-16 Uhr


I see that you modified the library to look for "eady" instead of ready (Thank you), but there is another place in the library that should be changed in the reset method.


no this crap!


why ?


This will throw an argument exception - as {0} is not present in the string.


if you look at the PlayerRoleComparator, it is done the same way there aswell. I am not sure about comparators, I have not worked with them before. let's go and ask Cay


Medium: Watch out!, assertEquals is based on equals method, and it compare the pointer to memory position and not the content. In this case works, but you must understand why and be ready when a method return a defensive copy of an array.


You are (almost) right :) The comparison is wrong. it should be a <=.

Can you change this?

Thanks!

El Tue Jan 27 2015 at 10:46:32 PM, aemartinez <notifications@github.com>
escribió:

> I think the comparison is the other way around, shouldn't it be this way?:
> if region.Address+uintptr(region.Size) < startAddress
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mozilla/masche/commit/57572033fb09781a1997d5942016500f4797ecb4#commitcomment-9471235>
> .
>


This is incorrect - switching {1} and {0} will make it say.

"Your Honorbuddy instance has changed profile from NewProfile.Name to OldProfile.Name"


I could certainly do that.


update_sqlite_coverage (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


@mmoll it's been normalised, sorry :)


@jbavari when you release 1.3.3 on NPM ? :-/


!. Seems a bit redundant hé


Medium: I see your code ok, but seems to be you didn't use TDD, I think you write the code first and test later. Is ok for now, but try to give a chance to TDD. Please fix the HIGH comments.

Great job!


```php
return $this; //Do the same at L12 and L20
```
Then you can do $PNObject->setMessage(x)->setReceiver(xx)->send();


@GM-Alex I understand the need for this for PHP 5.2 (yuck) or when you need to install ext without switching, but the old implementation wasn't working as expected. If we get an implementation that delivers without break anything, then +1 to have it back, nay otherwise :)


@psychon
This got lost in 355f106 - when splitting it..!


ChinaDNS里有个压缩指针的方法不知道是什么原理，其他好像就没什么特别了


Why not
rc = _preprocessStateMachine (next(i))
?


This change caused this bug: https://captureproof.atlassian.net/browse/CAP-1283

How come you needed to do this?


well, if the power is equal (which will only happen if it is max), it will already be at maxpower. So checking for that doesn't make much sense on a second thought.


Also OGHAM SPACE MARK seems to be blank only occasionally, so not sure if it is wise to use blank space as fallback.


hey @nknize I am afraid this import might causes problems here. The jts dependency is optional in our pom, but this import here makes it mandatory I'm afraid?


its in a submodule: https://github.com/rngtng/rainbow.ino pls check this out too..


Looks good - thanks!


Any reason for the version change?


@marcioAlmada Could you give me some examples what didn't work as expected, to get a point to start with.


Is meant as base for being able to communicate with users in their preferred language. Currently we can only switch per session in a certain language but when you initiate an action that triggers sending notifications we currently use the session locale. So, if someone is working in English with PP and triggers a notification then someone that uses PP in Dutch will get an English notification in its mail. That is why we need offer the option to register the locale for every user.


@glesiak Are you talking about `sut`? Actually it could be named as before `manager` but there's a problem to recognize what this code is actually testing (only name of spec will state about class under test) what in my opinion can lead to misunderstanding.

Please also notice that test should be descriptive, easy to follow even for non-developers (even with cost of 100 lines more) what is unacceptable in "normal development".


This is incorrect/bug, because row.permission will have same value for the entire model.


Could we add the 'pykickstart' metapackage pulling in both python-pykickstart and python3-pykickstart?


colocou as cores com letras minúsculas, e agora maiúsculo, bom padronizar toda escrita, no geral está incrivelmente melhor, agora só pequenos detalhes!


On Tue, Jan 27, 2015 at 01:29:09PM -0800, David Vossel wrote:
> why does this need to be exported if opt_trace_rc is not set?

That's actually partly unrelated: ocft doesn't set
OCF_RESOURCE_TYPE at all. Some bits/RA may expect this variable
to be set.


On testing I realized that this string is probably longer than required. Would probably suffice like this:
"The audio DSP has been enabled without any enabled DSP Add-on. Enable at least one in order to make use of this feature."
@jjd-uk - wording good enough?


isso não poderia estar na class de cima?


I did it! Yaaay.


Am Sonntag, 25. Januar 2015, 07:42:08 schrieb Daniel Marjamäki:
> I get this when I run the tests:
>
>     Assertion failed in test/testunusedfunctions.cpp at line 437
>     Expected:
>
>     Actual:
>     [test.cpp:2]: (style) The function 'f1' is used in this file only. It
> can be made static.\n

I'll look into this next week (currently fighting a flu).


fixed in next commit


quebrou a endentação aqui...


same as above ;)


I find these line hard to read. Please use the good-old if statement.


@jocel1 you are right, sorry


On Tue, Jan 27, 2015 at 01:29:31PM -0800, David Vossel wrote:
> also, why is this change necessary?

The trace filenames are constructed in a certain way. It just
seemed useful to have this parameter set so that we have then:

ocft_<configname>/<casenumber>....


@HelenaEksler You removed the tests because they take to much time to run.
So this means we do not have tests confirming the desired behaviour of the platform.
=> We loose the advantages of having Behat in place.

I noticed that @javascript tests take longer to run then @api.
Are all tests, that are now tagged with @javacript realy needed to be run trough PhantomJs or can we make some of them @api?


This patch needs a better explanation in the commit message, otherwise these changes look really unnecessary.


Т.е. можно добавить новый компонент, который наследует другой? Я ещё не знаю где-то можно использовать, но возможность радует. Это действительно "кирпичики".
регулярку можно заменить:
```php
'#^Component@(.++)$#'
```
быстрее + отключить регистронезависимый поиск. Почему? Это узкое место вызывается постоянно, поэтому тут экономия нужна.


> Reason is that we are using unit test coverage only

Others do ;)
Another thing: it would be cool (dont know if feasible) to set the branch coverage to zero too. Currently, this aspect is ignored because its difficult: to do that we have to find out how much and where the branches are. But it might be doable, cause we know where the branching statements are...


About the patch description, you are right. Not sure why I wrote it that way.


This change should go to the previous commit, right?


>Не фанат венгерки если честно

Если не секрет почему? Вот я не вижу ни одного минуса. Ну на жс разве что она явно лишняя.


Muito bom, uma ideia, isso poderia ficar assim:

```php
$nome =           $_POST['nome'];
$sobrenome =      $_POST['sobrenome'];
$sexo =           $_POST['sexo'];
$data =           $_POST['data'];
$email =          $_POST['email'];
```
E isso talvez ajudasse a sua leitura, o que vc acha?


Lol. I need to integrate FlurrySDK framework in my app using Cocoapods, so it is really useful :))


Старайся всегда писать фигурные скобки у циклов, это хороший стиль.


Changing the path to abspath at the params_list time is mostly necessary when relative paths are used. In such cases, commit 32457fac3dccd499ab507a54714d297543edef07 from @clebergnu does the relevant job.


Более понятные названия?


```java
int size = nums.length;
```


Should we execute this 1.7 code on a 1.8 server? `if (SpigotFeatures.isActive()) return`?


Tem um espaço sobrando, após as aspas dentro de class...


See https://translatewiki.net/wiki/Translating_talk:Mantis#Subtle_change_in_MantisBT_Timeline-related_language_strings_44832


:blush:


Никогда не пиши тип массива так - это очень плохой стиль. Правильно:
```java
int[] nums = ...
```
Внимание на положение скобок.


Chris Lumens had a problem with `except Exception as e`, because it doesn't work with ancient Python versions, so a version bump and dropping the support for these old versions was suggested.


Acho que BR não precisa fechar, ficando apenas
```html
<br/>

/* ou */

<br class="clear"/>
```


Could you please put the new definition of _ to pykickstart/i18n.py module? That's what we do in anaconda and blivet.


Yeah, that works. Do you know if there's a way to execute `Edit , d` from outside Acme so this script could do that?


This is from long time ago, so forgive the possible lack of accuracy:

If you were on PHP 5.4 and attempt to `phpbrew ext install --php-version 5.3 mbstring`, for instance, the installer would compile the ext source from 5.4 and put it on 5.3 which (hopefully) should give an error. The thing is, sometimes unmatched extensions that belong to PHP X don't not give errors on PHP Y but we can't allow that because it can cause unpredictable stuff :)

Steps:

- switch to a PHP version
- phpbrew ext install --php-version <other-php-version> <some-pecl-ext-not-available-on-disk-yet>

This is what recall. It just seemed safer to remove the feature since we haven't got a way to do it.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/606
success: http://build.servo.org/builders/linux2/builds/605
success: http://build.servo.org/builders/android/builds/641
success: http://build.servo.org/builders/mac1/builds/822
success: http://build.servo.org/builders/mac2/builds/822


Yes @magnusulf is right. Just return an int (probably 0) right away here.


All right, I'll change that.


Ainda acho que se vc está chamando as fontes do repositório do google, não tem pq existir as fonts, dentro da pasta fonts como está hoje, acho que já falamos disso... :)


`return Long.valueOf(r1).compareTo(r2)`. This way the comparator can also return 0 for the equals case.


Again something that looks like an unneeded change on its own. Please add an explanation or merge the change with some other commit to create a self-explaining change.


Isso aqui não está legal, fiquei confuso, é uma lista, onde dentro da lista tem um link, ou dentro do link tem uma lista?

Pq não usa ```nav```do html5?


The maximum power is different for different players. On MassiveCraft the premium members get more power than regular players. This is probably only the case on MassiveCraft but it's mainly our own server we develop for :smile_cat:


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=62c9125831f94a15a2acaef63d64a4ba
 plone-4.3-python-2.6 [FAILURE]
plone-4.3-python-2.7 [SUCCESS]


fast-forwarding master to auto = cc2523ee


That makes sense. Just please mention that in the commit message.


빠른 콜!
```js
Object.prototype.hasOwnProperty.call(obj, key);   // 느림  version 0.4.5
hasOwnProperty.call(obj, key);   // 빠름 version 1.0.0
// 한술더 떠서;
var hasOwnPropertyCall = hasOwnProperty.call;
hasOwnPropertyCall(obj, key);  // 가장빠름!
```


New user of NixOS so bare with me.

I've noticed that I can not use: "services.xserver.windowManager.windowmaker.enable = true".
I get the following error:

error: The option services.xserver.windowManager.windowmaker' defined in/etc/nixos/configuration.nix' does not exist.
(use ‘--show-trace’ to show detailed location information)

I've tried to do some research and I think this is because in the following file:

nixpkgs/nixos/modules/services/x11/window-managers/default.nix

windowmaker is not defined in the imports.

Can you please confirm this and fix it if I'm right?


This reads odd IMO -  maybe something like: "Enables digital signal processing (DSP) of audio channels ..."
Also, how is this related to the channel/speaker configuration configured in system settings? Anything we should mention in regards to passthrough (like that it doesn't apply to anything passthrough, ...)


que coisa linda! :+1:


> @tmpsantos: can you take a look? I've just followed what the upstream CL did, but I'm not sure if I should be calling Open() before OnSend(), and the new code in OnInit() is duplicating the existing error handling code for the Bind() call.

LGTM. The Open() is called in the OnSend() because the connection is not kept open like in TCP.


Why? It would unnecessarily pull in a whole Python stack of the other version. Now `pykickstart` contains the binaries and only pulls in `python3-pykickstart`, because we want to use it on the LiveCDs, where Python 2 is unwelcome.

I have consulted this with Chris and he said it's okay, provided that we fix `Requires` of packages that require `pykickstart` as a Python 2 library, of course.


Har laget en fiks for dette, så hvis jeg kunne få commitet endringen hadde det vært strålende! :D


I approve of using an educated guess like @ulumulu1510 is doing here.
It's simple and efficient. If we wanted to go advanced we could create a method that measure text length in pixels.
Such a method would need to have a map from char to width in pixels. Also it's further complicated by bold and italic. For now I don't think creating such a method is worth the effort.


That was fixed in one of the subsequent commits.


asdf


I'm not sure if showErrorMessages is part of the public Parsec API


Esse bloco todo, se eu não estou errado, eu tenho uma div box, dentro dela uma imagem com class relative, tenho também outra div com class legend-box, e dentro desta tenho um h3.

E tudo isso dentro de um a

Vamos lá, a ideia é um link, então logo de cara dentro de link não precisa ter h3, a hierarquia já está definida pelo próprio link, então dentro de um link conter um h qualquer coisa, h1, h2, h3, etc, não é bacana... Ainda dentro do link ter muito conteúdo também é ruim...

Eu editaria todo estilo necessário para o visual do link, em uma class do link, ficando mais simples de visualizar o html, e também mais simples de dar manutenção...

O que vc acha?


Good point.
```java
...
sortedBy = sortedBy.toLowerCase();
if (sortedBy.startsWith("r"))
...
```


No geral! Ficou ótimo!!!! Excelente evolução :+1:


Even in the updated pull request it is confusing like hell.


we don't have recovery partition and recovery patch is not needed as we are using release tools from cm now .


Okay, I will fix that, but as the last thing before merging, because force pushing deletes all the comments in the PR. I already did that once with this PR, so I'm not going to make the same mistake again.


It would be awesome to get access to this view without having to render a 401 response, something like ```/fake_cas_login```


i dont think so, need to check code.
for me 1.5 is all i need.


**jshint.W086**: Expected a 'break' statement before 'case'.


**jshint.W041**: Use '===' to compare with 'null'.


Yeah, I probably squashed it with the wrong commit. Do you insist?


For platform independence, I think you should replace `\n` with `System.getProperty('line.separator')`.
The value of this property should be stored as a private (static?) field so that it is only looked-up once


As per 1d5b3aadeaf6f0cfb49aa97176588bbbaee6c3c3 this is not needed


The filter() call should be changed to a generator expression/comprehension:
``sorted(c for c in bothSet if isinstance(toHandler.command[c], DeprecatedCommand))``


I can do that.


Pulled! :cake:


The list() call here and in other places is not necessary, AFAICT sorted() works on any iterable and produces a list.


i keep in sync with 6to5 major versions


OK. Same applies as with OrderedSets (https://github.com/rhinstaller/pykickstart/pull/1#commitcomment-9477890)


This not supposed to be inserted by script?


I think ``print()`` is better than ``print('')``


Could you fix the indent here?
I see spaces. I want tabs :smile_cat:


It is -- QtWebKit dependency


minor problems


这个好像有点问题，我的上面用了这个函数之后地址变成了 `//img/scrollup.png`，然后图片就显示不出了。


Awesome! :)


No problem, we can do this mass replacement on translatewiki.net itself and export it at next round. I think siebrand will notice the change and remember to perform the replacement when he syncs; but if you don't  see the other languages updated at next export please just poke me by replying here.


Good point!


> Т.е. можно добавить новый компонент, который наследует другой?

Нет, это просто возможность указания пути до шаблона с учетом различных вариантов метоположения компонента.
А уже сам шаблон можно использовать в любых местах, включая другие компоненты.


No. There really should be an empty line at the end of each file. It seems to be some sort of standard.


@marcioAlmada Thanks :+1:


Is it also possible to add libpng and libjpeg to the buildInputs.
This allows windowmaker to also use png and jped icons.

In the current build it only supports xpm icons and bitmaps.

Thanks in advance.


Is there a bug report for pylint to support six? We should really keep pylint in the game even with python[23] compatibility.


That is fine. I am sorry, I was not aware it would clash. It was not overly important, and that is why I put it into a #if DEBUG block. It is useful, but hopefully, failures would be removed during development time, so most users would never even see this type behavior.

Once things get completely merged, I will relook at this one. I will just keep a .patch file for the time being, to show me the changes that I had made.  It was pretty useful, detecting when to simply add a swapping of the binary result in the binary() function, etc, or when the end results were totally different, which means there was a byte swap needed somewhere within the crypt_all block.


What about this instead:
"Allows to configure available modes for different stages in audio processing and to arrange the steps within the processing chain of each selected mode." Does this still sound technically correct?

I didn't understand what I should expect reading this: "Also extended settings and information with help about available modes can be present."


update_sqlite_coverage (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (1573 of 1610 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-97.7%25-yellow.svg)


update_sqlite_coverage (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (614 of 614 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-2.75%25-green.svg)


You can easily promisify those callbacks.


Agreed


Qui ho giusto riordinato in ordine alfabetico


Any news on this one?


OK, I will squash it with https://github.com/tradej/pykickstart/commit/8fc02337c79c5448ffbc61290b80e4eab8814c3c and improve the commit message.


Because that's what ES7's `await` is going to work with.


This also needs some deeper explanation (i.e. the answer for the question Why?).


I'll change that.


Questo è sbagliato, lo tolgo nel commit dopo


Odd. I’m sure that when i removed them (4d6581d04df6d3) they weren’t anywhere in the draft at that time. Looks like they are back, even if not required.


Done b6f5c25


>Unfortunately this also means that alpha is handled incorrectly with our
own color management, but alpha isn't so important and can be fixed
later. (0.0 and 1.0 are unchanged by the transfer function, so it
"mostly" works.)

Actually, I just looked again, and this is completely wrong. Our own functions do not touch alpha, which is correct.


I see a potential security issue in replacing urlgrabber with urllib, as [urlgrabber apparently verifies SSL certificates by default](https://github.com/henrysher/urlgrabber/blob/master/urlgrabber/grabber.py#L987), but plain urllib does not. So I would advice using python-requests or some other module that does SSL certificate validation instead of urllib.


Oh, it's changed here. Ideally these changes should be merged with the patches that introduced the changed lines (they're all new, right?), but I'm okay with this separate patch too.


Questo pure è sbagliato: lo usavo per i test, dimenticato di rimetterlo a posto


위에서 설명한 GUARDING.


not lowercase (but you say that in commit msg)


This should be merged with the patch that added the try-except block.


Related to https://github.com/Mind4SE/mind-compiler/commit/21010e6c72521737a229e869865a33a7965c31d3


With this change, "SYSLOG_IDENTIFIER" renamed to APPLICATION, the same for some other attributes....Should it be so?


It does support some of the things `six` does, and there are bugs filed against it (like https://bitbucket.org/logilab/pylint/issue/200/pylint-doesnt-handle-python-six-very-well ). The `check` target is added back in https://github.com/tradej/pykickstart/commit/6fe0c7c7fa61729d1675ce93b68427cf1fca7323 after introducing directives for pylint to skip the import errors.


이름만 바뀜


See: https://github.com/jenkinsci/jenkins/commit/b105627b7357149f98f87ec2bf494e25bd2ed88b#commitcomment-8201680


_.all에서 _.every로 이름만 바뀜


I can squash & force push these, but that will erase this PR's comments, so I'll do it as the last thing.


Continua muito buggado :-1:


OK.


It should be changed to have `target_os = "ios"` too, sorry I've missed that before


I haven't tested this myself at all, i'm waiting on feedback:
https://forum.manjaro.org/index.php?topic=19399.msg180363#msg180363


this is a new comment


_.any하고 _.some하고 같은 기능을 한다.
any candy...
some candy...
생각해보니 다른점을 모르겠다...


le strcpy semble ne pas marcher?


That's a valid concern, thanks. What do other owners say about the proposed solutions?


Please do not use this commit, or 78620515d31488f4b2b94f275485692a6fe5ee6a.

See https://github.com/codesnake/xbmc/pull/1#issuecomment-71581723

`FBIO_WAITFORVSYNC` only exists starting `2015-01-15-321cfb5a46` kernel release, which already enforces vsync. On kernel releases prior to this one, it does nothing, since `FBIO_WAITFORVSYNC` is not implemented. On `2015-01-15-321cfb5a46` it makes the framerate crawl to ~30-40fps (half of vsync).

If you want vsync, just upgrade to `2015-01-15-321cfb5a46`, it works by default.

Please remove these patches (7731aa674e4d485da3255c3929e01a7583daeda7  and 78620515d31488f4b2b94f275485692a6fe5ee6a)


Oh merde ! C'est en prod ça ?


sounds good to me, thanks for the additional explanation


pouac !


We already have the link to github here:

https://github.com/openpsa/grid.js/blob/2e12a8aa178865462381fdcce1faabc3054da9d2/docs/template/partials/mainnav.hbs#L25

Displayed on the top right in the nav bar. Or am I missing something?


Tight code bro


Not the official reason, but if it would be me, I would revert this commit because the status text is jQuery's specific while the Backbone.sync and Backbone.ajax are designed to be easily replaceable by other XHR solutions.


Added some funny things


[вики ru] (https://ru.wikipedia.org/wiki/%D0%92%D0%B5%D0%BD%D0%B3%D0%B5%D1%80%D1%81%D0%BA%D0%B0%D1%8F_%D0%BD%D0%BE%D1%82%D0%B0%D1%86%D0%B8%D1%8F#.D0.97.D0.B0_.D0.B8_.D0.BF.D1.80.D0.BE.D1.82.D0.B8.D0.B2) + [вики en](https://en.wikipedia.org/wiki/Hungarian_notation) - считаю что недостатки более значительные чем примещества.


This image is cool!


While running in linux-ubuntu 14.04 first time it is scanning succesfully but next time it shows scanning... for a long time in terminal


I liked too!


Вот бы весь компонент можно было наследовать и менять только нужный шаблон, на примере как работает в шаблонах папки default и другого шаблона: если файла в папке шаблона нет, то он ищется в дефолтной теме.


r+


@magnum,  can this be reinserted now, or is there more change to wait for?


This has to stay, this is the version of the project and that is not the same thins as the minecraft.version which is MC and Forge version


We've generally shied away from creating new editions because it can confuse people who are in the middle of a workflow.  We've tended to either updated the most recent edition, or all non-archived editions, and use the publisher's editorial note function (if available) to state what happened.

In this case I don't think SP has an editorial note function, and your user set might be small enough that a new edition would be appropriate.

I don't think anyone would mind if you just did a Kansas City shuffle.


Do you have test cases for this? Does it past the existing tests that we have?

http://hstore.cs.brown.edu/documentation/development/testing/


Btw, the change I did prior to this was enough to cause the merge problem (when I added %x!=%x)


saw approval from jdm
at https://github.com/Ms2ger/servo/commit/f8ac1777ff9abcfcad5cb7a58ce3d7fbf61cf060


merging Ms2ger/servo/shadowing = f8ac1777 into auto


Ms2ger/servo/shadowing = f8ac1777 merged ok, testing candidate = 4a3e4032


Ah thanks!
On Jan 28, 2015 7:18 AM, "Tobias Bielohlawek" <notifications@github.com>
wrote:

> its in a submodule: https://github.com/rngtng/rainbow.ino pls check this
> out too..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rngtng/rainbowduino/commit/e6ea49ecbf2c47e226fe845184182fbb598b6c54#commitcomment-9477596>
> .
>


few minor things:

diff --git a/src/libostree/ostree-core.c b/src/libostree/ostree-core.c
index 907fb91..8db14b2 100644
--- a/src/libostree/ostree-core.c
+++ b/src/libostree/ostree-core.c
@@ -1105,7 +1105,7 @@ ostree_object_name_deserialize (GVariant         *variant,

 /**
  * ostree_checksum_b64_inplace_to_bytes: (skip)
- * @csum: (array fixed-size=32): An binary checksum of length 32
+ * @checksum: (array fixed-size=32): An binary checksum of length 32
  * @buf: Output location, must be at least 45 bytes in length
  *
  * Overwrite the contents of @buf with stringified version of @csum.
@@ -1434,7 +1434,7 @@ get_delta_path (const char *from,
   guint8 csum_to[32];
   char to_b64[44];
   guint8 csum_to_copy[32];
-
+
   ostree_checksum_inplace_to_bytes (to, csum_to);
   ostree_checksum_b64_inplace_from_bytes (csum_to, to_b64);
   ostree_checksum_b64_inplace_to_bytes (to_b64, csum_to_copy);
diff --git a/src/libostree/ostree-repo-static-delta-core.c b/src/libostree/ostree-repo-static-delta-core.c
index e8ddad4..2443cfb 100644
--- a/src/libostree/ostree-repo-static-delta-core.c
+++ b/src/libostree/ostree-repo-static-delta-core.c
@@ -132,7 +132,7 @@ ostree_repo_list_static_delta_names (OstreeRepo                  *self,
                     char checksum[65];
                     guchar csum[32];
                     const char *dash = strchr (buf, '-');
-
+
                     ostree_checksum_b64_inplace_to_bytes (buf, csum);
                     ostree_checksum_inplace_from_bytes (csum, checksum);
                     g_string_append (out, checksum);


Hi,

   This is the 86804234, which I assigned to Mark about download manifest.
This is for fixing the bug when toggling the cloud controller.
   Haven't been used in the cfbackup. But think it would be a good idea to
have toggle job and task to be available here.

   I assigned this story to you, since you already started

I feel the same way as you said about exported interface. But it feels
other ways that some of the functionality belongs to this package and
should not be exported. Especially working on low level programming that
require an integration point, feel like make something public or another
object is purely for testing purpose. That is why I saw a lot of library
testing having more in package testing.

1. We need to have a team discussion, seems we are working a lot on http
gateway related stuff
2 . The http gateway itself need to refactor to expose more arguments maybe
for easier usage.
3. Maybe use gogobosh for bosh director?

On Tue, Jan 27, 2015 at 11:33 PM, John Calabrese <notifications@github.com>
wrote:

> is this boshdirector stuff called anywhere in the code? i cant seem to
> find it.
>
> Also, while i know it's a pretty widely used approach, and you and Joe
> where talking about it.... I'm not sure i like the internal unit tests. To
> me it feels like that its an indicator of some things we should change
> rather than code around. like:
> ex.
>
>    - we dont have a testable exported interface, so we should write more
>    testable code.
>    - we are maybe not defining an appropriate exported interface, and
>    should rethink what is being exported.
>    - perhaps some of the 'testable' non-exported parts would make sense
>    as a standalone package or lib and then exported there and used to compose
>    a different packages functionality?
>    - etc.
>
> Perhaps a team discussion around this so we can just finalize a standard
> approach and unified opinion?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pivotalservices/gtils/commit/898cfa8a9da45ce317717258fcd84428d6564d93#commitcomment-9472466>
> .
>


@binderclip 不好意思, 我又提交了一个PR, 麻烦测试一下哈~


I think you meant 'canola'!


Perhaps:

"Audio DSP has been enabled without any enabled DSP Add-on. Enable at least one Add-on in order to use the DSP functionality."

To maintain UI consistency I substituted  "functionality" for "feature" as that is what the equivalent message for PVR uses.


Я думал у тебя собственное видение по этому поводу, а так в вики ясно что два лагеря накидали аргументов в блоки "за" и "против".


I'm quite sure that this breaks the link! The HTML &xxx style is just for stuff outside or inside tags, not in tag attributes itself, or ? but I'm unsure... do you see any problems when writing this with a "&" ?


Yep yep, it's a first update but I need to clean it and refactor it, give me a couple of days jaja :+1:


这个好! 能否PR上去呢?


Люто плюсую :)


makes me wait so long in dummy scanner :(


ami comment puis-je garder la traduction dans le projet? Je vois que vous suffit d'insérer les nouvelles chaînes dans mon cas est tout le temps vers le projet par défaut et doivent remplacer tout la traduction une idée de comment résoudre ce? (values-pt) tout le temps revenir à la valeur par défaut.


Now, the SPF support.

This is much quicker and much simpler, we only have to add a line in the DNS such :

DOMAIN.TLD 300 TXT "v=spf1 a:DOMAIN.TLD mx ?all"


The boot.IMG inside zip is smaller ,means it has no recovery. The one outside is built fine. Is it related to this?


What does this do? Do we expect ophyd to be issues signals like this?


Yes, my fault, I fixed that with next commit. Thanks


А лучше чтобы можно было в настройке шаблона задавать какой шаблон он "наследует"


Thank you, deleted them.


Get with full url using :query option


are the quotes needed ("1")?


Only if you do some rearrangement where you merge some other patches too.


I do not know why our system takes always sda for SD and sdb for USB, but it seems like this...


start coding in english! :D


We've already accepted the change from urlgrabber to python-requests in ananconda, so it should be okay for pykickstart too.


Build team has reviewed for next release.


Interesting point. It is somewhat sparingly documented :) But it has been around for quite some time (see [documentation from 2006](http://hackage.haskell.org/package/parsec-2.0/docs/Text-ParserCombinators-Parsec-Error.html#v:showErrorMessages), so it seems safe enough.


Can we leave this in, or something like it? I can think of a few places a horizontal scroller would be very helpful, and some mods may use it. On that note, perhaps it's best this whole PR not go ahead, it may break any number of UI mods.


Why here we have "1" missing next to sd_name? Non-consisten with before, a bit confusing...


the RandomUtilsMixin.js should be Upper case not lower case, please re-commit.


Much welcome, glad I can help :)


@kerbylav мы чуть о разных вещах: я про то, что компоненты могли бы наследовать себя так, что если файла в наследуемом компоненте нет, то он берется из наследуемого (логично), а как пример привожу то как работает шаблон плагина.

А ты предлагаешь весь целый шаблон (не плагина) наследовать таким же образом? Тоже один из вариантов: отдельный шаблон или отдельный файл рядом с оригиналом.

Как много дискуссий на гите получается :)


Looks great!


please, split this line. there are to much calculation in one line


Мое видение не может совпадать с уже существующими? :smile:


should be set final


https://github.com/bjimmykim/peer-assignment/commits/master/cachematrix.R

2599a33325fe074feda45099bce1f91589d819d8


I can send you my current code (servlet ...) but I need your email address.
We could try merging our complementary approach ...

Here is my address: bourges.laurent at gmail

Laurent


Did you want [SESSION_EXPIRE_AT_BROWSER_CLOSE](https://docs.djangoproject.com/en/1.7/ref/settings/#session-expire-at-browser-close) too, to match Webauth sessions?


Maybe not. I'll test without the quotes


i'm pretty sure there a better way to do this ;)


I think it's due to the USB2640 architecture. Not sure


Random question. What do you think? For me it is hard to read if statements within a for loop that has else statement as well.

But if I write something like this than it is obvious for me what is happening.

```
values = [self.__global, self._get_value_from_config(), self.__default]
ok_values = filter(lamda v: v != _no_value, values)
return next(ok_values, _no_value)
```

I am curious, what is you feeling?

P.S. I am not suggesting to fix it this way, just asking for your opinion.


Jorda, the variables ZLIB_LIBRARIES and BZIP2_LIBRARIES should be empty if SKV isn't configured for rocksdb. The corresponding "find_package()" calls that would set the paths and libs are in the main CMakeLists.txt and are encapsulated with if(LOCAL_KV_BACKEND MATCHES "rocksdb"). Does your build really try to pull in these libraries even if you configure for a back-end different from rocksdb?


start using constant for this kind of values


Very nice


Please add a dot at the end.


- Remove debounce option in ThrottleService
- The noTrailing option wasnt properly implemented and is no need at the moment


why this? Also, please do not add more trailing whitespace (see below)


@ft The backup ends next to the backed-up folder. This seems natural (`foo` and its backups are neighbors), and I count on it for the `-r` option.


version 0.4.5;
```js
if(_.isArray(obj)) return _.indexOf(obj, target) != -1
```
`Array`이면 `_.indexOf`를 사용하라. `Array.prototype.IndexOf`를 사용할 생각을 아에 안한다고 생각했으나 `_.indexOf`안에서 `Array.prototype.indexOf`가 있으면 사용한다;
```js
if (array.indexOf) return array.indexOf(item);
if (nativeIndexOf && array.indexOf === nativeIndexOf) return array.indexOf(item);
```
왜 `_.isArray`를 version 1.0.0에서 않썼을까?
`Array.prototype.indexOf`를 사용하는데 중요한것은 obj이 Array인가 아닌가라기 보다는 obj이 `Array.prototype.indexOf`를 엑세스 할수 있느냐이다.  (`String.prototype.indexOf`와는 다른것이다.)
만약 obj이 `Array.prototype.indexOf`를 엑세스할수 있다면 그것은 자동적으로 Array이자 ECMA5라는것을 동시에 알수있다.
v 0.4.5
`_.isArray`: `Array` 인가요? --> 맞으면  ECMA5있나요?  --> 있으면 `Array.prototype.indexOf`쓰시고요.
                                        |-->아니면 `_.include`쓰세요   |--> 없으면 `_.indexOf`쓰셔요.

v1.0.0
ECMA5의 Array.indexOf엑세스할수있나요?--> 있다면 당신은 Array이자 ECMA5이시니 `Array.indexOf`쓰세요
                                                                 |--> 없으면 _.include쓰세요


If a message such as "/tmp/mysql.sock" would be used by mysql instance "default" and mysql instance "2", set xy to ignore isn't descriptive enough then I don't how to make it better. Help me in that case, please :)


Is this commit related to test discovery at all?


I do not know, this script and test are a bit confusing and strange, the test is a bit wrong but this passing.
I left passing because the test was setTimeout and it does not work in jasmine, passing to the @deividmarques correct when return.


Give me a few moments to review that, I'm currently refactoring the whole code for Pimcore 3.x


It's a php-cs-fixer mistake, isn't it ?


> А ты предлагаешь весь целый шаблон (не плагина) наследовать таким же образом?

Да, так оно логично, тем более, что сейчас "компонентная" модель вполне это подразумевает, ибо изменения в компонентах от шаблона к шаблону будут далеко не у всех, да и то в основном косметические. А общая модульность/наследуемость шаблонов в принципе тоже намекает на то, что не стоит дублировать одни и те же файлы


That's fair, thanks for clarification!


Published to http://vui-docs-internal.s3-website-us-east-1.amazonaws.com/34_a06f8a8200700d29277dddbc6ffd31fc6b88875f


Is not aligned?


It is necessary due to the change of behavior in test discovery making the wrapper unittests to fail.


What do you mean?


c'est bof


repond si tu vois ça simon


Mein Ergebnisse aus 4 Seiten google zu "Stadtpolitik München":
 - http://www.nachrichten-muenchen.com/stadtpolitik/
 - http://www.muenchen.de/rathaus/Stadtpolitik.html


package name couldn't change, golang not support


This commit https://github.com/ks156/weio/commit/36ec54aafc157f904e7265c0ba7dde8c2c6ac86c fix the consistance liste here


```css
a {
    -webkit-margin-start: 1px;
       -moz-margin-start: 1px;
       margin-inline-start: 1px;
}
```


`_.each`를 언더스코 안에서 쓰니 Recursion같은 느낌이 난다.
`each`로 이름을 바꾸니 완전 같은 거여도 뭔가 언더스코 만들때 쓰는 low level utility같다.


nbghcgfcfc


Done


https://github.com/bjimmykim/peer-assignment/commits/86f472e8238985f844590ae71c3ea763e24eea10/makeVector.R

86f472e8238985f844590ae71c3ea763e24eea10


Looks good to me.


Nope,align works only by prefix. Not by `:`


Yay!


Yea saw working on it now, adding another fix at the same time. thanks


php-cs-fixer uses psr-1 and psr-2 and other conventions.

We are not coding javascript, it's not mandatory to remove last one `,` .
But it's not really important, i can remove it.


Please, let me know how you make the submitted homework. And check the unconfirmed error.


The semantics on the fields names isn't quite clear yet, we'll use this method for now but have to consider naming some fields application specific. for example: should we add an "ERL_" befor every erlang specific field (like method or pid)?


should use create! when testing...also since this is a unit test (not a feature/request spec) mock objects should be used instead of real database records


Not sure to what extent I represent the public opinion, but I find your code harder to understand. Generally not a big fan of filter/map/next – I prefer to use the built-in control structures


Throwables.propagate() is nice sometimes (not saying that it should be used here although)


I found it a bit suprising that C++ comments are generated in a file call `pythonjs_to_rust.py`.
Do you have plans to adjust the naming of modules and functions/methods?


We usually catch \Exception, not ExceptionInterface :)


Agree. I should set Temp flag, because it is only temporary. Will delete it soon


Method will be soon documented so this problem will be fixed automatically :)


TESTS FAILED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=cfcc93c8589240b493869bcea552f4f7
 plone-4.3-python-2.6 [FAILURE]
plone-4.3-python-2.7 [SUCCESS]


all tests pass:
success: http://build.servo.org/builders/linux1/builds/607
success: http://build.servo.org/builders/linux2/builds/606
success: http://build.servo.org/builders/android/builds/642
success: http://build.servo.org/builders/mac1/builds/823
success: http://build.servo.org/builders/mac2/builds/823


I want to make the massage for my work.


in progress


I think this line can be removed now.


Stashing this away for later in case there's no movement upstream.


lel


:+1: Thumbs up!

On Tue, Jan 27, 2015 at 5:02 PM, Oscar Gonzalez <notifications@github.com>
wrote:

> @rgpass <https://github.com/rgpass> done.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ogonzal87/talkative_robot/commit/c1be1d69dc3d8144d79529e815918f5d92dfd2ff#commitcomment-9468440>
> .
>


Because `UIView` animation method also has space:
```+ (void)animateWithDuration:(NSTimeInterval)duration delay:(NSTimeInterval)delay options:(UIViewAnimationOptions)options animations:(void (^)(void))animations completion:(void (^)(BOOL finished))completion NS_AVAILABLE_IOS(4_0);```


It's difficult to understand :-/
@MrColdbird has spoken a little about it his pull request, especially in this comment : https://github.com/hrydgard/ppsspp/pull/7369#issuecomment-71328878
I don't really know how to translate it, I'm thinking about translating it as something like "emptiness".


This shouldn't have a trailing ;, right?


Certainly the test at https://github.com/servo/web-platform-tests/blob/servo/XMLHttpRequest/send-accept.htm#L16 doesn't think so.


style: as `i` is only a counter, we'd generally write that the other way around, `for (i = 0; i < data->actions_count; i++)`.


True, I cannot decide which approach is better. Any idea or suggest?


Realmente, melhora a indentação e ajuda na leitura, não havia pensado nisso


Falling?


No, I don't know. I may look into this later today :)


fast-forwarding master to auto = 4a3e4032


https://github.com/bjimmykim/peer-assignment/commits/master/cachematrix.R

2599a33325fe074feda45099bce1f91589d819d8


r+


Doge bless


Void means emptiness, yes, it's a regular English word although perhaps not super common apart from C/Java function definitions...

Maybe we should replace it with some other word, got any ideas?


dd


This line should not be here. The project itself, when deployed, does not need helios.


Eu não tirei as fonts, pois achei que era dependências obrigatórias do Bootstrap, pesquisei e realmente são opcionais, vou retirá-las


line2 comm


Instead, this can be update to `^0.3.4`. Helios should be here, as it is only needed during development, not during production.


This is same as my homework!! Good job~


Hi I did your tutorial and it compile well. I have a AI that patrols between two target points and I placed one onto a platform which is not reachable by walking. I placed a navlink between the ground and this platform but the ai doesn't jump. Did I miss something?


Have you considered "lipo -thin -arch ..."?


saw approval from jdm
at https://github.com/Ms2ger/servo/commit/1b3e368bcddcf29e289f21d1f261450faaee8e0b


merging Ms2ger/servo/shadowing-layout = 1b3e368b into auto


Ms2ger/servo/shadowing-layout = 1b3e368b merged ok, testing candidate = 0f81e708


Of course, if you believe in Helios being nice and backward-compatible, you can even put `>=0.3.4 <1.0.0` or even `>=0.3.4` here.


Vou refazer este bloco


I *highly* recommend reverting this commit. Surely this is vulnerable to command injection ...


I'm too used to WebKit style of Type* and Type& that I do it automatically :-(


```js
$(obj).toArray
```
`.toArray` 메서드는 Native JS (ECMA5)가 아니다. jQuery나 Prototype 메서드이다.

언더스코에서는 JavaScript Native object 이아닐수도 있는 경우의 arguments를 넘겨줄때 이름을 `iterable`이라고 짖는 모양이다.


If I'm not mistaken, I think what we need here is:
```javascript
if(options.runtime) {
    content = 'require("imports?require=>false!' + traceur.RUNTIME_PATH + '");' + content;
  }
```
at least is what I use.


I highly recommend reverting this commit. Surely this is vulnerable to command injection ...


Because after the bump our curl version is different to the curl version in the system in away that the functions inside of the libs are not compatible anymore (might be even that we compile curl with different features/configure options then apple does). It worked by accident before. Either that or something changed somewhere else which now loads the system version of libcurl. Fact is - atm both libcurl versions are loaded into memory of the kodi process and we get a symbol clash.


No caso eu usei para o height:100% da siderar funcionar corretamente, mas poderia incluir adicionando o "html" ao seletor acima, vou corrigir


Oh. There are Smalltalkers that would frown upon these parentheses, as they think precedence rules are clear, easy to get and code should not put additional parentheses to bring confusion. I also lean more to that side, though it can be left at this form as well.


Done


@eli-jordan


thanks @t-cyrill it's works for me.


this one too
etc.


This one too


@playanth955 could you remove this empty line please?


LGTM


We'll still need this replacement. This just needs to be updated to work within Bitters' created directories.


+1, obvious oversight


Will need to add `FIRST` sets for every nonterminal. I don't want to commit to these, but maybe we can.


We can remove this comment block altogether, as it no longer makes sense.


I tried
```
nnoremap     <silent><buffer> gSac      vaBom`o<esc>xg``xdF\
```
and it seems to work fine.


you dropped this tests, does this mean you also want to drop the feature(s)/behaviour tested here?


Gtk3.14 sucks!!!. :cry:


This 3 lines break the concatenations of the scopes, returning only the last one in the hash, skipping all the scopes but the last. It should be reverted.


you can only forward declare enum class


I guess this one ahould be setProvidedValue()


Github needs a 'like' button for changes like this!


Thanks. Done


OK, so it's not related, but necessary.


Lars,

I got the following error:

> CMake Error: The following variables are used in this project, but they are set to NOTFOUND. Please set them or make sure they are set and tested correctly in the CMake files: BZIP2_LIBRARIES (ADVANCED)

You are right, I thought I was setting SKV_LOCAL_KV_BACKEND, but I wasn't doing it properly.

The issue, however, is that rocksdb is now the default, and it falls back to inmem when rocksdb isn't found, setting BZIP2_LIBRARIES, etc. to NOTFOUND:

```
find_package(ZLIB)
find_package(BZip2)
find_package(rocksdb)
if(NOT ROCKSDB_FOUND)
  message(STATUS "rocksdb not found, falling back to inmem backend")
  set(LOCAL_KV_BACKEND "inmem")
endif()
```

So, I guess it should either fall back properly and empty those variables, or fail earlier without falling back to inmem when rocksdb can't be found?


可是这个的结果和 `/img/scrollup.png` 的是一样的，所以你的环境下的错误是怎么产生的啊？

我刚开始弄，语法还不太懂 :sob:  ，试着改的。你再测试下，OK 的话我去 PR。


ddddd


You can only forward declare enum class.
What is used here is a MSVC language extension allowing you to also declare enum with specified underlaying type


Done


done.


LGTM


Here is the problem, we have to understand what meaning @MrColdbird was thinking about when naming that option... :-/


Hi @milkovsky. Indeed, it should, thanks. This is an old version, btw. I still have quite some refactoring to do.


Looks good


I agree. This needs to get fixed. Could you try running cmake with "-DSKV_LOCAL_KV_BACKEND:STRING=asyncmem" - that should do the trick until it's fixed.


I see, the obvious naming issue was that it did not start with `classloader.xxx` or `loader.xxx` so that if there are multiple JRuby class-loader options they share the same "name-space" ... also I probably would mostly switch it from an (embed) API not the command-line, but I might have missed a use-case, just wanted to give some feedback.


Great, thanks!

On 28.01.2015 15:06, Karl Hobley wrote:
>
> LGTM
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/torchbox/verdant-rca/commit/f6b5804b0ee175d8683c030e5d67466dae7c9a9b#commitcomment-9478965>.
>


It's okay. http://stackoverflow.com/a/1280969
4.6 gcc has it


Explain this to me? I have no idea what the commit title even means


How is that? If all fields are not equal to '' than you do submit? Do we really require all fields?


He added newlines.


typo 'intance'


Says using frfr for the maps, but DBC only gets enGB en US


for the dependency injection


no crap.  I totally didnt notice that....


Please remove all those empty lines.


# This is an <h1> tag
## This is an <h2> tag
###### This is an <h6> tag


is it worth checking that the domain is ``@rca.ac.uk`` here?


Done


why? (and the answer is because angular sucks balls is perfectly acceptable)


Stray import


Because long lines suck balls... and so does angular.


I agree with @WojciechTrzasko. I do put empty lines as well – it makes clarifies which dependencies are used in a source file. Here's my style, just for the record:

```objc

// 1. frameworks (modules)
@import Foundation;

// 2. libraries
#import <AFNetworking/AFNetworking.h>

// 3. Classes and protocols, alphabetical order
#import "XYZManager.h"
#import "XYZSerializable.h"
#import "XYZUser.h"
#import "XYZViewController.h"

// 4. Categories, alphabetical order
#import "NSDate+Formatting.h"
#import "NSString+Manipulation.h"
#import "NSString+Regex.h"
#import "UIImage+Archive.h"
```


- [ ] a task list item
- [ ] list syntax required
- [ ] normal **formatting**, @mentions, #1234 refs
- [ ] incomplete
- [x] completed


Mostly long lines though


so nothing to do with DI, so yeah


Warto by odpalić flak8 lub checkera.
Trochę dziwnie wygląda ten długi komentarz `#########` w testach.


Just wait, if he hadn't added those newlines, you would have had a horizontal scroll in every diff.


me? no, I would be reading angular to begin with


Classic angular multiline dependency injection ;)


I don't think this is the right place to introduce `fn:error(...)` calls. Until now we only introduced them in conditionally evaluated expressions, e.g. [`if`](https://github.com/BaseXdb/basex/blob/master/basex-core/src/main/java/org/basex/query/expr/If.java#L59), [`switch`](https://github.com/BaseXdb/basex/blob/master/basex-core/src/main/java/org/basex/query/expr/SwitchCase.java#L42) or [function bodies](https://github.com/BaseXdb/basex/blob/master/basex-core/src/main/java/org/basex/query/func/StaticFunc.java#L86). This way all cases that can only lead to an error are reported at compile time. As the current problem seems to be with XPath steps, I would rather see the `error(...)` be introduced there.


Would this return a redirect to the ``student-profiles:overview`` view?


I would probably do a separate commit for gem updates that require app changes (like attr_extras did) so you can tell what gem they're for, so they can be reverted easily etc.


that ^^ sounds like the tagline of a samccone Github project


`Classic angular multiline dependency injection based on Promises`


Version checking seems a bit stupid to me. When making a custom build you'd have to change the API too...

When the server is built, the API is automatically included so probably the API is compatible with the server.


不错


这个是正确的, 我测试了一下. 我也不太懂其实.
我的错误是因为如果url_for不带参数的话, 会返回根目录, 比如 /, 或者我的情况是 /blog/. 这样就会产生//img/scrollup.png, 访问不到, /blog//img/scollup.png, 其实也是两个斜杠, 但是能访问到.


Is this a change to the EspruinoTools repository? I get complaints `Object #<Object> has no method  flashBinaryToDevice'`


Check out this neat program I wrote:
```
x = 0
x = 2 + 2
what is x
```


Maybe this could be moved to the feedbag channel... :P

![](http://cl.ly/image/3w1P090U1V35/Image%202015-01-28%20at%209.13.09%20AM.png)


With DeprecationWarning, I mean https://docs.python.org/2/library/exceptions.html#exceptions.DeprecationWarning

Warnings are like Exceptions, but instead of crashing the entire programme, they will only print a warning in the console (and in our case also into the log file), when caught.


I think you should use an
`<addr>` element here instead.


As Kanye West said:

> We're living the future so
> the present is our past.


maybe


where is the fun in that tho?  think of this as feedbag channel++


First Header | Second Header
------------ | -------------
Content from cell 1 | Content from cell 2
Content in the first column | Content in the second column


Should we check that the user is in the students group?


with the added benefit that every one of these messages also is appearing in the feedbag slack channel for double the enjoyment


| Name | Description          |
| ------------- | ----------- |
| Help      | ~~Display the~~ help window.|
| Close     | _Closes_ a window     |


Ok. I will make it an option. Thanks for your feedback
On Jan 28, 2015 2:47 AM, "Baki Goxhaj" <notifications@github.com> wrote:

> That would take a while to load in a website with many images. I would
> suggest to make this check optional. I would not prefer it on my site
> personally.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gad2103/ajaxify/commit/f375bdad4552dc20e193bb7e95278217e1693908#commitcomment-9473970>
> .
>


Looks good. Just a few minor tweaks/questions above


| Left-Aligned  | Center Aligned  | Right Aligned |
| :------------ |:---------------:| -----:|
| col 3 is      | some wordy text | $1600 |
| col 2 is      | centered        |   $12 |
| zebra stripes | are neat        |    $1 |


You should create one solution to all this :afters.


Yes, it is - you need to pull in the changes for that too.


In this case, an extra array is a bit overkill just to keep around the notion that the number is negative. This code would work just as well:

```ruby
negative = false

if num_array[0] == "-"
  negative = true
  num_array.shift
end

# ... then, at the end of your method

new_array.unshift("-") if negative
```


First Header  | Second Header
------------- | -------------
Content Cell  | Content Cell
Content Cell  | Content Cell


Can you open this as an issue please?


Ja w zasadzie mam tylko stylistyczne zastrzerzenia (pep8, jak @quamilek wspomniał). Tak to wygląda ok według mnie.


use util.Inherits for this.  http://nodejs.org/docs/latest/api/util.html#util_util_inherits_constructor_superconstructor


Somehow this works but the strcmp not. Check it in dummy scanner


Yes, when no 'next' is set, it shows the same URL again which returns a redirect to student-profiles as of line 15.


This problem is about to be handled properly using http://lists.denx.de/pipermail/u-boot/2015-January/202306.html

The current demo image is available at https://github.com/ssvb/sunxi-bootsetup/releases/tag/20141215-sunxi-bootsetup-prototype . It can differentiate boards by the soc type, dram size and dram bus width, using the following table in /boot/setup/u-boot-binaries/sunxi-boards.cfg :

A10-OLinuXino-Lime       sun4i     512 MiB (16-bit)
A10s-OLinuXino-M         sun5i     512 MiB (16-bit)
A13-OLinuXino            sun5i     512 MiB (16-bit)
A13-OLinuXinoM           sun5i     256 MiB (16-bit)
A20-OLinuXino-Lime       sun7i     512 MiB (16-bit)
A20-OLinuXino-Lime2      sun7i    1024 MiB (32-bit)
A20-OLinuXino_MICRO      sun7i    1024 MiB (32-bit)
Auxtek-T004              sun5i     512 MiB (16-bit)
ba10_tv_box              sun4i    1024 MiB (32-bit)
Bananapi                 sun7i    1024 MiB (32-bit)
Cubieboard               sun4i    1024 MiB (32-bit)
Cubieboard2              sun7i    1024 MiB (32-bit)
Cubietruck               sun7i    2048 MiB (32-bit)
i12-tvbox                sun7i    1024 MiB (32-bit)
Linksprite_pcDuino3      sun7i    1024 MiB (32-bit)
Mele_A1000               sun4i     512 MiB (32-bit)
Mele_A1000G              sun4i    1024 MiB (32-bit)
Mele_M3                  sun7i    1024 MiB (32-bit)
Mini-X                   sun4i     512 MiB (32-bit)
Mini-X-1Gb               sun4i    1024 MiB (32-bit)
qt840a                   sun7i     512 MiB (16-bit)
r7-tv-dongle             sun5i    1024 MiB (32-bit)

Currently the user gets a setup menu on a HDMI monitor, which can be navigated at least with a FEL button (which, I believe, should be available on any devboard). The USB keyboards are also partially supported.

I may also implement automatic selection without any need for input from the user in the case if only one possible choice is available. For example, reducing the sunxi-boards.cfg to:

Cubieboard               sun4i    1024 MiB (32-bit)
Cubieboard2              sun7i    1024 MiB (32-bit)
Cubietruck               sun7i    2048 MiB (32-bit)
A10-OLinuXino-Lime       sun4i     512 MiB (16-bit)
A20-OLinuXino-Lime       sun7i     512 MiB (16-bit)

Unfortunately it is impossible to differentiate Bananapi and Cubieboard2 automatically, so the user's choice would be still needed.


```js
if (_.isArguments(iterable))  return slice.call(iterable);
```
`Array.prototype.slice.call(object, 0)`는 `arguments`와 같이 key가 0에서 부터 1씩 올라가는 Array like Object들을 array로 바꿔 주는데 쓰인다.

`_.isArguments`는 어떻게 `arguments`인지를 채크할까?
```js
_.isArguments = function(obj) {
  return obj && _.isNumber(obj.length) && !obj.concat && !obj.substr && !obj.apply && !propertyIsEnumerable.call(obj, 'length');
};
```
ECMA3에서도 존재하는 Array 메서드 들이 없으며 , `obj.length`가 `number`이자 건들지 않았으면 `arguments`라고 정한다. 여기서 `!propertyIsEnumerable(obj, 'length')`는 `obj.length`를 오버라이드 했는지 안했는지를 보는것이다. `obj.length`에 값을 지정해주면 Non Enumerable이었던것이 Enumerable로 바뀌는 특이한 현상을 이용한것이다.

그렇다면... Array인데 `_.Arguments`테스트를 통과해볼라면?
```js
//Copy Paste to Console;
var _ = {};
_.isNumber = function(obj){
  return (obj === +obj) || (toString.call(obj) === '[object Number]');
};
_.isArguments = function(obj) {
  return obj && _.isNumber(obj.length) && !obj.concat && !obj.substr && !obj.apply && !Object.prototype.propertyIsEnumerable.call(obj, 'length');
};
var obj ={a:1};
obj.concat = 0;
obj.substr = 0;
obj.apply = 0;
Object.defineProperty(obj, 'length', { value: 1, enumerable: false });
_.isArguments(obj);
```
:memo: 언더스코 1.0.0에서는 메서드의 존재들을 이용하여 Feature태스트를 한다. Modernizr같이.


I don't think so. It's just a convenience function for students who mistakenly use their email address. Actual login is handled by the LDAP backend.


Er bare å lage PR det, så ser vi på saken :)


test


The library should not be loading the config.  The config needs to be passed by the user when instantiating the class.
If you do this, you can only ever create one client per process, we need to create many clients each with a different token.


comments


Just my 2¢: I don't line resolving `nil`s to `NO`s. Writing `something != nil` makes code more readable, since you *instantly* realize that `something` is an object, not a boolean value.