labofsahil/github-event-dataset-2012 huggingface dataset:
What do you mean clears?  Does it create a visual artifact?  How often do we get a resize event where the width and height didn't change?  I suppose we added this for good reason, I'm just curious why.
This kind of comparison against fixed strings tends to be fragile over time.
This is actually intentional - it prevents inadvertently typing x = null (instead of x == null), which in some c varieties returns a boolean value, so that code works while performing.  This used to be a coding guidleines recommendation (but not requirement)
Name this ServiceBusNamespace
make the method synchronized itself.
This will resolve possible race condition when getJSValue called simultaneously from multiple threads, given that the return value is a single variable.
This way, you actually don't need codesnippets to be a stack, just a variable.
filename is undefined. (Functional test failed here)
missing a space between if and parentheses
Is it possible to refactor to reduce duplication?
Does `false` need to be added, I think its default.
We don't use defensive semicolons.
I will use a member variable for this. I need to save the state, so that the repository is not refreshed everytime the widget is shown
As was mentioned in pull request #29, this usage (using `>&` to redirect both stdout and stderr) only works under bash.  Unfortunately, the shebang line at the top of this script (`#!/bin/sh`) maps to different shells under different systems.  Under RHEL (and it's associated Linux distros) `/bin/sh` maps to `/bin/bash`, but under more recent Ubuntu distros (at least since 12.04) this maps into `/bin/dash`, which is an `ash` derivative, not a `bash` derivative.  As such, an error is thrown by any line that uses syntax similar to this (including line 16 in the new file, below).
Is this leaking?
hm. Simple CMS is the name of that bundle... maybe we should make this more explicit and say "configure SimpleCmsBundle"
we can check all the result on the page with these modifications
Being as generic as possible has its costs.
Yes it is, if I'm not missing something. The idea is simple: work as a compiler instead of a compiler. Just do everything it does but where we want it to be done.
Sorry, I don't understand why is Issue 8847 related to this.
Please use the VLCVideo "namespace" for these.
... and here + next 3 lines.
this will not work in a real case as the AnnotationReader validates the property name when reading an annotation and so will throw an exception if you try to set a default.
I omitted those on purpose when updating the CS 2 days ago, to avoid unnecessary conflicts when merging the PR deleting the validation code, meaning approximately all the changes you did outside the new files added by @schmittjoh today (which is logical as we used the same tool)
You're probably right, but I think that can be factored when we add the tabs.
Maybe `inputElementId` to keep with naming conventions?
Better use a runtime exception as assertion checking might be disabled at runtime. Perhaps an IllegalsStateException
oh cool.  I was struggling with this one.  this is great.
Shouldn't all these be configurable in some kind of config.js file? e.g. the topnavigation items ...
Nico indicated that `<button>` tags aren't draggable, which was the reason for this. They can be, if you use `{cancel:false}` in the draggable setup.

http://jsfiddle.net/eYRH7/
I'd made it public (not use internal at all). Maybe YandexApiSerializer?
So long as the ABI is fine... I was just saying that you won't get many people who would want to access this sort of data directly in D.
Hmm, this is `PBYTE  lpReserved2;` in the bindings from dsource.org. I'll have a look in windows.h and report back.
I am going to look at it
This assertion is always **false**. Why don't this one:
`throw new IllegalArgumentException("payload must be String or byte[]")`
?
And don't have `return` at all. Now it's a bit confused
Random `ALL_CAPS` variable?
What is the other command?
just for completeness: if we would use assetic, you would also want to ignore web/assets/ because assetic:dump renders js and css into that folder.
Right, my mistake. The code for adjusting the origin only has to be in one of the two methods, not both. The copy in preferredSizeOfLayer: is just left-over from some debugging I was trying to do in my app.
`form for user` -> `form for the user`
`this is done response is returned` -> `this is done the response is returned`
Someone could probably explain it better, but in the case of the following.  Ruby would look for Baz existing in Bar then Foo then "root (aka ::)".
    
    class Foo
      class Bar
        Baz
      end
    end
Sorry for the late reply, I somehow missed your comment. But we seem to have come to an agreement anyways :)
Any way to derive this `13`? If not, we should at least try to explain why it is 13 :-)
Alright changed and pushed. Actually I just noticed another oddity. Could you try starting a server, connecting, terminating the connection, connecting again and terminating again? I get odd duplicate messages for only one connection but it may just be another windows issue. Also they keep adding up so if I terminate the connection 3 times I get 3 messages instead of 2.


[SchedulerProtocolet|INFO|4:23:03]: Handler spawned for 127.0.0.1
[SchedulerProtocolet|INFO|4:23:03]: Handler spawned for 127.0.0.1
[SchedulerProtocolet|INFO|4:23:03]: Bootstrap message sent
[SchedulerProtocolet|INFO|4:23:03]: Bootstrap message sent
[SchedulerProtocolet|INFO|4:23:04]: Shutting down...
[SchedulerProtocolet|INFO|4:23:04]: Shutting down...
[SchedulerProtocolet|INFO|4:23:04]: Terminated
[SchedulerProtocolet|INFO|4:23:04]: Terminated
[SchedulerProtocolet|INFO|4:23:04]: Protocol successfully ended
[SchedulerProtocolet|INFO|4:23:04]: Protocol successfully ended
revisionflag is missing
Please indent lines 18-20
Nit: does it make sense to provide a default and then try to make it?  Maybe just test to see if you're passed an argument, test if it's a dir, and then load the ruby files from inside?
i agree with alex. i would leave this open ended. this requirement can go away. i want to see what people come up with.
counterproposal:

```<div role="banner">```
good point.  that would be simpler and more direct.  thanks.
verify a particular group was deleted.
Won't this clear any likes on a refresh? We still want localstorage to save the like array.
CACHE_PREFIX affects the whole project, so "askbot" is cleary not the right word
Use `$sn.confirmBox.dialogID` instead `#dialog`
Just replaced this for the actual position :D
ah ok, just wondered why it jumped from 5 to 20
It's not that I totally can't use them - i cannot compile via npanday, but i'm still using it to get all my dependencies together
Incorrect naming.

should be `private readonly AuthenticationService authenticationService;`
Sadly this function was written before our team worked on this project, and the amount and quality of commentary is insufficient to quicly fix this.
fixed in ff29e5775e8514b061215f4f0dde263a26a86124