5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1021 of 1021 tests passed, 0 test warnings)

![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-30.52%25-green.svg)


this is the same as good_file1 why doubly define it?


It's not an error, this checks if the args property exists in the component class. The args property is used for UI


same


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1579 of 1579 tests passed, 0 test warnings)

![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-70.82%25-green.svg)


this is a problem, will get it to it!


checkboes -> checkboxes ?


you could give them some better names, like `bad_file_spacing` or something like that instead of giving them numebr


Skulle det her filnavn ikke have været omdøbt til "minakasse.png"?


or should i just add comments to describe ?


"ftfa.jpg?"


This breaks backwards compatibility with Arduino IDE 1.6.5r5 and previous:
```
C:\Users\per\AppData\Roaming\Arduino15\packages\arduino\tools\avr-gcc\4.8.1-arduino2/bin/avr-gcc -Wall -Os -Wl,--gc-sections -mmcu=attiny841 -o C:\Users\per\AppData\Local\Temp\build3918553802547121879.tmp/sketch_mar05a.cpp.elf C:\Users\per\AppData\Local\Temp\build3918553802547121879.tmp\sketch_mar05a.cpp.o C:\Users\per\AppData\Local\Temp\build3918553802547121879.tmp/core.a -LC:\Users\per\AppData\Local\Temp\build3918553802547121879.tmp -lm
avr-gcc: error: C:\Users\per\AppData\Local\Temp\build3918553802547121879.tmp/core.a: No such file or directory
Error compiling.
```


ma è un update dei vendor anche? sto avendo qualche problemino.


Shouldn't be 8.1.6 before 8.0.11 ?


Yup, `gtk_menu_shell_insert(menubar, menu_vc, g_list_length(gtk_container_get_children(menubar)) - 1);` would work.
But as said, this is just cosmetics. If you have time, I guess @frlan would merge a PR happily or even fix it himself :).


names are way better than comments!


What's the state of the 9250 mag on pixracer? What kind of testing has been done and what remains?


3.50 -> 3.5.0


c


should be * 1000


Names are better as pytest shows the name which failed


Link to github issue is probably wrong. Also not sure about "filder"?


Link to github issue probably wrong


...Really?


please, can i get the code source of you're example, I'm new in ns3 and i couldn't understood how it works. thnks


changed read me file


5499-Electric-Service-Follow-Up (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (2129 of 2135 tests passed, 3 test warnings)

 * 9 tests had: AUD diffs.
 * 9 tests had: MDD diffs.
 * 9 tests had: MTD diffs.
 * 1 test had: ERR diffs.
 * 6 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


Condition is always true


it has a space on line 21, so when we run it with config file( which allows a space at end ) then it is a good file


config.hp -> config.php


Maybe, it looks like moving cursor up, but actually, you are moving it down.


tree is a beautiful command


calDAV and cardDAV -> CalDAV and CardDAV


an -> and


This condition will be changed to ask for demuxers capabilities to handle independed streams
INPUTSTREAM_CAPABILITIES


Those 2 lines are missing `final`


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2129 of 2135 tests passed, 3 test warnings)

 * 9 tests had: AUD diffs.
 * 9 tests had: MDD diffs.
 * 9 tests had: MTD diffs.
 * 1 test had: ERR diffs.
 * 6 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


Those 4 lines are missing `final`


What about we use C++11's auto?
```c++
for (auto it = m_components.begin(), end = m_components.end(); it != end; ++it) {
```
It's shorter...


Cool, but I would change the color too, then, to label-success or info or something


Ah I see where your point. The ID is in fact included but apparently godoc only shows a comment about filtered elements.
If you look at https://github.com/nlopes/slack/blob/master/channels.go#L20 you see that it contains groupConversation and that groupConversation (https://github.com/nlopes/slack/blob/master/conversation.go#L15) contains conversation (https://github.com/nlopes/slack/blob/master/conversation.go#L4) which indeed contains ID.

I just added https://github.com/nlopes/slack/blob/master/examples/channels/channels.go to the examples folder to give you an example of how to reach in to the (doc) filtered fields.


Wow its fix everything and now we can use map_foreachinshootrange  :+1:


@borsch  в карточці все описано. Був окремий період узгодження.


In my eyes this part was missing - if a stream is closed, it should be disabled in Demuxer.
Not doing this leeds to the fact, that an audio change with seek will be done on streams wich will be deisabled in a later step. Better disable this one directly.


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18456](https://teamcity.mesosphere.io/viewLog.html?buildId=155107&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:04:43


text should be annotated `@NonNull`


классы именуются с большой буквы, т.к. в задании он нигде не используется => должен быть удален


@ardiman
This is what is killing viewing your homepage. I will remedy this on production next week... however the url needs `%` represented as `%25` e.g. `encodeURIComponent` here. It's a *node* thing. If you are using QupZilla, or equivalent, it doesn't always encode correctly from the address bar copy to clipboard.


oh trailing whitespace - coala will cut that away!


well, I'm not sure, since i don't have a logcat, but since i use kirin930 as TARGET_BOARD_PLATFORM to build recovery and recovery work without problems, why, aren't same thing switch commits HISILICON_HI3630 to HISILICON_HI6210SFT?


@shama does this fix the issue @mafintosh and @freeman-lab had with dom nodes getting replaces during .update unless they manually set ids on them? p.s. you should get IRCCloud :)


you can just set `good_file2 = ['blablab\n', ' \n', ...]` to keep it explicitly there


Is this German by mistake?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18457](https://teamcity.mesosphere.io/viewLog.html?buildId=155111&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:04:48


Honestly, I have no clue about how to do it like this! Feel free to open a PL for it :wink:

#cssNoob :blush:


here also German


This commit is under test.


Did you intentionally remove Gordon or was that an accident as part of the reordering?


no, this is the *de*.lproj/Storyboard.strings file. As an example I translated a 2 words only "Regionen" and "Freunde"


Actually I was confused as well by this conflict... Don't worry about it, since I will merge the blurb next week :wink:


Works fine for me, I'll add the `final` statements, then merge into `release`, unless someone else has something to say :)


I have no idea, it was on the original patch...


see above


You should change the title to add the missing #


Oops. Entschuldige bitte!


Gordon didn't add himself to the contributors, so I added him later :yum:


No... Che problema?
I vendor brasali e riscaricali con composer update

Inviato da iPhone - Stefano T.

> Il giorno 05 mar 2016, alle ore 18:17, Md Ariful <notifications@github.com> ha scritto:
>
> ma è un update dei vendor anche? sto avendo qualche problemino.
>
> —
> Reply to this email directly or view it on GitHub.
>


NP, just translate the other lines too


Hmm, you might need to run this install in an Administrator terminal, because non-admins can't set symlinks on Windows.


@poteto - Hi. I'm curious why you removed the `this.attrs`. Isn't this what will happen when angle brackets land? So basically, if we use `this.attrs` we're future-proofing our work for angle brackets or am I missing something?


Is anybody using this?  Does it work well?  I'd like to buy 2 Insteon fanlinc controllers for my 65-device, wink-centric system.  There seem to be no wink-compatible options out there (ceiling-mounted controllers, not on switch) that are workable via Wink.  I'm willing to get an Insteon hub if need be, but would like to know that if I do so, the fan speed will be controllable via at least Siri and HomeKit, if not Wink.


These should be s->width, s->height.


Hm well, there seems to be a misunderstanding regarding the purpose of this repository. I intentionally left everything blank and bare-bones in the beginning, so that they can find out what and how to add it themselves. Imho the student pages should start only as some empty tags that prepare valid html pages, not more. As little stuff as necessary, so that they can understand as much as possible of it, and don't feel intimidated by unfamiliar stuff.

And there should be _no_ styles to start with. It's for them to add. Yes it starts out ugly. The greater is the opportunity to improve it. And why the reference to an external css resource? I'd like to get the point accross that for basic stuff, they can rely on themselves, and don't need external stuff from here and there without really understanding what it does. (Side note: it creates mixed content warnings when accessing the page through https, and the link looks like it could go away with the next deploy, when the hash changes)

Adding a reference to a shared base css is also something that I thought they could discover on their own while experimenting with this app. This way, they'd gain some understanding how referencing stylesheets works.

@pihme I hope you understand that for the reasons outlined, I ask you to revert this commit.


It's needed in Python


Why has this been removed??


1. static
2. Обычно вложенные классы объявляются в конце


This is true. But the reason for this is about 20lines above. From my point of view its ok/  better to write the code this way.


I will do a checkout this evening, run it and try to test every element of billing, record export and reporting to check functionality.


поля класса объявляются в начале, до его конструкторов


I see there is a new parameter H3,
but which seems to be set to 0 basically everywhere.
And it doesn't seem to cause any problem.

I suspect it technically impacts the hash table for 3-bytes matches,
and when set to 0, it defaults to MIN value, which is 12.

OK, but does setting this parameter make any difference for compression in the end ?
better ratio, or better speed ?

If not, could it be simplified to be always the same value (or automatically derived from another one) in order to reduce parameter structure complexity ?


This breaks backwards compatibility with Arduino IDE 1.6.6 only(tested on Windows 7):
```
"C:\Program Files (x86)\arduino-1.6.6\hardware\tools\avr/bin/avr-g++" -c -g -Os -w -std=gnu++11 -fno-exceptions -ffunction-sections -fdata-sections -fno-threadsafe-statics  -w -x c++ -E -CC -mmcu=attiny841 -DF_CPU=8000000L -DARDUINO=10606 -DARDUINO_AVR_ATTINYX41 -DARDUINO_ARCH_AVR   "-IE:\Stuff\misc\electronics\arduino\hardware\arduino-tiny-841\avr\cores\tiny" "C:\Users\per\AppData\Local\Temp\build7f79c2e27eef3adbb59d9f7386c82090.tmp\sketch\sketch_mar05a.ino.cpp" -o ""
avr-g++: error: missing filename after '-o'
```
I have managed to fix this error in 1.6.6 by adding the line:
```
preprocessed_file_path=null
```
which doesn't seem to cause any change in the commands in Arduino IDE 1.6.7(diff of the verbose console output is identical other than the temporary folder names).


1. final
2. Hashtable это synchronized версия, т.к. мы не предполагаем многопоточности, то используем HashMap


I'll move this to the object if possible


Where? :confused:


Oh nevermind I brainfarted and read it in the opposite direction lol


Really Good! Is this going to increase the size of the Go library?


:yum:


зачем указан пакет для Trie? они же в одном пакете лежат, можно не указывать


This is mentioned in the params detail below


"Fixes 158" => "Fixes #158"


char


If ll you want to achieve is to allow the compiler to deduce these template arguments via overloading rather than through explicit specification, you can use the `int2type` class.


Feel free to update the code. I can also add your findings in maybe 1 or 2 hours.


Concordo com o samuel


python manage.py makemigrations app
python manage.py migrate


un**e** installation par git


final, обычно root


Bei der Architektur des Session-Empfängers überlege ich, ob es überhaupt einen aktiven Thread geben sollte. Es wird ja eigentlich schon im Datagram-Layer entsprechend gepuffert. Ich denke gerade so in folgende Richtung:
- Receive-Aufrufe dürfen prinzipiell blockieren,
- wenn wir aber Dinge aus einem Endpunkt an mehrere Threads verteilen müssen (Multiplexing), dann brauchen wir einen Thread der das aus dem Endpunkt holt und in verschiedene Töpfe legt.
Das würde aber heissen dass man weder im "ganz grundlegenden" DatagramEndpoint einen Thread hätte, noch in diesem SessionEndpoint, sondern dass dazwischen ein Multiplexer wäre der aus einem DatagramEndpoint mehrere ebensolche macht.


Gør det nogen forskel?:)

On 5 March 2016 at 18:17, Kevin Simper <notifications@github.com> wrote:

> "ftfa.jpg?"
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kevinsimper/akasse/commit/81163f3bf276b96846cd5b9b763e82e240ffe745#commitcomment-16518939>
> .
>

--
Med venlig hilsen / Best regards

*Mathias Lund*

Web Crawler

*Graduateland <http://www.graduateland.com> - Europe's largest career
portal for students and graduates*

Check out our 2-minute video <http://vimeo.com/57135498>

*Tel*: +45 26255042
*Mail:* mathias@graduateland.com *Web:* www.graduateland.com
<http://graduateland.com/ref/pl>

Graduateland ApS
Njalsgade 21G 7th floor
2300 Copenhagen S
Denmark


lol ;_;


curNode.map - либо везде геттеры/сеттеры, либо обращение напрямую


@natouille tu veux tester l'installation sous Ubuntu ?


What is a tvp3026? anyone know? If not, can we remove this file?


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2161 of 2167 tests passed, 5 test warnings)

 * 9 tests had: AUD diffs.
 * 9 tests had: MDD diffs.
 * 9 tests had: MTD diffs.
 * 4 tests had: ERR diffs.
 * 6 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


OK to remove.


sorry no, it has private data, let me know if I can help by sending you privately


OK to remove, I guess.


Agreed.  In theory this isn't correct but in practice I believe it's a moot point.  I am fairly confident that the overwhelming majority of client calls to gethash that the request will readily fit inside a single packet.  I suspect that any requests that don't fall into that category are either maliciously crafted or extremely buggy clients.

As part of a discussion brought about as part of the changes here, I realized that we need some production metrics on what the size of requests to gethash really look like in order to tune numbers for reasonable upper bounds checks on the incoming request.  Some of that data can be grabbed from the load balancer and/or HTTP daemon fronting the WSGI service.  More detailed metrics will require some small changes here to record some types of request body errors, including how often we have a short read.


Wrap with #if 0, because it might beuseful.


It only showed that message once... now when I use `npm update` or `npm install`, it completes without any messages... and I just tried that in an admin windows as well.


Yes, I realized it a few moments ago :yum: Thanks!


remove.


修改建議：
>
最後一點正是將此實現與 [Chris Coyier](http://css-tricks.com/float-labels-css/) 以及 [Jonathan Snook](http://snook.ca/archives/html_and_css/floated-label-pattern-css) 區分開來的地方，後兩者均使用了 `:valid` 伪类。我認爲此 demo 無法突破這個特殊的限制，但正如我一開始所説，這些都是因瀏覽器支持各有所限導致的。

Notes:
>
* 注意强調作者的 demo 與別人的 demo 在實現上的不同之處，而不是“作者的 demo 與別人的 demo 不同”這個事實
* 個人認爲原文中 that particular limitation 是指代上文 `:valid` 偽類的使用限制吧
* 我認爲作者并非想表達其 demo “圍繞特定限制所實現”，而是想突出自己的實現和前文提到的實現一樣有相同的缺陷（step around 譯成“圍繞”是主觀色彩，像是意圖表達作者主動實現此 demo 是爲了展示限制，不妥，所以應該譯成“離不開，無可避免”等被動之意）


Sollte das nicht einfach ein Byte/Character Array sein? Auf diesem Level wollen wir uns ja eigentlich noch gar nicht mit Encoding rumschlagen. Messages sind einfach blobs.


popartx.com


This change shouldn't be here.


Wow, how many of these did they write? OK to remove.


Geez. Another one? OK to remove. Can we put this in libc?


And another hex dumper! OK to remove


This groovydoc is not currently available, but the static page generated by maven (one-level upper) is.
When deploying script is fixed, the upper level page will automatically have a link to groovydoc.
Should I leave the URL as what's available now?


I'm really inclined to remove all of cifs, any objections?


This doesn't make sense to me.  Why would `getBaseResources` return the village?


Momentan bin ich gerade der Meinung, dass die Session ID niemals nach "aussen" dringen muss, sondern nur für das Multiplexing "innen" von Belang ist. Nach aussen sollte das eigentlich durch das Objekt schon adressiert sein.


OK to remove.


Ok, that makes sense. I am using git commit --amend --no-edit to add the nano update to the commit. Will change the process until we can figure out a way to fix this annoyance.


@robske110 Could you also add something like "[world]" at the end?


you can simply write this as `.catch(callback)`


Wrap with #if 0 and all other usages/defintiions. Leave a warning that it is a security hole and should only be enabled for new ports.


ok


how many of these did they write?


Moment, heisst das wir codieren die Pakete gerade als JSON?


Great, thanks!!!


Looks good to me!
I will test in on CentOS and merge it on Monday.
Thanks!


And, no, a cpp version is not allowed.

Write a min that takes these parameters if you need it.


Не понял, приведи пример.


Yeah, lines 661/662, and I think the TD tag at 664.


@digawp. Yes I think you may be right. Have you tried changing it to only have 1 entry point and see if the tests are run properly?


another one? OK to remove.


I would argue for removing httpd entirely. It's old junk. But do whatever you want to httpd.


First of all, nice.
Second of all, I was thinking of maybe different infoboxes for the different paragraphs and stuff. I'll start moving stuff as soon as I finish this sine thing.
![sin cos wave thing](https://cloud.githubusercontent.com/assets/15935194/13549334/db659e12-e2d0-11e5-9919-e2081570d141.PNG)


методы определяются после конструкторов, конструкты и поля внутри себя сортируются по модификатору доступа: сперва public, потом protected, потом default, потом private


Thanks, that was probably a rebase fail on our part. We have been using this commit on your master for quite some time. The disabled tests above cause trouble on containerized hydra builders in our case.


imap4d? IMAP4D? remove all of imap4d.


ppp? It never worked that well anyway. Just remove all of ip/ppp.


Agreed.


Add Autor


Find out why this is commented out. DO NOT remove anything from stats. Fix it.


@peterqz because the text shared can only be accessed from the Fragment, therefore the menu must be the one from the DetailFragment.


This should also not be removed.  This is for INTEGRATED_AR...which we keep.


I have no idea what to do with libsec, so find someone who does :-)


Meri pull request merge ki hoti
@rohantaneja 😜


Feel free to update the code, I'm unavailable at the moment...


it seems that setDuration(duration: Integer) and getDuration(): Integer in SendAudio are missing. however they exist in Audio


падают следующие тесты:

        Trie.add("abcd");
        Trie.add("abcd");
        Trie.add("abcd");

        assertEquals(1, Trie.size());

и

        Trie.add("abc");

        Trie.remove("abc");
        Trie.remove("abc");
        Trie.remove("abc");

        assertEquals(0, Trie.size());

И еще, если у узла нет терминальных наследников, узел должен быть удален


Missed an Essence here.


They're confusingly named because the purpose of the ConstantsService functions changed as the VillageService was written. I have a fix for this but it's involved in a different changeset.


Missed an Essence here as well.


Ooh, thanks!


This is the last one.


1) ProductItem
     1.1) Color list with size ids
     1.2) Size list with color ids
2) update controllers, views to new model


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18458](https://teamcity.mesosphere.io/viewLog.html?buildId=155115&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:05:05


 - __information__: [src/chaiscript_stdlib.cpp:0](../blob/539ee3c84f50da96767b7741b1879a2bfdddaa10/src/chaiscript_stdlib.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 16 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/539ee3c84f50da96767b7741b1879a2bfdddaa10/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/539ee3c84f50da96767b7741b1879a2bfdddaa10/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/539ee3c84f50da96767b7741b1879a2bfdddaa10/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/539ee3c84f50da96767b7741b1879a2bfdddaa10/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/539ee3c84f50da96767b7741b1879a2bfdddaa10/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/539ee3c84f50da96767b7741b1879a2bfdddaa10/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 17 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/539ee3c84f50da96767b7741b1879a2bfdddaa10/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/539ee3c84f50da96767b7741b1879a2bfdddaa10/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/539ee3c84f50da96767b7741b1879a2bfdddaa10/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/539ee3c84f50da96767b7741b1879a2bfdddaa10/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/539ee3c84f50da96767b7741b1879a2bfdddaa10/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/539ee3c84f50da96767b7741b1879a2bfdddaa10/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


5499-Electric-Service-Follow-Up (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2129 of 2135 tests passed, 3 test warnings)

 * 9 tests had: AUD diffs.
 * 9 tests had: MDD diffs.
 * 9 tests had: MTD diffs.
 * 1 test had: ERR diffs.
 * 6 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


Oops! Sorry, that’s what I intended to write. Good catch!


We could. Can you remind me why we need it at all? Is client/check not a unique key across our infrastructure?


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (398 of 398 tests passed, 0 test warnings)

![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)


Got them in https://github.com/NucleusPowered/Nucleus/tree/refactor/rename. Thanks for pointing them out.


lol


Good call.


This change belongs with the libcurl commit, right?


You've added per-world weather, so you need to add at the end of usage "something like [world]" @robske110


@kzeng10 I removed the parseInts and it still worked. Do we need them?


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (390 of 390 tests passed, 0 test warnings)

![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)


Looks good.
Should we put the comments describing the problem to the top then start the code  part?Also should the problem number index be included in the comment?
Thanks


Ah got it


Actually it was invalid before - I don't care.


@YSRKEN revertします。
https://github.com/YSRKEN/KanColleSimulator_KAI/issues/22
を見てください。過去に私がやらかして直したのにまたやらかしてどうする・・・


release-5.x (lefticus) - i386-Windows-10-VisualStudio-14: OK (390 of 390 tests passed, 0 test warnings)

![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)


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Oh, OK.  I thought this wasn't needed now that we are using the find_package for libcurl...  But if this is just a standalone change that wasn't ever needed, please disregard.


CMakeLintBearTestEndSpacesAllowed  is this name too long


If you like I can make from it a third commit.


I get this error:
```
bpy.c:315: error: implicit declaration of function 'initGameTypesPythonBinding' [-Werror=implicit-function-declaration]
  initGameTypesPythonBinding();
  ^
```
If I uncomment this...


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1579 of 1579 tests passed, 0 test warnings)

![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-70.82%25-green.svg)


That is up to you, I'm fine either way.  My question was more about me trying to understand what makes this change possible.


same repo dif name


This was cherry-picked from master. Don't panic.


This doesn't seem to be necessary by the docs?


Yippee


Took out Traces


- [ ] Test
- [ ] Check this


У листа, как такового, нет поянтия tail. Лсит должен быть листом.. и ничем больше.. а сейчас у тебя это смесь листа с очоередеью.


how many time mumfrey for 1.9 dev? ;)

continu your good work :P


Вроде как, если размер говорит, что он 0. то tail уже не существует, а ты его создаешь заново.
И у тебя все еще размер счиатется за размер очереди.


Thanks for reviewing this code Art. This type of code review definitely needs several sets of eyes.


CMakeLintBearSpacingTest?


Yeah... They need to be done through the NBT values. You can't map them via meta as if you do, every time a plant type is added or removed it would break the entire map.

The conversion should be doable, I just have to find a way to ensure that all the variants will be baked still by the loader. But for now the multiple clipping items is an easy working intermediate.


would prefer that their involvement


Thanks.


do you test the dict conversion?


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (3048 of 3048 tests passed, 0 test warnings)

![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)


Phil, slow down, you're making us all look bad


even from commit messages? but yeah RST is also fine :+1:


You should be able to use the module import here: `@import MinimedKit;`


(assuming you're building MinimedKit as an embedded framework)


A librecad, mint 2D CAD program, maradhatott volna extra.


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (2124 of 2130 tests passed, 5 test warnings)

 * 9 tests had: AUD diffs.
 * 9 tests had: MDD diffs.
 * 9 tests had: MTD diffs.
 * 4 tests had: ERR diffs.
 * 6 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


@Yoann166 They'll need to update the plugin.


yes otherwise , it wouldn't have passed the test right? i also did explicitly test it, but not without coalang files


When I started to work with CoreFX it already had wrongly hard-coded optional path for pkgsrc dependencies it may be `/usr/pkg/include`, but it's purely optional path. Also pkgsrc is purely optional on NetBSD (and any other system, even if it's the default package manager like in SmartOS).

The correct way of handling it via CMake is finding includes (there was a macro for it) or discovering packages. I went for the latter.

BTW. The same issue is in FreeBSD code -- it's not valid to hard-code `/usr/local/include`. It may work only in the default circumstances. However I don't touch their parts.

I will put this change to a 3rd commit.


That wasn't a review, I was just playing yes-man to what you said. I
will try my best to review in detail (after today's planned fun with the
board meeting).  I put Terry up to this as much as anything else, so I
need to do my part.

On 3/5/2016 1:11 PM, Brady Miller wrote:
>
> Thanks for reviewing this code Art. This type of code review
> definitely needs several sets of eyes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/teryhill/openemr/commit/56e7f5ba9bf75174c0a1a5aecb166d292acd6162#commitcomment-16519292>.
>


i feel like this could receive slight improvements. Like: Hey there, friend! **It's** awesome **that** you..


OK


This looks good, your tests are cleaner than what I had done.


I know, it is my fault, sorry. But in ruby, all methods should be named in `snake_case`, not in `camelCase`. Please, fix it everywhere.


I don't want a release with this though, a temporary merge is fine.


C# SHARP


This change breaks existing code that assumes that `MaterialCheckBox` is instance of `CheckBox`, such as:

* `CheckBox box = new MaterialCheckBox();`
* `@UiField CheckBox box; //and the field on UiBinder is a MaterialCheckBox`
* `Widget w = new MaterialCheckBox(); ((CheckBox)w).getValue();`

and so on.

This kind of code is weird on new projects, but it happens a lot when you're porting an existing GWT app to Material - that's one of my projects.

This change should be notified at release notes or something like it because it can break things at compile time and runtime (such as casts).


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (390 of 390 tests passed, 0 test warnings)

![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)


can't we be a bit colloquial... hey this is newcomer bugs! :P


saw something like this in the documentation for the Lint module, i didn't know Bears had documentation, i just wrote these to help understand the code for our reviewers , i'll improve the documentation like it should be.


LGTM.


I made a pull request to disable this.


@nlf, this may lead to an issue where the hostname is different from the default one. Perhaps tap into `dlite ssh` which already resolves the hostname correctly?


funny guy


We need comment above this method, because there is not obvious argument list.

```
   # Erase the specified region.
   # The region describes the rectangle shape which need to erase.
```


What about making a backup copy just to be on the safe side?


Thanks, forgot to simplify after removal of log call


release-5.x (lefticus) - Win64-Windows-10-VisualStudio-14: OK (390 of 390 tests passed, 0 test warnings)

![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)


いっそちょっと重くなるけどスレッド数×試行回数分のuniqueな乱数配列でもつくるか


修改建議：
>
該版本改用 `:placeholder-shown` 偽類，并非僅爲了在 placeholder 文本不顯示時以不同的方式移動 label - 這展示了此模型是如何工作的。


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-NoThreads: OK (394 of 397 tests passed, 0 test warnings)

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


`-C "dlite"` :-)?


Try **96**


There is at least one little typo in line 35:  `We this can see (...)`.


そうじゃないとseedかぶりが発生するからその計算ムダになるし


Why not `upto`?


@blond забыли про render написать. И это 1.2.0 получается должно было быть


Oh okay. I'm working in 1.8.9


No, we can't move cursor.


No, we should write to the buffer.


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2129 of 2135 tests passed, 5 test warnings)

 * 9 tests had: AUD diffs.
 * 9 tests had: MDD diffs.
 * 9 tests had: MTD diffs.
 * 4 tests had: ERR diffs.
 * 6 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


* 1000


:clap:


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2162 of 2168 tests passed, 5 test warnings)

 * 9 tests had: AUD diffs.
 * 9 tests had: MDD diffs.
 * 9 tests had: MTD diffs.
 * 4 tests had: ERR diffs.
 * 6 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


was a misunderstanding; my bad


release-5.x (lefticus) - i386-Windows-10-VisualStudio-12: OK (389 of 390 tests passed, 0 test warnings)

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


```bash
$ pip install Django==1.8.11
Collecting Django==1.8.11
  Could not find a version that satisfies the requirement Django==1.8.11 (from versions: 1.1.3, 1.1.4, 1.2, 1.2.1, 1.2.2, 1.2.3, 1.2.4, 1.2.5, 1.2.6, 1.2.7, 1.3, 1.3.1, 1.3.2, 1.3.3, 1.3.4, 1.3.5, 1.3.6, 1.3.7, 1.4, 1.4.1, 1.4.2, 1.4.3, 1.4.4, 1.4.5, 1.4.6, 1.4.7, 1.4.8, 1.4.9, 1.4.10, 1.4.11, 1.4.12, 1.4.13, 1.4.14, 1.4.15, 1.4.16, 1.4.17, 1.4.18, 1.4.19, 1.4.20, 1.4.21, 1.4.22, 1.5, 1.5.1, 1.5.2, 1.5.3, 1.5.4, 1.5.5, 1.5.6, 1.5.7, 1.5.8, 1.5.9, 1.5.10, 1.5.11, 1.5.12, 1.6, 1.6.1, 1.6.2, 1.6.3, 1.6.4, 1.6.5, 1.6.6, 1.6.7, 1.6.8, 1.6.9, 1.6.10, 1.6.11, 1.7, 1.7.1, 1.7.2, 1.7.3, 1.7.4, 1.7.5, 1.7.6, 1.7.7, 1.7.8, 1.7.9, 1.7.10, 1.7.11, 1.8a1, 1.8b1, 1.8b2, 1.8rc1, 1.8, 1.8.1, 1.8.2, 1.8.3, 1.8.4, 1.8.5, 1.8.6, 1.8.7, 1.8.8, 1.8.9, 1.8.10, 1.9a1, 1.9b1, 1.9rc1, 1.9rc2, 1.9, 1.9.1, 1.9.2, 1.9.3, 1.9.4)
No matching distribution found for Django==1.8.11
```


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (398 of 398 tests passed, 0 test warnings)

![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)


@aboch Ok, if we agree on ignoring the internal network's endpoint we can do in a different PR.

Currently in populateNetworkResources() we go through the endpoints to identify gateway ep.  Since you have added a new function getGatewayEndpoint() lets use it in populateNetworkResources() also so that the logic to identify the default gateway interface resides in one function.


????? WOT ????
Translate dear boy :-)

Dave

On 5 March 2016 at 18:12, Pedro Fernandes <notifications@github.com> wrote:

> would prefer that their involvement
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dpjudge/The-Book-of-Mullan_GIT/commit/f038fac4128a21d4113881e70faae698d64bcea7#commitcomment-16519297>
> .
>

--

David Philip Judge


Not correct.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18459](https://teamcity.mesosphere.io/viewLog.html?buildId=155125&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:04:20


Agree. We can't remove the check..


OK, got you, fixing now


You can't do it n this way, for 2 reason:
1 - This will build Snap for all the devices, if someday we want to build optcm for a non-qcom devices Snap will be built for all the devices, no matter if it's QCOM or not

2 - You need the overlay for the restart preview, otherwise when you take a picture the preview won't start anymore... See here: https://github.com/JDCTeam/android_device_samsung_jf-common/commit/1622f1a428d2259caf8f5779459d83c225cc0f15


These tests now fail, and I'm not sure whether they were correct to begin with. `detail::const_item` contains a single data member - `const int values[10]`. So `values` is copy constructible, but it is not copy assignable.

The question is what does `is_noncopyable` indicate? Is it supposed to be *not copy-constructible OR not copy-assignable*? In that case these two tests should be moved to the `yes` section.

Or is it supposed to be *not copy-constructible AND not copy-assignable*? If this is the case, then the logic for detecting noncopyable types needs to be modified. It currently iterates through data members and if any are `const`, says the type is noncopyable. But it should really be checking for both not copy constructible and not copy assignable for all data members.


NIJE GOTOVO


I wont update optcm12.1 anymore,though


That's why I'm telling you that so you can learn it an make it better :wink:
This was no criticism at all :D


release-5.x (lefticus) - Win64-Windows-10-VisualStudio-12: OK (389 of 390 tests passed, 0 test warnings)

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


adding screenshots


It took me some time to understand it because of the double negation. How about we try to make it clearer? We can do either of:

1. rename the left hand side part of the conditional to `nextIsSubmitDisabled`, which makes everything clear.
2. invert the meaning of `isSubmitDisabled()` so we don't have the negation there, rename it to `isSubmitEnabled` (invert the use in `render()`), and remove the negation on the left hand side of this conditional

Your choice :-)


I think that this line is only accidentally right. It will prevent deleting all individual permissions when what you actually want ti to avoid deletion of the permission associated with the Group itself.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18460](https://teamcity.mesosphere.io/viewLog.html?buildId=155126&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:05:08


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1022 of 1022 tests passed, 0 test warnings)

![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-30.52%25-green.svg)


LGTM.


Does it make sense to you that gccxml is putting the cv-qualifiers on the right of the array declarator in `decl_string`?


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (398 of 398 tests passed, 0 test warnings)

![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)


Pagina pricipal
Favicon
Arreglo error ExecJS::ProgramError in Welcome#index


Привет, спасибо за вклад. Может тут заюзать `DecimalField`?


Sorry, bad ping.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18462](https://teamcity.mesosphere.io/viewLog.html?buildId=155131&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:06:40


nit: the `&&` should be outside the call to `maybeHas`. I think this breaks resetting the date picker otherwise.


Who are you?


Missed whitespace before `(. Please, fix it everywhere.


I'm really not a big fan of this method: the long comment shows how complex it is, plus this should be the caller's responsibility to make this kind of construct. Even if it's strictly more work, do you mind doing this on the datepicker component itself, plus having a single key for each month / short month? Like:

`datepicker: {
  monthsFull: {
    january: "January",
  }
}
`

etc.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18461](https://teamcity.mesosphere.io/viewLog.html?buildId=155130&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:10:23


Then if you decided to switch to cm-13.0 as stable build and drop 12.1 then please switch to cm-13.0 branches as default for our GitHub like i do in our MM, this is really confusing..


cc @joehand


Can you put `this.refs.elem.getDOMNode()` in a local variable so that this line can fit on 100 chars, please?


修改建議：
>
這裏是 HTML 相關代碼：
>> ***請爲 Code Blocks 添加語法高亮***


Won't this break the styling of this item with the original style sheets?


i feel so discarded


exactly what is needed; thx


nit: `pickerOptions.min`


fine


@YinghuaXu2 test


Here and below, can you use a ternary conditional, instead, please?

`pickerOptions.max = this.props.maxDate ? new Date(this.props.maxDate) : false`


No problem, I was sure you would not like it :p
But I was lazy.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18463](https://teamcity.mesosphere.io/viewLog.html?buildId=155132&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:10:43


Test testing


Actually a java service-api enhancement is added too: add condition checker callers for request method checks (GET, PUT, POST, etc)


Are `minDate` and `maxDate` automatically reset when calling `this.pickadate.clear()`?


Could all of this be done in the `componentDidMount` instead?


Shouldn't m4v also be included?


Ops, i saw now this: https://github.com/JDCTeam/android_build/commit/6697cec511bf73ecbbd9c4d70761377bdd78dc90

Take a look at this, probably must be reverted again


New edit


As it turns out, this happens because `<regex>`
is not actually implemented in G++ 4.8 (!). See #256.


ok


> Check _if_ session is set


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Remove the general else if - only use it for related validation tasks, in this case if email_incorrect else if email_unknown


Probably a little, but I haven't measured.

On Sat, Mar 5, 2016, 18:40 dbaroncelli <notifications@github.com> wrote:

> Really Good! Is this going to increase the size of the Go library?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/golang/mobile/commit/6fca37c69e243b777b0910e0fa6a32125159f4fc#commitcomment-16519088>
> .
>


Recommend the following here:
``````
if ($GLOBALS['css_header'] == 'style_bootstrap.css') {
$GLOBALS['titleBarHeight'] = 70;
}
else {
$GLOBALS['titleBarHeight'] = 40;
}


You copied a very bad style from the old codebase, never use the $r variable again. You can access the user data using $user->id, $user->name, $user->email etc.


Remove the else and move the whole block to the success method, similar to the register controller


typo


Please use $user instead of $result


Status is an integer and not a string, check out the database shemes


быть до конца честным


Please use 'test' for the name, user and password - I think language is not needed for the testing


release-5.x (lefticus) - x86_64-MacOS-10.9-clang: OK (390 of 390 tests passed, 0 test warnings)

![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)


Ah, please can you take this code change out again?


That would be also something useful to measure, beside the performance benchmark, as the size of the Go library is one of the most sensitive issues in GoMobile at the moment.


I see.

I'll think of something.


I don't want to get in the mess of maintaining backups for people. This command gets run every time the daemon starts, that means potentially a new backup every time the computer boots. I don't want to be testing to see if the backup belongs to me and all that nonsense.

I will document and add a note to the install command confirmation informing the user what files are modified and encourage making their own backups


good catch, fixed it


sure, why not :) added


No problem, thanks for reverting!


Sounds good to me, especially considering it runs every time the daemon starts (did not foresee this).


I have no idea how i changed this file. I even didn't open any source file. Should I add whiteline back?


This securecookie rule applies only to cookies with a host name which consists of one character (e.g. "x") and cookies with one character ("c") in my understanding...
I hope that's not true...

/cc @EFForg


Subtracting total health should be unnecessary here now that health is just set as zero.


bu satırda description: un alt satırda olması lazım.


I know this isn't the place for asking but I didn't want to open an "issue" ticket on Github just for the sake of a question.

Is it possible for this exploit to load a payload from somewhere on the 3DS' NAND? I.e. Instead of relying on the SD card to load the .bin.

For example, the arm9 payload could be injected into the main 3DS partition via a tool then on boot if the 3DS fails to load the payload from the SD card (i.e. The SD card has been removed) it falls back to loading the payload from NAND where we injected it to.

Sound possible? I'm asking because some people don't like the idea of their 3DS being temporarily "bricked" without the SD card inserted (I don't care about it but a dude who I talk to does).


修改建議：
>
我更新了 label 選擇器以使轉移的 label 僅在其對應 input 標簽中 ```placeholder-shown``` 偽類可用時才被使用。

Notes:
>
match 固有“匹配”之意，但此處直譯不通，```input```  標簽與```placeholder-shown``` 偽類應爲“擁有”關係，所以應該意譯。


Yes. Sorry that was a typo


@Zhangjd 校對完成！


This commit shouldn't exist. If you want change something in previous commits, you should use interactive rebase.

```
git rebase -i master
```


Как я говорил, __proto__ - устаревшее свойство, лучше им не пользоваться. Лучше для таких целей использоваться метод getPrototypeOf


Thanks. Fixed.


Это рассказывал на уроке


`getDbId` please


ayy


The default behavior is not to subtract a dark `dark_subtraction=False`, as user suggests and the block of dark subtraction is executed only if user explicitly give `dark_subtraction = True`


Было бы хорошо проверить, что N в диапазоне от 1 до 6


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18464](https://teamcity.mesosphere.io/viewLog.html?buildId=155140&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:04:43


probably, but it doesn't hurt anything for now. I figure it'll get cleaned up when we refactor the whole class


Greeeeeeeeeeat


Молодец. Здесь именно prop нужно использовать, а не attr


I see. That is a better approach. I will test it next time I get a chance


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1580 of 1580 tests passed, 0 test warnings)

![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-70.82%25-green.svg)


release-5.x (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (390 of 390 tests passed, 0 test warnings)

![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)


Shouldn't this just be `MachopKind:` `SuperpowerName:`?


CURR_GID=$(stat -c%u ${APP_CONFIG})
should be
CURR_GID=$(stat -c%g ${APP_CONFIG})


You are mixin html and css

     "height:100%"


PathTools is (and has long been) upstream blead.

I had made a repo very temporarily in order to get review on some work for a fix, and later deleted it, lest people try to use it for pull requests, etc.  The idea that it would end up in META.json is like the nightmare scenario of that problem.

Changes was never in blead before, but was only in tsee's repo.  I agree that this is no good — when I've found this problem in the past, I've added Changes to blead.  In fact, I started to do so here, too, but I couldn't build a normal dist from blead's repo directly because it needs ppport.h to exist and … well, there are complications.

Still, I can add Changes to blead, and will do so now!


Ah, derp this breaks, travis should we move the bundle task away from prepublish?


Routing.

The webhooks from pagerduty land in exactly one place, and we then need to be able to route those back to the appropriate Sensu cluster.


below this, you can access styling of the image by its id "embed".

    #embed {
        height: 100%;
    }

I think you access styles by id using a `#`.


if you replaced file('../wp-content/uploads/h5p/content/'. $contentId .'/videos/'.$filename); with a call to the framework interfaces getH5Pdir or something to that affect it would make more sense and would drop the WordPress coupling.


@yogesh0720  it's just a good idea to always msyql_real_escape data going into a sql query as well if you're going to run a shell command you want to escape it.: http://php.net/manual/en/function.escapeshellcmd.php


No longer needed?


Pretty sure this isn't actually being used?


Doh, i definitely removed it in another copy on my machine.


 - __information__: [src/chaiscript_stdlib.cpp:0](../blob/71c67bc76342bb3cb13208b56246d91fac90fd08/src/chaiscript_stdlib.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 16 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/71c67bc76342bb3cb13208b56246d91fac90fd08/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/71c67bc76342bb3cb13208b56246d91fac90fd08/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/71c67bc76342bb3cb13208b56246d91fac90fd08/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/71c67bc76342bb3cb13208b56246d91fac90fd08/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/71c67bc76342bb3cb13208b56246d91fac90fd08/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/71c67bc76342bb3cb13208b56246d91fac90fd08/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 17 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/71c67bc76342bb3cb13208b56246d91fac90fd08/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/71c67bc76342bb3cb13208b56246d91fac90fd08/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/71c67bc76342bb3cb13208b56246d91fac90fd08/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/71c67bc76342bb3cb13208b56246d91fac90fd08/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/71c67bc76342bb3cb13208b56246d91fac90fd08/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/71c67bc76342bb3cb13208b56246d91fac90fd08/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


@yogesh0720  thanks, i concur.


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (398 of 398 tests passed, 0 test warnings)

![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)


Yaa this is clear, thanks!


Yep it is expected, since this is a SNAPSHOT build there is more fixes to come.


I'll just add ccjs as an dev dependency


Looks a bit better than mine I must admit =p
It was something I quickly needed and thought maybe it could be of use to you.


Ty p:  next up - storm peakes wyrms vs icesteppe rhinos p:  Well already scripted it earlier just got a few creases to iron out p:


Hi! I'm from phone. The main idea here is to check exact connection, not any.

E.g. you defined ejc-sql-set-db function:

(clomacs-defun ejc-sql-set-db
               ejc-sql.connect/set-db
               :lib-name "ejc-sql"
               :namespace ejc-sql.connect
               :doc "Define ejc-sql.connect/db var.")

You can use it in any (e.g. SQL file) buffer. And you want to use "ejc-sql" nrepl connection, not "my-project".


@yogesh0720  thanks.


From ClearSky because there isn't any way to do it (i think)


Fixes #12


Probably want to wrap this in a try like https://github.com/SupportClass/lfg-twitchapi/blob/master/extension.js#L155 for when twitch is twitchy :)


*shrugs*


So stoked, and looking forward to this :+1:


To have it more efficient:

CURR_SID=$(stat -c"%u:%g" ${APP_CONFIG})
if [ "$CURR_SID" != "$APP_UID:$APP_GID" ]
   chown -R $APP_UID:$APP_GID ${APP_CONFIG}
fi


Looking for a better way to do this!!!!


Basicaly only EXTMODULE_RCC_APB1Periph was added, rest is just formatting.


@lavanoid Good idea


Dear.

I try your sketch..
Still got an error:
C:\Users\DM_PC\Desktop\Mysensors 23-2-2015\Sketches Werkend\sketch_mar05a\sketch_mar05a.ino:8:29: fatal error: Adafruit_Sensor.h: No such file or directory

 #include <Adafruit_Sensor.h>

                             ^

compilation terminated.

Meerdere bibliotheken gevonden voor "MySensor.h"
Gebruikt: C:\Users\DM_PC\Documents\Arduino\libraries\MySensors
Niet gebruikt: C:\Program Files\Arduino\libraries\MySensors

Where do i go wrong


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (390 of 390 tests passed, 0 test warnings)

![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)


Monica Perez Hernandez


Add after this:

```java
if (kitNotes.isEmpty()) {
    node = getDefaults();
    save();
}
```

That will save the file on startup.


Agustín Walabonso Lara Romero


I'll update the project soon (8-9th march).


Issue #6. Still wip.


Ta todo fora da formatação https://github.com/guilhermependezza/be-mean-instagram-nodejs-exercises/blob/1cb7235c36922f7ec33043c8dcecd1e3eb6f6724/class-02/class-02-resolved-guilhermependezza-Guilherme%20Pendezza%20de%20Sousa.md

Fora q ja vi esse codigo EXATAMENTE igual


@yellows8 , is there any way to get homebrew on old 10.6.0-31U


This block only runs on  Oracle and PostgreSQL where `features.can_return_id_from_insert=True`. https://github.com/django/django/pull/6245 may fix it (tests in progress as of this writing).


Eu pedi para CRIAR UM MODULO!

Ou seja separar a lógica em outro arquivo e USAR ele no app


I would also probably up this limit since most people would do =>30 second poll interval and getting more then 5 in 30 seconds is not uncommon.

As an aside, I'd probably also force a minimum 30 second pollTime to keep twitch happy :)


Incluir faja lateral


Did you make this?


Can you comment on what the issues with NDB concurrency are, and how they manifest?

I'm using managed VMs (with the default of gunicorn with gevent) alongside NDB code, so now I'm a bit concerned with this comment I see here. I have issues with servers mysteriously going unresponsive on me (deadlocking?), as well as OOM issues in the past (https://code.google.com/p/googleappengine/issues/detail?id=9610 that I'm tiptoeing around now), so am wondering if they're related.

Is the correct fix to switch to gthread in my gunicorn config (as you've done recently in this file's git history), or is the fix to try using the multiprocessing runtime in this codebase? (I couldn't find anything explaining the charter/justification for this project, so don't know what problems it is supposed to solve.) Thanks!


esta bien


correcto


If external module is configured to PPM at the radio startup, then is sort of works, WDT gets reset, etc.. When external module is switched to PPM the TIM2 interrupt stops being executed. But the radio still works.

If radio is powerd on with external module switched off, then it loocks up pretty soon after the external module is configured:
```
extmoduleNoneStop
extmoduleNoneStart
end
wdtr
GC Use: 8257bytes
|||
```


thanks!


I ment the :: `this.twitch = nodecg.extensions['lfg-twitchapi'];` :)


seems like a new line is missing here, strange that this doesn't trigger a travis failure...


derp


addaura is evil


release-5.x (lefticus) - x86_64-MacOS-10.9-clang: OK (390 of 390 tests passed, 0 test warnings)

![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)


In PPM mode both bits TIM_SR_CC2IF and TIM_SR_UIF are always set at the same time. In fact all CC bits are set (which is normal since all compare registers are at zero). Why then we use both interrupts, if they happen at the same time anyway?

Example log from here in PPM mode:
```
#*8.sr: 1f
dier: 101
#*8.sr: 1f
dier: 101
#*8.sr: 1f
dier: 101
|#*8.sr: 1f
dier: 101
wdtr
#*8.sr: 1f
dier: 101
#*8.sr: 1f
dier: 101
```


Thanks!


release-5.x (lefticus) - i386-Windows-10-VisualStudio-14: OK (390 of 390 tests passed, 0 test warnings)

![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)


Updated changelog with the 2.3.1a build.


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (3048 of 3048 tests passed, 0 test warnings)

![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)


Launcher WoW , toute version !

Le code est partiellement commenter pour vous permettre de le modifier et l'adapter à votre serveur !

Mais merci de laisser le COpyright !

Cordalement,
Sanctarius


up to preference I guess. i can change it if you guys prefer to have two separate labels


User Stories Part 1 and 2


These images show the basic flow and UI of the application. When a user enters our URL, they are directed to the login page. If they are registered users they can login and go to the Home page otherwise; the user can register using the registration form. Once logged in the user can view all profiles and browse them one by one. Moreover, logged in user is also allowed to edit and delete their profile. Upon deletion of profile, they are logged out, and their details are removed from the database.


@asaldivar will need to add what file to open in the repo for them to begin writing their code. (Example: "Go back into the project repo, wherever you've decided to put it, and open up the file RockPaperScissors.js (inside the app/js directory)."


jduudjdj


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2130 of 2136 tests passed, 5 test warnings)

 * 9 tests had: AUD diffs.
 * 9 tests had: MDD diffs.
 * 9 tests had: MTD diffs.
 * 4 tests had: ERR diffs.
 * 6 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


Thank you bae <3


Ja hvorfor ikke kalde billederne noget godt, så det giver bedre SEO? :)


@gabriel please do not just do this.
1. we should have a PR for all code
2. even if flow upgrades you shouldn't bump this else you'll break our CI as our builds will likely not flow pass anymore


Linea 57:
Por favor, no discutir en el código.
Alternativas:
- manda nos un email
- abre una issue y digalo en el chat
- si el corto, dígalo directamente en el chat


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (390 of 390 tests passed, 0 test warnings)

![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)


line break is confusing here, it's short so don't break it up


There are a few issues with this copy function:
1. Given a board, which is an array of arrays, r iterates over the *indices*, not the values.  Also, you do not want to use a for in loop to iterate over arrays for a few reasons.  In particular, you are relying on the rows being copied in order but objects are inherently considered *unordered* collections.  See http://stackoverflow.com/questions/500504/why-is-using-for-in-with-array-iteration-such-a-bad-idea for more.  Consider using a standard for loop or forEach or map.
2. Even if this did give you the rows in order, you'd only be copying *references* to the row arrays.
3. The makeCopy function should really be encapsulated in your Model class.


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (2130 of 2136 tests passed, 5 test warnings)

 * 9 tests had: AUD diffs.
 * 9 tests had: MDD diffs.
 * 9 tests had: MTD diffs.
 * 4 tests had: ERR diffs.
 * 6 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


These folders seem not to be correct, should be kodi-addon-dev-kit/include/kodi instead include
I'll post an PR for this


5528-Generators-Trap-Neg-SandiaPV-Zenith (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (2129 of 2135 tests passed, 5 test warnings)

 * 9 tests had: AUD diffs.
 * 9 tests had: MDD diffs.
 * 9 tests had: MTD diffs.
 * 4 tests had: ERR diffs.
 * 6 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


@asaldivar this template is definitely making the prompt a bit repetitive. Can we move the "Create" and "Check to see.." steps below to the **Steps** section and give a brief overview of the requirements here. _For example "In this section, you'll be working to create the cards that will make up your memory game. Each card will have a value of "queen" or "king". Giving it a value means we can use comparison operators to check for equality. Let's get started!"_


Another function you should encapsulate.  Also, as I mentioned, this is equal to window or undefined in strict mode.


Don't use == true


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-NoThreads: OK (394 of 397 tests passed, 0 test warnings)

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


Pienso que inicializar un array así el menos legible que:
`var meses = new Array ("Enero", "Febrero", "Marzo", "Abril", "Mayo", "Junio", "Julio", "Agosto", "Septiembre", "Octubre", "Noviembre", "Diciembre");`


It doesn't but I think the `id` attribute (or alternatively now the `data-bel` attribute) is the best way to indicate that a child node is the same though.

When `.update()` is called I played around with traversing the `.childNodes` to match up the child ids and tried a cache system to lookup but consider this example:

```js
var list = bel`<ul>
  <li>one</li>
  <li>two</li>
</ul>`

// ... later ...

var newlist = bel`<ol>
  <li>two</li>
  <li>one</li>
</ol>`

list.update(newlist)
```

Both `li` tags should not get updated, only the order and parent tag. The only way for us to know those tags are the same is if the end user indicates they are the same. So I think setting an `id` is the best way to do that, at least for now.


@asaldivar let's list the steps from above here in more detail and avoid replicating what's going to be in the deliverable. It'd be better if we avoided giving the answer right after each step.


If the function returns an object with a row and column, then this return statement is problematic since it only returns a number.


What!?


Seems to me that .ruby-version *should* be tracked by git.  Yes? No?


Am I missing something? Line 32 adds c11. line 35 restores old cflags. And just in case  c99 is not working, too, line 44 restores old cflags again.


release-5.x (lefticus) - Win64-Windows-10-VisualStudio-14: OK (390 of 390 tests passed, 0 test warnings)

![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)


I couldn't figure out how to get to ExprKind::Path from the rcvr_expr. Any suggestions if this line is not a suitable replacement?

Also is it possible let me know a situation where span_suggestion would fail so I can test this path?


@coderyi
嗯嗯，就像有两个点 2 这样的……故意的😂😂😂


#5518-Multispeed-Coils-used-with-AirloopHVACUnitarySystem-do-not-control-air-flow-properly-at-Speed-=-1 (rraustad) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


No reason. We have no strict standards except that the code in the header file not be too long (or complex) and that the header file should compile by itself.
Initially the logger was header file only, and this is an artifact of that.


@danielstuart14, the phone in its current position blocks the GPS / mini-map and de-sync icons.


UK should work if you change "type": "Polygon" to "type": "MultiPolygon", as the United Kingdom has two other islands.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18411](https://teamcity.mesosphere.io/viewLog.html?buildId=154810&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:06:30


I just copied it, I AM A TERRIBLE PERSON! MUAHAHAH! >jk


好吧  故意 地就行  害我强迫症


Done


Really nice work @nlf! Seriously great... nfs and vboxsf kills me. Dlite might be the first and only OSX project to make Docker actually usable.

:100:!


As you wish.


Interesting. I agree. I also noticed the `mf_close` is a bad name (there's a `mf_close_file` that has a very different meaning), but `_close` is used elsewhere to convey the same meaning: cleanup and dealocate.


Tá bugado issae


Globals are usually UpperCammelCase https://github.com/TrinityCore/TrinityCore/blob/3.3.5/src/server/scripts/Northrend/IcecrownCitadel/instance_icecrown_citadel.cpp#L122


They could be swapped for clarity, although I'm fairly sure it does the same thing either way around


What is wrong with passing a node ID? Seems Brett reasonable to me..


S/Brett/pretty. Phones.


see #11
need a release of this version for android 6.0 testing purpose


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1579 of 1579 tests passed, 0 test warnings)

![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-70.85%25-green.svg)


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18412](https://teamcity.mesosphere.io/viewLog.html?buildId=154830&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:07:50


release-5.x (lefticus) - Win64-Windows-10-VisualStudio-14: OK (390 of 390 tests passed, 0 test warnings)

![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)


develop (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1579 of 1579 tests passed, 0 test warnings)

![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)


bài làm đầy đủ, nhưng hình như bài cuối multil circle em nhớ là chưa học vòng for mà ? mình phải sử dụng kiến thức đã học từ vòng while chứ


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18413](https://teamcity.mesosphere.io/viewLog.html?buildId=154833&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:05:41


새벽 2시에는 주무셔야죠 ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ


added name to the error logged to std::cerr


Hmm, and I find the other check clearer ! However, the m_filepath check is redundant. Removing it.


Add "possible" in front of "threshold values ...".


We have allocated an ofstream object. We are simply deleting the object, not the file. Did I misunderstand your comment?


Done


Функция клонирования в данном случае не нужно глубокая, подошла бы простая и понятная:
```JavaScript
function cloneInterface(anInterface) {
  var clone = {};
  for (var key in anInterface) {
    clone[key] = anInterface[key];
  }
  return clone;
}
```


![image](https://cloud.githubusercontent.com/assets/16970990/13545089/701a5e6c-e2b3-11e5-9f4e-9566527847c3.png)
xuất sắc :dancer:


Yeah. Sometimes I am just paranoid :). I'll remove the check.


release-5.x (lefticus) - i386-Windows-10-VisualStudio-12: OK (389 of 390 tests passed, 0 test warnings)

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


héhé, pas mauvais. Mais je pense que tu te rends compte de la redondance de code qu'il y a. essaye de refactorer ton en enlevant les duplications. Mais bien joué.


Still pretty buggy, but will keep working on it


Duplicate. See Line 417


Решение рабочее, но переусложненное, как упростить, мы рассмотрим в следующей лабе


It is needed and tested now, but you don't need to special case it for `Integer`, `Rational` and `Complex`. All `Number` types should behave the same.


if abs(pos()) < 1:
       break
anh dùng chiêu abs(pos()) < 1 rất thuần thục. :)) nhưng bài mutil circle đâu mất rồi


pretty broken indeed

![mc-classic-anvil-save-fail](https://cloud.githubusercontent.com/assets/2346494/13545115/c75329f2-e281-11e5-8f75-d4b4596500ab.png)

I'm not sure what part is broken.

I think it might be worth having a look at what format other classic server use.

Maybe the level.dat format is not that much relevant and we should be using something else ;) (since it's based on java5 and https://github.com/node-modules/java.io/pull/17 )


develop (energy-plus) - Win64-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-39.12%25-red.svg)


hmmm, maybe we can change to the left side, but it'll block the clock, anyway.
So, idk... Also, we can set it in the middle of the screen, but I think this "GTA V like" position is good...
At the middle of the screen, it takes a lot of the "useful area".

I'll take a look in other possibilities.


bài làm rất tốt nhưng phần multil circle cuối cùng chưa được phép sử dụng vòng for, mình nên sửa lại chị nhé


Added for linux/Mac. Not sure if it works for Windows. Leaving that as it is.


I think I'm getting it, here's what I suggest - we can make another property of the strut class called TransformedAndAlignedCutPlane that returns the transformed cut plane after doing your rotation to it, this will be a public method on the strut class and it can take a vector or degrees etc. Does that make sense? If so I implement this and you can make any changes if I'm misunderstanding the exact rotation

> On Mar 4, 2016, at 8:36 PM, coten <notifications@github.com> wrote:
>
> Ok, this is what I mean:
>
>
> This same problem -- which I fixed for planes and coordinate systems -- is the reason that so many of the tool paths generated at the VT summit failed. The problem was that we neglected to intelligently define the X-Axis of each target frame or, in other words, the X-Axis of the "transformed cut plane" or "transformed coordinate systems." Note that these are relative such that target, work-object, and tool frames always align following a movement instruction (and therefore tend towards alignment during execution of that instruction). In our case, this means that when the robot executes a movement, it transforms the work-object and target frames so to match the tool frame and, if any of these are misaligned, then the robot will appear to strain while executing that movement. When this happens it looks like the robot is attempting to flip-over, rear-around, over-reach, over-turn, etc. So it's super important, haha.
>
>
> The fix, in brief, rotates each Cut plane or coordinate system about its Z-Axis such that the angular difference between its own X-Axis and that of the work-object (and, here, World) is reduced to zero. This means that each target aligns with the work-object and that the robot does the least work, so to speak, in order to reach that target.
>
>
> I tried implementing this bit of code in the same way as I had for planes and coordinate systems, but it seems like this just doesn't play-well with the node and strut classes. It makes sense, then, to implement this on the newly created "transformed cut plane" or "transformed coordinate system" associated with each strut.
>
>
>
> ________________________________
> From: Michael Kirschner <notifications@github.com>
> Sent: Friday, March 4, 2016 4:58 PM
> To: coten/TORO
> Cc: Nicholas Cote
> Subject: Re: [TORO] Working through integration; see a couple issues with Node class construction (d8fa402)
>
>
> Also- agreed, we should put as much logic as we can into the classes instead of separate nodes
>
> -
> Reply to this email directly or view it on GitHub<https://github.com/coten/TORO/commit/d8fa4027ed8eaa5943f0a3f954b06d59f94ffe40#commitcomment-16514593>.
> —
> Reply to this email directly or view it on GitHub.
>


Este commit me deu uma idéia, vou colocar o LaTex original aqui no repositório.


Why not just suggest to remove an extension rather than this workaround?


MOTIVATION:
The Illumina Infinium 450 k DNA Methylation Beadchip is a prime candidate technology for Epigenome-Wide Association Studies (EWAS). However, a difficulty associated with these beadarrays is that probes come in two different designs, characterized by widely different DNA methylation distributions and dynamic range, which may bias downstream analyses. A key statistical issue is therefore how best to adjust for the two different probe designs.
RESULTS:
Here we propose a novel model-based intra-array normalization strategy for 450 k data, called BMIQ (Beta MIxture Quantile dilation), to adjust the beta-values of type2 design probes into a statistical distribution characteristic of type1 probes. The strategy involves application of a three-state beta-mixture model to assign probes to methylation states, subsequent transformation of probabilities into quantiles and finally a methylation-dependent dilation transformation to preserve the monotonicity and continuity of the data. We validate our method on cell-line data, fresh frozen and paraffin-embedded tumour tissue samples and demonstrate that BMIQ compares favourably with two competing methods. Specifically, we show that BMIQ improves the robustness of the normalization procedure, reduces the technical variation and bias of type2 probe values and successfully eliminates the type1 enrichment bias caused by the lower dynamic range of type2 probes. BMIQ will be useful as a preprocessing step for any study using the Illumina Infinium 450 k platform.


<html>
</html>


release-5.x (lefticus) - Win64-Windows-10-VisualStudio-12: OK (389 of 390 tests passed, 0 test warnings)

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


develop (energy-plus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1593 of 1593 tests passed, 0 test warnings)

![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)


We're going to end up bringing back quite a few of these: touch, rm, mkdir, move, etc. For when we replace them all in `motion`


good :+1:


# welcome


https://github.com/OpenClassic/OpenClassic/blob/298815eae5c26bfa2a6295f865cb05d0cf6da38c/src/main/java/org/spacehq/openclassic/game/io/OpenClassicLevelFormat.java#L170 : an other format


https://github.com/dreadiscool/PowerBlock/blob/master/gg/mc/World.java#L94 : yet an other format


**jj  ##**


Okay, so at PMM's suggestion I've used hw/arm/raspi.c as a reference for introducing a SOC model amongst other guidance. I've reworked this section to largely mirror what's done in the raspi. Hopefully that's an improvement.


ghê gớm quá :))
![image](https://cloud.githubusercontent.com/assets/16970990/13545150/0b2cedec-e2b5-11e5-91af-ce5609b17873.png)


There is just too many rogue addons or extensions that insert nasty adservers and/or trackers, which should be dealt with via removing them, and not being dealt via an adblocker.


develop (energy-plus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Tại sao mình phải dùng đến 3 vòng while mà không dùng một phát  hàm turtle.circle(...) để vẽ luôn hả anh ?


Well, I added #defines,, then I reworked everything and I've thrown them away. I've now got two int arrays that act as maps for the timer and uart IRQs:

```
...
static const int uart_irqs[] = { 9, 32, 33, 34, 10 };
static const int timer_irqs[] = { 16, 17, 18, 35, 36, 37, 38, 39, };
...
```

Hopefully that's helpful?


https://www.classicube.net/server/host/ good list


Lel


https://github.com/Hetal728/MCGalaxy/tree/master/Levels/IO : plenty formats


bài cuối multil circle chưa học for nên em chỉ được dùng vòng while thôi nhé. kiểm tra lại đi


In fact, based on PMM's suggestion I've reworked this entirely, mainly to respect enabling/disabling coinciding with configuration changes.


bài multil circle 2 đệp quá ạ :)


https://github.com/Hetal728/MCGalaxy/blob/master/Levels/IO/DatFile.cs doesn't even try to implement save for level.dat


nice :)


good :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18414](https://teamcity.mesosphere.io/viewLog.html?buildId=154847&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:04:48


http://wiki.vg/MCLevel_Format maybe this makes sense


Would it be possible to integrate the GPS / Mini-map into the phone's buttons and screen?


Anyway. I think we should either:
* choose a format from that list up there ^
* just save the world buffer straight in a file and be done with it


Are you happy if I address this in the cover letter? I addressed it there when I posted the first two versions of the series.


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28330/testReport/) for this commit**


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18415](https://teamcity.mesosphere.io/viewLog.html?buildId=154848&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:05:31


@vitaliy-k if you don't mind I have implemented your fix into XGP3.


#5518-Multispeed-Coils-used-with-AirloopHVACUnitarySystem-do-not-control-air-flow-properly-at-Speed-=-1 (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1573 of 1574 tests passed, 0 test warnings)

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


it's possible, but idk if we should, it'll be a little buggy as the GUI has a timer to close itself


why?


It appears that 1024*1024 (2^20) is a magic number; the size of a record batch. Should we have a constant for this value? Else, how would the average user know to use this particular size?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18416](https://teamcity.mesosphere.io/viewLog.html?buildId=154849&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 245 Build time: 00:04:54


Get


both packets 0x3C and 0x74 is using the same order, it works on CC but strangely you said it doesn't work on old EC. I'm still trying to fix this packet to make it works on all clients (the correct way is use the same order on both CC/EC clients as RunUO does and works perfectly) but still not working

but don't worry, if something goes wrong I'll add back this workaround and mark it as resolved, since these buy packets is only giving me headaches instead an working price list :P


> 03:32 <@US200|Offline> Not really.
> 03:32 < AndrewPH> classicworld format is the greatest file format, and is supported by many softwares. but it's not really a standard ^^^yet
> 03:33 <@US200|Offline> .lvl files (MCLawl/MCForge/MCGalaxy etc), .fcm files (fCraft/800craft etc), .cw files (ClassicalSharp and ClassiCube)
> 03:33 < Skeltal> .lvl and try again
> 03:33 <@US200|Offline> Those would be the most common map files.
> 03:33 < Skeltal> Those are inside a background texture?
> 03:34 <+_123DMWM> hwat iz efkraft/eightyzerocraft? zounds outdayted
> 03:35 <@US200|Offline> The format originates from them, and has not been changed by the various forks.
> 03:44 < rom1504> we tried using the original level.dat files and... it wasn't a great idea :d (loading was ok, but saving not quite)
> 03:45 <@US200|Offline> Please nuuuuu not the .dat files.
>


Oops, typo. CAMELLIA *


As you know, I tend to prefer:

```
const bool canonicalOrient = true;
RDDepict::computeCoords(mol, NULL, canonicalOrient);
```

How does the canonicalOrientation work?  My old stupid way was to "canonical order" the atoms and bonds by running them through the smiles algorithm and assuming the depiction algorithm is deterministic..  Worked pretty well, actually.


singleton pattern相當於static variable以class的方式實作。直接寫static variable也是可以，可是在不同object file中使用時，要先宣告成extern，並寫if去判定是否已初始化等等會讓code可讀性降低(不過在C語言中沒有class，所以都是直接利用static variable)。


You could do

```
git commit --amend --author='Timothy Gu <timothygu99@gmail.com>'
```

so that Git tracks the authorship.


Note that the char array returned by c_str( ) is passed to connect where it is passed to the parse function that uses strtok to parse. As a result, the buffer has zeros written into it. Will this actually work other than the first time? Should we copy the string somewhere before we allow strtok to party on it? Plus, the altered string is then used below in an error message...


Commit title is wrong, this is a tiny fix for the stylesheet to prevent long unbroken words from breaking out of the div.


We are connected for connection i of n to some randomly selected drill bit. Is it worth saying the connection number? The drillbit to which we connected? Else, the user will just see a sequence of "Connected!" lines.


This just removed a block that, in addition to setting the view, also
cleared the query if the "favorites" tab was selected. This was because
there wasn't a way to search bookmarked images at the time.

I agree however that TabPane is a better name.

To answer the main point of what I think you are asking, the tab component
is stateless and reusable. It just needs a list for the tabs an active tab
and an eventHandler

On Fri, Mar 4, 2016, 2:34 PM Andrew Lenards <notifications@github.com>
wrote:

> It looks like this commit move the target of the "tab link" rendering into
> TabLinks. Am I right about that?
>
> I have unfortunately be preoccupied with QA test issues today, so I
> haven't end a chance to go back to the usage and definition of TabLinks.
>
> But, if I *am* right, then *"TabLinks"* may not be the best name for the
> component. If we're encapsulating a "tab link bar with the *target* pane"
> for display - we might want to consider another name. I think we definitely
> want a "TabPane" sort of components like this and have it as reusable
> because there several places we'd leverage it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iPlantCollaborativeOpenSource/troposphere/commit/d60c183f70d716d2508052f8afbbf850f827970c#commitcomment-16511122>
> .
>


Looks great Steve!


Probably in the next few weeks, as we move forward on protobuf.


does the commented out XML need to be here?


It seems we iterate over n connections. For each, we open the connection, start m queries, wait for m queries and close. This does not seem like a test of the n open connections, it really just opens one connection n times. Or, is that the point? There are no comments to describe the goal...


The logging fix was to handle the multithreaded case, but this test is all in a single thread.


:D Progress woohoo! Shame it failed to compile on the jenkins. :P


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (398 of 398 tests passed, 0 test warnings)

![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)


Por favor, indenta el código, sera mas legible.


Nice! I didn't know about this.


В този файл само поправих "две" на "два"


missing spaces, this is just unreadable


use my cm-13.0 branch on vendor


Hogan


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18466](https://teamcity.mesosphere.io/viewLog.html?buildId=155163&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:04:43


Whoops! Thanks for underscoring that.


BTW any idea how much extra time this block of regex adds to page load?


hardcoded constant here isn't better


Yes, IMHO it should be a **rule** of SCM functions (32 vars), giving undefined behaviour if you don't follow the rule.  First because it won't have different behavior between missions and non-missions, and second because it'll be _slow_ to copy 1024 locals.


:scream_cat: this causes an infinite loop! Was that your intention? If not you need to make a variable and increment the counter.


You are sending to the client the **function** date directly. Instead of **calling** it to send the date.


@asaldivar new repo for hosting code for memory game: https://github.com/ga-students/wdi-fundamentals-memorygame.


thanks :+1:


Looks like you have the idea to create a variable for the counter, however, you need to use the `i` in the loop condition. Line #52.


space


this line was in the original code, I didn't even saw it in sniffs, i would like to leave it be '-'


release-5.x (lefticus) - i386-Windows-10-VisualStudio-12: OK (389 of 390 tests passed, 0 test warnings)

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


I've got your password now!


Добавлю, я тогда не успел)


How is the game to store the max and min of the point values of all possible moves?  If it did, what would be the point of this function?  Instead use a variable to hold the value of the running min or max such as currentBest.  Then compare result to currentBest.


Yep.

--
Dan Kohn <mailto:dan@dankohn.com>
tel:+1-415-233-1000

> On Mar 5, 2016, at 14:58, David A. Wheeler <notifications@github.com> wrote:
>
> Seems to me that .ruby-version should be tracked by git. Yes? No?
>
> —
> Reply to this email directly or view it on GitHub.
>


`==` is used for comparison, where as `=` is used for assignment.


Oh no! There's a floating `for()` statement!


Turns out this change was unnecessary.  In the future, if you wait for me to notify you that I am done with a review before making code changes, it may save some work.  Github doesn't have a good way to indicate "starting review" and "ending a review".


Don't forget the semicolons!


That sounds like a healthy alternative.


Don't forget the semicolon!


You should have a makeCopy function (of the model) instead since you also want to keep track of things like the player's turn.  Encapsulate this function in your Model class.


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (398 of 398 tests passed, 0 test warnings)

![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)


@asaldivar let's do the same thing here as I mentioned for the previous unit. Keep this short and explanatory and move the details for the steps to the next section.


подумал, что последовательность этих цифр должна как-то называться, либо позиции единиц, дальше гуглил)


Looks like there's an extra `</html>` tag here!


No problem, I'm going to drop it in the process of applying this PR to the `0.15` branch.


nice


I confirm that this commit fixes #763104, thanks.


Это делает изображение множества Мандельброта пропорциональным.


Tags are typically in written in lowercase. 🤔 Hmm, most things in programming often are. Make sure you lowercase the start tag as well!


Won't this break the styling of this item with the original style sheets?


ack


@asaldivar Let's make sure that the deliverable look is consistent among unit projects.


`Choice` should be written in lowercase. This will actually always result in `false` since `Choice` will return `undefined`.

```js
if (`undefined` == "HIDE) {
  // this will never happen
} else {
  // it will always go here
}
```


oops, that was me. :P


@asaldivar add instructions to add and commit their code their repo.


Text won't show up in the `<head>`, you should move it the `<body>`. The head is for things like `favicons` and titles. Things that don't really show up.


A better approach to the CALL problem is how LCS/VCS does it, informing how many vars is used by the function on the command parameters, and saving only those, hundreds of times faster :)


release-5.x (lefticus) - Win64-Windows-10-VisualStudio-12: OK (389 of 390 tests passed, 0 test warnings)

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


@Martinusbe thanks mate I was waiting for this. I'll try with yours. Cheers


@asaldivar might be good to incorporate some of the coding advice given here: http://fundamentals.generalassemb.ly/04_chapter/09_assessment.html


get me informed once u test it,so i will be able to revert that REVERT commit


pire commit ever


@josevalim what is the reason for just taking `["Elixir", "AppNamespace"]`? This stops namespaces controllers, e.g. Admin from automatically picking up its own layout view. I know it's possible to change the layout in the plug pipeline but it seems inelegant to set the layout twice in the pipeline.

Was this a design decision, i.e. to enforce separation of concerns e.g. sibling apps in an umbrella?


PLACEHOLDER


Nem voltam benne biztos, hogy végfelhasználói, de igazad van.


Won't changing class like this mean it won't be supported in the original styling sheets???


Won't changing this class mean it won't be supported in the original styling sheets?


merge


Did you want a `{`here? If you don't have the `{`, `}`, it only assumes the next line is part of the `if` statement.

So in this case...

```js
if(choice=="blue")
  var choice2 = prompt("You have two one-way tickets. One to Saturn. One to Mars. Which do you take?");
```

is the same as

```js
if(choice=="blue") {
  var choice2 = prompt("You have two one-way tickets. One to Saturn. One to Mars. Which do you take?");
}
```

We typically always put the `{`, `}` since it's more clear for the reader.


remove commented out code


trebuia facut un .gitignore inainte de asta, ca sa nu avem merge conflict de la fisierele .xml


@asaldivar we're trying to move away from w3 schools. Can you reference https://developer.mozilla.org/en-US/docs/Web/API/Window/alert instead?


Not sure if this is intentional. I see two `if` statements back to back. Were you trying to do if `choice` is blue and `choice2` is mars? If so you can use the `&&`.

```js
// if choice is blue and choice2 is mars
if (choice == "blue" && choice2 == "mars") {
  // do something
}
```


This code should be kept outside this file, in main.qml as it is relevant to context in main.qml. Certainly nothing outside main.qml ought to reference `_votingSystem` directly.


а где открывающий тег <section> ?


Lots of white space here!


Should be >= 1.25 now.


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Yeah, definitely. I originally used W3 cus MDN can be awkward to read for beginning developers, but you're right, it's a great standard. I'll reference MDN.


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Won't using this class only mean it won't be supported in the original styling sheets???


Just delete your pocketmine.yml file. I didn't tell you to change anything.


@asaldivar how are students checking/testing whether or not their code works? Can you include instructions on how to run .js file? Or do you think we can stick to repli.it for the JS unit homework? We haven't taught students yet how to use text editors.


Awesome! Thanks.


Could delete this line, it's from Taddy's code


Why are you parsing this. Wouldn't it just make more sense to send/collect a GET parameter?


This comment is a bit cryptic.


Good luck!


Much cleaner if just use a get parameter.


I'll include instructions on how to run the `.js` in the memorygame repo.


Doesn't seem to be working on 1.9 spigot. Using the experimental builds off enginehub.


>  31 additions and 167 deletions.

Nice!


N/E/W/S doesn't really make much sense in the context of facedir nodes, you should use left/right/front/back.


Many thanks, it's really appreciated :) :+1: