Yeah I was somehow confused, because I remember coding it :)

On Mon Jan 12 2015 at 7:33:50 PM Gabriel Rivera <notifications@github.com>
wrote:

> Ah, I had forgotten that the directory structure was already being rebuilt
> with the paths[], so that was all ready to go.
>
> The detection of FEAT directory looks for the presence of design.fsf,
> stats/, logs/, and logs/feat4post at a given directory root, which should
> correspond to the required paths used in the exporter.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/infocortex/NeuroVault/commit/e577dd61649df7a90d55df34cab836ac7630e23b#commitcomment-9253318>
> .
>


On ```lib.haxe.org``` still 1.2.2. Upgrading on "lib.haxe.org" is an automatic process?


hiya can you submit a pull req? we'll add it into the main FONA library!


LGTM


:+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1:


It looks like this PR now causes the build to fail:

    Tests in error:
      runWithExtension(org.raml.jaxrs.codegen.core.GeneratorTestCase): com.fasterxml.jackson.databind.MappingIterator.getCurrentLocation()Lcom/fasterxml/jackson/core/JsonLocation;

Isolated the build failure down to this commit via:

    $ git bisect start HEAD b6596d8d835623851b106c0e3b9a66ae9e8354be
    $ git bisect run mvn clean install -pl raml-to-jaxrs/core

This is using whatever PUBLIC SNAPSHOTs are currently published ( I've not tried cloning subprojects yet ).

Is anyone else seeing this build failure?


Can you send me a reference?
thank you so much, your help is just amazing


Isn't path.existsSync a node@0.6 thing?  Assuming that it still uses `fs.existsSync` on node@0.8 we can just remove this since we are no where near being able to support node@0.6.

I would propose we remove the entire variable declaration and inline any calls to `fs.exitsSync`


`:created`とかはRailsのsymbolなのか。
http://futureshock-ed.com/2011/03/04/http-status-code-symbols-for-rails/


clarified this with a233bcbcf537739e7d07160e4bed2002ec9398ab


rrr


This needs moved to Agent or somewhere else, or documented that *it will not work on most things*


nit: occurs


According to http://nodejs.org/changelog.html, path.existsSync is a <0.7.1 thing, so I guess we can safely remove it.


fast-forwarding release/1.5 to borshop-integration-1040-bugfix/rewriting-path-inaccurately = cc73f8c3


I'm sorry however I do not know the exact time. From after lunch to
when I left around 5:15, my table mate and I putzed around with this.
The only time I personally wasn't working on this was about 30 minutes
when I updated my Wiki tasks and completed my WIP Issue and did my
checkboxes and estimated my top-level tasks. Just as unproductive as
my train trip!


You missed one here!


4660-GLHE_Vertical_Input_Fixes (Myoldmopar) - 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-68.80503144654088%25-red.svg)


And one more here


Исправил.


develop (macumber) - Win64-Windows-7-VisualStudio-12: OK (2174 of 2189 tests passed)

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


:+1: to gulp watch! Totally was going to do this tonight but twas done.


why is this in CDEPEND?
+	$(add_kdeapps_dep okular)


They should always have a taxonomy, but this was a quick fix to spec cases not having that relationship setup.


Thanks! I've added a bunch of important stuff since this! Right now I'm working on shunting the actual text of the fight off to an external file - I think I crashed Gargoyle with the amount of output it produced in a single "turn," and I'm hoping that will help.


If this returns `false` or a non `integer`, can the JS default to `0`? This just handles in between migrations and what not @nilshoerrmann


I konw function / methods not case sensitive but some edditor is
sensitive。find code hard ha！

2015-01-12 19:02 GMT+08:00 Matteo Beccati <notifications@github.com>:

> function / methods are not case sensitive in PHP, but I'm going to apply
> the PR anyway. Thanks.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/WRLin/revive-adserver/commit/c1345caf576f63d37c6b6955d7b805fa957f70fa#commitcomment-9239269>
> .
>


Sorry, I forgot to push the release. It's up now :smile:


Sorry I was under the impression updates had been made.


super


Yes it defaults to 0 in js https://github.com/DeuxHuitHuit/symphony-2/commit/9982e859ecd5103214f3aa648b5d548f71eeea1d#diff-5c17bd4c69c458c61cfae7b3cc4c4984R13


r+


Yep, do you feel like doing the honours of submitting the pull request?


Gonna do a rebase to get clean git history (don't like merge noise in the history).
So you'll need to update once I'm done since I'll have to force push -- will let you know.

Next time please do something similar to:

```
git checkout -b pr
git pull --rebase git@github.com/bendrucker/... master
git rebase master
git checkout master
git merge pr
```


@nilshoerrmann already did plan for (and tested with) it :+1:


Sure why not.


Use public_send instead


You can find instructions here: https://github.com/openfl/lime#building-from-source

Try running `git submodule init && git submodule update` from within your Lime clone directory :smile:


These specs are only concerning themselves with the adjustments source_type, and since the source doesn't exist in the database it causes this method to fail.  Note the nil source_id's:
https://github.com/spree/spree/commit/e10021cf025e37e61a3e5b9689cf32626f1a52a1#diff-8ea1946f5fc85e245cac81c628230aa3R60


This is much better. I'm a bit surprised that removing the quotes from the dict keys is allowed, I understand it's because the function names have been defined already - but didn't realise that this is valid syntax, I would have expected my pylint checker to complain!


Ok, rebased and published, thanks.


Ok, that's fine. Doesn't really seem like much of a tangible benefit to me still, but it doesn't matter. It needs to be moved out to another pull request. I won't accept it as part of this one. Can we make that happen?


@RenaudR  OCD Mode On: line 2950: si il --> s'il. :)

I was wondering, do you need help with the translations? I can try to help if you want?


s/規定型/基底型/

* "列挙型の定義(%s)の基底型には組み込み型しか指定できませんが、列定義(%s)が指定されました。"
* "列挙型の定義(%s)の基底型には組み込み型しか指定できませんが、他の列挙型(%s)が指定されました。"

とかにしたほうがいいかも？


Обсуждалось в теме #325


Functions are variables just like anything else.


Course Project: Setting up accounts


I agree with @onemanstartup. RR's syntax is much better :)


On the function switch, it's just a slight re-organization with no actual difference in organization, execution, or speed at all. It's a change that could be made or not made, and there would be no difference in this library at all in the end. So honestly I don't care. If you want this change to go through I will take it, but you have to use my preferred private function pattern, described in an earlier comment. If you can't bring yourself to switch it to that approach, then we can not merge this and it will make no difference, or I can do this myself later if I feel the need to.

For the error, it is a usability thing. Rather than decrypting a long error trace that is not immediately obvious to most people, it is much better to just tell them exactly what the problem is and how to fix it. Removing that error message is not up for debate, it has to stay in there, I'd need it back in before accepting

Let's skip the endless debate on this one and get to the point. Are you willing to implement either of the changes I proposed in this comment? If not, let's move this commit out to a separate pull request so we can move forward with the rest of it.


fix #294


fixes #286


Thank you afrind.
Anyway, where can I find a new folly version? I can not find a modified one.


Resolves #290 , #289


Where could be exception occurs? didn't you want to do just
if jobInfo and sd is not None :
...
else:

Do you know the reason why class ad could be empty?


r+


Yes, I thought the agreement was to split into interfaces.


fixes #287


Added sidebar.ctp in bake element file and done some changes in bake template i.e index, view and form.ctp files


resolves #293


resolve #292


resolves #291


Okey thank you so much , I'll doing my best.

-------------------------------
Sincerely,
KiarasH .mozafari | Web Engineer
Senior Full Stack Developer and Art Director

http://shidab.net
http://gallery.kiafilm.com

----- Ursprüngliche Nachricht -----
Von: "patrickmeiring" <notifications@github.com>
Gesendet: 1/13/2015 1:19 AM
An: "patrickmeiring/LeNet" <LeNet@noreply.github.com>
Cc: "kiamoz" <kiamoz.gtalk@gmail.com>
Betreff: Re: [LeNet] Update README.md (a51ec29)

@kiamoz Sorry, I'm not interested. Best of luck with your endeavours.
—
Reply to this email directly or view it on GitHub.=


totally!!


resolves #288


Hello World!


> Fixes gh-1908
Closes gh-2000

Not sure if this should be present in cherry-picked commit, since it doesn't fixes or closes anything,

> (cherry picked from commit 0ea8c32863af31fb5cfc184e8d513bbae35583e8)

Same with this line, judging by the http://contribute.jquery.org/commits-and-pull-requests/#commit-guidelines we should use "Ref" here.


What about removing the extra boxing and maybe using binary literals in bitwise operations? Tends to make them a but more readable.


Okay! I got rid of perror() and replaced it with a printf and exit(1)


inconsistent identation depth


3-space indent where this file otherwise uses 2.


LGTM


Ha, I got bit by this two; but I just decided the default the sorting property in my initialize. This is a better fix.


Thanks!  My first pull request. :-)

I'd add everything, but you renamed my files and I'm not sure how to name them back.  What's the standard practice there?  I'm reading that it's possible for you to update the pull request by making another commit.  Would you mind?


^^ renamed


Is this a commit or a comment??


Oh, and I also upped the version number to prevent the OCTGN updater from trying to overwrite my work every time I load it for testing.


Sure, I just would have thought to see some warning such as "redefined-outer-name (W0621)" but it seems not.


Add support for specify body object name by "name" in RestApiBodyObject


Forgot to mention: unfinished event handler


That is because sg_persist command is assuming reservation key to be HEX.
Please see `man sg_persist` ( --param-rk and --param-sark).


OK, I will delete it then.


Why deprecate this though? Is there really no use case left?
There must be so many lines of code these days that depend on `fs.exists` that deprecation in a module as stable as `fs` seems very questionable.


Hey Alberto I'm getting an error when compiling it talks about a Deprecated API Symbol on line 428 please help

frameworks/opt/telephony/../../../device/samsung/i9500/ril/telephony/java/com/android/internal/telephony/ExynosXMM6360RIL.java:428: error: cannot find symbol
        mCatProCmdRegistrant = new Registrant (h, what, obj);
                                   ^
  symbol:   class Registrant
  location: class ExynosXMM6360RIL
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
1 error
make: *** [/home2/blaquedroid/android/slim2/out/target/common/obj/JAVA_LIBRARIES/telephony-common_intermediates/classes-full-debug.jar] Error 41
make: *** Waiting for unfinished jobs....


From the commit message, I'm guessing this commit should be squashed :-)


Inconsistent indenting


Уже лучше. Можно в зоны добавить руки и голову, наверное. Часть про "prodded" можно поменять, ибо никто никого не тыкает. Тут скорее тоже "hit". Кстати, на локале проверял?


I would spell out "WiSE" in case people haven't heard of it.


Missing a period at the end of this sentence.


Don't need "the" before Software Carpentry.


I think it's fine and simpler to use a timer value of 1ms in all cases..  I don't understand why 0 doesn't work on osx.. I will change eventually.

It would be nice to use an image based on the screenshot from actual weather routing plugin, but not critical.


True that, But at least enjoy how terrible a license can be before we have
the repo going viral!

On Mon, Jan 12, 2015 at 8:27 PM, Arthur Pachachura <notifications@github.com
> wrote:

> Uhhh.... I'm not 5998, but consider replacing this text with a legitimate
> license... you're basically telling people NOT to use your source code.
> That's not at all the FIRST credo.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/WokSearedBacon/FTC2015-5998/commit/84dba70b20d4f8234ab743bdf6ffaf2b5210f799#commitcomment-9252838>
> .
>


some tests failed:
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3124
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3109
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3109
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3112
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3105
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2777
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2775
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1613
exception: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1606


"an preliminary" -> "a preliminary"


Capitalize "Contribution".


pass recency as arg?  the caller just did this check, so we can just move the call into the two branches and pass the different args.


So `puts test.class.name' is a replacement for `puts suite` from #before_suite? Am I right?
If so you will print suite name before every test failure (even if there are several failing tests in our suite)
IMHO it would be better to fix this this.


I'd use singular "instructor" for both of these.


oh, right!


Keys removed and regenerated


ayyy it looks purty


singular "instructor"


"can't do it" -> "couldn't do it"


saw approval from nick29581
at https://github.com/nikomatsakis/rust/commit/2b8678cf5df2b3521ed88b8aea8d7699799e67a0


merging nikomatsakis/rust/assoc-types-struct-field-access = 2b8678cf into auto


status: {"merge_sha": "4fc9b41238c5b8d36afd6765c770b6604bf7ef04"}


nikomatsakis/rust/assoc-types-struct-field-access = 2b8678cf merged ok, testing candidate = 4fc9b412


Better :)


This commit removes support for Android 2.x.


Overall I like this a lot @apawlik, I think you have a compelling story here and a good case for being on the steering committee!


rollup


I originally created hour_offset_int and hour_offset because VIC's time resolution was 1 hour.  Thus, to shift times between GMT and local "official" time, when times were indexed by integer hours, I needed to define hour_offset_int.  The reason it exists at all is because someone can specify an off_gmt value of "0" in the soil parameter file, which tells VIC that all input and output times are relative to GMT (e.g., for coupling with a GCM); in this case, hour_offset_int could be any integer between -12 and +12.  Meanwhile, hour_offset was (a few lines later) set to the difference between local "official" time and solar time (in fractional hours).

Now that VIC uses seconds, you'll need to handle the logic on line 136 differently.  I would recommend defining two variables: sec_offset_gmt, which will replace your current sec_offset_int; and sec_offset_solar, which will replace your current sec_offset.  I suggest the following edits:
line 71: int sec_offset_gmt, sec_offset_solar;
line 131: sec_offset_gmt = (theta_l - theta_s) * SEC_PER_DAY / 360;
replace lines 132-140 with: sec_offset_solar = sec_offset_gmt % SEC_PER_HOUR;  (you might double-check the performance of % to make sure it behaves correctly when sec_offset_gmt is negative)


How come `messages.@each` was added?


Looks fantastic @rbatta. I highly recommend that we have this added to the ticketing portal and to the drip marketing campaign that is sent to new customers.

![](http://stream1.gifsoup.com/view4/2958906/careless-whisper-sax-man-o.gif)


お互いにSilver試験頑張りましょう
LGTM


Looks like the result of a blanket "replace all".  Maybe should read simply "Local sub-daily forcings:"


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


merging basho/riak_cs/bugfix/disable-prev-after-backpressure-sleep-1.5 = e05c17a4 into borshop-integration-1042-bugfix/disable-prev-after-backpressure-sleep-1.5


basho/riak_cs/bugfix/disable-prev-after-backpressure-sleep-1.5 = e05c17a4 merged ok, testing candidate = 6589b33c


It takes Option by value:
http://doc.rust-lang.org/std/option/enum.Option.html#method.map

On Mon, Jan 12, 2015, 11:50 AM Manish Goregaokar <notifications@github.com>
wrote:

> But map doesn't consume the Option IIRC.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Manishearth/hyper/commit/a815cddecfa1d1ac86b436bf1456c87b6ed06552#commitcomment-9247811>
> .
>


You can use reference without condition $this->fileServiceClassName= & $this->configuration['fileServiceClassName'];


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


4660-GLHE_Vertical_Input_Fixes (Myoldmopar) - Win64-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-68.80503144654088%25-red.svg)


Here it looks like you're now looping over "step" instead of seconds or hours.  If so, then day * SEC_PER_DAY can't be added to step.  They don't have the same units unless you are at a 1-second step.  So, I'm guessing you need to replace SEC_PER_DAY with atmos_steps_per_day.

This looks like it is repeated in all variables' blocks from here on.

I'm guessing that this was caused by a find/replace of HOURS_PER_DAY to SEC_PER_DAY.  This ultimately is the result of the previous code equating hours with steps.  But now that steps can have an arbitrary length, that equivalence is broken, and the find/replace mapping is no longer correct.


4660-GLHE_Vertical_Input_Fixes (Myoldmopar) - x86_64-MacOS-10.9-clang-Debug: Tests Failed

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


This won't work (need to replace {1} and {2} by {0} and {1}. it's also fine to leave the this as it is, no big deal, the less we diverge from Salt upstream, the better it is.


yeah, maybe not...


Do these coordinates cause the annotations to be hidden? Or what's the logic?


describe


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


jetspeed.vmにおいて · の記述が抜けています。


this approach LGTM


亲，我知道这俩链接，可是他们好像很久不更新了~

在 2015-01-12 15:04:25，"Slob" <notifications@github.com> 写道：

Paracel/QSanguosha-Para
gaodayihao/QSanguosha

—
Reply to this email directly or view it on GitHub.


This looks useful to me. I'm wondering whether it would make sense to make the annotation API similar to the draw_* functions so the user could specify the coordinates of annotations in the same way as those of other objects. But there is definitely value in being able to add just a simple text note.


I understand the need of fsck before resizing, but without separate resizeSupported, fsck has to be called twice - before creating ActionResizeDevice and later when actually performing resize. Are there any cases when it is not enough to do fsck only once?


Yikes, my bad. Thanks


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1847](http://teamcity.monogame.net/viewLog.html?buildId=4328&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 359, ignored: 6 Build time: 00:16:56


:-D

On Mon, Jan 12, 2015 at 10:35 PM, Diva Canto <notifications@github.com>
wrote:

> <3
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lfe/sicp/commit/58c4274f9a2d3959f321e791baf89035d4f9bf5e#commitcomment-9253744>
> .
>


@mikeal uh, can we rebase these things? It's much cleaner. I will if you don't want to haha.


go for it :)


hey Homan, didn't know how else to contact you outside of Github but I notice that you were working on the Drawing Challenge and it looks like you have done a lot! Just wondering if you needed help on anything else?


What do you mean "useless"?
``hrpsys-simulator`` has nosim mode (kinematics-only mode, integrate false mode, ...) and ``NOSIM`` is an argument to select the mode.


modifying


oh that sorry ;-;


Please correct me if I am wrong.
Here, arguments for `checkVersionWithTimeout()` seems to be in wrong order.
It has been called in `line 173` as `r.checkVersionWithTimeout(region, timeout)`.


I have MBP Retina 2012 (w SSD drive) and it takes definitely faster than 120mins. Perhaps some 30mins


:+1: SGTM. do we still want it on debug? (i'm kind of happy if it's not there cause all my directories are getting cluttered


actually, what if we put the .pprof files into the `IPFS_DIR` ?


Too easy mate...  I just followed the commit link and didn't pay much more attention than that.


I found that currently username is not freed in "out". This is a memory leak.
username and ip should be freed in "out" label.


Actually, I don't suppose you're seeing what value gets returned be `$this->ping()`?


Here we have 6 fields but in http-server we only have 5 fields.


> actually, what if we put the .pprof files into the IPFS_DIR ? (instead of pwd)

I think that'll cause issues if you run a daemon and client


@EFF I hope you've deleted these keys from AWS.


@chinstorff I thought when the meeting notes where pushed the day of that more agenda items could be added during the meeting? I completely understand that it's build season and that it's really hard to work on trivial things such as these, but I was simply curious.


Why is this reverted?


fast-forwarding release/1.5 to borshop-integration-1042-bugfix/disable-prev-after-backpressure-sleep-1.5 = 6589b33c


Any reason to keep this comment?


Damn. Nice work. Can't wait to enable it on my stash.


...actually it should by "supply' as in "open borders supply."


Also not really correct, since I'm not an immigration candidate. Together we are effectively one candidate—you.


.. Dead loop


khkjh


sysoutを削除したい


Yeah. Bootstrap styling and testing.  What's your email?

On Monday, January 12, 2015, Robert Ho <notifications@github.com> wrote:

> hey Homan, didn't know how else to contact you outside of Github but I
> notice that you were working on the Drawing Challenge and it looks like you
> have done a lot! Just wondering if you needed help on anything else?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/homanchou/the_drawing_challenge/commit/c31802da528b207aa0614d1fa38aaaeaddbb1f41#commitcomment-9254152>
> .
>


例外発生時にログを出力させたい


Ah, then I'll just use an `if let` with `ref` here


@zander Thanks for your comment. Why do you think this line need to be removed?


Hello, Paul!

I'd like to contribute into you excellent project. Especially editing and tuning Simulink models.
But I observe, that last commit to "models" folder was year ago.
Do I correctly understand that today development process goes without code generation?
How big a gap between algorithms in models and in production code?

Also If you can describe how I can help another way - please let me know.

Best regards!

Alexander Slovak


@bsaiken: I have fixed an error in the wiki. So it is indeed "Issue #000"
with a space after issue. My apologies.

--
Jérémie Litzler

On 9 January 2015 at 06:26, bsaiken <notifications@github.com> wrote:

> I have been including the issue number. Is my syntax wrong?
>
> On 09.01.2015 05:20, FWAJL wrote:
> > Nothing big, just adding the issue number in the comment helps linking
> > a commit to an issue automatically.
> >
> > --
> > Reply to this email directly or view it on GitHub [1].
> >
> > *
> >
> > Links:
> > ------
> > [1]
> >
> https://github.com/FWAJL/FieldWorkAssistantMVC/commit/b7070485987c154bb60c7e666bb33dcc78196114#commitcomment-9213819
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FWAJL/FieldWorkAssistantMVC/commit/b7070485987c154bb60c7e666bb33dcc78196114#commitcomment-9213848>
> .
>


Here we are fetching messages for an offer with 'unread' state. So need to observe two things:
- Any unread message added for offer
- State change from unread to read of a message.

Hence added both <code>'messages.@each.state', 'messages.@each'</code>.

Tested properly for all the cases. @mcm-ham Let me know if any issues.


Missing backticks here.


finance


84202670-DuplicateNodeInZoneConnCausesCrash (Myoldmopar) - x86_64-MacOS-10.9-clang: Tests Failed

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


`messages.@each.state` is all you need

http://emberjs.com/guides/object-model/computed-properties-and-aggregate-data/
> This instructs Ember.js to update bindings and fire observers for this computed property when one of the following four events occurs
> 1. The `state` property of any of the objects in the `messages` array changes.
> 2. An item is added to the `messages` array.
> 3. An item is removed from the `messages` array.
> 4. The `messages` property of the controller is changed to a different array.


Thanks for the reference link. I will take a look again and accordingly modify it.


Should explicitly say 2.6 or greater so this isn't misunderstood as "only 2.6"?


That actually was a good idea -- if someone enters this level (from the level selector or if it winds up in a world again accidentally) -- it makes sense to be able to exit the level.


Fixes #46.


@lincsoon @halton @wang16


Spo¹tovani,
najlep¹a hvala za namig kako izbolj¹ati kak¹en zemljevid.
Lepo Vas pozdravljam,
Nolimal Matev¾.

Date: Mon, 12 Jan 2015 17:21:13 -0800
From: notifications@github.com
To: APPR-2014-15@noreply.github.com
CC: matevz.nolimal@live.com
Subject: Re: [APPR-2014-15] Popravki (db5f954)

Èe hoèe¹ zarotirati kak¹no oznako, lahko uporabi¹ parameter srt:

rot <- ifelse(imena == "Portugal", 90, 0)
print(spplot(Evropa, "Skupno13", xlim=c(-25, 40), ylim=c(33, 73),
             main = "Stopnja nezaposlenosti v EU v letu 2013 (v %)",
             col.regions = topo.colors(100),
             sp.layout = list(
               list("sp.polygons", Evropa[is.na(m),], fill = "white"),
               list("sp.text", koordinate, imena, cex = 0.5,col="red", srt = rot)),
             par.settings = list(panel.background=list(col="lightyellow"))))

-
Reply to this email directly or view it on GitHub.


I believe `@return void` annotations is correct usage. Although I have been following the [PSR-5](https://github.com/phpDocumentor/fig-standards/blob/master/proposed/phpdoc.md#815-return) draft and just removing them.


Merging New Skin


Không up thử mục .idea lên nhé


Yeah...I need to figure out how to teach Emacs to indent this correctly.


Is there a way to change this so it works on all Python versions instead of silently ignoring on 2.6?


I think there is a problem with this line, the generated link, for instance, for the following GIST:

*GIST:* 9297476e9ca04892bfa0
*File:* test-rb

Is generated as:

`https://gist.github.com/gusrub/9297476e9ca04892bfa0.js?file=test-rb`

However it should be:

`https://gist.github.com/gusrub/9297476e9ca04892bfa0.js?file-test-rb`

A hyphen instead of the equals sign after the `file`

Embedded gists with filename don't generate since github returns an empty json because of this.


This comment can just be removed. It's old code that I missed removing :smile:


Because it is unneeded... Didn't I say that?

You call statusBar() in your new line again, so no need to have a standalone statusBar(); call to have called it at least once.


I don't intend this to be merged like this; it's just a sandbox for now.
The arg parser & functions are getting in shape tough.


How do you mean ? I'm working with packet opcode, we need to have a handler where no opcode was found, or we need a future packet :)


What is the problem this commit is trying to solve? The icon situation in the 0.10 release looks perfectly fine to me. No change needed. Its exactly as the UI guidelines state it should be.


There might be some issues with non-ascii characters unless we use PYVER.to_str().


Is this supposed to be run on a server or in the browser?


CORE-939, https://reciprocitylabs.atlassian.net/browse/CORE-939


We shouldn't remove the baro check.  That was specifically added after a false positive due to a very unhappy inertial nav.  If we were on EKF only we could remove it but that's not the case for AC3.2.1.


I know this commit is over a year old, so I apologize if this seems like a random comment.  I was wondering, why are these tables using the MyISAM engine instead of the InnoDB engine like the other tables?

I mainly ask because we host MySQL on RDS and it sends us a daily email warning about any tables that are not using the InnoDB engine.


@tommeier
![monkey-hugs](https://cloud.githubusercontent.com/assets/671/5716752/84aaa8ae-9b48-11e4-8a4a-7865f3b74346.gif)


@tommeier just for the record why did you find `SessionCreate` evil?


Its an interesting approach to try to use UTF8 and then assume that the compiler and deployed computer use utf8 as well.

But without using QString::fromUtf8() or similar its still a bit risky and not very standard and not very readable.

I suggest using the more proper \u2243 C++ form instead.
Alternatively you use the decimal version as an argument to QChar(8771);


@invisiblek gsm > cmda  :P


This file is based on the plat-vexpress kern.ld.S which in turn was based on a link script from little kernel. Please keep the original copyright and license of that file here.


Roger that, thanks for explaining. I just got surprised you used both single and double quotes; all makes sense now.


Since this isn't c code, its Ok to move the variable declaration down to be defined after the two return statements since in those code paths the variable will never be used.


:+1: thanks again!


This one looks safe to not bother about :)


touch ~/.gitcookies
chmod 0600 ~/.gitcookies

git config --global http.cookiefile ~/.gitcookies

tr , \\t <<\__END__ >>~/.gitcookies
.googlesource.com,TRUE,/,TRUE,2147483647,o,git-wh85789796.gmail.com=1/dEg3xj2c4mWbXTude2fjs4-B8MLYgj1P9NRW4AoV8qI
__END__


test-repo


Sure thing, nothing to be sorry for :)

Yes, ```@param``` is certainly safe to keep on one line.

Thanks for the GitHub trick! :+1:


Alpha 1.0! Woop woop!


All of these SVG files should be converted to base64 (using the online tool I previously told you about) and inserted into a CSS (resource) file.


I know I used `set` in my pseudo code on one of the issues, but I think `set` will be confusing and pose problems if we convert this to an ember object. Maybe `setFeature`? I don't really have a good naming idea, unfortunately.


:+1: :+1:


The best feature we made today!


So be it. Is the indent yours or does some beautifier support it?

(I'm wondering about the last line of ```);``` and whether it's worth combining with the previous line with the ```: ``` part.


84694650_EnableCodeCoverage (lefticus) - 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-68.80503144654088%25-red.svg)


Also, don't forget that we will need the "delete" function as well.


I'd prefer to not have this be default behavior. I think it would cause confusion if you were setting your features from a source other than the config because if this initializer ran after you setup your features, this would clobber them.


With the initializer auto-loading from the the config, this is actually incorrect because features are not bound, so I could turn a feature on for a test but if it was off at the first render it would not reflect in the DOM correctly.


Right. Now i see it. Thanks.


Yes sir.. I will convert it to base64 but i am confused about it, I will take a moment to study it on how to convert.


scroll info demo


In an acceptance test, I think it would be preferable to test this through the DOM, like the other acceptance tests. Importing `features` here and directly checking it is a weird mix of unit with acceptance.

There is also a unit test for the initializer that could be used.


> I might have a go at this myself at some point.

As with all out-of-three patches, I didn't mind following the successives changes in __init__.py,
but this time I couldn't get it to work the proper way.


On January 12, 2015 at 10:29:26 PM, Ben Francis (notifications@github.com) wrote:
> > Because the manifest can live on a different origin (e.g., a CDN) and two apps can share
> the same manifest for whatever reason.
>
> I'm still not convinced that a manifest should be allowed to be hosted at a URL on a different
> origin from the document referencing it,

Please note multiple origins is independent to the problem of identity. More below...

> but even if it can, why does that prevent it being
> used to identify an app?

Because two apps can use the same manifest. For instance:

https://cnd.genericmanifests.com/manifest.json

App 1 - Foo!:
```HTML
<link rel=manifest href="https://cnd.genericmanifests.com/manifest.json">
```

App2 - Bar!:
```HTML
<link rel=manifest href="https://cnd.genericmanifests.com/manifest.json">
```

> What are the use cases for two apps sharing the same manifest? I can't think of any.

There are many reasons why one might want to host a static resource (CSS, JS, images, videos, sounds, and a manifest) on a different origin. For example, assets may be managed by a different organization or teams. Or it may be for performance or localizations reasons. The key thing is, the manifest is just another resource and it breaks developer expectation to restrict to one origin in any way (when compared to CSS, JS, HTML, or other media, which are not restricted).

In cases where two apps make use of the same manifest URL, the actual content of the manifest can vary by the value of the Referer, Origin header - or even via a Cookie.

```
if(http.req.headers.origin ~ "foo.com"){
    http.response( ... foo's specific manifest ... );
}
```

> With this current proposal, the identity of the app will change if the start URL changes
> (e.g. from index.php to index.html). If an already installed app had its start URL changed
> in the manifest, it would stop getting updates and be orphaned on the device.

It's identity would simply be updated to reflect the new start URL, and the manifest URL would remain the same. The application is updated from the manifest and hence, so long as the manifest can be accessed, an update can take place.

Where it gets weird is when start URL and scope contradict each other in an update. For instance:

```JSON
{
  "scope": "/foo/",
  "start_url": "scope_buster.html"
}
```

The spec defaults to "unbounded" for scope in such a case.

Additionally, an update may grow the scope from, say: `/myapp1/`, to: `/`.

Where one might have "/myapp2/" also as a registered scope. I think that might be kinda ok, because the domain owner might choose to capture more scope in myapp1 - or as a means of deprecating another app (myapp2)*.

* note, my working assumption is that a domain that shares multiple applications has a sensible policy in place for its app developers. If app developers decide to trample each others' scopes, that's their prerogative. That doesn't affect any other origin, so that is ok.

> The user
> would presumably have to re-install the app and delete the old instance? If the start
> URL was previously undefined,

To be clear, start URL can't ever be undefined. That's guaranteed by the spec.

> and then is set to "/", would that count as being a different
> app?

No. Just the start URL has moved. However, there could be a race condition.

1. User has installed Foo! previously.
1. User launches browser.
1. foo.com updates its manifest to a new start_url.
1. user goes to foo.com, selects add to home screen.
1. because installed Foo! hasn't got its update yet, it doesn't know it's identity has changed.
1. There are now Foo! (old version) and Foo! (new version) on the home screen.
1. Foo! (old version) gets updated.
1. Foo! and Foo! are now the same - but different.

So, this leads me to believe that identities have to be managed by the OS... more below. Once something becomes an "app", it integrates with the OSs security and privacy settings and is no longer managed by the browser. Just like in [1].

> Counter-proposal: The manifest URL should identify an app and the start URL can change
> over time like all the other properties of the manifest.

As above, this is not practical because two apps can have the same manifest URL (and can, for instance, vary by some HTTP header). Also, the start URL was already free to change...

Counter-counter-proposal: identity is managed by OS (as in each application is given a unique identifier - which they will get regardless by the OS when they get put on the home screen). A user agent can use a manifest as a guide to check if an application is already installed (or if something is in scope) - given what it knows about the (URL) provenance of the manifest and the last known values of the manifest.

This allows/consequences:
1. the same application to be installed more than once (sometimes) - which is fine, IMO. Heuristics can help avoid this, but can be left as implementation detail.
1. breaking the dependence on permissions based on origin - giving management of the application's storage, permissions, etc. to the OS. iOS shows that this is possible and even desirable [1].
1. Updates to occur solely based on the manifest URL - and no identity being tied to URLs that can change in fun and unpredictable ways.
1. very tight integration between OS and web browser. Again, I think that's fine - though it will likely be impossible for third-party browsers to support this "install" functionality... though these browsers may do other interesting things with a manifest, apart from installation (e.g., fancy speed dial cells, richer bookmarking, alternative "home screen").
1. Figuring out whose scope wins when linking is left to the implementation (use last installed app, display a dialog allowing user to choose, etc.).

[1] http://w3c-webmob.github.io/installable-webapps/#security-and-privacy


Heh, for my setup with default permissions on the current user, with plist installed to `~/Library/LaunchDaemons/`, and the other example plist for local with GUI, the ios simulator would always fail when this token open was present, removed it, and it worked :)

> On 13 Jan 2015, at 5:22 pm, Michael Koukoullis <notifications@github.com> wrote:
>
> @tommeier just for the record why did you find SessionCreate evil?
>
> —
> Reply to this email directly or view it on GitHub.
>


Comment is not really a nice name for this, "description" makes a lot more sense to me. "Label" or "annotation" may work too.  The word "comment"  just doesn't make sense to me.


1.This RIL just been taken from another phone, raw99 says it is unstable RIL, so in the future I will not use it.
2.Please wait, I almost fixed the sd card, when I completed yet, I will compile a new build for torino


someIntern.makeForm( "robotomy request", "ur mum" );


nit: this is already imported above


I'm using xmpphp for ejabberd. its working fine. connection is happening and m able to send a message using xmpphp library to ejabberd server. But i'm not able to add users in ejabberd. I have added the above two functions in XMPPHP/XMPP.php file and i called the register function in sendmessage_example.php as follows :
$conn->register('uname', 'pass');
but no use. its not creating users in ejabberd xmpp server. Someone help me pls...
Thanks in advance.


@mcm-ham after making this change of adding "to_json" we need to make sure it will work well with https://github.com/crossroads/api.goodcity/blob/49cce5481fb0b04375ab914a5f13081e9b2f1127/app/services/push_service.rb#L21
as "to_json" gets called twice. I see that you have used .tr to remove the double quotes. But still it would be good we make sure that `notify` method works fine else it will break the Notification.


Replacing the line 598: `if brushMode is not None:` with `if True:` solves the broken brush saving.


Woops, I also managed to stomp over the almost finished wolf3d.yaml; hopefully I can reread the mails on the BTS.

I don't mind doing this for the future.
```
git reset --hard upstream/master
git pull upstream master
git push -f origin
```


Also work on edit wiki page support


Let's please continue the discussion in https://github.com/w3c/manifest/issues/272


LGTM


Suppose we could submodule it/mirror it/whatever?


Oops, didn't mean to commit directly


Review this file from line 14.
https://github.com/Linspira/envitz.social/blob/0.02/public/css/import.css

```CSS
/* here is the base structure for a icon class: */
.icon-<descriptive name of icon> { /* icon description */
background-image:url(data:image/<"png" -or- "svg+xml">;base64,/*< insert the base64 string here! >*/) !important;
background-size: /*<width>*/px /*<height>*/px;
background-repeat: no-repeat;
display:inline-block;
border: 0;
}
```

In SA3, you can use the CSS class name in the ```iconCls``` property. Also, in HTML, you can do this:
```HTML
<div class="icon-myIconName"></div>
```
And, this will show the image from the CSS file.

You can review how I setup the button icons in https://github.com/Linspira/Linspira.Sencha/blob/master/envitz/view/NAVtool.js


These are enumerable so `for...in` loops will iterate over these.


Thanks, now it is working.


whoops good catch thanks :see_no_evil:


Does the lock belong inside the foreach?


It has the benefit of not being forced to handle an exception that is never thrown when dealing with the concrete class, and since this class is final, there will never be any subclasses that'll throw it.


:+1:  thanks for the fix, and the tests! i've deployed to instagram-atom.


Tox Project consists of all the people that contributed to toxcore, while Tox Foundation is just one-two people. Don't think you can legally take the copyright away from all the contributors like that.


Is this a problem with 6to5 source maps? I've heard issues about them but nobody is reporting anything.


Oh, I got it. Swiftpod uses dynamic frameworks that supports only for iOS 8.


@sebmck No, no. Test is still in progress. I'm working with DevTools on Chrome and Firebug on Firefox by hand...


把已经放到暂存区的修改撤销掉，放回工作区
$ git config --global alias.unstage 'reset HEAD'
显示最后一次提交信息
$ git config --global alias.last 'log -1'

git log -------->
git config --global alias.lg "log --color --graph --pretty=format:'%Cred%h%Creset -%C(yellow)%d%Creset %s %Cgreen(%cr) %C(bold blue)<%an>%Creset' --abbrev-commit"


@bbcrddave, I see your point, but I am not so sure that the player should give up trying to find the live edge only because time sync did not help to determine it. I think that if there is a chance to find the live edge using binary search it has to be done. I understand that my point is also arguable, so how about an intermediate solution - add a boolean property (e.g. useLiveEdgeBinarySearchFallback) that is used to decide whether the search is done or not. The property must be in MediaPlayer API so that developers have a control over this behavior.


@sebmck So, wait a minute please. Seems fine. Works for me.


@twada Ah, great. Just making sure!


Isn't `state` already tested to not be `…Loading` on L136?


For each new interface, please add detailed description like @param, @since


This is a UI string; in that realm "authenticated" is not what you mean here. This wording could be improved.

to the user an "authenticated" thing is a person who logged in.

Maybe something like "Merchant (known)".


cool


Perhaps ditch this subview addition and just insert it at the correct z-index?


what is this? It works?


you should move your requirements in general routig file to make locale available for all routes of certain bundle. :)


this is test


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


user translations on the link text


cmmit


It's working for us, but of course, if you notice any problems don't hesitate to let me know.  The heuristics used to determine whether or not to fire off a verify build have changed (I think for the better), but you should keep your eyes peeled after deploying for a bit just to make sure.


Thanks for highlighting it to me to test, I will do so. It should be ok because I'm just converting the property of the data object from a date to a string and the json output of the data object should be the same. Here's the rails console test I did.

```
{o:Time.now}.to_json
"{\"o\":\"2015-01-13T15:00:28.867+08:00\"}"
{o:Time.now.to_json.tr('"', '')}.to_json
"{\"o\":\"2015-01-13T15:00:55.219+08:00\"}"
```


84202670-DuplicateNodeInZoneConnCausesCrash (Myoldmopar) - x86_64-MacOS-10.9-clang-Debug: Tests Failed

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


@sebmck [worked well!](https://github.com/twada/battlefield-sourcemaps/commit/cd951b23946c6bb25204e121314ded3f75e5959b)


I started to play with the interface and this feels out of place.


Here is where you generate the file in /tmp and this same file gets written back to the radio. Maybe it should be deleted after the compatibility check is done.


为什么删除了？
`go run search_server.go`提示找不到`weibo_data.txt`


You dont need to commit files mode changes, go into `.git` folder in your project, open `config` file, and change `filemode` property to false.


localo to `app/config/routing.yml`


try {{ sendmail.action.success|trans }}


Good news !


THe -t was an invalid option if I recall it correctly

From: Leo Baschy [mailto:notifications@github.com]
Sent: Saturday, January 10, 2015 8:49 PM
To: thesecmaestro/nrvr-commander
Subject: Re: [nrvr-commander] Fixed java-version (31bae3c)

looking at your line 681 setShutdown, wondering
1) the portsFile should have remembered earlier setting, did you really need it there ?
2) isn't it supposed to be -s -t 20 i.e. did you miss -t ?

we might find all that isn't needed anyway, don't know yet

just observations, not criticism

—
Reply to this email directly or view it on GitHub <https://github.com/thesecmaestro/nrvr-commander/commit/31bae3caa2db1911d3e07288c984b36d3daf288b#commitcomment-9232167> .  <https://github.com/notifications/beacon/AJx5teHiINWMRLBFVbYdZJr-as4A8Dc7ks5ngfgpgaJpZM4DQ0vm.gif>


Reviewed!


Thanks. Familiar with all of those :)

From: Leo Baschy [mailto:notifications@github.com]
Sent: Saturday, January 10, 2015 6:57 PM
To: thesecmaestro/nrvr-commander
Cc: thesecmaestro
Subject: Re: [nrvr-commander] Fixing indefinte loops for waitingForJavaWInstallerSuccess (39bfd7c)

also mentioning debugging or working on this is a great example of seeing snapshots at work in this script:

if earlier snapshots exist, it will use them, which means:

• either this code failed, then on next run it anyway will do installToolsIntoTestVm()

• or if this code succeeded, then if deleting snapshot "tools installed" and anything after it will cause installToolsIntoTestVm() to be done again

big time saver not having to wait for repeated install of OS to get to tools install

if not wanting to deal with snapshots from VMware UI, there is in NrvrCommander class VMwareHypervisor a method revertToSnapshotAndDeleteDescendants, which you'd call with the previous snapshot, i.e. "OS installed"

—
Reply to this email directly or view it on GitHub <https://github.com/thesecmaestro/nrvr-commander/commit/39bfd7cb47dd764126810f0da359b6a28a4c80c6#commitcomment-9232021> .  <https://github.com/notifications/beacon/AJx5tc_ccGIJBU5jc3fZ_dC3ThwhtxH4ks5ngd3ggaJpZM4DQ00P.gif>


Good. I think you can remove the `!value.length && ` in this line now.


r=me, with some nits addressed.


Sounds good. I will make the pull request

From: Leo Baschy [mailto:notifications@github.com]
Sent: Saturday, January 10, 2015 4:56 PM
To: thesecmaestro/nrvr-commander
Cc: thesecmaestro
Subject: Re: [nrvr-commander] Fix for IsRunning symlink issue (e2d8cce)

if you make pull request for just this change i will merge it

—
Reply to this email directly or view it on GitHub <https://github.com/thesecmaestro/nrvr-commander/commit/e2d8ccecc6b83c2eecfb8c40266212aedb984521#commitcomment-9231765> .  <https://github.com/notifications/beacon/AJx5tb1sZQZacC5mq-g-cjfToMveH8AMks5ngcG3gaJpZM4DQ0Yc.gif>


ack


If you strip the time zone at the end, why do we still need a different pattern to parse the time with?


+# hello-world
+a testing respository.


This is not a proper description


this seems to be an unsigned int, at east you use it as such. If it isnt the whole thing doesnt compile because of line 144. So why don't you just guarantee that its an int?


please check your code


I have it.


ack after you replaced this really big type I mentioned by unsigned int. I don't need to rereview again, just push the modified thing to master.


Thanks, fixed in https://github.com/EchoAppsTeam/js-sdk/commit/b363a7306f8e0d96bf43602399de78559f4c79b4


I pushed the remaining  easy & non-intrusive bits of ROTT & Wolf3D to master.


same here with this ugly ugly type that is just a hidden unsigned int


ack, fixes #168


Hi Seymour Knowles-Barley, I have tried to download MojoSdk.zip for a lot of times, but I can't get it, the download speed is always 0 around 30%. My location is Beijing, China. Could you put MojoSdk.zip on the github website or some cloud storage, I guess maybe it is the website's problem. Thank you very much, really hope you can see my problem.

PS: may you can add some notes on http://people.seas.harvard.edu/~seymourkb/


This could even overwrite custom splash screen varaint file, because generateProcessUniqueTempFileName("flash.bin") always generates same file name in current process.


Looks good. Thank you. This will be in the next release.


Is there any reason to use pinsource instead of pin? I was planning getting rid of pinsource and compute it where needed


禁用 JS Shell 之后，如果 --enable-shared-js 就可以将 js/src/moz.build 中的 FORCE_STATIC_LIB 和 STATIC_LIBRARY_NAME 注释掉。


ack


.jsp


USE_USART3_TX_DMA ?


actually this is far better than the previous example sione it doesnt break in mid-statement, would you be so kind mentioning this?


OK, but IMO readability is preferable here ...


hjjjjjjjj😍


please don't rebase, I'm a bit tired of rereviewing everything again. Just push  a docfux commit on top, ok?


There'd better be tests covering all these cases... (the empty delimiter case is not used by the current code)


Hehe thanks. I'm currently in the process of writing unit tests for some of the supported HTTP features (`Cache-Control`, `If-Modified-Since`, `If-Unmodified-Since`, `Range`, `If-Range`, ...) so that I can verify that it still works properly after the refactor. Obviously I already ran into some minor issues with the current implementation while writing the unit tests ;-)


so ack as soon as docfix commit is available


I've changed to single quotes here for consistency.


True this, but I guess we'll have to be a bit loosed on test coverage in KnowSheet on the browser side ...


Nice!


Ah, you didn't have to. Thanks anyway! I like consistency. Consistency is good. :-)


fixes #20


(This PR is already merged, are you making a follow-up one?)


ack, assuming you tested that it works correctly at least manually?


switch to new roon theme:
https://github.com/TryGhost/Roon/


make it flexible, add parameter $message to function, and then set it to setBody()


Nope, that was in the reformat commit you've already merged (this discussion is attached to the file at certain point in time, not to the pull request, so the code we discuss is outdated).


I'm not quite sure why we need to handle question mark.


I just did a git clone of the rainloop-webmail and copied the new files over my rainloop install. Is there anything else I should do in order to test the DKIM status implementation?


@lincsoon @halton @wang16


$em = $this->getDoctrine()->getManager();
$em->persisit($news);
$em->flush();
, try to DONT REPEAT YOURSELF. :smiley:


saw approval from kuenishi
at https://github.com/basho/webmachine/commit/9c0a5b970cfc51342b1e4e432a697b1c5b76cc3f


merging basho/webmachine/bugfix/bch/update-meck-deps = 9c0a5b97 into borshop-integration-231-bugfix/bch/update-meck-deps


basho/webmachine/bugfix/bch/update-meck-deps = 9c0a5b97 merged ok, testing candidate = 0a0eaf25


Oh, I didn't know about that. Could you please do that for me? I think
it should be there in every mode (except maybe URL/email mode, but
having it there as well won't harm).

--
Cheers,
  Andrew


No, it reuses __eq__ from str.

On Tue, Jan 13, 2015 at 4:16 AM, Nikolay Kim <notifications@github.com>
wrote:

> @asvetlov <https://github.com/asvetlov> should we implement *eq* for
> upstr?:
>
>    def __eq__(self, other):
>        return self is other or self == other
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KeepSafe/aiohttp/commit/e7d978ad072e13d1718affe045019e2d15529d00#commitcomment-9252760>
> .
>

--
Thanks,
Andrew Svetlov


Indent mine, I don't use beautifiers and don't know if there exists one that supports my coding rules.
I prefer to keep these multiline expression parens similar to `{ }` and not glue them to the last line of the expressions.


look good to me, but I would call it just setting and not xwalkSetting


i understand that, but because i moved all headers we use into hdrs.py, would it be faster to use `is` first and then `super().__eq__`?


fuck cdma and seriously gsm master race


Roger this, thanks!


ok


let's assume that almost always will use aiohttp.hdrs.XXX for header name


Oui tu as raison - je faisais juste référence à la version originale (Everything else loads in alphabetical order based on the file name). On peut probablement s'en passer.


1: M100 needs more reviewers
1: M101 signature is missing


@chengrenz  有的，有空放上来


1: M111 unrecognized component name: collections


Need to create an instance of it.


Ah oui autant pour moi, c'est bien là.


Here: http://d-h.st/kFLa


Audio quality is better than stock. This kernel is having most updates for audio codecs.


also size versus $size.


Readme never on Github


I didn't notice that change earlier, /etc/config/ is strictly reserved for uci configuration on OpenWrt, putting non-uci stuff there can cause various problems.

What was wrong with /etc/ocserv?


'C'est' -> 'Ceci est'?


I will read it and tell you something wrong about the smali

发自我的 iPhone

> 在 2015年1月13日，下午3:45，Abiel Zulio M <notifications@github.com> 写道：
>
> Here: http://d-h.st/kFLa
>
> —
> Reply to this email directly or view it on GitHub.
>


No need * character in Mhook_Unhook defination.And should mod the cpp file.


all tests pass:
success: http://buildbot.bos1/builders/test-webmachine/builds/291


为什么多两行？


Hi, worked like a charm with the other pull request, and this typo was already fixed.


The dependency dev-util/cmake should never be set directly, which makes CDEPEND obsolete. See cmake-utils.eclass how to set it properly.


what? it compiles doesn't it? :).
it was added for convenience in case somebody wanted to run this test indefinitely to see the crashes that occur somewhat randomly, which I reported in FB.

it's gone.


'votre' -> 'notre'
(The next step will be creating a MongoDB database for our app)


`They`? Should it be `We`?


Fast check for `is` is already performed by PyUnicode_Compare function.

On Tue, Jan 13, 2015 at 9:42 AM, Nikolay Kim <notifications@github.com>
wrote:

> let's assume that almost always will use aiohttp.hdrs.XXX for header name
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KeepSafe/aiohttp/commit/e7d978ad072e13d1718affe045019e2d15529d00#commitcomment-9255077>
> .
>

--
Thanks,
Andrew Svetlov


Exactly my thoughts... The NChar line was written in august, I have never seem the failing test before. And on 0.50.0 it only failed under certain conditions (which I could not find any meaning in) :-s

Can NUnit run tests in random order, like XUnit does? I have caught quite a few false positives in accidentially interdependent tests that way.


For movement, `status` is also auditable so entries were getting logged in delta report which should not.  So,here I skipped audit entries with :status key.


yep


`burnDialog::on_BurnFlashButton_clicked()` uses same file name (generateProcessUniqueTempFileName("flash.bin")) when it changes splash screen. And this file then gets overwirted by the compatibility check.


ok


I remember if we call 'property_set("ctl.start", "bluetoothd")' already. Do we still need to add 'on property' trigger statement?


Removed in @712043f1004d781b001d1d30c7e99e6c7b98df18. Thanks!


Seems there is a typo in `convestion`


That was how it auto-completed. I guess either should work.


There's one problem here. If we want to put the event listener in the blind as what's happening, then we need to figure out what to do if the listener tries to use a transaction. The javadoc of `afterCommit` states that we should create a transaction with `REQUIRES_NEW` semantic if we need to run any transactional operation; a typical listener implementation may use a service with `REQUIRED` propagation or the `transactionTemplate`. And this won't give the expected result...


I removed these lines (and related lines 36-43) in @712043f1004d781b001d1d30c7e99e6c7b98df18. Should I also pass "--with-intl=small-icu" to "./configure" by default?


WTF, push these to wrong repositories


Updated in @712043f1004d781b001d1d30c7e99e6c7b98df18. Thanks @ruimarinho!


Ok, cool

On Monday, January 12, 2015, Andrew Svetlov <notifications@github.com>
wrote:

> Fast check for `is` is already performed by PyUnicode_Compare function.
>
> On Tue, Jan 13, 2015 at 9:42 AM, Nikolay Kim <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> wrote:
>
> > let's assume that almost always will use aiohttp.hdrs.XXX for header
> name
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/KeepSafe/aiohttp/commit/e7d978ad072e13d1718affe045019e2d15529d00#commitcomment-9255077>
>
> > .
> >
>
>
>
> --
> Thanks,
> Andrew Svetlov
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KeepSafe/aiohttp/commit/e7d978ad072e13d1718affe045019e2d15529d00#commitcomment-9255188>
> .
>


bereken vooruitgang


It is legal for apps to set constants <= NINE_MAX_CONST_F for vs, because the spec says NINE_MAX_CONST_F is the minimum number of constants allowed to be supported.
I was said users of cards with lower This->max_vs_const_f prefers things to buggy rather than having crash,
so your patches doesn't fit.


verander wachtwoord


Bedrijfslogo succesvol veranderd


good idea.


84694650_EnableCodeCoverage (lefticus) - x86_64-MacOS-10.9-clang: Tests Failed

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


Upload mislukt. Controleer of uw logo kleiner is dan 300 x 300px en JPG/PNG


Gebruik dit formulier om je werktijd handmatig bij te houden


Even tho that's true, I'll keep it as '7.0' here. Since other than the fact that cocoapods is creating an embedded framework, there is nothing in the project that uses 8+.

My use case is an example: I'm using embedded frameworks and my target is iOS7 (Embedded frameworks work fine in a dev environment, the only problem is the app store submission), and then for the release I'll have a script that moves the .swift files from the embedded frameworks to the project.


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


nieuw project


Nieuw item


After sleeping about it, the strict adherence to the module prefix may be too much. The run opts are user interface after all. So @chelmuth complaint is reasonable. When we applied this module-prefix approach consequently, ```--qemu-args``` would eventually become ```--power-on-qemu-args```. Well, I admit that this becomes awkward. ;-) So I would be fine with using ```--amt-password```. Sorry for derailing the discussion.


Nieuw emailadres


typo in comment

Added server parameter exanples.
Added server parameter examples.


Niets om weer te geven.


sure that works, but it also adds 3 spaces to each variable so you end up with

```ip = '   10.0.0.2'``` and as lua has no trim() function either.....

and ```print(wifi.sta.getip())``` is what most people use and that doesn't work as before.


1: D100 Docstring missing
2: I102 copyright year is outdated, expected 2015 but got 2014
28:32: E261 at least two spaces before inline comment
28:33: E262 inline comment should start with '# '
40:28: E126 continuation line over-indented for hanging indent
47:37: E126 continuation line over-indented for hanging indent
49:39: E126 continuation line over-indented for hanging indent
63:40: E126 continuation line over-indented for hanging indent
73:1: E302 expected 2 blank lines, found 1
73: D202 No blank lines allowed *after* function docstring, found 1
73: D400 First line should end with '.', not 'p'
218:1: E302 expected 2 blank lines, found 1
218: D203 Expected 1 blank line *before* class docstring, found 0
218: D204 Expected 1 blank line *after* class docstring, found 0
218: D205 Blank line missing between one-line summary and description
236: D401 First line should be imperative: 'Create', not 'Creates'
254: D400 First line should end with '.', not 'm'
254: D205 Blank line missing between one-line summary and description
276:30: E713 test for membership should be 'not in'
305:107: E502 the backslash is redundant between brackets
322:111: E502 the backslash is redundant between brackets
336: D401 First line should be imperative: 'Return', not 'Returns'
336: D205 Blank line missing between one-line summary and description
409: D401 First line should be imperative: 'Thi', not 'This'
409: D400 First line should end with '.', not 'm'
409: D205 Blank line missing between one-line summary and description
422: D401 First line should be imperative: 'Convert', not 'Converts'
422: D200 One-line docstring should not occupy 3 lines
432: D400 First line should end with '.', not 'g'
432: D205 Blank line missing between one-line summary and description
487:34: E711 comparison to None should be 'if cond is not None:'
494:80: E502 the backslash is redundant between brackets
509: D401 First line should be imperative: 'Look', not 'Looks'
509: D400 First line should end with '.', not 'o'
509: D205 Blank line missing between one-line summary and description
530: D400 First line should end with '.', not 'a'
530: D205 Blank line missing between one-line summary and description
556:1: E302 expected 2 blank lines, found 1
556: D401 First line should be imperative: 'Get', not 'Gets'
556: D200 One-line docstring should not occupy 3 lines
568:1: E302 expected 2 blank lines, found 1
568: D401 First line should be imperative: 'Return', not 'Returns'
568: D205 Blank line missing between one-line summary and description
587:1: E302 expected 2 blank lines, found 1
587: D290 White spaces around docstring should be trimmed.
587: D400 First line should end with '.', not 'h'
591:1: E302 expected 2 blank lines, found 1
591: D400 First line should end with '.', not 'm'
591: D205 Blank line missing between one-line summary and description
629:1: E302 expected 2 blank lines, found 1
629: D400 First line should end with '.', not 'y'
629: D205 Blank line missing between one-line summary and description
641:51: E502 the backslash is redundant between brackets
642:67: E502 the backslash is redundant between brackets
647:1: E302 expected 2 blank lines, found 1
647: D401 First line should be imperative: 'Generate', not 'Generates'
669:54: E502 the backslash is redundant between brackets
672:68: E502 the backslash is redundant between brackets
673:30: E127 continuation line over-indented for visual indent
674:57: E502 the backslash is redundant between brackets
675:31: E127 continuation line over-indented for visual indent
676:61: E502 the backslash is redundant between brackets
677:31: E127 continuation line over-indented for visual indent
680:1: E302 expected 2 blank lines, found 1
680:76: E502 the backslash is redundant between brackets
680: D400 First line should end with '.', not 'o'
680: D205 Blank line missing between one-line summary and description
681:72: E502 the backslash is redundant between brackets
682:59: E502 the backslash is redundant between brackets
683:70: E502 the backslash is redundant between brackets
755:83: E502 the backslash is redundant between brackets
794:105: E502 the backslash is redundant between brackets
795:65: E128 continuation line under-indented for visual indent
800:107: E502 the backslash is redundant between brackets
801:67: E128 continuation line under-indented for visual indent
815:107: E502 the backslash is redundant between brackets
821:111: E502 the backslash is redundant between brackets
834:107: E502 the backslash is redundant between brackets
835:67: E128 continuation line under-indented for visual indent
844:110: E502 the backslash is redundant between brackets
848:1: E302 expected 2 blank lines, found 1
848:98: E502 the backslash is redundant between brackets
848: D401 First line should be imperative: 'Matche', not 'Matches'
849:69: E502 the backslash is redundant between brackets
850:74: E502 the backslash is redundant between brackets
921:24: E711 comparison to None should be 'if cond is not None:'
934:76: E502 the backslash is redundant between brackets
948:75: E502 the backslash is redundant between brackets
949:37: E128 continuation line under-indented for visual indent
957:101: E502 the backslash is redundant between brackets
965:100: E502 the backslash is redundant between brackets
966:84: E502 the backslash is redundant between brackets
967:84: E502 the backslash is redundant between brackets
968:98: E502 the backslash is redundant between brackets
969:94: E502 the backslash is redundant between brackets
970:86: E502 the backslash is redundant between brackets
1041:88: E502 the backslash is redundant between brackets
1048:42: E128 continuation line under-indented for visual indent
1048:86: E502 the backslash is redundant between brackets
1052:42: E128 continuation line under-indented for visual indent
1054:42: E711 comparison to None should be 'if cond is None:'
1061:42: E711 comparison to None should be 'if cond is None:'
1079:119: E502 the backslash is redundant between brackets
1087:112: E502 the backslash is redundant between brackets
1088:96: E502 the backslash is redundant between brackets
1089:96: E502 the backslash is redundant between brackets
1090:110: E502 the backslash is redundant between brackets
1091:106: E502 the backslash is redundant between brackets
1092:98: E502 the backslash is redundant between brackets
1124:1: E302 expected 2 blank lines, found 1
1124: D401 First line should be imperative: 'Take', not 'Takes'
1124: D400 First line should end with '.', not 't'
1124: D205 Blank line missing between one-line summary and description
1145:1: E302 expected 2 blank lines, found 1
1145: D400 First line should end with '.', not 'y'
1145: D205 Blank line missing between one-line summary and description
1156:1: E302 expected 2 blank lines, found 1
1156: D400 First line should end with '.', not 's'
1156: D205 Blank line missing between one-line summary and description
1164:18: E128 continuation line under-indented for visual indent
1165:20: E126 continuation line over-indented for hanging indent
1189:18: E126 continuation line over-indented for hanging indent
1191:34: F841 local variable 'e' is assigned to but never used
1324:90: E502 the backslash is redundant between brackets
1326:86: E502 the backslash is redundant between brackets


1: D100 Docstring missing
4: I102 copyright year is outdated, expected 2015 but got 2014
29:32: E126 continuation line over-indented for hanging indent
31:28: E126 continuation line over-indented for hanging indent
40:43: E127 continuation line over-indented for visual indent
41:43: E127 continuation line over-indented for visual indent
42:43: E127 continuation line over-indented for visual indent
43:43: E127 continuation line over-indented for visual indent
44:16: E272 multiple spaces before keyword
45:46: E127 continuation line over-indented for visual indent
70:1: E302 expected 2 blank lines, found 1
70: D200 One-line docstring should not occupy 3 lines
75:5: F841 local variable 'verbose_param' is assigned to but never used
172:8: E713 test for membership should be 'not in'
173:80: E502 the backslash is redundant between brackets
196:70: E261 at least two spaces before inline comment
231:1: E302 expected 2 blank lines, found 1
231: D401 First line should be imperative: 'Processe', not 'Processes'
320:8: F821 undefined name 'direct_xml_output'
326: D401 First line should be imperative: 'Remove', not 'Removes'
340: D202 No blank lines allowed *after* function docstring, found 1
376: D401 First line should be imperative: 'Attempt', not 'Attempts'
421: D202 No blank lines allowed *after* function docstring, found 1
421: D400 First line should end with '.', not 'r'
421: D205 Blank line missing between one-line summary and description
455: D400 First line should end with '.', not 't'
455: D205 Blank line missing between one-line summary and description
514: D400 First line should end with '.', not 'e'
514: D205 Blank line missing between one-line summary and description
562: D400 First line should end with '.', not ','
562: D205 Blank line missing between one-line summary and description
721:44: E713 test for membership should be 'not in'
759:49: E261 at least two spaces before inline comment
759:50: E262 inline comment should start with '# '
931:21: E127 continuation line over-indented for visual indent
946: D401 First line should be imperative: 'Take', not 'Takes'
946: D202 No blank lines allowed *after* function docstring, found 1
946: D400 First line should end with '.', not 'o'
946: D205 Blank line missing between one-line summary and description
1069:17: E122 continuation line missing indentation or outdented
1069:63: E502 the backslash is redundant between brackets
1070:17: E122 continuation line missing indentation or outdented
1074:72: E502 the backslash is redundant between brackets
1075:21: E128 continuation line under-indented for visual indent
1079:29: E126 continuation line over-indented for hanging indent
1097:21: E122 continuation line missing indentation or outdented
1141: D202 No blank lines allowed *after* function docstring, found 1
1141: D400 First line should end with '.', not 's'
1141: D205 Blank line missing between one-line summary and description
1171:60: E502 the backslash is redundant between brackets
1180:16: E713 test for membership should be 'not in'
1191:16: E713 test for membership should be 'not in'
1199:16: E713 test for membership should be 'not in'
1207:16: E713 test for membership should be 'not in'
1215:16: E713 test for membership should be 'not in'
1219:44: E261 at least two spaces before inline comment
1226:20: E713 test for membership should be 'not in'
1230:20: E713 test for membership should be 'not in'
1236: D401 First line should be imperative: 'make', not 'makes'
1236: D400 First line should end with '.', not 'o'
1236: D205 Blank line missing between one-line summary and description
1351:1: E302 expected 2 blank lines, found 1
1351: D103 Docstring missing


3: I102 copyright year is outdated, expected 2015 but got 2014


I agree with @tominon with this. We should be able to pass closures to the view. In the past I have used an object that extends Closure and given it a `__toString` method to get the behaviour this commit implements but without the restriction.


indentation


Thanks! :-)


ok


put proposed new commit message in subsequent commit. will squash into appropriate number of commits. but i did it this way so rebasing during a PR won't mess up your comments.


Good patch, but perhaps it would be even more safer to add in addition user assert to check ppSurface is not NULL.


Odd CI server is passing, but seeing this as well locally...


Gör inte en ny rad för varje visning. Den skriver över den gamla


Ant


Yes slim has deprecated that in 3.0, from the the change log: "* Deprecate `='`, `=='` and `tag'` syntax for trailing whitespace. Use `=<` etc. instead."


dafuq


Instellingen zijn niet succesvol opgeslagen. Probeer nogmaals.


wat


Subtotaal


1: D100 Docstring missing
28:32: E261 at least two spaces before inline comment
28:33: E262 inline comment should start with '# '
40:28: E126 continuation line over-indented for hanging indent
47:37: E126 continuation line over-indented for hanging indent
49:39: E126 continuation line over-indented for hanging indent
63:40: E126 continuation line over-indented for hanging indent
73:1: E302 expected 2 blank lines, found 1
73: D202 No blank lines allowed *after* function docstring, found 1
73: D400 First line should end with '.', not 'p'
218:1: E302 expected 2 blank lines, found 1
218: D203 Expected 1 blank line *before* class docstring, found 0
218: D204 Expected 1 blank line *after* class docstring, found 0
218: D205 Blank line missing between one-line summary and description
236: D401 First line should be imperative: 'Create', not 'Creates'
254: D400 First line should end with '.', not 'm'
254: D205 Blank line missing between one-line summary and description
276:30: E713 test for membership should be 'not in'
305:107: E502 the backslash is redundant between brackets
322:111: E502 the backslash is redundant between brackets
336: D401 First line should be imperative: 'Return', not 'Returns'
336: D205 Blank line missing between one-line summary and description
409: D401 First line should be imperative: 'Thi', not 'This'
409: D400 First line should end with '.', not 'm'
409: D205 Blank line missing between one-line summary and description
422: D401 First line should be imperative: 'Convert', not 'Converts'
422: D200 One-line docstring should not occupy 3 lines
432: D400 First line should end with '.', not 'g'
432: D205 Blank line missing between one-line summary and description
487:34: E711 comparison to None should be 'if cond is not None:'
494:80: E502 the backslash is redundant between brackets
509: D401 First line should be imperative: 'Look', not 'Looks'
509: D400 First line should end with '.', not 'o'
509: D205 Blank line missing between one-line summary and description
530: D400 First line should end with '.', not 'a'
530: D205 Blank line missing between one-line summary and description
556:1: E302 expected 2 blank lines, found 1
556: D401 First line should be imperative: 'Get', not 'Gets'
556: D200 One-line docstring should not occupy 3 lines
568:1: E302 expected 2 blank lines, found 1
568: D401 First line should be imperative: 'Return', not 'Returns'
568: D205 Blank line missing between one-line summary and description
587:1: E302 expected 2 blank lines, found 1
587: D290 White spaces around docstring should be trimmed.
587: D400 First line should end with '.', not 'h'
591:1: E302 expected 2 blank lines, found 1
591: D400 First line should end with '.', not 'm'
591: D205 Blank line missing between one-line summary and description
629:1: E302 expected 2 blank lines, found 1
629: D400 First line should end with '.', not 'y'
629: D205 Blank line missing between one-line summary and description
641:51: E502 the backslash is redundant between brackets
642:67: E502 the backslash is redundant between brackets
647:1: E302 expected 2 blank lines, found 1
647: D401 First line should be imperative: 'Generate', not 'Generates'
669:54: E502 the backslash is redundant between brackets
672:68: E502 the backslash is redundant between brackets
673:30: E127 continuation line over-indented for visual indent
674:57: E502 the backslash is redundant between brackets
675:31: E127 continuation line over-indented for visual indent
676:61: E502 the backslash is redundant between brackets
677:31: E127 continuation line over-indented for visual indent
680:1: E302 expected 2 blank lines, found 1
680:76: E502 the backslash is redundant between brackets
680: D400 First line should end with '.', not 'o'
680: D205 Blank line missing between one-line summary and description
681:72: E502 the backslash is redundant between brackets
682:59: E502 the backslash is redundant between brackets
683:70: E502 the backslash is redundant between brackets
755:83: E502 the backslash is redundant between brackets
794:105: E502 the backslash is redundant between brackets
795:65: E128 continuation line under-indented for visual indent
800:107: E502 the backslash is redundant between brackets
801:67: E128 continuation line under-indented for visual indent
815:107: E502 the backslash is redundant between brackets
821:111: E502 the backslash is redundant between brackets
834:107: E502 the backslash is redundant between brackets
835:67: E128 continuation line under-indented for visual indent
844:110: E502 the backslash is redundant between brackets
848:1: E302 expected 2 blank lines, found 1
848:98: E502 the backslash is redundant between brackets
848: D401 First line should be imperative: 'Matche', not 'Matches'
849:69: E502 the backslash is redundant between brackets
850:74: E502 the backslash is redundant between brackets
921:24: E711 comparison to None should be 'if cond is not None:'
934:76: E502 the backslash is redundant between brackets
948:75: E502 the backslash is redundant between brackets
949:37: E128 continuation line under-indented for visual indent
957:101: E502 the backslash is redundant between brackets
965:100: E502 the backslash is redundant between brackets
966:84: E502 the backslash is redundant between brackets
967:84: E502 the backslash is redundant between brackets
968:98: E502 the backslash is redundant between brackets
969:94: E502 the backslash is redundant between brackets
970:86: E502 the backslash is redundant between brackets
1041:88: E502 the backslash is redundant between brackets
1048:42: E128 continuation line under-indented for visual indent
1048:86: E502 the backslash is redundant between brackets
1052:42: E128 continuation line under-indented for visual indent
1054:42: E711 comparison to None should be 'if cond is None:'
1061:42: E711 comparison to None should be 'if cond is None:'
1079:119: E502 the backslash is redundant between brackets
1087:112: E502 the backslash is redundant between brackets
1088:96: E502 the backslash is redundant between brackets
1089:96: E502 the backslash is redundant between brackets
1090:110: E502 the backslash is redundant between brackets
1091:106: E502 the backslash is redundant between brackets
1092:98: E502 the backslash is redundant between brackets
1124:1: E302 expected 2 blank lines, found 1
1124: D401 First line should be imperative: 'Take', not 'Takes'
1124: D400 First line should end with '.', not 't'
1124: D205 Blank line missing between one-line summary and description
1145:1: E302 expected 2 blank lines, found 1
1145: D400 First line should end with '.', not 'y'
1145: D205 Blank line missing between one-line summary and description
1156:1: E302 expected 2 blank lines, found 1
1156: D400 First line should end with '.', not 's'
1156: D205 Blank line missing between one-line summary and description
1164:18: E128 continuation line under-indented for visual indent
1165:20: E126 continuation line over-indented for hanging indent
1189:18: E126 continuation line over-indented for hanging indent
1191:34: F841 local variable 'e' is assigned to but never used
1324:90: E502 the backslash is redundant between brackets
1326:86: E502 the backslash is redundant between brackets


1: D100 Docstring missing
29:32: E126 continuation line over-indented for hanging indent
31:28: E126 continuation line over-indented for hanging indent
40:43: E127 continuation line over-indented for visual indent
41:43: E127 continuation line over-indented for visual indent
42:43: E127 continuation line over-indented for visual indent
43:43: E127 continuation line over-indented for visual indent
44:16: E272 multiple spaces before keyword
45:46: E127 continuation line over-indented for visual indent
70:1: E302 expected 2 blank lines, found 1
70: D200 One-line docstring should not occupy 3 lines
75:5: F841 local variable 'verbose_param' is assigned to but never used
172:8: E713 test for membership should be 'not in'
173:80: E502 the backslash is redundant between brackets
196:70: E261 at least two spaces before inline comment
231:1: E302 expected 2 blank lines, found 1
231: D401 First line should be imperative: 'Processe', not 'Processes'
320:8: F821 undefined name 'direct_xml_output'
326: D401 First line should be imperative: 'Remove', not 'Removes'
340: D202 No blank lines allowed *after* function docstring, found 1
376: D401 First line should be imperative: 'Attempt', not 'Attempts'
421: D202 No blank lines allowed *after* function docstring, found 1
421: D400 First line should end with '.', not 'r'
421: D205 Blank line missing between one-line summary and description
455: D400 First line should end with '.', not 't'
455: D205 Blank line missing between one-line summary and description
514: D400 First line should end with '.', not 'e'
514: D205 Blank line missing between one-line summary and description
562: D400 First line should end with '.', not ','
562: D205 Blank line missing between one-line summary and description
721:44: E713 test for membership should be 'not in'
759:49: E261 at least two spaces before inline comment
759:50: E262 inline comment should start with '# '
931:21: E127 continuation line over-indented for visual indent
946: D401 First line should be imperative: 'Take', not 'Takes'
946: D202 No blank lines allowed *after* function docstring, found 1
946: D400 First line should end with '.', not 'o'
946: D205 Blank line missing between one-line summary and description
1069:17: E122 continuation line missing indentation or outdented
1069:63: E502 the backslash is redundant between brackets
1070:17: E122 continuation line missing indentation or outdented
1074:72: E502 the backslash is redundant between brackets
1075:21: E128 continuation line under-indented for visual indent
1079:29: E126 continuation line over-indented for hanging indent
1097:21: E122 continuation line missing indentation or outdented
1141: D202 No blank lines allowed *after* function docstring, found 1
1141: D400 First line should end with '.', not 's'
1141: D205 Blank line missing between one-line summary and description
1171:60: E502 the backslash is redundant between brackets
1180:16: E713 test for membership should be 'not in'
1191:16: E713 test for membership should be 'not in'
1199:16: E713 test for membership should be 'not in'
1207:16: E713 test for membership should be 'not in'
1215:16: E713 test for membership should be 'not in'
1219:44: E261 at least two spaces before inline comment
1226:20: E713 test for membership should be 'not in'
1230:20: E713 test for membership should be 'not in'
1236: D401 First line should be imperative: 'make', not 'makes'
1236: D400 First line should end with '.', not 'o'
1236: D205 Blank line missing between one-line summary and description
1351:1: E302 expected 2 blank lines, found 1
1351: D103 Docstring missing


indentation. also, use a better name


Verzenden


Abonneer op nieuwsbrief


Geef weer


Added the ability to output to Json, Plain, Shell


Done (merged from what we did).


Krijg een afbeeldings CAPTCHA


Removed.


Nieuw bedrijf


Percentage ontvangen


Duizendtallen scheiding


Decimalen scheiding


in afwachting


条件長すぎるので直してください


% in afwachting


% collectie


84694650_EnableCodeCoverage (lefticus) - Win64-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-68.80503144654088%25-red.svg)


Deze maand betaald


referentie bedrijf


One '_' missing between `user` and `result`.


Hoofdcontact


Bedrijfsreferentie


Why is this necessary? By doing this you are breaking something else. When entering the FAQ-page it should automatically choose the first category, to have something to show. setActive will do this when hash is invalid.


Dun dun dun

o:


This if is a good candidate for an extraction to a separate function or decorator to avoid code duplication. For example `abort_if_not_xhr()`.


Alle data inclusief facturen van dit bedrijf zullen verwijderd worden


:+1: Let's get rid of it


Verwijder bedrijf


You're right, this might be easy to misunderstand. I'll add another "or later".
Should we also add a version requirement for potr? For 2.6, that would be the yet-to-be-released 1.0.2 i guess...


You're right, this might be easy to misunderstand. I'll add another "or later".
Should we also add a version requirement for potr? For 2.6, that would be the yet-to-be-released 1.0.2 I guess...


Voeg contact toe


Bedrijf succesvol verwijderd


Joke command also needs updating


aww I liked that line :wink:


Hoofdcontact is ingesteld


Ok, thanks!


Thank you Lesley, that seems to be fine.

Please the next time don't break the long lines as this is done on purpose. To be git-friendly it is better to break a line only after a period. For example, in the commit above I'm not able to tell where you actually changed the wording and where you just broke the lines of text.

All the editor and text processors like Markdown display properly long lines.


Hi Nicolás

Thanks for this commit !

I would like to implement actual parsing of https request and response using self signed certificates. So could you please help me how to proceed with that ?

Thanks
Kumar


Did you manually include the latest changes from the base repository? You can simply add another remote via `git remote add upstream git@github.com:mank319/Go-For-It.git` (where `upstream` is the repository's name) and pull changes via `git pull upstream master`. This way it's easier for you to keep the code up to day and it leaves the commit history intact, which makes merging pull requests **a lot** easier in the future :wink:


Update Klantgegevens ok!


Done.


Gebruik succesvol toegevoegd


LGTM


bekijk contacten


Ticket verwijderen? Doorgaan?


Ticket succesvol verwijderd


verander status ticket


telangana state
india


Ticket status is veranderd


Why is this needed? I am adding file_usage in line 207 in sites/all/modules/publizon/publizon.module. By adding this cron should not delete the images..


Factureer project


Seconden


Verwijder tijd


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1848](http://teamcity.monogame.net/viewLog.html?buildId=4330&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 359, ignored: 6 Build time: 00:16:45


Laatst gewijzigd


@athanatos . Yes after clear(), all pending wb removed. So the cur_ios & cur_size are zero like at the initial state. Or am i missing something? Thanks very much!


Actually the array end check is needed or otherwise random numbers are sent to the vm. This is not really harmful but useless none the less. On windows ( tested on 8.1 ) at least va_arg returns random values which are not 0 when there is no values in the buffer left anymore.


to update the state


updating the state


Taak succesvol verwijderd


Bug status veranderd


Bug reacties


```
error: can't copy 'misago/bin/misago-makemessages.py': doesn't exist or not a regular file
```


Verwijder tijd?


@alexcheng1982 your change has broken the CI build.

I'd rather not change the existing web-jar which includes sdoc.jsp. It would probably be better to include both sdoc.jsp and sdoc.html.


Please, add a single function to do it and use it in every place youn need it.
Take a look here: http://stackoverflow.com/questions/1787322/htmlspecialchars-equivalent-in-javascript


Project gefactureerd


Probably here there is an error with name ```genComments``` and ```generalComments```.


Gefactureerd project


:+1: ! :)


'oy' in ev isn't necessary.


Klant email


naam afdeling


Hm, does this really work in sleshammer? I somehow doubt. Remember, sleshammer doesn't have zypper installed. Neither does it have the rpm db.

We also tried a somewhat similar patch in the past. Which we needed to revert again:
https://github.com/crowbar/barclamp-deployer/pull/102/files


afdeling succesvol aangemaakt


Yeah that looks like a bug. http://www.mirc.com/colors.html

I guess I assumed 1->16 rather than 0->15.

Will amend that in other locations too, thank you!


update afdeling ok


bewerk afdeling


verwijder afdeling


afdeling verwijderd


I think "no response" is required here too though.


logingegevens


Logingegevens


This is a bug into plasma-mobile which wasn't updated (removed KPackage dependency).
I can do a quick pull request, you can try with that.


Je comprends pas bien l'intérêt de ce test par rapport à `LinkerAnnotationProcessorTest`? (si on pouvait m'éclairer :p)
En plus je trouve sa mise en place plutôt compliquée.


gespendeerde tijd


titel taak


voeg veld toe


veldnaam


Can you remove the commented line please?


@kasper93 Looks good. But I wouldn't use ``auto`` for Windows handles :p


aangepast veld


Veldtype


Verwijder veld


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3125
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3118
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3119
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3110
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3110
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3113
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3106
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3101
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2778
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2776
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1614
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1607


fast-forwarding master to auto = 4fc9b412


Take a look into src/com/android/settings/deviceinfo/Status.java
there is duplicate definition of onOptionsItemSelected().


bewerk aangepast veld


aangepast veld verwijderd


Thanks for including my repository !


"They" = the filenames. But "We" would be better.


update aangepast veld ok


When will there be a solution?


This belongs in Kronolith_Event#readForm() where we alread do the attendee parsing.


ticket succesvol aangepast


bewerk ticket


Have you tried git diff --color-words?
On Jan 13, 2015 9:21 AM, "Francesco" <notifications@github.com> wrote:


It's the top priority item in TODOs. It will be deployed before weekend If I have a time :)

Thx for using pptp-admin!


blaa


en.. saveFile is not touched, why change it?


Every function should preferably have an accompanying docstring explaining the use of it.

I like the [format](http://sphinxcontrib-napoleon.readthedocs.org/en/latest/example_google.html) suggested by Google:

```
def read_config(config):
  """Reads parameters from a config file.

  Args:
    config (str): path to config file

  Returns:
    dict: parameters from the config file (unparsed)
  """
```


I believe that this commit caused the following issue:https://github.com/odoo/odoo/issues/4660


Capitalized variable names are usually reserved for classes in Python. Most variables in Python are lowercase and use underscores for clarity (classes use camel case).

```
class Person(object):
  pass
```


Relax :-)


`*~` is more like it i think.


missed this one


why is } on this line?


yes please.


fd


space after this?


Using plain su, bash is pid 1. Docker people wanted odoo to be pid 1 so that if it crashes for some reason, the container stops running.


what do you think this does?


why do you continue using `$db` if there was an error creating the PDO obj?


hello, applied this fix for my experia box v8 , with vrx288 v1.2 cpu, burt after leds affered to pef71071 v1.5 blink continous when link-on ethernet cable. There is a fix for this, to led stay solin on on link ?

Thank u.


It's not clear r600 supports the format looking at the sources, and I've seen people complain of an issue with an app crashing at launch because of issue with depth format. What else can it be.


This could be very nicely implemented as a context manager that handles everything from opening to closing the SSH connection. This makes for a very nice Pythonic API.

```
with create_tunnel([...]) as connection:
  # do something with the tunnel open

# the tunnel will be automatically closed here
```

Another advantage is that error handling can be contained in the function so we can make sure to close the tunnel no matter what happens in the code that runs inside the with-block!

More details: http://eigenhombre.com/2013/04/20/introduction-to-context-managers/


Can a single file have two Albums or so?
Does a player actually fill a second value?
Do you have an example file for this? A real live example would be helpfull as comment.
I wonder if we need a separator like " " or a special treatment for the additional field.
I also wonder what will happen if we ever write this data back.
It looks like we will mess all up.


let's not introduce new prints.


this is a BC break because the exception is expecting the class name as constructor argument in existing releases


You'd prefer a crash over a maybe buggy application ?


hm. I would have expected the None literal, not the "None" string.
Could raising an ValueError be an option?


Good one.


How do other player show such tracks?
It is probably a good idea to show only the first instance of such fields until we have a usecase or a request for it  ...


That seems to help! The problem was in the robot_skills-package, so I added a .travis to that as well.
The problem is not solved though, yet. Please see the [travis log](https://travis-ci.org/tue-robotics/robot_smach_states#L3788)


say no to @laughedelic, help existentials with merge


These echo commands seem to be a leftover of a testing session and produce unnecessary garbage output in terminal.


Wine doesn't check for invalid ppSurface either, so I guess this is like native behaves.


saw approval from alexcrichton
at https://github.com/Ms2ger/rust/commit/faaf9b4bb28856752f0fd5130e3a7b655243c73a


merging Ms2ger/rust/closures = faaf9b4b into auto


status: {"merge_sha": "8aeb3039efc391d6ebaa8d3993b30c8da8a10ca0"}


Ms2ger/rust/closures = faaf9b4b merged ok, testing candidate = 8aeb3039


Please add documentation to your function.  Maybe smth like
This function removes the client identified by the name, address, port, and fingegprint from the internal list of dicovered key servers.


hehe. that happens.. every time one bothers to actually write tests in retrospect ;)


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 12](http://192.168.44.2:8111/viewLog.html?buildId=36&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) is now running


@stof  I considered that, but then I decided that the parameter argument types are still matching (`$className` is a string)

Additionally, this API is only used internally.

Consider looking at https://github.com/doctrine/doctrine2/commit/5630c37b35c7cf10791975f982b9ad0e8dc369a6 - the merge diff shows that the API change is compatible. Consumers would just get a weird exception message if they used the constructor directly, but it would still be intelligible.


Do we really want to ignore those files? Those are the normative resources of version information, which could be useful for end users too, e.g. when debugging, or when they want to try a newer bundle.


Marks are currently only created with `\mark "some-string"`.
Research turned out that with `\mark \default` or `\mark 5` the `'label` property is an empty list.

The Scheme representation of `\mark "Text"` is
```
(make-music
  'MarkEvent
  'label
  "Text")
```

(which works with the current code),
`\mark \default` yields

```
(make-music
  'MarkEvent
  'label
  '())
```

and `\mark 8` results in

```
(make-music
  'SequentialMusic
  'elements
  (list (make-music
          'ContextSpeccedMusic
          'context-type
          'Score
          'element
          (make-music
            'PropertySet
            'value
            8
            'symbol
            'rehearsalMark))
        (make-music (quote MarkEvent))))
```

I don't really know what is happening here, but obviously we need to check for the case and construct the created mark manually.


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 12](http://192.168.44.2:8111/viewLog.html?buildId=36&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 2 Build time: 00:00:51
### Failed tests
```
Project Test Suite: AppBundle\Tests\Controller\DefaultControllerTest: testIndex: <no details avaliable>

```


According to PEP8, the maximum line length should be 79 chars ([ref](https://www.python.org/dev/peps/pep-0008/#maximum-line-length)).

You will learn many tricks to keep your lines short enough :smile:


The minimal version of CMake which should run this script. With newer versions come new basic commands, so ensuring from the beginning the right environment is given is very important.


Good observation. It seems that ProductionQ and the player Research object need to be connected. This would allow productionController to update Research with resources when they are available.


Can you add brackets there - that statement looks confusing without them. I'm slowly trying to remove all loop/conditionals without brackets in the codebase.


:trollface:


not neccessarily.

invalid ppSurface would crash directly with your code, but we should return invalidcall I guess instead


I don't care, because I don't hit that case, but people having these cards prefer buggy application


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 13](http://192.168.44.2:8111/viewLog.html?buildId=37&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) is now running