I'm running `git submodule foreach "git checkout master"` because that's the only way to make git work with PhpStorm. How can it check out such an old version if I check out master?


May I know your rationale behind not using else if?


I really don't like this. >_> Even if non-instant chemistry recipes are to be implemented (it's a concept I personally love, but it requires study since it's probably the biggest gameplay change you could do to Chemistry), it should be done in a better way and in a posterior PR. This one is big as it is.


I think it's because master points to the last fetched version; you have to do git pull after git checkout master.


`bundle-collapser` is probably better as a devDep.


ok thanks so `git submodule foreach "git checkout master & git pull"` will be my new favorite command


Don't need to use enabled=true always. Test cases are enabled by default. Use it only if you want to disable tests explicitly.


Try to throw only the specific exceptions.


also the `browserify` could be updated as it's `9.0.3`.


So it's possible to change both settings at the same time


This method is useless here, it was used in moves stats only.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2212](http://teamcity.monogame.net/viewLog.html?buildId=5626&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 441, ignored: 6 Build time: 00:14:07


Shall we try to introduce unique API names and Context. May be Test Class name + some number for API name and test method name as the API context.


to fix errors such as these:

     2> src/Compiler/GhcjsProgram.hs:184:3:
     2>     Ambiguous occurrence ‘liftIO’
     2>     It could refer to either ‘GhcMonad.liftIO’,
     2>                              imported from ‘GhcMonad’ at src/Compiler/GhcjsProgram.hs:11:1-25
     2>                              (and originally defined in ‘transformers-0.3.0.0:Control.Monad.IO.Class’)
     2>                           or ‘Control.Monad.IO.Class.liftIO’,
     2>                              imported from ‘Control.Monad.IO.Class’ at src/Compiler/GhcjsProgram.hs:32:1-39
     2>
     2> src/Compiler/GhcjsProgram.hs:186:3:
     2>     Ambiguous occurrence ‘liftIO’
     2>     It could refer to either ‘GhcMonad.liftIO’,
     2>                              imported from ‘GhcMonad’ at src/Compiler/GhcjsProgram.hs:11:1-25
     2>                              (and originally defined in ‘transformers-0.3.0.0:Control.Monad.IO.Class’)
     2>                           or ‘Control.Monad.IO.Class.liftIO’,
     2>                              imported from ‘Control.Monad.IO.Class’ at src/Compiler/GhcjsProgram.hs:32:1-39
     2>
     2> src/Compiler/GhcjsProgram.hs:277:3:
     2>     Ambiguous occurrence ‘liftIO’
     2>     It could refer to either ‘GhcMonad.liftIO’,
     2>                              imported from ‘GhcMonad’ at src/Compiler/GhcjsProgram.hs:11:1-25
     2>                              (and originally defined in ‘transformers-0.3.0.0:Control.Monad.IO.Class’)
     2>                           or ‘Control.Monad.IO.Class.liftIO’,
     2>                              imported from ‘Control.Monad.IO.Class’ at src/Compiler/GhcjsProgram.hs:32:1-39
     2>
     2> src/Compiler/GhcjsProgram.hs:313:26:
     2>     Ambiguous occurrence ‘liftIO’
     2>     It could refer to either ‘GhcMonad.liftIO’,
     2>                              imported from ‘GhcMonad’ at src/Compiler/GhcjsProgram.hs:11:1-25
     2>                              (and originally defined in ‘transformers-0.3.0.0:Control.Monad.IO.Class’)
     2>                           or ‘Control.Monad.IO.Class.liftIO’,
     2>                              imported from ‘Control.Monad.IO.Class’ at src/Compiler/GhcjsProgram.hs:32:1-39
     2>


@xmo-odoo I believe there's a similar issue with logfile, see

#3852

and

https://www.odoo.com/forum/help-1/question/disable-logfile-from-command-line-68282


You could just ignore submodules unless you have to modify them, that's what I do :)


did you consider using `util.format` here?


If there is no version of ghcjs installed (or if closure-compiler is not installed) use ghc to prepare the setup scripts.


Adhere to java method declaration conventions. "SubscribeAPI" should be "subscribeAPI". Method name should start with simple letter.


嗨,

hjwdzh@gmail.com 想關注您

****** hjwdzh@gmail.com 是您的朋友嗎？ ******
好/是: http://invites.infoaxe.net/signup_e.html?fullname=Hjwdzh/physbam-drive-pc&email=reply@reply.github.com&invitername=hjwdzh&inviterid=37132591&userid=0&token=0&emailmasterid=ca7f9a68-c6f3-4ca8-96df-351fbaba693b&from=hjwdzh@gmail.com&src=txt_yes

不/否: http://invites.infoaxe.net/signup_e.html?fullname=Hjwdzh/physbam-drive-pc&email=reply@reply.github.com&invitername=hjwdzh&inviterid=37132591&userid=0&token=0&emailmasterid=ca7f9a68-c6f3-4ca8-96df-351fbaba693b&from=hjwdzh@gmail.com&src=txt_no

按一下這裡，將您自己從所有這類電子郵件移除
http://invites.infoaxe.net/uns_inviter.jsp?email=reply@reply.github.com&iid=ca7f9a68-c6f3-4ca8-96df-351fbaba693b&from=hjwdzh@gmail.com


Yeah. In general I prefer using formatting functions, but in this case it will cause the code to wrap to a new line, making it a little less readable overall.


@thet the only way to tell if your commit is responsible is to revert. This is why it is so important to revert right away, if things fail. You did the opposite and committed further, now it is impossible to tell.

All three builds are broken. Please revert all the commits until the builds is green again. If the build is still red, then you have the prove that your commits were not responsible. It is as easy as this if you stick with the CI rules...

Apart from that I still think that git submodules are a bad idea and I would really like to keep them out of our CI setup.


@davidmason I think you left this one on your last PR. It's generating a bit of unwanted noise in the logs.


you're right.

But it's worse:  for the rc4 function it segfaults even with bash >=4.2. That's why I commented that, see https://www.mail-archive.com/bug-bash@gnu.org/msg14428.html. So in generally lastpipe needs to be avoided and to be done differently for the next stable release.

As you mention this: I'll should label this as a bug. Thx.


From glancing at the current failures, I thought they are due to the daylight savings time change, not @thet's change.


We still want Cppcheck to be compilable by VS2010 .. so this syntax can't be used yet as far as I know.


never found a way to do that in phpstorm, and the risk would be that I never catch the latest versions of the modules which is about as useful as not installing them at some point. But yeah I think I'll try to remove them entirely…


Use testng static imports.


Plural here too


Using 1 here seems like kind of an over-optimization, especially since the whole idea is that the user might have a few


(The US changes to daylight savings several weeks earlier, because insanity.)


I make a coding mistake in function `_uuid_to_str`.  In `_uuid_to_str`, `seq_mode` should be assigned `uuid->clock_seq_and_mode`.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2213](http://teamcity.monogame.net/viewLog.html?buildId=5628&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


On 8 Mar 2015, at 17:29, Timothy Jones <notifications@github.com> wrote:

> Mostly ease of embedding. It's significantly easier to integrate Minigrace into other tools if only one file is needed. I can split this up into two different parts if you want, so the local editor still has the individual files but you can generate the concatenated form as a separate Make command.
>
> —
> Reply to this email directly or view it on GitHub.
>
It would be very useful to have a version that lets the browser load multiple separate files.   I would use this for debugging.  Having a “production” version that concatenates everything into a single file would be fine, so long as they have the same content.  It seems to be that it would be safer to have an explicit list of  included files in the makefile, than to grab everything (except a few exceptions) that happens to b win a directory.

I was also wondering about removing the “use strict” pragma; is that just because of the concatenation, or is there another reason that you don’t want the code to be in strict mode?  I moved to strict mode for two reasons: I get more useful diagnostics (e.g., this defaults to null rather than window), and I may get better optimizations from the Javascript JIT.

It would also be useful if you subjected the built-in lists to the same tests that are used on the prelude lists.  These are in tests/t150.   A simple test like
	def l = [7,8,9]
	l.remove(9)
	l.remove(8)
	print(l)
shows that the remove method doesn’t work.


But wouldn't tapping on two cards theoretically register two onChecked events?


i'll check right now..


@tuan08
I havent looked at the code thoroughly but do you take care of Exceptions that are not worth retrying?
e.g. If a message is too large the Exception is org.apache.kafka.common.errors.RecordTooLargeException. This message will retry indefinitely.


Ah, ok, I don't use PHPStorm to do git stuff, so I guess that's why it works for me.


@davisagli could be. Though, I prefer to have proof instead of guessing. And I prefer to have a green build instead of a broken build for days. Our CI rules are a technique to solve those kind of problems easily. If we ignore the CI rules, we end up in a situation like this, things become hard to figure out and the build is broken for days...


Set<String> volds = readVoldsFile();


Fix these errors:

     2> src/Gen2/Object.hs:530:8:
     2>     No instance for (MonadTrans (ReaderT ObjEnv))
     2>       arising from a use of ‘lift’
     2>     In a stmt of a 'do' block: l <- lift DB.getWord32le
     2>     In the expression:
     2>       do { l <- lift DB.getWord32le;
     2>            replicateM (fromIntegral l) g }
     2>     In an equation for ‘getListOf’:
     2>         getListOf g
     2>           = do { l <- lift DB.getWord32le;
     2>                  replicateM (fromIntegral l) g }
     2>
     2> src/Gen2/Object.hs:744:11:
     2>     No instance for (MonadTrans (ReaderT ObjEnv))
     2>       arising from a use of ‘lift’
     2>     In the expression: lift (fmap fromIntegral DB.getWord16le)
     2>     In an equation for ‘getIW16’:
     2>         getIW16 = lift (fmap fromIntegral DB.getWord16le)


Hey,
        I am trying to connect Raspberry Pi as device to my PC as a demo in which form the response in coming. I am using FTDI cable using which UART to USB communication is take place.  And also passing the --serial COM6 --baudrate 115200  as the command line argument but my device is not opening.

I got the error in the below function:-
m_stream.open(m_portName, err)

err={m_val=5 ....}

Can you help me with this?


:+1: works like a charm.. thank you very much..


I cannot reproduce it but from the error it seems the plugin thinks the proto file has "option cc_generic_services = true;" somehow. The file does not contain that line and the option should default to false. (https://developers.google.com/protocol-buffers/docs/reference/cpp-generated#service)


test this part @sujithvm


Really, all the plugins you make are something that can be generated by a plugin generator.


wut ?


No, few lines ago you do that, here you force the sidebar to be displayed unconditionally.


Remove this


I ignore them, but of course it's annoying to see them all the time and having to ignore them. Another solution is to add them in a Changelist, but you have to remember to add the submodule when you actually modified it... tricky... that's why I was so keen to remove submodules in #6605


Lol yeah. You're right. Gonna change it back to if else then.


Is it really needed that each template, in fact, defines extra status bar? It would be better if status bar is defined and created just once (in FormSimulator class).

Moreover icons should have opaque background and centered images. Your icons are not centered in the picture.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2213](http://teamcity.monogame.net/viewLog.html?buildId=5628&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 441, ignored: 6 Build time: 00:13:54


Marie ask us to move typing text to padding-left 5px on all input but this make a lag effect with placeholder. Do you think it could be better to use padding-left 5px on placeholder instead of simple left:5px ?


I agree but it matches the existing style so I used it anyway.


use testng expectedExceptions annotation for negative scenarios.


Uninitialized value could give some random memory and leak informations.


The travis vms only have 3g total, hence the maven limit at 2.5...


@hxim:I know. It's intended because i think we should test this idea on the other stat stuff (moves and gain stats). If their is nothing to gain we can later remove unneeded stuff from this struct.


This is not working on/for TIDAL
And I never saw anything similar to "Local Playlists" on web app for TIDAL either


I think if legacy us enabled and we find message then we should print a warning. Likely to be ignored but at least more visible than nothing.


All private methods can be make utility methods and improve reusability.


You are right it possible to move _SOURCE_DIRECTORY & _MYSQL_EXECUTABLE out of the file and add it as definition.
But I thought its similar to git revision hashes (the location of the source and mysql bin could change if you rerun cmake) thats why its defined here.

But I don't mind if you want me to add it as compiler definition.

_FULL_DATABASE is located here to make it easy to change it on new TDB releases.


Fixed.

Looks like glibc still doesn't do closefrom.


Copyright for new files shouldbe 2015..


where did the bzero come from?


how to change it


RowMajor (DeadParrot) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Hello! I tried GCC4.9 (Linaro) too, and that's what I get: http://pastebin.com/5uFhwH8g
I see something about LTO but didn't use it... Any help?


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/d0fa359a0dd289977b7ba999af23e0ed073965be


merging basho/riak_cs/bugfix/yessir-rc = d0fa359a into borshop-integration-1083-bugfix/yessir-rc


basho/riak_cs/bugfix/yessir-rc = d0fa359a merged ok, testing candidate = 5c5d53c0


Good :)
しかし改善の余地有り。
具体的には
（１）行列をprintをメソッド化
（２）行列の各種演算をメソッド化


yeah, i fixed it like we do on BSD's but with this commit i make the code portable for Windows :) (and also fix a variable name)


あと、prob01ディレクトリ以下に移動したほうがいいかも


asdf


Your build has LTO enabled.
Disable it.
On Mar 9, 2015 3:47 PM, "Sebastian Jena" <notifications@github.com> wrote:

> Hello! I tried GCC4.9 (Linaro) too, and that's what I get:
> http://pastebin.com/5uFhwH8g
> I see something about LTO but didn't use it... Any help?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_build/commit/2dd7b438743d066ecbea92dc1f180b49692fdd65#commitcomment-10093475>
> .
>


We need to introduce Test Class per single scenario and make the test class more readable. At the moment you are covering full scenario inside a single test method. Which might cause difficulties when debugging the tests.


I just made some changes (6cf8db08a2787c4f5b9cefd9f59e282e2b320d2d) that will probably break your code (sorry, didn't see the point in backward compatibility to a temporary state).

I already tested it for the R packages, and there is works fine. If you can test it and it works, we can push to master.


sf


[Particular TODO comment](https://github.com/UnifiedViews/Core/commit/168fb84d95a5ddf30fe42fc732640876802c0b91#diff-470740c3363ebb5c2d69b53721e42c37R86) is my personal opinion to the functionality not implemented by me. If it should be implemented, please create a new change request (GitHub issue) with low priority and let @tomas-knap approve that it can be implemented.


Here's an attempt to fix those tests: https://github.com/plone/plone.dexterity/commit/3f073d0d3397684d4c9c65781e101ffe28935de3


ergh, I haven't seen that one -flto in Android.common_build.mk in art/build, found it after I posted it lol.
Thanks anyway! Btw, why Linaro doesn't support it? Is it possible to fix it?


Hello Eric

For Planet we need the RSS or Atom feed - can you implement such for Clojure tag? If not, I can filter your blog through Yahoo Pipes.

Thanks


Yep. It really should be replaced by GSettings or whatever the cool-aid is this week.

It should be pretty straightforward. Nearly all configuration is abstracted in librawstudio/conf_interface.c with a few gconf_*() in other files mainly for enabling notifications.


GCC and Linaro both supports it.
It's Android that it makes trouble with LTO.
On Mar 9, 2015 3:53 PM, "Sebastian Jena" <notifications@github.com> wrote:

> ergh, I haven't seen that one -flto in Android.common_build.mk in
> art/build, found it after I posted it lol.
> Thanks anyway! Btw, why Linaro doesn't support it? Is it possible to fix
> it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_build/commit/2dd7b438743d066ecbea92dc1f180b49692fdd65#commitcomment-10093517>
> .
>


I will trying..


This is a stupid bug! Correct!


Colors shall not be moved to the theme. They used to be but should not be now, they are used to customize the layout in order to visually distinguish different instances - instance name is not enough.

I can imagine a trade-off, which enable you to fill colors in configuration file or use theme's colors as default.

If anything then the comment should be removed or changed to "trade-off version",


@kallestenflo From OSGi point of view it seems to be ok, though I had no time for extensive tests.


well, actually if anything goes wrong here the cache seems to be corrupted and should be rebuilt anyway ... As we are reading from serialized file there could occur further RuntimeExceptions (if properties were deleted, constructors has changed, etc.). I think it is ok (for the moment) to catch any type of exception here.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/880


@locutus2
OK. Also using this stat for history LMR reduction (step 15) might be useful. Especially with more threads i think.


Lol got it NIVEN!

Well spotted for the journal. Good thing you checked it. Passed right by me


The travis builds are still failing - cannot create thread oom intermittently....

Rich

> On 9 Mar 2015, at 06:40, Simon Matic Langford <notifications@github.com> wrote:
>
> The travis vms only have 3g total, hence the maven limit at 2.5...
>
> —
> Reply to this email directly or view it on GitHub.
>


Shouldn't that be at the end because then you will get the 1st tutorial (not the 0th)?


svn-info

svn简明操作指南，reveal.js

检出主干
svn co svn://localhost/demo/trunk ~/ppt/demo
查看主干信息
svn info
创建分支
svn cp svn://localhost/demo/trunk svn://localhost/demo/branches/branch_demo -m "创建分支"
检出分支
svn co svn://localhost/demo/branches/branch_demo ~/ppt/branch
查看分支信息
svn info
修改主干与分支的文件 造成冲突
svn up
svn ci -m "message"
在主干上合并分支
svn merge -r 14:16 svn://localhost/demo/branches/branch_demo
解决冲突
svn resolve index.php --accept mine-conflict
svn resolve index.php --accept theirs-conflict
标记冲突已解决
svn resolved index.php
提交主干
svn ci -m "message"


This appears to break uploads when I test with it. If I disable these lines, the images upload fine.


参考サイト: http://girigiribauer.com/archives/724


this one mark https://github.com/texttochange/vusion-backend/commit/25443c523958a47fe43bf312b90f9f07b6fe4d10#diff-16cef359f4eaff5fecfa4c1d2127a9ddR916


@tisto i did not change anything for the 4.3 buildouts. i think that's a good proof that my 5.0 commits are not responsible for the failures.
@davisagli tnx for the daylight saving change hint and for the fix. that's the obvious reason, since the failures it's timezone related. i didn't thought about daylight saving changes, because we change end of march.
@tisto which timezone is the jenkins server configured for?


WOAH WOAH WOAH WOAH YOU JUST DELETED LIKE THREE DOZEN THINGS

Did you just commit this right away without pulling the other stuff first?  What happened?


Naming or boolean returned is off. Try read: "if (categorizeClassElements) { seriousError..};
I would at least reverse the boolean returned, returning false if failure.
generateMetaData also returns false in case of failure.
The boolean shouldn't be needed. Alone the fact that it is needed shows the problem :-)


fast-forwarding develop to borshop-integration-1083-bugfix/yessir-rc = 5c5d53c0


This was pushed to `master` by accident, without a PR, and was reverted by https://github.com/18F/foia-hub/commit/2de17450fe926c0ada55f18132e14dfa3e938c1d.


missing a comment above this section.


@bors: retry


I think we do not need support area right now.
Because we do not have this contents.


saw approval from larsbergstrom
at https://github.com/saneyuki/servo/commit/27dfbb69c1911c45714632fc3081dabfccbb3963


merging saneyuki/servo/ignore = 27dfbb69 into auto


saneyuki/servo/ignore = 27dfbb69 merged ok, testing candidate = ae5175cb


@spilist `Status.during` 이 date range 를 인자로 받아야 함.. 116f16f


Touchè.


If both arguments are really fixnums then does this make any sense?

I mean, ##eq? is faster than ##eqv? .

(This comment also regards the subsequent commit, 418ee69cf612aec7989d3cb0624b0092e0799732 .)


具体其他详细说明，[请点击](http://www.sundexin.com
)：http://www.sundexin.com

#####检出主干

````
svn co svn://localhost/demo/trunk ~/ppt/demo
````

#####查看主干信息

````
svn info
````
#####创建分支

````
svn cp svn://localhost/demo/trunk svn://localhost/demo/branches/branch_demo -m "创建分支"
````
#####检出分支

````
svn co svn://localhost/demo/branches/branch_demo ~/ppt/branch
````

#####查看分支信息

````
svn info
````

#####修改主干与分支的文件 造成冲突

````
svn up
````

````
svn ci -m "message"
````

#####在主干上合并分支

````
svn merge -r 14:16 svn://localhost/demo/branches/branch_demo
````

#####解决冲突

````
svn resolve index.php --accept mine-conflict
````

````
svn resolve index.php --accept theirs-conflict
````

#####标记冲突已解决

````
svn resolved index.php
````

#####提交主干

````
svn ci -m "message"
````


great!


saneyuki/servo/ignore = 27dfbb69 merged ok, testing candidate = e9908e57


This seems weird. Where is it there is code that can't handle that? Shouldn't that just be updated then?


И все пак, не мога да пропусна cheap shot-а:

> Намирам, че употребата ми е коректна.

С тоя език да не си се засилил да ставаш депутат? :)


asdjsadjkl
assdads


ROMA does not need these line 41-42.
After installation of ROMA by gem, we can use all command directory.

ex.)
```bash
mkroute localhost_10001 localhost_10002 --replication_in_host
```


We don't really need that in iso19139 as it is related to iso19139.che ? or you depend on iso19139 XSLTs for that ?

For ISO19139 specific functions, it may be better to group them in the same namespace eg. https://github.com/geonetwork/core-geonetwork/blob/develop/schemas/iso19139/src/main/plugin/iso19139/layout/utility-fn.xsl#L4


I had though I had removed all refrences to multilingual urls.  I only wanted multilingual name and description not the URL.  I will remove that reference.


ROMA does not need these line 52-53.
After installation of ROMA by gem, we can use all command directory.

ex.)

```
romad localhost -p 10001 -d --replication_in_host
romad localhost -p 10002 -d --replication_in_host
```


Better name would be: `PARALLELISM` (parallel is an adjective)


x


Hi, I try 'hash_array()', but I got some negative number which is obviously wrong, is this a bug?
```
postgres=# select hash_array(ARRAY[1]);
 hash_array
-------------
 -1905059995
(1 row)
```


Unnecessary trailing space between function name and parenthesis; `sort_problem_files` doesn't have a trailing space while `check()` and `check_files()` do. (consistency)


@damirius

This merge should include the dev code. Please let me if you are having issues with it. Since the Milestone D and E aren't complete, I cannot yet merge to dev.

I will try to notify you if I merge from dev to the feature mapping so that we don't create conflicts.

Thank you.


new data type Extension material


Done see: 8f010fb8ca31494033c5101eddb65dc72645d4e2


This should not be changed, it should still be ldap.yml that contains your ldap server details.


You are correct.  It's a typo on my part.  I'll fix it.


I think "then change the auth strategy in the user model:" should become something like "change the auth strategy in your application config in file `config/application.rb`"


Correct for deleting inputs.html in github and not pulling into local directory


yeah sorry didn't realized before merging is more of a performance issue since there's nothing which state can be modified here but being math functions potentially called a ton of times it probably makes them faster to pass a const reference like:

```cpp
float ofSignedNoise( const ofVec4f & p );
```


тут получается ты "светишь" oauth данные на весь github :)
лучше их вынести в отдельные настройки через https://github.com/railsconfig/rails_config
например в Settings.local.yml задать
а тут в коде уже
config.omniauth :twitter, Settings.twitter.token, Settings.twitter.secret


Thanks Jesse.


![radio-icon 4](https://cloud.githubusercontent.com/assets/11302015/6551339/9cb6b646-c63f-11e4-8f1b-298b011ea06d.png)


Are you able to open COM6 using an another terminal program?

2015-03-09 9:27 GMT+03:00 thakorjay <notifications@github.com>:

> Hey,
> I am trying to connect Raspberry Pi as device to my PC as a demo in which
> form the response in coming. I am using FTDI cable using which UART to USB
> communication is take place. And also passing the --serial COM6 --baudrate
> 115200 as the command line argument but my device is not opening.
>
> I got the error in the below function:-
> m_stream.open(m_portName, err)
>
> err={m_val=5 ....}
>
> Can you help me with this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-10093317>
> .
>


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/b2bae7e802d53c647cee456b1b2a4b41c2021c77


merging basho/riak_cs/feature/introduce-getopt-into-inspector-rebased = b2bae7e8 into borshop-integration-1084-feature/introduce-getopt-into-inspector-rebased


basho/riak_cs/feature/introduce-getopt-into-inspector-rebased = b2bae7e8 merged ok, testing candidate = 38a7bf07


We had an internal discussion about changing `codeboxes` to `code_boxes` - but most of the team didn't like the change, so we didn't changed it :|. Naming, branding, etc.


Эх, лишний тэг параграфа. А перевод строки нужен был, я не учел.


А чего вернул mb_strtoupper?


Yes, would be better but note that if they fail they are trivial to fix.  Currently most of the specs don't use the I18n version, I can convert if you think its a priority?


yea its opening in putty terminal. I share the screenshot of it.

On Mon, Mar 9, 2015 at 1:05 PM, Sergey Polichnoy <notifications@github.com>
wrote:

> Are you able to open COM6 using an another terminal program?
>
> 2015-03-09 9:27 GMT+03:00 thakorjay <notifications@github.com>:
>
> > Hey,
> > I am trying to connect Raspberry Pi as device to my PC as a demo in which
> > form the response in coming. I am using FTDI cable using which UART to
> USB
> > communication is take place. And also passing the --serial COM6
> --baudrate
> > 115200 as the command line argument but my device is not opening.
> >
> > I got the error in the below function:-
> > m_stream.open(m_portName, err)
> >
> > err={m_val=5 ....}
> >
> > Can you help me with this?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-10093317
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/devicehive/devicehive-cpp/commit/3319387fab4cee5747676aa5bf61924a8ef893cc#commitcomment-10093816>
> .
>

--
Thank You

Regards
Jay Thakor
CDAC-Student
7745079370


FWIW, Metasm is now smart enough that specifying .i8 suffixes doesn't seem to be neccesary anymore,


lol, you didn't like the example? ;)


Also adding the multi threading!!


Actually it should be ./gradlew assembleAsciidoc


При копировании (Ctrl+C) текста, стили не учитываются и получается херня. Надо переписывать варианты, чтобы убрать mb_strtoupper. Либо добавить стиль text-transform:uppercase, но вместо mb_strtoupper поставить ucfirst(strtolower($text)), что не самый хороший вариант.


`const Camera* camera`


Let's add a `get_dtime()` method to `Camera` maybe?


I have added this because when using only proxy, calling localhost/proxy/ was redirecting to localhost/


Add validation for checking proper format of `:telephone_number`, `:mobile_number`, `:fax_number`


interesting way :)


Star position.


Use ruby's new Hash syntax.


`git rm Kadai.java`とかでファイルを削除しつつGIt管理から外せますよ。
ちなみに、本当は`git mv Kadai.java prob00/Kadai00.java`とかってするべきでした


Oops, fixed the issue by defaulting to zero if the project end date has passed.


Redundant scenario. Remove it.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/883


Can you test this change with the new -Profile related changes:
1. Create two profiles in Powershell - for two different subscriptions A and B.
2. Import ASR vault settings of one subscription A.
3. Run cmdlet with Profile A. See that it passes.
4. Run cmdlet with Profile B. What happens?
5. Run cmdlet with Profile A. See that it passes again.


test preview comment


Thanks!
I have changed on 1374e2bbd4653dc29d5ae7aa30c95309967eee52


@mikelow I removed the reverb in change d0014c4.

One day, I was using a SF2 file with BASSMIDI Driver and tried controlling reverb amount of each channels by CC91. However, I could not kill reverbs even though I am sending CC91=0. After all, I found that the issue was caused by the reverb preset in pgen chunk.

I am not sure if it is a bug of BASSMIDI, but I have removed the reverb for such purpose. (Also, a SF2 file I have (SGM-V2) does not have the reverb preset value, too. That is one of reasons that I have decided to do the change.)

Thoughts?


Thanks!
I have removed unnecessary spaces on  1374e2bbd4653dc29d5ae7aa30c95309967eee52 and 3f83b0a1d874a8158b96dd46906159d5e144fb5e


With rails there should be 2 space indentation


FUN! :) :+1:


Division by zero if shutter open == shutter close, which is a very valid configuration.

Also, this code should somehow be centralized in one place.


Does this mean that the new version won't support spring 3.2 any more?


I didn't meant to ask for the removal of the spaces around operators like pipes, but it's your call. :-)


Grade 1/2

I don't think you did the Todos part or you forgot to add the files.  Watch out for indentation.  Ruby has 2 space indentation as a guideline.


Testing the onLoad event handler of an iframe requires attaching the container to the DOM.


Indentation should be 2 spaces


New lines at the end of files


I think I would just call this `Time`. The type suffix is only used for typedefs, when we need to distinguish the template type and the concrete instantiation of that type.


@serverfire The links to the editprofile view were not found.


Relative to what? I think `normalized` would be much clearer than `relative`.


This is annoying:

    *** To confirm this, enter YES: yes
    Aborted


Grade 1.5/2

I couldn't find your work for TODOs.  You might have not added them to your git.  Good work other wise


Hi, Dear Michael
I want to use sonar to monitor the junit test code coverage for a grails project. firstly I use grails code coverage plugin to generate the coverage.xml and configure sonar.groovy.cobertura.reportPath property, but there is no data in junit part. So I change the sonar.sources from grails-app to grails-app/controllers,grails-app/services,grails-app/domain. then the junit code coverage appears but the console shows resource is not index. How can i solve that? thank you very much.
sonarqube 4.5.1
Groovy sonar plugin 1.0.1
please help me.


I think `relative` should be allowed to be 1, what if `absolute` == shutter close?


Please work with exceptions instead of return values, that is more pythonic and will help if this method is used from somewhere else.


Looking good! Especially the scaling to LTC! :)


Grade 1.5/2

Good work!  I just couldn't find you Todo's.  You might have not added them.


没问题的（回复得太迟了吧喂=。=


Maybe `TimeType` should provide methods such as `make_absolute()` and `make_relative()` to ensure we don't repeat conversion code all over the place.


You can consider to use fabulous or colorama for colored output


fast-forwarding develop to borshop-integration-1084-feature/introduce-getopt-into-inspector-rebased = 38a7bf07


Grade 2/2
Good work!

Seems like you recommitted every file though.


True, a bit annoying


See my comment below for colored output


This is clearer indeed!


I guess thats a copy & paste error. Please get that corrected as you are clearly not Fabian Jakobs. ;)


As below


Ruby usually has 2 space indentation


Maybe a custom exception?


Try having new lines on every file


I'd rather have this commented out by default, so that when people customize their `config.php` they don't overlap


TokenTypeOf!"("


These assertions are redundant. Only keep those in `TimeType`.


Grade 2/2

Great Work!


nice, but I find it questionable whether a method is even needed on the model just to wrap datetime.now(), why not just put: user.last_login = datetime.now() straight in the view?


Add new lines on the end of files


Grade 2/2
Good Work!


No, and this is standard osC behavior, only the tell a friend and the Paypal modules send messages to the header.


y u no UA_LOG_WARNING(logger, UA_LOGGERCATEGORY_USERLAND, "msg") ?


I added the missing podtype (timeval) by 95923cbfea052e7cfe55c230047c559ca8faba4d. Could you please verify if --check-library is still unhappy?


This line throws a NoClassDefFoundError when using the plugin without BungeePerms.


but if head is NULL, can can you access the ```->val``` field from it?


this regex is only matching 1-digit indices. Is there a guarantee elsewhere that there will never be more than nine?


why not std::size_t ? instead of unsigned long long


this needs a SIMD test case


it's funny how you first get what is at the memory address pointed to by ``** head``` and then get the memory address of the exact same thing :))
Try it simply using ```del_first(head)```


Looks good


some tests failed:
failure: http://build.servo.org/builders/mac2/builds/1187


Thanks! The windows build process looks less frightening now :)


Читается пока хуже, так как вы еще мало знаете про особенности.
Через пару занятий Вы поймете почему лучше писать именно так.


might be a good idea to have some sane defaults here, eg. 'Safari', so that people don't get an error message when visiting the AppVersions report with an unconfigured `config.php`.

Either that or work out a "hide if not configured" method


Yes, thanks! I just pushed the fix.


Hi.
Please use the mailing list (http://www.sonarqube.org/get-support/) for help requests and do not comment on (random) commits, here on github. Some users may face the same issue.
Don't forget to provide more information related to how you run the analysis of your project (sonar-runner/maven?).


J'aurai besoin de changer ma config donc.


OK, we will send new data binding document include mt8173 to public


Единственное, что мне не нравится в jslint. Это Forward Declaration для функций, а так все по делу.


All the element?


Should be element


also, needs to call user.save(), you could leave the method there I guess, but call self.save() in this method or the change won't go to the db


Maybe we could allow y/yes during debug mode (and only require the full YES outside debug mode)?


anyway, during development you can always use `echo YES | indico db downgrade`


Heu what ?? :p


What about not requiring uppercase?


The idea behind that was that this is a destructive command, which can easily destroy lots of data, so you should be damn sure that you really want to run it. Maybe the better option would be disabling the prompt in debug mode altogether?


toto netreba zakomentovat - pokial OpenCV nenajde, tak sa nebude kompilovat kod pouzivajuci OpenCV (ak nahodou kod bude kricat, ze potreju OpenCV, tak je to chyba a treba #ifdef)


set(...
treba nechat - len nastavuju premenne - nic to nepokazi (pokial ta premenna nie je pouziva)

podobne nizsie


Note: I'm open to finding a better fix for this (if it messes with vim over slow SSH connections)


newline doesn't belong here


toto by malo byt "na konci cmake-u", tam kde su ostatne install prikazy, idealne obalene if-om
len pre OSX a bundle


but here a newline would be nice :)


Hello, i have changed this "sm_pugsetup_requireadmin", "1", - but you can still use .setup without admin privileges. Any clue?

-spot


we currently don't specify what a function raises, maybe it would be cool to do that but at least for now it's not consistent so I would rather open a bug for it and not do it here for now.


see #370


After this commit I have problem with rendering `<button>` element.
test for reproduce example:
https://github.com/mzk/forms/commit/7726c6ab2725db55d93f5a38dd8a06077b436a1b


What about the `ZincScalaCompiler` case then? (honestly, I don't really care myself, as I don't use Scala nor intend to ever use it, but still)


I recall we came up with something shorter during last review?


Jednolinijkowe docstringi zgodnie z pep257 robimy w jednej lini:

"""tekst docstringa"""


Done


unneeded doc comment


wow that's so helpful...


What, exactly, about it? Is there something that our tests aren't covering?


zgodnie z pep8 nazwy zmiennych piszemy małą literą, kolejne słowa odzielamy podkreśleniem - get_day


no, your code is good, really, it's mostly the comments ;)


`ZincScalaCompiler` uses `includeClasspath(false)` but I think we should pass a `-sourcepath` in all cases (whether explicit or the default `null`)


${FLAGS_FOR_DEBUG} treba pouzivat, pripadne treba odstranit flagy ktore clang nepozna
(predpokladam, ze pouzivate na OSX clang compiler)


@wajatimur Can you give a detailed use case to reproduce this issue, please?


unneeded comment


@dg
After this commit I have problem with rendering `<button>` element.
test to reproduce:
https://github.com/mzk/forms/commit/7726c6ab2725db55d93f5a38dd8a06077b436a1b
its feature or bug?


if this was over the skeleton fat sprite I still feel it should have one, honestly just by referencing the non fat sprite, for code quality purposes.


It looks like, the linker do not find the libkodiplatform.so.


Why set it if there is no classpath?

When we were setting it, all of the zinc tests started failing. I only looked briefly, but it looks like the zinc compiler can't handle there being source path values.


Tą funkcję można by uprościć z wykorzystaniem modułu datetime i użyciem metody timedelta:

>>> str(datetime.timedelta(seconds=3666))
'1:01:06'


OK. I remember we had other issues with mpl integral beign inserted into other hierarchy and the value type beign wrong which is what that fix. I thougth we had an open issue on this but I was wrong.


> > Намирам, че употребата ми е коректна.
>
> С тоя език да не си се засилил да ставаш депутат? :)

Добре знам как звучи :) Забавлявам се вътрешно като използвам такъв изказ.

Депутат? Бъдещето може да крие всичко :smile:


@pombredanne thanks for saying so :)


OK, I will merge two pio pieces into one


this is also obvious IMO

If one wants to add this info one could do:

```
    compiled_pattern = re.compile(pattern, flags)  # For better performance
```

in one line.


Was that rename intended?!


czy to ma być funkcja prywatna? Jak tak to trzeba dać jeszcze podkreślenia na końcu nazwy __moja_funkcja__. Pytanie tylko czy w tym wypadku to jest zasadne :)


+ za list comprehension i od razu zwracanie tego!


Ah... nice. I just extracted the URDF visualisation (see RobotWebTools/roslibjs#157), and this and the STL loader where broken. This patch should be backward compatible though, while the STL Loader I guess is not.


thank you ;D

Ok, I will re-arrange it, I think I can just move it one step upwards and
commit or I will clone a new one or something.

/Johannes

On Mon, Mar 9, 2015 at 1:11 AM, Malte Münchert <notifications@github.com>
wrote:

> Johannes, first: thumbs up for committing code on a Sunday (and boy, did
> the sun shine today!). I noticed you put your app inside a subdirectory in
> your repository. This might cause problem when deploying to Heroku. It will
> try to determine which programming language and framework you use and will
> expect the project to be on the root level. So you may want to move your
> stuff. OR you can try something like this
> https://coderwall.com/p/ssxp5q/heroku-deployment-without-the-app-being-at-the-repo-root-in-a-subfolder
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/StartupInstituteBerlin/Spring2015JohannesHildinngsson/commit/cdd283467f3258501eacc9fdec3c89e658817805#commitcomment-10091551>
> .
>


Do you mean "comment" part ? OK, I will fix


nope, fixed in later commit.


Will fix it


All good now.

Thanks
On 09/03/2015 7:05 PM, "orbitcowboy" <notifications@github.com> wrote:

> I added the missing podtype (timeval) by 95923cb
> <https://github.com/danmar/cppcheck/commit/95923cbfea052e7cfe55c230047c559ca8faba4d>.
> Could you please verify if --check-library is still unhappy?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/danmar/cppcheck/commit/3d9c2b4d979cf6693e0cf1cfa3db14a6c3a17586#commitcomment-10094093>
> .
>


upravit stylom:

option( OSX_BUNDLE "Create an app bundle on OSX." OFF ) # TODO: disabled, because currently it does not work...
if( APPLE AND OSX_BUNDLE )

resp.

option( WIN_BUNDLE "Create an app bundle on Windows." OFF ) # TODO: disabled, because currently it does not work...
if( WIN32 AND WIN_BUNDLE )


good point. fixed in 0905fb9

Thanks


eline saglik.


uvedene zmeny by mali byt "future-proof" a aby sa dali dobre mergnut
- komentovanie zrovna nie je dobre riesenie pri mergovani...


Hello,
For the `get_request_header(...)`, it's `Accept-Language` the good variable name. Why this change so because they are the same values in theory, no?


Why was Feign added? It's an almost carbon copy of Retrofit?


Нехороший ты человек Слава...

Интерфейс командной строки имеет уже устоявшиеся принципы:
Короткие (однобуквенные) опции начинаются с одного знака "-":
-t, -f
-tf - это две опции -t и -f (короткая запись так сказать)

Длинные же опции (более одной буквы) начинаются уже двумя знаками "-":
--from-file, --points


Контакты нужно научиться доставать из таблицы https://gist.github.com/ilyubin/f7995f18a9adc1b0590b
Неверно вытаскивать что-то из скрытых аттрибутов чекбокса.


jw. odnośnie jednolinijkowego docstringa (wiem, że możę nie w całym tym projekcie jest to tak stosowane, ale warto wiedzieć - w VW tego staramy się trzymać :) )


Oh, OK then. Maybe there should be a similar integ-test as we added for Java for the Groovy and Scala cases? (I didn't know where and how to add them, and I'm neither a Groovy or Scala user so I don't really know how things work in those cases)


Is it a good idea to have language dependend text in this icon? Must this png be "translated" for every new language, then?


Митко президент!


Why not put this into a regular doc comment? Also all the information given in that comment is provided by just reading line 6 once.
search for ... generator ... pattern


wiem, że tutaj to jest mało zasadne, ale ogólnie staramy się by importy były w kolejności alfabetycznej w modułach.


Hi,

Sorry for the delay, give me your email adresse, I'll send you the full print.


Thanks @tomquas. Looks good, and I'm going to merge it.
Though looking at this I wonder if it wouldn't be better passing websocket params proxy creation time. It would become something like:

```JavaScript
var ws = require('net.iamyellow.tiws').createWS({
   url: 'ws://<IP:URL>:<PORT>',
   protocols: ['chat'],
   headers: [
     'If-Modified-Since': 'Tue, 15 Nov 1994 12:45:26 GMT'
   ]
});

ws.open();
```

I'll try going in this direction with the next release.


greenlighteado y a early acces


tutaj też mozna by to pd razu zwracać bez tworzenia dodatkowej zmiennej.


ogólnie spoko!


чтобы не удалять лишние ряды. Ищите tr[name='entry']


А email и телефон не будем доставать?


mind the semantics of `computeThumbnail` to resolve https://github.com/shaarli/Shaarli/pull/153

In short: `computeThumbnail` has to return a non-empty array for soundcloud or you won't never ever see a thing.


Hi,

You can send it to jocelyn.fournier@gmail.com


What's wrong with this way of implementing it?


:+1: Thank you - awesome!


test1,2,3 pass.


I gave up and set up a py3 test environment for ePad to test things like this.


Results for parsing and pretty printing Symfony 2.6 (4969 files) on PHP 5.6 and PHP 7 before and after this change:

PHP 5.6 | before | after | diff
---|---|---|---
parse | 29.5s | 25.6s | 13%
print | 5.8s | 3.7s | 36%
memory | 148MB | 126MB | 15%

PHP 7 | before | after | diff
---|---|---|---
parse | 11.8s | 10.0s | 15%
print | 2.7s | 1.7s | 37%
memory | 121MB | 89MB | 26%

Tests weren't particularly rigorous ;)


They are not executed when they are added to the `dependencies`. The `assets` Class will exectue them vor every view later.


Hi!
I'm challenged making replicaset by chef-solo and vagrant.
But formula cookbook is for chef-server.
Can this cookbook break through chef-solo?


Merging Archit's code


this change makes springlobby hang on windows xp. see #367

i've made this a runtime config which defaults to disable:
https://github.com/springlobby/springlobby/commit/367d4b42d572ca43c96da64ea17e39a7c671f9b4


I just challenging!
Please Wait.


in the whole testsuite you don't ever give search_for a valid non-default `max_matches` value... thus you don't test this functionality. You basically test re.finditer the whole time, do you?

Also when reading the code above I found a whitebox cornercase which should be really tested: matches that are one char long. This needs testing with your generator function because you do `pos = current_match.end()` and I think you'll need a `+1` there. (Bug!)

Another corner case: It is also possible to make a regex so it does not match anything, a zero char match. I think this will result in an endless loop (maybe not with the `+1` above).


imo to really fix flickering the ammount of full redraws has to be reduced. atm it seems every update of a list control does full redraw it which apparently is a bug. afaik all listctrls in springlobby are a child class of https://github.com/springlobby/springlobby/blob/master/src/gui/customvirtlistctrl.cpp so it shouldn't be this hard to fix i guess.


>  What if the code before the timeout added a second dependency

This method collects all dependencies added in the same tick. So the scenario you describe is the standard case for this method.

`@dependenciesAdded.fire(@jsWaitlist, undefined)` will notify listeners of the added dependencies and then immediately deletes them.


could do the old click on the right half of the screen for jump, left for slide

buttons are hella lame


It worked!

![eX0 in browser over WebSocket](https://cloud.githubusercontent.com/assets/1924134/6551972/9e2e7104-c5fd-11e4-9974-67bd98be924e.png)


Should that not be ```'|', '|', ('id', 'in', ids),```  ?


````c++
for (int i = 1; i < argc; i += 2){
    std::string  opt = std::string(argv[i]);
    ...
}
````

Кстати, шаг 2 наверно не лучшая идея, можно иначе:
````c++
for (int i = 1; i < argc; i += 1){
    if( argv[i][0] != '-' ) continue; // Опции начинаются со знака `-`
    std::string  opt = std::string(argv[i]);
````


sorry, but basically almost all the comments say: remove that comment :) so it shouldn't be *that* much work...


If you want to execute something like `alert("hello world")` then it has to be executed every time.


That's awesome!
On Mar 9, 2015 1:43 AM, "Dmitri Shuralyov" <notifications@github.com> wrote:

> It worked!
>
> [image: eX0 in browser over WebSocket]
> <https://cloud.githubusercontent.com/assets/1924134/6551972/9e2e7104-c5fd-11e4-9974-67bd98be924e.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shurcooL/eX0/commit/7ba48d00cfd143bca2e5652cd04bc92c262a29d2#commitcomment-10094637>
> .
>


Sorry, I forgot to report here. Tested, works in py3k, with normal file paths and file urls.


Hi, I was looking into the AppCompat source and in Material design there is a different animation:

   <style name="Animation.Material.Dialog">
        <item name="windowEnterAnimation">@anim/popup_enter_material</item>
        <item name="windowExitAnimation">@anim/popup_exit_material</item>
    </style>

I suggest you extract this animation from the source and add it to the library. Now you are using animation which was there before API 21.


this comment is also obvious from method name don't you think?


done


Looks good to me. One question though: is there a special reason for the 'find_in_dict_or_clone' method to be protected instead of private?


No, although it can be the same result, I think it twice, and the correct form is this. This equals to:

(invoice.payment_mode_id = False OR invoice.payment_mode_id = payment_order.mode.id) OR id in ids


closes #1


This is Livingdocs internal markup decoration. And the decorator is for user specific decoration.


saw approval from shino
at https://github.com/basho/riak_cs/commit/43be47631e7e5f93f61da9ee4af4424691f1f035


merging basho/riak_cs/feature/check-signed-date = 43be4763 into borshop-integration-1067-feature/check-signed-date


basho/riak_cs/feature/check-signed-date = 43be4763 merged ok, testing candidate = 66b997c6


I'm not sure whether we want this. The original URL makes more clear that we're the "official" Git for Windows. However, you're right that people looking at the `PKGBUILD` are probably more interested in the internals / source code, and for these the new URL would be better.


możnaby się pokusić jeszcze o testy tych metod, które sam dodawałeś np. tej zamiany sekund.


Right. Will do.


@atteroTheGreatest it's not about `code_boxes` vs `codeboxes` vs `code_bo_xes`. I'm fine with 'codeboxes', but it's about consistency. `code_boxes` implies `CodeBoxes` endpoint name, `codeboxes` `Codeboxes`, `traces` `Traces`, not `ScheduleTrace` or `WhateverTraces`. The way the API is implemented is inconsistent, confusing and requires quite some ifs (and more to come, ex. for `traces` if you aren't gonna change names). Or we shouldn't have used HATEOS in the first place ...

"A REST client needs no prior knowledge about how to interact with any particular application or server beyond a generic understanding of hypermedia." http://en.wikipedia.org/wiki/HATEOAS


Thank you.. Fixed!


Please to not keep out-commented code.


Yes this will attach `component` and all of its descendants to the DOM


Please remove this empty line.


In German you don't use “»” e.g. as quotes. If it should represent an arrow in this case I think “»” should be alright...


Ahhhh. copy+paste :boom:


done


Fixed, it'll be in the next commit.


Ok, good point. Vlan reporting is still a little rough around the edges, I just put it up so people can take a look at it and comment.


@Matth--  thank you


Created [OGM-766](https://hibernate.atlassian.net/browse/OGM-766).

We definitely can avoid the looping/re-creation here, but I'm a bit skeptical about the fine-grained error reporting. At least for MongoDB we cannot tell which elements of a batch passed/failed.


Tip: if you want to commit something that definitely doesn't change the code, you can add "[ci skip]" to the commit message to, well, skip CI. :)


Right, they are equivalent indeed.


that's a good point, I suppose I was trying to avoid over-complicating it but it's probably worth spelling out the choices available there.


+1. your proposal should emphasize the fact that a websocket created via #createWS should not be reused by multiple calls to #open with different parameters.


This does not seem to be right without 4a83e89d8d18fcf5b4d746d162479f23bc68faad, because you basically still fetch the same source code (just from a different source) but don't apply the patches anymore. Also, patches should be removed in the same commit they are not applied in anymore. I propose to just change URLs here, and squash the removal of the patch file entries with 604d81e598af1c42f7e3f3926e7cbb83f489813b.


Actually, no I can't. I believe that, as of Unity 5, gameObject.transform is deprecated and gameObject.GetComponent<Transform>() is favored.


Do you have time to create a PR?


hm .. ah oui il faudra que je te crée quelque chose pour toi. Je m'en occuperai mardi midi.


I'll do that for other pulls, thanks for the hint!


Easy do. If input ... X is < screen size / 2 do something.


Sure np

On Monday, March 9, 2015, Arjen van Bochoven <notifications@github.com>
wrote:

> Do you have time to create a PR?
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/munkireport/munkireport-php/commit/036187e407cdc4b31fa813672832d6e139918d58#commitcomment-10094792>
> .
>


@temasek this not has sense have the old cmhw manager due to cm switch to new cmhw it requires move sepolicy from system_app to system_server and it will gives fc to devices using KeyDisabler


Alright, I can. But as far as I know, it doesn't negatively impact performace to use GetComponent<Transform> over .transform and I kinda prefer it that way.


Hum... weird
"For Unity5, we decided to bite the bullet, and remove all the quick component property getters with the exception of .transform (since that one belongs in the core module, there’s no need for it to go, and it’s also by far the most used one)"
http://blogs.unity3d.com/2014/06/23/unity5-api-changes-automatic-script-updating/


done, but can't be a oneliner


kkk


lgtm, ack


Hehe as you like :)
I know that GetComponent is slower than just accessing transform (now with Unity5). If it's done just once no big deal, but multiple times a frame it should be .transform.

Also I was saying this because, there is no need to get the RectTransform because we have no use for it, that was the point ;)


A mistake.


Ah right, hadn't realised - yeah, that's fine then. They are at least more readable this way.


Note that the function 'set_global_data_param(...)' in settings.c is strictly copy pasted from within 'communications.c:handle_mavlink_message(...)'


Done


Use a guard clause instead of wrapping the code inside a conditional expression.


Use %r only for regular expressions matching more than 1 '/' character.


Use `described_class` instead of `PhoneNumberValidator`


Spec path should end with `phone_number_validator*validate_each*_spec.rb`


4: L102 copyright year is outdated, expected 2015 but got 2014
21:1: I201 Missing newline before sections or imports.
23:1: I100 Imports statements are in the wrong order. from mock should be before import pkg_resources
24:1: I101 Imported names are in the wrong order. Should be ImportPathRegistry, PkgResourcesDirDiscoveryRegistry, RegistryProxy
24:1: I100 Imports statements are in the wrong order. from flask.ext.registry should be before from mock
24:1: I201 Missing newline before sections or imports.
29:1: I101 Imported names are in the wrong order. Should be InvenioTestCase, make_test_suite, nottest, run_test_suite
43:1: E731 do not assign a lambda expression, use a def
45:1: E731 do not assign a lambda expression, use a def
47:1: E731 do not assign a lambda expression, use a def
586:42: E231 missing whitespace after ','
586:44: E231 missing whitespace after ','
599:17: E127 continuation line over-indented for visual indent
599:48: E228 missing whitespace around modulo operator


ack


remember to add the `g` flag


21:1: I201 Missing newline before sections or imports.
23:1: I100 Imports statements are in the wrong order. from mock should be before import pkg_resources
24:1: I101 Imported names are in the wrong order. Should be ImportPathRegistry, PkgResourcesDirDiscoveryRegistry, RegistryProxy
24:1: I100 Imports statements are in the wrong order. from flask.ext.registry should be before from mock
24:1: I201 Missing newline before sections or imports.
29:1: I101 Imported names are in the wrong order. Should be InvenioTestCase, make_test_suite, nottest, run_test_suite
43:1: E731 do not assign a lambda expression, use a def
45:1: E731 do not assign a lambda expression, use a def
47:1: E731 do not assign a lambda expression, use a def
586:42: E231 missing whitespace after ','
586:44: E231 missing whitespace after ','
599:17: E127 continuation line over-indented for visual indent
599:48: E228 missing whitespace around modulo operator


this comment doesn't add any real info, does it? I usually use comments to provide a reason for what I'm doing. What one is doing is (and should) usually very obvious.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/885


@Montellese Thanks! That did the trick!
Will cleanup a bit and create a PR for discussion

cheers


I know this ;). This is why I wanted to do this. However, sometimes decisions aren't ours. @23doors and @mkucharz wanted to leave it as it is now.


hold on a sec, you're saying that the default section holds itself as default section. The Section class should have an assert that prevents self-referencing this way. Did you run tests on this commit?


I just took what worked well on the imx6 (also armv7) and ported it to the bcm2807 (armv7). Wierd how this does work well on the imx6... @mk01 and i also discussed this issue on github, `who is allowed to push what fix directly on github?`


Hmm... but then this is not really a `Callback` but rather a list.


I understand the point. It's only that it feels weird that you type `yes` instead of `YES` and you get `aborted`. But on the other hand, you force people to truly think **twice**.


Test functions for analysis.py


OK, at least now it's clear we need more `if`s ;). Any decision about other inconsistencies, like various `traces` -> `XxxxTraces`?


what I meant with some more corner-cases, is that you provide one section on one of those coafiles only and one other section in the other coafile only. This way you test if the None check introduced in the last commit works right.


I'd also keep those here and add the others.


you also should pass `ignore_defaults=True` in the `update` call in `_merge_section_dicts`, shouldn't you?


Think of a `public class FreshClassLoader extends ExternalResource`.
Then you can use it like this:
```java
@Rule
public MethodRule freshClassLoader = new FreshClassLoader();
```


Done


We would have name collisions internally if we had all those endpoints having the same names. It would need some additional work that would make our code rather worse than better.


Got it i'll remove this comment.


@tractorcow Could you investigate why the German translations are out of date? A few people have mentioned it to me now, not sure if it’s something wrong with the script (IIRC this process is automated?): https://groups.google.com/forum/#!topic/silverstripe-dev/PcGki_saaak


Yes, for the comment part and the line itself, add space if needed so it look like this:
reg = <0 0x14000000 0 0x120>,
         <0 0x1400C000 0 0x1000>,


K<5F>  + <nF>


Les données de l'utilisateur sont cryptées et brisées en chunks (morceaux). Ces chunks sont ensuite distribués à d'autres Vaults dans le réseau. À aucun moment les autres Vaults n’ont d’informations sur les données qu'on leur a demandé de stocker autre que des informations anonymes d’adresse réseau.


In the next commit this test is removed. It is present here because earlier self referencing was happening and in a later commit it was removed. https://github.com/Udayan12167/coala/commit/f034774e9996c8d44d0d8e02a10803303f501071 take a look at this commit.


sorry, forgive me this time, it is urgent for me, but no one answers me on stackoverflow.. Could you answer me ? http://stackoverflow.com/questions/28937941/resource-is-not-indexed-in-grails-project-and-sonar. thank you very much. the last comment here


2/2
Good work!


<5F> +  <nF>


Projekt start.
 - projekt created in: htdocs, githubs.
- index.php and screen.css - start


Il a été demandé de conserver sans traduction certains mots clés de SAFE. cf le thread de traduction du forum


fast-forwarding develop to borshop-integration-1067-feature/check-signed-date = 66b997c6


I didn't mean to change `ScheduleTraces` to just `Traces` and have a few traces endpoints, sure it'd collide. I meant to change links` names, so it's link `schedule_traces` -> `ScheduleTraces` endpoint and so on.


Will do.


`Schedule` with `schedule_traces` is in my opinion worse than it is now, it's unnecessary info here.


Try to replace the $results line by :

```php
			$results = Db::getInstance(_PS_USE_SQL_SLAVE_)->executeS('
					SELECT  p.`id_manufacturer`, COUNT(DISTINCT p.`id_product`) as nb_products
					FROM `'._DB_PREFIX_.'product` p USE INDEX (product_manufacturer)
					'.Shop::addSqlAssociation('product', 'p').'
					LEFT JOIN `'._DB_PREFIX_.'manufacturer` as m ON (m.`id_manufacturer`= p.`id_manufacturer`)
					WHERE p.`id_manufacturer` != 0 AND product_shop.`visibility` NOT IN ("none")
					'.($active ? ' AND product_shop.`active` = 1 ' : '').'
					'.($all_group ? '' : ' AND EXISTS (
						SELECT 1
						FROM `'._DB_PREFIX_.'category_group` cg
						LEFT JOIN `'._DB_PREFIX_.'category_product` cp ON (cp.`id_category` = cg.`id_category`)
						WHERE p.`id_product` = cp.`id_product` AND cg.`id_group` '.$sql_groups.'
					)').'
					GROUP BY p.`id_manufacturer`'
				);
```


Changed it.


HHVM 3.3.5 已经解决了 die exit 的问题了吧？https://github.com/facebook/hhvm/issues/993


Removed.


I dont get the requirement of ignore_defaults here. This is a simple assignment of defaults?


Okay. Now I have something like this while booting http://pastebin.com/GBsuAeRs
Oh and I didn't have any segfaults on ART while compiling


Can't think of a nice way to do this, so will probably drop this commit.


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


symbols won't be exported with mingw/cygwin, use _WIN32 instead of _MSC_VER
then you need to define dl_EXPORTS with makefiles too maybe.
be careful than dl_EXPORTS is not defined when building static libs.


Ok, I submit a PR then !


licence


Its simpler yes, but it omits the recent functional addition that made it callable from outside the git repository. It also does not redirect any git error message. I have made a few small modifications in :   1efee5e7d90ca988df490d649d48d5b86c84477d to address this.


But for every commit all tests should pass. (This way you can revert commits without danger if something goes wrong.) I suppose this isn't the case here.


The difference has been cleared up for me.


No its a generic icon I either put some text or no text at all.


OK, I can merge it


When runing the console(rails c or rails console) you are running like the application. In everywhere when you do Post.all or Wall.all, or wall.posts.all, it returns ordered by the id, by default. But like I said to you we can cannot rely on those ids.


I mean  line 146: https://github.com/Udayan12167/coala/commit/f034774e9996c8d44d0d8e02a10803303f501071#diff-a0e4f723f6df10e75b0dc8fc4e75584dR146


github prevents me from commenting there.


Solves issue #2 and #1


All tests passing :+1: .


Looks significant enough change to do a LKG.


will fix it


Just a side note. API schema has nothing to do with hateoas. And API can be traversed without any schema knowledge. When it comes to names - schema outputs views name that are internally used. That's not something dependable. And I agree with @atteroTheGreatest that forcimg unique endpoint names in nested relation is not a good solution and too verbose.

And about CodeBox -> code_box. There is an open discussion on aha and it's not really all that simple. I for one disagree about having to split it when we're dealing with a product name comboned of two words that are not meant to be split. Similarly GitHub should remain github not git_hub and so on.


So by passing ignore_defaults=True we are saving on some extra bit of computing. If i get you correctly ?


Will fix it


This creates an object of the type wall, using the factory, with the defaults from there.
```ruby
let(:wall) { create(:wall) }
```
if you do wall.posts it will return a empty collection, you can try it yourself by raising it like:
```ruby
it "raise something" do
  raise wall.posts.inspect
end
```

Here this one returns an object of the type wall with the defaults from the factory but I also put 2 objects of the type post(using the factory), inside the wall. so this returns a wall with 2 posts:

```ruby
let(:wall_with_posts) { create(:wall, posts: [build(:post), build(:post)]) }
```

I call it wall and wall_with_posts to distinguish the objects so I can use them in the tests. Does that not make sense?


> probably handleAppliedoperations and handleException could receive the errorHandlerManager

Moved this all to the dialect as per your suggestion


Alberto Rodríguez de la Cruz


> Still really bothered by that name ErrorHandlerManager

Renamed it into `OperationCollector`.


Go add Os to your sensor source's Android.mk
On Mar 9, 2015 6:15 PM, "Sebastian Jena" <notifications@github.com> wrote:

> Okay. Now I have something like this while booting
> http://pastebin.com/GBsuAeRs
> Oh and I didn't have any segfaults on ART while compiling
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_build/commit/2dd7b438743d066ecbea92dc1f180b49692fdd65#commitcomment-10095232>
> .
>


Please stick to formatting ;)


I dont understand what you mean...

```ruby Post.most_recent``` returns all the posts ordered by the created_at, when I do ```ruby Post.most_recent.first``` it returns the first of those posts...

Can you  be more explicit about your doubts?


Yes, good idea. Done.


And we're also saving complexity and potential bugs.


Moved to separate class and documented it.


Adding installation information in the README.


Formatting ditto. Brackets on same line as blocks (except for functions), whitespace after `if`, `while`


If I understand [GitHub Pages](https://pages.github.com/) correctly, this should be a seperate repository with a very specific name rather than a branch of the main repository. More specifically, that one should be the repository for the future docs, we're probably going to need another one for the homepage. I think. Maybe. ;)


hm, we don't compare the reference but the contents... I suppose that might be ok. Ok. THats ok then.


current_user ? social_network_data_processing : find_or_create_user


Since this is the third place such a "Mono.Runtime" check would be used, it would be better to put it somewhere into `util`.


This lack consistency between nspi/emsmdb for the same portion of code. Either all of nothing, but given that it is the only place where we use [nspi], then let's go for nothing and revert this one.


`return rkApp.GetValue(AppName) != null` seems more compact


Actually, HC Yen already send to public:
http://lists.infradead.org/pipermail/linux-arm-kernel/2015-February/324757.html

He will continue this discussion and finish the upstream


private readonly?


Ok, I missed that, sorry.


too  much -> too many


Since `AllUsers` is never true, I'd remove that. Otherwise it make it an enum, not a bool


@ajs-sun I don't think we need to bother about this for now. We will most likely handle this in another way soon.


well, you know about the newlines, do you?>


my comment


дважды повторяется фактически одно и тоже условие

`wrong_social unless @credintial && (current_user.id != @credintial.user_id)`
перенести вверх. Рассмотри плохой случай сразу и откинь его (сделай return из функции)

а дельше в функции оставь только полезную нагрузку.

и тут ты уже видимо сможешь не проверять `@credintial && (current_user.id == @credintial.user_id)`

а просто использовать `wrong_social`


@alviteri My hlte has the sepolicy already in place long ago in system_server. No issues for hlte using old cmhw as of now. Works for now till your goodself update the changes to use new cmhw as I'm using crdroid implementation of navbar + hw keys combo. Thanks.


`setWinBoot` is not the best name probably. Something like `InstallAutoRun` would seem more appropriate...


@ajs-sun @pascal-brand-st-dev ok, then I suggest to remove it for now.


`UninstallAutoRun`?


my comment2


So it will be there? :
/device/lge/hammerheadcaf/libsensors/Android.mk


so, you changed documentation here but not in the commit before. Most of the comments are generally applicable.


A more provocative commit message might have been: 'israel and the palestinian territories no longer exist'
:+1: for your editorial restraint


cute, looks good!


what you don't define in that comment is whether the split pattern is contained in the parts.


Yeah, you're probably right, sorry.

For now we can work inside that repo and move it to a fresh one before
it gets live. Main reason would be to allow different read/write access
on the seperate repository, so as soon as that happens we could make the
move.

On 09.03.15 10:26, Matthias Mees wrote:
> If I understand GitHub Pages <https://pages.github.com/> correctly, this
> should be a seperate repository with a very specific name rather than a
> branch of the main repository. More specifically, that one should be the
> repository for the future docs, we're probably going to need another one
> for the homepage. I think. Maybe. ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/s9y/Serendipity/commit/bdf090b80be08bb3155b53187928c6ddfeaaaba8#commitcomment-10095420>.
>


It's used in different places on `mapiproxy/servers/default/nspi/*`:

    $ grep -irn "\[nspi\]" mapiproxy/  | wc -l
    13

On emsmdb I have not updated it because the debug messages needs a lot of change (in relationship with #143)


Thanks, I'm fixing it!


Ca test


Plase adjust trimVcal to be not a fixed variable.


Done


ret


maybe we can add this for preparation as well?


I don't think audio is bad. But I guess I'll need to watch videos of it being recorded from console.


Wooo, Isn't LocalPickUp a delivery module ?


Maybe
On Mar 9, 2015 6:34 PM, "Sebastian Jena" <notifications@github.com> wrote:

> So it will be there? :
> /device/lge/hammerheadcaf/libsensors/Android.mk
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_build/commit/2dd7b438743d066ecbea92dc1f180b49692fdd65#commitcomment-10095518>
> .
>


Hey @dhanvi - I think we should introduce a rule which writeups should appear first. E.g. there are several types of writeups (writeups that explain the solution in English, writeups that explain the solution in another language, writeups that just present the solution - for example gist/pastebin link). The order should be as just listed and in between the types the order is FCFS :).


I don't see the point of this commit. It does the same but has less type information.


soryy.. thats was my fault... ive started using cloud 9 (https://c9.io/) webbased development environment.

it says to use process.env.PORT as the port..    i had meant to put it back before i commited that change.

but thats cool if you can do it that was so it works both ways.

teh rest if cloud9 is mostly fine.. with 2 exceptions.. i havent got livereload to work as it needs a second port.. and there is not enough disk space provided to compile the flash file. (or more specifically install eth flash sdk)


Not sure why the wrong provider name was in here, it has been exported correctly by me (https://scr.meo.ws/paste/2015-03-08-02-58-52-LghAdsn1.txt) but @jedisct1 apparently made a mistake. Anyway, thanks for fixing!


@todrobbins re blog subsection i'm happy either way (i guess we need some way to link to them and its standard so why not). Re the pull the request - normally you can just extend it re-adding stuff or submit a new one.


#3 @arrowrowe  change the url here.


translated everything except 2013 OWASP Top 10 (commented)


you may also need to change the form data. don't know if array will be serialized properly.


https://www.facebook.com/v2.0/dialog/oauth  à test


Hello, two remarks on that diff (yes, I know about the $iocroot change)
1. why not `cp -p`
2. why the `cp` at all.
`chown(1)` and `chgrp(1)` are the same hardlinked binary and you are breaking that while losing `chown` in base.

Am I missing something?


trust me its completely broken just like wdc

is fine with azimer's audio though


Cleanup.


Here's what I thought (but did not put in the Titanpad, sorry):

* s9y/s9y.github.io – that will *have* to be the repo for docs.s9y.org because that's supposed to be powered by GitHub pages
* s9y/blog-theme – the s9y theme for blog.s9y.org
* s9y/board-theme – the phpBB theme for board.s9y.org
* s9y/website – the onepager for s9y.org

However, I'm still mulling about how to handle the common assets (CSS, JS, images, icon font?) for these. I think it would make sense to have kind of a “s9y web framework” that all of these can use, but on the other hand, the forum theme for example will need quite a lot of styles that the other pages/sites/theme don't need …


@ThiefMaster, I think you mentioned something at this line before. I lost the comment due to force push. What was it? Thx.


This is already what I have..


is there a reason why mmc2 is missing?


客户端连接本地1234的udp端口，监听本地的8787端口作为socks5服务，传输内容加密，默认建立5条固定通道（防止大数据传输或者网络不好时单条通道拥堵）

I can see your optimize for connections. i thought we can provided a connection pool to manage the connections automatic.

Set a min connection and a max connections

open new connections when need.

for example. Min 3 connections, Max 50 connections.

if current active connections increase, just increase connections , until reach the max amount 50.

when no active connections, auto close no need connections, to free the memory and the cpu resource usage.

It would be useful for heavily usage.


Actually the braces around the GROUP BY are not at the same place :)


I think this has clossed #634 by error when you wanted to say #633, could you be able to re-open it?


Fair enough.


you can still check the comments by going to the old commit's page by modifying the url

anyway, i think i suggested using `to_unicode()` here. but i just remembered that we have a jinja extension which handles it automatically if necessary, so it's not absolutely required. still cleaner to pass unicode objects imo


I didn't know `arguments`. Also I remember I had a compile error before that commit, but this is not the case anymore.


selected ? maybe. for `enum` does not help reader to understand the code.


I can do that if you guys want it.
The change isn't that big though, only some .d.ts files will change.


No this is not an error, the glossary is sort in the new `gitbook-parsers`: https://github.com/GitbookIO/gitbook-parsers/commit/901542df30483f8c4b349f64737e0b33954aa788


Avoid using id selectors


`end` at 12, 4 is not aligned with `expose(:projects_sorted) {` at 6, 2


Avoid using {...} for multi-line blocks.


Use 2 (not 0) spaces for indentation.


Avoid using `rescue` in its modifier form.


Line is too long. [101/100]


Unnecessary spacing detected.<br>Trailing whitespace detected.


Use the new Ruby 1.9 hash syntax.


1 trailing blank lines detected.


The first argument to describe should be the class or module being tested.


The test shouldn't be tied to a specific implementation.


Line exceeds maximum allowed length


Unnecessary fat arrow


Line contains inconsistent indentation


Color `#2C6CA3` should be written as `#2c6ca3`


Selector `moveLeft` should be written in hyphenated BEM (Block Element Modifier) format


Selector `moveRight` should be written in hyphenated BEM (Block Element Modifier) format


`0px` should be written without units as `0`


Avoid using id selectors<br>Opening curly brace `{` should be preceded by one space


Color `white` should be written in hexadecimal form as `#ffffff`


Avoid using id selectors


`0.6` should be written without a leading zero as `.6`


Oh, sorry for the inconvenience! Thank you for your work :-)


Merge rule `.show-notes` with rule on line 38


Merge rule `&:checked ~ label` with rule on line 188


Avoid qualifying attribute selectors with an element.


Shorthand form for property `margin` should be written more concisely as `5px 0 0` instead of `5px 0 0 0`


Each selector in a comma sequence should be on its own line


I think it's good to have the possibility to choose between more than two implementations (default + platform specific), but as long as you have a PRNG with enough entropy for cryptographic operations, then I can accept only having two choices. So, please update based on Pasal's comment. This comments relates to the comment below about CFG_WITH_PRNG_SOFTWARE also.


Yep, saw it but it doesnt work better :/


That looks wrong to me. It's not the responsibility of the Table class to convert the data to display. You should instead do that before feeding the table with data.


I don't understand the commit message. can you be a bit more specific what was there to fix?
`crowbar machines list` gives you a list with new lines. if you call it with backticks it will truncate the new lines.
i think this commit is not really necessary here


Jo jag förstod det. Men om man använder parameter expansion och sen gör tvärtom inne i metoden. Stöds detta?


@temasek  and you having this configuration .. you have still display and vibrator cmhw working

https://github.com/temasek/android_device_samsung_hlte-common/blob/cm-12.0/sepolicy/system_app.te

mmm I saw now

https://github.com/temasek/android_frameworks_base/commit/c54885a1097eaa50f59baf91b034aed22ee9eeb4


There is nothing to do here: there is no change, I just reordered lines to make the comparison possible.


anyway if you can see any conflict for have 2 services running at the same time .. for the same function tell me ... Thanks


+[src/cls/rbd/cls_rbd.cc:2161]: (warning) Calling 'sizeof' on 'sizeof'.


Thank you, yes you are right.
This has been fixed in the next version (that I am currently developing with our 3rd party).
The next version's goal is be able for public review, and after it is in the mailing list,
I will commit it back to dev-3.18.


I have an error in Rainlab.Pages plugin, when I want to create a Menu:
Call to undefined method RainLab\Pages\Classes\MenuItem::resolveAttribute()" on line 917 of /modules/backend/Widgets/Form.php


Ah you're right, guess it's been a while since I used Jabo's audio for this game.


the API name is "displayTask"


Hmmm I'll play around with it today.
On Mar 8, 2015 10:43 PM, "Jay Dolan" <notifications@github.com> wrote:

> Hm, weird. Maybe I'm calling wcstombs incorrectly? We do need to convert
> from wide char (16 bit) to char (8 bit) here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jdolan/quetoo/commit/2cef172e817bce6af03aed4b87adb0e11c842906#commitcomment-10092240>
> .
>


Why not a static map?


I am using Editpad pro and I cant for the life of me figure out how to fix my indentation, its really weird.


I don't think libsass guarantees to be thread safe, but I wonder why this could be an issue. But I'm not against you're solution. IMO it is the one also mentioned on stackoverflow, where you prepare the string, so atof will recognise the "dot" in the expected locale. I thought it is a bit over engineered, but if this is what it takes to make that working as it should, we should do it! Maybe you're willing to create a PR to get this included!? Thanks!
Although I'm still curious what the issue could be. IMO it could mean that other threads that do execute atof could get a wrong locale, while we switch it for to "C" for our atof call (and to switch it back later) ... right?


One reason: it allows us to build a `FetchSpec` into a single string of params before we know the URI. But mainly just 'cos it turned out this way.

(This code originally used `URI.resolve`, but it turns out that that method doesn't always work the way you'd expect. Code evolves.)


ok


His culture was changed to adnanite a few lines below that. I wasn't sure how set_graphical_culture worked, Ill go ahead and fix it.


Hmm, so I think we actually don't want this.  We'll want the underyling type to be able to elide itself as necessary.


Did you post a completed file that is zipped, that I can download?

Bob Beroza
Sent through my iPhone 6

> On Mar 9, 2015, at 4:44 PM, Dave Reed <notifications@github.com> wrote:
>
> Just tried it on my iPad. This is the error I was getting:
>
> 2015-03-09 19:31:45.015 AppSalesMobile[285:26334] *** Terminating app due to uncaught exception 'CALayerInvalidGeometry', reason: 'CALayer position contains NaN: [35.5 nan]'
>
> I set an Exception breakpoint in Xcode to find where it was and did a hack to fix it. See code here:
>
> https://github.com/dave256/AppSales-Mobile/commit/4fb1d1d48eda7a2415f49bfffe9f071ade1b0b44
>
>
> > On Mar 9, 2015, at 5:50 PM, RBConsulting <notifications@github.com> wrote:
> >
> > Thanks Dave
> > I have been using App Annie but I really like AppSales better.
> >
> > Please let know if you figure it out.
> >
> > Thank...Bob Beroza
> > Sent through my iPhone 6
> >
> > > On Mar 9, 2015, at 1:59 PM, Dave Reed <notifications@github.com> wrote:
> > >
> > > It still works on my iPhone 6, but stopped working on my iPad a while ago (sometime last fall I think).I haven't looked into why.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub.
>


Your sync instincts mislead you, young jedi.

Reading list doesn't use timestamps for syncing; it uses change flags. "Modified" here means a set of records with certain flags set.


@mikaelpatel did not want this to be included in previous pull - but looking for some feedback on this one.
Should I go further and break backwards compat a bit to save valuable memory given moving to registry is probably going to break things a lot?


It's 49.92Hz ;-)


Done!


Woops! Ill fix this.


Good question.  I thought about that but decided against it for two reasons.

1) This is really a Linux specific issue so keeping the fix in the already Linux specific zpl_* layer seemed best.  If we can avoid changing `zfs_inactive()` that's a good thing, the less surface area for conflicts with other platforms the better.

2) While putting this in `zfs_inactive()` would have eliminated the need to replicate this code in the end I felt it was less readable.  Keeping it in the zpl_* layer I think makes the intent more clear.  Basically, we're setting PF_FSTRANS for the entire VFS clear callback.


why not?


Is this intended to be `$local`?


OK.


Good to me and Good catch. So the root cause for typing slow down when specific words are present is not updating ast correctly and immediately?


Yup! TODOtastic.


I always want `$type` mostly for consistency.


That's exact. I don't remember the exact details, but the default http-transport-layer implemented by php for the soap calls was not flexible enough for my needs, so in rev. 5aa7f58ac009a4b76fe42daebbede43fb4d0c495 I swapped it out to use the default transport layer from the lib itself.
It's a bit like the 3-cards trick: I ask to php to create the soap request and then tell it to use my own code to send it ( in ggPhpSOAPClientTransport::__doRequest(), which in turn calls ggPhpSOAPClient::_send() ).
The auth headers should be added to the call by ggWebservicesClient::payload() or ggWebservicesClient::send(), depending on the fact that you use curl or not.

It should be easy enough to troubleshoot the problem by setting the debug option to the client, and inspect the request sent by using $client->requestPayload() immediately after the call, looking for proper auth headers.

Of course if you have a proxy, or reverse proxy, or even just a packet sniffer such as wireshark between the soap client and server, you can capture both requests, the std one and the one after your patch, and compare them


I think this is a browser specific thing as I'm notificing differences between Firefox and Chrome here.

Its all about window.pageYOffset vs documentElement.scrollTop vs body.scrollTop.

Setting formatBarContainer to document.body breaks Firefox. Setting it to document.documentElement as an alternative breaks Chrome.

Since we've introduced the concept of identifying the container via the formatBarContainer option, then perhaps we should be using something like this instead:

scrollTop = Math.max(this.el.parentNode.scrollTop, window.pageYOffset, 0)

In my testing so far this seems to be consistently working across FF, Chrome and Safari. I don't have access to IE variants to check it further.

Since this change came about (I assume) because "This is useful if you're using Sir Trevor inside a scrollable element (i.e. 'overflow-y: scroll`)." as per the docs comment, can you test this as well?

Alternatively we can just revert back to using window.pageYOffset:
http://stackoverflow.com/questions/19635188/why-is-body-scrolltop-deprecated
http://stackoverflow.com/questions/19618545/body-scrolltop-vs-documentelement-scrolltop-vs-window-pagyoffset-vs-window-scrol


Well, the problem's been avoided now that the Qt UI's been completely and
utterly junked.

On Mon, Mar 9, 2015 at 6:08 PM, Peter Tissen <notifications@github.com>
wrote:

> Have you tried reading the license?
>
>
>    1. You may copy and distribute verbatim copies of the Program's source
>    code as you receive it, in any medium, *provided that you
>    conspicuously and appropriately publish on each copy an appropriate
>    copyright notice* and disclaimer of warranty;
>
>  —
> Reply to this email directly or view it on GitHub
> <https://github.com/MoochMcGee/Emotionless/commit/e8d9487fd40255ee2e2e59c87965eb57535bda5a#commitcomment-10110736>
> .
>


Betcha this is faster. Four linear string equality checks are probably cheaper than hashcode and a map lookup.


Another similar way, which will bail out with a try statement on version numbers.

``` python
>>> import pkg_resources
>>> from pkg_resources import DistributionNotFound, VersionConflict
>>> import subprocess
>>> required_deps = subprocess.check_output(["cat", "requirements.txt"]).splitlines()
>>> required_deps
['Babel==1.3', 'Flask==0.10.1', 'Flask-Cache==0.13.1', 'Flask-Limiter==0.7.4', 'Flask-SQLAlchemy==1.0', 'GitHub-Flask==0.3.4', 'WTForms==1.0.5', 'ua-parser==0.3.5', 'nose==1.3.1', 'blinker==1.3']
>>> pkg_resources.require(required_deps)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/karl/.virtualenvs/webcompatcom/lib/python2.7/site-packages/pkg_resources.py", line 669, in require
    needed = self.resolve(parse_requirements(requirements))
  File "/Users/karl/.virtualenvs/webcompatcom/lib/python2.7/site-packages/pkg_resources.py", line 576, in resolve
    raise VersionConflict(dist,req) # XXX put more info here
pkg_resources.VersionConflict: (Flask-Limiter 0.6.6 (/Users/karl/.virtualenvs/webcompatcom/lib/python2.7/site-packages), Requirement.parse('Flask-Limiter==0.7.4'))
```

(/me just realized that he needs to upgrade Flask Limiter).


I'm sure it is, but it's pretty gnarly.


Makes sense - thanks!


I like this. Good use of least powerful primitive rule.


this is only a temporary build to get everything working mvp style. lots of TODOs added that need doing.


These links are all dead now


all tests pass:
success: http://build.servo.org/builders/linux1/builds/978
success: http://build.servo.org/builders/linux2/builds/977
success: http://build.servo.org/builders/android/builds/1012
success: http://build.servo.org/builders/mac1/builds/1193
success: http://build.servo.org/builders/mac2/builds/1194
success: http://build.servo.org/builders/gonk/builds/164


Nit: this comment seems inaccurate


Nit: remove commented code


r=SimonSapin


Should this be "test your mettle"? Great addition btw!


This patch corresponds to one in the ghc tree: https://github.com/ghc/ghc/commit/dfc32cd54e1b6d624ebb9d26772ce787aac3421d


![roc figure](https://cloud.githubusercontent.com/assets/10427178/6567134/c8b7e0b8-c67f-11e4-899f-408c0d871884.png)


79453838-AutomateSomeDocumentation (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Very buzz-word, such unfinished!


fast-forwarding master to auto = f4a36272


It's misleading that this starts `accumulate` when it's not touching the accumulator.


Perhaps `collect` is more appropriate?


Or `filter`, since that's what is happening.


saw approval from SimonSapin
at https://github.com/pcwalton/servo/commit/f9cdd05d58e7f1a8cd6198dad3058d86bf479a08


merging pcwalton/servo/counters-redux = f9cdd05d into auto


pcwalton/servo/counters-redux = f9cdd05d merged ok, testing candidate = 4d2efb56


in the Dev Tools, Network Tab, there is a Fonts filter. we can check that together tomorrow.


I was just reading through these and noticed this is the only correct property value that was an array rather than a single string. I have no idea what this is used for or how, but I'm guessing this is a mistake.


this project is kind of a library. only resources linked to it directly should be inside it. all other resources should be in frackend.


Simply changing from `OnUpdate` to `OnFixedUpdate` makes me happy. :)


Why set it to 16 max? 32 worked great on rpi2. I set (without seeing this) to 80000000 via the overlay in config.text and it was enjoyable. After updating and upgrading it stutters bad.


This looks like it should be gone, since this functionality exists above.


No meu caso usar: ``` test "${cpf}" -eq "0" ``` tem o mesmo efeito
É uma sugestão apenas, mas funciona do jeito que está

E trocar:
```
echo 'CPF inválido (não pode conter apenas zeros)'
```

```
zztool erro 'CPF inválido (não pode conter apenas zeros)'
```
Apenas para compatibilizar com essa mudança recente.


Talvez mudar esse trecho:
```
: ${cpf:=0}
```

por algo como:
```
if test ${#cpf} -eq 0
then
	zztool erro 'CPF inválido (não pode conter apenas zeros)'
	return 1
fi
```
 Economizaria uma passo que no contexto que você apresentou não faz sentido ter.


Just so you know, this is already being done by a community-maintained organization.

http://laravelcollective.com/docs/5.0/html

http://github.com/LaravelCollective/html


Well, they probably should learn to use branches.
http://stackoverflow.com/questions/9790448/how-to-update-a-pull-request

On 09/03/15 22:37, josmil1 wrote:
>
> Not sure you understand by "their branch is on the right", isn't
> web-tutorial the branch we're all working on? Do they need to know this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/josmil1/web-tutorial/commit/7b3acdf94cc9016d35703af732de456493eb8f88#commitcomment-10110318>.
>


This seem like a nice addon. The only thing I'm wondering is that we cannot exclude roundRectangle in favor of performance with this. I call for regular rectangle vs  10 four rounded.


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (637 of 637 tests passed)

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


这里试试用 [trait](http://php.net/manual/en/language.oop5.traits.php) 组合到 controller 上面


Also "sneak peak".

![sneak_peak](https://cloud.githubusercontent.com/assets/39553/6567252/d0487d18-c69a-11e4-859d-7a5969be2110.gif)


Ok.

On 2015-03-09 12:25, LanceBooth wrote:
>
> We have not been able to interface with the camera because it needs to
> be plugged into the d-link (which is bagged with the robot), hence the
> code being untested.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Team2550/roboRIO-Code/commit/767ea3b781ed6025d4ee1286942818d7a297ff83#commitcomment-10106864>.
>


The social API doesn't use `since`, but it does let you send a `length` parameter to say how many updates you want. Try changing the data you pass in for your $.ajax options.


develop (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2165 of 2183 tests passed)

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


This may be because we were inadvertently discarding the value `0`, since it was falsy.


develop (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1146 of 1146 tests passed)

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


Wasn't aware PMD used that idiom to prevent warnings, awesome! (thought we needed an annotation)


If you don't need `object data` or the return value, you can simply use the non-generic version of `DepthFirstAstVisitor`.


@amitmurthy This series of commits broke MPIManager. After executing `addprocs(manager)` the REPL just hangs and I get something like
```julia
julia> addprocs(manager)
^CERROR: InterruptException:
 in read_worker_host_port at multi.jl:1018
 in connect at managers.jl:217
 in connect_n_create_worker at multi.jl:1202
 in start_cluster_workers at multi.jl:1112
 in addprocs at multi.jl:1067
```


if platform_name == 'Windows':
        cmd[3] = "--clean-css"
...and it's work on windows


Your code looks great and I'm just being nitpicky about this, but maybe this could be helpful:

Defining your velocity as a string can be confusing, since the purpose of keeping a velocity would normally be to use it for calculations. Saving it as an integer would make it easier to work with. Then, appending the "km/h" would  became a presentation detail and not part of the data itself. If you store it as an integer, it might also make sense to store the units separately, something like:

`@airspeed_velocity = 15`
`@airspeed_velocity_units = 'km/h'`

Again, I'm being nitpicky here; this was the only ting that drew my eye, and it's probably not even relevant since this is a practice assignment with no practical application. So anyway: Good work :)


develop (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2166 of 2183 tests passed)

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


None of the validation works on the initial sign-up, but it does work editing later from the patient info page. Must be a bug in the flow? I just successfully saved a new account with a birthday of March 32nd (which moment translated into April 1st).


Show!
:+1:


Hmm, I see. Maybe there is a deeper bug in schema validation then? I wonder if it's worth digging.


Looks great to me. Can I help in some way?


#resolve


FOSSCRUM-1 #resolve


83782874_LBNL_ThermostatOffsetFault (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Yep! My mistake. Thanks for catching this.


Huh?


You're right. It should be a string.


Is this correct 'unauthorized' ?
On my side I get 'Unauthorized' (first letter capital)


some tests failed:
failure: http://build.servo.org/builders/linux1/builds/979


Out of curiosity, does this actually work for you? I just ran the setup for this and I had to go in and manually insert the passwords and database name.. (sorry, there appears to be no other means of easily communicating directly about this docker build :/)


thanks for catching that, @shaneriley !


Wouldn't it be simpler to take busybox binary from aml device, rename it to fbset (as it's part of aml's busybox) and use set_display_mode script for example? :)


Please edit frontend-edit.css in templates/your-template/css directory, add this line
.icon-edit {
  display: block;
  font: normal normal normal 14px/22px FontAwesome;
  font-size: inherit;
  text-rendering: auto;
  -webkit-font-smoothing: antialiased;
  -moz-osx-font-smoothing: grayscale;
  width: 22px;
  height: 22px;
  text-align: center;
  background: #fff;
}
.icon-edit:before {
  content: "\f044";
}


The triangles certainly look cleaner.  I still see some value in the numbers as, for me, damage is not just a binary thing (fully upgraded or not), but maybe not enough value to justify the clutter they add.  I think there is potential for a lot of trade-offs like this (functionality vs aesthetics) so maybe adding a settings/preferences menu can be justified, with toggles for things like "show attack/light values".


Where are the labels?


Trivial, but shouldn't this have a space after the comma?


we will want to setup a npm task to run the lint

`npm run lint` or something .. then we will just configure travis to only run that task on these branches :)


I'm sorry for posting this here, I joined in order to report a bug but couldn't find a place to put it but here. Whenever I go to pput a comment in someone's flairhq reference sheet, I can't without pushing tab to get to the text box. Clicking on it is as if there's nothing there.


the edit icon is not visible due to it using <i class="icon-edit"></i>
that code is only available for old version of font awesome which is used bootstrap version 2, so if you use T3 BS3 version you need to go to template manager > your-template on style column > General tab > Legacy Compatible = on.
Unfortunately it will load more file, so, my suggest is edit frontend-edit.css as above coment.