labofsahil/github-event-dataset-2012 huggingface dataset:
As far as I can see loading indicator is bound to 'load:start' and 'load:end' events, that are handled by model.fetch method (which is called by model.load). model.load will fail early (immediately after route change) and it doesn't use bindToRoute anymore.
Same as for the other job, but here we can spent a lot of time when doing a pull only if the repo has already been cloned.
The blanket _icon was picking up other icons and adding the "background" of action-icons.
Nice catch; just did some testing with this:

    > 1 / 0;
    Infinity
    > (1 - (1 / 0)) * 100;
    -Infinity
    > typeof((1 - (1 / 0)) * 100)
    'number'

I wasn't aware that this is how Javascript handled division by zero, but it seems legit.
you can use StringUtils.isEmpty for this kind of checks
The name of the file mentioned above. It's `queue.txt` for us, I think.
Link is the wrong way round!
Storing the result as a string of 0s and 1s means we use 8x the memory we need before writing out the value. why not just build it up as bits and bytes as you go?
gah needs to be .passForm
removed :-)
Test failed at this line:
self = <class unittestzero.Assert at 0x0320E840>, first = 0, second = 1
msg = 'Addon reviewed was: Scrapbook Lite\nStar_rating was: 5'

    @classmethod
    def equal(self, first, second, msg=None):
>       assert first == second, msg
E       AssertionError: Addon reviewed was: Scrapbook Lite
E       Star_rating was: 5
I ran the test and it failed (actually 5 failed) at this assert.
We shouldn't always compare 'new_rating_count' with 1, since we test for addons with 2, 3,4 and 5 reviews.
I like this approach quite a lot...mostly because I hate defrecord, and this covers like 90% of the reasons why I've ever thought about using records. :)
Just saying..
rows will already be defined at this point because of the above code. I think commentRows makes it clearer what they represent compared to detailRows.
I think we should probably provide a tenant override on an entire Config option (and not just the individual elements). Actually, it might make sense to only allow for the tenantOverride flag on the entire config option. We'll also need the lock in the admin UI to prevent these overrides.
Indentation fell a bit unaligned here
Are you sure this is equivalent? Working copies are a bit fuzzy for me, so I don't know if there is a one-to-one relationship between editors and their working copy. To be on the safe side, log something if you get back `null`. That way we can track possible regressions.
At a minimum ``stackFrame`` should be ``volatile`` because ``ScalaStackFrame`` are shared between different actors (look at ``ScalaThreadActor``). However, I'm afraid you may need more than that.

What I'd suggest is (if possible) to make ``ScalaStackFrame`` immutable. Then ``ScalaStackFrame.rebind`` should simply create a new instance of ``ScalaStackFrame``, i.e.,

```scala
def rebind(newStackFrame: StackFrame): ScalaStackFrame = ScalaStackFrame(thread, newStackFrame) 
```

But I don't know if that would be acceptable from a performance POV (but you know what they say, make your code correct first, and then tune it for performance ;))
Should this be fixed?
а если меньше 0?
Да, согласен выводить в через объект 1 объект 2 не очень хорошая идея.
угу. сначала не внёс намерено, чтоб выяснить обязательно ли выносить все методы класса в интерфейс, а потом проморгал, что он используется в мутаторе.
GenericTool is a stub because @Pamelloes never finished em and someone pulled it, they're not supposed to be like that since they're used in MaterialData ;)
The HTML5 spec suggests storage quota with the limit of 5MB for localStorage

Which means browsers can implement your own quota.
replace with `js.$functionProxy(f)`.
Since `distance` isn't a common field, would you mind rewriting this to mimic what is done for the `$file` property?

```
if ($this->file) {
   $serialized[] = 'file';
}
```
Sequencing them would mean that their subscriptions are sequential too. It'd be easy enough to include the results from `signal1` in the return value if necessary.

Again, this seems to be serving a mostly theoretical need, and I think we should remove it until there's a really concrete case to be made for it.
I don't really consider this a "fix" as much as a "workaround". The fix is of course to get rid of the leaks. What I do here is log the entire result in the build log so that the user can determine whether there's a real error or not. Consider that it took a long time for somebody to figure out where the "corrupt repo" error was coming from since there's no diagnostic info logged. If there's a severe error where we really can't do a rev-parse, then this method will return a junk line, and ObjectId.fromString() will throw an exception instead. That will still trigger the "corrupt repo" logic.

I think this is the better solution, since looking for specific strings like "The process has leaked file descriptors" will be brittle. That addresses only very specific cases. For example, suppose that the same message comes back in German text - the string match won't find it, but we still have a valid repo. Or suppose a new version of Java or of Windows changes the message - same problem again. With the way I did it here, if we can get a valid SHA1, we assume we're good. Any extra output is captured for the user to handle. We may run into errors later on if the repo truly is corrupt, but that can happen even without my patch. (Assume for example that your refs are good, but you have a corrupt pack file. rev-parse will succeed, but checkout can still fail.)
You could check $? right after you run the program.
Ok, I think @dahlby's comments on `Fetch` apply to clone. So something like:

```
void Clone(string sourceUrl, string workingDirectory, CheckoutOptions checkoutOptions, ProgressHandler handler, Action<Repository> onComplete);
```

You might need to tweak that a bit as I'm not sure if you need 3 callbacks for the 3 different `GitIndexerStats` type, or just a single one that has the type of callback as a property. Either way, the basic pattern is to make this have callbacks. Having read @dahlby's comments, the callback approach is certainly easier to setup.

And this makes it easy for us to wrap Rx around it. :)
I agree maybe we can live without the self.raw.copy()

That lead to bugs with raw gets modified between 2 sets of epochs but that can be considered bad practice
As much as I hate to kill all this code, I think dealing with it at the raw level is probably a better solution. I'll close this one and add the relevant parts!
Can you enhance this a bit? Maybe focus on why service availability is important to the business/end-user?
If some method can return ```nil``` (```@page.content_for``` can!), I'm using ```raw``` to avoid ```NoMethodError: undefined method `html_safe' for nil:NilClass```.
``$user`` ?
because you are loading ``app/bootstrap.php.cache`` which loads it. And the goal of this chapter is precisely to explain you how to do to **skip** the ``app/bootstrap.php.cache``
Great! I missed the PSR part the first time. I think the CS docs should emphasize that sentence...
I don't think it makes sense at all now that this is not part of an article on compiler passes. I have removed it altogether in GH-1335
包含一个或者多个存有映射值的文件。
已改为 默认页
use akka.util.NonFatal
You can simplify it to just:

```
if feature:
    blah
    blah
```
insert line break

kept -> uses the same arguments...
should add float and clearfix ?
I think the indentations are off here.
Same comment as above; this always evaluates to True
@danielpassos thanks; @secondsun can you weave that into the doc?
Do we really need both hotRodWrapping and rawValues?
Rebase side effect. The validation used to be located here, but Tristan's changed cache store config around.
Weird indentation throughout this function.
It's highly possible… I've also noticed that some of my more recent reset code is throwing build errors. Also the fact that this reckons the submodule pointer was changed is a bit strange.
Ah, I guess it could. I was just being consistent with the ones below it. I didn't know if something changed in TB13 that made it more consistent with the other Gecko products, so I left it be.
Won't this cause a syntax error?
The Homescreen was already launched in the setup so you don't need to launch it again. Also the homescreen is just always open anyway so instead of launching it you can just find its iframe and switch to that frame when you need to use the homescreen.
It is not generated. It cames from my architecture of a SOAP envelope
Put `);` on the next line.
I don't think we should add the "fixed" option. Especially since it only sets the class of position-fixed and you have to manually add the CSS as you did in the example.

We should add a class of "tour" to all popovers and a unique id "tour-@index" to each popover instead. Then you can add the position:fixed to .tour or #tour-@index when you need it.
Woo hoo input validation!
Can't we run over `sakai.config.skinStore` instead?

This would avoid the 2 each loops
@dooremont seems like this line needs to be removed as well
We actually can patch our true multiclass SVMs to propagate num_classes to the strategy when training
Well the real reason is because I found 150px allowed enough room to easily navigate the items in the dropdown, without looking broken. My changes in IE7 change the design slightly, and leaving 200px looked like the page was broken or had failed to load an extra element.
Das Thema ist wichtig.
Why is 'enabled' stored as a string?
You may have noticed that lots of this code "works" to do fast-delete in column-stores as well as row-stores.  The reason is that it should work, but I haven't gotten that far, yet.   Anyway, this is the hook that keeps it from firing for column-stores, the underlying cell on the page will never be WT_CELL_ADDR_LNO, so fast-delete will never act on column-store pages.
I made this change, and it works, but I actually am not entirely sure why it was necessary.   Obviously there's something about how WT_BTREE_DISCARD works that I don't understand.
Should extra spaces for consistency: `function (Request $request) use (&$middlewareTarget)`.
This seems fine, however $this->stats is never cleared if this code path is taken. If the strpos early exit happens, stats is cleared. However if that exit does not happen and stats was previously initialized then while will never be executed yet we'll still attempt to get estimated row count.

My proposal would be to bring `$this->stats = array()` out of strpos conditional and execute it unconditionally before everything else.
What about the attached diff?
It is still possible to do what you want with original design.

And then, we can still use a WSF_HANDLER as termination of filter chain.


On Thu, Nov 8, 2012 at 5:19 PM, oligot <notifications@github.com> wrote:

> In library/server/wsf/router/context/wsf_filter_context_handler.e:
>
> > @@ -17,8 +17,8 @@ inherit
> >
> >  feature -- Access
> >
> > -	next: detachable WSF_CONTEXT_HANDLER [C]
> > -			-- Next handler	
> > +	next: detachable WSF_FILTER_CONTEXT_HANDLER [C]
>
> I've updated the filter example. Can you have a look at it ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/EiffelWebFramework/EWF/pull/15/files#r2071624>.
>
>
Are you sure that ``$this`` (of type ``object<Assetic\Asset\AssetCollection>``) is traversable?
honestly i have seen this line a million times
String formatting should be fine! ``'%s.%s' % (str(adler...), task_name)`` should work.
What you are doing here is different than what I did in my pull request which relate to the warning from fsockopen()  at line 256 similar to what you did with fopen() in stream.
Could move this to PixelUtils as well !
Alguma razão para teres feito isto com `raise` e deixares o `head :bad_request` para o fim? No teste local que fiz, a mensagem do `raise` não aparece nos logs nem na resposta, por isso não vejo porque o fizeste.
Usar este sistema de excepções tende a ser mais lento do que disparar logo o erro. Eu já tenho modificações prontas para fazer commit e push, tirando o `begin` … `rescue` e usando simplesmente 

```ruby
if [data OK]
  respond_with(…)
else
  head :bad_request
end
```

... que me parece mais straighforward e evita o overhead do begin rescue. Se estiveres de acordo, faço commit e push.
Este arquivo não deve fazer parte do pull request.

Voltar este arquivo para a versão anterior.

```
git log -10 {meu_arquivo}
git checkout {sha do commit} {meu_arquivo}
git add
git commit
git push
```
Any reason not to use lists here? It would probably be best to maintain some sort of ordering here of the attributes rather than go totally random with a hash set.
John, sure, xsl file shouldn't be used to hide secure data, I agree. However, I do feel that when I want to release my xsl templates, I'd do so on purpose (on GitHub). If I don't release them, Symphony shouldn't decide on its own to do it for me. For me, not releasing a template can have multiple reasons like economic (I get paid to code it, and am not allowed to make it open source) and pride (the template is written quickly, because of time or money constraints).
Yeah, that is a bummer.
yes it works!!

my script 

´´´
#!/bin/bash
source env_piplmesh/Scripts/activate
path="$PiplMesh/manage.py "
$path manage.py celerybeat &
$path celeryd --loglevel=info --concurrency=1 --maxtasksperchild=10 &
$path runpushserver &
$path runserver &
´´´

I think i have still that python bug:
 if datetime.datetime.now() > horoscope.date + datetime.timedelta(days=HOROSCOPE_OBSOLETE):
I doesn't. It leaves url's the same. Ok 72.
Check the implementation in `django.contrib.auth.models.User`.
Why all this could not be simply default CSS?
If you check output of `/i18n/js/` you will see this function:

```
function pluralidx(n) {
  var v=(n%100==1 ? 0 : n%100==2 ? 1 : n%100==3 || n%100==4 ? 2 : 3);
  if (typeof(v) == 'boolean') {
    return v ? 1 : 0;
  } else {
    return v;
  }
}
```

I see that it returns values `0`, `1`, `2` and `3`. But in your translations you are missing `3`. So for Slovenian there are four possible values, only only three. (0 dni, 1 dan, 2 dneva, 3 dnevi, 5 dni)
> Mixin is for another pull request I guess?

As you want.
I agree, no point of having class for non-changing  once-occurring messages. Will change it now.
Please explain XPaths above.
Empty lines before and after this URL.
But OK. This is ugly. Then do:

```
newuser = auth.authenticate(username=form.cleaned_data['username'], password=form.cleaned_data['password2'])
auth.login(self.request, newuser)
```

Mitar
This is a very nice feature... I'll take a closer look asap. Thanks for the very thorough docs!
I'm being a little bit of the Devil's Advocate here, but what exactly are we trying to prove here?

A public class is being meticulously tested, but no other test makes use of the fact that this public class has been introduced. Wouldn't it be simpler to make `UnknownParametersCountMethod` internal and move it to the AutoFakeItEasy library?

Is a class even required here? The class only does one extremely simple thing? Wouldn't it be simpler to inline that single line of code in the `FakeItEasyType` class?
How would you pass `Int64.MaxValue` into an `int[]`?
`int` is an internal type
I see no way to reference, though I could be missing something. I suppose they're just Ruby files, so maybe we could use require? Seems like a refactoring that could be done after this gets merged, though.
mhm, why uint32?
will wait for more things to change and ill commit it tomorrow
Jeez....
Mismo comentario que más arriba, los paréntesis aquí no son necesarios.
IMO we could fallback to native `Array::some`, but not a big deal.
I'm not very comfortable with depending on this. Can we self-host the plugin? If that site goes away for whatever reason we can't build. That's not a situation we can find ourselves in.
I'm not entirely sure what I'm looking for, but there was no difference in the output of 'mvn help:effective-pom' with the file as it is now, and with the <scope>test</scope> removed. So I'm assuming that it's safe to remove that line?
I believe you have this backwards. If resultsLimit is set to "30", you're passing the ajax param data of "30=limit".  Switch resultsLimit with limitParam.
Please, add a docstring here.
you should add in some hilarious invalid usernames.

i'm not serious. don't do that.
I've no idea. I don't really understand what `inst` is, anyway.
As above, looks wrong to call a private member on something else than this.
Is the `this._tag_manager` check so it runs fine in tests?
Private member access.
You'll want to leave version changes up to Opscode so they can decide when / how to rev
The order you pass these variables match the signature of the Ebook __init__ function on line 23
assumes all kwds are arrays, dangerous?

I like ``kwds`` better than ``kwargs``
может засунем настоящий урл? я тебе скину на dev-версию ссылку когда залью
did you mean to commit mwhenry.com? :D
Done. Though I'm unsure why this would ever be unset, would it be cleaner to ensure to default it to {} when creating the dygraph object? I had merged this from a recent patch adding a null check.
yes, it's very naive right now and only does browserish validation (but i don't even count on that, that code is somewhat optimistic and forward looking but there's nothing there to support it yet)
Simplification:
    if (DocumentTransformConstants.EBXML_DOCENTRY_PATIENT_ID.equals(slot.getName()) {
handles the null case without extra logic.
... and it occurred to me now, that while it seems simplest to remove the defaults from C++ so that size specification is required for Sersic, deVauc, Airy, Box, that does have a downside, which is that the radius-checking and exception throwing is happening through different means than for the methods that go through checkRadii in pysrc/SBProfile.cpp.  If we want to change this behavior later, then it seems bad that different SBProfiles will have the size arguments checked in different ways.  So maybe i should change to use checkRadii in the pysrc for everything (which would also help if, later, we want to add another radius specifier for Sersic).
Global variables should be declared at the top (module level) of a script.
would it be possible to use a fix url instead ?
Interesting, hadn't appreciated that we'd have to get the domain from the HOST header, but of course we don't know that from the request itself.  Bueno.
parameter `e` and exception `e2` risks being confusing. I'd rename the parameter to `entity`, but that's just my preference.
Same here: good to include the entity toString in the log message.
The location is important due to php imports.

The file will be deleted as soon as the setup has completed.
Guava's Objects.equal is a nice shorthand for this.
Perhaps you should add tests for “invalid” `array` and `n` arguments.
Here you can use `User._meta.object_name`.
I'm a little concerned by this. It's a big slowdown for "proper" databases, just to let test databases work better. I wonder if there's a way we can catch this error for SQLite and fix it? Or a database-specific switch (pretty hacky, mind).

Alternatively, this can be fixed by changing some of the config params for SQLite.

I'm not sure what to do here.
Gone to Logging.scala and AssetsPlugin.scala
I tried without `raw` and it works correctly. The `<a>` is rendered as is.

Why are the `raw` methods still here?
I'm cool, whatever works for you mate :)
If we move non *.class.php files to src/main/resources
- for maven there is no problem as they get copied to target/classes in the process-resorces goal
- for non-maven we can add src/main/resouces to module.pth
Sorry if this is considered bikeshedding, but I still disagree. WindowManager.launch isn't Gaia's way of launching apps, it's the homescreen app's way of launching apps inside itself. If apps have to call special methods in the Gaia homescreen then they're Gaia apps, not web apps. I feel that now we're past the Demo Phone milestone, we should be figuring out how to do this properly, not hacking something together. I suggest leaving this as it is and then discussing our plan to turn Gaia apps into real Open Web Apps hosting on an actual web server.
Thank you for writing a test! I've ran locally and verified that w/o the fix things do indeed break.
Does something else set is_aurora? If not, why can't you set this in the previous if block?
Instead of repeating this.id two times in the parameters you can say: 
Post.find("SELECT p FROM Post p, IN(p.author.friendedBy) u WHERE u.from.id = ?1 and (U.accepted = true or u.to.id = ?1) order by Date desc",
this.id).fetch();

Please fix this
Are you saying that you'd like the read-ID function to be part of the generic API?  I think I'd like to keep that out of the generic API since I don't really want users of this to ever know anything about the details of the underlying device so I can be certain that we can swap one driver out for another without changing the API users.  That's one of the things I'm trying to abstract away via the driver API.  Callers shouldn't even know if the flash is external SPI flash or internal STM32 flash (or a file in the unittest).

It does make sense to have the JEDEC driver properly check the device ID during init though.  I think my preference would be to pass the expected device ID into the JEDEC driver in the config struct and have the init function properly verify the value.  That keeps HW details contained in the board_init + driver init rather than exposing it to users of the generic API.

Would that cover your use case?
I can remove the added test, but that seems senseless because it exposes a problem that already existed and still needs to be fixed.  I simply don't have the fix to it in this patch because I'm not sure how to solve it.
This tests the creation of a 10000 long defer chain nothing more
I disagree, and think they belong here or no alias at all.  This is based off all the code I see in Chase and Serve with one method with slightly different implementation and expectancies.
Stack в однопоточных приложениях не надо использовать.
Проверь это условие. Объяви метод с return type LinkedList.
Typo:  "The procy" should be "The proxy"
Can you just do new JSONArray("{" + registrationID + "}") or something
docstring comments should be used to document the function/method per [PEP 8](http://www.python.org/dev/peps/pep-0008/)/[PEP 257](http://www.python.org/dev/peps/pep-0257/) (that is, immediately after the `def` and only then). any inline comments should always use `#` as @m8ttyB says above.
h_cl is the new table, h_complearning is the old table.
`RidgeClassifier`, `LinearSVC` and `sklearn.multiclass`. I see now that the latter exposes its binarizer.
Shouldn't this be a `property`?
Ok, sorry. I didn't run the coverage tool yet.
Yup, good idea!
i get an error message: undefined local variable
Can you add a test for this case, please?
Consistent as in all of dmd uses the same methods for reporting/detecting/displaying errors.
Exactly.  Meaning, these code points should only be used *inside of a program*, not *across program boundaries*, like programming language text.  I think the Unicode standard makes this very clear.

Also, note that the specified ranges are *rejected*, so commenting out the last row implicitly *accepts* the final range, except that it is already handled on line 83.  It's just a micro-optimization to eliminate 2 more checks.  I just wanted to document that this range was explicitly rejected.
@jystewart I'm changing this on master now
Definately not easily. The first thing it tries to lookup "ldapAddress" is not in the environment in that form
Well, even though its deprecated this actually makes the usage more consistent with other places.  Not sure it matters either way.
Ok, have removed (see last push)
Perhaps this code was added to the wrong function. It seems to me now that perhaps I should be adding this code to "mangleLocalTempFilename" instead of "mangleHelperFileName" , will investigate but from a precursory scan it's not clear.  If any of you can have a look please do...
actions A, B, C, D, where "B" is the rename and "C" lock times out.

* A executes, success
* B executes, rename the file to "newName", success (file now has newName, not logicalname)
* C lock fails, retry (release the lock)
* B retry, need to re-rename back to original name, release the lock
* A retry, release the lock
* run again...

Why not just wait until the others succeed? Because that breaks the transactional model contract. And if someone CTRL+C after roll-back and before re-run, the FS is in a good state.
you can use `{+silent}` as a shortcut for `silent: true`
`#name` is a shortcut for `#{name}` (works only for simple interpolations)
Also, the arguments here too are strings.
`@set \name "#name (#count)" {+silent}`
This is too rough. If `str(e).startswith('5')` we should bubble that up.
Right, scratch that.
Для того шоб додати користувача "адмін"  з алмінскими правами, його блог і пр. плюшками
Впринципі самого користувача  і блоги можна додати через фікстури але проблема з  перенесенням користувача в групу  Адмінів
scripts should be inside head tag in layout.jade.
Fixed :D (I hope)
This script is not perfect, but feel free to keep it as-is if it is easier to explain!
Here, we register a `keyup` listener each time a `show`event is dispatched.
We end up registering only one at a time, as we overload the previous one with a new one.
It would be better to register this listener only once, outside of the `show` event listener.
So If I understood `parse` implementation correctly, `parse` and `serialize` aren't bijective. Is it expected ?
(Not a big deal, but we should at least mention it)
Could you elaborate a bit what you were trying to do here? Is this part of the functionality you wanted in the PR? Otherwise I'd remove it.
it is new ? When the branch does not look like a version, it is ignored by the guessing AFAIK.

Btw, you should mention that an explicit version in the composer.json will win over the branch name (unless you discussed something else not released yet ?)
sounds like an improvement indeed. Lets take a look at it in the AM. Looking forward to seeing this running without errors. Is this an error in your code or an error in the application?
I see... Would you like to fix it or should I try it?
Already made the changes locally but wanted to wait until the API is finally solid before I need to change te example again.
Isn't it possible to use one of the utilty functions in ``skimage.util.dtype`` (also imported as ``skimage.img_as_float`` etc.)  At least, in ``skimage.util.dtype`` there are methods for providing the appropriate ranges.
added locally already
Don't assume they passed in a `Cartesian3`; they may have passed in an object literal with `x`, `y`, and `z` properties.  Use the static `clone` function instead.  This is a general rule of thumb for arguments to all public functions.
It's private.  Marked it as such.
We need to change my crappy code that precompiles the shaders in Track Santa to match this.
Perhaps an `@example` for this and the other cartographic to/form Cartesian functions - not because they are complicated, but because they are so commonly used.
Talked with Blain yesterday and basically if you don't set the other properties, aka "middleName", "firstName", and "lastName", those will not be set. For fullName, however, iOS has a default for it, which is "No Name".  This may not be the correct behavior, but it is with parity for now.  So I may change this behavior in my next PR with Contacts, when I implement modify/delete contacts.
I guess this really depends on what distinguishing factor we are using to define standard vs hybrid.  If the factor is showing roads then yes, they are the same since you can't turn off roads in satellite.  Even so - with the requested addition of blue google street view lines in hybrid mode under Android we can't truly say they are the "same".

I feel it would be better to just call out this behavior.
I understand but I would wager that most of our developers have had little to no experience with endian handling so I figured we should provide or link to some information on the topic.
I guess so. That could be HI LAR I OZ
resolved this issue in a new widget
Let's add some new lines.
这写法太老了哈
Another benefit to this is that you could [change your specs from this](https://github.com/resolve/refinerycms/pull/1304/files#L3R15):

```
section.is_title.should == true
```
to:

```
section.should be_title
```
s/ranges/bidirectional ranges/
how about removing this line and replacing the next line with:

    enforce(_data.RefCounted.isInitialized || (i == 0 && j == 0));
Oh I thought the getBoolean() method had an optional constructor like getBoolean(String path, boolean default)
(can't make a comment below this line, but it will be really helpful to make TreeNode hashable (so we can easily map analysis info from it). It should be easy to do here, for instance:

class TreeNode implements Comparable, Hashable {
  ...
  int hashCode() => span.start << 7 ^ start.end;
  int compareTo(TreeNode other) => span.compareTo(other.span);
...
Y U NO INCLUDE BMP?
I think you can use `MailServices.accounts` here.
Are the parenthesis necessary?   Is there a parsing error otherwise?
Code was moved to the Model
just refreshing the page to constantly experiencing the duh!! moments
es algo que pensé, si tener la abstract factory o solo el método... compro lo del método, pero recién lo puedo cambiar mañana a la noche. O el sábado.
Can you describe what this condition does?

As far as I can tell it will never be true, because `len` is a CIDR integer and `IN4MASK`/`IN6MASK` are hex values.
why linking to your fork ?
Just something I thouht about myself: Whats about

* No variable/parameter/property/...-names (in short: identifiers) longer than 30 (?) characters.
* Line endings `\n`. I just ran into issues while I `grep`d over source, that contains both `\n` and `\r\n`. Thats just scary...
* Line length: soft limit 120, hard limit 150

?

I tried to choose fair numbers, that should reflect, that exceeding is not only ugly, but may lead to serious comprehesion (? right word?) problems. In fact they are just placeholders.
Can you add `;` at the end of each line in this file?
This loop is basically to pre-render the date facets. Normal facets just need 'solr_field' => array('value1', 123) etc, but date facets need the data rendered in different forms.
1. key: 2011-01-01T00:00:00Z
2. results: 123
3. filter to create the link. [start TO end] or [2011-01-01T00:00:00Z TO 2012-01-01T00:00:00Z]
4. formatted value: 2011 - 2012
Right, but IIRC, they wanted multiple MS-backed channels in the same app, with a different region for each.
Many of these atts are now taken care of by the super parser.

Since you will be back in here; you should change RLR and RLR.R to classes instead of String.
Not sure ;), this is what mine did.
do you want me to change it?
True, i must have missed that after we talked but yes I agree that that should be the case.
In other words we need to call the postProcessFilteredMessage regardless of TX or no TX
I'm afraid this is a race condition. If the second changes during this test, the test will fail.
After the rename, this will be `  - cd vendor/doctrine/doctrine-mongo-odm-module/tests`
Should be blank line.
This logic should go in the deprecated plugin itself (it should listen for the tiPropReferenced event and then do this logic).
Indeed... I don't feel good about adding a dependency to the named parameter `subclass`... @rande is it ok for you, or should we provide a way to retrieve the value from the admin class ?
https://github.com/wilds/TrinityCore/blob/master/src/server/game/Movement/Spline/MoveSpline.cpp#L113
Why is this property of this Shape mutable, but most others (that I've seen so far at least) are not?
Après verif', c'est bien ça. Toute les lignes se sont vue agrémenté d'un jolie '^M'...
Bon vue la portion de code qui merde c'est pas très grave, pour ce coup-ci osef. Mais va vraiment falloir que tu trouves une solution parce que manifestement le problème vient de toi.
I expect this will improve the stability of the gate.
Hide this for now with "if TEST_MGMT".
Or another example:

https://github.com/fivethreeo/cmsplugin-blog
```python
duplicate_message = parameter[1] in self._messages
```
Probably a good idea to name the new parameter something like ```personal_ids``` to avoid putting the type in the name (see my other comment below).
You shouldn't specify the column number here because that will change in different mapping files. To make this more general, this function should take a parameter called something like ``personal_id_field``, and then you should look up where that falls in the header line. I think that should look like this:

```
try:
    personal_id_index = header.index(personal_id_field)
except ValueError:
    raise ValueError, "personal id field (%s) is not a mapping file column header" % personal_id_field
```

Then, you'd use this as:

```
if i[personal_id_index] == personal_id_of_interest:
```

You should make this change throughout the code where ever you're using the index directly.
akhe mikhastam background coloresho rgba(0,0,0,0.5) bedam ke bad ba js roosh ke click mishe rgba(0,0,0,0) beshe ke khode
<div style=\"background-image:url('$path/$each_img');\"> 
 background color ghabool nemikard
warning : it is not always necessarily a Document. better use a window instead.
Ok, i've remove type attr from link and style tagnames:
https://github.com/telefonicaid/Gaia-UI-Building-Blocks/commit/5e435e05c15abd060af1fdd25a8378dc3086fff8
Why do we have this as well as `require_moderator` when they both seem to do basically the same thing? There must be some way to merge them?
Yes, but I didn't want to divert away from how the rest of the tasks do it to keep things consistent.
oh, I see. nvm
Can you verify it was called with an array?
the thread is a detached thread and will quit when main process exits.
"... *the* tuple initializer."
Never enjoyed that rule in C++ :o).
Not return Null ,need to return the last child