labofsahil/github-event-dataset-2012 huggingface dataset:
Flush queue at exit

Adds flush command to send queued messages on demand, fixes race condition w/ .synchronous behavior, adds timeouts for TCPConnect.new and at_exit's thread join.
Fix to make ui.dialog return an object

Fixes blackberry/BB10-WebWorks-Framework#354
Docs: blackberry-webworks/WebWorks-API-Docs#221
(2/2) 1% battery increments and no charging light on shadow

Use the stock 605 OTA version of battd on shadow. This, combined with my other pull request for android_device_motorola_shadow, eliminates the annoying charging light and enables 1% battery increments.
I HATE LIFE

oh god
DBOChangeDAO now exposes DTOs instead of DBOs
[twitter] Store entire JSON response instead of specific fields
Plockat bort tester för obsoleta sdk:er.

Tjena Mattias!

Jag har plockat bort tester för obsoleta sdk:er i master-branchen. Jag behövde det för att få hudson-servern upp å snurra.

Förhoppningsvis dyker det upp lite automatiserade instruktioner till dig här på denna pull-webbsidan med exakta git-kommandon att köra får att få in det i ditt repo och din master-branch.

/Hugo
Group API for LinkedIn

Finally got round to implementing the Group API for LinkedIn.  This should be the last piece of the LinkedIn API.
Adjust form type

Adjust the tree model type to latest Form Component changes

This fixes #42
Api updates

Making changes to the API docs for the following issues:

https://github.com/blackberry/WebWorks-API-Docs/issues/240
https://github.com/blackberry/WebWorks-API-Docs/issues/300
https://github.com/blackberry/WebWorks-API-Docs/issues/292
https://github.com/blackberry/WebWorks-API-Docs/issues/293
https://github.com/blackberry/WebWorks-API-Docs/issues/278
New Module: Persist Hidden Comments across Page Loads

This is a little module that stores a list of hidden comments in localStorage, and then re-hides them on the page load.

Tricky bits:

1. It uses an array of keys to prune if the object gets too large. Right now it prunes at 500.

2. There was a little rendering issue with the comment navigator that I worked around with a setTimeout. I assume it was interacting poorly with another module. Feel free to play around with that if you want to see what I'm talking about, otherwise it works well.

Any thoughts appreciated!
Renumbering regexes convention

0000-0999 Regexes that cover unwanted attributes issues
1000-1999 Regexes that cover unwanted elements issues
2000-2999 Regexes that cover empty elements issues
3000-3999 Regexes that cover structure issues
4000-4999 Regexes that cover HTML entities issues
HTML에 대한 참고 부분을 추가했습니다.

틀린점이 있을 것 같습니다;
Adds a data-graph-xaxis-labels-enabled option

, which allows one to disable the labels on the X-axis (useful when there are so many items that labels overlap each others)
epalace510

It's definitely worth adding my profile page.
Use the command InCm to know when the init is finished

A cleaner way to know if the init is done.
Changes up to 1/27

Most of these changes are things like wonder cost, test string fixes, etc, so those are good.  I did however make changes to athe AssingStartingPlots.lua in anticipation of the new resources, and the code that was added has not been tested.  It is disabled now so there won't be any problems, but if you see something wrong with it don hesitate to fix it.  In the NewResources.xml I again added disabled code, but I left some tags blank because they require the icons and stuff.  Besides these though, everything else is working. :D
ARQ-780 add glassfishResourcesXml property and deprecate sunResourcesXml
Webviewfactory event fix
Improved i18n of panels and important fix to logging panel counter
'Database kann nun Clientstatus abrufen
Conditional Object Actions

Allow generator.yml to supply a conditional function to object actions, along with an "inverse" option on whether the object's conditional result should be inverse.  In the generator.yml, specify the condition like this:

```yaml
  object_actions:
    delete:
      condition:
        function: canDelete
        inverse: false
```
In this case, the generator's model object must contain a "canDelete" method:

```php
class ProductEntity 
{
      public function canDelete() 
      {
            return $this->countProductSales() == 0 ? true : false;
      }
}
```

The link in the object_actions.php.twig template should then be wrapped with: 

```twig
{% if action.conditionalFunction %}
	{{ echo_if ((action.conditionalInverse  == 'true' ? 'not ' : '') ~ builder.ModelClass ~ "." ~ action.conditionalFunction) }}
{% endif %}

###---- LINK HERE ------###

{% if action.conditionalFunction %}
	{{ echo_endif () }}
{% endif %}
```
Added version for update handling

Added the version line for allowing auto-update of package if new version available
merci

Merci pour tous
Note on protection's role in name lookup.

This is an often misunderstood part of how protection works, especially at module level, so I think it is worth explicitly stating in the documentation.
trying to make mssql work

not sure if you are using a different version of win32 bindings, but i had to make a few changes to make this work with win32 from https://github.com/AndrejMitrovic/WindowsAPI.git
As there are three components to the URL (Press URL + "catalog" + "series" + "path", the only thing that seems to work is the following: 

{url|assign:"sampleUrl" router=$smarty.const.ROUTE_PAGE page="catalog" op="category" path="path"}

I'm not entirely sure if this is using the url assignment function correctly, but it seems to work fine. If that's ok, I'll clean up the series and category URL assignments.
Again, could be a matter of an unfortunate merge but both of these lines are not needed. pyflake it.
Merged by r34197. Thanks.
It's generally preferable to use {fieldLabel} rather than <label>.
Understandable. Although is just a different word phrasing which wouldn't harm if not translated and maybe has to be made sooner than later in case translations will be update.
Well what do you know...
I prefer a block. A second boolean argument is a little bit ugly =)
looks good, I'd have kept/sorted alphabetically sorting for the grep/case clause, but that may be done later anyway.
Most probably, but some of these entities might not even be part of Extensions module - we're probably looking at creating a core bundle in the end.  I'd expect we refactor the class name again at some point.  At this point I'm just trying to get the conversions started.
Travis Y U LAME.
This still looks like an unintended change. The extra checks in the if statement are needed.
You should reference a tag in a repo here, for example:
```
s.source   = { :git => 'git://github.com/AgileBits/SGHotKeysLib.git' , :tag=>'1.1'}
```
Again, --auth not required
Nice perf win.
Without skipping after setting :fixed attr, my carefully chosen fixed widths were no effect on final xlsx. At least in LibreOffice.

I sent a pull request for it but it seems that I've missed the edge case you mentioned. ;)
Well, it sure shouldn't be me... who should it be?
Capturing group named "module" removed from all templates.
Answer: much faster to do datetimestruct (I added Year to test with the old method). It's trivial to build a structured dtype cache. Should take 10 mins... I'll do it to compare.

In [4]: dti = DatetimeIndex(start=datetime(2005,1,1), end=datetime(2006,1,1), offset='Min')

In [5]: dti
Out[5]: 
<class 'pandas.core.index.DatetimeIndex'>
offset: Min, tzinfo: None
[2005-01-01 00:00:00, ..., 2006-01-01 00:00:00]
length: 525601

In [6]: %timeit dti.year
10 loops, best of 3: 114 ms per loop

In [7]: %timeit dti.Year
10 loops, best of 3: 24.2 ms per loop
No worries, I still kind of know how imperative programming works :-P.
My point was that generally `isEmpty()` is the safer bet because it can be internally represented as `size() == 0` if that's fastest. It doesn't work the other way round, so one should use `isEmpty()` every time one doesn't exactly know how the underlying collection is implemented. And it's more readable, too!
As these are the flattened files, you might be interested in commit @16c8cb39edcdfaa5279f372f5e21fcd294e3bbdc which applies your tweak to the source files.
:) Okay, willing to update that:
```
cat('\n', rep('=', getOption('width')), sep='')
```

Or you'd have a better solution?
why not `2.2.*`?
good catch, that explains some warnings I've seen lately. I thought maybe rake was just stomping on tango methods, never bothered to check is cd was actually an alias for in_directory
Accumulator must be undefined on "use strict" mode.
CloudController is creating service when requested through vmc commands and then checking name (alias is db column). This is leading to an abandoned (un-attached) services in case of Mongodb. Changed the behavior to check it before creating.
Done in an updated patch here:
https://github.com/Bertram25/mana/commit/cb15794bc7121604fcee52d1de5e7d3b4b86ed8c
You forgot to add the options has to the 'to_comma' call.

So, for example, specifying options such as the :style will be ignored.
Here is a small example on what I'm telling:

```
@> as.integer(factor(paste("a", 1:11, sep="_")))
 [1]  1  4  5  6  7  8  9 10 11  2  3
```

maybe the possible fix is to use `id.length` instead of `id` in `polylineGrob` like this:


```
polylineGrob(..., id.length = rle(group))
```

In this case, all points in the same line need appear in succession in the data.
Oops, I'd removed that line since I last pushed, but thanks for the pull request :D
C est pour le portage du module assortiment. C est vrai qu il suffit d
avoir le meme hmlname definie ds la fonction, c est juste pour etre
exaustif dans l appel du hook et de le definir a l entree du hook. Apres on
s en sert ou non. Mais c est comme tu veux, ce n est pas blocant de ne pas
le pass. Dit moi comment tu le vois et j avise en fonction.
Le 7 mars 2012 12:25, "Regis Houssin" <
reply@reply.github.com>
a crit :

> ca passe mais ceci peut tre dfini dans ta fonction
> 'formCreateProductSupplierOptions'
> quel est l'intrt ?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/FHenry/dolibarr/commit/0b9b92dedd0f4af3e3f21f6e105299ce9cbd4480#commitcomment-1052841
>
`ReqCreatureOrGOCount1` = 0 for to complete the quest at the time of the socket. Time to find a debugg. (it does not create any error it just validates the quest when it is taken)
I don't know if this update was aimed to fix the current currencies bug on reloging, but if it was, it didnt work :p
:P true
Ok, I see it is not yet loaded by default.

julia> load("printf.j")

julia> printf("| %-8s | %3d |\n", "foo", 3)
| foo      |   3 |
Beim ersten durchlauf 6 Fehler... Folgedurchläufe 7 Fehler
Ja, war eigentlich so gedacht - aber wollen wir uns wirklich heute noch die Arbeit machen und das erweitern?
Because an email address is not allowed and to provide feedback it is not allowed?
Хе. Ну это гораздо корректнее, но зато не даст совершенно никакого эффекта - в случае статичной позиции на земле Z будет одинаков что там что там.
Yes yes You do.
@RSA You mean all VMAP ?
I extract all this on windows xp 32bit , did i need to extract it on Unix 64Bit?
No, latest rev unstable. Get back to [mr1608] until we find source of problems ;-)
yeah, thx for noticing
Hi,

I thought your first note was part of the emacs mailing list Opps... :-(

I remember fighting with this but I don't know what I finally came up with.  Seems like I had to do some type of external hack.  That was long ago.

I sit on the emails mailing list.  I mostly ignore it but its convenient to ask quick questions like "how do I ....".  Also, every once in a while a message comes across with a really cool lisp library.
I've changed the getter to check for an empty value and set them to being
null. Haven't committed the change yet though.

  $this->_modelName = empty($name) ? null : $name;

On Thu, Mar 8, 2012 at 5:00 PM, Maurizio Domba <
reply@reply.github.com
> wrote:

> Check my additional comment on the pull request #471
>
> This has already been discussed before (I did not see it)... no need to
> check empty or false, this is a non valid value.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ronaldfenner/yii/commit/b03e2d04f4a4ff9d1467ef2c186a4f267afb957f#commitcomment-1063326
>
Looks good. Pulling thanks!
Yes, but I think it will take to fix it sometime, so I  think that filter these matter isn´t bad idea.
Should all of this not be redundant with the advent on ``Desktop.open(File)`` in Java 1.6?
I don't think we actually use it with URLs.
Yes, in fact, I can just revert the 'backported' patch and I can hear the sound then.

'Before the byuuapu merge' would be essentially my repo/fork as it is.

The ROM I'm using I've listed earlier ('Donkey Kong Country 2: Diddy's Kong Quest (USA) (En, Fr).sfc') - it's from the No-intro romset.

I went to another level that has the Rambi sidekick (level 1-1) and the jumping sound still seems to be in that level - so it could be that the background music is somehow conflicting with the jumping sound in that stage (Red Hot Ride), clipping it out in the process. This didn't happen before the fixed-point changes - that's about all I can confirm.
you need to override only `background-position`
Great. Scroll of the Monk even has the rules in the sidebar on page 16:  
Pay half again as much.

Maybe we should drop the explicit cost in the template and just calculate it from the base cost?
Maybe this would be nice as an option for theme-designers? :D
Sadly, google did not fix this prior to pushing Chrome 17 public. I have incorporated a fix (similar but with a bit less code) into the latest commit, along with a thanks to you for finding this and providing the initial fix for it.
just to let you know, you broke this.
It looks like this still needs to be merged into OOICI. Can you check to see if it is still upto date and if so move the PR to OOICI. 
Thanks
Great work! I didn't test it. r+
Unfortunately it's not that. I'll try to dig into it and find the real reasons once I've got a little time...
i would say we should only require the implementation. its the job of the implementation to depend on the interfaces and if it doesn't, it needs to be fixed. if you need to work with a misconfigured impl, you can still add the interfaces dependency to your project as a work around.
the positive side is that the implementation could reference i.e. a temporary fork of phpcr with some bugfix and things like that.

i'd say lets just see this in action and find out what is better.
I suspect you could use filelib:fold_files/5 here.
If you want this to be a tuple, it needs a trailing comma inside the parens. As it stands, it is equivalent to:

    VALIDATORS = validators.has_method(('GET'))

because the parens just tell Python that this value should be evaluated before the others in the expression.
Ruby-style constructor.
Why would you change indentation to mismatch the rest of the codebase?
I would add the gthread-2.0 in configure.ac. I need this to build dawati on my system as well.
мы перелезли на kaminari и не жалеем
senseless, since the formatter will put the brackets below the method's name the next time anyway ;)
MKPersianCal is a simple way to present Persian date "Shamsi - Khorshidi". It's include one class, MKPersianCal, and occupies less than 100 kB compiled. It uses only public APIs, to show Persian date of course you can create events for your dates and ensure that your application is safe for submission to the App Store.
yes, should work too.
The `:` breaks the design . See auraphp.github.com/Aura.Router , which was fixed in https://github.com/auraphp/Aura.Router/commit/50913dc3ca1368f445552eb0c12d7de4c4376560 . Applies to all gh-pages branch .
You deleted a lot of what I said in this paragraph and replaced it with something shorter and different.  Why?  Is it because we explored so many water physics possibilities that we weren't going to be able to explain them usefully in this comment anyway?  (Which is a reason I'm in favor of.)
Maybe it could be the autostart causing the issue, as i have not tested with that, could you disable autostart.sh and see if you still get the issue??

When it freezes on boot can you still ssh in to the box or not?
Because I'm a noob and I didn't know it existed?  :)  Actually, for this
particular test I'm a little paranoid about making sure I have a full clone
as opposed to possibly sharing references between the resulting hash and
ENV.  Frankly, the ENV object scares me because it seems to be implemented
somewhat differently in 1.8.5 as compared to newer versions, and I don't
have a complete understanding of those differences.

"to_hash" would probably work fine though.  I'll read up on it tomorrow and
try it out, thanks for the pointer.

On Wed, Feb 29, 2012 at 5:32 PM, Nick Lewis <
reply@reply.github.com
> wrote:

> Is there a reason `ENV.to_hash` doesn't work?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/cprice-puppet/puppet/commit/85077ea13ce9b12aee6179158c3c2176617e0251#commitcomment-1029000
>
Added a UIOutlet
Added a IBAction
IBAction changes text value of UIOutlet
Please use a better variable name than $chk_123.  Even just $chk_num would be better.
As above re inline structures
Only needs it once. You'll notice that `$last_slash` was being used as `X + 1` in both locations. It is not incremented in the first instance and so no more maths has to be done to +1 again.
我刚刚突然想起来坑掉的倚天设计赛了
要不要赶在游卡二将成名之前填掉？免得技能又被他们拿去了
要是作者自己投稿的也罢，要不是自己投稿的，论坛相应版块又关着，可死无对证了╮(╯_╰)╭
@ boomy D you shouldnt start to talk about that :D if i remember right it was you who had a 1gb db errors log :D
retriman has some good fixes for battlegrounds and other misc inside ;)
same comment above
For `script:`, we at least need to fall back to `make test`. I cannot bring up the link immediately but there was a CI tool that detected `Build.pl` and a dozen of other options to run tests. We need to explore this, treating all projects the same is not a very good idea.
Added action onmouseover for dropdown 
allow click action if set data-selector
example

```
<li class="dropdown">
    <a data-toggle="dropdown" class="dropdown-toggle" href="/profile/" data-selector="#">Profile</a>
    <ul class="dropdown-menu">
        <li><a href="#">Edit</a></li>
        <li><a href="#">Show history</a></li>
    </ul>
</li>
```
You need a little extra setup for testing now that we moved stuff into Rack
middleware. It's documented in the README:
http://rubydoc.info/gems/clearance/0.14.0/file/README.md#Testing
I thought that it is what you wanted. I have badly understand. In any event that does not have importance because one can put at it what one wants like name. It is this same name which will appear in script. Thus one can modify "English" by "en_GB" or "en_US". 

When the user goes in the module “Configuration” it is this name which will appear.

Now the question is to know is we put “French” or “fr_FR”? :-)
I think that "witch database to use" must be defined in a config/db.php file.
Thx for the hint. Such a regex would be a better solution.
Try to avoid whitespace noise.
This is just a problem I've run into many times depending on how the ruby on that particular machine was built. rvm hasn't always built correctly to prevent this issue, and with the many various machines with different versions of ruby it's easy to mistakenly release a broken gem.

I didn't mean for this to be passive aggressive. It's a problem that's been easy to cause lately. It's just unfortunate that there was ever a version of rubygems that resulted in this error, even if newer versions don't. I assume it will clear up as we get all our development machines running good versions of rubygems. (I found the issue in the rubygems repo that talks about this problem, Evan thinks he has it fixed once and for all. This has been my experience with my last 1.9.3 releases with the newest rubygems.)
Clarifying the previous comment: Driver's station display is best (if only) made in labview
`eval` is probably the worst think you can do here :)

Not only just a presence of `eval` is wrong here, the link data keeps `slide[1]` text - a reference to local (internal!) variable `slide` and it will simply break if this variable is renamed  by simply minifying the script.
These methods should also do `var object = Object(this)` and throw if `this == null`.
I'm currently on this commit and I'm getting that cross-classloader completion bug thingy again:

```

  ~/code/bultitude $ lein repl
#<Agent$Closeable$b14108b6@2df30035: {:ss #<ServerSocket ServerSocket[addr=0.0.0.0/0.0.0.0,port=0,localport=64732]>, :transport #<transport$bencode clojure.tools.nrepl.transport$bencode@15384ed5>, :greeting nil, :handler #<session$session$fn__405 clojure.tools.nrepl.middleware.session$session$fn__405@74b70648>}>
Welcome to REPL-y!
Clojure 1.2.1
    Exit: Control+D or (exit) or (quit)
Commands: (help)
    Docs: (doc function-name-here)
          (find-doc "part-of-name-here")
  Source: (source function-name-here)
          (sourcery function-name-here)
 Javadoc: (javadoc java-object-or-class-here)
Examples from clojuredocs.org:
          (clojuredocs name-here)
          (clojuredocs "ns-here" "name-here")
nil
user=> (require 'bultitude.core)
nil
user=> (bultitude.core/java.lang.ClassNotFoundException: bultitude.core (NO_SOURCE_FILE:1)^C
#<RuntimeException java.lang.RuntimeException: Invalid token: bultitude.core/>
nil
user=> Bye for now!
```
Is there no way to generate those two huge files? The Visitor one is generated in spec I think…
Sure can Sanne ;)
BTW, if you have any idea on how to constrain the number of marshaller instances, I'm all ears :)
Unfortunately we use the classloader from the invocation context as well as the one configured at the cache level, so we can't use a simple Map<ClassLoader, Marshaller>...
In the end I kept it with a Parallel call per directory - given it's each top level directory, there should still be pretty solid advantages in using Parallel.
@bkeepers thanks, but no thanks. I have too many gems to maintain as-is, and don't personally touch Rails 2.x anymore. :) I think I'll follow the "try it and see if it works" strategy.
Ideally this would have a check that $7.queryExpr() wasn't non-null, and generate an error about COUNTER not legal here.
OwnedHqlExpr counter = $7.getExpr();
if (counter)
   reportError(...);
j'ai rien dit xD
Dziwne wcięcia przy assertach.
It's adding as a development dependency.  Which you still need in order to properly develop and test Apartment, but it's not required as a runtime dependency, so people who install the gem to use in their app won't have to use Rails, just ActiveRecord.

This basically came about because I'm using a Sinatra web service to build a public API and I wanted to use apartment, but didn't want to use Rails.
Fixed. Dam'nt Mac French keyboard :(
it's a red herring
My setup automatically removes end-of-line whitespace.
`Date time`, LOL.
At first I was using the approach of an array starting with a index of 1. But that was kind of odd, because normally an array should start with a index of 0. So I think I'm going to stick with an array stating with 0.
This seems like a monkey patch. Will investigate this issue further, to see where it goes wrong.
It seems like converting the array to a set should count against the set benchmark.
thıs would fıx updateobject ın 43x because ı thınk value 44 = map ıf ım not wrong
Rules are now injected.
Woops, fixed ^_^
Sorry, i've been follwing this project for a while, can you explain me what SLATER is?
Dear reviewer(s) note that same method exists in DLFileEntryFinderImpl. I was thinking it is not good but still didn't know where to extract that method to make it available to others. Also, it would be great that such util methods are protected so we can easily write JUnit test for them.
Ideally, the app would remove subscribers who unsubscribe. This was my sledgehammer first-attempt at cracking that problem. Nasty, I know.
good catch, I hope you did not waste too much time debugging this.
Sorry about that :-( I did run the tests before pushing... Strange!
Yep, I'll do that. I also realized that the same problem appears in some other places
@sebastianbergmann @gsherwood I'm going to re-run CodeSniffer here and give the feedback as soon as possible. =)
@gsherwood @squizlabs Verified and working correctly.

Already closed bug #18717:  https://pear.php.net/bugs/bug.php?id=18717

Thanks a lot! =)
These lines should be linked up.
Not sure why it is there, it was there already when I made these changes.
I agree with Emully, that must've been a hard one.
I keep trying to read this as `MORE` ... something.
It was part of another commit.
Add you name to the license header
parece en el r2 lo arreglaron to eso XD
Glad you fixed the bug on `index.mapper.geo.GeoPointFieldMapper` causing infinite loops for lng = -360-180 = -540 (among other) because of the while test comparison being non strict.
However -180 and 180 can still be returned, they should instead be equal to the same value. Idem for -0.0 and 0.0.
Moreover, this implementation can be **very slow** for huge positive or negative values.

I suggest using Java's floating point modulus operation (see [the JLS](http://java.sun.com/docs/books/jls/third_edition/html/expressions.html#15.17.3))

```java
// Definition: floatingpointModulusCentered(x,d) = y such as y is in ] -d/2 ; d/2 ] and -0.0 becomes 0.0
// Note that -inf, +inf and NaN will yield NaN
// IMPORTANT: divisor is supposed to be a positive value. (A negative won't mess the modulus but will mess with the rest).
public static double floatingpointModulusCentered(double dividend, double divisor) {
        double rtn = dividend % divisor; // yield a value between 0 and dividend (or -dividend and 0)
        if (rtn <= 0) // get rid of the negative range for the positive one
                rtn += divisor;
        if (rtn > divisor/2) // transform the positive range to a 0-centered one
                rtn -= divisor;
        return rtn;
}
```
Cool. The state atom approach is definitely works beautifully when you have some state and just redraw the widget based when it changes. I'm still struggling a bit with places where state transitions have side-effects like add/removing/changing widgets. All we can do is keep experimenting and see where it leads us :)
I don't know what your code is doing but you need to do

db.open(function(err, db) {
  //if db leve otherwise db.admin().authenticate
  db.authenticate("a", "b", function(err, result) {
     // the rest of your app.
  });
})

if the driver is not started up correctly it's in an unknown state and any things fired during that period will be lost or cause undefined behavior.
Just thought I'd help out.
My copy is missing 116 upwards fork'ed found you already have them added 123 and 124 as missing :-D
jajaja estoy haciendo grandes progresos como veras...
Where did the "https" go?
いいような気がします！
Really can't wait for tomorrow! I did buy pickaxe, so many questions...
We don't need to raise here since we're already doing it in `find_article` filter
ben du coup je ne vois pas ce qui a été corrigé ^^
We should not that `FirstLast` and `EvenOdd` are no longer methods available within code in the upgrade guide.  The `API-CHANGE` comment on this commit doesn't mention that explicitly.
yep it does. See my previous comment
Even with a Close connection?
If I load a FileInputStream in this manner do I need to close it? if not won't an open filestream cause a memory leak?
what's the reasoning behind the second part of this condition? If we fire 'a.b.c' then we probably ought only have it firing on listeners tagged with each of those (and possibly more). So listeners on 'a.b.c', 'a.b.c.d', etc. would get triggered but 'a.b', 'a' wouldn't. The `this.namespaces.length === c` would allow both of these through. See note (that I'm about to write) on your last test.

Also, you can ditch the `if` completely and just return the condition: `return checkNamespaces.length === c`.
OTOH, :  )  in the case where someone just specifies `--upgrade-recursive`,  it is conceptually weird for `RequirementSet.upgrade` to be False with `RequirementSet.upgrade_recursive=True`,.  Maybe just fix the redundancy I mention.
We could conditionally display the app creation instructions only if it's your first time authorizing an app.
While I agree about global funcs, this is the most obvious place to put it until we have a plan for major refactoring.
sorry, i changed that variable name from _use_matrix_coloring to _supports_64b_atomics without changing the truth values, fixed in last commit to match the variable name semantics

http://buildbot-ocean.ese.ic.ac.uk:8080/builders/pyop2-testing/builds/433
Now that this function has two completely different operations it should probably be split into two functions (maybe load_config, and load_config_fragment?).
Are we using the Queen’s English now? Or is this a stylish aberration? ;-)
see sbt-site-plugin tickets for substitution
I think we should check who's Terminated here
badass so worstcase O(N+1) bestcase O(1) ;-)
In doc code we have switched to 2 space indentation for java code also, and line breaks after some 70 chars (unsure about the exact number).
This is due to limited space in pdf output.
You might want to check AkkaSpec.getCallerName for a quicker way to obtain this information.
Apparently some stuff in iterpase.pxi also uses `_MultiTagMatcher`, but I’m not sure how that translates into the public API.
What is X?  Typically not used to indicate dimensions.
No, it is not high. 20 is quite good because it will cover more acquisition schemes. Leave it 20 please.
Thankyou :-)
dint see any print statement here. May be you meant else where?
I don't think "TCP timeout" is the correct term here. Perhaps "HTTP timeout"? TCP is designed to allow connections to persist for long periods of time without activity. A web server like apache though could reach a user-configured timeout while handling a request, but this is not at the TCP layer.
Removed because it is useless in the root project. Kept in the zookeeper project.
think it'll erase a random entry without this if-statement? :)
Stream hints already contain width/height of current stream. Should be usable instead.
@Empact I'm not using shoulda at all.  `test_helper.rb` requires shoulda as of d8a950668878de619aebb30e7897a03d5c12716b.  This causes `rake test` to fail.  I expect that somewhere someone is requiring `bundler/setup` or equiv. which will change the load path according to what's in `Gemfile`.

I added this because just to make it easier for people unfamiliar with the repo; it's common to expect `bundle install && rake test` to be able to run the tests.

I have no idea whether the codebase uses shoulda elsewhere, so I don't know whether it's safe to remove the require in `test_helper.rb`.
It would make it harder to read/understand though...
SimpleDelegator is the magic that sets up __getobj__ and similar that you will see.  As long as you use methods appropriate the the objects you have, this passes them through.  You will see check of the type of __getobj__ to know what to call, for example look at to_s
no empty lines (same below)
I am concerned about that as well, and I'm working on simplifying it down a LOT.
That's another one that got in there somehow... I'll do that PR later. I've got simply way too many branches needing work.
You might just place this in `init`, because this method by default has nothing special in its implementation, and calls `init` anyway. Also, the if statement should be `if ((self = [super initWithNibName:nibNameOrNil bundle:nibBundleOrNil])) {` without empty lines between the brackets!
Sure, `CC.strip().split()` would be fine. Change it to that and I can merge. Have you signed the CLA?
```javascript
else
  assert(0); // or throw Error("unknown socket type")
```
grunt has lint and quinit tasks
"override" rather than "overrides"
Unfortunately this leaves a huge white scrollbar placeholder dividing the sidebar and content.

http://cl.ly/image/0t092d1q3q3b

I knew about the problem, but hadn't really got to a clean fix for it.

This also has the nasty effect of leaving a big fat scrollbar when the page is in lower-res mode (resize horizontally)

http://cl.ly/image/2b1X3z022a0P
Yep, also returns EINVAL in 10.8.
Снова что-то с табуляцией.
I think we should explicitly check for text instead of everything that is not an html. We should probably do it via a configuration option too. So people have a better control.
Remove this whitespace diff please.
Adding a parameter list (even implicit) can mess with existing code, e.g.

```scala
// 2.9.2
scala> Set[Any]().par(null)
res3: Boolean = false
```

But the alternatives aren't appealing: (e.g deprecate `.par` and introduce `.parr` or `.||`)
Will become easier once https://issues.scala-lang.org/browse/SI-6749 is fixed.
Learning my lesson from "val origin" I think you already know this, but from the days before macros were a glimmer in anyone's eye, in TreeDSL:
{code}
    // Add a null check to a Tree => Tree function
    def nullSafe[T](f: Tree => Tree, ifNull: Tree): Tree => Tree =
      tree => IF (tree MEMBER_== NULL) THEN ifNull ELSE f(tree)
{code}
Yes, much better. Will change, also for ``test_vq``
This function apparently expects integer arrays with non-negative values as input (otherwise ``log2`` call fails), but this is neither documented nor checked.
Also worth pointing out that trace has another different (but mathematically related) meaning in algebra.  See http://en.wikipedia.org/wiki/Field_trace.
As this is self-evident, this comment can be deleted.
not sure this is used anywhere. Maybe we should avoid adding it to new functions...
>Maybe I simply do not understand the notation. I suppose that ^ is for upper indices and _ for lower.

Yes.

> And what about indices that are not grouped by upper and lower? For example Tuluul where u is upper and l is lower. Is this supported.

Yes, for instance in the second example in `double_coset_can_rep` `T^{d3}_{d1, d2}^{d1}_{d3}^{d2}`
I did miss that link somehow. I was not aware of such a use case. So yes that facebook example makes sense - someone could have a custom registerVideoTags() method that calls registerMetaTag() several times. Sorry for not getting it earlier.

In regards to the defaults: Since the index is removed, it now makes more sense to me to have an optional $key/$id like qiang suggested.

````
// default
$cs->registerMetaTag('example', 'description', null, array('lang' => 'en'), 'description_en');
// replacement
$cs->registerMetaTag('123456', 'description', null, array('lang' => 'en'), 'description_en');
````

What you think about this @klimov-paul? An additional method still seems too much for me.
Good catch.  That's some cruft left over from development.
Turn that back to two spaces
Well it formats the ctor to look like:

```java
@Inject
public MyBundle(
                 Provider<Foo> fooProvider,
                 Provider<Bar> barProvider) {
        super(2);
        ....
}
```

If it is indented once it looks like:
```java
@Inject
public MyBundle(
        Provider<Foo> fooProvider,
        Provider<Bar> barProvider) {
        super(2);
        ....
}
```
Usually the ctor arguments are indent further, but in generated code it does not really matter, so I don't know :)
Use `changeType` instead of `type`. The variable `type` is reserved for Model classes throughout Ember Data.
1. Where does @tcp_client gets updated?

2. Should it be: "#{host}:port"  - (" and not ') ?
mkay...?
I remember trying to remove the if and it errored out somewhere
Indentation ....
Library upgrades should go into the release notes. Speaking of release notes, they don't seem to have been written for this patch.
Rule of thumb: oplog should be long enough to hold all transactions for the longest downtime you expect to take on a secondary.  Operationally, I'd say that 24 hours was a minimum; 72 hours a good value; and lots of folks run with a week's worth of oplog, just for good measure.
s/command/operation/

s/:program:`mongo shell`/:program:`mongo` shell/
This is confusing. It can be used while in fsync/lock state. The confusion comes from replication and writes which will block reads (mongodump) and your backup.  Newer version of mongod will not cause this situation but I don't think the change made it in to 2.2.0 so currently it may be true that once you fsync/lock + queue a writer that all read will block -- it was true pre 2.2.0 for sure.

If you are using mongodump then you should not use fsync/lock.
The edit to this section obscures a fact, and so should say: 

.. will be able to use this index for a larger portion of *these* operations.

(i.e. For the queries that include both fields, with a compound index MongoDB will be able to use that index for a larger portion of *these* queries.) Without the "callback" it sounds like MongoDB can use the index more often, when really, MongoDB can only use the index for a larger portion of these specific queries, without needing to scan as many documents (which is comparatively expensive.)

I'm not sure of the best way to convey this concisely, so feel free to experiment, but it's an important distinction.
``records`` in the process.
We shouldn't see Dir.pwd be called, so maybe this should be Dir.should_not_receive(:pwd)?
Just for my own edification, why respond to `:[]` over `is_a?(Hash)`?
Fixed everything (I think)
add_sources, etc.
I disagree, this is adding a new feature but adding an option for it to be disabled.
These are not very long lines, so I do not see the problem. If I put them on their own lines these 4 lines will take up 11 lines instead.
As this represents the post ID it should be named `$post_id`
How is one any different than the other?
I don't think this is required for UPDATE queries.
"has" is a verb, a form of "to have". But it is not a complete sentence, so that should probably be changed.
let's try and use string.format or the fmt() alias
removed. no one is listening for check.run
<pre><code>
{- | This module re-exports "Data.Time" wrapped in 'LIO'. It is important to note that this module is only safe with the latest version of "LIO", where @toLabeled@ has been removed and timing attacked have been addressed. In similar vain, when executing a piece of code that you do not trust, it is important that the time primitives not be directly available.
-}
</code></pre>
Така теста е куц. Ако се промени името на класа, получаваш false positive. По-добре е да stub-неш конкретен факултетен номер и да проверяваш дали него го има.
https://github.com/jeanphix/django-rq/blob/master/django_rq/tests/tests.py#L125

for now get_worker creates a worker for all queues...
if you are using bash or the command line you do :

git add /path/to/file/
git commit path/to/file -m 'added file'
UserLoggedInRedirectResponse is a static method so I can't access FormsAuthenticationRedirectQuerystringKey property from there if it is non static. Is there a way to get FormsAuthenticationConfiguration from NancyContext?
kept. added note about ticket
please dont pull request with lots of commented code. it's better to keep this in your local fork / branch.
should be camelCase, ```getBillItemId```
I think this should be ```iris.exceptions.TranslationError```.
I think maybe I didn't explain properly what I meant. I understand that ``typeConstructor`` is supposed to be called only with dtype='float32', but then I think there should be an assert inside this function (the one defined in ``theano/sandbox/cuda/opt.py``) to verify that it is true (instead of doing the same thing regardless of the given dtype).
My reasoning is that if there had been such an assert before, the error Mark has been facing would have been easier to identify.
Minor: jar file name has gotten out of synch with test case name.
This procedure will overwrite the 1st long in the VALUES column.  This 1st long is where the count is stored. This is probably the cause of the strange looking values in the log output I showed you.
Whitespace damage? Unrelated even so, so no thanks.
should be moved to PaginationMixin
WRONG DOCSTRING
also test a failing POST
Um, when?
m_arenaPoints is of type uint32, please don't use %i
Are you sure this piece of code does not belong to BeginSaveAsFile?
Yes, you're correct. Reject this chunk.
* だいぶ良くなりましたが、まだまだ抽象的です。カレンダーのフラグというのは分かりますが、じゃあこのフラグがどうなら何？ということが読み取れません。Flagということなので、値はYESかNOが入りますよね？例えば、YESの場合を変数名で表現すると良いでしょう。
Two reasons:
1) Using "fid" as distribution key will cause data redistribution, since the input table is distributed by id. Especially for the big dataset (such as kddcup, it has about 5M rows, and 40 columns), the redistribution is very expensive according to our test. However, using "id" as distribution key, there is no data redistribution.

2) If the number of columns of the input table is less than the number of segments of GPDB, then the data can't be distributed among all segments.
Can we distinguish convex/lmf.hpp and task/lmf.hpp a little more in filename?  Whenever possible I prefer keeping unique filenames in a project to reduce confusion.
CYGWIN still needs to be tested if someone is up for that, but I'm supposing it would work.
Good point! Corrected this in commit: https://github.com/monlog/monlog/commit/44ddf00205f1ee96b949778a0ac258cfa755794b
On OS X, NSWindow is not a subclass of NSView, so in order to serialize NSWindows, we have to pass in an NSObject instead. Since NSView is significantly different than UIView, and NSWindow different from UIWindow, I felt that preserving type checking on iOS was necessary in this case.
Where is browser_version defined before this line?
1. Yes, I did mean `auto_now_add`, oops.
2. The replaced code has the same behavior (set to current time when making a new object), except it can be overridden when creating the object. I think this is preferable, because with `auto_now_add`, specifying a non-default creation time **fails silently**, which is very confusing.
Will do so in next commit.
Needs a stable download; we aren't accepting head-only formulae into core anymore.

Also the `:using => :git` is superfluous.
Can't assume `/usr/local`, but if you use `HOMEBREW_PREFIX/etc/anacrontab`, Homebrew will do string replacement on `HOMEBREW_PREFIX` before applying the patch.
I know it looks clunky, but after the possible return >

else:
    data = self.migrate_object(cls, data, data_version)

instead of just >

data = self.migrate_object(cls, data, data_version)

is easier to follow - particularly given the loop & if
Yes, but in this case the error only have a message.
I think an empty implementation for this method needs to be present in the ``Layout`` base class for 2 reasons:

1. To document the method signature
2. To avoid generating errors if a new layout implementation is written and not all needed methods are implemented
jopp, die reihenfolge wird hier dann hergestellt... warum ordner nicht gelöscht werden schau ich nach
It was like that before, and there is a lone of them like this, shall I change them all?
should we have two different registers? One for subclasses and only for only each class match?

My concern is that due to the Java type system, you can't write a codec that can't deal with subclasses (in scala, you actually can using an implicit ev: =:=[A,B] to see that type A is equal, not a subclass, of type B).

Therefore, there must be cases where there are Codecs that shouldn't be used with subclasses, or we don't want to use them that way.
use example hub id from docs. make global constant.
Use memcpy
possibly yes, see my next commit.
You're right. I've changed it.
@dsaff 
Are you keeping the track with our discussion. I like these discussions because we can uncover a lot. Lets bring a statement after @kcooney . 

Thx.
Section 4 of the [Coding Guidelines](https://github.com/OP2/OP2-Common/wiki/Coding-Guidelines) states the function signature would be on one line - but I'm not too bothered.
This was the only thing I took from the previous implementation of Block-comment. I'll use `Ctrl-Shift-/` for both
This line now has nested single-quotes. Restore to original code.
The reason I decreased the font size is because the "pills" (numbers in ovals) start to float drop to the next line for some test names that are very long. This gets worse when there are partial errors because it adds another pill to the line.
 
The reason I didn't want to increase that column to span5 is because it also requires that the right column (where results are shown) is decreased to span7.

When do you see the font size change as jarring? It seems in most cases that someone running the test will either keep it as it is by default, or make the window wider, which will cause the font size increase, but doesn't seem like it would be "jarring". Maybe I'm missing a case.

Another option would be to always keep the font size at the smaller size.

Thoughts?
Added an optional url parameter. Defaults to window.document.location.search.
Array accesses aren't range-checked in JS, so if index is out of range we'll just get `undefined` back as the result of `lastFilterResult[index]` -- which seems like the right thing to do (analagous to the return-null case at the top of the fn).
@joelrbrandt Promises don't have that problem in general; it's just jQuery's rather unfortunate implementation. Consider using Q promises. Whole debate here: http://bugs.jquery.com/ticket/11010
well the thing is that normal copy has the same behavior as duplicate, you see the underline files twice when you paste them
because that's what we do
Dude I probably dropped and re migrated the database 15 times and
couldn't get rid of the user with ID 1 not found bug.  That fixed it;
I can give you the Stack Overflow link.

Dan



On May 9, 2012, at 6:25 PM, "Gareth G. Aye"
<reply@reply.github.com>
wrote:

>> @@ -6,5 +6,6 @@ class ApplicationController < ActionController::Base
>>
>>   def current_user
>>     @current_user ||= User.find(session[:user_id]) if session[:user_id]
>> +     rescue ActiveRecord::RecordNotFound
>
> Kill this. If there is a user_id key in the session hash, then it should be in the database. The only reason you'd ever lose it is if you change the database in which case you'd want to have the user log out.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gaye/rideshare/pull/22/files#r798494
Even for a private inner class, a short description will be welcome.
Oh I see, you've moved to the modern directory contents method which takes care of this feature for us. Let's just drop this line of code then.
или blockinflate? или blockinflect? :)
No worries. Ruby is more familiar to me now than C, but I rewrote in C. Hopefully it's not too offensive to your eyes.
I don't see it fixed?
How to commit this file? Since it is compiled, it won't do it automatically is there any way to override this?
For convention's sake, mind making this `_tracksitionsSupported`?
I didn't realize we already handled textboxes and dropdowns, so this looks great.
I send back an error so the app can keep moving instead of blocking on waiting for a callback from a save action. Unnecessary?
I try to use a rule of 3's to decide things like this. If it appears in >2 places, put it in a function.
Replicated fixes mentioned above.
This is *actually* nothing that uses `comm` anymore. Nothing. I checked. I would be ok with removing it and its tests entirely. :)
Here you use 5, which I think is better, yes.
Is it possible to get `timeupdate` to occur without touching `play` or `seeking`? If so, it'll be a problem to remove this line.
e.g. `http://www.lollercats.com/flicker.com/photos/987129387123` or something.
After "follows: ", can you put the `template-name` definition on a new line? It does look so understandable here: https://github.com/humphd/butter/tree/t2340
This does not work (nothing happens). `id` is a valid `id` attribute (if I use `#` links directly it works, but it becomes hidden behind the navbar.)
Should return null for all paths.
now checks the error codes of input body filters more carefully to avoid socket leaks.
```js
return a.slice(from, to).join(separator);
```