labofsahil/github-event-dataset-2012 huggingface dataset:
Howdy Curter, I don't understand why you renamed DEFAULT_CONFIRM_EMAIL.

Also, could you please sign this so we can accept contributions from you: http://vanillaforums.org/contributors
Here's an idea: `dup2` is only ever called by a child process, so there is no code for it in the cache, so each newly forked child has to compile it.
Yep, feel free to commit this change directly - I definitely agree the implied string continuation is better... I just have some bizarre mental block about the implicit string trick - I always forget that it exists! So if you see this from one of my commits in the future, don't hesitate to fix it - definitely not intentional.
Quick update: 

Setting `display_name` to null in `frontend/modules/profiles/actions/register.php` results in the user receiving an email with a `display-name` of the value of the `core mailer_to` setting, due to the defaulting in `addEmail `  here https://github.com/forkcms/forkcms/blob/master/frontend/core/engine/mailer.php#L44
This is slightly less code... and more clear?
> :) 1300-15**6**0 - heavy unstable period...

*fxd?
grid unload not related to CPU load. need profiler use... i not make this result, sorry.
we will test it but have compile problem whith this line:

Radical/src/game/Unit.cpp:679:59: error: taking address of temporary [-fpermissive]

I don't saw this problem before, anyone know how to solve it?
should use contentType "application/json"?
There exists some other candidates for constantization: GC_[debug_]end_change_stubborn, GC_exclude_static_roots, GC_add/remove_roots, GC_general_register_disappearing_link(const obj), GC_finalized_malloc. Correct?
I think I prefer the CSS comment.
This is fine by me.   I can commit this version if you like.
Wouldn't expect you to. Did I push this by accident?

It's going to take me some time till I understand git enough and have got my changes in a fine-grained enough format to push them back to your ocupy.
You're doing this just because it's a separate window/tab?  Should not be necessary... you can do "opener.top.restoreSession()" to invoke the one from the main window.  More generally, cloning blocks of code must be avoided.
an ok solution, but needs a comment.
that's where line 25 comes in, it changes the UA from settings to lowercase as well -- not sure what you mean by cookie check
this was simple the least invasive way I could use any copy & pasted list of user agents without having to verify capitalization
It keeps spawning up, doesn't it?
Thanks for the info. Really interesting. Can make some optimizations to my own code now.
I'd suggest that every public function accepting a GError ** have this,
or something like this, at the beginning:

``g_return_val_if_fail(err == NULL || *err == NULL, something);``

This makes sure that the caller either invokes the function with error return location
set to NULL (which means he ignores errors) or that he properly initializes the location
with NULL. Of course, one does not need to do this in functions that just pass the
error location to glib functions that do have these checks.
Ok, I'll change it now.
the quotation mark for `'name` should line up with the one in the line above.
can we get away with not duplicating code?
Nicely done. Though it's boost, not bullet. ;)
I guess you want to use `get(key, default)` and not `popitem()` (which doesn't take any arguments). I don't see any need to remove the pair from the dict before passing `kwargs` on to the superclass.
Why? What's wrong with leaving it configurable, but in .config/razor by default?
Du får använda mitt nya [gg-kommando](https://github.com/oleander/dotfiles/blob/master/bin/gg-git). Bättre än någonsin. 17% av alla kommandon som används på min laptop är gg, (172 av 1000)

``` ruby
#!/usr/bin/env ruby -w

# GG git, version 0.2.0, 2012-01-25
# Warn you if you're trying to commit large files
# Usage: gg "A commit message"
# Copyright (c) 2012, Linus Oleander
# Released under the MIT licens

max_file_size = 0.2 # In MB

files = `git status --porcelain`.
  split("\n").
  map do |raw|
    split = raw.split(" ")
    media = split.last
    type = split.first
    if type == "??" and File.directory?(media) 
      next `find #{media} -type f`.gsub(%r{\/{2,}}, "/").split("\n")
    end 
    media
  end.
  flatten.
  map do |file|
    next unless File.exists?(file)
    size = File.new(file).size.to_f / (10**6)
    next unless size > max_file_size
    "#{file} (#{size.round(2)} MB)"
  end.reject(&:nil?)
  
if files.any?
  puts "#{files.count} large file(s) found."
  puts files.join("\n")
  puts "Continue?"
  abort("Aborting") unless STDIN.gets =~ /y/
end

puts `git add . && git commit -v -a -m "#{ARGV.join(" ")}"`
```
ага ... яэт я дурилка фанерная...они в другой ветке... изза моей криворукости и неполного осмысления гитхаба
Grant, out of curiosity why did you comment out provided for the scope of the hadoop dependency?
hm... i think i've come up with something...
PEX doesn't run on spout server last I checked, nor does it use our namespace or have spout in it's name :S
This one shouldn't be changed, it's checking the environment.
Yep, p.port is unsigned.
i am already using 2.0 for production, when do u release stable 2.0 version?
start quests not acepting

If function is enable
So there should be done in better way to turn off mmaps ... When you turn off mmaps in config after using fear it is very bad also charge spells etc ... to be included in core this has to be fixed
And crash http://paste2.org/p/1888023 was so promising ...
I'll try to fix this commit since it is late your time
If I could "Like" this commit, I totally would.
Ha! foreman *cough*

On Thursday, 16 February 2012 at 00:00, Brian Ewing wrote:

> Aye. There are reasons, but none strong enough to excuse such an atrocity as this. I profusely apologise to the wider developer community, and will surrender my Rubies and SQLs and NoSQLs and Pythons at the next available opportunity.
> 
> (Using env variables was annoying everyone)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/weetidy/twandom.me/commit/ec12f3d76a6436ab7e1ba6585dc786524f2b5576#commitcomment-971957
> 
>
@unforgiven512 https://github.com/CyanogenMod/android/commit/89acec784fd50305cc55d05ecb3416fcd7c3eb0e
had to add `.to_sym` for `3.1`, in next versions it is not necessary.
I'll plan to yank this file modification at some point in the future (likely before the official sourcebase commit)
xD c est du timmi hein faut pas trop lui en demandé avec github :P
Indeed, I was quite fond of this functionality for I used the ssl connection with a signed certificate for remote connections and the non-ssl connection when I was on my own lan.
Not a problem.

On Sat, Jan 7, 2012 at 11:08 PM, Strife Onizuka
<reply@reply.github.com>
wrote:
> It's too much work to track signatures on the wiki (which article of a set of articles that all use unarticled constants contains the signature definitions for those constants). The solution is to put the declaration in the template that builds table of constants used in all of those articles so that the signature shows up in all of the articles. Consequently any script that reads the signature block needs to ignore redeclarations.
>
> Example:
> https://wiki.secondlife.com/wiki/LlGetPrimMediaParams#Signature
> https://wiki.secondlife.com/wiki/LlSetPrimMediaParams#Signature
>
> I should make a template to make it easier to add signatures to the block and to track who is adding to the block.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/BlindWanderer/lslint/commit/06977c8cefa28b76ef8ae7d9d16a6c0d30f1f01a#commitcomment-847217
Just remove the boiler plate code and use @Canonical class annotation
Or just @EqualsAndHashCode and @ToString if you dont want to tuple constructor.
Good feature, thanks.
This change re-opens bug #164

13:09:58,675  WARN org.eclipse.jetty.server.AsyncContinuation:496 (qtp151042754-23) - 
java.lang.IllegalStateException: Request object no longer valid. This object has been cancelled
	at org.atmosphere.cpr.AtmosphereResourceImpl.getRequest(AtmosphereResourceImpl.java:384)
	at org.atmosphere.container.Jetty7CometSupport.action(Jetty7CometSupport.java:178)
	at org.atmosphere.cpr.AsynchronousProcessor.action(AsynchronousProcessor.java:1)
	at org.atmosphere.cpr.AtmosphereResourceImpl.cancel(AtmosphereResourceImpl.java:673)
	at org.atmosphere.container.Jetty7CometSupport$1.onComplete(Jetty7CometSupport.java:103)
	at org.eclipse.jetty.server.AsyncContinuation.doComplete(AsyncContinuation.java:492)
	at org.eclipse.jetty.server.HttpConnection.handleRequest(HttpConnection.java:495)
	at org.eclipse.jetty.server.AsyncHttpConnection.handle(AsyncHttpConnection.java:48)
	at org.eclipse.jetty.io.nio.SelectChannelEndPoint.handle(SelectChannelEndPoint.java:586)
	at org.eclipse.jetty.io.nio.SelectChannelEndPoint$1.run(SelectChannelEndPoint.java:44)
	at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:598)
	at org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:533)
	at java.lang.Thread.run(Thread.java:680)
Do we really need separate rules for border, radius and shadow? We may, but if there's no variation in HTML, I would call it extra burden. :)
네 그렇게 할께요 :)
@emerleite cool! :) Thanks
The ZTE Blade .35 source code removed support for other ZTE devices in a few places like this, check kconfig & other files too. For my patch I just copied the blade source code over the untouched zte turies source code & made a commit of the difference. I never bothered to add support for other ZTE devices back, because I don't own any others. I think Tom G fixed these problems in his patch.
Ahh that's nice. I like it!
Apparently there's a problem, template resolution fails. If that helps, when I add this at line 103 of boss_mail.erl:

        io:format("ViewPath = ~p, ViewModule = ~p, filelib:is_file(ViewPath) = ~p, code:is_loaded(ViewModule) = ~p~n",
              [ViewPath, ViewModule, filelib:is_file(ViewPath), code:is_loaded(ViewModule)]),

I get this output:

        ViewPath = "src/mail/view/confirmation.html", ViewModule = cb_who_src_mail_view_confirmation_html, filelib:is_file(ViewPath) = false, code:is_loaded(ViewModule) = false
        ViewPath = "src/mail/view/confirmation.txt", ViewModule = cb_who_src_mail_view_confirmation_txt, filelib:is_file(ViewPath) = false, code:is_loaded(ViewModule) = false
Summary should describe what the control does. The creator method should be moved to the description.
Hey, sorry for the delayed response. I've been without a laptop for a few days, so I'm just getting back into the swing of things.

I like this idea--but I don't like the helper blacklist function. It just looks kinda hack-ish in comparison. I'm trying to think of a better way to do it. Maybe just a ``is_blacklisted`` function in a ``helpers.py`` module? I think that may look cleaner.

What do you think?
What is the most generic, yet informative, exception I can throw that will make sure all users of jstream will be able to catch it and treat it correctly?
If I understand - you'd like the path of the added queue entry. Can you describe the need? What are you doing with the path after you get it? 

Also, the only reason add the foreground insertion is to get the Path, right?
orly? not implemented?
Should be a const function.  Not too keen on returning an array, but I can't think of anything better.
Want to create an issue (milestone later) for that?
thanks Dan. Yes I see that, like I said the ones locking multiple keys make the possible solutions more complex. I agree it's not a priority, still an interesting puzzle to solve :)
I tried that at first, before realizing what the real problem was. Just move the
```java
if(health < 0) {
  dropAllMoney();
}
```
code into the `tick()` method instead of the `render()` method. That's where it's supposed to be.
The `new Random()` way makes the synced random generator drift out of sync sometimes.
Уже сделано в f9613c3c3c378932f3df8807b27f41e9640b7284
furula bien parece...de momento vale asi
These two imports break "python setup.py install"
Tweaked in commit f39755a0755cdd13226a6815c3f999c70bbc4dc2
https://github.com/scottkellum/web-app-spec/commit/f39755a0755cdd13226a6815c3f999c70bbc4dc2

More tweaking needed?
They make us tough out here in the colonies =)

I've been offline a couple days because I've been working like a dog 
and I just was playing Star Wars to blow off steam rather than hacking ( 
more slashing, less hacking as it were ).

About the asio.hpp, I noticed also that the generic header includes the 
world and is massive.  This made sense to do this when I wasn't sure 
what features I'd be using, but now that we've ironed most of that out, 
it's a good place to cull a bit.

I did take a look at it originally, and I noticed that boost have some 
really gross macros that surround including ... I half thought the 
preprocessor would intelligently cull it out.

Also, I'm leaving the original headers just commented out for now ... 
the last round of commits for this effort will be to remove all those 
comments.

I'll have a go tonight and push another round.  Take a look tomorrow 
and eyeball it again.

/sr
double next =pnextlevel-pexp;
I would love to, but there's an issue with it, can you please reply to me on IRC?
I've tested it on all my machines, and it seems that it works. Just to be sure, I started a new Mint instance in a VM, with only git, zsh 4.3.15, and this repo. When I do:

    fpath=(~/zsh-completions $fpath)
    autoload -U compinit
    compinit -i

so I don't think it has to do with my config. The version of zsh doesn't seem to matter, either. I don't really know what could be the problem.
I'm not sure how this needs to be implemented, so for now I just added these dummy methods which at least allow the Android examples to build.
please contact me when completed, if you can.

I hope Japanese translation is included in the next release,
so I want to push it after you complete.
Any reason this is required? I'd like to avoid adding a gem dependency (if we do need it, please add it to the gemspec).
Please take into an account that for the plain sockets the on_tls_init method will be never called.

>But you don't need a connection to set up an ssl context, you need the io_service, so this is more direct!

Do you need the io_service during the context initialization? If yes, I have no questions.
Oh wait Game said that.... damn you.
I'll try to merge your branch later today.
Cherry pick is going to be a pain; and when I use Ikarosdev's framework/base, I was getting a strange error. http://forum.xda-developers.com/showthread.php?p=2120631 so I scratched out the HAL_TRANSFORM_ROT and only left the numbers, and It ekpt on compiling this time, but i got another awsomeplayer error some place else. So I switched to GCC 4.4 and used the google source framework/base, and the 4.4 version didn't get any errors from awsomeplayer, but i'm getting errors from https://github.com/ikarosdev/android_hardware_msm7k/blob/gingerbread/liboverlay/overlayLib.cpp overlaylib.cpp like it has couple messed up lines, that I am not sure how to fix, and thats my current issue right now. Do I need ikarosdev's framework/base, couldn't I just use google own? I building AOSP can such a pain...
working like a charm, thank you!
When DisplayObjectContainers without children calculate their bounds, they get a Matrix relative to the target space that includes their translation. Then the translation is included again in the calculation of the position by the matrix. This change removes the translation from the call to the matrix.
pushed the cs fixes, thanks for checking that.

about separating: we originally said that we want to provide routers in this bundle too. this router is not too specific on the cmf, it should work with any doctrine implementation. (some minor glue code/doc for the ORM is missing but it should be easy to use)

i just made the doctrine router no longer load by default, so that people can use the chain router without any doctrine routing: 
https://github.com/symfony-cmf/ChainRoutingBundle/commit/fce88421b82567db40a3ee7a9f8f605b79e2a8ee
Can you please tell me how in the love of God can I build with this option??
where do I put --configid 1 before build??

or do I have to put flag NAM_VARIANT=inspire in board config??
Please be more specific
Ok, sorry. I like to rebase distribution branches onto my "master" branch, but now I realise that I should not rebase other peoples commits on a public repository. I will fix that.
will you do a PR of this? 

It is definitely should be there.
My concern is the explosion of methods on `Validator` that become unreadable. That's why I proposed this approach to limit its pollution.

The cons I can see is that if at some points constructors have setting that methods don't have or vice versa, we will need to revise this approach many by subclassing of `MethodValidator`.

Personally, I think this approach is worth it. Also note that the design is consistent with `getConstraintsForClass`. My idea was to let major methods on `Validator` and put minor ones behind an indirection.
Was there a reason why it was reversed?
lo siento no hablo espanol muy bueno, pero que es la information correcto en espanol o en ingles
This is why changes should require a second verification before the submitter can approve them.
typo "documnet"
I've compiled and am trying to use the module, but the success callback is never executed. Do you have the same issue ?
shouldn't it be /usr/lib/i386-linux-gnu instead of /usr/lib/i386-linux/gnu
screen_name 属性の扱い、どうしましょうか？
/users/:id で twitter へのリンクを表示するために使っているようです。
name 属性に twitter から返って来た name ではなく敢えて screen_name を使っているのには意図がありそうですね。
The constants you have here are item constants, not material constants.  You'll want to have MATERIAL_BAR_BRONZE, MATERIAL_BAR_IRON, etc. for the Materials field in this example.
What I mean is that instead of duplicating the test code that checks whether the browser supports touch events everywhere it's needed, move this test into `Y.UA` so its logic is maintained in a single place. Like `Y.UA.supportsTouch()` or something.

I believe the Chrome sniff is there because some versions of Chrome reported a false positive for the `'ontouchstart' in Y.config.win` test.
Where is used this class ?
Please no "this" in front of log :-)
&& $cityId == F::app()->wg->CityId needs to stay
I'll see if I can use ActorSystem.registerExtension to plug in the Cluster subclass for testing purpose. If that is possible the internal actors can use Cluster(system).
Avoid the closure for msg.map(_.getClass)?
case n: BigInt =>
case n: BigDecimal =>
I'd recommend not using + for a destructive update
I think this baby has got to go
`var todo` is scary, but I guess you guys know what you are doing here
Doesn't work ;-)

[error] /Users/viktorklang/Documents/workspace/akka/akka/akka-actor-tests/src/test/scala/akka/serialization/SerializeSpec.scala:189: overloaded method value must with alternatives:
[error]   (notWord: SerializeSpec.this.NotWord)SerializeSpec.this.ResultOfNotWordForArray[Byte] <and>
[error]   (haveWord: SerializeSpec.this.HaveWord)SerializeSpec.this.ResultOfHaveWordForSeq[Byte] <and>
[error]   (rightMatcher: org.scalatest.matchers.Matcher[Array[Byte]])Unit
[error]  cannot be applied to (SerializeSpec.this.BeWord)
[error]       byteSerializer.toBinary(ba) must be theSameInstanceAs ba
its wrong, i'll change it
Performance, perhaps, I kept that because the original impl did something like that and also the article http://www.lexemetech.com/2007/11/consistent-hashing.html

I'd be happy to remove it if we don't think it add any value
Delete these?
looks like we need to create a new less var for this assuming it is the lang dropdown - needs fixing in footer.less too
True this was just a quick fix
Good comment but might be clearer if located in the except block along with the pass.
Another case where the binding is created, and then saved below
why do you need to initialize this?

Worst case, put it up into the signature as an optional parameter.
finally {
close()
}
I test snippets. I was just trying to get the doctester working, but it's indeed quite hard because of the recursive importing.
bad hashcode.
Can you capitalize descriptions ("Enable putting...")?
Duplicate block.
Yes, I am indeed abusing it and drawing into nothing: if you look above where the track is drawn, I need the slider's `knobRect` to highlight the slider at exactly the location of the knob. This `knobRect` is only passed (below in `drawKnob:`) after `drawWithFrame:inView:` is called, which is why I draw into nothing. The cell's drawing is deferred to allow the `knobRect` to "propagate". It's really not a performance hit, but I agree it is a massive abuse of API. However, as justification, that's the only way the knob can be drawn highlighted.
This is subject to race conditions. Please use `dispatch_once` instead.
New code should not declare instance variables, since it's no longer necessary for the compiler.
Shouldn't it be `>= 2`?
I revised the patch to just convert the input data frame to a list to remove the ugly environment copying I was doing.  This also improved performance, however the conversion still makes this version slower than the non iterative version.  This version is much closer to the mutate implementation.
And then what we'll have here is coding updating the list of image file *names*, and deleting old files when their names are deleted from that list as it grows.
Won't work on architectures that require aligned loads.
I didn't see the comment in the other pull request.  Can you elaborate? Code example?
That's a good argument. However, I just noticed, that `NPY_UNICODE` is just an "enum", see `ndarraytypes.h:71`. So I don't think that's a good idea to do sizeof(), because that will always return 4, no matter what the actual size is. Is there some other solution?
I don't want that branch to be executed, since I already got different strides in that case. The needview is set for the next if just after.
You should be able to easily calculate the actual length
@TazeTSchnitzel I think your Facebook just went wild ;)
Apologies, not use to this yet, will check over more thoroughly.
it's not 'dead code', we need to turn, for example, `:only` into `options[:only]`. It just so happens that `:only` is stored in a variable named `from`, which we then assign the result of `options[from]` to.
I didn't get why we need to symbolize the keys. Could you explain?
Yes, please remove it.
Why do you need `session_payload` here, if you're using `response.body` in the line below?
it used to be public, you want me to remove the specs and resquash?
`Creates a copy of this attachment with the payload slot of T added/updated with the provided value`

Note that update first erases stuff of type T from current attachments, and only then appends the argument.
stap should be using the same args as "quick", otherwise odd things could happen....
Typo: "The _names_ of the groups"
intuitive?  I'd love to hear what this was.
How do these classes relate to the generators in `generators.py`?
Pyflakes will catch such errors (I added one to demonstrate):

```
$ pyflakes sympy/core/sympify.py
sympy/core/sympify.py:7: 'from sympy import *' used; unable to detect undefined names
```
-> `know = diff.equals(0, failing_expression=True)
I tried deleting this ``elif`` block (L253 to L255) and all the tests still pass.
The nice thing about map is you only have to import one thing. But I changed it and fixed the failing test -- which shows how one can get into trouble when using map :-)
It should probably just return `self` instead of raising.
well, it can still handle 'coulomb potential' (electric potential due to a point charge). The function doesn't determine the potential energy but accepts a user defined function.
If it's internal use only, why did you add it to `__init__.py`?
Let's get this unXFAILed. If it's real it can't be non-commutative. So removing that will give you:

```
>>> X, Y = symbols("X Y", commutative=False)
>>> Z = X + I*Y
>>> Z.conjugate()
conjugate(X) - I*conjugate(Y)
>>> (Z*Z.conjugate()).expand()
X*conjugate(X) - I*X*conjugate(Y) + I*Y*conjugate(X) + Y*conjugate(Y)
```
Actually, the code is in an ``if version > 0.11`` block.
Depends on what we decide about simplification of expanded forms.
The calendar could not remember the current date, when it was opened a second time.
doesn't matter though, not in the new branch
It was being called. The patch nukes the production call being made
when removing a pair.

On 25/05/2012, at 18:17, Ben Hood
<reply@reply.github.com>
wrote:

>> @@ -102,11 +96,6 @@ trait DomainDifferenceStore {
>>   def lastRecordedVersion(pair:DiffaPairRef) : Option[Long]
>>
>>   /**
>> -   * Removes the entry for the latest correlation store that was recorded with the diffs store.
>> -   */
>> -  def removeLatestRecordedVersion(pair:DiffaPairRef)
>> -
>> -  /**
>
> If it's necessary functionality, I wonder why it is not being called from production code already. Also, I'm wondering where it should be called from.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/lshift/diffa/pull/166/files#r880232
also remove this one, no need to nest role="navigation" :)
Do you need an instance variable here?
It's the opposite direction that we're worried about:

`cdb_save` passes in `self` which eventually gets stuffed in to a Chef::REST call, where it gets converted to JSON via an implicit `ApiClient#to_json`. Since this patch omits the behavior change for `to_json` (which is present in the Chef 11 version), we're protected against saving private keys to couch.  It doesn't show up in the diff context, but you can see the code here: https://github.com/opscode/chef/blob/a553d4edc07b8e95c0a99a7bd66d261da0437368/chef/lib/chef/api_client.rb#L145-162
I didn't understand what is this prompt about...
this should be in session just like add_lang or add_ext_lang
Because it hasn't been added to the page yet, feel free to do it instead of asking why it hasn't been done, its quite obvious looking at the code and event name what it does, think like an extension author, not a developer. What might you want to put there. You seem to have the idea of events wrong. Authors look for events in the right place to add what they want. They don't look at events and think what can I put there.

It allows you to add further info blocks like Statistics and Who is Online.
Hmm, weird.
okay, reverting back to phpbb_style_is_active()
Is this going to work with the timezone changes?
The location of that event appears to be off at all. Makes more sense to have this at the end of the `$forum_data` block in the add case
Drop quotes.
Same as border-radius - it should not be in colours.css
I am OK with this. However, should we make this a generic function in luvit's path.lua to match an absolute path?
fixed in d03dab0
`logging.warn`
I remember that discussion, sorry that I've forgotten to test my virtualgl installation. But these changes are ok for your development, not for repository.
In my original code I was using:

HB = require('yui/handlebars').Handlebars

instead of the regular "requires" statement. This is the way @ericf is using it in photonearme project. I wasn't sure about the implications. I think this approach will isolate HB into its own Y instance, but I remember having some problems with the way HB.compile works when using the "requires". But it if works fine, then we can keep using the require :)
в последней ноде этот код не работал. если у вас старая то верни старый код.
Indentation - `case` should be 4 more than switch, and everything inside it a further 4 more.
Sure, you can delete that one. Its from before we had our conversation :)
Ik geloof niet dat deze docblocks kloppen?

Suggestie:
/**
 * Pacman  (https://github.com/Enrise/Pacman)
 *
 * @link      https://github.com/Enrise/Pacman for the canonical source repository
 * @copyright Copyright (c) 2012 Enrise (www.enrise.com)
 * @author    Rob Quist (<evt je email adres>)
 * @license   http://framework.zend.com/license/new-bsd New BSD License
 */

Rinse & repeat. Gaat ook voor andere bestanden binnen het pacman project.
I'd recomment using `if not os.path.exists(self.pathfile):` here
The missing indention here confused me at first as to whether those were local or global variables.
When we POST to update/create a new patientlist, these fields are required:
-name
-description

and not searchquery.
I know it hasn't changed, but the docstring note re datatypes is a little confusing. It appears from the result files that the handling of data types is more consistent so perhaps the docstring could be tidied up. The use of the word ' updated'  seems inappropriate. It may also be worth pointing out how the datatype of the interpolated coord(s) depend on the datatype of the sample points (not the original cube?).
// if posting a form, post as usual
// if posting an object, post as json content type
remove old comment?
Everything wrong.
Don't use incomprehensible variable names.
I was grasping for straws when I added the PULSE_SEND() macros in place of
enif_send() as things were breaking all over the place at that point in my
code.  This was something I did because I thought it was a requirement when
using PULSE to test NIFs, so if it's not then I'm happy to remove it.


@gregburd | Basho Technologies | Riak | http://basho.com | @basho


On Mon, Dec 3, 2012 at 5:13 PM, Scott Lystig Fritchie <
notifications@github.com> wrote:

> In c_src/bitcask_nifs.c:
>
> > @@ -1646,7 +1646,7 @@ static void msg_pending_awaken(ErlNifEnv* env, bitcask_keydir* keydir,
> >      for (idx = 0; idx < keydir->pending_awaken_count; idx++)
> >      {
> >          enif_clear_env(msg_env);
> > -#ifdef PULSE
> > +#ifdef PULSE_NOWAY_JOSE
>
> FWIW, the gsb-async-nifs branch uses:
>
> #ifdef PULSE_NO_REALLY_JUST_USE_THE_REGULAR_SEND
>
> Removing the unused code would be good because it's, well, unused.
> However, I think it's also a useful placeholder for later discussion with
> Hans & Ulf & the rest of the Quviq folk: it's only 52% clear to me why
> you'd want to use a PULSE-instrumented send here in the NIF code, but it's
> 100% clear that actually using a PULSE-instrumented send here in the NIF
> code causes all kinds of deadlock problems for PULSE. Some kind of "TODO"
> reminder would be good. {shrug}
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/basho/bitcask/pull/73/files#r2299322>.
>
>
Do you invision more remote calls like this being made in the future?  If so it would be nice to add a `riak_core_capability:get/2` for remote calls that does all this work and returns `true | false | {error, Reason}`.
thinking of a better way.
x.haveselectxpathregion and xhaveattributevalue don't seem to be initialized to false anywhere. Use 

x.haveselectxpathregion = (x.select_xpath_region = GetConstraintValue("select_xpath_region", pp, CF_SCALAR);

for tighter code that initializes in any case.
rather than commenting this line, it's best to just remove it.
I've made this a lot cheaper by getting the type directly.
If POSTGIS_TEMPLATE exists, it will be a string, not a tuple. So you'd better make the tuple in the execute method below instead.
Nitpick: s/integer/int/
Hmm, yeah, I can try something else. I'll just import `expanduser` from os.path and use that.
This is based on an hlint "redundant $" warning that wants to translate:

f <infix> (g param $ some stuff)

to:

f <infix> g param (some stuff)

So the parens are just replacing the $
For my benefit, can you explain what it is from the .cabal file that we need to store in the index? Is it the package name and version? What should happen if a user `--link-source` a local dir and then changes e.g. the package name? Since we're linking and not copying presumably the package index needs to be updated?
Putting a path as a key is a weird
Excellent; this is much nicer than the way I had it and almost the same size. You can save a byte gzipped by testing `scale !== 1` first, for a total size increase of 35 bytes.
This file is generated after configure. Don't include it.
Indeed. In fact, it would cause an error if the schema name is not madlib!
The type of verbosity should be the same for all the functions. Please change it to INT.
Nit: This change adds a 't' to any.
Got it. So... you can leave it like it is here and then do another commit that fixes this and the other instances. That'd be fine.
We probably don't want this file.
Just checking, was this change expected?
This condition should never be met, since `from_addr` should never be a tuple.
Not sure this is the right place to start talking about the advantages of distributed version control. It is the next section's content.

And I think it is too early to start introducing the word "commit"
Так лучше?
> При клонировании репозитория, как правило, автоматически создаётся ветка `master`, которая отслеживает `origin/master`, поэтому `git push` и `git pull` работают для этой ветки "из коробки" и не требуют дополнительных аргументов. Однако, вы можете настроить отслеживание и других веток удалённого репозитория. Простой пример, как это сделать, вы увидели только что — `git checkout -b [ветка] [удал. сервер]/[ветка]`. Если вы используете Git версии 1.6.2 или более позднюю, можете также воспользоваться сокращением `--track`:
hmm gute Frage. Danach gibt es noch die Methode getFullPath(): https://github.com/redaxo/redaxo/blob/5d14db0f8ac748017b92195f8831f4ace8d72bef/redaxo/src/addons/mediapool/lib/media.php#L248-251
Wenn, müsste das dann auch getFullUrl() heißen? Passt aber auch irgendwie nicht so toll, oder?
Ich glaube, ich würde es erst mal so lassen, wie es ist.
Could the imports in this file be moved to the top of the file?
Please drop the word 'simply' here. That's offensive.
Is maven capitalized?
Use backticks for `pom.xml`!
It's used by `HttpEngine`. I made it `public` to make that more explicit.
With this for loop I don't think you get to index = 0, try to do a:
```cpp
for(uint32_t i = size-1; i > 0; i--) 
{ std::cout << i << std::endl; }
```

I think you can do it like this ``for(uint32_t i = size; i --> 0; )``
why the negation wtih ! here?
@marcphilipp 

+ nothing will change. they still will use junit:junit:4.11.
+ accorind to the felix documentation this is really alright using packaging: bundle.
+ the plugin will handle manifest with packaging: bundle.
+ i used this project in integration tests and any other project dependent on junit, and it really works as if packaging: jar. Otherwise i would calim as well as you -but i proved.
+ we need default plugin's goal: bundle because we need hamcrest in it and this is easy. Next time when #421 and #332 is ready with I will change to packaging: jar and goal: manifest. I can inject hamcrest with shading plugin, but this is safe solution as well, and solves our problem (other for junit-dep).

Today I will update my project and add new OSGi entries. I modified the entries and the itegration test work fine. I will maybe tomorrow extend the integration tests.
Still thinking through this, so no complete thoughts yet.  Wouldn't it be easier to extract a protected methodBlock(Description, Object) method, as we discussed in the other thread?  It would also be more forward-compatible, in case we add something to the base methodBlock at some point in the future.
If namespaces argument omitted then 'og' will be default.
He didn't understand the concept :P

I will change the GUI system to use a ChatStyles list appropriately.
_xhr is used a couple places, and it does handle the deferred bit cleanly. I'd just as soon leave it for now.
I will add a comment that the location may be adjusted to prevent clipping. I can't think of a reason right now to return the location so I'll hold off on that.

-----Original Message-----
From: Randy Edmunds [mailto:reply@reply.github.com] 
Sent: Thursday, June 07, 2012 4:11 PM
To: Ty Voliter
Subject: Re: [brackets] Context Menu API (for review only) (#1012)

> +     * 
> +     * @param {number} x - page relative x coodinate
> +     * @param {number} y - page relative y coodinate
> +     */
> +    ContextMenu.prototype.open = function (x, y) {
> +        // TODO: positioning logic
> +
> +        $(".dropdown").removeClass("open");
> +
> +        $("#" + this.id)
> +            .addClass("open")
> +            .css({"left": x,
> +                  "top": y - 30}); // todo: compute offset
> +
> +        $(this).triggerHandler("beforeContextMenuOpen");
> +    };

May need to return the x,y coords in case context menu gets displayed in some place other then the x,y coords that were passed in, but I'm not sure if that's of any value to the caller.

---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/1012/files#r948109
I wondered about that.  It seemed harmless (and slightly simpler) to just leave it active all the time since finalizeDocumentNavigation() will harmlessly no-op if that flag isn't set.  But I'm happy to change it if you'd prefer.
I set both Opera and Chrome to German, and the messages are still the same.
Do we actually have a German Getting Started page and is reviewed?
I broke them out, so we can add the JSDocs as part of the JSDoc task.
We need a bunch of new dialogs for diverse error messages and the open dialog. In the future this should probably be replaced by a smart template (alongside the other dialogs) as the current approach does not scale well (just keep adding more dialogs bloats this file too much in my mind).
Nice performance data! 

I agree with you. We can make the decision later on.
The apf layer is used as a data holder - we only do `getAttribute("attr")`
This helps to gurantee the consistency between what the user sees and what the debug is doing
Adding a layer may introduce state inconsistency
This replaces NLog's LogManager
Is the idea making this a dict instead of just emitting the name that in the future you will emit other categories of data?
This shouldn't be necessary when it's installed as a gem. Should be removed.
Oh, whoops. Yeah, I meant to add it as the default for the field. Your other comments make much more sense now. :P

I don't think it will change the DB, and certainly it doesn't need to change it, so even if it did not having that change wouldn't break anything.
Change this to + Layer
Same layout as above.
Can you just add two more buttons to the layout and show/hide them as necessary instead of cloning/appending things here?
multi-lined object
`$httpCode` devrait être en paramètre de la méthode, non ? parce que notice, là, à priori
No, I'm saying we should maybe consider renaming it in react core.
You can find a list of the Capybara-provided RSpec matchers [here](http://rubydoc.info/github/jnicklas/capybara/Capybara/RSpecMatchers/).  The options hash is not very well documented, but you can write the above test more succinctly as:

`page.should have_css('#state', :text => 'creating')`

There is also a :count option to find a specific number of matching elements.
Consider using an Error Payload for this, since it is basically a validation failure -- remember each client side module will handle errors differently.  The IRC module may want to render errors in the same way it renders system messages, for instance (or maybe slightly different using red text).
for brevity we can omit the "be able to"
Yeah that's a bit clearer
Let's keep this one out of master, I think we had some problems with it causing the build to fail.
So we no longer distribute the Gemfile. This is fine. We delete it... +1
This should handle all valid cookie strings.
Yes, that's the whole point. The method providers can attach specific invokers to the method descriptors they provide, to achieve desired behavior. By default there is a `MethodInfo.Invoke` inside, called on the provided target IHub, but it can be just anything (like eg. firing some event).

Eg. in the Orchard implementation, I'd put there some code firing a system event with given method name and given parameters, without even having to implement a method on the hub object itself
Two typos in this line:

* whether/if duplicated
* automatically (missing 'o')
Also, `.. versionadded: 2.5` should be added to all new functions and macros, as this will be a part of the 2.5 release.
this.view -> view
conv**e**nience
Probably a minor point -- but generally I would prefer duck-typing here for the mapping type.  What about things that behave like a dictionary, but don't inherit from `collections.Mapping`?
it's definitely a noticable performance difference:
```
In [20]: timeit re.search(pattr, '+12:52:32.423')
1000000 loops, best of 3: 1.47 us per loop
In [24]: x=re.compile(pattr)
In [25]: timeit x.search('+12:52:32.423')
1000000 loops, best of 3: 809 ns per loop

```
So I will adjust it as @iguananaut suggests while trying to preserve readability.
Yea, I'll make this change so things are less confusing.
Do we really want to rename functions in log messages alone?
I like getting nit picked sometimes, at least people read the code and help making better code :). Oh and updated!
Why can't this formula be used in the justcheck case as well?
Why is this being cached? And why are you caching a pointer that will be reported as a memory leak at shutdown by memory leak catching tools?  (why not just static Array mutable; if mutable.size == 0 ... then initialize....)
Indentation is not correct here. Also there should be space before and after `=`. You can use the [codesniffer](https://github.com/cakephp/cakephp-codesniffer) to avoid such errors.
I know this isn't related to this commit specifically but this `firstType` thing looks suspicious to me, because items of an object literal aren't ordered.
Haha agreed.
I'm ok with `allow_exponent`

@mchilvers I'm not arguing with the theoretical correctness of your statement.  I'm arguing with the practicality of putting such a long name into production, then having to deal with the griping of the hoardes of people that will chastise me for the name being too long. I already get complaints that Enaml is "too verbose", I don't need to add fuel to the fire.
Let's change this to `type` and make it `"buy"` or `"sell"`.
Again, you should make this call somewhere during initialization so that it's only called once. OpenGL Get calls can actually be quite expensive.
After using it more, I think we should make the default background color be some shade of gray, so that the user has a visual indication of the boundary of the region which is active for mouse event handling.  Having it be white is really annoying to use, because it's hard to tell if you've gone off the edge or not.  We could use the same color as our boxes or something else, I don't care much.  But white is suboptimal here, I think.
throw Throwables.propagate...
Store and use result of tenantNameAndUsername.indexOf(':').
I think you are walking the reflog from oldest to newest here instead of newest to oldest so -1 is grabbing the first checkout instead of the last.
:metal: Loving these tests
Updating comments. :cake: for you!
Interesting. Would it be acceptable to use a regex in `further_checks_on_spec`? I know we like to keep our CPU time down, but this api isn't called all the time, and this would only happen if there's a "-g" in the spec.
To make gcc (4.7.2) happy (-Wunused-variable|-Wunused-parameter), we need to mark those unused, i.e.:

```c
#ifdef GIT_THREADS
  int result = git_mutex_lock(&pb->cache_mutex);
  GIT_UNUSED(result),
  assert(0 == result);
#else
  GIT_UNUSED(pb);
#endif
```
Thanks for the quick work.

I'll just mention that known limitation in the release notes and if we can resolve it easily enough during beta testing that would be ideal but if not we could always add newsgroup support in 1.2 later in the year.
I'll try again - but in the past I've found I do need to put in the non-breaking space...
Instead of having the `node` and `nodeview` functions, what if you just add `*args` to the `Node` constructor and store them as the children of the node? Then you could just nest nodes naturally and allows for the future extension of having parent nodes associated with pages, rather than only child nodes having pages.
I would rather see the 3 views and 3 pairs of templates merged into 1 template and a pair of views (add/remove), that took a parameter to decide between leader, reviewer and editor. Do you think this would be too obnoxious?
oh yeah, oops. but yeah, we prefer parens over the ugly `\` :)
I can't imagine there ever being more than two, but there could be with new identity solution for b2g.
Please don't do this. `create_switch` is sufficient.
I think it would be a good idea to make it more generic and define the configuration as "Croogo.corePlugins" for instance (and update the docblock)
An important attribute of writing a successful software library is to not only support well the intended scenarios, but also make the design so open that other developers can use it in new and unexpected ways. While you and I may have certain use cases in mind, I believe it's a fallacy to think that these are the *only* possible cases.
I don't know Italian, but isn't there an extra 'b' in this one ?
On Tue, Nov 06, 2012 at 02:01:17PM -0800, Simon wrote:
> >  
> >  app = Flask(__name__)
> >  app.config.from_object(__name__)
> >  app.config.from_envvar('C2CSTATS_SETTINGS', silent=True)
> >  
> > +Environment.directory = os.path.join(CACHE_DIR, 'assets')
> > +# add version number in webasset objects URLs
> > +Environment.url_expire = True
> 
> C'est la valeur par défaut, tu préfères le spécifier ?

Selon
http://elsdoerfer.name/docs/webassets/environment.html#webassets.env.Environment.url_expire

 « The default behavior (indicated by a None value) is to add an expiry
   querystring if the bundle does not use a version placeholder. »

... ce qui correspond à ce que j'observais chez moi: les fichiers générés
étaient servis par gunicorn sans numéro de version dans le query-string.

C'était pour activer cela que j'ai suggéré cette modif (ça nous sera utile
lorsqu'on mettra l'appli derrière varnish).
Even better than using the actual encoding? Just explode if the actual encoding isn't UTF-8. UTF-8 is the only encoding we should use here.
навіщо нам ці таблиці?
it's $content :)
`if ($this->callback)`  is enough.
I could make `ABSOLUTE_URL = true` and  `ABSOLUTE_PATH = false`, so there would be no BC break at all and the conversion would not be necessary. The disadvantage is that the referenceType param could be both a boolean and a string.
`if (INF !== $this->maxDepth)`
Blank string and SQL NULL should never be conflated.
Does this need to be lazy?
user id issue mentioned above.
Is the "inline-block" needed? Maybe simple

```javascript
$('#uid-error').show(); 
```
would be sufficient.
I'll need to review (rust), but if memory servers this syntax is correct. I also didn't write this test. This is one of the many things in this branch that just got moved not actually written.
已修正
for pedagogical purposes, remove comp
You can't remove an migration. If you need add an old model, you need create an new migration adding the model again.
oh le code non formatté! :-)
This is clearly a stub method but I think this should be refactored in the a view class method since it will require the current request/user to determine this info.
like 0 < bits_needed > 128 better?  I think it just kind of looks odd anyway (too xmlish).  Should I change to 0 < bits_needed or bits_needed > 128?
I think you probably also want to check that there is only one parameter and no hash parameters. This will future-proof the transform if we decide to add more features to these constructs.
I think it's good practice to make things like this objects with a well-defined interface. Otherwise, I can't tell what information is in here without searching the whole file.

If you want to keep things extremely light, even a ``collections.NamedTuple`` would do.
Il n'y a pas de "e" à "profil"
I would actually do a lookup here to make sure it worked
There is no need to load resources directly using the TCCL. Shrinkwrap has much nicer convenience methods for doing the same thing
you would not need override registerOperations method at all if you register add and remove methods in constructor.
It should be 'lets' and 'change'
Could you add a test for `track.playable()`?
This is REALLY slow. Please only `transition` the `transform` property unless the effect you want is not achievable from that. (a quick glance seems possible to me)
@lightsofapollo <3