labofsahil/github-event-dataset-2012 huggingface dataset:
01-18 04:18:58.412: E/AndroidRuntime(1286): FATAL EXCEPTION: main
01-18 04:18:58.412: E/AndroidRuntime(1286):
java.lang.RuntimeException: Unable to create service
com.android.systemui.SystemUIService: java.lang.NullPointerException
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
android.app.ActivityThread.handleCreateService(ActivityThread.java:2263)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
android.app.ActivityThread.access$1600(ActivityThread.java:123)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
android.app.ActivityThread$H.handleMessage(ActivityThread.java:1201)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
android.os.Handler.dispatchMessage(Handler.java:99)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
android.os.Looper.loop(Looper.java:137)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
android.app.ActivityThread.main(ActivityThread.java:4424)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
java.lang.reflect.Method.invokeNative(Native Method)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
java.lang.reflect.Method.invoke(Method.java:511)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:784)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
com.android.internal.os.ZygoteInit.main(ZygoteInit.java:551)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
dalvik.system.NativeStart.main(Native Method)
01-18 04:18:58.412: E/AndroidRuntime(1286): Caused by:
java.lang.NullPointerException
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
com.android.systemui.statusbar.phone.PhoneStatusBar.prepareNavigationBarView(PhoneStatusBar.java:503)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
com.android.systemui.statusbar.phone.PhoneStatusBar.access$100(PhoneStatusBar.java:98)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
com.android.systemui.statusbar.phone.PhoneStatusBar$NavPanelObserver.onChange(PhoneStatusBar.java:431)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
com.android.systemui.statusbar.phone.PhoneStatusBar$NavPanelObserver.observe(PhoneStatusBar.java:426)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
com.android.systemui.statusbar.phone.PhoneStatusBar.makeStatusBarView(PhoneStatusBar.java:324)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
com.android.systemui.statusbar.StatusBar.start(StatusBar.java:64)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
com.android.systemui.statusbar.phone.PhoneStatusBar.start(PhoneStatusBar.java:269)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
com.android.systemui.SystemUIService.onCreate(SystemUIService.java:93)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	at
android.app.ActivityThread.handleCreateService(ActivityThread.java:2253)
01-18 04:18:58.412: E/AndroidRuntime(1286): 	... 10 more

On Wed, Jan 18, 2012 at 3:59 AM, Project Codename: Android
<reply@reply.github.com>
wrote:
> Add me to gtalk at codenamegit@gmail.com
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/AndroidUser00110001/AOSP--Custom-Settings/commit/d4cce4ce49db434b1e06a7d953e01fd8e41d62c5#commitcomment-877476
Do we still need to add the apks?
We can checkout and make a nightingale-1.11 branch from the commit before.

On Wed, Feb 29, 2012 at 2:32 AM, Antoine Turmel
<reply@reply.github.com>
wrote:
> awww This should be reverted IMHO, we are not releasing 1.12.0a but 1.11.0 maybe we can properly branch this before ? (as nightingale-1.11)
> then rename sb-trunk-oldxul to nightingale-trunk ?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/nightingale-media-player/nightingale-hacking/commit/d4a70b55653fe7424bf80d060cc8ee613e59a548#commitcomment-1025062
Yes I have thought about it, and I still think it's a good idea. I don't want to break the current API, however.

Currently—and I admit it is a bit yucky—you can enqueue a function by passing in a `timeout` kwarg, which gets gobbled by RQ and other kwargs are passed along the function. This renders it impossible to enqueue a function that expects a `timeout` kwarg itself.

My proposal would be to make a function with the signature you are proposing here, and have the `enqueue` method call that function internally, without the ability to use a timeout value. (For that, you should reside to calling the proposed verbose enqueue function.)

I'm not sure what a good name for that method would be, yet.
This method is called only once. I suggest you modify the type directly
I know but that's not relevant here I'm trying to find the handler of an arbitrary url for which I don't have context.
It may look strange. Though, you have to keep in mind that the ProvidersDefinition class belongs to the scribe-up library, which is independent from Spring (as it's used in Apache Shiro project for example which does not use Spring).
I could call the init() method any time or anywhere I want, but in fact, the ProvidersDefinition will be initialized only once, that's why I just put it in the OAuthAction class because this class is absolutely required for OAuth support.
If it was like that already then just leave it be, I just got curious :-)
added the check in SystemGuardian that it is the user guardian that terminates
created a separate class for SystemGuardian, more clean
OK, thanks for the feedback once again!
Nice catch! Saving Copy-Paste people from some headache :)
don't like this approach.  tons of code for simple task.
"created"
.NET 2.0?
Weird () around pending.
Just whitespace diffs here, so the tests now work and don't throw warnings.
Just put the Dll include in the ffmpeginternals. It's only for codec id's right?
Would it be difficult to have a fallback to ffmpeg in case the WMACodec rejects the files for whatever reason?
(or maybe the other way around :) )
mrjob.tools.emr.audit_usage.
Hmm, I guess this would mean somehow getting hold of the project then (resp probably simpler then hold of the DependencyFacet to delegate). Is there something around which can do this without dependency injection (as we don't have DI in e.g. the MavenPluginBuilder)?
Message should be "The EJB class already exists, do you want to overwrite it ?"
Same 2 blanks lines.
Ok. Will put it in the new pull request.
Yes, that's what I did already.
I couldn't get Pyramid to return a proper 404 or other status codes. This is also the reason for set_response_notfound()
yes I did :) I'll try to merge tonight
This whole block of code duplicates the vertical case. It should be factored out into a common method.
Make this a `const kMaxSessionSize = ...`
Espace avant le 1.
What do you mean? Yes, test case is necessary.
I just noted it's missing. :)
So would you rather it worked like this?

```c
PEM_write_bio(out, X509_PUBKEY_get(spki->spkac->pubkey));
```
Thanks for pointing it out. Copy paste boo-boo.
либо не переносить, либо : на другую строку
I think this doesn't belong to AS, making a copy is not a good solution neither.
We could put the shared code in Action Pack but then we can't make an Action View if that is the intention.
Another alternative could be having another gem to share code through AP itself and AC, AD, AV.

Hmm I'm not convinced by any solution :/
Changing this is the line that broke the test. ha!
Please make sure you name it :)
You can break the line here in your name, same as other entry.
Maybe `sym.forceStubFailure()` would be clearer/safer.
No, I'm completly fine with this PullRequest.
The important part is that the functionality goes into M4, the implementation can be tweaked after that.
I keep wondering where this error message comes from
Yes, I wasn't searching for it the right way - sorry for the noise.
an analog of the bisection
I am not sure why this was removed. Could you please explain why?
For the record: according to the recent [discussion](https://groups.google.com/d/topic/sympy/0JIcCbgAXRA/discussion) on the list, strings are _not_ allowed.

I've changed to list to `Tuple`.
init_session and init_printing both seem to use the unicode printer in the qtconsole and the notebook.
Sorry my bad what i meant to say is if <code>arg.is_nonnegative </code>is placed instead of <code> arg.is_positive</code> and <code> arg.is_zero</code> then its showing errors such as http://paste2.org/p/1979595
Yes, the require.ensure IS needed
Same as with cut and mkdir.
I'd just replace this with two calls to `add_work()`. Less code, easier to read. :-)
Drop the redundant `type, `media` attributes.
Ok I will make it same as the command parsing which cast all types to string is this is the defined type
I guess
so I guess I'm checkint to see if the other projects are not defaulted as well?
This is not relevant any more as next update will shift towards update_at modification.
This `endif` tag closes after `</ul>` and `<ul>` staring element is not included in the `if`
var cache on #searchform
hmm does this need to be broken up into multiple lines? seems like they would fit ok on a single line :)
I will not be  engaging in this argument with you.
Perhaps the manager should have a `get_normalized_valid_drivers()`. `clean_row` would still be part of the manager, but only used internally.
There was a core.generate_smilies_footer, this is now core.user_update_name. Pick one of verb-noun or noun-verb and stick to it.
We explicitly decided against making inline js any easier so that is not something that should be implemented. However I agree there should be a way to use includejs for files in the assets directory.

So yes, please create a separate ticket for that.
Not really as it has that as an input
note to self: this really should be a table.
[yui-sandbox.js](https://github.com/yahoo/mojito/blob/develop-perf/lib/yui-sandbox.js) is clever, but wasn't available when I wrote this, and doesn't address the issue I had. I needed to specify the path to YUI to `require` while working on the build stuff as an external command (i.e. an npm module mojito-cli-cmd-build).

That requirement doesn't apply anymore now that the code is in the mainline, so I'll take it out.
what's the purpose of redefining `base` here? I guess u will have to use `combine: false` otherwise it will continue use `comboBase` instead.
Using `libpath.basename(files[i], libpath.extname(files[i]))` instead of the `replace(/\.(js|css)$/, '')` is a little more correct/robust, but might not be worth the effort.
missing comb and it dependencies
as I commented below -- this inefficient use of set would be used only for cases when buggy numpy can't do "right"

anyways -- these cases should not occur often anyways ;) (atm it is just python3 - speicifc ;) )
Okay, we should use <property name="drug" here with the unique="true" ..../>
don't think we need these lines
For now it is just zeros_like(x) which is correct "almost everywhere." If we wanted to get fancier, we could change it to eq(x,argmax(x,y).dimshuffle).
Integers. And can you put this inside the method body, without the ****?
No problem. Actually my eclipse editor has a 4-spaces setting. It's changed to 8-spaces by accident when I remove the trailing whitespace long long ago with Vim editor. The style of code that is edited later is consistent with 4-spaces. I will fix it. Thanks John.
self.results should be just results, here and below as in "results[key] = self.permanent[key]"
Может стоит переименовать опцию `verbose` в `verbosity` или `log-level`?
Ditto for ornament.
useless doc line.
This should be a `tuple`.
looks like this is the only real change here :P
Please change this docstring to a comment.
What if tag_name->length != 1?
I think so, see this line on jQuery repository

https://github.com/jquery/jquery/blob/master/src/deferred.js#L53

```javascript
promise.pipe = promise.then;
```
I agree that it's not worth spending much time on this right now, but as I said, proper collation support is easy. The only "difficulty" is passing the result of PG_GET_COLLATION() into the comparison function.
It isn't really obvious to me how this is a fix for the issues mentioned, perhaps add some more details?
No need to check for null at this point since you've already called =~() on the key above.
Ah, the second token should be {name of course} could you please change the token? Thank you Dirk!
If you're doing high enough volume through this transport for a growing redis to be a problem, you probably need some kind of monitoring to pick this kind of thing up anyway.
Same as above, this description should be more... descriptive especially since this is an abstract class and there is no mention of that anywhere
in der config.inc zeigt $this auf die rex_addon instanz des addons selbst.

Daher kannst du hier statt

```
rex_config::get('debug','firephp_maxdepth')
```

```
$this->getConfig('firephp_maxdepth')
```

schreiben.

Den Default kannst du übrigens als 2. parameter übergeben, falls an dieser stelle nicht zwingend die config vorhanden ist
I think it may be nice to make a `VWikiPageName` validator that runs the validity regex and normalizes the name.
s/Gets a specific or the current loaded language config./Returns the configuration object for a given language./
Used mapper so that we can pass extra parameter once the async dbus call has been performed.
Is there any reason to use `PipelineExtractor` when `len(queue) == 1`?  It's just unnecessary overhead.
Pretty sure I didn't write this.
I believe we also need to pass "response" to Exception.__init__()
Hi Kasper, I believe we should not include testFramework as part of universal node_module. Given that this module is "require"ed for production we don't want to load the code that is specific to testing.
Yes, I think we should; I like to use the JUnit tests themselves as examples of "the right way to do it".
Wrong icons
Personally Region threads running without their World's thread running doesn't make much sense to me.
And I can't find any place where SpoutWorld would need a running region thread.
It does seem a little weird to return the name here vs. something like the startTime. Reconsider?
Please use double quotes to match the rest of the file.
Good catch! Nit: there are multiple empty lines before this one.
You are right, but applies to English root locale. See here: https://github.com/adobe/brackets/blob/4d972353589e8ed5d2eebc82640389b749b4dda8/src/nls/root/strings.js#L68
If we ever use the hash, this won't work. How about ``window.location.path``?
Comment should say appReady
Ah, I see that you're positioning it dynamically. Nevermind. :)
i was curious about struct alignment/padding issues since we're persisting this to disk and this made me :)
Hm, `function` may be chanko's `function`. So we can keep it. (but certainly it's confusing...)
Generally speaking, unrelated changes like this in a PR are bad. Not sure if @hone cares or not, just a thought.
gah, ^
Maybe use guava's Objects.hashCode() - for this and the other hashcode impls
The onclick is javascript, right? Can we move that to the assets/javascripts?
Is this really correct for the plural form of repositories?

Just want to double check.
Also, "asyncTest", "Sync API"? ... lol!
Can you make an `if` block out of this line too?
I don't think this will work. Here's what my console says:

`cues`:
```
[
Object
command: "media-editor"
start: 7
__proto__: Object
, 
Object
command: "plugin-list"
start: 14
__proto__: Object
]
```
`extObj`:
```
Object
butter: ButterInit
command: "media-editor"
start: 7
__proto__: Object
```

`start` was overwritten. I also don't really love the dependency on Popcorn API here. How else can this be accomplished?
why the space after <code>function</code> here?
spacing, add a space before "dblclick"
This is a view object, needs .id.
Just for you! ;)
I'd like parser to be setup here instead.
Again, these 3 instance variables were just set in the prior step, so why overwrite them?
Hmmm, although it's used again here and having that ternary operator twice is not ideal either. I'm agnostic... you can leave it, change it to what I suggested in both places or implement another solution if you think of something,
probably should make this a constant... looks like a magic number to me ;)
Tohle a následující nelze bez přerovnání přeložit tak aby to neznělo divně.
Also, it's worth creating a test that does a post for video that has no information in it.

    def test_post_video_no_data(self):
        cat = category(save=True)

        data = {}

        resp = self.auth_post('/api/v1/video/', json.dumps(data),
                                content_type='application/json')
        eq_(resp.status_code, 201)

        # Get the created video
        resp = self.auth_get(resp['Location'], {'format': 'json'})
        eq_(resp.status_code, 200)

I'm not sure what that does offhand, but it'd be good to know what it does when it gets no data. Does it shoe-horn a video in? Does it go to defaults? Does it kick back an error? (Hopefully the latter.)
Missing parens :)
Since the edit action still exists on the password_resets_controller, shouldn't we have updated the test to account for the changes vs removing it?
I am curious about the question eric asked.  Given that a template
exported to a manifest only contains a package name, is seems like the
only purpose of specifying a NVREA would be for promotion.  Correct?

The generated image (from the exported template) could under most
situation have a different version than what the user supplied.  Which
seems bad.

-Justin

On Fri, Jul 20, 2012 at 10:48 AM, Justin Sherrill <jlsherrill@gmail.com> wrote:
> I'm not a huge fan of checking at promotion time only, just because
> you could potentially have a failed promotion at that time, and failed
> promotions are bad (can't roll them back, etc..).
>
> On Fri, Jul 20, 2012 at 10:41 AM, Ivan Necas
> <reply@reply.github.com>
> wrote:
>>> @@ -205,20 +205,19 @@ def export_as_tdl
>>>
>>>
>>>    def add_package package_name
>>> -    if pack_attrs = Katello::PackageUtils.parse_nvrea_nvre(package_name)
>>> -      self.packages.create!(:package_name => pack_attrs[:name], :version => pack_attrs[:version], :release => pack_attrs[:release], :epoch => pack_attrs[:epoch], :arch => pack_attrs[:arch])
>>> -    else
>>> +    package = Resources::Pulp::Package.name_search(package_name)
>>
>> I'm not talking about checking that in the promotion phase. And that should also be the time, when checking the package specified in the template, can be found in the environment. What I'm suggesting is, in this place, use only
>> ```
>> self.packages.create!(:package_name => package_name)
>> ````
>> without any conditions, and do the checking when promoting (or alternatively when adding the template into a changeset)
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/Katello/katello/pull/335/files#r1205772
Just curious. In Fedora 17 is default tomcat7. Version 6 is awailable as well. But we will have to migrate to tomcat7 one day. What will happen to this script? It is marked as 'always' so it will be run even when tomcat6 will not be available.
I'm just thinking loudly.
We will have to edit this script instead of creating new. With something like
```
if [ -x /lib/systemd/system/tomcat7.service ]; then
  service-wait tomcat7 start
else  [ -x /etc/init.d/tomcat6 -o -x /lib/systemd/system/tomcat6.service ];
 service-wait tomcat6 start
fi
```
OK. Seems good.
The license header should not mention you but the copyright of the library. Authorship is kept by the ``@author`` annotation on the class.

The license header would become a huge mess if we were adding the name of each guy modifying the file in them.
again, cool down: since looking into `Monolog`'s code is not my primary job, I **forgot** you can use `handle` before `write`
Strictly speaking, a launch path should never contain "http://" or "https://", it is just a path inside the origin the app comes from https://developer.mozilla.org/en/Apps/Manifest#Definition_of_the_fields

I understand this is somewhat tricky at the moment given that the app manifests aren't served from their real origins, but are we happy with this approach? This should at least have a comment to say that it needs changing.

Also, it would be nice to add comments to the manifest files which use http:// in their launch path to say this should be changed, or find another solution. We could temporarily add a field to the app manifests called origin which we hard code for now, but which will eventually be automatically detected?
the intentation is a bit wacky around here - I think you've mixed tabs and spaces.  Please could you fix?
Interesting point. Is the result of `u.cm ** -2` a `Unit` or `Quantity`? In the test in the line above, are the resulting values `Unit`s or `Quantity`s?
I tend to prefer to pass some meaningful values to fromlist.  Though as you discovered it's not strictly necessary--just a wart in the import system.
All this does is swap a `.` for a `_`. Not needed.
I went by the guide here:
http://ruby.railstutorial.org/chapters/following-users#sec-following ,
which I assumed was a pretty definitive source.

Would this not return true/false appropriately if it were to find a
Relationship?

*Nikhil Goel* | Vanderbilt University, 2013 | Computer Engineering,
Economics**

LinkedIn <http://www.linkedin.com/in/ngoel1> |* ngoel36@gmail.com | ( (256)
503-9183 | NikhilGoel.me <http://nikhilgoel.me/>



On Mon, Nov 19, 2012 at 2:18 PM, Luke Steensen <notifications@github.com>wrote:

> In app/models/user.rb:
>
> > +
> > +  has_many :favorites
> > +  has_many :favorited_videos, :through => :favorites, :source => :video
> > +
> > +  has_many :relationships, foreign_key: "follower_id", dependent: :destroy
> > +  has_many :followed_users, through: :relationships, source: :followed
> > +
> > +  has_many :reverse_relationships, foreign_key: "followed_id", class_name: "Relationship", dependent: :destroy
> > +  has_many :followers, through: :reverse_relationships, source: :follower
> > +
> > +  has_many :bookmarks
> > +  has_many :bookmarked_videos, :through => :bookmarks, :source => :video
> > +
> > +
> > +  def following?(other_user)
> > +    relationships.find_by_followed_id(other_user.id)
>
> Methods ending in a question mark should return true or false, not a
> Relationship.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bitcast/bitcast/pull/11/files#r2172131>.
>
>
I would leave this as is, because Bitcoin-Qt is a Bitcoin client, so either Bitcoin-Qt or leave Bitcoin client, which is fine here IMHO.
I think you should do this:

```
layer.mergeNewParams({floor: floor || null});
```
I prefer to statically allocate memory rather than dynamically for performance.  You can declare an array of greenify_watcher statically and just check if nfds exceeds the boundary.
i'm not an expert in JS, but would 
```javascript
i = this.get_selected_worksheet_index() || 0
```
?
Are you sure about this? I expect in Python 3, line will already be `str`, so decoding it again will fail.
Guava 14 will introduce BaseEncoding.base64Url which addresses this.
s/detected/detect/

Or maybe:

    def test_does_not_detect_bad_code_in_a_valid_file
I know this is old code, I'm just curious why it matters if this is in the footer?
oh cool.  thanks!  done.
remove the `c = `
I can explain more about why if you like but I understand that you're under pressure right now.
How is this test different from `test_generate_signature_10`? All I can see is that it's using a more "realistic" input.
"format" is not a very good name for this variable.
No its not It is good practice though to prevents new comers from doing common mistakes like accidentally creating a global. I add some comments in there about it
Commented line to remove
This change looks suspicious.  True, there is y0+y1 in the denominator of the else-clause, but I don't see any physical representation for abs(y0+y1).  I wonder if the equation itself is wrong?  I can't remember where I got that algorithm from...
Just intend with one tab for this stuff.
Put this in it's own file. One class per file
The most common needed. I guess i was seeing an oppertunity to have less BC
Why not `@expectedException PHPUnit_Framework_Error_Warning`?
Funny, never saw that before :) Fixed.
@sstok no, it's an array in any case.
`!isset($options['database'], $options['collection'])` ?
do you really need to log each call to the cache ? It looks like it will be spamming logs
Sure it can be generated here. The salt will be in the first 29 characters of the returned value, and the actual encoding of the password in the next 31 characters, giving a total of 60 characters in the returned string. (See php's documentation of the function crypt)
how about `str_replace(' ', '-', ucwords(str_replace('-', ' ', $name)))`?
why two lines here?

`#pragma mark - WEPopoverControllerDelegate implementation` should be enough?
Isn't the naming bit confusing? Either use "stdout" & "stderr", or "output" and "error_output" or something.
感觉硕士和博士好像没有分太清楚，这个问题好解决么？
isCover这个名字和方法是否开放，或者是否发布可以再讨论下。
Why the ```(as)```? It is more readable without?
this comment is weird
I don't think this should be here
So pep8 recommends for each work to be capitalized... this should probably be FilterRule then.
...especially when they are not familiar with Git
So then, how comes that they are familiar with javascripts and other programming stuff at all?
Why do these components need a view, rather than just using the component instance directly?
Does this makes sense? Checking for an array and casting it to an object?
I did :) That's the reason I'm pointing out this. Try adding fields one by one, and retrying to scroll on each one. There is a size where you should be able to scroll and you can't.
We could not only use absolute position for container because it already apply float to set the relative position. If you want to set absolute position, you have to apply both input and button and set absolute position position for them. I think using the container div to wrap the button and only set position for button would keep the chances as less as possible.
Your line wrapping is extra weird here.  Use emacs and meta-q, man! :)
Quick question: Why is 11000?
Ideallly GAIA_DOMAIN should point to a fake domain by default but I don't want to do that now otherwise all the phones distributed at the JSConf in Arizona will not work with Gaia.
If holdend events are all you need the gesture detector for, can you remove the gesture detector and use contextmenu events instead?  I think that touch and hold generates a contextmenu event by default.  I know the homescreen uses it, and I suspect you could use it here.
Maybe one more thing I forgot to explain: It's possible that 'about' is also top ranked for the prefix 'a', 'ab' and therefore also ends up in the word list for that node.
I know we might need to update the ThreadListUI after delete in message thread, but is it necessary  to put the ThreadListUI.renderThreads in the ThreadUI.renderMessages callback? Do you want to update ThreadUI at first on purpose?
I went for "if (telephony)..." :-p
Add bug related! ;)
I am not sure we need this, but let's do this just to be on the safe side.
please unprefix all `calc()`
Can we reuse the entryCountDonde variable from above? Just to save the second querySelector.

(just asking :) )
Thank you for removing useless `en-US` locales property (since the default text is specified as in en-US)
This is my mistake, I never pushed a change set this will break calendar etag handling.

Can you please fetch from master (in caldav) and re-copy paste the file over I pushed the fix here correctly upstream.
Nit: full uppercase please (I correct a lot of those…)
don't you want to add `#modal-dialog-custom-prompt` to

https://github.com/ochameau/gaia/blob/fda8e3cffc4e360d3f85731c91dee714c1dc8f09/apps/browser/style/modal_dialog.css#L26

and

https://github.com/ochameau/gaia/blob/fda8e3cffc4e360d3f85731c91dee714c1dc8f09/apps/browser/style/modal_dialog.css#L130

?
Nit: this line is too long, please split it
azure is the name of the script the generator is part of (azure-cli), I could rename it to GeneratorScriptPath if it makes more sense...
updated.  Thanks.
以下に示すでは

以下のイラストでは
I'm not happy with `didPersistRecord` either :/
I guess you don't if the test passes (but I guess you are checking the isEqualIffZero case rarely for these random tests.
I'll add another test for these.  I am really short on time right now so it may take a bit.
I think you should set errno before returning -1.
Make this a class constant instead of a global.
Improve this description.
Are the explicit closes necessary? hsFileStream is RAII, right?
thread in thread?
Alright.
Is there an elegant way to make a getXForView function that also takes e.g. Errors and Infos as parameters, rather than having two separate functions that do essentially the same thing?
The above change was in response to Marijn's pointing out that using offsetWidth isn't robust here because it will be zero if the editor is hidden on init. He also wanted all feature detection to happen on script load rather than instance init.
Heck if I know. I'll fix that.
Nope; tests show that it works fine without it.
Yeah that was the problem. I pulled from gles20 recently and there were a few DOS linebreaks. Let me dig out the commit...
you should add the curly braces (and add an empty line before the next part):

```php
<?php
if ($this->identifiers) {
    return $this->identifiers;
}

$this->identifiers = $this->importNodesFromURL(self::REGISTRY, self::EXPRESSION);

return $this->identifiers;
```
Can this method be private or is this something that someone want to use externally?
Geoffrey has added slf4j, so best to log errors there now, instead of System.out.println
If I am not mistaken, MVEL uses ~= as its matches operator. The way this grammar is written, it might make it impossible to parse that operator in expressions.
Adds support for not found
Why is this method asynchronous? there is nothing I/O dependent here. I'd use return instead of callback(message).
>> Should I remove /dev/null ?
yes, please
Playing around with HTMLUnit a bit (http://code.google.com/p/selenium/issues/detail?id=141) let me set the request headers ok.

https://gist.github.com/3246120

Need pointer on if/how to integrate this with the existing test code.
the /(?<![0-9])/ is a negative look-behind saying "There should be no digit immediately before the IP address"

Your change means "there must be a /<[0-9]/ match ahead of this position"

I'm reasonably certain the current expression is correct, since it prevents '1234.1.1.1' being matched as an IP because '234' is preceeded by a '1'. 

Can you help me understand your change? What was not working before? What works after this change?
Is this just to have it as an option instead of putting in the logstash config file?
add comments why we use these methods
I think `next` is a better name here
I think `isLoaded` should work just fine as well.
Sigh. Ya, will update.
again orthogonal, a step towards fixing the debug locale in our dev environment
just leftover from when calls were synchronous.
@salmonz - These commands are Debian/Ubuntu specific but BrowserID is meant to run on any Linux variant or OSX.  Perhaps put this under a Debian based sub-section?
all the types get wrapped in DotDict on line 68 at then end of the method.  doing here would be doing it twice.
More idiomatic modern Python here would be:

```
with open(file) as f:
    json_data = f.read()
```

It also is safer, in that it will close the file even if an exception is raised in the body of the `with` statement; something you'd have to use a try/finally block to emulate otherwise.
what's the point in attributing the object to a new variable(search_results_page) if this is not being used anywhere in the test?
Makes sense! Thanks. Looks good to me.
Isn't the first check now duplicate since if the last one matches, the first one has to match too by definition?
https://github.com/rubygems/rubygems/pull/353
There were a few places that needed them untainted, it seemed easier to untaint them here - could move it back if that's what you want.
I have limited understanding of sbt (Scala newbie :)), but where are these versions being picked up from?
Actually, it is not. I think if/else is readable, switch in case of 3 items is not better. Additionally, I would need to check for something else than string value.
Removal of these is a BC break that I couldn't yet fix (I think I will simply enforce a service with name `Di` in the factory itself). The two rows are commented because this is a cyclic dependency. I might solve it, as said. That way, the functionality would be fully restored.
Please use seperate cleanup commits for such changes in the future.
We could also switch to SAX parsing with apr_xml_parser_feed + apr_xml_parser_done. No allocation - on-stack, heap or otherwise - required in m-a-c.
Is there a URL for the latest version?
PhpStorm 3 was using use statements to generate the phpdoc but not to parse it. But this has been fixed in PhpStorm 4 which does not have any issue with relative class names in the phpdoc
Kannst Du mir auch erklären warum? Intuitiv hätte ich einen Response mit Statuscode 304 zurückgegeben und diesen dann im Client verarbeitet.
I don't think we need an exact match if it makes no sense.  @zidane can comment.

Does this also include creating portals in other worlds?
@samiluukkonen Tässä olisi tärkeää tarkistaa, että sarjanumero on vielä ns. 'vapaa'. Eli myyntirivitunnus = 0. Jos useampi extranettaaja ostelee samoja tuotteita niin käy muuten aika huonosti jos molemmat valitsevat drop-downista saman sarjanumeron.
Remaining todo
use `@arg` here, not `@type` 
also line too long
two empty lines  before class definitions
General remark. And a very annoying one too. We should pick one style for interpolated strings. Either " " or ' '.
There should be a package docstring...
This one seems consistent with others we have.
fix typo: "initialized" (or do it)
Nit: Should be separated by two empty lines?
I'm surprised that this doesn't break tests... It looks like it should.
remove all the files from the results folder
Я так понял — тут приведен пример, поскольку эта ошибка отличается от контактового Validation Error именем поля? Тогда надо про это написать.
Replaced by tests in `TestDMEnd2End`
Also, this test is skipped entirely, so all my changes may have been for not.
After the monkey patching in gevent occurs, it's legal to keep using `time.sleep`, it will automatically use the gevent version.
(www.) escaped to (www\.) after helpful comment from Naomi!
I'm not sure about this one. Before the patch, the test where we passed the 'Digest' instead of 'Basic' would have passed, but this one would not pass and throw a 500 because we tried create a buffer out of parts[1] which is undefined since the split returns an array of one element. So I would keep that test in.

Thoughts?
replaced this by a whole new content .
i need to double check that this works across all compute resources
@ohadlevy , this added to the domains_controller by @jpoppe
https://github.com/theforeman/foreman/pull/200
When it should do

```vim
let old_wildignore = &wildignore
set wildignore=
so `=file`
let &wildignore = old_wildignore
```

I think this is only way to avoid fnameescape(), and annoyed problem about escaping filenames.
No indeed. Made them final.
not info-level
Make final what you don't intend to mutate.
My general idea had been to be able to declare the "locations" of icons within the sprite separately from defining semantic, full CSS defined icon blocks that would be included into the views.  I sort of tried to merge the two in a way that makes icons stand-alone entities you include when you need one (as opposed to including the "image" inside some other CSS/HTML entity).  Which is why the switch to declaring the icons as classes with a background position and the auto-selectors.  Thoughts on this approach?
I don't know what correct LaTeX is, but sometimes there doesn't seem to be enough space between the \angle symbol and the letters. Is there a correct way to do this, or could you manually add more space?
The exercises should not require knowledge from watching the videos, I was told.
In the original pull request, #304, it was set to 1 because of the weird behaviour in safari 3 and down. So I assume it needs to be there for that reason.
Accessing `$_SERVER['PHP_SELF']` directly twice is inefficient. Please cache the result so as not to incur unnecessary dereferencing overhead:

```
$self = $_SERVER['PHP_SELF'];
$me = $self;
```
The row of centroids is the index for the centrroid.  clusterAssessment stores this in column 0.  Check the latest commit where I put up an ASCII diagram to show how the matrices relate.  I probably still have some FiltCols lying around that need to be cleaned up.
I saw this but did not want to change anything there. if you would like i can  also fix this indention
сделай отдельный класс, наследованный от графа тогда. Разделять реализацию --- это похоже на бред
You can do this: `@browser.query("link[href]").getAttribute("href")` (I think)

I'd rather we not use the private API ;) it might change, but the public API will not.
unless python does inheritance in a way that does not match my expectations, i think that while this block may make the code more clear to readers, it does not actually change anything when running. self.SearchResultItem was returning the version of SearchResultItem located in the class that self is.

on second glance i thought you made this more specific in order to be able to do the isinstance calls in click_result, but those could have been done with CollectionSearchResultList.SearchResultItem just as easily as with CollectionSearchResultList.CollectionsSearchResultItem
Whoops, good call. My bad!
If it is default, leave it as it is. No need to change.
This doesn't seem quite accurate.
Since creating replies is now going through the same endpoints and API functions as creating comments this is already tested elsewhere. I'll just remove this test.
This should be changed to groupId, to be consistent with userId and contentId.
Should be inline with the documentation above.
I'm not entirely sure if we should return an error here. That would result in a lot of code where'd you have to do something like:

```javascript
var Config = require('oae-config').config('myModule');
...
var someConfigValue = Config.getValue("foobar");
if (someConfigValue.code && someConfigValue.code === 400) {
     // An error occurred
     ...
}

// Deal with the value
```

As the validation only checks for  a non-empty feature and element maybe we could just return `null` ?
I realize this goes against most of the semantics against validation and error handling but it should result in cleaner code.

ie:
```javascript
var Config = require('oae-config').config('myModule');
..
var someConfigValue = Config.getValue("foobar");
if (!someConfigValue) {
     // An error occurred
     ...
}

// Deal with the value
```
In the future I'd like to be able to define other data types that aren't Objects. And so there's some confusion over what to do if an object isn't a direct instance of Object. Is it a non-JSON data type e.g. function, or a different class, e.g. UUID or ObjectId, or a binary string (Node.js Buffer or UInt8Array)? Or, is it strictly a data type that is a JSON object (maybe it's a datatype that was cloned with e.g. `var childInstance=Object.create(parentInstance)`)?

If the object was created with `new` it should have a different `constructor`. This can be tested with `instance.constructor==Object`. However this fails if the object defines a "constructor" property.

Maybe:

    instance && (typeof instance.constructor!='function' || instance.constructor==Object)

But this still fails if later I were to try and validate an ECMAScript instance {constructor: function(){}} against a schema where 'function' is a legal type.
haha, ok :)
My commit fixes the memory leak by 0x425949: htsbuf_append (htsbuf.c:110)... not sure about the later ones... I was testing with two connected clients and with switching between h264 and mpeg2 chennels for about 5 minutes. I'll make another test tomorrow asap to se if your streaming improvements fix the same leak as my commit.
Wick.
Won't behave properly:

    php > $str = array();
    php > var_dump(($str == ''));
    bool(false)

... and should be checked against `$units` anyway.
А можно подробнее?
Я ничего не понял.
Возможно это метод для отладки? Согласитесь было бы странно, если бы у класса NSNumber был бы метод printNums и при вызове
[[NSNumber initWithInt:1] printNums: [NSNumber initWithInt:2]] выводил бы:
Первый инт: 1
Вторй инт: 2
Что будет если [elements objectAtIndex:randomElementIndex] совпадет со старым элементом? То практической замены не будет. 
Поэтому по теории вероятности ДНК мутирует на 0.75*x  процентов.
Секунд 30 тупил, пытаясь понять какой блок начинает int. Вот что значит сила привычки
И плюс — называть переменную count в данном случае — не очень хорошая идея, так как в массивах есть метод count, и это вызывает путанницу
should be: 
    <unzip dest="${forge.dir}">

Noticed after saw jenkins failind to go into forge dir, since it didn't unpack it in there, may need to create dir first.
Sure I'll  update the path accordingly.

I understand the concern with the RNG.  But like you say we want to catch template errors before build time, we need to make sure we are in sync with factory at all times.

One way to address this would be to ask the factory guys if they can expose the rng as a resource.  We then could then request the RNG on boot.
Yeah, this can be used on all of our tests, a nice little spec helper for another PR
The naming here doesn't make it especially clear how the dictionary is organized. How about `argIndexesBySubscribable`?
OK let's take a step back.

So the problem this is trying to solve is what? Waiting, blocking, for a value until we get one. That value could arrive on a different thread or it could arrive on this thread. It could arrive within this runloop or it could arrive after some delay.

The reality is that in our particular use (*ahem*), we won't deadlock since we always call `-first` with the immediate scheduler. But it seems Pretty Bad® to deadlock if it *is* called on some other scheduler. I'd much rather it spin the runloop than deadlock.

Maybe we should split the actual subscribe on to another queue?
Please add jsdom to the dependency section in the documentation above.

Also, not many things use jsdom, so I'd recommend perhaps using xml2js or cheerio for scraping the results out instead.
I would append something like “ – META-SHARE” at the end of the title as this is common practice.
Very nice we now have this. We can use it for working on DICS or other frequency / time frequency beamformers.
ok now i get the plot False

so you should have a private function to compute tfr_data and extent and do not expose the show param to the user. makes sense?
> I've always heard (and worked with) 5 cycles. That gets my vote since we're just throwing a warning anyway (and why not warn people more conservatively), but I won't complain about 3.

I guess it depends on what frequency. 5 for low freqs is a lot.
Usually epochs are too short.
But I agree for beta 5 is good and for gamma I would go up to 10.
Committed changes, see if you like them.
oups I had not seen this. Maybe you can avoid the inheritance from dict and raise warning that accessing attribute like a dict will be deprecated in v0.6
nit: is the indentation correct?
Right. I was suggesting making it monadic, but what you're saying makes sense given that only routeTop and routeName use it.
设置授权工作进程使用的非授权用户名称。安装完成后，该名称

设置工作进程以某非特权用户的身份运行。安装完成后，该用户名
This shouldn't be necessary. Engineer includes a dependency on the [path.py](http://pypi.python.org/pypi/path.py/2.4.1) module which wraps the os.path module in some syntactic sugar, including overloading the `/` operator. Since most, if not all, path-handling code in Engineer uses that module, I'd prefer to stay consistent.
Needs extra indent.
> Why not let S handle this

I tried it; test_lpoly.py runs 2% slower
Currently you are creating the Settings object as an extension, which makes it scoped to the whole ActorSystem. I think in this case (with user/passname info etc.) you might want to scope it to the actual mailbox instead, i.e. not make it an extension but create it in the MailboxType constructor; that is why the specific config subtree is passed as a constructor argument.

And yes, it would make sense to look into the other durable mailbox types as well whether this change is applicable to them also. (ticket created)
Gracias, mañana lo arreglo.
ok i check and this meta is deprecated :) http://www.w3.org/TR/2011/WD-html-markup-20110113/meta.http-equiv.content-language.html
Instead of `@stock[food] || 0`, I recommend replacing with

```
@stock.fetch(food, 0)
```
revert this too; i'll publish `wav` asap
Do we want to keep this shebang line here in sankey?
Enable validation on incoming files
Yes. First commit in months and I'm making a fool of myself. :-(
Basically we need to explicitly declare the 'url' property only to change item's styling: without the URL it will look like the menu subheader (currently boldfaced pale gray).
can we not name this rsdns? Seems specific to rax
Hmmm, why does Interpreter get a ctor parameter for cache manager? In the current arch (which I'm not saying is the best), this kind of components should be received in @Inject methods.
It's no longer a wheel because it does not warp at 0, ie segments can start or end at 0 but never have 0 in the middle. So we should think of the hash space as linear rather than a wheel.

Regarding number or segments, some (future) implementation (but not current one) could round up the number for example to closest power of 2 because that enables more efficient computations based on bitwise operations. Users should think of numSegments as an indication rather than a strict requirement.
well spotted 
Here is how I'm thinking to solve this: 
- I don't think there's any point for the state transfer to migrate the local transactions that didn't leave this node for either prepare or lock acquisition. Why? because the tx state will be send across  anyway when the transaction will either lock something remotely or it will prepare remotely, so sending the tx state before that is redundant
- the LocalTransaction keeps a list with all the remote nodes where this transaction was sent: LocalTransaction.getRemoteLocksAcquired() (name might be a bit misleading) 
- modify the StateProviderImpl to only send a local transaction as part of the state transfer IFF the transaction left the node, i.e. LocalTransaction.getRemoteLocksAcquired().isEmpty() == false
I went through the ML thread and I'm fine with this. Testing...
I propose `roflims` for the next one...
So how do you know this isn't used? It could be that upstream users are using it...
I agree with @WeatherGod here, xsteps==1 (and similar for y) is better here.
As @efiring pointed out, trailing backslashes can be alleviated by being a little more proactive. Perhaps create a temporary variable to hold the value of `fontsize/text2path.FONT_SCALE`?
@marshall indeed. This is because gaia first put stuff in $(TARGET_OUT)/b2g/webapps, so we must keep it before wiping $(TARGET_OUT)/b2g
as much as you love the EmptyQTool do you think we should keep it around?
я немного не про то. Код, который внутри eval можно написать прямо в код (просто убрать eval). Наверное это остатки старого, может там раньше какие-то переменные подставлялись
I will remove it but why? You directly see if everything worked as it should.
Now I remember why I ditched the document.createElement override. It's because I changed the detection to only call document.createElement once, and keep the style object for all future tests. Since the stylePrefix method re-uses jsPrefix with a style object, it would have to do document.createElement for every call. Or I have to make the jsPrefix more complicated somehow to avoid this. Either way I think this change will make the code worse instead of better. What do you think? Do you see any alternative solutions? How about renaming _mockStyle to setStyleTestElement and take an HTMLElement instead of a style object?
don't do multimethods that dispatch on class, use protocols
@arnodeclercq please use single quotes in this file
Why do these 3 lines need to be repeated?
Why are you mixing "_" delimitation with camel case?
you might want to speed this up by getting the whole int label vector in best_map and only handing it into this function. this might help as get_in_label will do array bounds checking, conversion double -> int and testing if label is really an integer