labofsahil/github-event-dataset-2012 huggingface dataset:
I'm a little confused as to why you're adding the // if there is no scheme.
This one does not work as expected ...
Use "..." as a workaround
(The parameter comment is still to be rewritten, I assume?)
Today not much. But in future could help to drive selective cache expiration for example...
@pzi1win2go3 我不太懂这个为什么有两个 `search`
funny word wrap. OG--
they do, but you don't need `prefix_only`
What we're trying to do is add the id into the parent array if it isn't already in there. array_key_exists() is the wrong function to do this job. That may have been after a long night at the keyboard. 

Instead it should just be in_array() which should do the right thing. This would've been a heck of a problem to track down, so you did good. I'm going to close this request and put in the right fix. Thank you!
Does `Docco` uses now `Scaladoc`?  If it is, can you make a pull request to `inca`'s circumflex repository?
jeje ok no hay drama
For now I made both Bool and Char neither signed nor unsigned since they both have "different" behavior. With the proposed rules, they'll both be normal unsigned integers. Interestingly, Uint1 and Int1 are exactly the same.
Da hast du Recht. Aber keins von beiden wird durch `Gruppieren von Werten durch Collections` deutlich.
Ich finde den Syntax-Parser auch toll, vor allem das show-Kommando
Not sure if this can be fixed, but .mo is an extension mostly used by compiled GNU gettext (translation) files. Those are now recognised as Modelica files.
@Iriskin I think here the logic is better. Please check this.
You totally RTFW'd m...
Hmm, this now makes this method locale specific - we need to start using Zend_Date consistently. That applies to other methods in Date as well though, so beyond the scope of this pull request.
feel free to make an `inapp-payments.less` stylesheet
There is no difference between class and struct in C++ except for the default accessibility level.

I don't get this.
Done, sir.

On Sun, Feb 12, 2012 at 10:37 AM, Raghesh
<reply@reply.github.com>
wrote:
> Yes you are right. Please fix it.
>
> On Sun, Jan 29, 2012 at 9:13 PM, Ershad K
> <reply@reply.github.com>
> wrote:
>> Sir, this must be sys.exit(1), right?
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/raghesh/gcalread/commit/556a9955c3f6f110b32f1f8eebf3b36c1d56b343#commitcomment-912220
>
>
>
> --
> Raghesh
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/raghesh/gcalread/commit/556a9955c3f6f110b32f1f8eebf3b36c1d56b343#commitcomment-957387



-- 
Sincerely,
Ershad K
http://ershadk.com
Dont' we need this?
agreed! ha I can barely read it a few days later. But is the if/else one
too clunky?

Btw, on the HS application, do you have an issue with me referring to you
as a JS mentor?

On Wed, Mar 7, 2012 at 8:02 AM, Cameron Westland <
reply@reply.github.com
> wrote:

> While the shortest version might be the least amount of lines of code. It
> may not be the most readable. The ternary operator is especially confusing
> to read weeks after writing it. In this case you have three nested ternary
> operators on a single line!
>
> -C
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/mcranston18/katas/commit/c664d4c594660719cb80d5cc57d824b594daee64#commitcomment-1053094
>
le patch ne passait pas, j'ai du le faire manuellement, mais j'ai
rajouté ton nom dans le code


Le 06/01/12 10:16, Yann Droneaud a écrit :
> Oh, I thought you would commit my patch using git am, just to keep author name, give credit, etc.
>
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/Dolibarr/dolibarr/commit/33049e9f1997849cd86f5a09fcf9c1f36528cd81#commitcomment-843046
>


Cordialement,
-- 
Régis Houssin
---------------------------------------------------------
Cap-Networks
30, quai de Verdun
71700 TOURNUS
FRANCE
VoIP: +33 1 83 62 40 03
GSM: +33 6 33 02 07 97
Web: http://www.cap-networks.com/
Email: regis.houssin@cap-networks.com

Dolibarr developer: regis@dolibarr.fr
Web Portal: http://www.dolibarr.fr/
SaaS offers: http://www.dolibox.fr/
Shop: http://www.dolistore.com/
Development platform: https://doliforge.org/
---------------------------------------------------------
I've got something similar already implemented, just haven't pushed it up.  It's called at the end of .launch instead of doing a polling loop.
lol, I don't even know how you managed this
Fixed in 330000cc9ac1288855ed4e15ad8fc619fd289780.
adding google's crypto for SHA256 support
![So much win](http://thelaughingmarcus.files.wordpress.com/2009/08/so-much-win.jpg?w=445)
bug #?
Can we re-use the stuff in services/utils.py? I think this is all there already...
don't you want to get the count before we take the slice?
not storing the callback name on the image object breaks the DOMCleanup
change endpoint back to URI, then do .toAsciiString later
Breaks my build on linux, Mandriva-2010.2 x86_64.  I get the following error:

`/devel/games/zelda-solarus-dx/solarus-20120219a/src/movements/StraightMovement.cpp: In member function ‘double StraightMovement::get_speed()’:                                                        
~/devel/games/zelda-solarus-dx/solarus-20120219a/src/movements/StraightMovement.cpp:89: error: ‘hypot’ is not a member of ‘std’
make[2]: *** [src/CMakeFiles/solarus_static.dir/movements/StraightMovement.cpp.o] Error 1
make[1]: *** [src/CMakeFiles/solarus_static.dir/all] Error 2
make: *** [all] Error 2
Command exited with non-zero status 2
Not sure if that would do it in mysql-base, depending on what you like to achieve. Existing installations with this problem will not be fixed when upgrading if placing it there. Fixing it with mysql only is tricky, but running through it with php and run utf8_encode/utf8_decode on it and store it back(with the proper encoding set for the connection) will fix it.
wofür braucht man das?
This is definitely going to be a "nice to have" feature for later. But one idea is to publish multiple versions of the assets (one for each year), and leave the option of choosing which one to use when you load the page.
This probably doesn't work, but I hope illustrates the intention:

``` js
if (!has('amd')) define = (function(def){

	return function(name, deps, callback){
		def(name, deps, callback);

		if (typeof name != 'array') name = [name];
		var module;
		while ((module = name.shift()) Core.global[module] = require(module);
	};

})(this.define);
```
u can have a try
Unstable affliction dispel should ignore resilience, http://www.threadmeters.com/p9pd3x/Unstable_Affliction_Dispel_Ignores_Resilience/
Basado en: http://www.warcraftmovies.com/movieview.php?id=133081 minuto 36:20... Gracias @Pimbai
@kamranayub: You could steal it from [Mono](https://github.com/mono/mono/blob/master/mcs/class/corlib/System.Collections.Concurrent/ConcurrentQueue.cs).  Note licensing terms.
Use object based parameters instead of array based parameters since wants allow the users to pass custom parameter values through the `nmModal()` call with `ajax` option.
Why is this defined here if it's only used in one spot? Why can't we have it inside the `finalized?` method and then move it out when we need to.
That's it. Looks like if I set »Clip Subviews« via IB it has no effect, but it works when set it programmatically. Thanks.
Funciona no linux agora o dojotimer?
I see that singleOpt is not necessary anymore, just leaving it out works, thanks a lot!
strtotime() is called in quite a few places, did you check that none of the remaining calls are affected by the behavior you're trying to fix here ?
Thanks for your good commit.
This should probably just be `admin`, though.
@rotty3000 I could not find the format-source in current build files. Is this part of a toll package?
Didn't do this because it would overwrite the value if the user had purposely save it as None.
is fixed, thanks.
 also added the missing filter for vc100
You make a persuasive argument. ;) I will commit a patch shortly.
I can't begin to express how much I like this and how much I'm learning in the process. Thanks Ross.
@odorcicd what's your release policy? I noticed there are several dozens PRs waiting to be reviewed.

Thanks!
Hehe, no problem then, I just got carried away by the recent typo-drama :P
Damned trolls... where's the ban button?
It's used in all other spell_X.cpp files
If a file is unused just delete it.
Good call, it was indeed broken. Fixed: https://github.com/mozilla/amo-validator/commit/9fb9cad

btw, the assumption that manifest files end in .webapp or .json was incorrect, they can have any extension: https://bugzilla.mozilla.org/show_bug.cgi?id=721875
License changes are always a bit tricky and I am not a lawyer, so I can't speak on what is legally correct, but as long as you give your permission for this change, I don't see a problem.
Oh I see it just gets passed through. That makes more sense than prepending it to _property_ in the model but is more awkward to deal with as you need to check for null & add an extra '.' if not null.
Part | Points
------------- | -------------
base | .5
*Total* | .5

Lots of errors and input requested (if you use it, comment it out for the next unittest based homework)

```
'tree' is not implemented
'normalize' is not implemented
----------------------------------------
test_greeter_basic (__main__.GreeterTest) ... ok
test_greeter_case (__main__.GreeterTest) ... ok
test_greeter_int (__main__.GreeterTest) ... ok
test_box (__main__.BoxTest) ... ok
test_box_error_too_small (__main__.BoxTest) ... FAIL
test_box_error_type (__main__.BoxTest) ... ERROR
test_distance (__main__.DistanceTest) ... 5.0
FAIL
test_point_in_box (__main__.PointInBoxTest) ... out of box

Point is in the box
out of box
out of box
out of box
FAIL
test_point_in_box_edges (__main__.PointInBoxTest) ... 
Point is in the box
FAIL

======================================================================
ERROR: test_box_error_type (__main__.BoxTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "hw08.py", line 69, in test_box_error_type
    self.assertOutput(box("", ""), self.error_msg)
  File "/Users/jacklaxson/Projects/homework-cs112/jackerma/hw08/basic_funcs.py", line 40, in box
    elif int(w)!=w or int(h)!=h:
ValueError: invalid literal for int() with base 10: ''

======================================================================
FAIL: test_box_error_too_small (__main__.BoxTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "hw08.py", line 64, in test_box_error_too_small
    self.assertOutput(box(0,0), self.error_msg)
  File "hw08.py", line 31, in assertOutput
    self.fail(msg)
AssertionError: 
Expected:
	Error: Invalid Dimensions
Actual:
	Error: Invalid Dimensions 1


======================================================================
FAIL: test_distance (__main__.DistanceTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "hw08.py", line 118, in test_distance
    self.assertEquals(distance((0,0), (4,3)), 5)
AssertionError: None != 5

======================================================================
FAIL: test_point_in_box (__main__.PointInBoxTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "hw08.py", line 145, in test_point_in_box
    self.assertTrue(point_in_box((15,15), box))
AssertionError: None is not true

======================================================================
FAIL: test_point_in_box_edges (__main__.PointInBoxTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "hw08.py", line 153, in test_point_in_box_edges
    self.assertTrue(point_in_box((10,10),box))
AssertionError: None is not true

----------------------------------------------------------------------
Ran 9 tests in 0.002s

```
ok... yes, there are all the things I've said here and in my pull requests. So, your proposed resolution is no change?
I use it as "Cache".
Sqlite is reading very fast!
I save it for 5 Minutes.
So i can answer Questions very quickly!

...
Question: Send SMS to Billy
Answer: Okay! What you will say?
Answer from User: Hello Billy, whats up?
Answer: Okay, Should i send the SMS to Billy:
Hello Billy, whats up?
Answer from User: Yes
Answer: Okay
...

For Commands like this, you must safe this Data.
With Assistant and Speech IDs you can made Answers and Requests easy.
The SMS Text and the Receiver saved, too!

All for 5 Minutes, only.
After 5 Minutes -> or <- after a finish Request "Send SMS", "Yes/No" the
Datas will be deleted.

Without Saving... i cant image how a person will realize "Send SMS" oder
"Send Email"... All Text-Saves ^^
Phone ... i mustnt save ^^
Mr. Blondel, you are going strong ;)
不算
在 2012-02-25 12:15:41，"Tusooa Zhu" <reply@reply.github.com> 写道：
>在不同universe注册账号算犯规不。
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/TeliuTe/teliute.github.com/commit/e838412ada4b6c273a5aab8ce1a6103179732c99#commitcomment-1010256
...a litte readme would be great to close #35 :)
Do you think that a string representation like this is sufficient? We may want to at least do something like java.lang.String[], which will avoid confusion.  It could then be assumed that anything not included in the json is provided by the language.

Probably not necessary for primitives, but then primitives may vary from language to language, so possibly we should define a set of 'approved' primitives from the intersection of C++, Java, and C# (the top three languages on [Tiobe](http://www.tiobe.com/index.php/content/paperinfo/tpci/index.html)).
"this method fire*s*"
That was a bit of confusing behaviour to return what you set..
<strong>1.0.4</strong>
Nice Icons! :)
I changed the casing somewhat (variables start lower-case, classes start with upper-case normally), but I hadn't noticed it being used in that variable. I don't think there's too much impact otherwise.
What are the sources of all those coeffs numbers you give? I can't find any info about those numbers in addons 4.0.6a or wowwiki, wowhead, wowpedia, or wow patches.
Post output of git remote -v

You got full access so no way this can happen
 On Feb 4, 2012 8:44 AM, "Scott Brady" <
reply@reply.github.com>
wrote:

> The problem is I only have read permissions to the two repositories I
> already moved to sgt7 from my account, if you can't change the permissions
> delete them and I will fork it directly to sgt7.
>
> On Fri, Feb 3, 2012 at 9:57 PM, Chirayu Desai <
> reply@reply.github.com
> > wrote:
>
> > Fork it from cm github, then cherry-pick your commits or add your
> > repository as a remote, then fetch n merge it
> >
> > And yes - setup remotes properly
> >
> > Git@github.com:sgt7/xyz.git
> >
> > Use git remote -v to check
> > On Feb 3, 2012 11:32 PM, "Scott Brady" <
> > reply@reply.github.com>
> > wrote:
> >
> > > Thanks for adding that, can you make our frameworks/base and settings
> > > read+write.  When I moved them from my github it revoked the
> permissions.
> > >
> > > ---
> > > Reply to this email directly or view it on GitHub:
> > >
> > >
> >
> https://github.com/sgt7/android_packages_apps_Phone/commit/59f7eaaf203b634ce39f6a0625ebcfda4198d77a#commitcomment-931174
> > >
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/sgt7/android_packages_apps_Phone/commit/59f7eaaf203b634ce39f6a0625ebcfda4198d77a#commitcomment-932531
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/sgt7/android_packages_apps_Phone/commit/59f7eaaf203b634ce39f6a0625ebcfda4198d77a#commitcomment-932552
>
This is a way to get the versionName of this application from Manifest.xml .
I'M WORKING ON IT.
The problem is not that all files are concatenated but that the jasmine runner is executed before require.js executes the require() functions with the specs inside. It seems that waiting for window.onload to execute jasmine is not enough to include the require() wrapped specs.
@pcfighter You're only missing those 2?
I wasn't really seeing this being used anywhere, so I eliminated it. Let me know if it needs to go back.
does this work for xhdpi as well or do we need a new icon for that and thats not the drawable i was referring too for settings the one from the pulldown changed to white
Oh, and the page header is wrong.. the bug queries are for all components in the D project, which includes phobos, druntime, the website, etc.  Not just dmd.
This commit causes strange bug.
In example: Given a media file named: _01. Smoke On The Water (Deep Purple Cover).flac_, this code will try to treat it as an album art image! ( _"SkImageDecoder::Factory returned null"_ in logcat).
Moreover, the media file is copied to _albumthumbs_ cache directory...
I suggest a small change to the condition - file extension check:
```java
(...)
if (entry.equals("albumart.jpg")
|| ((entry.contains("cover") || entry.contains("front")) && entry.endsWith(".jpg")) {
(...)
```
Any thoughts on whether an empty list of specs in any_of should be an error? If the spec is dynamically generated, maybe better to be an error since that seems like it should never be the intent? *shrug* Fine as-is too.
Does the `sub_vector` argument needs to be a pointer, or could it be a `std::vector<double>`? How do I know how long the `sub_vector` is?
I actually think it's easier for people to distinguish between classes and variables, when the classes hang off the `Y` object.
Yep, the default search seems to handle them.
Per Scott's comment up above, please ignore this one :)
But I can change it to a comment :-)
this should be in `finally` block as it was before, because handleSupervisorFailing is not trusted code (potentially)
Perhaps mention what a PDU is?
protocol data unit?
only those which were sent within `Failed` in the other direction already.
Use collectFirst instead of find
I can add check (stupid user exception)
condApply?
So what happens when the queue is full? How does Akka handle queue length internally (I am asking so we get similar semantics...). For the time being, I have not changed this.
The first thing the user would do in transform they would cast it so partial function seems to be good from the usability point of view. ie.
override val transformResponse = { case msg: CamelMessage => msg.bodyAs[String] } 

so if we change it to (Any) => Any we are forcing user to type more:

override def transformResponse(msg:Any) = msg match { case msg: CamelMessage => msg.bodyAs[String] } 

I agree with the val vs def let's change it to val.
why move this here?
it's not something that should be done in general
This part might belong in one of the config/* files
You're asking me?  Oh, you mean just the Card:: prefix.  Not sure about that.
s/at al./et al.
Not sure why this change, looks like it breaks the default group type setting
These don't belong in the static configuration for a repo. They are one-off options that should be passed to the publish call itself. It doesn't make much sense to store a start/end date in the repo itself, that's a config option for a particular run.
This is a good call, will make it easier to test.
Could replace these two lines with:

```
distributor_config.setdefault(k, False)
```
This is not the right way to go about this. Try `rb_enc_asciicompat`.
What about ZookeeperConnected? Should we restart the cluster?
Yeah, that would be the best fix, as you could tack them straight onto the season fetch easily enough.
Never see the artist type in my tests but will update the commit in 2 min with your suggestion.
please remove these extra empty lines in the phpdoc
It is important to retain WebDAV as well as Exchange in this description.  Newer Exchange servers use ActiveSync, which we don't yet support.
Sorry - I did not mean to commit the second part of that pull request.   I
noticed that there was a fix that was supposed to handle this so I have not
looked into that.

However, it is broken when I run the specs here on the updated version:

 3) Dzl::Examples::FunWithParams /foo/:bar (:bar as time) converts :bar to
time
     Failure/Error: JSON.parse(response.body)['params']['bar'].should ==
'2012-01-01T00:00:00-05:00'
       expected: "2012-01-01T00:00:00-05:00"
            got: "2012-01-01T00:00:00-08:00" (using ==)
     # ./spec/fun_with_params_spec.rb:113:in `block (4 levels) in <top
(required)>'
     # ./spec/fun_with_params_spec.rb:111:in `block (3 levels) in <top
(required)>'



On Fri, May 11, 2012 at 10:55 AM, justin-vitrue <
reply@reply.github.com
> wrote:

> > @@ -110,7 +110,7 @@ def app; Dzl::Examples::FunWithParams; end
> >      it 'converts :bar to time' do
> >        get '/foo/2012-01-01' do |response|
> >          response.status.should == 200
> > -        JSON.parse(response.body)['params']['bar'].should ==
> '2012-01-01T00:00:00-08:00'
> > +        JSON.parse(response.body)['params']['bar'].should
> include('2012-01-01T00:00:00-')
>
> Are there + offset timezones?
>    include('2012-01-01T00:00:00')
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/vitrue/dzl/pull/15/files#r809908
>
We shouldn't reformat the license layout
I had problems with this.  I think I just have to find the right place for it.  We need a "Wagn initialized" event and hang this on :before_init
probably i was too cautious,
copy wouldn't be called when ace is hidden or not in document so this might be ok
This fails sometimes because `bigInt2bits` trims leading zeros and `r` is random bits. So, there's a slightly greater than 1/256 chance that bytes are missing. From the spec,

> (MPIs must use the minimum-length encoding; i.e. no leading 0x00 bytes. This is important when calculating public key fingerprints.)

We could set the most significant bit to `1` to make the test pass consistently, but that doesn't seem right.

    // bigInt  randBigInt(n,s)        //return an n-bit random BigInt (n>=1).  If s=1, then the most significant of those n bits is set to 1.

Any thoughts?
On 07/06/2012 11:15 PM, Tuncer Ayaz wrote:
>>       (catch eunit:test([list_to_atom(Suite) ||
>> -                          Suite <- string:tokens(Suites, ",")], EunitOpts)).
>> +                          Suite <- string:tokens(Suites, ",")], EunitOpts));
>> +perform_eunit(EunitOpts, _Modules, {generators, Generators}) ->
>
> OK, so ``tests=foo_test,foo_test_`` should only run the test ``foo_test`` but not the generator, right?

No, as I said before, if you list both as you did now, the "foo_test" 
should cause foo_test() to run, because it's a shorter match, and 
"foo_test_" should cause foo_test_() to run (since it's not a prefix of 
foo_test()).

    /Richard
very cryptic name indeed
No, this isn't. The old remoting is not removed yet!
it should not differ. If you have a different extension, it will trigger a new compilation when auto_reload is true.
@cryptocompress concat and divide again ? where ? There is only concatenation here
This is useless. If ``__call`` is defined, none of the ``deunderscoreItem`` cases can be reached
why the check on ``extensionInitialized`` ? Initializing the extensions does not register the globals anymore
IMO it should be `if ($withContext) {`
Because you want merge the context only if `true ==$ $withContext`.

I looked at the tests, and I think there are wrong.

May be I really miss someting.
This does not at all address the issue I raised before. `rangeFlavor` is already an `NSUInteger` – it needs to be cast to a `long` in order to be used with the `%ld` format specifier.
Why change to :type, the C code doesn't expect self
good catch - I fixed with a new commit.
``pendingPromises``
Do you mean 8 spaces in addition to the 8 spaces already there?
Am I understanding right that the issue here is that:
* There is some special-case code for handling indexing in 1-d arrays
* This special-case code does not actually work
* You're not feeling inspired to fix the special-case code (or maybe it's not even worth fixing, because special cases are hard to justify...)
* So this code is just disabling that special-case code
?

If so, I really think the proper way to disable it is to delete it all. Keeping dead code around is really not worth it. It's not as useful as it seems (if anyone really wants it, it will still be available in git, and anyway if anyone gets inspired to optimize this code there's a good chance they won't find the old code particularly useful anyway), and the maintenance cost is much worse than it seems (dead code builds up and it soon becomes impossible to figure out what is actually going on). Let's keep the code clean.
You probably have an indentation issue here (or some \t are left somewhere). Moreover I don't often use "and" when it is not mutually recursive.
Just for consistency - those are all indeed case-insensitive, but RFCs all mention them with starting capital letters. I know it's irrelevant to the feature itself, but figured that since it's not a 300-line diff it won't be such a distraction.
Not that it makes much of a difference, but is this `memset` needed? If I see it right then you are fully overwriting `K2` anyways in the `php_hash_string_xor_char` call below.
The first sentence here is pretty close to the first sentence of the intro above. Both begin with "Migrations are a convenient way." It could probably just stand as...

Migration Overview
--------------------------
Migrations use a Ruby DSL so that you don't have to...
no need to say `self.` here anymore
Done https://github.com/rails/rails/commit/e6e3317c5409b069782e7b701a11bcd5283794ae review please /cc @frodsan
Since those are exceptions and not Context errors strictly speaking this really doesn't belong to this file.

I don't really like that exceptions again start to be part of the control flow because it can easily get out of hand. Macros.scala is your territory really so I am not going to chime in but to me it just doesn't look good. Making it more resilient to working with erroneous trees/symbols etc is a better way IMHO.
Yes I will. Documentation will come in the RC phase.
right - good catch
Space after `Runnable` before `{`
since you only ever use `done_list` as a set you can just write `done=set()` and then instead of using append use add (done.add(i)).
I would suggest something like the following

```

>>> def check_pair(a, b):
...     idx = -1
...     for _, (i,j) in enumerate(zip(a, b)):
...         if i != j:
...             if idx == -1:
...                 idx = _
...             else:
...                 return -1
...     return idx 
... 
>>> c=check_pair
>>> c((1,1,0),(1,1,0))
-1
>>> c((1,1,1),(1,1,0))
2
>>> c((1,1,1),(1,0,1))
1
>>> c((1,1,1),(0,1,1))
0
>>> c((0,1,1),(0,1,1))
-1
```
This one amused me. I can imagine a hardcore python syntax evangelist going insane over it.
I certainly think that we should use an unordered structure (frozenset or dict), but as long as we support `Add(2, 2, evaluate=False)`, we can't .
Why was this change necessary? It is used as a tuple in the next line.
stdlib imports at top (I think you and Ronan had mentioned that this is the preferred place, though this is inconsistent across sympy).
I think it should be particular model class.
Replace with `CPagination|false`.
If we break out a positioning helper:

top, top-right, right, bottom-right, bottom, bottom-left, left. offset & threshold.
You're right; malloc.h should certainly be in `#ifdef _WIN32`.  (I only included that for `_alloca`)
Changed them back to the old version.
Looks really good, almost there. testify takes a --no-color option, which is used by ColorlessTextTestLogger (on bb, for example). It's not perfect (--summary ignores it), but you'll want to set the `color` attribute on your class from this option in test_program.py (keep the isatty default, though).
Funny that you had to write all that code up there to return a good error message when all you really care about is this line :).
Please change this to the more explicit `Object.keys(obj).length === 0`. Also, Object.keys will throw an Error if something other than an object is supplied - so it should probably be coupled with `typeof obj === 'object'`
could you please elaborate a little on where the code does not comply with the SF2 coding standards? I've found the standards and can't really spot anything that violates it unless there are specific coding standards for the SF2 unit tests?
Сделать все xml в файлах. Пусть пути будут заданы явно.
Return 0 on success, -1 on error.
I see the problem:
darwin contains win and it checked after Mac checked after Windows.
I will fix it.
Creo que tendríamos que devolver el ID acá, o lanzar una Exception
well MongoDB isn't sorting in these notional operation. it's just using the index order as the sort order, so returning docs from the cursor in the order they appear in the index?
Hrmm... this looks a bit awkward to me.. don't you want something more like:
```
tokens = search.strip().split()
query = Q(is_active=True)
for token in tokens:
   query = query | Q(owner__last_name__icontains=token) | Q(owner__first__name__icontains=token) | Q(title__icontains=token)  | Q(description__icontains=token)
projects = start_set.filter(query)
```

Same goes for below.
There is a ui.confirm method that has the benefit of respecting the -y option.
Missing docblock for the third parameter
This value should be unique for a particular web server as sphinx may be being used for indexing apart from phpbb.
I would clarify somehwere that `parse_github_response` takes an object, data, that has either a message field to indicate an error, and a data field with a pulls field that contains a list of Github pull request objects.

And then comment here on that is why this extra work is being done. And maybe change that above contract.

You see, my first thought was why not concatenate the two results together? And then map the JSON.parsing. We don't need extra checking.

But then I realized that parse_github_response had an error case. 

Another option is to call parse_github_response on each result, and the concatenate the results of that together. 

I'm not sure, but the logic here is hard to follow whereas it could make sense or reminiss on something familiar.
te rog folosește construcția „ceva in altceva”, nu e nevoie să apelăm metode de tipul __contains__.
De asemenea, poate ar trebui să nu trimitem asta în clar în session?
aici cred că e suficient de adăugat "seen".

Poți face `if action in ("login", "seen")`.
This is all about proper method names. See, there are 3 different options for shadows: Mesh can cast shadow, Mesh can receive shadow and Light can emit light that gives shadow or not. I will reconsider these methods' names and document them.
The lookup would find `SwtShoes::Shoes` in 1.9, which would be recursive in this case :/

As I haven't been around too long, I'm not sure why where's both `Shoes` and `SwtShoes::Shoes`, but it was necessary to make the specs pass.
nice!

I was thinking that we might just want to move this to `./lib' at some point since it's not really a part of the app
Why are we exporting `dump`?  This appears to be an internal method.
Both property names should be prefixed with `this`
the handler on lines 17-25 is identical to the one on lines 31-39. Can you abstract to a common function 
```
var enableButtonsHandler = function () {
  ...
}
```
This should return and error (e.g. -1) or throw an exception. The calling method handle the error code / exception appropriately, rather than "quietly" failing with a console.log statement.
read renamed to seen
See for example link.py line 177
The indentation of this line and the lines below may need to be cleaned up.
``if owners else ''`` unnecessary in the ``else:`` block.
`m` используется один раз, можно в одну строку записать
Please use 4 spaces for the indentation, not tabs
Any chance this can be kept readable while still valid for edoc?
Both do it wrong, what we need is a Transport:secure() that says if the connection is secure or not.
Here is the "fix"
Эта строчка должна быть такой `if not value and auto_values:`. Пустой список вернёт False: http://docs.python.org/library/stdtypes.html#truth-value-testing
All these error list related functions, if they're going to be included at all, should not be in the public header. If they are in the public header, that means they're part of the API and need a WEBVTT_EXPORT annotation, as well as the webvtt_ prefix to indicate that they're part of the libwebvtt "namespace", avoiding potential name conflicts.
@jdalton a minifier probably shouldn't do stuff like this. See: `"11" % 10 === 1`. It's safe here because we **know** `nodeType` is always a number.
You are right, this has nothing to do with openID protocol. It's an attribute from user's profile(https://graph.qq.com/oauth2.0/me) and would be required as a parameter 
(plus access token of course) for any other QQ API requests, like SNS, E-Commerce etc...

The package persists this attributes as 'id' in user document.
blank and null values are not permitted. And on bug_526469 branch I've added HTTP_ACCEPT_LANGUAGE languages to the top of the choices list (which will be the default). Do you think we still need a default value here?
I'm getting an sha1 mismatch. Did the author update the tarball?
Please use a URL that includes the revision hash.

`:p1` is the default, so this whole thing can just be

```rb
def patches
  "url"
end if build.include? 'with-dtrace'
```
If the `--python` option was not given, will this cause an error because the files are not there?
Да уж. Я, пожалуй, пойду ещё дальше и упрощу «вы можете слить или вы можете перейти» до «вы можете слить или перейти»:
> Вы можете слить (merge) её в одну из своих веток или перейти на эту ветку, если хотите её проверить.
Why move the optional/`*`d portion to the beginning of the regex?
We don't need a lesson on what volatile means.  That's easy to Google.  We need to indicate why these fields need to be volatile, and the others don't.
No need for deleteOnExit now.
SIGINT is a signal primarily created by Ctrl-C in the terminal--for any circumstance other than that (timeouts, restarts, etc.) it would receive a SIGTERM or a SIGKILL, which we don't trap.
if the experimental flag is set, _enabledAgentNames is replaced by agents, which in effect loads all agents.
This might have side-effects if, in the future, _enabledAgentNames is used somewhere else.
Also, I think once you fix the issue notes on line 73 the ```selectionInfo.endCh === 0``` case is unneeded here.  With _getSelectionInfo() changed, I think ```endCh``` can only ever be 0 if there's no selection, and the other clause of this 'if' covers that case already.
Please add Closure-style type documentation for params and return values (for this and all the other new functions below).  For details: https://developers.google.com/closure/compiler/docs/js-for-compiler#types
Do _handleKeyIn() and _handleFilter() both run every time the query string changes?  It seems like this case is covered in both methods, so it feels like one of them is redundant code.  _handleFilter() walks the whole list of plugins, and if none of them return true from match() it sets currentPlugin to null.  So it also covers that case.

Does _handleFilter() only get called some of the times query string changes?  E.g. it doesn't get called on backspace?
Yup, will fix when I finish up the unit tests.
It seems inconsistent that the fileViewFocusChange event calls _updateListSelection() directly, and the documentSelectionFocusChange event calls _handleDocumentSelectionChange() (which simply call _updateListSelection()). I think it would be easier to understand this code if both events called the same function. Can we get rid of the _handleDocumentSelectionChange() function?
I will remove it
DON'T MERGE NOW

I will add a change today to track popups and reload them on file save complete (as Ruben suggested) - (like the Chrome extension Vlad and Harrytun did)
Right now the design is to only show icons for completions that are type-driven; completions that always make sense. That allows users to distinguish them from other completions like textual ones or snippets. (See https://github.com/ajaxorg/cloud9infra/issues/1101.) I don't think we should deviate from that design, or we need something different.
Yeah. It made sense once I paid more attention. :)
Someone's editor has weird default indentation.
Here the Gravatar URL is hard coded to use http.
line-break, shoot for <= 78 characters per line.
Multiple tests were removed but I see only a single test added?
agreed on Virtus, will update it.

for the "Proper type validation could be added later when Virtus will get a way to tell us." line the intention was to warn that in its current form if you have this:

```ruby
required :id, type: Integer
```

It does not means you can access params[:id] and blindly consider it will always be an Integer, if the input is "dfdf" you will get that in params[:id]. I wanted to mention that somewhere but was not sure where, any idea ?
Yep. I'll make the sequence number arbitrarily large.
Let's kill this line since it's a console.log() and it's commented out.
Should `Report-Msgid-Bugs-To` be an e-mail? Maybe there is some other syntax for URLs?
Can you make these consts instead? `color` and `drawText` never really change.
Given the above comment, you just need:

_uiConfig.value( "ui" ).onLeaveDialog !== false
Can we just use a counter instead of `Date.now`? Might it be possible for two things to receive the same namespace if it happens quick enough? Something like `__guid++` guarantees that it won't.
You probably want to remove the listener once it fires.
Move this stuff to a module (not the part from 26-30, which can stay here).  Basically, we want to have something like src/util/storage.js and then require Storage, which should be the same as your __butterStorage var above.
For now, yes. No need for `set` here.
Yep, Pkoin a remonté ce soucis sur mon repo.
Je pensais passer par là, puis je ne l'ai pas utilisé.

Je vais corriger ça, et mettre Atoum en place, histoire de pouvoir tester + code coverage
Strictly speaking, to the user this is a button.  I know it's actually a link in the HTML, but to the user (which is how we write scenarios) it's a button.  The step def should be able to handle the fact we use links as buttons - that's why Capybara has link_or_button methods.
Хорошо бы убрать копипаст между `css` и `css-fast`
Not really except for trial and error (start empty, and see what's missing, add them in one by one), although the new build kit will pull in cross-project dependencies if defined.
This should also be added the listing. Merge vs mirror stream validation should validate empty minute directories as well
Might not want to add the backtrace join here. Aren't we already doing this in the controller when you raise an exception?
yes, 'and' is evaluated after % and before if, see http://www.techotopia.com/index.php/Ruby_Operator_Precedence#Operator_Precedence_Table
On 08/28/2012 09:34 AM, Eric D Helms wrote:
> In src/app/views/common/_footer.haml:
>
>> @@ -1,15 +1,2 @@
>> -%p.copyright
>> -  #{_("Copyright &copy; ") + (Time.now).year.to_s + _(" Red Hat, Inc.") }
>> -%ul
>> -  %li
>> -    %a{ "href" => "#" }
>> -      #{_("Disclaimer")}
>> -  %li
>> -    %a{ "href" => "#" }
>> -      #{_("Terms of Use")}
>
> In Satellite that link could potentially be appropriate after reading
> through it, but in the upstream version, linking to a set of Terms of
> Use for a website that is not even the web application itself just
> doesn't compute.
>

Yank um.

-- bk
Forgot about that, thanks!
well, try to use a FirePHPHandler as fallback here for instance. It will explode in your face (not even cleanly but with some PHP notice or errors as the code of the handler is not written in a way to be safe when the protected method is called with invalid data)
Change to `IList<HubDescriptor>`
I see this removed all over the place. Where is it triggered from now?
remove `su`.  the cron should be run as `postgres` user
nit: expand the content of lockscreenNotification
I think these were originally here as conveniences for the spectral density equivalence classes.  These should probably be moved to local variables in the `spectral_density` function.
just spotted another typo - sphinx
Perhaps add an example case usage case here in the docs if you can make a simple one?
A token/reset model works for me - at the risk of some extra complexity.
So remove ipcThread() and ipcThread2() from here, but leave the OSC ifdef and ipcInit()?
local variables strNotifyTx/Block are overriding the static strNotifyTx/Block here?
+1 @elemoine 

@asaunier we should talk about it, but I thing that I we want to create a stable version of a coherent package we should synchronised the version of CGXP to the version of c2cgeoportal.
I dont quite get this one
hmm, I see that now... My other idea is to force emoji to be prefixed with a space so that "some text;)" would not display winkface, but "some text ;)" would display winkface. that wouldn't require any fiddling with html encoding/decoding.
simply ``if`` :)
It'd probably be nice to have a friendlier mapping that doesn't use CCP's camelcase
I don't buy that :)  please enunciate exactly what you are referring to
I suspect there's a nicer/guice-ier way to do this rather than a static?
Got it.  Thanks. I will update the pull request.
Comment is pretty straightforward~ish. When setting a `<select/>` to an value that isn't in an `<option/>`, IE would reset the value to undefined where other browsers would just keep the previous value. So I force it back.
If h1_text is the same as title, why not do?:

    {% set h1_text = title %}

I'm just curious.
What about ``delete``? Do you consider that, as we currently have no DELETE in our middleware we should not make it an available option here?
Yes, we actually did start out just trying to call the same function as the
submit button, but we realized that it depends on having all of the fields
filled in on submit.  We then thought about modifying the existing function
and again realized it was better to leave it as it was so it would still
work for the submit button and if JavaScript was turned off.  So yes, we
did have to duplicate some of the code to make it work with what we were
passing in.
Team Bean Counters
On Nov 25, 2012 7:54 PM, "Todd Sedano" <notifications@github.com> wrote:

> In app/controllers/peer_evaluation_controller.rb:
>
> > +          unless user_alloc.nil?
> > +          alloc_answer << user_alloc.first[0] + ":" + user_alloc.first[1] + " "
> > +          end
> > +        end
> > +
> > +        allocation.answer = alloc_answer
> > +      end
> > +      unless allocation.save!
> > +          head :internal_server_error
> > +          return
> > +        end
> > +    end
> > +
> > +    render :nothing => true
> > +  end
> > +
> >    def edit_report
>
> This code looks very similar to def complete_evaluation, and is not very
> DRY.
>
> I had hoped that this implementation would improve the code readability of
> the existing code base, not make it harder to maintain.
>
> The original code successful did edit / update, right? So then the main
> change would be just calling it more often. I see that the control flow is
> different. Was there a bug with the original control flow?
>
> Or maybe I understand this better, your new code is just updating for one
> text box?
>
> My long term goal for peer evaluation is to refactor it into the model,
> however, that doesn't need to be done today.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/professor/cmusv/pull/135/files#r2218177>.
>
>
what is happening here? Please try and avoid whitespace changes.
Is this old leftover content?
How the argh I guess I rebased the wrong branch from an old checkout or something. Oh git, why you gotta. Rererererebased, the new patchlist is correct.
Thanks for the suggestion. Moved over the majority of the properties at e99de50a7fb3c2ab09117c9586c1b065ab1646a4 .
Mhhm, somehow "transparentGunzip" also looks strange. How about: "transparentGzipInflate" or "transparentGZIPInflate"?

But I'm also okay with the first solution. Just reply with what you think is the best solution, and I will change it accordingly.
Exactly the same ? not really as there is a ``if`` :)

But my point was that it cannot be compared with the ConstraintViolationList
why not naming it `createTemporaryFiles` like you have `createEmptyFiles`?
is it intended to keep a reference to the params if you don't use them ?
I would love some advice on naming the two `*Service` classes. I hoped for some feedback here on what they should be called based on what they are doing.

The goal of this class is to provide an implementation that can be used by `CommandBundleInterface`. Similar to a trait. This would allow a third party framework or application to extend `Bundle` and implement `CommandBundleInterface` or `ContainerBundleInterface` as appropriate and not have to duplicate the actual logic for `registerCommands()` or `getContainerExtension()`.

There may still be some copy and paste involved but most of the copy and pasted code would not be logic.

Traits would be ideal. Is there a better way to accomplish what I'm trying to do here? How would a class that does what I'm describing best best be named?
there is no typehint because we don't have an interface for it, and we don't want one: it would make the use of all call loaders more difficult by requiring to include the interface too instead of just the autoloader, and it would limit its use to the Symfony autoloaders whereas it currently works with the Composer one too for instance
Don't you mean `Regex`?
I am not sure if this extra commet is really needed ?
this change still kills this logging addition. Y U NO LOG?
Its in main.css and appeared to be used once we went back to the wide version of openbook
I wrote this for the json api, since we don't have visibility yet.  I'll modify it to something like comment.find where visibility > friends after that gets implemented.  I did write a quickie html template to display all comments just to avoid a template does not exist error when you go to /comments but I don't expect that to be linked anywhere.
Decorators don't extend APIs. This does. It works kind of like a decorators, but it's actually something else.
Chris I think you should merge this, and then I can add error printing.
Larry merged that pull request, so I will be able to use that code.
Es gab hier gar kein flush() mehr, daher wurde auch kein Training erstellt, wenn man den Command alleine ausgefuehrt hat. Ist das bei den anderen noch drin!?
remote = code_name
Haha! I like this. Good luck finding bugs if you do this all the time.
After some experimentation with criterium, it looks like Math/abs is indeed faster -- but only if type hints are added.
Yes.  dnsmasq functions as a DHCP server too.
Yes, i don't want to change the content.
No way. :-)

Use the management API or (shudder) xslt in the testsuite to add any destinations needed by tests. Use management APIs in quickstarts to set up any destinations needed by the quickstart.
This should have ThreadFactory so the threads have meaningful names instead of "pool-1-thread-2"
Nope. Not sure how this happened; the idea all along was to bind to `blur` (which the specification mentions), but that's clearly not what I typed.

https://github.com/jquery/jquery-ui/pull/772
American preferred spelling is focused.
Remove L43-L45
nit: `var gWpsPbcLabelBlock = document.querySelector('#wps-pbc-button a');`
It isn't, sometimes it has a number in it. See line 652 :)
Is it possible for `fetchSettings` getting settings after the settings SET on line 203? They are all async and you are presuming the mozSettings behaves as first come first serve.
also, this conflicts with my patch for Bug 802605, so depending on which patch lands first, the other will have to be modified
We should disable all the functionality when no SIM but UX is thinking about this edge case.

Anyway, I don't understand your commentary. What are you suggesting to do? Build a constant such as `NoICCId` = 0 or something like that?
It will never checked because you have same IF condition in line 317. We could remove this IF
It is strange to call a function with this name on success.  I'd suggest you just change this to self.debug() everywhere and get rid of cannotGetBluetoonInfo().
Yup.

input,output[,error in,error out,discard in,discard out]