This commit does not work for me. It constantly loops over the array of all packages until a timeout occurs.

```
[2016-02-17T09:30:39+00:00] INFO: Processing zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] action install (net::default line 1)
[2016-02-17T09:30:39+00:00] DEBUG: Providers for generic zypper_package resource enabled on node include: [Chef::Provider::Package::Zypper]
[2016-02-17T09:30:39+00:00] DEBUG: Provider for action install on resource zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is Chef::Provider::Package::Zypper
[2016-02-17T09:30:39+00:00] DEBUG: ethtool
[2016-02-17T09:30:39+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:40+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 4.0-5.2
[2016-02-17T09:30:40+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:40+00:00] DEBUG: tcpdump
[2016-02-17T09:30:40+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:42+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 4.5.1-2.2
[2016-02-17T09:30:42+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:42+00:00] DEBUG: nmap
[2016-02-17T09:30:42+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:43+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 6.47-9.10
[2016-02-17T09:30:43+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:43+00:00] DEBUG: telnet
[2016-02-17T09:30:43+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:44+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 1.2-167.1
[2016-02-17T09:30:44+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:44+00:00] DEBUG: iftop
[2016-02-17T09:30:44+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:45+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 0.99.4-7.2
[2016-02-17T09:30:45+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:45+00:00] DEBUG: whois
[2016-02-17T09:30:45+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:47+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 5.2.10-4.2
[2016-02-17T09:30:47+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:47+00:00] DEBUG: wget
[2016-02-17T09:30:47+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:48+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 1.14-2.2
[2016-02-17T09:30:48+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:48+00:00] DEBUG: iperf
[2016-02-17T09:30:48+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:49+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 3.0.11-3.2
[2016-02-17T09:30:49+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:49+00:00] DEBUG: ipset
[2016-02-17T09:30:49+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:51+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 6.25.1-4.17
[2016-02-17T09:30:51+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:51+00:00] DEBUG: ethtool
[2016-02-17T09:30:51+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:52+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 4.0-5.2
[2016-02-17T09:30:52+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:52+00:00] DEBUG: tcpdump
[2016-02-17T09:30:52+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:53+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 4.5.1-2.2
[2016-02-17T09:30:53+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:53+00:00] DEBUG: nmap
[2016-02-17T09:30:53+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:55+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 6.47-9.10
[2016-02-17T09:30:55+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:55+00:00] DEBUG: telnet
[2016-02-17T09:30:55+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:56+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 1.2-167.1
[2016-02-17T09:30:56+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:56+00:00] DEBUG: iftop
[2016-02-17T09:30:56+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:57+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 0.99.4-7.2
[2016-02-17T09:30:57+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:57+00:00] DEBUG: whois
[2016-02-17T09:30:57+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:30:59+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 5.2.10-4.2
[2016-02-17T09:30:59+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:30:59+00:00] DEBUG: wget
[2016-02-17T09:30:59+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:31:00+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 1.14-2.2
[2016-02-17T09:31:00+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:31:00+00:00] DEBUG: iperf
[2016-02-17T09:31:00+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:31:01+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 3.0.11-3.2
[2016-02-17T09:31:01+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:31:01+00:00] DEBUG: ipset
[2016-02-17T09:31:01+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:31:02+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 6.25.1-4.17
[2016-02-17T09:31:02+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
[2016-02-17T09:31:02+00:00] DEBUG: ethtool
[2016-02-17T09:31:02+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] checking zypper
[2016-02-17T09:31:04+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] version 4.0-5.2
[2016-02-17T09:31:04+00:00] DEBUG: zypper_package[ethtool, tcpdump, nmap, telnet, iftop, whois, wget, iperf, ipset] is installed
... until timeout...
```


Please revert the line above.


Does kScrollWindowAdd really get any type of optional parameter? And not specifically an integer?


I don't know much about this part.
@tsauerwein @ginold OK for you?


All `NULL` cases in `TestJsonOperators` were broken. Actually better approach is to comment them in previous commit, and uncomment with fix in a very next commit. But i forgot to do that.


@Ismailrizvi  : now please remove the dummy data and keep only 1 single row in html table
rest all will be replaced by jsp code to fetch data from db


Proba


Thanks a lot for the link!


oh yeah! great job, Amina! Many many thanks!
please have a look at issue #86, should help to check the tissue annotation


@Ismailrizvi  : please remove admin login from dashboard too it will be also be accessible only after login


Not a big deal since the layout/placement of this tool will probably evolve, but perhaps you should add the ``translate`` directive to the interface strings anyway?


keep only one row here too instead of dummy data


CLI not working with images


nice!


Is the ``string`` type actually needed?


Thanks for the fix Ben


I've managed to create images that can be flashed from the stock web interface. I'll upload them later today!


@ToNict : Thank you for the bug report!

Because of merging separate modules in one `jquery.jqgrid.src.js` the problem will be not exist in `jquery.jqgrid.src.js`, but it will a problem in case of loading modules separately with respect of [RequireJS](http://requirejs.org/) for example. I posted the corresponding bug fix [here](https://github.com/free-jqgrid/jqGrid/commit/13847fee48af9216c55d9ab5e4fb85e4df49e08a).

Please post in the future the bug reports as separate issue here: https://github.com/free-jqgrid/jqGrid/issues


@tchaikov With regards to the missing atexit() I'm not sure where the problem is. I have seen that on regular completion ceph-mon does write the exit strings and removes the PID file. ceph-osd's that receive signals do not show the atexit() strings and leave the PID file in place.I don't get a core. So somewhere inbetween the program terminates. I asked at hackers@freebsd.org, but no replies yet.
To work around this, there are two options: 1) live with it and document this feature, and repair the fallout from that; 2) see if we can attach it to the signal catcher that runs to cleanup when a program terminates.


This gives me compile errors in VS2013.

```
2>c:\users\tthomas2\sourcetree\glfw-tr\glfw\src\wgl_context.h(119): error C2061: syntax error : identifier 'GLboolean'
2>c:\users\tthomas2\sourcetree\glfw-tr\glfw\src\wgl_context.h(120): error C2061: syntax error : identifier 'ARB_multisample'
2>c:\users\tthomas2\sourcetree\glfw-tr\glfw\src\wgl_context.h(120): error C2059: syntax error : ';'
2>c:\users\tthomas2\sourcetree\glfw-tr\glfw\src\wgl_context.h(121): error C2061: syntax error : identifier 'ARB_framebuffer_sRGB'
2>c:\users\tthomas2\sourcetree\glfw-tr\glfw\src\wgl_context.h(121): error C2059: syntax error : ';'
2>c:\users\tthomas2\sourcetree\glfw-tr\glfw\src\wgl_context.h(122): error C2061: syntax error : identifier 'EXT_framebuffer_sRGB'
2>c:\users\tthomas2\sourcetree\glfw-tr\glfw\src\wgl_context.h(122): error C2059: syntax error : ';'
```


@alexanderdean can you confirm this is correct? I don't quite understand what is "15.05".


@AGritsevich But you did change it. We agreed to leave the better code after us.


Just out of curiosity, can you post the output of
`# hexdump -C -n 1600 /dev/mtd1`


@adamschwartz Done! https://github.com/phippuuuu/inactivity-message/releases/tag/v1.0.1


jstl....感觉好老的技术了，还是freemarker好用


Does this really work?
It's possible to pass angular expressions to a JS function?


Wrong use of std:cerr.


I can get past these by replacing with `GLFWbool` - but I run into other errors after that.


and an an → and as an


So what happens if I write here...


Once I get past the `GLboolean` errors I get a lot of other errors:

![image](https://cloud.githubusercontent.com/assets/192418/13105825/5914fa98-d565-11e5-83db-4afea70033ff.png)


web项目的gradle build.gradle中需要加入
apply plugin:'war'
这样gradle eclipse 才会生成能正常运行的web项目


Add scenario that tests URI that has fragment only, no path.


Not made a token for new card convenience, should definitely do this!


aaaaaaaaa


If you want to print only numbers in the stdout, remove all other prints to the stdout, do not move them to cerr. That's a bad thing, blame on you. :rage:

lol.


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/40ccd6998198ff5fa4d2dcab178683fbbe629d25


merging graydon/stellar-core/bug-969-catchup-in-progress-when-manually-triggered = 40ccd699 into auto


graydon/stellar-core/bug-969-catchup-in-progress-when-manually-triggered = 40ccd699 merged ok, testing candidate = 72a9800d


Actually, it is my personal build folder. I Have worked on the CMakeUnix.cmake in another commit in which the Build is removed from .gitignore


I see, they're enums.


You are right I can Change it to CMakeWin32 spec


[![Luxembourg Gardens at Twilight](http://api.artsmia.org/images/77/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=77)


[![Moorish Courtyard](http://api.artsmia.org/images/121826/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=121826)


Looks good.


set Column Type similar to Table Type.
Name is the fully qualified name
Column_Name is the unqualified name


Looks like that was missing already; fixed now.


I guess it is but i can test it without. (Actually the Module keyword of add_library should do it)


Badges #5


No, I ll remove it


Best changeset!


Super bien!!!
Podrías simplificarlo más si dejas filterEnv con un sólo argumento:
```
filterEnv func(keyval []string)
```
ya que al envsMap se tiene acceso directo y se puede usar sin problemas dentro de la función anónima


Ah this commit takes care of gamejolt/issue-tracker#585 I guess


5479_OAandPLANTEquipList (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2131 of 2131 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)


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


I added a semicolon here to fix this line.


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


Very nice,

I just committed this to the codebase (it will also be in the 4.3.0 release).

thanks,
-brady


@Nadahar so what we should do with regards to your PR? Are you going to make changes in the plugins?


Add back the `readonly` modifier to these two arrays


I this this should move into a separate project completely. Leaving Spring Integration Kafka just with the channel adapters.


No it is not working when building from another folder (see [CMAKE_INCLUDE_CURRENT_DIR](https://cmake.org/cmake/help/v3.1/variable/CMAKE_INC) )

I guess there is the possiblity to add :
```
set(CMAKE_INCLUDE_CURRENT_DIR ON)
```


Thanks for this!!


Actually may i suggest that zoom is infinite and the spectator's viewbasex and viewbasey increases as he/she zooms out?


Add doc for growDetectedFootprints.  Should it be defaulted to 0/False?


I don't think this follows the naming standards: http://developer.lsst.io/en/latest/coding/python_style_guide.html#naming-conventions
(e.g. DETECTED -> detectedMask)


Minor nit-pick: double quotes about "sky" are used above.


"e" is not used


Should this log message indicate that not filter was found because it is a sky object?


I think in general it is harmful to have partially initialized `AutoCOMPtr` objects floating around. Next time when we try to initialize them, the pointers have reference to older objects. Hence I didn't check for exception type here. Do you think of any scenarios why we would want this to reset only in OOS/OOM?


5479_OAandPLANTEquipList (Myoldmopar) - 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)


The must overlap the same target so if mapped they are on the same chromosome for sure.


@KelseyDH you can use `link_to`'s block option to break up long lines and avoid concatenating html


typo


> kekekekeekekekekekek i added a line to my readme and it only took an hour


Went ahead and removed it as all the access check is done in the the controller.


Transactional use default to_name if missing #7


This commit breaks compilation on linux with std::min error. Either you should cast both its params to the same type or to explicitly mention argument. GCC seems not to be able to deduct two values where markov_chain_size is unsigned int and fixed_new_word.size() is size_t


好深远的typo


I still have no clue how this was working before. Looking through genGraphs, there's absolutely no reason this should have worked..

I don't really have anything constructive to say here.. just re-expressing my surprise


Electric-Service-Refactor (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (2076 of 2084 tests passed, 543 test warnings)

 * 547 tests had: AUD diffs.
 * 477 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 3 tests had: EIO diffs.
 * 1 test had: SHD diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

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


`set_property(TARGET mod_websocket PROPERTY C_STANDARD 11)`
that should be true

`set_property(TARGET mod_websocket PROPERTY POSITION_INDEPENDENT_CODE ON)`
that also true


Wow ... I thought I was fast enough with aborting the push :(

Sorry! I actually wanted to open #8 but accidentally clicked "push to master" :S


Oh, this is an official policy now? I did not know that previously...


@valib No, I need to know where this goes before I do anything. If I'm going my own way it won't matter, and if not then we'll have to figure out how to handle it.

Updating the plugins is easy enough, but I don't have write access to any of them so that would mean that whoever has accepts a PR. I don't know which plugins use this. I don't know why a plugin would need to get the GUI in the first place, but it sounds like it must be a hack of some sort.


Nice. Eventually, `blocChatDb` could become a more polished abstraction/wrapper around Firebase, but it's a good starting point.


I pursued the code on the left because I didn't like how the connective phrases in `to_sentence` were also being bolded.


Looks good to me, can you open  a PR upstream ?


No it is also working on Mac OS. The FindApache search for either apxs2 or apxs (which exists on Mac OS)

The probleme I have with Mac OS is that not all the modules are availble.


크흠..? twrp를빌드해본결과 twrp부팅이전혀되질않습니다 어떻게된거죠?


`VehicleDamageEvent.setDamage()` - it from where?


Thats true but I wanted to keep a unique CMakeLists.txt. We can split it for the example.


:+1: will rewrite them


Yeah my change preserves the "and"s not being bolded.


Good job Alison. Can I ask what was the longest part of the assignment for you?


Also change `fp-section` for the constant SECTION.


You are right that's why I removed them


See `SentenceHelper`.


`stdout` & `stderr` are unused here aren't they?


Of course, it's a `TargetWalker`!  Thanks for explaining.


some tests failed:
exception: https://travis-ci.org/stellar/stellar-core/builds/110007157


indeed


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


fetch_stealth2 returns a different format (it's a different function on the gateway)

this is because of small changes on the stealth spec... fetch_stealth and fetch_stealth2 are part of a backwards compatibility scheme, so both formats can be queried by old and newer wallets. fetch_stealth will get proxied to older obelisk, fetch_stealth2 is querying the new libbitcoin-server that supports a different stealth scheme (different filtering of data contents).


aren't you missing a docker run command to actually run this image?


Please don't do this. It's a simple three-line idiom, and doesn't need to be a function. I'd rather have it be inlined for this purpose.


Oh, wait yes I do. Apparently we repeat the repeat the default .dat file name if there's not enough .datilfes listed for each key ([Source link] (https://github.com/chapel-lang/chapel/blob/master/util/test/genGraphs#L775-L779)). So it worked for you because you're using the default .dat file name (basename is the same as the .chpl file) but Ben used a non-default .dat file name.

Still, I think repeat-files is the right thing to do


I understand how it solves the NPE, but i am confused on what the expectation should be (and why it is ok in that control layer to sometimes have null attemptId and sometimes not)


I'm pretty sure this isn't what you meant.


Same


Oh I see, I didn't see `normal_weight_sentence_connectors` because line 9 being singular didn't show it.


null entity? kek, I remove them :D


As the build configuration for the examples is quite generic, I wanted to reflect that (in case some other examples pops in).


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


Does it mean we were supporting `credit` with an existing credit card but not with a new one?


I spent quite a lot of time thinking about this wording and I think the language "View as Default Role and Divisions" isn't ideal as what we really have is a state of ON -- my view is different -- and then we need to give the user an easy escape hatch to get back to normal.  This is why I used the language that I did.


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


Is the plan for Firewall.md to have all firewall related TILs?


Maybe just pass these directly to the function rather then making them variables? Although that might be less readable. Maybe kwargs?

It's not a blocker either way.


Hmm, 'cancel' to me implies that there's a change in progress which hasn't happened yet, and 'cancel' would abort it.  Maybe just "Go Back to Default View".


Yeah, I just wanted to make it super clear what was happening in each block, since they are so similar otherwise.


@JustAkan http://buf.jollaman999.com/multirom_test.zip


No the reason is the merging of reads to fragments (as merged reads), done in line 322 if needed, combined with the fact that I cannot instantiate a new ReadContext with the faked merged reads; the way ReadsContext is coded the reads must come from a file (or at-least a SamReader) so it seems overkill to go to that extend as to preserve ReadsContext here.


The consumer is documented as thread-safe, though.

http://people.apache.org/~nehanarkhede/kafka-0.9-producer-javadoc/doc/org/apache/kafka/clients/consumer/KafkaConsumer.html


Close #8 ... this is not a problem :P


One minor addition I might suggest here is to verify the type is a string. That way people who create additional importers can see what type of property it needs to be. We're going to be making a blog post about the import framework, so this will just be a nice check to have.


Yup, I am confused too. I decided to go ahead with the fix since the caller creating `PaymentStateControlContext` has a `@Nullable` for the attempt id. I could try to reproduce it in a test if you want.


Hello back.


This doesn't look so great at the call site:

```swift
buffer.ensureUniqueStorage(minimumCapacity: newLength, allocate: myMalloc)
```

I think the argument label here should probably still be `allocator`.


Nothing some development phase left over.


Good point, I hadn't considered that.


ActiveMerchant supports `credit` against a token only.


It's true though.


Yup!


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


Why? it is the "subject" of the test... doesn't sound good in English?


Dont we want to add this -notrait to outerloop ubuntu/osx runs?


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


This is version 3.1.1


ℑ 𝔴𝔞𝔰 𝔥𝔬𝔭𝔦𝔫𝔤 𝔴𝔢'𝔡 𝔤𝔢𝔱 𝔰𝔬𝔪𝔢 𝔣𝔯𝔞𝔨𝔱𝔲𝔯 𝔦𝔫 𝔥𝔢𝔯𝔢 𝔞𝔱 𝔰𝔬𝔪𝔢 𝔭𝔬𝔦𝔫𝔱...


It is important that wrapper be made :P


Bukkit


yeah probably, took it verbatim from a test. will look into it tonight.


@xeno-by I enabled the presentation compiler flag `-Ymacro-expand:discard` but it still gives an error here.


I want you to do TWO prompts firstName and last name DON'T FORGET YOUR SPACE


The scenario is `EnterScriptObject` constructor throws OOM and it catches it but sets `hr = E_OUTOFMEMORY`. `VerifyEntryScript` detects this and [throws](https://github.com/Microsoft/ChakraCore/blob/master/Lib/Runtime/Base/LeaveScriptObject.cpp#L70) OOM which is of type `ExceptionBase`. This doesn't get caught in existing catch but gets caught here.
```c++
catch (Js::ExceptionBase)
    {
        AssertMsg(false, "Unexpected engine exception.");
        return JsErrorFatal;
    }
````


Good call on the constants being BLANK, I think it's a useful way to remember it not only for the computer, but whoever is reading it. Good formatting on this. Doing most of your work outside the main function cleans it up nicely.


Actually, this has changed in 0.9 it's been listed as not thread-safe in the release doc.

http://people.apache.org/%7Enehanarkhede/kafka-0.9-producer-javadoc/doc/org/apache/kafka/clients/consumer/KafkaConsumer.html


Shouldn't this #ifdef be FEATURE_MULTIREG_RET?


Great!

Should we consider any other PR in the next release? Such as
JENKINS-26287 Prevent the console-outline to overlap the console output #26

> On 18 Feb 2016, at 00:55, George Simpson <notifications@github.com> wrote:
>
> Wondering if you would ever need to support a combination of -P and -D properties. I suppose if you were changing the config to use -P you would be crafty enough to know that -D would no longer work.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/jhinrichsen/gradle-plugin/commit/a9d9945728d2c0711db1289d622538b873f17955#commitcomment-16163340>.
>


This needs testing. Will report if this breaks CRON or not :)


Had to spend time figuring out what you were doing here. A comment would be nice. (As I see it, you are reading ahead and keeping the values in a queue. You pull a value out of the queue only when the size of the queue goes more than skip footer.)


One thing that helps is the using statement. It goes using std::cout; at the top of the code. It is works like using namespace. It helps work out the bugs if you forget a std::. You would have to use it for every one though. One for cin, cout, string. All of them.


Why just method body, shouldn't this entire method be under #if FEATURE_MULTIREG_RET?  I see all the calls of this method are already under #if FEATURE_MULTIREG_RET


Yup I imported the wrong Pair class... there are quite a few.


The only thing I would even critique  is that some of your curly braces are in odd spots and it seems cramped, but the functionality seems to work well.


Ah gosh I thought you had accepted the PR, didn't even realize you hadn't put it upstream. I'll revert this part of the change.


Aquí cambiamos el destinatario?


You can use the using keyword instead of the namespace, it works about the same but makes everyting neater. It goes like using std::cout;. You will have to use them for all of them though. One for cout, cin, string and all the others.


I need some help, When I rotated the device, the frontOverlayView did not resize it bounds, frame or limits and the user stills can select parts of the view because the frontOverlayView still has its originaal size, do you knowk how to fix that?

Thank you very much!


Will do some refactoring here.


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


@Nadahar this is on the @SubJunk as an owner of this project to decide.


jsでニューラルネットワークを実装ですか。
狂気を感じます。
このレベルまで慣れたらひらがなを書くが如く、プログラミング言語で色々なことを表現できるんだろうなぁ。
精進します。


depends on: https://github.com/blockchain/My-Wallet-V3/pull/121


Yeah Return to Default View isn't bad.  Part of the issue is that it makes sense to us but users don't think of the normal state as a default view.  That's a distinct more relevant to us when we program this.  The user just knows about the site as they are used to seeing it, and then also this concept that they can view the site as other people.  For them going back is less about changing state to a default view, and more about unapplying it.

Anyways, I find that _Cancel_ is used quite universally in UI/UX to indicate a way to safely end something that could change or cause harm.  In this case cancel is getting the user back to their default state which to me is the priority as I don't want them stuck or confused about the fact that they have a mask applied.

I had a few other ideas when thinking about it (it's quite the rabbit hole if you think about it):

"I am finished.  Take me back to normal."

"Cancel viewing and go back to default view"

"Finish viewing and go back to default role and division." (not ideal language if the user is just an admin)

-----

However as I thought about it also, I found that the "Change view" button could be removed to instead facilitate two dropdowns:  One for `role` and one for `division` (which could actually be layered quite nicely as a dropdown over bold text).  If so, then I think a normal "Cancel" could actually work quite well as its context next to the other actions is clearer.


여전히 cm을설치시 system을마운트시키지못합니다. 이걸어캐해야해야할까요?;;


makes sense.


地域メッシュの追加


condition should be
(callReturnTypes.getFirstReturnRegisterType() == EA_UNKNOWN) ? ...

(Since you are editing it please correct it; it works because EA_UNKNOWN is zero right now)


you can try the varargs like `RecordPointer(int ... fields)`


done.


What are exceptions can be here? Don't want to see repeat reset if the exception is deterministic.


i tested these new re-sampled sounds out and they are less cavey, so I just committed them.  If we have suggestions or changes we would like to request, please submit them to ender23 or myself.


To be honest, I don't see how `password` can ever be `NULL`, but it was in the original code.


Oops, I missed that one. Thanks Priya.


:+1: , remember, every time you commit a change to your `Gem` file, add the `Gemfile.lock` as part of the same commit. I usually create a single commit for updating my gem file (along with the gemfile.lock) There is a high chance of opportunity a new gem can break something, so identifying gem changes quick can help you on debug faster a problem.


LGTM. Did you try to run these test cases?


I believe that we might need three modes here: relying on consumer's autocommit essentially will commit on fetch. Until now we have committed offsets after processing (even in the automatic mode). I suggest we use three settings: FETCH, AUTO, and MANUAL (with the later two being essentially manual, from the consumer's point of view).


For consistency, I copied the logic from `sig_listen` below, which uses `server_find_chatnet`.


99% of this commit is changing values += 0.5. Why?


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


This is simply about showing something in the preview saying "this pckage version is used in these projects: X, Y, Z, etc."

I really don't want that list to be ginormous.  Consider hte case of something like roslyn.  Where we have 100+ projects and we might reference the same package in all of them.  I don't want to list all 100 projects here.  Just a few so the user can get a sense of where the package is currently used.


파이썬에서 권고 하고 있는 최소한의 코딩 스타일 가이드 입니다.
https://www.python.org/dev/peps/pep-0008/


@JavierJia Yes, I passed the test cases.


Oh right, so it updates if it has an ID but doesn't exist in the database. Weird.


I'm not sure what this means.  I'm calling into an external API that i'm assumign may throw for any reason.  As such i'm just being resilient to that.  This is purely to show additional informatoin (an installed project version).  So if it fails, it's not something i want ruinging the rest of the experience.


Make sure you spend sometime reading about the different devise modules and what they provide.
Devise is by far the most widly 'complex' gem used across rails projects, so it is very handy to know well this library. 90% of the project i have worked on with rails, use devise

For example this `confirmable` module offers something interesting. Some apps use it and others not. Why it could be?


ここ試してただけだから、実際は1です


It's also described [in the docs](https://github.com/torch/nn/blob/master/doc/convolution.md#spatialconvolution).


Mac에서는 정상적으로 제 계정으로 되는데 회사에서 윈도우를 사용하고 있는데 윈도우에서는 unknown으로 나오네요 ㅡㅡ;


I don't think the project name is that necessary.  it's just the project you're currently in.

I do think 'package' and 'version' can be removed though.


결국은 unknown도 저입니다. ^^;


:smile:


You may want to update this, this can be used as your app defult.


Nice, you can go ahead to replace the array with your SVector.


Would be good to see this with parameter replacement.


@aammd
Here I decided to add a multiplier, that changes depending to what uses the add_donation


Doesn't OpenSSL provide an API for this? https://www.openssl.org/docs/manmaster/crypto/RAND_bytes.html

Since we're already using OpenSSL cross-platform that'd probably be a more workable solution than implementing this ourselves per-platform.


5479_OAandPLANTEquipList (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (997 of 997 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-28.67%25-green.svg)


For some reason, often when I'm opening Xcode, all the views in this one view controller get misplaced, so I just update the frames. When I redesign this view controller this should go away. Also, you should watch this repo.


@marcus-h : once again, thanks for your hint and your careful reviews. I will take care (at least to revert this line of my commit)

(BTW: yes I got a notification and read it ;-))


2016년 개인적인 다짐 입니다. 어렵지만 조금씩 조금씩 실천해 나갈 생각 입니다.


Read all different options :neckbeard:


As we discussed that is the way the Indirect emitCall was calling it - twice. The refactoring was just to refactor the code and not the functionality of the original code.


Electric-Service-Refactor (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2082 of 2090 tests passed, 546 test warnings)

 * 550 tests had: AUD diffs.
 * 480 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 3 tests had: EIO diffs.
 * 1 test had: SHD diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

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


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


Nit -- additional white space before "and" can be removed.


Waiting on #33.


Undo/redo is basically "cross your fingers and hope it works".  Because the system is not actually build on immutable snapshots all the way down, you're always invariably just running arbitrary code and hoping it can properly roll back anything it did.  So, in general, this will work, but i would always describe it as something that has hte potential for failure.


Shouldn't this #if be FEATURE_MULTIREG_RET?


That didn't work for me ("AttributeError, object has no __exit__"), had to change it to stream = urllib2.urlopen(url).

Could you confirm this please ?


Shouldn't this #ifdef be #if FEATURE_MULTIREG_RET?


Right. Changed. It was in the existing code before the refactoring as well.


There are two reasons why it would be good to understand:
1. Make sure we don't cover something by handling the NPE correctly here (that may very well resurface somewhere a bit further). That's the main concern at this point
2. Understanding better how our code works (what is expected and not and in which case). That part is only interesting if we can document it, if not we gonna forget in 2 days (at some point i know the answer, i just don't anymore)

So short story, if you are confident for 1/, don't anything more unless you also want to enhance [our payment doc](http://docs.killbill.io/0.16/userguide_payment.html) which does not seem to talk vey much about attempts.


This looks to be a separate bug fix.


Thanks for the quick fix and the reminder!
I'm being having that for a while and it didn't cause big trouble..


@thebigsmileXD
A Chinese phrase:别给脸不要脸


So far, CRON is broken. Need to confirm this before reverting.


yo no tengo TOC pero creo que este commit podría estar unido a 8d43b83abc4fc75a2669a438baceebbfc4c921c7. Estos dos hacen que pasen los tests :smile:


I just thought of mentioning on record:  This PR combines multiple fixes listed below:
1) gc-hole fix for struct return in two registers
2) gc-hole fix for passing structs containing gc-pointers on stack (genPutStructArgStack() changes)
3) Modelling of GT_RETURN(retbuf) in case of methods returning structs through a hidden retBufArg
4) Above bug fix in cloning call nodes
4) Refactoring changes

It would be a good idea to separate them into multiple PRs for better tracking and for risk mitigation.


I left creating a singleton as an exercise to the reader.

For example, you might want to make two date-time properties to share the same transformer, and one date-only property to have its own. This method has no idea that you want to do that.


When the project is closed the undo stack is cleared (fortunately).


멀티롬 second롬 설치시 Failed to mount '/system' (Invalid argument) 가뜨면서 멀티롬설치가되지않습니다.
실제로 구글로고에서도멈추구요. 아마도시스템이텅비어서그렇지않을까싶습니다.


Was this deletion intended?


@mangogogos Good point, didn't realize the sources were still useful for that use case. Adding them back in the next release: 403176ae9332fb7c8beac339dd41cbb067f995fd

Sorry about removing them in the first place!


Remove this and just have getTenantForKey always return something. In the properties file case return the key.


Waiting on #34.


_NuGetVisualStudioPackageReference is defined above.

What this is is verifying that we shoudl have been able to find that reference so that we can add the 'EmbedINteropTypes' flag to it.  If we can't find that reference then something went very wrong with the nuget restore.


Why the above code is needed since constructor of GenEmitCallInfo() is already ensuring that argSize is zero for non-x86 case?  The above code can be deleted?


whoops!


Same deal here


I don't like this at all. Maybe you should just use `self._queue.get(True, some_timeout)` instead, so it will re-evaluate `self._thread_running` from time to time.


db stuff should be commented out by default


No, that looks suspicious.


Space after if


Can we get rid of this?


done.


sorry for laterly update, it works fine w/ my workstation.


It is needed in this now. The code now gets the classification in the GT_CALL member structDesc.
The SetRegisterReturningStructState(structDesc);  has been eliminated. If the copy is not performed, the member structDesc of GT_CALL gets not copied properly.


Yes.  not called or used.


Damnit - Fixed!


PropertiesFileKeyManager should be the default


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


Fixed!


k


This interesting. By moving the module loading from the airframe to the flight plan, what are the expected benefits for the overall architecture?
It would be really nice if the flight firmware could serialize `module` objects, a linker, if you will, without having to flash the firmware each time there is a change to a flight plan and/or airframe. That opens up other issues related to security and formal verification of flight software, etc... but it might be fun to have. :)


thankss


debugging?


Back to fixtures--great idea!


No swearing


Looks good!


This is great


Nowhere, as far as I know. I took them from here: https://github.com/daid/Cura/blob/SteamEngine/package.sh#L177


this is literally just piping /dev/(u)random to the user.
I would like to have access especially to /dev/random. (for the pro-users)

I will try the openssl rand thing now.


Sup country computer?
Good job.
How about next time name your functions something that would help you identify what they do?
Like alertName() and basicAlert() maybe?


Please note that it was needed even before as well.  Whenever a  GT_CALL node is cloned, we are not cloning structDesc. Just because we eliminated definition of SetRegisterReturningStructState() doesn't make it necessary now.  The reason to eliminate SetRegisterReturningStructState() is that the lone call to that method in importer is not necessary and hence can be cleaned up.


We never used ActiveRecord fixtures before in this project. The only fixtures I've seen before this are the web request fixtures that we generate with VCR.


Yup.


@mkonicek Won't having the sources in the npm package increase the size? people who want to compile can always clone from GitHub.


Canola was misspelled. Need to update this word before pulling into master branch.


I spoke to lasconic a few hours ago on IRC and he said he would merge my PR in the next day or two. When he does, you'll need to rebase because I changed a few things. You might get some conflicts when you rebase, so you might want to test the rebase in a new branch first and only update this one once you are sure you've got it working. That way you can still fall back to this one if something goes wrong.

Come to think of it, you probably want to make a new branch anyway so that you are tracking MuseScore:master rather than shoogle:portable-linux-appimage. Of course, you can't do that until after the PR is merged.


Please help 92721644
2/16/16
Email : adietze@msn.com


I don't think you can do that in vanilla agar.io since there is a limit on how much you can zoom in or out.


If `flags` equals `MAKE_IDT_FLAG(SEGMENT_PRESENT, RING_OS)`, which expands to `3 | 4`, which is obviously `7`, why do you need to force the value to `0x07` on this line?


@mkonicek Seems the `ReactAndroid` folder is `4.8M`. IMHO we shouldn't have it in the NPM package.


I believe the query() will return FALSE on failure (http://php.net/manual/en/mysqli.query.php).
So in case of failure here, you just don't do anything and the function ends?

In the previous implementation, the TRUE/FALSE return value was something of an error check. You probably want to raise an exception on failure here or stick with the original implementation.


comment about why this is x'ed?


# To run as an app on your android phone do the following:

$ cd  /pathtolocalrepo/LCSC-Android-App/LCSC Campus Life Android App/
# Build the application apk
$ cordova build
# Install the application on your phone (Make sure your phone is plugged into your computer)
$ adb install platforms/android/ant-build/CordovaApp-debug.apk

# That's it! :smile:


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


This is bad. You're simply ignoring the failure case of query(). Not to mention there is no point simply returning TRUE which is what the $result value will be inside this if block. If you're not going to throw an exception then at least stick with the original true/false return values which make more logical sense.


you'd need to update these too :)


Why is this in a try catch just so taht it hides what type of an error first occured?


maybe... just like at first we thought it was impossible to use premium skins...


As said earlier, I think this breaks CRON. Not too sure because I've only tested a bit.


Django m2m things aren't really my jam, but I think I understand the issue here. The way this is coded, it picks the first one and then moves on with life. If we're really concerned about this situation--i.e. it indicates something is seriously wrong--then we should probably send someone an email with details about the problem or log it somewhere where it's going to be seen or something.


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


You can get rid of one level of indentation here by taking this approach:

if (!$result = parent::query($sql)) {
    # $result is false...
    throw new Exception(...)
}

# Your '$result is true" code goes here without needing to be inside an if block with an extra indent.

This approach is especially useful where your if/else blocks become large or you have further nested if/else blocks inside like in this case.


You can get rid of one level of indentation here by taking this approach:

    if (!$result = parent::query($sql)) {
        # $result is false...
        throw new Exception(...)
    }

    # Your '$result is true" code goes here without needing to be inside an if block with an extra indent.

This approach is especially useful where your if/else blocks become large or you have further nested if/else blocks inside like in this case.


This line needs to be here. We need it for creating a user. When you create a user you need the password twice.


moved to `/tmp` RimeOfficial/opsworks-cookbooks/issues/1


There is a typo in the dependencies causing the CI task to fail. I'll be more than glad to merge this if you want to fix it.

This is what it gets by merging:

"cocktail": ""cocktail@>=0.5.1 <0.8.0""

Thanks for taking the time to do this!

Max


lol. To get this to work, you have to make the cells render smaller. But the issue is, the amount you can zoon in or out will always have to be the same. Unless MoneyClip made an update, a couple months ago I checked and remade the client which showed the amount you can zoom in and out was counted in the client. Not sure yet if it's still a thing though.


"*an* artist's" ... "*an* HTML page"

#grammarpolice


Wow, thanks for the detailed exposition and references!
I still think that the language would be better if nameability and reachability were unified, but you've convinced me that probably won't be feasible until 2.0 and that this draft may not be the best approach.


*rectangle*


we will have to see...


首先,如果你觉得有什么不爽,请说出它.但你自己必须明白你为什么要这样说.
不要像一个小孩子耍脾气,我们讨厌这样.你觉得这样很酷吗?我们觉得你很烦.
其次,我们的代码贡献/参与者全部在README.md上面.

看来之前高看你了.


Yep.


Should be able to display the angle brackets by writing < for < and > for >


You're making the same mistake of not handling the failure case here.
I understand you switched to exceptions only where an error/warning message was being displayed, but then why did you modify these functions and in a way 'broke' there original true/false return value implementation? Now you're returning True when things are successful or null, which is just weird.


How about simply silently falling here, and not reporting a rate if this sysctl call fails?


a setTimeout though makes an assumption about when that dynamic camera is getting set


since moneyclip made my bot extension zero out all text, it also affected premium skins. so i added (almost )all of them into customskins.txt lol


This line is the fix, along with `s/order/self.active_orders[nick]/` in the rest of the file.

For brevity, i'd have just put that instead of the `self.db.execute()` call; am I missing something here?


dis da head


line note


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


嗯，实现 #59 中提出的功能。


All failure cases for functions where some value is expected should be handled with an exception.
Other queries which are not returning a value (like INSERT, UPDATE, DELETE queries) should return true/false for error checking.


Very well, when can we talk?

On Wed, Feb 17, 2016 at 1:50 PM, tkx <notifications@github.com> wrote:

>
>    - The code is Python3 compatible if print arguments are in parentheses.
>    - The complexity of this code is O(n^3). Additional memory is needed
>    in particular functions for comparison.
>    - Next I would place cols&rows check first, because there is no need
>    to proceed if it fails. Just a notice, only 3 inner squares are actually
>    checked according to the way validate_inner is called in validate, the
>    rest of the validate_inner calls do nothing, maybe this needs to be
>    refactored.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tkx/py/commit/ec56ac4ab495502a69ba26ab714585faed629a28#commitcomment-16145475>
> .
>


also is there a way around the text zeroing out? see this:
![capture](https://cloud.githubusercontent.com/assets/15255724/13130711/22867b88-d5b4-11e5-8c02-f553ea4127aa.JPG)
notice how the agar.io client removes text from most buttons and options when it senses the bot extension running. any way around it?


This is a potential workaround for #388 and allows the merger to go ahead from a fresh PR if permitAll is set (i.e. no comment body exists).


This looks great. Thanks! You could do the test that handles only showing location to friends next if you wanted!


I'm gonna accept this pull now because I pulled it onto my computer and the tests you wrote pass.


MapGet, MapDelete는 BTreeGet을 기반으로 작성했습니다. 원래의 코드를 수정하는 과정에서 포함되었습니다.
BTreeGet의 map, getMap()도 따로 사용하는곳을 발견하진 못했는데 확인이 필요할 것 같습니다.


this happened before... i remember agario-powerups was in its early stages. same thing happened =\


:+1: on this commit.


OK no problem. I updated https://github.com/jgamble/Algorithm-Networksort/pull/11 to keep it inline.


The argSize member of GenEmitCallInfo only exists for _X86_ target. The constructor also initializes it to 0 only for that target.


MapDelete에 포함된 부분은 삭제하도록 하겠습니다. 감사합니다.


Looks good.


A couple remarks here for future reference. I'm making the changes locally in the interest of time, but wanted you to know for the next one :)

* In general, I organize the enumerations in alphabetical order, so I'm moving this one up in the file. I notice I didn't follow my own convention quite rigorously in this file as the top one is out of order, but I'm going to leave it rather than add noise to the commit for now.

* Generally I copy the description for each enumeration member directly from the MS API enumeration. Sometimes they are a little clumsy or don't apply in the python-docx case so I might reword them a bit. But generally they do a pretty good job. Note that this particular enumeration is used for more than highlight color, so the descriptions should be applicable to the general case. Also, they should not be active, as in "Set highlight color to ..", rather, they should be noun phrases of some sort. It's the method that does the action, these just indicate the state of affairs, either existing or required. They can also be used in the getter, for example, which doesn't do any setting. The MS API values are generally consistent with this.

* Buried in the Enumeration base class somewhere is code that auto-generates the documentation page for the enumeration, which is then referenced in API documentation. Here's how you get it:

  * complete the enumeration
  * start Python in the project directory and run these lines:

  ```python
  from docx.enum.text import WD_COLOR_INDEX
  print WD_COLOR_INDEX.__docs_rst__
  ```

  * cut and paste the output into a new file named 'WdColorIndex.rst'. This is the value of `__ms_name__` in the enumeration.
  * Add that page file name (less extension) to docs/api/enum/index.rst in alphabetical order.
  * Generally then I re-'make' the docs and check to make sure everything turned out right.
    ```bash
    $ make docs
    $ open docs/.build/html/index.html
    ```


Suggest you go through the PHP database classes again and fix the true/false return cases to:

    return parent::query($sql);


It seems that there was NOT a significant performance hit as I reported here. Some strange dark magic caused my computer to run at 1/3 normal speed today. It seems to be entirely unrelated to the addition of the particle effects, and I've confirmed that it is running at near the same fps as the previous commits.


NOTE: referenced performance issues are a fluke... see my comment on 3073b050e848267f9100a387fc503df776257159


Awesome feature!


So would I! That's what my test said originally but I changed it when I realised the actual behaviour was different.


L604-606 duplicated.


SZVAV-Fan-Control (rraustad) - i386-Windows-7-VisualStudio-12: Tests Failed (1724 of 1986 tests passed, 274 test warnings)

 * 536 tests had: AUD diffs.
 * 191 tests had: EIO diffs.
 * 262 tests had: Table big diffs.
 * 11 tests had: BND diffs.
 * 14 tests had: RDD diffs.
 * 6 tests had: ESO small diffs.
 * 2 tests had: MTR small diffs.
 * 4 tests had: ESO big diffs.
 * 8 tests had: MDD diffs.
 * 8 tests had: MTD diffs.
 * 1 test had: SHD diffs.
 * 3 tests had: Table small diffs.
 * 2 tests had: MTR big diffs.
 * 1 test had: ZSZ big diffs.
 * 2 tests had: ERR diffs.

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


No conscious reason. I reimplemented it rather than copying the what was there and didn't notice that I'd done it different. I'll change it to use `#has_key?`.


this file is really messed up... need to restore it to as it was.


I don't know if this works. I just tried in an interactive console to do `my $dr = 0; $dr = 0 || warn "asd"` and `print $dr` returned `1`


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


Whoa, that was fast. Nice!


@ogrisel , @jarfa , this is a material change; we've been down this road before with regressions: https://github.com/scikit-learn/scikit-learn/issues/4184


Whoot! https://github.com/blog/2111-issue-and-pull-request-templates


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


Be careful not to clutter up the global scope with your vars and functions. It would be very easy to load in another script file that also defines a `var canvas` or `var score`, and if they're in the same scope, the last one defined will win. This can lead to a very difficult bug hunt.

It's a best practice to isolate your code in [modules](https://addyosmani.com/writing-modular-js/) (advanced topic) and/or [namespaces](https://addyosmani.com/blog/essential-js-namespacing/) (fairly straightforward, but a little less safe than modules).

At the very least, consider putting all of your JS here in its own file instead (say, `test.js`), and then include the file with your `<script>` tag: `<script src="test.js"></script>`

As far as namespacing this code, you can do that rather simply by just wrapping it in an Object. i.e. instead of:
```
var canvas = document.getElementById("myCanvas");
var ctx = canvas.getContext("2d");
var ballRadius = 10;
// etc
function keyDownHandler (e) { /* ... */ }
function keyUpHandler (e) { /* ... */ }
// etc
```
wrap them in an Object (plays the part of a namespace), like this:
var buda = {
  canvas : document.getElementById("myCanvas"),
  ctx : canvas.getContext("2d"),
  ballRadius : 10,
  // etc
  keyDownHandler : function keyDownHandler (e) { /* ... */ },
  keyUpHandler : function keyUpHandler (e) { /* ... */ },
  // etc
};
```
Then whenever you want to access these properties/methods elsewhere, just add the `buda` namespace in front: `buda.canvas`, `buda.keyUpHandler`. It's extremely likely some other script on the page might define a function named `keyUpHandler`. It is far less likely that someone else will name something `buda.keyUpHandler`.


C++ objects that inherit from `lsst::daf::base::Citizen`.  Most large C++ classes are supposed to do so; many small ones do not.  Note that even pure-Python packages without C++ or SWIG source may still call other packages that do have C++, so it's a best practice to include the `MemoryTestCase` in all cases.

In the future, a more advanced `MemoryTestCase` could be written, but it's perhaps more likely that a tool like `valgrind` would be applied externally.


Please tell me someone is building a nightly artifact.


s/4 six/4 seconds/


@matthewisabel :heart_eyes:


For efficiency, maybe do this with NumPy instead?

First, convert the images to NumPy arrays per here:

  https://pyscience.wordpress.com/2014/09/06/numpy-to-vtk-converting-your-numpy-arrays-to-vtk-arrays-and-files/

Then just add them together with the '+' operator:

```
outputImageDataAsArray = inputData1AsArray + inputData2Array
```


The previous commit you've linked to updates the checksum data for marriage-abroad. This one updates it for register-a-birth (which also relies on `CountryNameFormatter`).


The earlier commit you've linked to updates the checksum data for marriage-abroad. This one updates it for register-a-birth (which also relies on `CountryNameFormatter`).


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


unack


Don't use `**/*.md` as it would include `node_modules`, `.eggs`, `build`, `venv` and other folders a user may have too.


My bad :(


nice. I'll try and do that next


这个文件貌似看起来不像源文件


You are welcome to rework it along those lines if you wish, it was like
that for a very long time and I was trying to keep it minimal.

On Wed, Feb 17, 2016 at 5:10 PM, Tim Moody <notifications@github.com> wrote:

> why is one in fedora and the other in yum?
>
> why not fedora and centos?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jvonau/xsce/commit/47e10f138d5dc69a5cbb0deffb87871f18e28175#commitcomment-16162529>
> .
>


whats dat ?


> I consider FMOD's behavior to be what is proper and logical

In case you've missed it, this is a clear case of "your usecase != generic usecase".
Perhaps it somewhat makes sense in ZDoom usecase (though I'd argue that multichannel 3d sounds should have been marked as a clear case of "don't do that" back in the past...nevertheless they weren't), but in a more generic context, there would be usecases where such choice would make no sense at all.


Will this do the right thing? I swore `''`ed strings won't have variables interpolated, so this should always fail?


We could have a pubs section at the start and reference them. See #22


Что ещё осталось:
Дошлифовать по pixel perfect hotels, сейчас там небольшой сдвиг относительно макета, и фильтр с ползунками тоже не совсем где надо.
Анимация формы поиска отелей на главной.
Сделать тень для body.

Пункт из файла specifications:
_Фильтр: верстать с помощью формы, кнопка «Показать» отвечает за отправку формы, при выключенном JavaScript должен осуществляться переход на отдельную страницу (отдельную страницу верстать не нужно)._
Значит нужно делать ссылку и менять её на кнопку из JS, когда он подгружается?

Пункты из критериев:
_Д17. Вёрстка проходит тест на переполнение контентом.
Б12. Вёрстка идентично отображается в последних версиях браузеров Chrome, Opera, Firefox, Safari, а также в Internet Explorer 10+.
Д12. Файл стилей представлен в двух вариантах: с форматированием и минимизированный, к странице подключён минимизированный.
Д18. Критический функционал сайта работоспособен без JavaScript (использовано прогрессивное улучшение)._
Здесь не совсем понятно, где граница критического функционала.

+ проставить vertical-align для всех инлайн-блоков(так и не понял или забыл, почему  это так важно)


Yeah, we need to swap the quote types here...


Make sure to do this in all the html files.


Is there anything that prevents a custom extension from using the context path `/NIFI-API`? If not, a custom extension could use a case-sensitive-unique context path but then the logic here would pass through.


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


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


Yeah, I would recommend:

```java
byte[] randomBytes = new byte[32];
secureRandom.nextBytes(randomBytes);
final SecretKeySpec secret = new SecretKeySpec(nextBytes, HMAC_SHA256);
```


Nit: I usually prefer `preTickFunctions.length = 0` as [promoted in this David Walsh blog post](https://davidwalsh.name/empty-array), although [this post](http://www.2ality.com/2012/12/clear-array.html) suggests that your approach here is faster and better, as long as you're sure there aren't any other references to the array being cleared.

So, I guess I'm fine with this as-is.


<input name="<{$f.dictionary}>" mytime="time" id="date_<{$f.dictionary}>"  readonly="readonly" type="text" placeholder="请选择时间">


The invariant is that only one of addr or indir will be non-null.  We should also assert in genEmitCall() that if indir is non-null then addr is null


Some of your function names are very active verbs, like `drawBall`, while others are very passive, like this one. Typically the active form is used for names of functions that perform an action, like `doSomething`, `detectCollision`, etc.

One exception is functions that do a check and return a boolean value; in that case, you typically name the function as if you were asking the question that the function check: `isNaN`, `isPerson`, `hasPoints`. That way, conditionals that utilize these functions read very naturally:
```
if (isPerson(data) && hasPoints(data)) {
   // Do stuff for data that is a person who has points
}
```

Another exception is functions that convert one type to another. `toString`


Hi @jpwchang, are these Bluetooth-related changes _necessary_ for Melonee's driver to work with Sphero BB-8? (I just bought one and want to test drive it.) Thanks :smile:


Wow, Stash connection can resolve some bugs related to atomikos and provide performance improvement. Good Job.


sticket/Application/Home/View/CrmUser/index.html
515
生成工单，并随机分配给：


Will do.


In this case (indir != nullptr) we should assert that addr is nullptr.


5479_OAandPLANTEquipList (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1549 of 1549 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.77%25-green.svg)


Does this introduce a new dependency on `app.tickCount`?  I thought the original implementation guaranteed that  a `runOnAppTick` method would only happen once, where this one might run over and over if the app in question wasn't incrementing `tickCount` reliably.

I guess we always depended on `tickCount` to decide when to run this though, so... never mind.


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


This test should still exist. Just make the behavior such that if the destination transfer is somehow already fulfilled it should GET the fulfillment as a separate request and pass it on. Extreme edge case, should never happen, but that is I think the safest behavior.


I *strongly* recommend setting up coffeelint to ensure that you maintain a consistent code base. Check out teascript for the example of that.


This change is subtly wrong. An empty macro in an `#if` expression evaluates to false (and a compile error), not 1. The previous behaviour was more correct, in a way.
```
$ cat preproc.c
#define FOO

#if FOO
#warning FOO is true
#else
#warning FOO is false
#endif

$ clang preproc.c
preproc.c:3:8: error: expected value in expression
#if FOO
       ^
preproc.c:6:2: warning: FOO is false [-W#warnings]
#warning FOO is false
 ^
1 warning and 1 error generated.
```

Issue #1704 is entirely about the handling of `env['DEFINES']`. GCC and clang handle arguments of the form `-DFOO=bar` by acting as if the line `#define FOO bar` was inserted before the first line of the compilation unit. The compilers special-case `-DFOO` to mean `#define FOO 1` instead of `#define FOO`. An empty macro is defined on the command-line with `-DFOO=`.


reack


I changed this if/then to a one-liner, you can have a look in the commit; it's a little more elegant I think :)
https://github.com/python-openxml/python-docx/commit/f3be324455d2637b8e77ae58a41ff02cb77321a2#diff-ef89d5063878646ebb226b61acf90383R111

I also used `expected_value` rather than `expected_result` as it's more specific (and used throughout so far (or at least should be :)).


Interesting. I don't feel strongly. However, I do feel like it's slightly more clear what I'm doing with my approach (setting preTickFunctions to be empty).

For example, suppose I had the following code:
```
var preTickFunctions = {
  fn1: function () {},
  fn2: function () {},
  length: 2
};
...
// a bunch of other code that makes it non-trivial to go find the original declaration
...
preTickFunctions.length = 0;
preTickFunctions = [];
```
Perhaps contrived, but with the second approach I can know exactly what I have without needing to look elsewhere.


Ok, this runs perfectly. I've committed it to feature/highlight on the main repo:
https://github.com/python-openxml/python-docx/commit/f3be324455d2637b8e77ae58a41ff02cb77321a2


Whops! Thanks fixing!


What is the intent of this name field? How is it different than item.metadata.itemName? Why would someone want to change it?


Is this function used anywhere?


comma should be on previous line.


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


It doesn't seem to make sense to me to have this input field in the "adjust size" section.


I hadn't considered that case. :+1:

On Wed, Feb 17, 2016 at 5:54 PM, Brent Van Minnen <notifications@github.com>
wrote:

> Interesting. I don't feel strongly. However, I do feel like it's slightly
> more clear what I'm doing with my approach (setting preTickFunctions to be
> empty).
>
> For example, suppose I had the following code:
>
> var preTickFunctions = {
>   fn1: function () {},
>   fn2: function () {},
>   length: 2
> };
> ...
> // a bunch of other code that makes it non-trivial to go find the original declaration
> ...
> preTickFunctions.length = 0;
> preTickFunctions = [];
>
> Perhaps contrived, but with the second approach I can know exactly what I
> have without needing to look elsewhere.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/code-dot-org/code-dot-org/commit/84d5dc5180e2d97d7e1a9ca0b2a2669bcf45f6fb#commitcomment-16165249>
> .
>

--
Brad Buchanan
Software Engineer, Code.org
brad@code.org


return should be on newline.


@TheDiamondYT1 Wtf? You changed ur username?


break is unreachable and should be omitted.


Great - thanks


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


@TheDiamondYT1 Prove that you are RandomAltThingy.


The English is fine, but `subject` simply conveys less.


This code is so crappy structured, I can't even


Good point. A custom UI's context path is driven by the war's filename which is typically a combination of artifact id and version. However, there is nothing preventing someone from trying to be sneaky. That said, in this case, it would fall through into the else block which is what we want, correct? We only want to attempt to verify a download token if it is a download request through the REST API (/nifi-api). An incoming request to anything other than /nifi-api is considered a request to a UI extension which is verified as such on line 73 below.


Nice, I want to see the solution of this problem! I would go solve to... but I working in #2.


Date


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


mismatched


Thanks XD


Yes, --until could also be useful.
I fixed the where clause and a few edge cases in the latest commit.


You can use assertIs


Use assertIs


I didn't add the delivered counter directly in the ProduceRequest class, as it has nothing to do with its inner workings. Here it seems to me natural to mark the instance of ProduceRequest with the number of deliveries externally.


This will still loop through every single blood test result in the test batch even if we can tell from the very first one that re-entry is required. I think that if we have already determined that there is a test result which requires re-entry for a blood test type then we should skip over any other results for that type. It would also make sense to exit the loop if we find a test result for each type that requires re-entry, just not sure if there is an easy way to do this.


@bradenmacdonald I don't know of a way to do that no, although postfix arcana can get pretty convoluted, and I clearly don't claim mastery of it : )

That said, I don't mind maintaining this list - it's pretty rare to have to change that list while we are still small, and there are also plenty of addresses that don't go to Google (besides my email, things like urgent@, dev@, etc.) - so at least this way I don't risk sending emails to GApp by mistake and ending up losing emails that Google doesn't know how to handle because they should have gone elsewhere.


Please couple this with changes to documentation (we have a .rst file for config code)


Ouch, sorry.

-----Original Message-----
From: "Lasse Schuirmann" <notifications@github.com>
Sent: 18-02-2016 15:13
To: "coala-analyzer/coala" <coala@noreply.github.com>
Cc: "AbdealiJK" <abdealikothari@gmail.com>
Subject: Re: [coala] find_user_config: Handle files, directories better(8e858bb)

you cherry picked this commit from somebody else, please keep the author info in that case.
—
Reply to this email directly or view it on GitHub.


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


Use :: and check how this renders in documentation, with a few tweaks you can make stuff that looks terrible look great. It looks okay here but I suspect it would render badly.


I don't **need** it byt when I have a bitfield, I am used to `and` the result with the right amount of bits so that the code is more readable. But it is a total choice and preference.


@lpointet wow. And I thought I was secure by using get_home_path() instead of ABSPATH directly...
Ok so yes I'll copy this wonderful piece of poetry:
```
$slashed_home      = trailingslashit( get_option( 'home' ) );
$base              = parse_url( $slashed_home, PHP_URL_PATH );
$document_root_fix = str_replace( '\\', '/', realpath( $_SERVER['DOCUMENT_ROOT'] ) );
$abspath_fix       = str_replace( '\\', '/', ABSPATH );
$home_path         = 0 === strpos( $abspath_fix, $document_root_fix ) ? $document_root_fix . $base : get_home_path();
```
to our code...


It does as far as my reading has gone.

Ive tested PermissionError, FileNotFoundError and "delete dir instead of file"
All gave OSError.

Apparently a while back this was `IOError`, but 3.3+ `IOError == OSError`  (I checked this too)


Lol no prob bob


Мне кажется, лучше вместо цифр ввести константы, которые можно настраивать


En effet, cela fonctionne à présent. Merci pour le dévouement ;)

cf. a877fc442bbe8c47520b426ce6502c3e6b31f2c2


you should make it clear what will be the done call order


This should check ":$name"


согласна, чтобы имена классов были более специфичные


Add saving and fix the highlighted issue, then +1


FYI, the MPL / LGPLv2 version of the iTextSharp is still in the official git repository at https://github.com/itext/itextsharp. Look for the older tags.

There are no builds attached to the older tags, but neither are there in this repo. iText Software has no intention of removing the source of the older MPL / LGPLv2 version. It's just not going to be maintained and there won't be any official binaries.


Instead of shell/command you could use the apt_key module https://docs.ansible.com/ansible/apt_key_module.html

There is a module for everything :)


Cool, yeah, I think I switched between 'Insert' and 'Import' about twelve times. Couldn't decide which one was better. :smile:  About the dropdown, my thinking was to rather not have the destructive option as the default, even though it might confuse users who are accustomed to the their content being replaced and appending feels like the least aggressive option. You sure you rather want replace as the default?

Ah, that's a really good point about the translations. Will change that.


It's best to keep it there because it stops the 'catch up' calculation code from running, making the ABM slightly faster.


Thanks , @weshaggard


Nice work! :+1:


For now, yes.


:P


I'd feel better if we'd mock the glob to return bogus files so the branch gets tested with the right exceptions


Me three!


Wow, my fault.


Disregard this comment - I've changed my mind after thinking about it some more. I don't think that the performance impact is big enough to require a change here.


do we need a reference here ?


This is the checkbox which I need to determine is checked or not


Because I am used to seeing the names in blue :stuck_out_tongue:


Always returns null. I had tried redirected to "openmrs/users.list" is it returned null and it always redirected. No clue why. I followed [this](http://stackoverflow.com/questions/15775412/checking-which-check-boxes-are-selected-using-java-a-jsp) tutorial.


Lol


Woohoo!


You could just put this into the bowl: ``context.bowl.launcher = launcher``.


`MailQueueAction NOOP = success -> {};`


我是发现make test时比以前慢了。。


It's more a Provider than a Factory (no parameter given to `newAction`).


Let's make it an optional arg.


I don't have a problem with using stubbing instead of the WebMock-based (?) `worldwide_api_xxx` helper methods. However, I think the amount of stubbing in this test suggests that the implementation isn't ideal. It might be worth considering introducing a method something like this: `WorldLocation.find_fco_organisation_for_country(slug)` to hide some of the implementation and make it easier to stub. More of the implementation could then be tested in the `WorldLocation` unit test.


I wonder why we're doing this in the init, rather than the build and inserting the modules in __all__ ?


NOOPFactory = () -> NOOP;


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.592.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=69337&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 931, ignored: 56 Build time: 00:15:48


เพิ่มแล้ว commit มาเลยครับ


Prety Gud. :+1: :camel:


FAIL tests/filesys/base/syn-write
run: ((null)) open "stuff": FAILED
pass tests/userprog/args-none
pass tests/userprog/args-single
pass tests/userprog/args-multiple
pass tests/userprog/args-many
pass tests/userprog/args-dbl-space
pass tests/userprog/sc-bad-sp
pass tests/userprog/sc-bad-arg
pass tests/userprog/sc-boundary
pass tests/userprog/sc-boundary-2
pass tests/userprog/halt
pass tests/userprog/exit
pass tests/userprog/create-normal
FAIL tests/userprog/create-empty
FAIL tests/userprog/create-null
FAIL tests/userprog/create-bad-ptr
FAIL tests/userprog/create-long
FAIL tests/userprog/create-exists
pass tests/userprog/create-bound
pass tests/userprog/open-normal
pass tests/userprog/open-missing
pass tests/userprog/open-boundary
pass tests/userprog/open-empty
pass tests/userprog/open-null
pass tests/userprog/open-bad-ptr
pass tests/userprog/open-twice
pass tests/userprog/close-normal
pass tests/userprog/close-twice
pass tests/userprog/close-stdin
pass tests/userprog/close-stdout
pass tests/userprog/close-bad-fd
pass tests/userprog/read-normal
pass tests/userprog/read-bad-ptr
pass tests/userprog/read-boundary
pass tests/userprog/read-zero
pass tests/userprog/read-stdout
pass tests/userprog/read-bad-fd
FAIL tests/userprog/write-normal
pass tests/userprog/write-bad-ptr
pass tests/userprog/write-boundary
pass tests/userprog/write-zero
pass tests/userprog/write-stdin
pass tests/userprog/write-bad-fd
pass tests/userprog/exec-once
pass tests/userprog/exec-arg
pass tests/userprog/exec-multiple
pass tests/userprog/exec-missing
pass tests/userprog/exec-bad-ptr
pass tests/userprog/wait-simple
pass tests/userprog/wait-twice
pass tests/userprog/wait-killed
pass tests/userprog/wait-bad-pid
pass tests/userprog/multi-recurse
pass tests/userprog/multi-child-fd
FAIL tests/userprog/rox-simple
FAIL tests/userprog/rox-child
FAIL tests/userprog/rox-multichild
pass tests/userprog/bad-read
pass tests/userprog/bad-write
pass tests/userprog/bad-read2
pass tests/userprog/bad-write2
pass tests/userprog/bad-jump
pass tests/userprog/bad-jump2
FAIL tests/userprog/no-vm/multi-oom
FAIL tests/filesys/base/lg-create
FAIL tests/filesys/base/lg-full
FAIL tests/filesys/base/lg-random
FAIL tests/filesys/base/lg-seq-block
FAIL tests/filesys/base/lg-seq-random
FAIL tests/filesys/base/sm-create
FAIL tests/filesys/base/sm-full
FAIL tests/filesys/base/sm-random
FAIL tests/filesys/base/sm-seq-block
FAIL tests/filesys/base/sm-seq-random
FAIL tests/filesys/base/syn-read
FAIL tests/filesys/base/syn-remove
FAIL tests/filesys/base/syn-write
23 of 76 tests failed.
make: *** [check] Error 1


real good documentation now!


This should only change `reEntryRequiredTTITests` if the value is not already true, otherwise it might be set to false in another iteration of the loop even though a previous rule result has tests which require re-entry.


Лучше, наверное, сначала присвоить значение 0, а потом проверять его наличие


Cool.
If I find some time I volunteer to optimize this template according to my regular experience.


Fred style?


Let's start to use constants for stuff like this.


## Fixes this error:
```make
$ ./autogen.sh
autoreconf: Entering directory `.'
autoreconf: configure.ac: not using Gettext
autoreconf: running: aclocal
autoreconf: configure.ac: tracing
autoreconf: configure.ac: creating directory build-aux
autoreconf: configure.ac: not using Libtool
autoreconf: running: /usr/bin/autoconf
configure.ac:10: error: possibly undefined macro: AC_PROG_LIBTOOL
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.
autoreconf: /usr/bin/autoconf failed with exit status: 1
```


I think it would be a good case to use Custom Validators: http://guides.rubyonrails.org/active_record_validations.html#custom-validators
What do you think, @rschumann?


changed


looks good.


These are not ASCII double quotes (`"`), they are UTF-8 opening and closing double quotes (`“”`) which cause a compilation error.


Just make sure that the zip file starts with "./" or "/". But note that unzip does some extra option parsing after the zipfile, you need to make sure that those are safe. (Or make sure that no files in $MYFILES start with '-').


Same as above.


LGTM


I'd like to avoid using shell here. It's so easy to get this wrong in the end (quoting). Let's try to get this to work without sh


The thing is, I need to know what kind of token the string is split with. If I put all splitting in a single regex, this information gets lost. (Also I need the indices to do the computation.)


feature: addde new API bind


Много несодержательных файлов


Does this file should go to commit "bricks: add diferent types of bricks(dipole, multipole, monopole)" ?


Same


这是一个测试文件


Don't % expand logging calls, it's done internally

``_logger.warning(_(".... %s", "stuff")``


`Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>`


this is only part of the fix, you can still change span in the general tab option, maybe this option should be removed there? not sure


Let's make sure that we can check this at the time we --validate


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


Is this fixing a real vulnerability? Seems like it... ChangeLog doesn't specifically call out any problem though.


Javel


You need to define the operator `D` with all the tokens (somewhere in lines 11–26).


Please push your changes into DEV branch not in the stage.


achtergrondkleur Chocolate


That comment would be for case 3 actually.


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


don't we use `analyticsName`? if no, please remove from constructor


You've just reverse the mappings of ``Bio.AlignIO.StockholmIO.PFAM_GR_MAPPING`` which was effectively part of the public API (no leading underscore).


Typo in recursive call


You can drop this function and just do a split on "[, ]". It'd be nice if you also had support for quoted strings.


I've been testing using node.js 5.6.0. Is there any other way to return 4xx response using node.js 5.x, when a parse error occurs?


@Patater Please verify that name is pointing to flash and all the characters of *->name are within flash.


Same here, does this should go to previous commit ?


I'd really handle the p_h >= 1.0 case instead, so that we don't make broken computations for f, p, q and t and fix it afterwards, but handle the problem at its root.


Can you consider here the suggestion in [Issue77](https://github.com/Z-Wave-Me/zwave-smarthome/issues/77) to use the scene icon in the elements view?


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: .travis.yml


typo: element, transformation

I'm not sure what this comment is trying to say...


@ggouaillardet if (opal_memory->memoryc_set_alignment) should be here to avoid collision with memory/malloc_solaris.

I think these changes should fix issues came from previous commits but I see one more lack in my original PR. See description in https://github.com/open-mpi/ompi-release/pull/967#issuecomment-185250851


hello


These and other contexts with only one test... Should we merge the before with those tests, or you plan to have more tests for those contexts?


question: what happens when user restart SR at midnight ? it will take 27 hours to run next showupdater ?


This is not used. Remove.


@mr-sneezy - you might find these changes handy.
Also, nick has pulled in my updates to SoftwareSerialWithHalfDuplex (updated using SoftwareSerial for Arduino 1.6.5r5 as a base) - should be smaller and more robust to timing issues as 1.6.5r5 doesnt use static tables.


Please create a centralised sanity check function for box_id and replace all occurences of this check in other functions, too.


```
>>> import shlex
>>> shlex.split('ala "ma kota"')
['ala', 'ma kota']
```


This is only required in the else, right?


Those one line "it" should use { } instead of do-end blocks. And also remove the description (if the test is self explanatory).


Please remove this import now that you’re importing it with the library style below.


This modification was made in a previous "nop" commit, does this should go to "bricks: add diferent types of bricks" ?


why those submodules are no longer in the project ? What is the procedure if we use sub modules within the project without using maven.


I don't get the link with nic brick


Why is needed to run it 3 times?


Changed to:
# making a list of 1 element a string allows features
# such as ID/ACC in Pfam to be checked
# with a string comparison

It is just to explain the rationale of converting to a string only features that have one element


ack


Ok, in my opinion this is not that nice.

However, i will merge and publish this weekend. Thanks for your PR!


No newline at EOF.


We should not have to access private state of session assistant.


i think it's still nice to do the comparison at https://github.com/xiexingguo/ceph/commit/9062954f01f76c8016478e6bcb9fa66655b1a2ca#diff-0f4fc4acc5679fcbb07632c4346f8628R206 also, so we don't wait CPU cycles looping in the shard once we've collected enough number of warning messages.


s/seem/seam/g


this added an extra level of `/julia/` to this path which is not correct for binary installs - #15130


We should return a useful return code. This is a long standing bug in plainbox-based tools.


and for the same reason, no need to continue looping in the `while` loop once we have enough warning messages, so a `break` is desirable.


s/php/PHP/g

I'd also format any inline code with ticks (\`), e.g. `$myVar::myStatic()`.


cleanup needed


Hmmm, I've checked code multiple times, there was no multiple calls to defer/await, but there was exceptions thrown sometimes, and looking at `Fiber.current._defer` - it seems (although I'm not sure) like there could be a situation when `Fiber.current._defer` could be set to `true` then exception thrown and it would be never set to `false`. I'll be looking into that bug these weekends.


== not =


Yes, that was the way it was before. The dictionaries in the Iterator and Writer were reversed, but needed to be kept in sync.
By making it a global, there's no need to keep them in sync. At module import, the dictionary is copied into the Iterator, while it is copied reversed (as it is needed) in the Writer.

I didn't prepend an underscore because it wasn't in the previous class attributes.


again


probably not


Help code public on framework.jar.out/smali_classes2/com/android/internal/R$string.smali not patch all


commit and push มาแล้วครับ


![](http://i.imgur.com/AGZ3mge.png)
OS : Window 8.1
Eclipse : Mars.1 Release (4.5.1)
환경에서는 위 코드 정상 작동 하는거 같습니다.^^


Let's add a bug about not saving user-made selection, refer to the bug here and fix this later.


if it fails, you shoud @Ignore it


Cherry-picked to v0.1.


Is there need for 'if def'? Surely 'find node near' depends on finding a node registered with 'group flammable'?


internal


Hi,

Just for your information @teichsta :

since this commit, the openhab cloudbees fails on the smarthomatic binding : https://openhab.ci.cloudbees.com/job/openHAB1-Addons/1161/consoleFull

Thanks !


s/creates/creations ?


`warned > 0` would suffice. but personally, i think `return warned` is also acceptable =)


404 https://github.com/MeTaNoV/toggle-icon


5 ème correction bien


Out of curiosity: Will close consume or do we make sure on the server to pull all messages so that all side effects get executed?


Help for patch code public string no auto patch on framework.jar.out/smali_classes2/com/android/internal/R$string.smali with public.xml on framework-res/res/values???
Sorry my bad english


Может вообще заюзать функцию?


Overall, looks good. Just needs the noted typo and formatting fixes. Thanks! :+1:


why are you generating so much blog routes? I only need once :-S You don't need to generate a blog route per post... right?


For boolean variables, CMake has `option()`.


Sort this set


it would be cool to use the right (eclipse) import rules to avoid messing with imports everywhere


Why not name the inner class and test is separately?


This should be initialized in opal/mca/memory/malloc_solaris/memory_malloc_solaris_component.c:73 by the parent commit https://github.com/ggouaillardet/ompi/commit/066d91487195c4983346be11b39d9689c41f8b08


Fixed


you should document why you use two assertion sets for the same field


@vilchik-elena I think we should include at least one property and one method which are compliant.


I think it makes more sense to put it in @Before


toppie


I'll have a look on this, but have never written any ruby specs before. I just wanted to use the module with hiera and found it a usefull addition. Should i also add the same implementation for users?


is it intended to not make it public ?


Let's file a bug on Session Assistant to store the selection *and* the test plan that was used so that we don't have to use side channels to save this. This will also fix the issue of overwriting the selection that we implicitly do by re-selecting the test plan.


convert ?


s/anyParameter/anyParameters/g (with an "s")


Let logging do the % expansionn


you should extract submethods


use test case


It's fugly.
Let's try doing a simpler matching


Please follow the project style conventions for the option lists (use existing plugins as examples).


Ah, I mixed up with an `or`, you don't force the value to be at least 7, you just restrict the number of bits. Ok :)


@rasjostrom Created a new develop branch to separate the project more. If you do PR, please send to develop :)


Noticed that also this commit prevents triggering the event of `recordingState` since this state would not return any `recordingId`. This would just return as a log.error rather than triggering the event. `recordingState` events will be only triggered based on the received events from the MCU message


The plugin is named dnf. Having a `dnf-` prefix on options just looks silly:

```
# sosreport -o dnf -k dnf.dnf-history=True
```


:thumbsup:


多谢，已经修复：https://github.com/magicshui/qingcloud-go/commit/b768871a10365de0414f2b3d775e79ef32276e81


An sos plugin doesn't display things - suggest 'capture', 'collect' etc. instead.


This apply to whole table even plugin table, and default string is set to 255 for plugin table.
Why not restrict to really needed table ?


Why make this an option if it is fast and enabled by default?


Whoa, comments :)


Sorry, yes, comparing against the master branch you've not changed any existing module level API.


Awesome, thanks for your work! I like the trick with `#[allow(non_snake_case)]`, allowing you to use the macro args both in the argument and the type parts of the function.

Now I just need to learn how to read ASM to determine if something vectorized correctly.


We should probably detect what happened and not show empty selection lists or stuff like this. During development people sometimes get their environment.


Happy to modify it. It's your code after all :bow:


This patch breaks Q in a major way. We have just hit this in our production server - after autoscaling and getting new EC2 instances (which automatically does `npm install`) we got the latest version of `newrelic` module 1.25.2, all hell broke loose after this.

```
   [ ESC[32m'TypeError: Q.nextTick.runAfter is not a function'ESC[39m,
     ESC[32m'    at trackRejection (/var/app/current/node_modules/q/q.js:1057:20)'ESC[39m,
     ESC[32m'    at reject (/var/app/current/node_modules/q/q.js:1131:5)'ESC[39m,
     ESC[32m'    at _fulfilled (/var/app/current/node_modules/q/q.js:836:20)'ESC[39m,
     ESC[32m'    at self.promiseDispatch.done (/var/app/current/node_modules/q/q.js:863:30)'ESC[39m,
     ESC[32m'    at Promise.promise.promiseDispatch (/var/app/current/node_modules/q/q.js:796:13)'ESC[39m,
     ESC[32m'    at /var/app/current/node_modules/q/q.js:604:44'ESC[39m,
     ESC[32m'    at runSingle (/var/app/current/node_modules/q/q.js:137:13)'ESC[39m,
     ESC[32m'    at flush (/var/app/current/node_modules/q/q.js:125:13)'ESC[39m,
     ESC[32m'    at doNTCallback0 (node.js:419:9)'ESC[39m,
     ESC[32m'    at process._tickDomainCallback [as _tickCallback] (node.js:389:13)'ESC[39m ] }
```


Ditto for capitalisation, prefix and default enablement.. Why would a user ever want to disable this?


Hi,

you were missing " as it is done eg. in object_checkcommand.conf.erb. The line should be:

`command = [ <% if @cmd_path -%><%= @cmd_path -%> + <% end -%><% @command.each_with_index do |cmd, i| %>"<%= cmd -%>"<%= ', ' if i < (@command.size - 1) %><% end %> ]`

The second problem is that since `@cmd_path` must be a string, there is no way that the if statement will ever be false. Hence, in the worst case when it is en empty string, the line starts with `+`. If a string such as `/etc` is defined double quotes are missing.


Ditto.


@franktoffel, **chacheme** does not exist :sweat_smile: I already fixed that!


That would be great.


ack


cейчас у тебя `this.giftBoxes = Promise` а нужно чтобы был массив с которым ресолвится этот промис
```
giftBox.getList().then(giftBoxes => {
});


Isn't `ReadTrace` missing the function body? That's not legal Go code, but it seems to work with the way natives are used. Should we do something about this? /cc @neelance


I've seen it, but it

    Select the data instances based on conjunction or disjunction of filters
    derived from :obj:`ValueFilter` that check values of individual features.

I would need at least one *that checks values of pairs of features*, like *(a = 1) and (b = 5) or (a = 3) and (b = 2)*. This filter handles just conjunction or disjunction. We can add it. And/or we can refactor the filters, some of which contain some amazing relicts from their C++ ancestors, like https://github.com/biolab/orange3/blob/master/Orange/data/filter.py#L252. This was to needed be quick without overusing STL maps. It not longer makes sense since `Domain.index` uses dicts.


Neuer Tinyfy key


You could just set this to ```None```, and below use ```if actual_nr_of_agents < lowest_load or lowest_load is None:```. Then there never will be an issue where 100 would be insufficient. Or use ```sys.maxint```.


It would actually be better if the call was being patched - keeping a local copy is messy and bound to go wrong at some point...


Think you meant "it increases" rather than "in increasing."


Are these managed by logrotate or dnf itself? If they are not size-limited or rotated by default then they should be collected using `add_copy_spec_limit()` and the current global `size_limit` value.


<img src="http://thewritepractice.com/wp-content/uploads/2012/05/Alot-vs-a-lot1-600x450.png"/>


Are these values ever likely to change? Is it worth breaking them out into general.yml config keys?


> vary