labofsahil/github-event-dataset-2012 huggingface dataset:
One more version of implementation:

    recipeList:
        - mysql:
            tunable:
                max_allowed_packet: 1
            server_root_password: qwe123
            bind_address: 127.0.0.1
        - yoursql:
            tunable:
                max_allowed_packet: 2
            server_root_password: secret123
    nodeAttributes:
        tunable:
            max_connections: 800

What about that?
if str[0 didn't get catched in the switch block above, an invalid row would be saved. Perhaps you need additional check here to see whether the row.version and row.name has any valid content.
In [2]: response._headers
    Out[2]: 
    {'content-type': ('Content-Type', 'text/html; charset=utf-8'),
     'location': ('Location', 'http://testserver/en-US/editors/queue/pending'),
     'strict-transport-security': ('Strict-Transport-Security', 'max-age=2592000'),
     'vary': ('Vary', 'X-Mobile, User-Agent'),
     'x-content-security-policy-report-only': ('X-Content-Security-Policy-Report-Only',
                                               'policy-uri /services/csp/policy?build=5d5d'),
     'x-frame-options': ('x-frame-options', 'DENY')}

so you would do `self.assertRedirects(response, reverse('editors.queue_pending'), 302)`
When the thread is interrupted, the connection is not properly closed. That seems to be the cause of some ClientRequestExecutor leak as per the java dumps I have.

As a side effect is producing a "Too Many Open Files" problem given enough time.
this is unrelated to the pull request's purpose: to proxy eval()
Right, but `by_dataset` won't contain any keys with empty lists as values.
It's useful data for the support role. Either it says everything is set up correctly for paypal, or it displays what's wrong, like permissions not set up, or no permissions to do refunds, etc. Things a support person should probably know to be able to do support.
内存泄露啊，你为毛非要new呢.....
Pour Dorian et Denis : Il faudra mettre ça en fonctions pour que ce soit compatible avec le reste du programme :)
Exactly
On Jan 20, 2012 9:49 AM, "Per Djurner" <
reply@reply.github.com>
wrote:

> Good idea, similar to how we test redirects, right?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/cfwheels/cfwheels/commit/2289bcf1dc359eda3d391891712c405070a53012#commitcomment-886230
>
think do not need this condition?
fair enough. Feel free it to do that after the merge..... :)
This is a bit scary, but I'm running the full test suite now to see if anything breaks from this.  In code, there are no more "Editors:*" rules, so whatever does break should be updated/fixed.
Excellent choice of method :)
Здесь ```reverse``` не надо делать. Если тебе нужно сортировку сделать, то лучше сделать ```default_scope```  в модели ```comment```, как это сделано в ```page```. Либо в ```page``` в определении ассоциации ```has_many``` прописать``` order```
<?xml version = "1.0" encoding = "UTF-8">
Confirm. No work. How to download? :D
whoops thought this was trinitycore commit :D sorry.
@spicyj This logic basically says if you enter 0.5 and the correct answer is 5, it'll try putting a comma in there that later gets ignored and the answer is marked correct.

I think that's what's causing the issue in #10100.

Seems like there should be some more complete i18n library that'll do this for us correctly, but I have no idea.
maybe this should even be left up to each individual model to decide that they have or not (much like FriendlyId does now):

    class MyModel < ActiveRecord::Base
      extend Videoclip
      has_video *opts
    end
`import org.cain.cmdbin.listeners.*;`
Me too, but I couldn't work out how to force the inclusion of those functions from library.js. What do I need to do, to ensure that those functions are included whenever the intertyper gets to a landing pad?
Just earning my commit bit ;-)

I'll have a new play soon, when rack calms down. I kinda want to run through the zguide and implement every one of the patterns. I've done MDP, ADP and Titanic before with something resembling the ObjectProtocol that's in EM (but way earlier).

I think @erikh implemented something similar to titanic when he took over one of my older commercial projects too.
That's how it has always been named. the name transformOutgoingMessage does sound good.
shouldn't this use the ``:class:`` syntax ?
Sphinx が正しくパースできるように、「、」と「``」のあいだに半角スペースを入れていただけるでしょうか？
I quite liked having the version number in there. Do we want to avoid it?
This is not needed, because there is no 'news' type in the feed. Currently all news is pulled through the twitter aggregator account :-)
I think that's not a good way to do it.  I think there should be a separate help message for the password field than the validation error message for an empty password.
Cool. Send one for that too, if it works out. 

Glad people are still finding it useful.
This commit fixes a bug reported by @spiderr where API::getFile() was expecting a file parameter. This only occurred when a call was previously made via POST (i.e. acquiring an access token) and was caused by the default cURL options not being reset between requests.
I'm non insisting, but using sprintf() might be just simpler.
render-preRender    

1  2012 . 14:20  Denis Rykovanov <
reply@reply.github.com
> :

>             
> ,            
>      ?     
> ?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Etherlords/reality-0.1/commit/8fb8a5543d7b2cf0ef34022cdacee1f5a3352a56#commitcomment-922269
>
This is not fully correct.
This would make it easier to create mobile versions of a website. As a developer, it is easier for me to decide by domain which version should i show. And i can give the user a chance to decide if he wants to see the mobile or desktop version. If i use media queries the user has no decision. If i use a second page tree, i have more editorial work.
I know this would generate a duplicated content problem, but for complex website construcs there are more positive arguments for this feature, than negative arguments i think.
Forgot to delete this line?
Well, in my eyes the possible options are:

```ruby
check_at = upcoming.delay_minutes.minutes.since((Time.now - issue.posted_at).seconds.ago)
Delayed::Job.enqueue(EscalationJob.new(issue_id), 0, check_at)
```

which I think is convoluted, and my current solution on the other hand. (Btw, Time + something is only defined for a something of type `Numeric` by itself and not further specified in Rails itself, it seems.)

Also, I still don't fully understand your suggestion, I think. For using the `minutes`, `seconds` etc. magic I already have to know which dimension my given number is in, if it's a variable, otherwise the result would be pretty random.
It's actually quite tricky to strike a balance between the use of `pickle`—the `#{capture_model}` stuff and the automagical _Given a xyz exists_ steps—and less generic steps that are more verbose and repetitive. Took me half a day to figure out just how `pickle` works, what I can really use it for and what not.

Writing more features will hopefully be more straightforward now.
var p
currently elevator is in move by default, that's why I removed this code.
I don't know how to make this elevator stop. gameobject_template is the same in UDB and in YTDB, this  must be related to code.
OK, I consider the port done now as of the latest commit.

Both the 'libsnes' and 'ps3' directories can be considered mergeable with Genesis Plus GX. You will have to import the two Makefiles too - Makefile.libsnes - for the libsnes ports - and Makefile.ps3 for the standalone PS3 port.

I've done as few code edits as possible - I did have to make a few ifdefs here and there though - here are the key defines to look for -

(two underscores)CELLOS_LV2(two underscores) - PS3 port define

(two underscores)LIBSNES(two underscores) - Libsnes define

I might follow up on this with some MSVC-specific defines later on. Just let me know if you have any problems to do with merging this.

A couple of comments -

1 - I had to make the system hardware detection based on file extension optional in load_rom - on PS3 I have to perform this step before calling load_rom itself, and in the libsnes port I have to do a similar thing. It has been split up into a separate function called 'detect_system'. It also now recognizes upper-case ROM path extensions in addition to lower-case ones.

2 - What does __extension__ actually do in osd_cpu.h? The MSVC build is tripping over this - I don't think it's necessary. I ust blank it out for now with a MSVC compiler global define in the solution. Perhaps you could get rid of it altogether though if it doesnt' serve any purpose.
After perfoming extensive tests to the warden system, I found a configuration that fitted perfectly: Warden.ClientResponseDelay = 30 and Warden.ClientCheckHoldOff = 5, this config takes around 1-2 mins detection which works for me, now another doubt assaulted my mind. Are there any other 3rd party non-cheat programs I should know of that might trigger a check failure thus leading to an unfair account ban? thanks everyone for responding btw.
Its bcz server send some cheat data, wait for client reply, send cheat data, wait for client reply, etc, etc, etc.

So, sometimes this can take 5min or more to detect a cheat :\
If shuffle wasn't implemented by converting to a grouped sort, then it would be possible to implement a native shuffle activity which worked on grouped data, and shuffled within a group.  The code generator can't guaranteed that I can restore the original groups so I can't implement it while the implementations are non native.
I don't know if it's in any spec, but I think the "Proxy.for" syntax is being abandonned in favor of the more natural:

var p = new Proxy(target, handler).
So a more relevant test would be: (typeof Proxy === 'function')
Esta sin meter desde que se hizo el merge.
https://github.com/Radicalmangos/Radical/blob/master/src/game/Player.cpp#L11970

A ver si lo podeis cambiar ;)
what?. o.O
Hi!  Heh, could i ask you to submit also through click wrap license http://www.antlr.org/misc/feedback to keep license clean? then i can integrate.
Ter
Missing 'usage'
Also, why is the lock release not related to a specific key? It looks like we're awaken by the first key release.
Krati is a great KV store. I didn't run performance tests on it yet, but I know for a fact that it provides **great** value to LinkedIn. 

You can see a krati/bdb comparison on http://sna-projects.com/krati/slides/krati_vs_bdb.pdf
Thanks, good commit!
@xcambar: I'm not sure I follow your comment, but the global registration needs to be there when an amd loader loads underscore and another script that depends on underscore but does not been call define() for their dependencies. For instance, backbone/master. 

If underscore only calls define in that case and does not export a global, then an error occurs. It is also useful when multiple versions of underscore could be loaded on a page (even outside an amd loader), but the loader may only want one of them. If the underscore definition disappears into define, another script that wanted to use it as a global would have a problem. 

Basically the same forces are at play for underscore as they are for jQuery, so the same pattern is used. Most other libs do not have this problem though. It just happens to be that underscore and jQuery are really popular, and modular loading is really new for most developers.

If you want to talk about it more, it is probably best to do outside of this commit, to avoid the extra noise for the underscore collaborators.
Wow. That's crazy.

Thanks for the links.
Thanks :) I'm agree, I need to make my code more drying, I tried to avoid inserting logic in views. Perhaps I can use your approach using DB calls.
() is needed
I think you should cherry pick this commit also in the 0.10 branch if you want it in the release (not sure if @fabianp would otherwise to that?).
If it's in both banches I'm pretty sure Fabian will take care of the rest when building the docs.
Yeah i should clean it up a bit...but for now i don't care as far as it
works fine
Il giorno 26/feb/2012 00:14, "adridu59" <
reply@reply.github.com>
ha scritto:

> I see some unproper stuff here...
>
> for example '#include <linux/version.h>' should have been removed.
>
> You might want to try the one on my repo, is slightly different of your
> and works perfect so far. :)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/simone201/neak-kernel-sgs2/commit/6f8f76b347fac4d69a9a4331dbf326576a76f8f3#commitcomment-1011613
>
not finish yet, be patient :)
Fantastic news, thanks for sharing !
Thanks. This will meet my needs but it doesn't quite comply with the animate specs. They say the complete function is called in the context of the element being animated. Should the call be opts.complete.apply(el, null)?

Not that I need it to be that way but that seems the most like what animate would do.
Excerpts from ZyX-I's message of Mon Feb 27 22:23:23 +0100 2012:
> > And it is debuggable if it ever happens. A simple grep for the key is enough.
I've reverted my patch.
Why the change?

Foursquare says the page is still /oauth2/authorize
either take out the "the" before current thread, or add a "the" before main thread.  I prefer having the "the"s.
sfsafasfas f
I don't really think so since the PCMD commands are sent to the command port and not the config port ... Maybe it's the overwhelming data the drone gets when PCMD commands are sent without a threshold between consecutive commands.
The multiplication on this line isn't adding the tax onto the total. You should be multiplying by 1.13.
`number_format()` requires a few more arguments to write the proper number of decimal places you're looking for.
`'href': '#'` должен быть заменён на страничку с объяснением как пользоваться шаблонами.
It seems to work under 1.8:
```
Using /Users/jason/.rvm/gems/ree-1.8.7-2011.03
extract:~ jason$ irb
ree-1.8.7-2011.03 :001 > %w(foo bar baz).map(&:to_sym)
 => [:foo, :bar, :baz] 
```
As has already been mentioned, the Jenkins angle is irrelevant. Code is written for human beings not continuous integration tools.

As for "evaluate things differently", almost all of them suppress important details and/or render the comments nonsensical (FIXME describes a problem; TODO describes **the opposite** - a fix; they're **not** interchangeable). I've reverted most of them. Feel free to restore any changes that were motivated by actually clarifying/improving the code rather than "appeasing a reporting plugin".
おつです！
What is the behavior if ClassRegModuleInfo.open_class_registration == False, or if there are no class sections with section.parent_class.category == esp.program.models.class_.open_class_category()? The behavior should be "do nothing," since that is what we want for programs like Stanford Splash and HSSP, which don't have walk-in seminars.

I can't tell whether or not that is what will happen.
we are planing to move to strictly follow semver while will look something like 6.0.0-alpha5/beta the only way to differentiate between the new version is to use AssemblyFileVersion. We are also trying to align it to the same version in nuget.
Haha my bad

On Jan 17, 2012, at 19:27, Rinat K<reply@reply.github.com> wrote:

> that was the only thing i found, since you removed action_admin_init it was throwing up notice and sorry about new lines :) my laptop Komodo IDE had that "trim trailing whitespaces and EOLs".
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/danielbachhuber/Ad-Code-Manager/commit/516430126c97a3c1d56fe1b390426ef68b80b59d#commitcomment-876857
Thank you both for reacting so swiftly!
¬¬ And the others languagues
a scalar defaulting to an array ? really ?
But my DB is diferent... ¬¬
Crap, sorry!
yups, but dont forget use framework base, from https://github.com/GioICS/Android_Frameworks_Base
Hi, DerekJay, I was trying to use this commit in my kernel,  , but i keep getting this set of build errors:
drivers/mmc/core/mmc.c: In function 'mmc_suspend':
drivers/mmc/core/mmc.c:558: error: 'init' undeclared (first use in this function)
drivers/mmc/core/mmc.c:558: error: (Each undeclared identifier is reported only once
drivers/mmc/core/mmc.c:558: error: for each function it appears in.)
drivers/mmc/core/mmc.c:558: error: expected ';' before 'err'
drivers/mmc/core/mmc.c:565: error: 'err' undeclared (first use in this function)
drivers/mmc/core/mmc.c:572: error: expected expression before '}' token
cc1: warnings being treated as errors
drivers/mmc/core/mmc.c:572: error: no return statement in function returning non-void
oh,yeah!so cool
"checkout", c'è un typo. Inoltre preferirei non fossero globali.
it's very dangerous. just remove element with "countrie" class name and your code will be broken.
Which version were you trying? You should be able to compile with Java 1.6 or 1.7.
Ok. I'm finished with commenting on your comments. It's a **base**, nothing else. Please invest your time in doing a fork, adding what you think is valid, do a pull request, finito. It takes the same time as commenting in here and if you've insights or find missing stuff: add it. The idea of open source and git is to collaborate on code and not (only) to discuss in comments.
just change to:  `posixpath.join(main_index_url.url, version) + "/"`
@agronholm  or @iElectric  you want to handle this in a new pull?
this are less concerning though, because often these are 404's anyway.
If I don't get a response, I'll just close for now, since part of this was already handled in pull #695.
The lookups are handled by the MCP. We wouldn't want to do this in the parser because the parser also runs inside `tronfig`, which may not be on the same machine.
wouldn't it make sense to have two methods?

    setReceiveTimeout(timeout: FiniteDuration)
    clearReceiveTimeout()
On 2012/05/31 07:46 AM, patriknw wrote:
>> @@ -52,7 +52,7 @@ class RandomRoutedRemoteActorSpec extends MultiNodeSpec(RandomRoutedRemoteActorM
>>    def initialParticipants = 4
>>  
>>    "A new remote actor configured with a Random router" must {
>> -    "be locally instantiated on a remote node and be able to communicate through its RemoteActorRef" in {
>> +    "be locally instantiated on a remote node and be able to communicate through its RemoteActorRef" taggedAs LongRunningTest in {
> it's a MultiNodeSpec and should therefore be tagged as long-running. It's that in master also.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/akka/akka/pull/476/files#r905537
okay, thanks for that
How about user passing in a Promise[CurrentClusterState] as the callback, and then they can just add callbacks to that with whatever EC they need.

    val p = promise[CurrentClusterState]
    p.future.onComplete { … }
    cluster.registerOnMemberUp(p)

Then it hooks in with existing Future/Promise machinery, and it is clear that the same rules for closing over state applies to the callback/thunk passed into the registerOnMemberUp call.
As I wrote in the commit comment:
* It's no problem using after, but scalatest will output the test method
  as completed (green) before running after, so it looks confusing in the logs
* Using unique barrier names adds extra traceability in case of failures.
I guess you know what Donald would say ;-) (and for such a small match at that)
How important is it that we see the updates and that they are in synch (consistentHashRoutees and consistentHash match up?)

If they need to be in sync, then put them into a Tuple in an Atomic, and get them in the beginning, and if discrepancy between routees calculate and try to CAS, if fail just use what you calculated or reread.
Sorry, I was looking at the wrong word... yeah, for the "may" in line 82, I'd leave it as-is.
but without uniqueness types :-)
good, should we also clarify, if not already done, that `watch` of remote actor will not generate `Terminated` if remote jvm crash or network drops?
I was thinking about going the other way and adding some annotations for the JUnit tests and enhancing the JUnitSuite so it produces the same tags `"timing"` et.c. so we can classify JUnit tests as well.

We definitely need to discuss where we want to go.
So we give up permanently after the seed node time out?
thanks, will fix
the MIGRATE_PERMISSIONS stuff is gone now.  is that what you mean?
I debated this.  Most likely the plugin (gofer) will need to override every method in gofer.lib.conduit.Conduit but /may/ not have to.  Seemed proper to subclass here since the handler APIs expects a conduit that isa gofer.lib.conduit.Conduit.  Since python is not typed, we can go either way here.
strange line break.
that's how your add() method works. It is more consistent and convenient to have everything append to the same StringBuilder
This is wrong. This will read the composer.json of Behat (which does not rename the vendor dir), not the one of the project
It's easier to write "with pkgs.lib;" at the top of the module.
That line can be removed now, the amount of code shared with that has become insignificant. :)
The bounds need to be preserved: use (i think its this) builder.add(Real.valueOf(1 - position.doubleValueOf(i), position.boundsOf(i)));
same for the builder.add below
The layer proxy is just a one-line accessor to the internal `indicator.layer`. It just lets you do fancy things.
But isn't this how NSPopover defines it?
I could work on that, but it's out of the scope of that commit/this PR. I'll open an issue.
Style: `window.frameOrigin = o`
In that case, the static function can accept pointer arguments which it will set upon return (similar to how `NSErrors` are typically returned in Cocoa).
You can actually add an empty implementation here to be the default. Scala's traits are not pure interfaces -- they can have implementations for some methods. So just

   def postStartHook() {}

And now only YarnScheduler has to override it.
It's covered - see for example https://github.com/nipy/nipype/pull/492/files#L2R328 The template just needs an extension.
The leading space before description_len is wrong here
Угу. Тернарный оператор, на мой взгляд, лучше использовать только в выражениях, заменять if им не стоит.
К тому же не очень понятно поведение этого кода при изменении типа возвращаемого значения одной из функций.
Do you need to reimplement the update using arel, or can you use `update_all`?
I think I'd remove this one
How about adding something like

    // A precaution against future additions to FlagsNotPickled turning out
    // to be overloaded flags thus not-pickling more than intended.
    assert((OverloadedFlagsMask & FlagsNotPickled) == 0, flagsToString(OverloadedFlagsMask & FlagsNotPickled))
The section title should also be `Attributes` so that it interacts well with the documentation machinery. You can mention above that there may also be solver-specific attributes there.
I feel like this is a bit too messy, just let the coupon input remain styleless. Leave it up to the dev to create styles for their specific theme.
On 23.05.2012 00:39, Aaron Meurer wrote:
>> +>>>  F
>> +    QQ[x]**3
>> +>>>  F.contains([x, 1, 0])
>> +    True
>> +>>>  F.contains([1/x, 0, 1])
>> +    False
>> +    """
>> +
>> +    def __init__(self, ring, rank):
>> +        from sympy.polys.domains.polynomialring import PolynomialRingBase
>> +        FreeModule.__init__(self, ring, rank)
>> +        if not isinstance(ring, PolynomialRingBase):
>> +            raise ValueError('This implementation only works over '
>> +                             + 'polynomial rings, got %s' % ring)
>> +        if not isinstance(ring.dom, Field):
>> +            raise ValueError('Ground domain must be a field, got %s' % ring.dom)
>
> Why this restriction?
>

This class implements actual algorithms (syzygies etc), and I know them 
only for fields. In fact they use groebner basis computations, which 
sympy only implements for fields.

There are possible extensions of this, but it fairly is non-trivial.
ok, reformatted the dictionary.
if you wrap the lhs in signsimp this works.
No tests are broken with the changes made. Please see #1505 where

```
>>> a, b = S(3)/4,S.Half
>>> gcd(a,b)
1/4
>>> gcd(a*x,b*x**2)
x/4
>>> gcd(a*x,b*x**2/1.0)
0.25*x
```
The one that does not take a RoutePattern: `not_found`, to be consistent with Sinatra.
The one that does: `matchAll` or `all`.
Ah, I thought since #1628 is "just" the PR, I could leave it out. Adding right away ...
ae801f5
Did the first one.
since SessionData implements Comparator<Object> don't we not need to put in an Override? It returns an error when I do put one in.
Loads of these whitespace changes.
This would also make it easier for Metrics to be composed of other Metrics. The parent would call `#each` on its children passing `block` instead of merging all responses into a single hash.

On that note, is there a reason why things like `Histogram` and `Meter` keep their own counts instead of delegating to an internal `Counter`?
Should I start the `if` block here ... or can we skip the whole transform thing altogether and put the `if` above `var t  = ` ?
Preferably embed it in another struct and use container_of so you don't have to go pointer chasing.
I've split the original test into three separate tests.
if we're going with collection named ``things`` then we should state assign .. to the variable named ``c``
should we *require* the If-Match etag to be sent to ensure we don't collide? I sent requests with no etag at all and they blew away changes made by the site.
on centos/redhat, pciutils package installs lspci in /sbin/lspci
This is about the shadow topics. I don't really know about why that code is there and what is that code for, it isn't mine, ya see. I just placed the filter where it needed to be.
`request_var` is deprecated.
Should be wrapped in `/* */` style comment per the coding guidelines
Why @ here?
About 2: Removing should be fine.
will then be
Actually, $phpbb_root_path may not need be prepended at all. I added it when I was trying to fix the broken chunk uploads but forgot to remove it.
this looks unintentional?
each check implements a timeout. i am unsure what this timeout would be.
debug stmt.
* gcc 4.3+ is required (min msvc version?)
* gcc 4.7+, msvc 2010+ is recommended
Did you decide against moving this into `post_init` or just didn't want to do it in the auth feature?
Else condition with nil is pointless. If the condition is not evaluated, the result is nil.
Also, you can condense everything using a block

```
node(:registrar_id) do
   # ...
end
```

The same principle applies to other methods.
this should probably go into the staticHandling on environment:dev section.
`RewriteCond %{REQUEST_FILENAME} !-f [OR]`
Let's remove these commented out lines, or add an additional comment for why they're commented out.
Failed review then ;)
This map has already been merged into OpenRA.
LOVE that sencha can do this :) Good job making use of Sencha
Style: please name with more clarity on its usage, rather than abbreviation
allow no argument to be passed to method `$format = null`

`@return string`

```
// formats as config, first one is default
$formats = array('json', 'xml');
$format = !in_array($format, $formats)) ? reset($formats) : $format;
```
if you edit the file anyway - put an assert in here that the compute_map is 1 at [src], not e.g. 0 or 2
I'm not sure what happened here.
I don't remember and can't find the information It work without the try:except, so I removed it.
зачем ты bundle update делаешь? не трогай версии гемов.
Fix it already, thanks.
This will definitely make it hard to filter by projects and tags
unicode(x)
@jackpoz Ok, now I see why it is a stupid idea to use * / or */ or whatever in a comment... Thank you, again!
`PATH.resolve(__dirname, 'data/util', file + '.bemjson.js');`
Для чего понадобилась это проверка?
I told you ... !
If we stop storing auth and cookies our shared sessions, it seems like they lose most of their value
you should simplify it as the adapter is optional in the browser constructor (it will create the curl adapter in this case)

And you should add a setter for the browser to be able to set a pre-configured browser when you don't want to use the default config
As a general rule, it's better to use `something is not None` rather than `something != None`.

See http://www.python.org/dev/peps/pep-0008/#programming-recommendations, second bullet point.
typo: "dato" => "data"
An iterator is a sequence (since iterator(i) = i)
The ticket was always going to incur a performance hit, but I *really* like the elegance of this solution. Note that `if ( !selection || jQuery.inArray( elem, selection ) < 0 )` will probably compress a little better, but what's a De Morgan transformation or two between friends?

**ALSO**: This changes the return value of `buildFragment`, and https://github.com/PaulBRamos/jquery/blob/e67daf2018c4d7661d4c45e3ba5ad38db9a36bf9/src/manipulation.js#L317-L336 will need some attention as a result. @dmethvin, we've been thinking about dropping the fragment cache anyway... it would probably simplify things here. I'm inclined to place the burden of proof on someone who wants to keep a ~100 byte eight-clause `if`, but I know there's a lot of history behind that precedes me.
Read further, information that a shortcut is being resolved IS just a debug, but information that shortcut hasn't been resolved is warning or error.

Additionally, logs are full of such milestones that shortcut is being resolved and the log is then unreadable.
Yes. Some other components, such as sparse vector, also have this issue. We have a separate dedicated issue number to resolve this.

We think we can submit another pull request to address this issue.
I think it should be internal.
needs to be pipecommand, not command.
Care to be more specific? (i.e. post the install log)
It would be better to have a separate `MacOS.xcode_installed?` method. Lots of things depend on `MacOS.xcode_version`'s current semantics, including external formulae, so we can't break compatibility.
Double http:// here.
Actually I had an issue when compiling Gegl on a fresh up-to-date install of Snow Leopard.
Gegl didn't compile at all because of some OpenCL 1.1 files missing.


On May 4, 2012, at 6:12 PM, Charlie Sharpsteen<reply@reply.github.com> wrote:

>> @@ -1,13 +1,28 @@
>> require 'formula'
>> 
>> +class NeedsLion < Requirement
>> +  def satisfied?
>> +    MacOS.lion?
>> +  end
>> +  def message
>> +    "Gegl requires Mac OS X 10.7 or newer: It uses OpenCL 1.1 which is not implemented in Mac OS X before Lion"
>> +  end
>> +  def fatal?
>> +    true
>> +  end
>> +end
> 
> Appears to build fine on my Snow Leopard machine---do we get runtime errors if OpenCL 1.1 is missing? Would be nice to have a link to a mailing list post/issue number that explains SL depreciation---can't find anything about it in the GEGL NEWS file and Google searches pull up these pull requests as the top hits.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/mxcl/homebrew/pull/12063/files#r775122
This is to support passing parameters to a SQL query that is not prepared. In that case the data types are not available.
Ok je supprime.
Do these need `create_session` calls need a `yield`?
Will this break when riakasaurus hits 1.0.10?
In what scenario would we want to set both the value and the index here?
Sayli, I would prefer to maintain the location of the regex inside of the validation code for a few reasons. The primary benefit is that it allows anyone who wants to look at the validation code to see in one place what is happening. If we move the regex to a constant, they would look at the validation code, and then need to look for the definition of that constant to see what's happening.

The other reason I hesitate to move it is that I haven't thought of a compelling advantage to have it as a constant. The re.match method will cache the compiled regex, so storing the precompiled regex doesn't lead to a performance benefit. This regex is also not used by any other code, so there's no need to share it or to avoid code repetition. Is there a specific advantage that you know of to having it at the module level?

Jay, thanks for catching the slash thing. That would have made a sad Randy when a bug inescapably came in about that. I'm fixing that and adding a test to check it right now.
Since defining `__all__` changes import behavior, I think it might be better to not implicitly add the variable to the module. It does create a slight amount of overhead for the first person to add an `@export` declaration to the module, but the explicitness would be worth it.
Yep. We can get venue name and address. The current venue's name is set as "University of California San Diego". I'm not sure if that was manually entered.

If someone wants to make the appropriate changes to the Meetup.com event(s), I'll update this PR to use the event's venue name. We'd also want to update the map/directions link as well.

Since the map and directions are more specific than a standard map route, what would be the best way to store different direction links? Default to the tinyurl link for the UCSD campus, then use a standard google map with directions for any other venue?
Please rename to fLock
can you try doing it here after the arg.toString(), without the extra \\, since client.escape() should properly escape those? (feels like the safer place to apply it?)
It seems like there are no circumstances when `_setupAds` runs that you would not want to tear down the previous ad. If that's correct, you can just run the deletion at the top always, and then bail on `if (isHomeNode)`.

Alternately, `this._deleteAdMobAd` seems reasonable?
This builds a map, and then normalizes as it builds a second map. Would it be more efficient to normalize the map as you build it in _bindingsToMap() (i.e. only build 1 map) ?
JSLint errors. Missing spaces around ``>=`` operator.
I guess the question is whether users will see it that way.  "Text" is a generic word, not a proper noun like the other strings being used here... so normally it would be translated.  Since we're no longer showing the full mimetype string, will users get that this label originates from one?
This should be $searchField now.  I believe NJ changed it but I'd bet it's easier when you merge to just ditch his changes to this file and redo the rename yourself.
The code here is very close to moveLineUp().  Is there a clean way to consolidate these?  For the parts that just vary +1 vs. -1, you could pass in a "direction" argument.  Then the only real diffs would be the corner case checks.
REMINDER: Removing id parameter will break existing extensions, so we need to announce this to the community.
(meaning the name, not the privacy - tell me if my description is insufficient.)
Great, thanks for letting me know, I'll go ahead and merge this in.
Looked it up and it appears "comply" is the recommended word here. Sentence should be "... new listing which complies with our ...".
Use find and replace in netbeans to catch'em all :-)
dunno, I always prefer to make it explicit. Quick hint where src of method should be looked for.
Why are you defining your own user class?
*This class defines document type for text posts.*
You also have to remove `django.middleware.transaction.TransactionMiddleware`!
Doesn't this go back on our 200 vs. 200px debate?  I thought we agreed on numbers?
add some spacing between each of these inputChangedHandlers
This should not be a url input.
on is not part of the interface. You should either mock the exact calls or use a real implementation.
Why this makefile? Also, there are many Python versions and interpreters, so I'd rather have users install them manually. Please remove this change. If you feel strongly about it, though, feel free to submit another pull request just with this change, and we can discuss there :-)
haha so this is copy-paste fail :P
This can/should use {} syntax an avoid the guard.  String.valueOf( searchable ) is superfluous.
Пользовательское — это отрендеренное в html? Я предлагаю не на абзацы бить, разделяя пустой строкой, а делатьпереносы (вот как в этом письме). Такие переносы при рендерингев html никак на документ не влияют. Вот что я имею ввиду https://raw.github.com/bem/bem-tools/master/README.ru.md 19.11.2012, 21:30, "Sergey Kryzhanovsky" <notifications@github.com>:In README.ru.md:> @@ -0,0 +1,49 @@
> +# Borschik
> +
> +Borschik is a extendable builder for text-based file formats.
> +It's main purpose is the assembly of static files for web projects (CSS, JS, etc.).
> +
> +# Относительные ссылки на картинки
> +
> +Упоминание картинок в CSS файлах можно изменить с относительных путей на абсолютные (или другие относительные) с помощью конфига (файл `.borschik` в любой директории). Конфиг считается относящимся к той директории, в которой он находится. Конфиги в директории более верхнего уровня (ближе к корню проекта) важнее конфигов в директории более нижнего. Пример синтаксиса:
Но тогда при чтении (пользовательском) на широких мониторах эти строки будут обрубаться, а не занимать ширину, предусмотренную вёрсткой. Посмотрел мануалы в apw, bem-tools, csso — везде смысловое деление текста.— Reply to this email directly or view it on GitHub.
This should be a unicode. Also, I think it should include the site's title which is in settings. e.g. something like this:

    return u'%s: Videos of %s' % (settings.SITE_TITLE, category.name)
Heh, not that I'm aware of.  Just calling attention to what feels like a dirty hack.
i would consider having one sendupdate-script that takes more arguments - right now somebody needs to inspect both command and the script
If we're requiring bash instead of sh (are we?) this should use the `$()` construct instead of backticks, which should avoid having to do the escapes.
Ok, good to know. Thanks!
The pattern we use is Infrastructure instead of Util.
A blank line is missing here
There is a small caveat there. HSQL does allow number types conversions
like FLOAT to INT, for example. The statement

select I from T1 union select D from T2;

where I is of integer type and D is double compiles but it fails to
initialize union executor in run time. I guess, it's better to add a schema
check there to be consistent.

On Mon, Sep 24, 2012 at 10:49 AM, Paul A Martel <notifications@github.com>wrote:

> In src/frontend/org/voltdb/plannodes/UnionPlanNode.java:
>
> > +            child.produceCopyForTransformation(childCopy);
> > +        }
> > +        return copy;
> > +    }
> > +
> > +    @Override
> > +    public void generateOutputSchema(Database db)
> > +    {
> > +        // Should be at least two selects in a join
> > +        assert(m_children.size() > 1);
> > +        for (AbstractPlanNode child : m_children)
> > +        {
> > +            child.generateOutputSchema(db);
> > +        }
> > +        // @TODO MIKE - what should be an output schema for union?
> > +        m_outputSchema = m_children.get(0).getOutputSchema();
>
> Since HSQL is validating number and type of columns, this code should work
> fine as is.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/alexeevm/voltdb/pull/3/files#r1673718>.
>
>
I see it in a few other places, but it looks like it might have been done intentionally in a few places, so I'll wait to see if there was a definite reason.
Fascinating--I always assumed that `** 0.5` would use the sqrt() function as a shorthand.  But I guess that would mean adding an extra if statement to every ** operation.
I just noticed now that you're calling this as `type,value, exception` ... the third argument is a traceback object, not an exception, right?  it looks like you're using it as a tb below, so maybe it's fine, but I just wanted to double-check that this is what you meant?
Given the comment and that this seems to update the status of an existing transfer, why is this not a cast? On the other hand, why was it important to crash the receiver in the original version? /cc @rzezeski
Once the test has reported back as "passed", the process has quit, which in turn has closed this process, so you won't get here after the nodes have stopped.
This shouldn't be in the pull request, keep this to yourself.
I don't think many people except for Satoshi like this coding standard, actually, but I must say I like the consistency.

That said, I never realized it wasn't used in the Qt code...
It's ok if you submit it in another pull request
Well, the issue here is that the construcotr does the logic even if you never call a getter to get the value, making the initialization of the object a consuming task, thus making it unusable when your code follows the DI pattern
Some whitespace in between these different items would be useful to help group the comments with the assertions they apply to.
Missing a closing paren
Can there be? The sample data didn't show that - not sure what the xml would look like? If you have a character with that, get me a dump :)
Cleanup needed.
I was thinking more like `shims <- new.env(); shims$system.file <- ...` and then I think you might be able to reuse the namespace code to copy in
I've been thinking about that solution too, I guess will keep thinking some more, void and manual return works until we're sure :)
indentation looks odd here. tabs!
Devil advocate here.
I would do an `os .rename` then write on it, just for consistency so that some filesystem ( like btrfs) can still now that this is still the same file.
Tohle nefunguje pro: dd if=/sys/kernel/debug/pb171/core of=/dev/null bs=1

Jsou tam 2 chyby...
In newer JS this is actually valid, i.e array comprehension. I bet its not supported everywhere though.

```` js
[ stuff for (stuff in someArray) ]
```
http://www.htmlhelp.com/tools/validator/problems.html#amp
Why not do it by using the dust.isEmpty ?

dust.isEmpty = function(value) {
  if (dust.isArray(value) && !value.length) return true;
  if (value === 0) return false;
  return (!value);
Either remove this or make it controlled by an option, maybe another verbosity level.
Capitalization correction suggested by Matej
nice, I like that
So this was a block that used to be in the base template but I grepped "intro_text" and didn't find anything. So this was dead.
The hacking howto should not be covering ElasticSearch requirements. So this shouldn't be here.
It'd be a pretty non-trivial set of query parameters on account of the boosts needing to be model-specific. But that is a possibility with the bucketed search results view.

With the unified search results view, we can do that, but it applies to everything you're searching for--not a specific model. This is sort of what I was getting at when I said I think we need to move the boosts to the document scope. We can talk more about that later, though.
That was just a copy from the installer code excluding the renaming to simply 'error'. Why I shouldn't use this name?
out of date copied docstring
crud, fixed.  also used the case__idprefix trick.
It doesn't seem like this message gets handled by anything.
Yeah we might want to use a different code for syntax error versus "n" to confirmation, I'll just make the latter 2
oops this line shouldn't be here, one sec will push a fix
This really is a no go, we can't get the cctools tarball on every creation. I suggest to cache it in `~/.virtualenv` as we're using it for virtualenv.conf now, too.
I don't quite understand.  Why would you have to pass two managers to the request?  Your request would be secure not, not both.
This is currently my only concern here. We are going to do memory allocations (resize the buffer) for each mip level. This will be slower than just allocating the whole lump. I could adjust this to be two for loops, fist to determine the final buffer size and the second to write to it. Would it be worth it, comments?
wrong indentation. This is Symfony, not Drupal :)
it was protected, I made it public in order to test it though
Request getHost() returns the unfiltered value of $_SERVER['HTTP_HOST'] ... which is the value of the "Host:" header sent in HTTP/1.1 requests.

Apache can be configured to passthru requests (eg to support HTTP/1.0 requests).  In that case the Host header is not sanitized.

Let's say you have a password reset email that uses a generated password reset URL that indirectly calls getHost().  A token is included in the URL.  The attacker would modify the Host header in his requests and submit reset requests for other users.  Target user clicks on email link that goes to attacker's site, leaking said token. (XSS + phishing)

So , yes, avoidable by securing the web server configuration.

The PR addresses the fundamental point of not trusting user input.
You need to add the composer autoloader if it exists.
@fabpot rejected to use an id in the toolbar a while ago, saying it should be possible to display the toolbar several times.
500ms last time i measured.
If you put this in the postgres home directory (/var/lib/postgresql on recent 12.04 instance) then it will work for the postgres user.
= = # 这个是因为，我喜欢3这个数字。
Can you make sure this header is public?
Could you please add a short description for this details::class ?
Does this also work with https endpoints?
Should probably be called "Chestplate" since that's the in game name
ok, c'est bizarre ca me l'affiche pas dans le diff...
`$ele` is not a very telling variable name. Consider renaming it to `$data`. The same goes for `$tmp`.
Hm, this sentence confuses me.
This is going into Pyface, which knows nothing about Envisage, the Tasks application, or the event manager service, so we cannot use the event manager for this. 

Besides, we shouldn't be using a global event dispatch for low-level context menus *anywhere*. That's a gross breach of abstraction boundaries. I know there are other places where we are doing this, but let's not introduce any new ones.
it used to be 1.0 (was defined in Namespace.java)
@bstansberry Brian, here is an attempt to achive the behaviour you/we wanted. While at it, does this line need i18n?
i think the \d should be \d+ here
no XML
pill_images[color+'_'+rotation] && ctx.drawImage(pill_images[color+'_'+rotation], x*block_size, y*block_size);
Let’s not redefine headers here please. For the sake of the HTML demo we can use an external stylesheet:
```html
<link rel="stylesheet" type="text/css" href="../headers.css" />
```
Sure! ;)
I don't understand why the existing code didn't work.  If the app is fullscreen, won't its heght be the window height and its top offset be zero?

Does this fix the bug where the activity frame isn't quite the right size?  I've seen that happen, but don't know how to reproduce it now.
oug, can we name the parameter or the variable in a different way, is a bit confusing and names are free :), something like

addEllipsis: function kh_addEllipsis(view, fakeView, ellipseSide) {

    var side = ellipseSide || 'left';
We always get something since the "phone type" is a mandatory field so at least we would get the type of phone on screen... :-)
please remove the es, fr, it, pt-BR, zh-TW sections if the corresponding files don’t exist.
I feel like CustomDialog.show and NotificationHelper.send would love to have a js object as parameter but that's not related to this pull request.
these are now all exposed as globals should be inside the suite block
What if contact.tel = = [{}] ?
var AppManager = {
  isLocalized: true,

 init: function init() {
   ...
  }
}
If we can make sure the pending message must be the latest in the thread, we may not need to sort the message after concat (but we have to know hot to concat because the message sequence in ThreadListUI and ThreadUI is different). If the pending message might interlace with original messages because user decide not to send the old pending message, we still need to sort anyway.
What we really want here is for:
1) the message reader display to close if we invoked moveMessage() since the message will intentionally be in another folder now.
2) the move toaster to show up on the message reader.

Since you're not dealing with the 2nd thing here, this is fine, I just want to make sure you understand the gameplan.
Of course we can. I prefer your option but I mimic the existent API
I think is better to put the FB string to denote that this is an error from the FB integration. Taking also into account that FB integration might be disabled with a setting ...
do we really need this?, as the FTU manifest will be a well known one ...
Since this actually alters the stream position, it should be named accordingly (UpdatePositionGetEffectiveCount)...
I use JObject.ReadFrom(reader)