labofsahil/github-event-dataset-2012 huggingface dataset:
"with no luck" is very colloquial and may be confusing to non-native speakers. Might be clearer:

"[WARN] Analytics: %d transmission attempts failed."
I'm not saying that we NEED it here, but I just think it's good practice to check for it when possible.  I know you are passing constants here directly, but there is nothing preventing someone else who is not as familiar with the code from accidentally doing this.   I don't see any drawbacks doing that here of getting a free null check when no extra code is needed.
Where is the memory leak? i dont see it. I copy children in an autorelease array to make sure i wont do something wrong  (like removing while in a loop)
Default behavior when `backgroundImage` is unspecified depends on the type of view and the platform. For generic views, no image is used. For most controls (buttons, text fields, and so on), platform-specific default images are used.
era um negócio que eu já tinha visto, não faltam as aspas ali?!
Why are you using the showEvent to call the backend?
No problem.  I wasn't aware that was the case, fixed now.
Oh sorry. It works with the current master! Thanks.
> Why not just give an example?

How is that not an example. If you have an object which owns an object, and it has a function which relinquishes ownership of that object to the caller and returns it, then if `destruct` takes `ref`, then you'd have to assign it to a variable first just to destroy it. e.g.

    auto obj = owner.relinquishOwnership(key);
    destruct(obj);

instead of

    destruct(owner.relinquishOwnership(key));

Regardless, unless specifically requiring an lvalue buys you something (and I don't see why it would), then I see no reason to restrict the function to lvalues. Just use `auto ref`.
`else static if (is(typeof(range[1 .. $] = range.front)))`
> If int is equivalent to any kind of static array, it's int[1], not a zero-dimensional array.

Why? Looks like you just don't want to accept zero-dimensional arrays and all.

> as I disagree that it should accept anything other than static arrays, but I don't see how you can argue that MultidimStaticArrayElementType should result in anything other than void when given a scalar.

Do you hear me? These templates are **practical**. Its behavior is needed as it is anyway. More than that I see no applications for a behavior you propose.
yes, good point.

I changed it here and in other places where it was obvious.
yes, the default version for the test objects which are not in the OSGi environment
If these two options only end up being set on the ODM's Configuration object, why both specifying them on the connection block of the config structure? We can simply define them within `Configuration::addDocumentManagersSection()`, no?
Fair. I just mean we shouldn't restrict it to "assignment."
That's fine. I just wasn't sure if the higher number is the higher priority (lower numbers are higher priority with the lottery), and I'm too lazy to look into the code to find out if this is the case (or if the code currently doesn't make a distinction between higher and lower priorities).
amended owrap command
This is likely too long to fit on the dashboard layout, probably want to keep it as Loc:
Awesome. Fixed.
Could we, per @christianv's suggestion above, pass in an argument to a single function and use the `.toggle()` and `.toggleClass()` functions to handle the toggling?
A lot of this code appears to be the same as the htmlblock widget -- can we combine the common pieces so we don't have to maintain the same code in two places?
Can you remove the `px`?
yeah, evidence that it once existed in the mgmt command...  :) fixed.
Not necessarily, removed it.
Try inserting some XSS strings in and see what happens. That will lead you down the right path to solve this.
Why so? "Library functions" are no different than any other piece of code – it's not in the API, just a local variable. Why not ask him to spell out the types out in the foreach below as well?
I think it's worth starting to change the style. Pretty clearly the old style's main motivation was avoiding compiler bugs.
@deadalnix: `cast(REG_TYPE) &sigsegv_userspace_handler + ((rip != addr) ? 0x4 : 0)`?
Now that channel can be used for passing the Message directly (when no 'expression' is provided), I think we should remove "result" from the variable and method names. That would be consistent with the attribute in the XML which is only named "channel" within each sub-element now. Of course, the JavaDoc (as mentioned in the comment above) is necessary to clarify the role of that channel in the 2 cases: 1) when there is an expression and the result of its evaluation becomes a payload and 2) no expression is set so the original Message is sent to the channel as-is.
Yeah, this is the simplest way (and the standard way, based on what I've seen).
The correct type is `txt`
Thanks. I'll update the PR accordingly :-)
at what version?
Actually--how did you end up with a non-string value for your resource?  Is this possible when using this type/provider via manifests or only when using it programatically?
I will document once the PR is ready ;)
This should be removed. It is meant to hide the collector in the toolbar if its empty. E.g. hiding the database collector if no data base interaction occurred during the request.
"Copyright 2012 Cesium Contributors"
Here it should use the same format as the request that generated this call and not hardcode it to html.

@merk @asm89 the format is directly related to the way the client side scripts are going to handle it. So, when some other scripts need the data in xml or json it will need to explicity say so.
You shouldn't be overriding the source
Ok, I agree, I'll change it.
why is x an Exp and not a Sym?
самодокументированности кода - всмысле, что код должен быть понятен и без коментариев. Коментарии только для не очевидных штук (алгоритмы, датафлоу, пояснения к аргументам функции итп).

Совсем не посмотрел на имена функций - все они тесты. Оставь как есть. Не заморачивайся :)
Extra spaces between variable name and `=`.
midoonin chon har kodom az image ha 7px fasele beyneshoone bara hamin nemishe image akhari radius bokhore chon tamame image az slider kharej nemishe
yep `transition` and `transform` are all good for b2g
take off the constant thing again
I wish I'd knew what that method actually does. All impl return nil except IPadSplitWindowProxy.
the google style guide is to not use "withFoo" other than for the first param (WithTargetDir). Just do: rename:(BOOL)rename
With `reduce` this would be simply:
~~~clojure
(reduce toggle #{} a-seq)
~~~
How is that "others"?
from here to line 59:

what about using [`glob()`](http://php.net/manual/en/function.glob.php) instead of opendir and various checks (like extension, etc.)?
What was this? Old method that sent all $data parameters separatelly, and from now it serialises and sends only in one parameter `q`?
Ok, please do so. IMO, these MBean retry calls look hacky and look to me that they're just a hack for the real problem, which is that AS7 is not being deterministic in exposing things.
This test fails because the PackageValidator assumes every Package is an import, is that really correct?
use this class in an auto-wired test.
is this whole class empty now?
I don't think this should be phrased as a recommendation to do, but rather a possible suggestion. And you definitely have to get consent from the hackers signing up. Be clear about what info will appear where online. Some hackers don't want their info to be made public, or e.g. are only okay with their twitter handle online, but not their email address.
Don't quite follow this, why would you replace this with the gwbe namespace? Makes sense if you're going to just roll the BDE code into the GW base package, but in this case, it would remain part of the managed package itself-