labofsahil/github-event-dataset-2012 huggingface dataset:
I would just call the class "Backend" The "Abstract" is kind of redundant and doesn't add a lot of information.

Maybe the class should be more named even more specifically, e.g. "PackageBackend".
Lint doesn't like functions inside a loop.

I'm fine with creating a named function inside `Pool` constructor although I prefer this approach.
constraint?
Impossible. It is used two `emplace` overloads.
```d
ubyte[] buffer = [66, 0, 0, 0, 65, 200, 0, 0];

        enum Float: float
        {
            one = 32.0f,
            two = 25.0f
        }

        assert(buffer.peek!Float() == Float.one);
```
This might be a bad assumption... Preceding steps might add the datastream, which would now be wiped...  Keep track if it was _created_ in the ingest form?
This is really a per-server thing. You could store that per server, or you can store it per account and make sure you delete it when you get a new cluster assignment.
Fix the tabbing on this line.
This is never assigned. Good to remove.
i rather wonder why not make the method static?
Misses check on type, check on hashcode should be irrelevant.
I will change to that.
This seems inefficient since it retrieves all the users and then iterates them in O(N) time. Is there a way to pass criteria to NHibernate to have it do this in MySQL?
calling it `add_dicts` is fine
can you give `i` and `v` more obvious name so they show what they do?
Can you make this CSS file conform to the new style guide?
The value is an internal value of CP_UTF8.
I got it from MSDN[2]. Because I accorded with well known code page ID, I used it without thinking deeply.
According to the MSDN, it is "This parameter can be given the value of any code page that is installed or available in the system."[1]
Therefore, it is not a mistake to appoint with an integer value as a parameter of MultiByteToWideChar.

[1] http://msdn.microsoft.com/en-us/library/ms886760.aspx
[2] http://msdn.microsoft.com/en-us/library/dd317756%28VS.85%29.aspx
You never call pthread_attr_init to allocate the attr struct.
http://www.unix.com/man-page/freebsd/3/pthread_attr_get_np/
Although instead of doing Assert.fail for both of these, you should just let the exception propagate up.  You have "throws Exception" on the method, so you don't need a try/catch around the parse method at all.
I agree unless you envision them to have different attribute set in the future. At this point they are the same and it does remind me of Redis outbound where we have a very similar situation with 5 collection types supported by a single namespace element.
Gunnar, i actually agree with Artem.
I understand the desire to make it backward compatible but that is handled by the fact that you maintaining the method and its signature. What happens inside is entirely up to you and in order to stay consistent with how we do things in other components that utilize expressions I would go the path of getting a file path and creating LiteralExpression here so in the end you can always have expression to evaluate instead of relying on what is null and what is not.
The negative feels awkward.

Howabout 'usingDefaultTemplate' ??
looks good with the wording you modified when making the method name change
No; I am saying the existing attributes should work the way they always have. That's what we agreed. If someone wants to do the delete as part of the transaction, they have to NOT set the should delete to true.

Otherwise, this will break existing apps that have a transactional poller, and should delete/mark read.
this could use ``:class:``.
Instead of `!($blah =~ /regexp/)` you can write `$blah !~ /regexp`.
Quand j'affiche en local le résultat de l'affichage est à droite de "affiche :", ca serait peut être mieux de le mettre en dessous (eventuellement avec une pettiet div, avec beige cassé et border-radius, histoire de différencier du reste du texte).
if the triggered_element array is the same as clicked_button, i'd use triggered_element.
This looks like it bled in from your other branch? Watch out for that kind of thing, yo
Nice.  I wasn't even thinking about this when I/we filed that issue.
formating is off.
make this provate
Before we merge, we'll want to pull this out.
Yeah, we have, but can re-do it. What's the preferred method to replace it with?
Hi, this lines are nesessary for Excel UTF Support. Otherwise I had troubles with special charachters like ö/ä/ü,...
you should exclude the Resources folder from the coverage too
We're checking for the use of tabs here, and raising an error if so. But WP standards (for better or worse) advise using tabs over spaces. Is the intention to raise errors when multiple spaces are used instead of single tabs here?
By default, the profile exec-functional-test won't be executed unless with -DenableFunctionalTest=true in mvn command line, so for SCM build, it won't trigger this profile
@nukulb I don't see why not. Why can't we show the user both of them and let them use whichever one they want? The browser shows both as well.
oh sorry, I didn't see L59, I thought it wasn't being created, is there a reason you made it a pointer though? if you just declare it normally it will be automatically freed from the heap when it's done and you don't need to worry about deleting it
if statements can be merged.
Same is true for a BlackBerry.