@johnnykv I got another error when conpot is starting:
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/gevent-1.0.1-py2.7-linux-i686.egg/gevent/greenlet.py", line 327, in run
    result = self._run(*self.args, **self.kwargs)
  File "/usr/local/lib/python2.7/dist-packages/Conpot-0.4.0-py2.7.egg/conpot/protocols/ipmi/ipmi_server.py", line 423, in start
    assert self.bmc == None
AttributeError: 'IpmiServer' object has no attribute 'bmc'
<Greenlet at 0xb4b4aa2cL: <bound method IpmiServer.start of <conpot.protocols.ipmi.ipmi_server.IpmiServer object at 0xb4b0606c>>('0.0.0.0', 623)> failed with AttributeError

ERROR:root:Stopping because <Greenlet at 0xb4b4aa2cL: <bound method IpmiServer.start of <conpot.protocols.ipmi.ipmi_server.IpmiServer object at 0xb4b0606c>>('0.0.0.0', 623)> died: 'IpmiServer' object has no attribute 'bmc'
2015-07-14 15:48:34,108 Stopping because <Greenlet at 0xb4b4aa2cL: <bound method IpmiServer.start of <conpot.protocols.ipmi.ipmi_server.IpmiServer object at 0xb4b0606c>>('0.0.0.0', 623)> died: 'IpmiServer' object has no attribute 'bmc'


ok this was my error. pull it and try it now


thx:
https://github.com/motom001/DoorPi/commit/d8a7e4bd03337bfbb57f661a73452cc6ea95b434


Why create a Bucket w/o a client?  Its methods won't be usable.


These two fields are non-standard (see http://www.softwareishard.com/blog/har-12-spec/#response), so it makes sense to add a comment.

I like `url` attribute, but not `method`. There is no "response method", there is only a "request method". A better solution (as you said earlier) is to add `request` argument to the callbacks.


You don't need an extra boolean flag here you can return immediately with the error message instead of breaking the cycle.


I think if autoCreateIndices is false Predicates.alwaysFalse() could be used


how about justt `The requested coafile '{filename}' does not exist.` :)


Security purpose - for URL matching


CM uses other implementation in cm-12.1 branch. If you basing your codes with cm-12.1 then you don't need anything else to work. - it just works.


Hey there!


No, please Vaggelis, it cannot work like this. We are more than pleased to have your features and bug fixes included into the project, but a pull request should ideally contain only one feature and should in any case be complete in order to be merged. I'm really sorry it took me 20 days to review the code, it couldn't be done earlier. I hope it won't happen again.


SDB_MAX_VALUE is not available in cdb.c
loading sdb.h from there doesn't seem to elegant either
considering <<0, <<8, <<16 is right above it doesn't seem so wrong


The purpose of this is to match the existing `zfs list` behavior where snapshots are not listed unless they are either named or recursion is requested. Removing this would break enforcement of the depth parameter.


Ugh.


Maybe it's not the best idea to go the db every 5sec to ask for the same data. You can pass this value at the caller side. Same applies to the other checker.


What bothers me is that coala anyway does something, it executes an empty default section and the user thinks: "Did I do something wrong?" (At least for me the first time)


thanks


just out of curiosity, where is the repo we are copying this from?


Nice catch.


well `The requested coafile '{filename}' does not exist.` is pretty obvious is it?


ah great. thank you


TODO?


Since we want to support some defaults for quick linting I think the message is not that bad, we just need to specify the defaults in our doc.


This commit LGTM.


maybe something like "testThatIndexMissingExceptionIsntIgnored" would fit better? Otherwise it looks kind of obvious that you'll get the exception if indexItem is overwritten to just throw the exception


asp.net c#.net


Not really ... The user would wonder why is it running if the file didn't exist

-----Original Message-----
From: "Lasse Schuirmann" <notifications@github.com>
Sent: 14-07-2015 19:25
To: "coala-analyzer/coala" <coala@noreply.github.com>
Cc: "AbdealiJK" <abdealikothari@gmail.com>
Subject: Re: [coala] main: Refactor warnings when no coafile existent(c690300)

well The requested coafile '{filename}' does not exist. is pretty obvious is it?
—
Reply to this email directly or view it on GitHub.


Shouldn't `_DefaultsContainer` go away too?


Хорошо


@vito I'm worried about the merge here. Was there a typo?


Likewise the whole `_testing` module, which exists only to support the `_implicit_environ` pattern.


This looks good to me.


yea, i got it :)

i fixed the xposed bug and my old rom working well with my own kernel but you mean our patchroms cant boot due to the miui binary inside bin ?
Sorry for disturb you ;)


Sorry @sils1297 @AbdealiJK I mean my message with "using default"


Could you please re-test this?
E.g. I tried the following scenario and it didn't work as expected without this change:
```
zfs create mypool/fs
zfs snap mypool/fs@snap
zfs list -r -t snapshot mypool/fs
```


hm interesting, does the original sql-lite repo not have this logic?


@lfryc @lholmquist - looks like this is not working - I am not sure, how to increase the 'scope' of the `contextPath` -> works in the snippets


it was the master one, & i compiled with Qt5 (but last time I compiled with Qt5, it ran).
here's my CMakeOutput.log :

The system is: Linux - 3.16.0-43-generic - x86_64
Compiling the C compiler identification source file "CMakeCCompilerId.c" succeeded.
Compiler: /usr/bin/cc
Build flags:
Id flags:

The output was:
0

Compilation of the C compiler identification source "CMakeCCompilerId.c" produced "a.out"

The C compiler identification is GNU, found in "/home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/2.8.12.2/CompilerIdC/a.out"

Compiling the CXX compiler identification source file "CMakeCXXCompilerId.cpp" succeeded.
Compiler: /usr/bin/c++
Build flags:
Id flags:

The output was:
0

Compilation of the CXX compiler identification source "CMakeCXXCompilerId.cpp" produced "a.out"

The CXX compiler identification is GNU, found in "/home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/2.8.12.2/CompilerIdCXX/a.out"

Determining if the C compiler works passed with the following output:
Change Dir: /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp

Run Build Command:/usr/bin/make "cmTryCompileExec1093653383/fast"
/usr/bin/make -f CMakeFiles/cmTryCompileExec1093653383.dir/build.make CMakeFiles/cmTryCompileExec1093653383.dir/build
make[1]: Entering directory '/home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp'
/usr/bin/cmake -E cmake_progress_report /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp/CMakeFiles 1
Building C object CMakeFiles/cmTryCompileExec1093653383.dir/testCCompiler.c.o
/usr/bin/cc    -o CMakeFiles/cmTryCompileExec1093653383.dir/testCCompiler.c.o   -c /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp/testCCompiler.c
Linking C executable cmTryCompileExec1093653383
/usr/bin/cmake -E cmake_link_script CMakeFiles/cmTryCompileExec1093653383.dir/link.txt --verbose=1
/usr/bin/cc       CMakeFiles/cmTryCompileExec1093653383.dir/testCCompiler.c.o  -o cmTryCompileExec1093653383 -rdynamic
make[1]: Leaving directory '/home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp'

Detecting C compiler ABI info compiled with the following output:
Change Dir: /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp

Run Build Command:/usr/bin/make "cmTryCompileExec3514964000/fast"
/usr/bin/make -f CMakeFiles/cmTryCompileExec3514964000.dir/build.make CMakeFiles/cmTryCompileExec3514964000.dir/build
make[1]: Entering directory '/home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp'
/usr/bin/cmake -E cmake_progress_report /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp/CMakeFiles 1
Building C object CMakeFiles/cmTryCompileExec3514964000.dir/CMakeCCompilerABI.c.o
/usr/bin/cc    -o CMakeFiles/cmTryCompileExec3514964000.dir/CMakeCCompilerABI.c.o   -c /usr/share/cmake-2.8/Modules/CMakeCCompilerABI.c
Linking C executable cmTryCompileExec3514964000
/usr/bin/cmake -E cmake_link_script CMakeFiles/cmTryCompileExec3514964000.dir/link.txt --verbose=1
/usr/bin/cc     -v CMakeFiles/cmTryCompileExec3514964000.dir/CMakeCCompilerABI.c.o  -o cmTryCompileExec3514964000 -rdynamic
Using built-in specs.
COLLECT_GCC=/usr/bin/cc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/4.9/lto-wrapper
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 4.9.1-16ubuntu6' --with-bugurl=file:///usr/share/doc/gcc-4.9/README.Bugs --enable-languages=c,c++,java,go,d,fortran,objc,obj-c++ --prefix=/usr --program-suffix=-4.9 --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --with-gxx-include-dir=/usr/include/c++/4.9 --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --enable-gnu-unique-object --disable-vtable-verify --enable-plugin --with-system-zlib --disable-browser-plugin --enable-java-awt=gtk --enable-gtk-cairo --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-4.9-amd64/jre --enable-java-home --with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-4.9-amd64 --with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-4.9-amd64 --with-arch-directory=amd64 --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --enable-objc-gc --enable-multiarch --disable-werror --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib --with-tune=generic --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu
Thread model: posix
gcc version 4.9.1 (Ubuntu 4.9.1-16ubuntu6)
COMPILER_PATH=/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/:/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/
LIBRARY_PATH=/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/:/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib/:/lib/x86_64-linux-gnu/:/lib/../lib/:/usr/lib/x86_64-linux-gnu/:/usr/lib/../lib/:/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../:/lib/:/usr/lib/
COLLECT_GCC_OPTIONS='-v' '-o' 'cmTryCompileExec3514964000' '-rdynamic' '-mtune=generic' '-march=x86-64'
 /usr/lib/gcc/x86_64-linux-gnu/4.9/collect2 -plugin /usr/lib/gcc/x86_64-linux-gnu/4.9/liblto_plugin.so -plugin-opt=/usr/lib/gcc/x86_64-linux-gnu/4.9/lto-wrapper -plugin-opt=-fresolution=/tmp/cchjG3Wz.res -plugin-opt=-pass-through=-lgcc -plugin-opt=-pass-through=-lgcc_s -plugin-opt=-pass-through=-lc -plugin-opt=-pass-through=-lgcc -plugin-opt=-pass-through=-lgcc_s --sysroot=/ --build-id --eh-frame-hdr -m elf_x86_64 --hash-style=gnu --as-needed -export-dynamic -dynamic-linker /lib64/ld-linux-x86-64.so.2 -z relro -o cmTryCompileExec3514964000 /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crt1.o /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crti.o /usr/lib/gcc/x86_64-linux-gnu/4.9/crtbegin.o -L/usr/lib/gcc/x86_64-linux-gnu/4.9 -L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu -L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib -L/lib/x86_64-linux-gnu -L/lib/../lib -L/usr/lib/x86_64-linux-gnu -L/usr/lib/../lib -L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../.. CMakeFiles/cmTryCompileExec3514964000.dir/CMakeCCompilerABI.c.o -lgcc --as-needed -lgcc_s --no-as-needed -lc -lgcc --as-needed -lgcc_s --no-as-needed /usr/lib/gcc/x86_64-linux-gnu/4.9/crtend.o /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crtn.o
make[1]: Leaving directory '/home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp'

Parsed C implicit link information from above output:
  link line regex: [^( *|.*[/\])(ld|([^/\]+-)?ld|collect2)[^/\]*( |$)]
  ignore line: [Change Dir: /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp]
  ignore line: []
  ignore line: [Run Build Command:/usr/bin/make "cmTryCompileExec3514964000/fast"]
  ignore line: [/usr/bin/make -f CMakeFiles/cmTryCompileExec3514964000.dir/build.make CMakeFiles/cmTryCompileExec3514964000.dir/build]
  ignore line: [make[1]: Entering directory '/home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp']
  ignore line: [/usr/bin/cmake -E cmake_progress_report /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp/CMakeFiles 1]
  ignore line: [Building C object CMakeFiles/cmTryCompileExec3514964000.dir/CMakeCCompilerABI.c.o]
  ignore line: [/usr/bin/cc    -o CMakeFiles/cmTryCompileExec3514964000.dir/CMakeCCompilerABI.c.o   -c /usr/share/cmake-2.8/Modules/CMakeCCompilerABI.c]
  ignore line: [Linking C executable cmTryCompileExec3514964000]
  ignore line: [/usr/bin/cmake -E cmake_link_script CMakeFiles/cmTryCompileExec3514964000.dir/link.txt --verbose=1]
  ignore line: [/usr/bin/cc     -v CMakeFiles/cmTryCompileExec3514964000.dir/CMakeCCompilerABI.c.o  -o cmTryCompileExec3514964000 -rdynamic  ]
  ignore line: [Using built-in specs.]
  ignore line: [COLLECT_GCC=/usr/bin/cc]
  ignore line: [COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/4.9/lto-wrapper]
  ignore line: [Target: x86_64-linux-gnu]
  ignore line: [Configured with: ../src/configure -v --with-pkgversion='Ubuntu 4.9.1-16ubuntu6' --with-bugurl=file:///usr/share/doc/gcc-4.9/README.Bugs --enable-languages=c,c++,java,go,d,fortran,objc,obj-c++ --prefix=/usr --program-suffix=-4.9 --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --with-gxx-include-dir=/usr/include/c++/4.9 --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --enable-gnu-unique-object --disable-vtable-verify --enable-plugin --with-system-zlib --disable-browser-plugin --enable-java-awt=gtk --enable-gtk-cairo --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-4.9-amd64/jre --enable-java-home --with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-4.9-amd64 --with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-4.9-amd64 --with-arch-directory=amd64 --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --enable-objc-gc --enable-multiarch --disable-werror --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib --with-tune=generic --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu]
  ignore line: [Thread model: posix]
  ignore line: [gcc version 4.9.1 (Ubuntu 4.9.1-16ubuntu6) ]
  ignore line: [COMPILER_PATH=/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/:/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/]
  ignore line: [LIBRARY_PATH=/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/:/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib/:/lib/x86_64-linux-gnu/:/lib/../lib/:/usr/lib/x86_64-linux-gnu/:/usr/lib/../lib/:/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../:/lib/:/usr/lib/]
  ignore line: [COLLECT_GCC_OPTIONS='-v' '-o' 'cmTryCompileExec3514964000' '-rdynamic' '-mtune=generic' '-march=x86-64']
  link line: [ /usr/lib/gcc/x86_64-linux-gnu/4.9/collect2 -plugin /usr/lib/gcc/x86_64-linux-gnu/4.9/liblto_plugin.so -plugin-opt=/usr/lib/gcc/x86_64-linux-gnu/4.9/lto-wrapper -plugin-opt=-fresolution=/tmp/cchjG3Wz.res -plugin-opt=-pass-through=-lgcc -plugin-opt=-pass-through=-lgcc_s -plugin-opt=-pass-through=-lc -plugin-opt=-pass-through=-lgcc -plugin-opt=-pass-through=-lgcc_s --sysroot=/ --build-id --eh-frame-hdr -m elf_x86_64 --hash-style=gnu --as-needed -export-dynamic -dynamic-linker /lib64/ld-linux-x86-64.so.2 -z relro -o cmTryCompileExec3514964000 /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crt1.o /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crti.o /usr/lib/gcc/x86_64-linux-gnu/4.9/crtbegin.o -L/usr/lib/gcc/x86_64-linux-gnu/4.9 -L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu -L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib -L/lib/x86_64-linux-gnu -L/lib/../lib -L/usr/lib/x86_64-linux-gnu -L/usr/lib/../lib -L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../.. CMakeFiles/cmTryCompileExec3514964000.dir/CMakeCCompilerABI.c.o -lgcc --as-needed -lgcc_s --no-as-needed -lc -lgcc --as-needed -lgcc_s --no-as-needed /usr/lib/gcc/x86_64-linux-gnu/4.9/crtend.o /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crtn.o]
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/collect2] ==> ignore
    arg [-plugin] ==> ignore
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/liblto_plugin.so] ==> ignore
    arg [-plugin-opt=/usr/lib/gcc/x86_64-linux-gnu/4.9/lto-wrapper] ==> ignore
    arg [-plugin-opt=-fresolution=/tmp/cchjG3Wz.res] ==> ignore
    arg [-plugin-opt=-pass-through=-lgcc] ==> ignore
    arg [-plugin-opt=-pass-through=-lgcc_s] ==> ignore
    arg [-plugin-opt=-pass-through=-lc] ==> ignore
    arg [-plugin-opt=-pass-through=-lgcc] ==> ignore
    arg [-plugin-opt=-pass-through=-lgcc_s] ==> ignore
    arg [--sysroot=/] ==> ignore
    arg [--build-id] ==> ignore
    arg [--eh-frame-hdr] ==> ignore
    arg [-m] ==> ignore
    arg [elf_x86_64] ==> ignore
    arg [--hash-style=gnu] ==> ignore
    arg [--as-needed] ==> ignore
    arg [-export-dynamic] ==> ignore
    arg [-dynamic-linker] ==> ignore
    arg [/lib64/ld-linux-x86-64.so.2] ==> ignore
    arg [-zrelro] ==> ignore
    arg [-o] ==> ignore
    arg [cmTryCompileExec3514964000] ==> ignore
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crt1.o] ==> ignore
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crti.o] ==> ignore
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/crtbegin.o] ==> ignore
    arg [-L/usr/lib/gcc/x86_64-linux-gnu/4.9] ==> dir [/usr/lib/gcc/x86_64-linux-gnu/4.9]
    arg [-L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu] ==> dir [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu]
    arg [-L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib] ==> dir [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib]
    arg [-L/lib/x86_64-linux-gnu] ==> dir [/lib/x86_64-linux-gnu]
    arg [-L/lib/../lib] ==> dir [/lib/../lib]
    arg [-L/usr/lib/x86_64-linux-gnu] ==> dir [/usr/lib/x86_64-linux-gnu]
    arg [-L/usr/lib/../lib] ==> dir [/usr/lib/../lib]
    arg [-L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../..] ==> dir [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../..]
    arg [CMakeFiles/cmTryCompileExec3514964000.dir/CMakeCCompilerABI.c.o] ==> ignore
    arg [-lgcc] ==> lib [gcc]
    arg [--as-needed] ==> ignore
    arg [-lgcc_s] ==> lib [gcc_s]
    arg [--no-as-needed] ==> ignore
    arg [-lc] ==> lib [c]
    arg [-lgcc] ==> lib [gcc]
    arg [--as-needed] ==> ignore
    arg [-lgcc_s] ==> lib [gcc_s]
    arg [--no-as-needed] ==> ignore
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/crtend.o] ==> ignore
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crtn.o] ==> ignore
  remove lib [gcc]
  remove lib [gcc_s]
  remove lib [gcc]
  remove lib [gcc_s]
  collapse library dir [/usr/lib/gcc/x86_64-linux-gnu/4.9] ==> [/usr/lib/gcc/x86_64-linux-gnu/4.9]
  collapse library dir [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu] ==> [/usr/lib/x86_64-linux-gnu]
  collapse library dir [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib] ==> [/usr/lib]
  collapse library dir [/lib/x86_64-linux-gnu] ==> [/lib/x86_64-linux-gnu]
  collapse library dir [/lib/../lib] ==> [/lib]
  collapse library dir [/usr/lib/x86_64-linux-gnu] ==> [/usr/lib/x86_64-linux-gnu]
  collapse library dir [/usr/lib/../lib] ==> [/usr/lib]
  collapse library dir [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../..] ==> [/usr/lib]
  implicit libs: [c]
  implicit dirs: [/usr/lib/gcc/x86_64-linux-gnu/4.9;/usr/lib/x86_64-linux-gnu;/usr/lib;/lib/x86_64-linux-gnu;/lib]
  implicit fwks: []

Determining if the CXX compiler works passed with the following output:
Change Dir: /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp

Run Build Command:/usr/bin/make "cmTryCompileExec1443612808/fast"
/usr/bin/make -f CMakeFiles/cmTryCompileExec1443612808.dir/build.make CMakeFiles/cmTryCompileExec1443612808.dir/build
make[1]: Entering directory '/home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp'
/usr/bin/cmake -E cmake_progress_report /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp/CMakeFiles 1
Building CXX object CMakeFiles/cmTryCompileExec1443612808.dir/testCXXCompiler.cxx.o
/usr/bin/c++     -o CMakeFiles/cmTryCompileExec1443612808.dir/testCXXCompiler.cxx.o -c /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp/testCXXCompiler.cxx
Linking CXX executable cmTryCompileExec1443612808
/usr/bin/cmake -E cmake_link_script CMakeFiles/cmTryCompileExec1443612808.dir/link.txt --verbose=1
/usr/bin/c++        CMakeFiles/cmTryCompileExec1443612808.dir/testCXXCompiler.cxx.o  -o cmTryCompileExec1443612808 -rdynamic
make[1]: Leaving directory '/home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp'

Detecting CXX compiler ABI info compiled with the following output:
Change Dir: /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp

Run Build Command:/usr/bin/make "cmTryCompileExec2300559841/fast"
/usr/bin/make -f CMakeFiles/cmTryCompileExec2300559841.dir/build.make CMakeFiles/cmTryCompileExec2300559841.dir/build
make[1]: Entering directory '/home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp'
/usr/bin/cmake -E cmake_progress_report /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp/CMakeFiles 1
Building CXX object CMakeFiles/cmTryCompileExec2300559841.dir/CMakeCXXCompilerABI.cpp.o
/usr/bin/c++     -o CMakeFiles/cmTryCompileExec2300559841.dir/CMakeCXXCompilerABI.cpp.o -c /usr/share/cmake-2.8/Modules/CMakeCXXCompilerABI.cpp
Linking CXX executable cmTryCompileExec2300559841
/usr/bin/cmake -E cmake_link_script CMakeFiles/cmTryCompileExec2300559841.dir/link.txt --verbose=1
/usr/bin/c++      -v CMakeFiles/cmTryCompileExec2300559841.dir/CMakeCXXCompilerABI.cpp.o  -o cmTryCompileExec2300559841 -rdynamic
Using built-in specs.
COLLECT_GCC=/usr/bin/c++
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/4.9/lto-wrapper
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 4.9.1-16ubuntu6' --with-bugurl=file:///usr/share/doc/gcc-4.9/README.Bugs --enable-languages=c,c++,java,go,d,fortran,objc,obj-c++ --prefix=/usr --program-suffix=-4.9 --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --with-gxx-include-dir=/usr/include/c++/4.9 --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --enable-gnu-unique-object --disable-vtable-verify --enable-plugin --with-system-zlib --disable-browser-plugin --enable-java-awt=gtk --enable-gtk-cairo --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-4.9-amd64/jre --enable-java-home --with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-4.9-amd64 --with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-4.9-amd64 --with-arch-directory=amd64 --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --enable-objc-gc --enable-multiarch --disable-werror --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib --with-tune=generic --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu
Thread model: posix
gcc version 4.9.1 (Ubuntu 4.9.1-16ubuntu6)
COMPILER_PATH=/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/:/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/
LIBRARY_PATH=/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/:/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib/:/lib/x86_64-linux-gnu/:/lib/../lib/:/usr/lib/x86_64-linux-gnu/:/usr/lib/../lib/:/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../:/lib/:/usr/lib/
COLLECT_GCC_OPTIONS='-v' '-o' 'cmTryCompileExec2300559841' '-rdynamic' '-shared-libgcc' '-mtune=generic' '-march=x86-64'
 /usr/lib/gcc/x86_64-linux-gnu/4.9/collect2 -plugin /usr/lib/gcc/x86_64-linux-gnu/4.9/liblto_plugin.so -plugin-opt=/usr/lib/gcc/x86_64-linux-gnu/4.9/lto-wrapper -plugin-opt=-fresolution=/tmp/cce10SSD.res -plugin-opt=-pass-through=-lgcc_s -plugin-opt=-pass-through=-lgcc -plugin-opt=-pass-through=-lc -plugin-opt=-pass-through=-lgcc_s -plugin-opt=-pass-through=-lgcc --sysroot=/ --build-id --eh-frame-hdr -m elf_x86_64 --hash-style=gnu --as-needed -export-dynamic -dynamic-linker /lib64/ld-linux-x86-64.so.2 -z relro -o cmTryCompileExec2300559841 /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crt1.o /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crti.o /usr/lib/gcc/x86_64-linux-gnu/4.9/crtbegin.o -L/usr/lib/gcc/x86_64-linux-gnu/4.9 -L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu -L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib -L/lib/x86_64-linux-gnu -L/lib/../lib -L/usr/lib/x86_64-linux-gnu -L/usr/lib/../lib -L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../.. CMakeFiles/cmTryCompileExec2300559841.dir/CMakeCXXCompilerABI.cpp.o -lstdc++ -lm -lgcc_s -lgcc -lc -lgcc_s -lgcc /usr/lib/gcc/x86_64-linux-gnu/4.9/crtend.o /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crtn.o
make[1]: Leaving directory '/home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp'

Parsed CXX implicit link information from above output:
  link line regex: [^( *|.*[/\])(ld|([^/\]+-)?ld|collect2)[^/\]*( |$)]
  ignore line: [Change Dir: /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp]
  ignore line: []
  ignore line: [Run Build Command:/usr/bin/make "cmTryCompileExec2300559841/fast"]
  ignore line: [/usr/bin/make -f CMakeFiles/cmTryCompileExec2300559841.dir/build.make CMakeFiles/cmTryCompileExec2300559841.dir/build]
  ignore line: [make[1]: Entering directory '/home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp']
  ignore line: [/usr/bin/cmake -E cmake_progress_report /home/arnaud/build-ifcplusplus-master_berndhahnebach-Desktop-Default/CMakeFiles/CMakeTmp/CMakeFiles 1]
  ignore line: [Building CXX object CMakeFiles/cmTryCompileExec2300559841.dir/CMakeCXXCompilerABI.cpp.o]
  ignore line: [/usr/bin/c++     -o CMakeFiles/cmTryCompileExec2300559841.dir/CMakeCXXCompilerABI.cpp.o -c /usr/share/cmake-2.8/Modules/CMakeCXXCompilerABI.cpp]
  ignore line: [Linking CXX executable cmTryCompileExec2300559841]
  ignore line: [/usr/bin/cmake -E cmake_link_script CMakeFiles/cmTryCompileExec2300559841.dir/link.txt --verbose=1]
  ignore line: [/usr/bin/c++      -v CMakeFiles/cmTryCompileExec2300559841.dir/CMakeCXXCompilerABI.cpp.o  -o cmTryCompileExec2300559841 -rdynamic  ]
  ignore line: [Using built-in specs.]
  ignore line: [COLLECT_GCC=/usr/bin/c++]
  ignore line: [COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/4.9/lto-wrapper]
  ignore line: [Target: x86_64-linux-gnu]
  ignore line: [Configured with: ../src/configure -v --with-pkgversion='Ubuntu 4.9.1-16ubuntu6' --with-bugurl=file:///usr/share/doc/gcc-4.9/README.Bugs --enable-languages=c,c++,java,go,d,fortran,objc,obj-c++ --prefix=/usr --program-suffix=-4.9 --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --with-gxx-include-dir=/usr/include/c++/4.9 --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --enable-gnu-unique-object --disable-vtable-verify --enable-plugin --with-system-zlib --disable-browser-plugin --enable-java-awt=gtk --enable-gtk-cairo --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-4.9-amd64/jre --enable-java-home --with-jvm-root-dir=/usr/lib/jvm/java-1.5.0-gcj-4.9-amd64 --with-jvm-jar-dir=/usr/lib/jvm-exports/java-1.5.0-gcj-4.9-amd64 --with-arch-directory=amd64 --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --enable-objc-gc --enable-multiarch --disable-werror --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib --with-tune=generic --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu]
  ignore line: [Thread model: posix]
  ignore line: [gcc version 4.9.1 (Ubuntu 4.9.1-16ubuntu6) ]
  ignore line: [COMPILER_PATH=/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/:/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/]
  ignore line: [LIBRARY_PATH=/usr/lib/gcc/x86_64-linux-gnu/4.9/:/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/:/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib/:/lib/x86_64-linux-gnu/:/lib/../lib/:/usr/lib/x86_64-linux-gnu/:/usr/lib/../lib/:/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../:/lib/:/usr/lib/]
  ignore line: [COLLECT_GCC_OPTIONS='-v' '-o' 'cmTryCompileExec2300559841' '-rdynamic' '-shared-libgcc' '-mtune=generic' '-march=x86-64']
  link line: [ /usr/lib/gcc/x86_64-linux-gnu/4.9/collect2 -plugin /usr/lib/gcc/x86_64-linux-gnu/4.9/liblto_plugin.so -plugin-opt=/usr/lib/gcc/x86_64-linux-gnu/4.9/lto-wrapper -plugin-opt=-fresolution=/tmp/cce10SSD.res -plugin-opt=-pass-through=-lgcc_s -plugin-opt=-pass-through=-lgcc -plugin-opt=-pass-through=-lc -plugin-opt=-pass-through=-lgcc_s -plugin-opt=-pass-through=-lgcc --sysroot=/ --build-id --eh-frame-hdr -m elf_x86_64 --hash-style=gnu --as-needed -export-dynamic -dynamic-linker /lib64/ld-linux-x86-64.so.2 -z relro -o cmTryCompileExec2300559841 /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crt1.o /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crti.o /usr/lib/gcc/x86_64-linux-gnu/4.9/crtbegin.o -L/usr/lib/gcc/x86_64-linux-gnu/4.9 -L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu -L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib -L/lib/x86_64-linux-gnu -L/lib/../lib -L/usr/lib/x86_64-linux-gnu -L/usr/lib/../lib -L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../.. CMakeFiles/cmTryCompileExec2300559841.dir/CMakeCXXCompilerABI.cpp.o -lstdc++ -lm -lgcc_s -lgcc -lc -lgcc_s -lgcc /usr/lib/gcc/x86_64-linux-gnu/4.9/crtend.o /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crtn.o]
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/collect2] ==> ignore
    arg [-plugin] ==> ignore
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/liblto_plugin.so] ==> ignore
    arg [-plugin-opt=/usr/lib/gcc/x86_64-linux-gnu/4.9/lto-wrapper] ==> ignore
    arg [-plugin-opt=-fresolution=/tmp/cce10SSD.res] ==> ignore
    arg [-plugin-opt=-pass-through=-lgcc_s] ==> ignore
    arg [-plugin-opt=-pass-through=-lgcc] ==> ignore
    arg [-plugin-opt=-pass-through=-lc] ==> ignore
    arg [-plugin-opt=-pass-through=-lgcc_s] ==> ignore
    arg [-plugin-opt=-pass-through=-lgcc] ==> ignore
    arg [--sysroot=/] ==> ignore
    arg [--build-id] ==> ignore
    arg [--eh-frame-hdr] ==> ignore
    arg [-m] ==> ignore
    arg [elf_x86_64] ==> ignore
    arg [--hash-style=gnu] ==> ignore
    arg [--as-needed] ==> ignore
    arg [-export-dynamic] ==> ignore
    arg [-dynamic-linker] ==> ignore
    arg [/lib64/ld-linux-x86-64.so.2] ==> ignore
    arg [-zrelro] ==> ignore
    arg [-o] ==> ignore
    arg [cmTryCompileExec2300559841] ==> ignore
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crt1.o] ==> ignore
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crti.o] ==> ignore
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/crtbegin.o] ==> ignore
    arg [-L/usr/lib/gcc/x86_64-linux-gnu/4.9] ==> dir [/usr/lib/gcc/x86_64-linux-gnu/4.9]
    arg [-L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu] ==> dir [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu]
    arg [-L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib] ==> dir [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib]
    arg [-L/lib/x86_64-linux-gnu] ==> dir [/lib/x86_64-linux-gnu]
    arg [-L/lib/../lib] ==> dir [/lib/../lib]
    arg [-L/usr/lib/x86_64-linux-gnu] ==> dir [/usr/lib/x86_64-linux-gnu]
    arg [-L/usr/lib/../lib] ==> dir [/usr/lib/../lib]
    arg [-L/usr/lib/gcc/x86_64-linux-gnu/4.9/../../..] ==> dir [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../..]
    arg [CMakeFiles/cmTryCompileExec2300559841.dir/CMakeCXXCompilerABI.cpp.o] ==> ignore
    arg [-lstdc++] ==> lib [stdc++]
    arg [-lm] ==> lib [m]
    arg [-lgcc_s] ==> lib [gcc_s]
    arg [-lgcc] ==> lib [gcc]
    arg [-lc] ==> lib [c]
    arg [-lgcc_s] ==> lib [gcc_s]
    arg [-lgcc] ==> lib [gcc]
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/crtend.o] ==> ignore
    arg [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu/crtn.o] ==> ignore
  remove lib [gcc_s]
  remove lib [gcc]
  remove lib [gcc_s]
  remove lib [gcc]
  collapse library dir [/usr/lib/gcc/x86_64-linux-gnu/4.9] ==> [/usr/lib/gcc/x86_64-linux-gnu/4.9]
  collapse library dir [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../x86_64-linux-gnu] ==> [/usr/lib/x86_64-linux-gnu]
  collapse library dir [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib] ==> [/usr/lib]
  collapse library dir [/lib/x86_64-linux-gnu] ==> [/lib/x86_64-linux-gnu]
  collapse library dir [/lib/../lib] ==> [/lib]
  collapse library dir [/usr/lib/x86_64-linux-gnu] ==> [/usr/lib/x86_64-linux-gnu]
  collapse library dir [/usr/lib/../lib] ==> [/usr/lib]
  collapse library dir [/usr/lib/gcc/x86_64-linux-gnu/4.9/../../..] ==> [/usr/lib]
  implicit libs: [stdc++;m;c]
  implicit dirs: [/usr/lib/gcc/x86_64-linux-gnu/4.9;/usr/lib/x86_64-linux-gnu;/usr/lib;/lib/x86_64-linux-gnu;/lib]
  implicit fwks: []


Not anymore, removed.
Thanks


1: M190 line is too long (52 > 50)


@lamagnifica  , I'm sorry I may be missing this -- is there a note here that mentions that non-Never settings of Randomization will make the Answer Distribution report confusing, especially for non-numeric problems?


Ok, thanks.


Like this?
https://github.com/GirlsCodersWarsaw/CakeMaster_Ania/commit/2b65a9d21878b3fc99cbed7fb4bc06eb5a38fcea


Thanks


s/Inc/Inc./  please


@mondok A completion handler is required here otherwise there's no reason to call the method. I added nullability keywords so consumer should get a compiler warning if they're not using the API correctly and a hard crash if they ignore the warning. I prefer hard crashes for programmer mistakes like this. Thoughts?


Why not mention " warning, filename not found. We're using a default config file which should give you some basic analysis of your project"

-----Original Message-----
From: "Makman2" <notifications@github.com>
Sent: 14-07-2015 19:26
To: "coala-analyzer/coala" <coala@noreply.github.com>
Cc: "AbdealiJK" <abdealikothari@gmail.com>
Subject: Re: [coala] main: Refactor warnings when no coafile existent(c690300)

Since we want to support some defaults for quick linting I think the message is not that bad, we just need to specify the defaults in our doc.
—
Reply to this email directly or view it on GitHub.


You don't need to put everytime at cases { } because that break; will leave that case.


missing semicolon at end of line


I don't get it. Isn't `906609` the biggest palindrome? and if so, isn't it the first one encountered if numbers are going from 1000 downwards?


точнее вот что.


@guanlisheng You mean append it at the end? (Like: push_back after all other currencies were added?)


On the first version of this PR I had removed but Gustavo asked to put back, we can always add a fake contiki.h and add to cflags and be able to generate a configuration with Contiki platform and mainloop.


does this really need `&mut self`?


ah, of course, for path compression. ignore me.


@edgardmessias what is the difference between your extension and the old one forked by @Romariosss ?


Erm...maybe device tree needs to confirm if its good to go.

For your reference with my device tree commit
https://github.com/temasek/android_device_samsung_hlte-common/commit/3d242e371728edeef1909fefd62265e70199c84f


BTW, the fact that the special properties and generic properties are set in different passes means that `zfs_set_prop_nvlist` is still non-atomic even if we set the corresponding parameter to true.
Maybe we should given up on the idea of setting multiple properties atomically and support setting only one property at a time?
Non-atomicity is okay for the creation case, of course, because we can just destroy the dataset.


Yeah not bad, but the problem is we don't analyze something by default right now :(


I like this approach with the warning  :+1:


JSX and ES6 processing is done by only webpack. react-rails gem does not know where your javascript files are. It only looks at files that are referenced by component.js and there is nothing there for the gem to compile because everything is already compiled in client-bundle.
The two lines here
//= require react
//= require react_ujs
are needed for react-rails gem to inject the markup in the right position when rendering client side. In fact, you can comment these two lines and it will still work for server side rendering.


Don't we need a corresponding test for the `get_for_service_account_json()` function?


I don't have any crashes if I disable hair.

But I don't have sound effects either, i.e. only `playstream` produce sound and `playsound` is silent.


what does this fix exactly?


can you clarify that this does not apply to int literal variables etc?


@hodgestar Read for review


Just for interest, how do I get whether `--save` is set? :/


This does not compile.  It comes up as a SASS error

```bash
Error: Invalid CSS after "body:not(": expected ")", was "".admin") .erro..."
        on line 222 of sass-modules/_panels.scss
        from line 15 of screen.scss
  Use --trace for backtrace.
```


I've tested it on a local build on Mac OSX 10.10.4 on Safari, FireFox, Chrome, and Opera and it seems to work fine in all these cases.  I have not tested it on PC, but since the code came from the typekit project, I am inclined to trust it.

Functionally, I don't know that it is superior to just using the direct <link> to fonts.googleapis that TWiStErRob showed.  The only thing that this gives you is the ability to use webfontloader hooks if you want.

Since it is no longer asynchronous, it does delay the display of the text until the fonts get loaded.  Simulating a 50Kbps connection, it didn't seem bad and still seemed less disconcerting than the FOUC, especially after the initial caching.

I'm happy to contribute as I can.  We're using this for our team blog at tech.popdata.org, so I'm happy to share anything that seems general enough to benefit your project.

I've not really done active github contributions before, so if I misstep, I apologize - for instance, I think I wasn't supposed to delete my branch of the footnote fix before you had a chance to review it.


@helios-ag could you please also create a new tag e.g. 1.0.1 ? i'm using helios-ag/fm-elfinder-bundle and it did not load new changes you merged.


Does this work considering what you were saying earlier...?


It is too late to change the behavior of snf-image now. We cannot limit the set of values for which a property is considered True. The code has already made a choice, once a property is defined, no matter *what* its value, it is considered True.

This makes absolute sense and is the intuitive thing to do. Also consider a user which already has a property set to 'true', or 'True', or 'SET', or 'set', or 'OK'. Their Image has been working great so far. Making such change will break backwards compatibility with numerous Images.

The only logical thing to do is to fix the documentation so it describes the behavior of the code accurately, especially since this behavior is also intuitive.

About the novice user deploying Windows XP: The Image must have already undergone processing through SYSPREP. This is the part that needs to be automated, it does not make sense to insert extra, complicated code to detect the Windows version just to enable OFFLINE_NTFSRESIZE automatically. We can expect that a user who can run SYSPREP can also read a clear warning in the Image deployment log. Alternatively, an automated program which we expect has already run SYSPREP while preparing the Image can also set this property.

In general, I think automatically determining the value of missing Image properties in runtime is the wrong approach. In the interest of security and simplicity, snf-image should minimize the number of things it has to guess. At Image build time, the automated program which we assume prepares the Image initially can use all sorts of heuristics to determine sane values for Image properties.

So, I would propose to move forward with this patch and then I can contribute patches to fix the documentation so it is synchronized with the code, in a separate Pull Request.


I guess this resolves my [comment](https://github.com/GoogleCloudPlatform/gcloud-python/pull/988#commitcomment-12156120) from 8b1ecb2


charset utf8, no?


This commit LGTM.


any particular reason you commented these out?


I had not expected a negative performance impact. But I also had not expected the impact of integer operation on the Dslash to be that noticeable. Anyway, for multi-GPU that is clearly not a performance relevant thing.
I was more being curious about the positive effects and I am always in favor keeping things in higher precision where it does not have a negative effect on performance / memory pressure.

Btw. it’s the same way as I had it in the Bielefeld code ;-)


here too


@CSRedRat да, проморгал этот момент. Конечно GitHub.


cool :)


bool(section["save"]) --save will simply translate into a setting with the
save key

2015-07-14 16:05 GMT+02:00 Makman2 <notifications@github.com>:

> Just for interest, how do I get whether --save is set? :/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/c6903009db1132779ef86fbc9071a78d70148037#commitcomment-12156457>
> .
>


I think we want to include the bam file name as a column in this? Also, can we add a header row to the csv?


Nit: can you put the types in backticks? e.g.,

    "conflicting type parameter defaults `{}` and `{}`"


backticks here too


"Для подключения укажите название библиотеки и имена модулей или пути к ним для модульных систем."
Предложение читается сложно:
"к ним" == библиотекам, именам модулей или к обоим ?


I would probably format this like:

    "...a default was defined here..."


ack :)


This commit LGTM


ack if CI passes


similarly. You could maybe also drop the leading `...` here.


This change was left here due to rebase issues. Fixed. This should be covered by https://github.com/kborchers/react-globalize/pull/19


Sure, I can add the bam file name as a column.

I didn't want to add the header as this still creates an HDFS file, where we don't want the header if loading it elsewhere. We can add a second option to create a local CSV with header?


This needs further revision: taking indices 0 and 1 for a URL string formatted like "http://foo.bar.com:8080" (as it is in localize.py) will give you "http" and "//foo.bar.com".


Runs the command in a subshell, to separate the actual command from our output captures.


Why the reacharound?


@moyni has the global version of this line been removed? It's not coming up on my screen but I may have missed it. Other than that looks good to me!


It builds, but I am not sure if everything works as it should :)


**Beautiful!** Saves me having to update this plugin myself! Vertical split for the win! :trophy: :+1: Thanks!


Hi @mkormendy - it's simply to gain access to the object instance from within other functions / methods.


Ok, thank you. I'll remove this option.


Nit: seems like you should take `Option<&ty::TypeParameterDef<'tcx>>` (note the `&`)


@jesssummerfield - it's the overall BD category, as generalist & global lead were the only two roles in.


No - `cloudman-test` is about 3 weeks behind tip now and contains the RC1 code. `cloudman-dev` bucket will (most of the time) have the tip (or the development version not yet committed) but it can also (& often) have broken code there as well.

I had sent you the instructions and scripts to easily create your own bucket with any modifications so I thought you were using that. I've since then added that bit of documentation here as well: https://wiki.galaxyproject.org/CloudMan/CustomizeGalaxyCloud#Using_custom_CloudMan_application


You need to roll it forward, not backward. Also the comment should be indented.


The variable could be set from Crosswalk itself to avoid forcing users to do it.


does it matter here?
otherwise sure.


906609 is the biggest palindrome for the 3 digit number. And no...is not the first encountered.I mean, if I go with both numbers, the first palindrome that will encounter will be 580085 because the first two number would be 995 and 583.
995 is bigger than 993, but that doesn't prove that the first number that will encounter will also be the biggest.

For 4 digits, accidentally works, the first found is the biggest. But for 3 digits I had different results.
(There is a probabilty that I didn't really understood the question, my bad then).


Wrong hash: this is the same hash that is in Chromium's `DEPS`. You are supposed to roll it forward so that it contains the commit that introduces the `ffmpeg_component variable`.


Delete this commented out code?


Wrong approach. The build was failing just because `media::InitializeMediaLibraryForTesting()`. Does not exist anymore. Looking at the original commit upstream, it is easy to see that the calls just need to be replaced by `media::InitializeMediaLibrary()`.


Ditto.


I see the comment below:  this feels wrong (maybe we should have some mechanism for finding the `http` bit cleanly).


Instead of pasting some SHA, could you paste here the same changelog message from previous patched version? This `ChangeLog` file is intended for end users.


Removed by https://github.com/danmar/cppcheck/commit/fa03a93f892af3272c91df729fcdd1a5407001dd


Not really, just to maintain standard
On 14 Jul 2015 16:15, "Rochet2" <notifications@github.com> wrote:

> does it matter here?
> otherwise sure.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Rochet2/TrinityCore/commit/6db9fc6d3f6f4b77833876676472945813ee8d15#commitcomment-12156726>
> .
>


I've left a few inline comments above for future reference. I'll fix them myself and try to land a newer M44 than .61.


We are planning on making changes to the buffering logic in this area to optimize POST scenarios and will be exposing the buffered stream via a different internal API.  I would suggest holding this PR until then.


I get a division by zero if I set `bins > 100`. If possible it'd be good to do the binning without rounding to nearest percent first.


Nice effort here. :)


Time to switch to ESLint?


Why you did this?


Probably we should in general track the cause of a unification for type-variables, which would obviate the need for this search. But this is good for now.


esta linea sobra, no? que no se esta usando `new_entry` para nada :)


I don't like using the "real" client here, especially as we have to stub out other things which aren't needed.  Setting `_connection` on it just ices the cake.


funny indentation?


How this works? The new method has to be manually added into .rst? I was expecting this to be autogenerated from docstring - that means do it differently than how it is on DNF project.


The csv output is slightly confusing, as the bins are things like `5-9` followed by `10-14`. What if my VAF is between 9 and 10, which bin does it get assigned to? Probably want the csv output to look like `5-10`, `10-15`, etc.


I am guessing you did not mean to modify which version of the rust-installer we use.


This is the README.md of my test repository.


Hello


I see what is wrong. I had not considered the possibility of recursively searching for snapshots by specifying non-snapshots. Your change would have the the side effect of breaking things when `-d $N` is used rather than -r. The snapshots are technically 1 level below the datasets. I will attach gdb and see why the traversal does not want to go down 1 more level.


remove commented lines


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


seems unnecessary.


Is this update not in the `bower` version?
I reinstall `ng-admin` by bower. I find `ng-admin-only.min.js` has not exists.
And I run my App. I can‘t find `updateMethod`  in my entity object.

Regards.


Hi @lefeverd, thank you for your comment. Can you create an issue for this, mentioning me? Thanks.


I think a simple test case for this command would be worthwhile


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


normalmente querremos borrar un entry por los datos que tiene (o teniendo una instancia de entry), no por su posición en donde lo guardamos (que debería ser transparente al usuario). Por lo tanto la implementación de `remove_entry` debería ser algo distinta. Que opinas de algo así? `entry` es una instancia de `Entry`

```
def remove_entry(entry)
si no existe entry en entries, return
buscar por nombre, por ejemplo, y eliminar de entries la entry con name = entry.name
return entries
end
```


Call it calculationStartDate


lastCalculationDate


Shouldn't this follow the [ZCML Style Guide](http://docs.zope.org/zopetoolkit/codingstyle/zcml-style.html)? That'd be 2 spaces for ident but 4 for attribute listing.


You are right, I mixed the assignment requirements and thought of something else.
It's fine


We shouldn't be targeting `.red` -- and this shouldn't have a colour value? (that's the point of the red class)


Can you use `python3-shinx`, please? RPG project is based on Python3 only so far.


Doesn't it also need insertchar function?


:)
do you also have links to commits added in cm12.1, for this to work?


@mariuz
I had helped develop for Yii1.
I developed from scratch, to be supported 100% of Yii2. Also I added all unit tests Yii2.
Only lack a few things to finish, as truncateTable and alterColumn for use in migration. All functions for the ActiveRecord are working


Iniciando con NodeJs


I'm not sure the descriptive statistics stuff in `variantLociFromReads` is necessary. Doesn't estimating the histogram essentially capture all of that information already?


If we have multiple same import in the same file, let's move it to the top


hmm? isnt that Hercules have it since very long ago ??
https://github.com/HerculesWS/Hercules/blob/master/src/map/script.c#L14703

Been using this function long ago in my test server, just wondering why Hercules doesn't add it.


On 14-07-15 15:46, Cornelis G. A. Kolbach wrote:
> Folder name and description
>
> A long outstanding wish from my side to include folder descriptions in
> item-selector. Also used this area for folder name (to a point of
> @maartenkling), which is useful for use cases when you for instance end
> up on a folder though a mailed link and not through clicking your way
> there.
>
> @gyst, Approved?

Yes! This fixes an important criticism.

--
    Guido Stevens  |  +31.43.3618933  |  http://cosent.nl

    s o c i a l   k n o w l e d g e   t e c h n o l o g y


"универсальный" и "совместимый с обоими шаблонизаторами" -- это не одно и то же свойство? универсальность (из контекста) сводится именно к тому, что его можно использовать в обоих шаблонизаторах.
" по факту выполнять шаблоны BH."
- выполнять шаблоны == выполнять СБОРКУ шаблонов?
- что здесь прячется за фразой "по факту"?
+ тут однокоренные "используется/использовать в соседних предложениях.


Open an issue. Not up to me.


Wilbur,

after these changes sensors on cabinets disappeared. Double-check attentively, please.

Very thanks for the support.


hey, I did the same thin in ajk/pylint a few hrs back.

Would be good to rebase first


I don't think it matters whether or not you keep the remote branches. I do the same thing in the FreedomSoft repo. I wonder if the problem lies in the branch being committed to? Maybe since I haven't committed anything to the `game-player-associations` branch, it doesn't notify me since it probably doesn't concern me.

I'll do some digging and let you know what I find.


I don't think there's much we have to do in `onCreate` and `onUpdate`, you can read more about their functionality [here](http://stackoverflow.com/questions/21881992/when-is-sqliteopenhelper-oncreate-onupgrade-run).

If anything, it might be best to delete the database on `onUpgrade`, since SQLite is really just a cache of the cloud DB anyways.


Is this a serious pull request? Where is the line wrap?


Good, that kills 2 birds with one stone then. A fairly small stone even :).


Fair enough - if I apply `.description` (https://github.com/halfdan/Ghost/blob/59188f7f19c2241b1b7b20c137539423a2317a7f/core/client/app/styles/layouts/users.css#L103-L109) though the `.red` class doesn't work (since .description is more specific).


OK. I pull it in and have a deeper look, if I have the time. Thank you very much and I am happy to see, how you guys use the theme.

Thanks for the link :)


不用翻译(#) 里头的内容哦 ;-)


I guess I could define #a1a1a1 as --grey and move it out of .description


I really don't understand what I am doing wrong. I will go through it carefully again.


多进程创建模块


Gotcha. Sure, making the header optional works for me. My use case is just to pull this tiny file into a pandas dataframe and have the columns labelled.


This code was all written in a local repo before I got access to this repo. When I got access, I copied my code over in this commit.


You MUST check result `TypeQueryVisitor::as<StringElement>(e.value.first)` against NULL.
It works in simmilar way like `dynamic_cast<>`
```
if (StringElement* s = TypeQueryVisitor::as<StringElement>(e.value.first)) {
  assign(...);
}
else { // key is not string - handle error
}
```


Okay, perfect. Thanks Seb!


modulo comments this LGTM -- thanks for working on this, @arahuja !


`height` is the only variable here, the rest are "static" style properties that could be put directly in the markup (part of a template). Then this class wouldn't be necessary.


This might be the issue: https://help.github.com/articles/receiving-email-notifications-for-pushes-to-a-repository/

It looks like you, the repository owner, have access to the repository settings. Can you check to see if the repo is set up the way the help article explains?


Yeah. This was from the original bglib so I hadn't changed it.


TAK SIĘ NIE ROBI


wyżej


都发完了才开始 Review ;-) 这个咱们可以尽快搞一个网站起来～ 这个方式很不错的说。


For python: "Floating point numbers are usually implemented using double in C; information about the precision and internal representation of floating point numbers for the machine on which your program is running is available in sys.float_info."

https://docs.python.org/2/library/stdtypes.html


We could use https://docs.python.org/2/library/os.path.html


i thought the same thing.


1: M111 unrecognized component name: legacy


This document used to outline policy related to x and seek assistance from y to help improve it.


Pataisyta


See b994c1c0fd19369485837f53e3137c438a934da6


By data API I mean, what is the type of `data`:
> @return `Array<Hash{Symbol=>String}>` where the keys are `:key` and `:value`


Too many blank lines.


I think this should be enough.


@varund7726 ask falcon and titan users if someone can test a fix for ota, if so then ill give them an apk


Hey here!


maybe I'm misunderstanding this, but it looks like we're writing the histogram in parallel? I'd suggest might be easier to always write it from the driver -- it's super tiny.


No problem - all the best!


I found the issue and just pushed up a correction.  I reused a device object that made the sql just completely wrong.  This might take more testing to get dialed in just right.  @eab8f4b


I have uncommented  the debug log lines. This is the content of the log file:

loading app /home/ubuntu/code/RyuApp/ControllerSTP.py
loading app ryu.controller.dpset
loading app ryu.topology.switches
loading app ryu.controller.ofp_handler
instantiating app None of Stp
creating context stplib
instantiating app ryu.topology.switches of Switches
instantiating app /home/ubuntu/code/RyuApp/ControllerSTP.py of SimpleSwitch13
instantiating app ryu.controller.dpset of DPSet
instantiating app ryu.controller.ofp_handler of OFPHandler
OFPSwitchFeatures received:
	datapath_id=0x0000000000000001 n_buffers=256
	n_tables=254 auxiliary_id=0
	capabilities=0x0000004f
OFPSwitchFeatures received:
	datapath_id=0x0000000000000002 n_buffers=256
	n_tables=254 auxiliary_id=0
	capabilities=0x0000004f
OFPSwitchFeatures received:
	datapath_id=0x0000000000000003 n_buffers=256
	n_tables=254 auxiliary_id=0
	capabilities=0x0000004f
	 ***switch dpid=0000000000000001
 	 port_no=1 hw_addr=02:b7:00:7b:6b:13 name=s1-eth1 config=0x00000040
 	 state=0x00000000 curr=0x00000840 advertised=0x00000000
 	 supported=0x00000000 peer=0x00000000 curr_speed=10000000 max_speed=0
hub: uncaught exception: Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/ryu/lib/hub.py", line 52, in _launch
    func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/ryu/topology/switches.py", line 821, in link_loop
    self.links.link_down(link)
  File "/usr/local/lib/python2.7/dist-packages/ryu/topology/switches.py", line 329, in link_down
    del self._map[link.src]
KeyError: <ryu.topology.switches.Port object at 0x7fd8510c3210>


:abc:?


:abc:? (and elsewhere in this file)


I guess for POC, it is ok to use `range`. Let's say we want to build a SaaS. `range` will use a lot of mem later on. So better use lazy approach, like `xrange` or generator.


1) If a user has misused the image properties, then he is to blame. The documentation stated it clearly.
2) We are already using get_distro() is order to determine how to assign the hostname. We could in the future let the user assign this as image property and use the function only if this is not available or its value is not functional.

Please proceed with the already mentioned changes in order to have the patches merged.


Ok, my bad


don't worry about updating this, I've got it.


//Commiting the REST API for Health Statistics UI// is not a very descriptive commit message. Rather you should be describing what the changes in this commit are.


>"универсальный" и "совместимый с обоими шаблонизаторами" -- это не одно и то же свойство?
здесь имелось в виду писать одинаковый JavaScript-код для обоих шаблонизаторов.

>выполнять шаблоны == выполнять СБОРКУ шаблонов?
не одно и то же.

> что здесь прячется за фразой "по факту"?
по факту = при выполнении шаблонов (думаю, можно убрать)

>тут однокоренные "используется/использовать в соседних предложениях.
заменю, спасибо.


yes, they are. fixed.


Updated PR.


Yes. I got the scripts. I need to focus on the data analysis more now and try to find a stable way for my analysis environment. I'm trying to balance new features with stability and decided to wait with galaxy on dev until the new release in a few weeks.
For the mounting of another drive, I will try to use `cloudman-dev` for now. Can I also attach and mount an existing EBS instead of a snapshot?


Nice! Just curious where you fund this info? :)


1. 使用gdb调试linux内核 --> 使用 gdb 调试 Linux 内核 （中英混排时在英文周边加空格，显得视野更舒适一些）
2. (#使用gdb...) --> 修改回英文原来的链接（这部分需要在后面自动构建中文目录的内部链接，gitbook暂时不能友好支持中文内链）


previous indent had a mix of tabs and spaces - so coverted to space along with the use of "L"


it is better to import specific module. similar to java, we don't want to import * a lot :)


minor formatting. spaces between = sign.


@moyni ohhh ok - ignore me! In that case, all looks good, thank you!


Will we also include a field for the folder ID? If I created my folder as "Projection Materials", the ID is projection-materials, and if I change that to "Forecast Papers", I will want the ID to follow suite - sometimes.


Yes it is, since that's a more standard workflow than writing to local in Hadoop.  Without NFS the local file would be lost on some cluster machine.  We can have a separate local option.


I think the following is the proper fix. The compiler is pointing a problem related to libgcov, it seems it's missing a link against it. Try the following patch:

```patch
diff --git a/tools/build/Makefile.vars b/tools/build/Makefile.vars
index 52f34f5..ad0c495 100644
--- a/tools/build/Makefile.vars
+++ b/tools/build/Makefile.vars
@@ -183,6 +183,7 @@ endif

 ifneq (,$(filter %coverage,$(MAKECMDGOALS)))
 COMMON_CFLAGS += -fprofile-arcs -ftest-coverage
+COMMON_LDFLAGS += -lgcov
 endif

 DEP_RESOLVER_CFLAGS := $(CFLAGS) -Werror=implicit-function-declaration

```


Actually, I think we should just use $ScriptBlock as is, not calling NewBoundScriptBlock or Ast.GetScriptBlock.  That wouldn't have worked before when some helper functions were internal, but now that they are exported, it should work better for modules that register their own completers.


@yiqinglan do you have an example for this? Scrolling works for me in 1.0.1. Please open an issue and provide a codepen example


"Для правильного функционирования не требует подключения исходных файлов шаблонов."
Сейчас акцент на первой части (если будет иначе -- не будет правильно функционировать).
Именно это и имелось в виду, а не то, что может функционировать, вне зависимости от того подключены исходные файлы шаблонов или нет?


Nece da radi kada koristis apsolutnu putanju.


@Thilas, if you look the rest of extension method I've changed all reflection query with `T`.

As you know `X is Y` does exactly the same thing of `typeof(X).IsAssignableFrom(Y)`.

So if for some reason a developer want to invoke `WithParsed<T>` using a base type of `Options` the check should passes.
May be it's not common to have:
```csharp
abstract class OptionsBase { }
class Options : OptionsBase { }
```
in options scenario, but without such change:
```csharp
var result = ...
result.WithParsed<OptionsBase>(...)
```
will fail. Or am I missing something?

For `WithNotParsed<T>` the thing is exactly the same (also if the instance will not be accessed)... I'm taking advantage of the internal `Value` field loaded with a default empty instance (for the moment something needed to help subsystem) to make the check.

Is just for coherency with verbs overloads.


File to open a Shader wirted in GLSL.


> I'm not sure the descriptive statistics stuff in variantLociFromReads is necessary. Doesn't estimating the histogram essentially capture all of that information already?

I've been using that a bit as  a summary of histogram, I think both in tandem are pretty standard, histogram and marker of percentiles.


Hi,

i) so for Windows operating system, we have to open a separate section somewhere here,
https://autocmake.readthedocs.org/en/latest/ , right ?

ii) Git tools are needed by developers.  I think this new section could be separated into two parts: one for users, and one for git equipped developers. Users could be advised to download the file by a web-browser.

Comments, please ?


On the change form you mean. Yes, but shouldn't that be automatic? Plone has this nice forwarding feature anyway until the old ID is taken by a new object.

> Op 14 jul. 2015, om 16:40 heeft Alexander Pilz <notifications@github.com> het volgende geschreven:
>
> Will we also include a field for the folder ID? If I created my folder as "Projection Materials", the ID is projection-materials, and if I change that to "Forecast Papers", I will want the ID to follow suite - sometimes.
>
> —
> Reply to this email directly or view it on GitHub.
>


@sevko look who it is! thanks for the fix. hope you're having a great summer!


Typescript-angluar: update `todomvc-common` to latest
=>
Typescript-angular: update `todomvc-common` to latest


:100: open source


Should disable before merge ?


@Thilas,
now I got... You're right for options scenario, the check is completely redundant!  I'll remove it... Thank you.


Yeah - we should talk about how those buttons work.


shouldn't offset be optional?


Not like that's gonna confuse anybody :p


We don't need to use HttpServletResponse class here. Please refer to other places in StratosApiV41 [1] to see how it is done.

[1] - https://github.com/ashandk/stratos/blob/5ab9159744f136e649174f7293adabe5c77fb4bd/components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41.java#L171


On 14-07-15 16:42, Cornelis G. A. Kolbach wrote:
> On the change form you mean. Yes, but shouldn't that be automatic? Plone has this nice forwarding feature anyway until the old ID is taken by a new object.

Should not be UI exposed IMO

--
    Guido Stevens  |  +31.43.3618933  |  http://cosent.nl

    s o c i a l   k n o w l e d g e   t e c h n o l o g y


" так как предполагается, что в `bh.js`-файлах не используется `require`." -- это пояснение к тому, почему "частично"? если да -- то читается тяжело и нужно здесь пояснить связь -- кем предполагается?


target folder ne bi trebalo da se nalazi na git-u


What Qt Version is your OpenSceneGraph compiled with? I use OpenSceneGraph binary from Debian Jessie or ubuntu 14.04 Distribution packages. They are compiled against Qt4 (https://packages.debian.org/jessie/openscenegraph and https://launchpad.net/ubuntu/trusty/+source/openscenegraph). That is why I compile IfcPlusPlus against Qt4. If your OpenSceneGraph is compiled against Qt5 you should use Qt5 to compile IfcPlusPlus. You could take Fabians Repository instead. Up to date the only change between my repo and fabians repo is the use of Qt4. See https://github.com/berndhahnebach/IfcPlusPlus/commits/latestgit66


> Op 14 jul. 2015, om 16:44 heeft Guido Stevens <notifications@github.com> het volgende geschreven:
>
> On 14-07-15 16:42, Cornelis G. A. Kolbach wrote:
> > On the change form you mean. Yes, but shouldn't that be automatic? Plone has this nice forwarding feature anyway until the old ID is taken by a new object.
>
> Should not be UI exposed IMO

That's what I meant ;)


ni settings folder ne bi trebao biti ovde


A link should be Ok I think.


Yep.


Vaffanculo


So when changing the title, we want also an ID rename, right?


There is a typo : "$http_accept_lang**ua**ge", not "$http_accept_lang**au**ge"


Что подразумевается под "таргетом"?


Let me try the volume option (never did) and will let you know.
(currently fixing your other bug report about volumes larger than 1GB)


> Op 14 jul. 2015, om 16:45 heeft Alexander Pilz <notifications@github.com> het volgende geschreven:
>
> So when changing the title, we want also an ID rename, right?

Yes automatically.


don't we need `return timeout || timeout = $timeout(processQueue, 0, false);` ?


中英文混排的格式问题同上，全文都调整下吧，会很快的。


Why is this in here?


It's fine. It'll go away by the end of the PRs.


isinstance(list_delimiters, Iterable)


I'd really like to help fixing the sound problems, but I only have my laptop with me right now, which is only set up for emails. I suspect some memory corruption in the hair logic, which is broken anyway and may bleed through to all other game logic if it's really corrupting memory. Maybe it's some use-after-free problem.


If you are adventurous, I recommend you to squash these two commits, it saves git a lot of data (because it can then do a delta).

Try: git rebase -i HEAD~2

and change one line of 'pick' to 'squash'


Forthcoming in ensuing PRs.


这里需要同步下最新的上游，eLinux.org 已经更新到本地了，或者直接用脚本更新吧，执行：

$ tools/build-cross-refs.sh


ok


ok!


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


I think we should use `var nextTick = Util.nextTick;` instead of a `this` reference.


Silently replaced BUILD_TAG by BUILD_URL (could even allow multiple Jenkins server) and remove GIT stuff which is not useful as we can have more than one repository (e.g. soe-ci and acme-soe)


zasto veliko slovo u imenu paketa?


While this code is compact, it has become a little bit harder to read. We should not compromise readability as it is crucial in maintaining code.


linux 最好首字母大写：Linux


`--target` 和 `--host` 用命令标示符括起来。


You should make the same change to the `getBackendListCountedBuilder()` method so that the number of results is correct. The condition using `$fullNameReversedExp` is also missing there.


@Makman2 could you go through https://github.com/coala-analyzer/coala/commit/120c2364bea75eb8516e4782927a647e6c32239e#commitcomment-12153317 and fix it for all the classes where I had done that ?

I was about to do it - but seeing as you have already started it here ... could you do it for all classes ?


Hello!
Thank you for plug-in!
Could you change tabs bar  background color #691919 at startup Brackets on #333639
![bar](https://cloud.githubusercontent.com/assets/6720515/8676063/e7d89d3c-2a4f-11e5-84e6-1936dd62dfd7.png)
![tab2](https://cloud.githubusercontent.com/assets/6720515/8676224/d98af116-2a50-11e5-8719-d22ee65d8fd3.png)


Feels weird to rely on the format-string of a sub-function here. Kinda hard-codes the formatting at two places. Why not parse the format-string as argument to format_timestamp_dns()?


还是少了逗号。

"cheering.caf"
"bizParam"


if (Players.Any() && InactiveTime > Envir.Time)
                InactiveTime = Envir.Time;

Monsters do not attack the problem right there when the user appears.


Javadoc on classes and methods missing.


This is quite a heavy commit but it is necessary to be able to use CCVs as part of SOE-CI. It means you can create CCVs containing the CV given as parameters and attach the CCV to a test node, and it will work. The difficulty is that there is no hammer command to tell 'update this CCV with the latest version of the CVs it is made of', so this needs to be programmed in bash, which is ugly.


For async applications you commonly have to implement a state machine. Just like you do here with your boolean flags. There are python packages around which simplify these state machines. Maybe it is worthwhile to look into those.


> If we add every pattern we currently have in Screendoor, these are no longer design guidelines; it's a visual QA suite.

Gotcha -- I am currently using it as both.

Should we create a separate set of pages for visual QA?


@parasyte committing this part to a separate branch for now, but this actually working just fine ! :)

few improvements are required though:
- I had to "hack" the Tween object and add a fake `pos` object, as else it is causing issue when added to a container. Shall we say that from now on, only object with a position vector can be added to a container ?
- we lost the ability for the z value to be automatically defined when an object is inserted into a container (and if the z value is undefined), like for the HUD where I had to specify an actual z value now,  see here https://github.com/melonjs/melonJS/commit/52a0a02648aa47595a98dffc3e7e792c1f595d7f#diff-d795862bb72efc855abac0dda154e799R14 and here  https://github.com/melonjs/melonJS/commit/52a0a02648aa47595a98dffc3e7e792c1f595d7f#diff-e34e58676109676155a3545841186476L139
- some weird code now where a z value is needed, but not actually used :)

if you could have  look and comment when you wake up ?


This line should not be commented


this is super close from completion anyway  :)


comment this


"суффикс" и использовавшееся ранее "расширение" -- это одно и то же? По форме -- да, а по сути?


Or maybe you can just shorten the whole thing. I mean you don't look at 'errno' anyway:

```
r = format_timestamp_dns(expiration, sizeof(expiration), rr->rrsig.expiration);
if (r < 0)
        return r;
```

becomes:

```
struct tm tm;
...
if (!gmtime_r(&rr->rrsig.expiration, &tm) || strftime(expiration, sizeof(expiration), "...", &tm) <= 0)
        return -EINVAL;
```

..not sure. The helper function does look kinda nice..


if (Players.Any() && InactiveTime > Envir.Time)
InactiveTime = Envir.Time;

Monsters do not attack the problem right there when the user appears.


other one


i) My general and perhaps not so helpful answer (sorry) is: Please write the doc such that a Linux person quickly finds information without having to read Windows workarounds and a Windows person quickly finds the info she needs without reading through pages of irrelevant information that does not work on Windows.

ii) Are Git tools really needed? Why? Is this a dependency? I would assume that every developer is able to fetch a file from the web via the command line. I don't think we need to explain it.


Fixed here: https://github.com/discourse/discourse/commit/46b61627cdd71af6980305e92ba131678e2f89aa


Please add the authorship of the MAV'RIC team


codeline. не?


Yeah, I was going to add that today, it was a bit too late last night.

But something struck me today. This trait does not allow one person to have multiple `Permissions`. It's a bit silly if `Admin` can't reach one route because he is not `None`, right? Could we perhaps do:
```
pub trait AuthorizeSession {
    type Permissions =  enum { Admin, User, None };

    fn permission(&self, Permissions) -> bool;
}
```
Instead?


这里翻译时看下是否要尽量让专有名词的符号保持一致，比如 openocd --> OpenOCD ? 不一定遵守原文？


Why define a new colour variable, if this colour is only being used in a single location?


It was a while ago, but I believe it has to do so that the contraction opcode will work for the alphabetic wordsigns.

What is the problem that these lines are causing?  I believe these lines could be removed.

MRG
mgray@aph.org

________________________________
From: Bert Frees [notifications@github.com]
Sent: Tuesday, July 14, 2015 5:39 AM
To: snaekobbi/liblouis
Cc: Michael Gray
Subject: Re: [liblouis] Fix context rules with uppercase string tests not working. TODO: Find out what the goal of the commented code is. (35dfbb3)

@MikeGray-APH<https://github.com/MikeGray-APH> can you shed light on this? Why were these to lines added?

—
Reply to this email directly or view it on GitHub<https://github.com/snaekobbi/liblouis/commit/35dfbb3eace45d2c46cc122b65cf22e5768c26e3#commitcomment-12151705>.


"Список файлов предоставляет технология files"
Это важно? Требуется ли установка этого пакета для того, чтобы пользоваться этой опцией?


@davilima6 you are totally right. plone.api recommends that (http://docs.plone.org/external/plone.api/docs/contribute/conventions.html#indentation). Will fix.


Shouldn't need to define a colour here at all


I would use textCapWords instead of textCapSentences for people with compound first names, multiple last names, and people who want to write their middle name.


comment 1


If we let this like this, the default configuration requires to use a gimball system. This is specific to your project, therefore you should have done this on your project files...

Next time, DO NOT PUSH ON MASTER DIRECTLY BUT FILL IN A PULL REQUEST FROM A SEPARATE BRANCH!


This is a default values for heartbeat and timeout from the implementation of SockJS for Node.js.


Removed in the following commit =p


I would probably rename @state to something like @completed or @complete, otherwise what happens if you have some other type of state later when you have a new feature?


Or, if my newer commit is messing it up, give a higher number than 2 ;)


I would have totally gotten these wrong. I was under the false impression that you should do subtractive prefixes wherever possible, e.g. 49 would be `IL`. Not the case, obviously!


We can go -releases way so we don't need to build duktape .c / .h (so issue #2 is gone).
We also could define a version to be used by our build (they can't go to the past and change the license of already released code). It also avoid changes on code that could break our build (so issue #3 isn't a problem).
For issue #1, ok, you'll spend 10MB more when fetching this repo. How relevant is it? Zero relevance, you have to download a lot of other dependencies already, our repos, etc.

Dropping the code in our repo, in the other hand, lead to some issues:
1) Maintaining it in our repo doing commits with code drops (a lot of unrelated changes in a big blob)
2) Adding thousands of loc that aren't intended to be maintained by us.
3) Provide code that we didn't write increasing our exposure to possible copyright infringements.

When we deal with libs, we don't drop them in our repos, we just depend on them. Duktape doesn't provide a lib, you must to use a c module in our project. But it doesn't mean you need to have it copied in your repo.

I continue to believe dropping this code in our repo isn't the way to go.


could you use keyword syntax i.e. `verbose=save` to make it explicit to the reader how this weird save variable is used here?


Ah, good call - although I'm not sure we actually need to return the timeout.  Either way, I'll submit a fix for this.


@Revenaunt updated.


isinstance(list_delimiters, Iterable) should do it...


The "Sockjs-node" implementation raises InternalServerError in this and other cases.


Суффикс в ENB это не тоже самое, что расширение.

`file-name.suf.fix`

Расширение это то, что после последней точки — `.fix`.
Суффикс это всё после первой точки — `suf.fix`.


The default value is there so that link has some value even if somebody calls the function view without any GET parameter.


It already exists. For (most likely) historical reason we have

```python
class TestCredentials(unittest2.TestCase):
    ....
class Test_get_for_service_account_json(unittest2.TestCase):
    ...
```


Cool! Thanks!


ack


It goes in #988.


I agree with this


correct since the order will indicate the final primary key in database.


I do not know why this is necessary, but that's what works "SockJS-node".


I have no major concerns with this, but want to wait until we ship the beta to pull it in.


I don't like this. You should change the comment.


Does in #991


Yep good catch. My vim macro failed me.


Indeed.


atom does not activate package if it has activationCommands. So with these present auto-fold will not work since the package is not activated until you manually use one of these commands at least once


I thought of something pretty nice for this - you could do the following:
`section_name_surroundings=( ("[", "]") ),`

and later in the function, just do `dict(section_name_surroundings)` ... if the user gave a dict, it will remain as it and if it is a double tuple type object - it will automatically convert to a dict.


FYI ruby standard is to use snake case for instance variables, so you would probably call this @task_list instead of taskList.


@hodgestar Ready for re-review


@infojunkie plz review this commit


you mean `section_name_surroundings=( ("[", "]"), ),`


You can set multiple attr_accessors on one line like this:

attr_accessor :title, :priority, :complete


I tried, but couldn't get it to work. I get a `examples/example.rs:19:1: 30:2 error: the impl does not reference any types defined in this crate; only traits defined in the current crate can be implemented for arbitrary types [E0117]`.


yeah!!!


you tested this manyually?


str is an iterable, you can remove checking for str I think


1. Thanks for pointing out. I meant to add a comment above this, like I do in other places.
1. Some public property like `client.http` or `client.credentials`?


you're right - the comma is needed

On Tue, Jul 14, 2015 at 8:32 PM, Lasse Schuirmann <notifications@github.com>
wrote:

> you mean section_name_surroundings=( ("[", "]"), ),
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/2a421c5f11066cf334141fae1e8b506e733c0d51#commitcomment-12157889>
> .
>


This is the backoff between retries right? Should probably lower the default down to something like 250ms.


Важно. Требуется, чтобы результат был получен с помощью технологии.

Результат с помощью `files` ожидается всегда, даже если не указывать значение опции явно, т.к. есть значение по умолчанию — `?.files`.

Использовать `enb-bem-techs` пакет нужно для всех БЭМ проектов.


:+1:  :)


you can use that cool or syntax:

`something = a or b`

something will be a if bool(a) evaluates to True, else b.


Ah yes, thanks!


I stupidly did a git update which included this change while I was trying out the pfsense_ng_fs theme and now every GUI change I make I get:
Fatal error: Call to undefined function pfSense_fsync() in /etc/inc/config.lib.inc on line 856

I set the theme back to pfsense_ng via the developer shell but I don't think it's related to the theme...


Here, you suppose that for the default case, we don't want to put the failsafe values on the servo 5 and 6 (when i=4 and i=5), as the failsafe value depends on the servo type, this should work anyway. In addition, this function is called almost only in project specific files, therefore, you could just create a new function

I don't get why you do that...


Why not `throwIO . ErrorCall` in `sendMail` here in this module?  I think that whoever calls this besides `sendMail'P` should do more or less the same logging-and-crashing.


:)

On 07/14/15 11:05, Christian Wach wrote:
> Ah yes, thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cuny-academic-commons/social-paper/commit/cdc5da8c9f2ab6077fb30bb3e96161a26b5e43d7#commitcomment-12157947>.
>


Found this out the hard way :) Thanks


that's ugly api


yes and works like I want


Hmm, I don't think that extracting this part into a method makes things more clear - at least it doesn't for me


Catch `SomeException` here instead of `ErrorCall`?


I did it to avoid implementing the `_batch_stack` mechanisms on the mock class. It seems somewhat necessary to test the true behavior here.


Yes I started investigating yesterday but ran out of time. I'll continue today.


I don't think we need all these parameters, do we?


the problem with bad/good and do/don't is that there will almost certainly be times when it makes sense to do the "bad/don't" thing. Maybe that's my problem with style guides :)


ack


It's two places - but happy to remove if that doesn't qualify for a new variable


done


Move to `Thentos.Smtp`?


Hey,

I just stumbled upon this via TK's twitter feed and saw your frustrated comment on on line 11.

I think you've made a typo there because
[o for o in bpy.data.objects if o.type == 'MESH']
is valid and gives the correct result. To make this a little more efficient, you can use a generator expression:
(o for o in bpy.data.objects if o.type == 'MESH')

BTW, I'm the creator of the Freestyle SVG exporter and have worked extensively with Blender addons and Freestyle, so if you experience any further problems or need some help on this, please ask


`\#`?

/me looks at the RFC

Wow, this is indeed used to delimit the RDATA. "Nice"..


Um, I do not like this global modification of a Ruby base class. Prefer Ruby refinements which allows to close such a change into a specific scope only (so the code outside is not affected).

See e.g. example how to define a refinement: https://github.com/yast/yast-ruby-bindings/blob/master/src/ruby/yast/core_ext/ansi_string.rb#L4 and how to use it: https://github.com/yast/yast-yast2/blob/master/library/system/src/clients/view_anymsg.rb#L49 and https://github.com/yast/yast-yast2/blob/master/library/system/src/clients/view_anymsg.rb#L191.

(This is a new Ruby 2.1 feature, see http://blog.rkh.im/ruby-2.1.)


I thought we don't want such lazy evaluation constructs?


Good catch, thanks


Yes I agree with you :)

If you want, you can go back as it was.


Also, I would just crash in `sendMailOk` instead of returning a boolean and make the caller crash.


I'd suggest renaming this package from io/sqlite to provider/sqlite, seems like a better fit - these are adapters to external libraries and resources, io is more about reading/writing questions and answers.


@fneu began to use them and I didn't like them. @AbdealiJK began to use them and I liked them a bit by then because they're really charming. Now I want to use them too. No really, I do think they're cool and we can use them, it's not that complex.


@Makman2 you disagree?


first program


I think the if/else structure is fine -- it's inherent to the algorithm for now that there are several scenarios to detect and consider.

Likewise, recursion (or perhaps conditional iteration) seems relevant for handling the subcases that are identical to the larger cases.

If the if/else is bothering you, the `case` statement may provider nicer syntactic sugar (but takes exactly the same approach):

```ruby
    case arabic_number
      when 0      then ''
      when 1..3   then 'I'  + convert(arabic_number - 1)
      when 4      then 'IV'
      when 5..8   then 'V'  + convert(arabic_number - 5)
      when 9      then 'IX'
      when 10..39 then 'X'  + convert(arabic_number - 10)
      when 40..49 then 'XL' + convert(arabic_number - 40)
      when 50..89 then 'L'  + convert(arabic_number - 50)
      when 60..99 then 'XC' + convert(arabic_number - 90)
      when 100    then 'C'
    end
```


Hi,

ad ii) IMHO, git is needed for developers working on(for) MS Windows.

For example, myself (as humble Windows developer), I am versioning my changes, doing some programming/adaptation, testing on Windows machine(s).    And on top of it, the "C:\Program Files (x86)\Git\bin\curl.exe..." is  fine alternative to the Linux wget command.


What about interaction of cache and skip? We should try to use the cache even when we don't have an API key, imho.


I wanted to avoid catching `SomeException` because that would also catch things like sigint or asynchronous exceptions that we don't really want to catch here.


如果你的调试器支持 GDB ?


Those `10..39` thingys are actually ruby `Range` objects. And the Ruby `case` statement is designed to make these kind of "soft" comparisons (via the `===` operator). Very handy for succinctly notating scenarios like this.


You're right : [OSG on Utopic](https://launchpad.net/ubuntu/utopic/+source/openscenegraph) is Qt4 compiled (like Wily, the earlier one).
I will try with Qt4, but I agree with Fabian on [issue #39](https://github.com/ifcquery/ifcplusplus/issues/39#issuecomment-120822085) : I wonder why Ubuntu uses Qt4...
I will ask Ubuntu's communty, try compiling with Qt4, and be back.
Thank you


Changed as per mengxr


Done.


R0 and Z0 are not used recurrently, so the initial values can be simply set to None in outputs_info


Did you track back when this was added to see if it was ever used? Perhaps ping the person who added these files?


Yeah, I wasn't sure about that either. But the ticket was fairly specific that `sendmail` should return, not throw an error.


Imho this is a breaking change, you have to re-insert all your plugins. Despite that this will not work at all if you dont modify ext_tables.php


It's actually not the same information.  From MSDN:

> **FindExInfoBasic**
> The FindFirstFileEx function does not query the short file name, improving overall enumeration speed.

We do not support Windows XP or earlier, so we would prefer the faster routine.


Ok, will do (both of these).


Certainly I think that every developer should always use Git for everything. But I don't think that Git is a requirement for Autocmake.


Included a cleaned-up version.


I did it to avoid duplication, and I thought the purpose of those two lines was well summarised in the method name.

I can revert it if you think  inlined is clearer.


this is likely as vix build nashu full-hd skins.  they are in openvix skins git.
i havent updated them though in a while but if you want to add them to oe-a skins and keep them updated, i will remove from openvix-skins

or you can add enigma2-skins-openvix to your feeds.bb and keep them updated in the vix git?


Included softmax, sigmoid was already there.


这里的 find 命令咱们也可以简单的用 `find . -name "*gdb*"` 包含起来，更好看一些。


Ah, I see! I will change this and submit another pull request.


ack


AipoLocalization_ja.propertiesに宣言したTODO_SETFIELDNAME_URLを使うようにすると良いかと思います。
言語ごとに簡単に文字を置き換えられるよう、直接文字列を打ち込まずにAipoLocalization_ja.propertiesを介するようになっています。


What should I put instead of 'key'? for me json is all about key/value pairs.


S


These variants I would leave out of core funktional. Maybe just keep them in a separate module.


Cool!

2015-07-14 17:14 GMT+02:00 Grzegorz Chrupała <notifications@github.com>:

> Included a cleaned-up version.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kadarakos/funktional/commit/01401bd0e3a20d2d995e3f4f1851960c1e1fd7e9#commitcomment-12158146>
> .
>


@samccone Yes, I did. It was simply forgotten when moving the app to use Bower.


permanent_widget. only Qt functions use CamelCase.


基本 Review 完毕，内容本身稍微浏览了一遍，问题不大，格式方面稍微调整下就 OK。


The trait does allow multiple permissions, the associated type can be something with bitflags or a `Vec<PermissionType>`. Alternatively the user's type could implement `Ord` so `Admin > User > None` and permissions can be checked for `> None` or using `match`. I don't agree with reducing it to `bool` return as that would mean the trait method needs to be aware of the current path (which would be combinatorial explosion).


I think the confusing part for me is to multiply by two inside the method

I get you need to do that since you are sleeping that time, but from the outside you could think that it is retrying with the same backoff, instead of with `backoff * 2`


👍 sgtm


good catch!

--
Lane McIntosh
www.lanemcintosh.com

On Mon, Jul 13, 2015 at 1:23 AM, Niru Maheswaranathan <
notifications@github.com> wrote:

> also we forgot to include clim as a argument to the function
>
> silly silly
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/baccuslab/pyret/commit/a67362a185f22a5ed0e8e206d3e800faf2857cdf#commitcomment-12128085>
> .
>


nope. this isn't working is it?


这里添加下，详细格式请参考最新的 zh/doc/README.md

> 原文：
> 翻译：
> 校订：


Are parentheses necessary?


"Enter name (optinal

fix this


this isn't wokring


1: M101 signature is missing


good luck

Even debian sids osg dev packages uses qt4 https://packages.debian.org/unstable/openscenegraph

I would like to use Qt5 too. I tried to compile osg myself against qt5 a few month ago but faild.


Wouldn't this run 4 machines on Travis, two of them quite useless?


I'm not sure we want to push this into `SerDe.scala` -- this is used for all communication between R and the JVM and its better to keep this clean.

Could we have this `doConversion` block in the bytesToRow function ?


It seems that your fix is entirely in userland, but I initially encountered the problem while using `lzc_list()`.
Maybe I just passed wrong parameters to it... or maybe there is another issue.
What parameters should be passed to `lzc_list` to list all snapshots of a given filesystem?


Ok, the trait might need to be adjusted to make it less work for the user, but it's okay to show the idea for now, can correct it later.


You are going to get your educational accreditation yet!

I did the: git rebase -i HEAD~3
followed by changing the "delete" commit line to squash
but git push yielded:
```
$ git push
To git@github.com:opengapps/arm64.git
 ! [rejected]        master -> master (non-fast-forward)
error: failed to push some refs to 'git@github.com:opengapps/arm64.git'
hint: Updates were rejected because the tip of your current branch is behind
hint: its remote counterpart. Integrate the remote changes (e.g.
hint: 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.
```

NOTE: I was up-to-date with you 22556af commit first...


And this is also probably not valid yaml, without the space


To reduce the number of changes necessary for adding new measurements, you could add an optional field to the objects listed in your `measurements` array (which I assume will be generated based on what is implemented serverside). Eg `"dependencies" : ["Weight", "Hight"]` for BMI.


it's a complement to this (tests would be failing without zmq)

https://github.com/josegonzalez/python-beaver/blob/master/beaver/tests/test_zmq_transport.py#L8

I can make a separate PR for this if you'd like


See https://github.com/adsbypasser/adsbypasser/commit/9b3226f3dffbad4286a0da8ed54840a472087282


For the same reason as above, you could use an optional `"default" : true` field for default measurements.


The parentheses shouldn't be necessary.


Yeah, @cburgdorf, come bikeshed with us!


This dropdown can (and should) be generated from the `measurements` array. Make sure that in the final version the names are translatable.


#foreach ($record in $result.List)から#endのループ内で、一覧表示用のデータを表示しています。
このループ内で$recordは使える変数名になっていますので、記述をそちらに移すと上手くいくかもしれません


Not a fan of the `will look something like` ... would rather see a clearly delineated list of all the possible properties followed by the example.


can you change the date here to:
Thu July 16 2015


I know the feeling - I've been told many times to refactor some code because it was "bad/don't" or not canonical :)

Still I think that it is worth mentioning some things otherwise it is "legit unless specified".
For the same reason I came to the conclusion that tools are better than style guides


I haven't been asked about my knowledge of web dev yet; removed!


Actually if I don't provide a value it raises `AttributeError`

<img width="588" alt="dash" src="https://cloud.githubusercontent.com/assets/18167/8677058/8cadb75a-2a7f-11e5-9de3-280f41f0f708.png">


got it... the "-f" was needed


PS: Sorry I haven't merged that yet, I'm still investigating something I don't understand. I'm working on fonts for the AA branch and that should lead to the proper fix.

If you ever merge the AA branch (you don't need right now), those four lines will now conflict because I have renamed a few fields. But the fix will be trivial.

glyph.U0 = ((float)pc.x0 - PixelCenterOffset) * uv_scale_x;
glyph.V0 = ((float)pc.y0 - PixelCenterOffset) * uv_scale_y;
glyph.U1 = ((float)pc.x0 - PixelCenterOffset + glyph.Width) * uv_scale_x;
glyph.V1 = ((float)pc.y0 - PixelCenterOffset + glyph.Height) * uv_scale_y;


ack


Shouldn't this invocation of `main()` be `main(sys.argv)`, since `celery.bin.celery.main()` takes `argv` as a parameter that's defaulted to `None`?

I'm experiencing a bug where Celery 3.1.18 quits quietly after invoking the init script because `sys.argv` isn't passed in. When I change it to `main(sys.argv)` as stated above, Celery starts without issue.


--> \`log_start\` 和 \`log_end\`


That's a good point. This is prototype code though, so I expect it to change significantly anyway before it becomes the final component.


Sorry, brain is in javascript land.


test Comment


Don't need to add `TEST-EXPECTS-ERROR-REGEX`. The `TEXT-EXPECTS-ERROR` refers to the error code being returned. Is independent of the output.


Paths also fixed.


@fskirschbaum
![image](http://media.giphy.com/media/XreQmk7ETCak0/giphy.gif)


It seems that you have forgotten to remove the **main()** method.


BULLSHIT just add PPUThread& argument to function which fails to link.


@eregon @chrisseaton @nirvdrum I added these excludes to get the MRI tests passing again. There is also some regression when the test suite starts `jt test mri`:
```
ERROR IN TIMEOUT THREAD

    internal implementation error - UnsupportedOperationException Undefined Rubinius primitive. org.jruby.truffle.nodes.rubinius.UndefinedPrimitiveNodes$UndefinedPrimitiveNode.undefined(UndefinedPrimitiveNodes.java:34) (RubyTruffleError)

Backtrace:

/Users/brandonfish/Documents/jruby/test/mri/ruby/test_regexp.rb:557: warning: (VAliasNode 557) does nothing - translating as nil
/Users/brandonfish/Documents/jruby/test/mri/ruby/test_regexp.rb:558: warning: (VAliasNode 558) does nothing - translating as nil
/Users/brandonfish/Documents/jruby/test/mri/ruby/test_regexp.rb:559: warning: (VAliasNode 559) does nothing - translating as nil
/Users/brandonfish/Documents/jruby/test/mri/ruby/test_regexp.rb:560: warning: (VAliasNode 560) does nothing - translating as nil

```


This is a good point. We could document which properties can be set atomically and those that cannot. Alternatively, if we can re-factor things to handle them together, we could resolve this.


怎么直接加到这里了？


Doesn't work :(


getUrl()がurlの値を読みだす処理(getter)なら、setUrl(String url)はurlに値を設定する処理(setter)になります。
他のsetterを真似して同じように処理を書き加えてみましょう


Yeah, that seems to work!


Zasto Pera?


Solves https://github.com/ApiGen/ApiGen/issues/625


Be consistent, use array like everywhere else in this file
`...only: [:index]`


基于最新的远程仓库，合并为一条记录提交吧。


Why is this /sbin/busybox-static and not $BUSYBOX_BINARY?


@claudiosmweb and @mikejolley use of `$( 'form', this.$el ).serializeJSON()` should also be here :P https://github.com/woothemes/woocommerce/blob/master/assets/js/admin/order-backbone-modal.js#L110-L125


Please do.


1st commit


Besides the issue pointed out, looks OK. Remember to fix the README accordingly. If no other changes besides that, feel free to push directly.


The fact is that when you set your timeout to 0, it should normally disable it. Here, when the timeout was 0, it didn't pass in you condition so the timeout was reset to its default value, 2000 ms.


Apply


Use `obj.comments.count()` and `obj.files_attached.count()` instead of `len(...)`. It's more efficient: In your approach, you get all objects from the database and count, whereas `.count()` translates to `SELECT count(*) FROM ...`, i.e. you count relevant records on the database, which is quicker.


(#...) 后面的内容不要翻译哈。这个 gitbook 没有办法生成正确的内部链接，对中文支持不好，需要保留后我们用脚本自动生成一个新链接。


请先阅读 zh/doc/README.md ，然后基于最新的分支，把所有变更合并为一条修改再发 pull request。合并分支可以用 `git rebase -i HEAD^^^^^`


Instead of doing all this to grab the model  name from here and etc, please make use of the features I've been adding to the code where the server can specify url patterns ia inline_form_config.  See admin.py andthe get/fill*Url methods (as well as getInlineConfig/getFieldsConfig) for details!


Ah, I see the BUSYBOX_BINARY has $ROOT_DIR in front.


It seems to me even when the input y contains matrices the code should work. Can you put up a test case?


Why the extra blank line?


Yeah, that sounds like a solid needed test.  Are you sure we're running 32-bit right now?  Is the runtime we pull down to execute the tests 32-bit?  I didn't think that there was a 32-bit coreclr published yet.


中英（数字）混排时请在英文/数字周边添加空格确保阅读体验。


Nice, the 3 threads also goes well!


ack


:-) great


Pierwszy moduł w node.js


If `server` is given it should always be set to the value of `src`, regardless whether it's `NULL`, right? Otherwise, you might end up with stray pointers on the caller side.


Again, prototype code...
The goal here is to quickly explore different UI concepts. The final code will look much different.


The link to the schema points to the 2004 version, I recommend using http://www.w3.org/TR/xmlschema11-2/#dateTime


1: M100 needs more reviewers
1: M101 signature is missing
1: M110 missing component name


good idea!