labofsahil/github-event-dataset-2012 huggingface dataset:
Add Perl to the shootout with Dancer

There are no perl frameworks in the shootout! Perl is still the glue of the web, still very popular [1] and with the Modern Perl movement perl is becoming a force again.

The big frameworks in perl-land are Catalyst, Mojolicious, and Dancer [2]

Dancer is similar to Sinatra, while the other two are closer to Rails / Django in scope.

Given there are multiple frameworks from the other languages, including at least one representative of Perl can't hurt.


[2] https://github.com/languages
[1] http://www.google.com/search?q=perl+web+framework
copyright-french-translation
maybe l2norm here or a specific line length call? The above is only valid for RC coordinates.
fixed issue #208
setupCamera already does a glLoadIdentity(). It's not needed here.
Hey @sorccu any update regarding dev status of Cufon ?
来吐槽一下吧，这工作用gerp就能搞定
//cc @whtsky
I thought it used to depend on that.  I've noticed that calling make test doesn't trigger a rebuild if there are changes.  I wonder why it changed?
I partly agree.
1 these changes should actually be in the geoexplorer specific version of
the files if at all
2 local has a special meaning and is assigned that id by inner working of
either gxp or geoexplorer and is not likely to ne modified by application
code without a good reason.
3 only local layers should default to lazy not remote layers from other ems
etc

Matt Priour
OpenGeo - http://opengeo.org
Expert service straight from the developers
On Jan 7, 2012 3:21 AM, "ahocevar" <
reply@reply.github.com>
wrote:

> Wouldn't it be possible to set the ```forceLazy``` option on the
> application level instead? It's a bit awkward in my opinion to determine
> behavior based on ids.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/opengeo/gxp/commit/7f63d013d96235d490bf5b88026ec2839917182a#commitcomment-845882
>
si tu veux il y a isinstance() qui est un peu mieux que type() car il test aussi l heritage et il peut prendre une liste:
elif isinstance(value, (strt, unicode, int)):
If you have a number you can get the correct behavior with "%016d" instead of s. If it's not a number then things get more complicated.
I've updated the tests:
  * one test required predictable ids, it is still overriding
  * other tests rely on default implementation
  * explicit tests for generation
I would put the ``gamma`` in the first place
I guess this page should be updated accordingly: http://trippi.sourceforge.net/ "Trippi connectors currently exist for Sesame, Kowari, Oracle Spatial, and MPTStore."
Funny to see it still mentions Kowari, and not Mulgara ;-)
It should be 'a' useless function. lol
I don't understand your change here. After this change the comment would be completely wrong. The functionality originally named does exist on certain browsers (namely, older IEs). We should do a version check to be precise.
Just removing the knowledge from this method about the structure of each entry within `attribute_values`. There's still some coupling between that and `set_attribute` but I don't think you can remove that. I figured it was an unnecessary detail at the level of this method.
just to make sure it wasn't has anything to do with any extension i installed, i tried running it in incognito mode, it failed the same way, for pressing spacebar with cause it to stop working very quickly.

I tried the same on my other linux machine, using chrome as well, the problem did not exist.
It only hang when using windows, I am using windows 7 64bit.
Yes, you are right. I will take this only for following parsing errors.
Ah right, so USER_HEADER_SEARCH_PATHS is actually not an array, but a space separated list?
Not working correctly so don't use this!
submit renamed due to reported bug with jquery.form:
http://www.pricesofindia.com/blog/error-form-elements-must-not-have-name-or-id-of-submit

"i found that form having name=\u201dsubmit\u201d will cause an error when you are using jquery.form.js.
Renaming \u201csubmit\u201d with something else will work"
I don't like the profile/details thing. Let's put everything under `profile/_profile`, and we'll deal with the UI bloat later.
I'm not sure how to fix this. I can't see anything obviously wrong.
On Feb 27, 2012 4:54 AM, "Jordan Moldow" <
reply@reply.github.com>
wrote:

> I seem to have introduced a bug whereby friction doesn't always work when
> you're moving left (and, oppositely, friction in the right directions seems
> to be stronger). I'm not sure where this bug is coming from.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/jmoldow/Science/commit/0d4dd48a9beffc77b0104a55f4937bf1af1147ca#commitcomment-1014236
>
This is not a valid calculation, too much healing recovery.
более 80% сообщений уже здесь, в бранче (а это считайте после фильтра ГМов и фильтра технарей) - полная лажа. представляете сколько процентов информации непосредственно от игроков подтверждается? ноль целых хрен десятых...
<IMG SRC="javascript:alert('XSS');">
INSERT INTO `world`.`pool_template` (`entry`, `max_limit`, `description`) VALUES 
('600', '15', 'MASTER Herbs Isle of Quel''Danas zone 4080');

PLease, not every one have a database call world.
The interface to `u8_strwidth` need to change in that case because it *does* expect NUL-terminated strings.
What happens is when loading the remote inspector and switching to the Scripts tab, no script is visible. The only script file is "about:blank" in the script dropdown, and there's no content.

I see the script tag in the dom tree, but it's not exposed properly to the scripts tab.

As to why this is related to webkit...I don't know why it doesn't work, but it has to do with how webkit's inspector works.
no the unique is an SQL level change which means I would only be able to have 2 entries in this table: one with True and one with False
The solution is to change the field name, the problem was that related_name creates another field on this end, which means this class now has 2 "subtypes" fields. To fix rename one of them to something that works
thx for answer rsa and your good compiler knowledge
Compiled fine on freebsd just now... The thing is that it must not use FastMM on unix at all...
this commit freezes both windows build (yes i have vc100 now ) 8p 1400 dollars and one reformat later
heh, tho not sure if this spell should have some cooldown, as currently it can be up for 100%
@carlos93 I will test
https://github.com/Reamer/mangos/commit/c2a9b8cec059fb8e3c7b7248c854ae89cecb5b84
this stuff's gotta go, too.
Функция db_create_table принимает три параметра
1. массив, в который будет загружен результат её выполнения
2. имя будущей таблицы
3. массив полей таблицы
Всё как в Друпале
Always comment your queries for better understanding with the -- format.
DO NOT leave spaces between values or field names, save as much space as possible.

http://www.trinitycore.org/f/topic/2588-sql-guidelines/

y luego me jodes a mí con el codestyle... T_T
Fantastic idea! Could certainly prove useful...
Sets alternative model name to be used in forms
Nope, this was intended.
DRY'd! https://github.com/mozilla/zamboni/commit/eca97180ac0734fb39846c66a52c2fc84987c512#L1R1124
nope, no error
oh that's what that does :)
I added all hooks into standard code. This was an opportunity to uniformize hook names. All hooks to add information on
create/update/view cards are called fomrObjectOptions instead of sometimes formObjectOptions, sometimes showInputFields, sometimes
showOutputFields.

For triggers into pdf build function. I'am not sure if it must be a trigger or a hook. There is already triggers XXX_BUILDDOC ran after doc generation but it was a bad idea. It should be hooks. Can you provide me an example of usage of what you want to do with PREBUILDDOC hook ?
Yes, I'll create the sample commands and the windows template. Then I'll
change the dicovery rules to just add the good template, and not create the
services. Managing template is more easy for user than managing N*M
services (n hosts, M services on each). :)

On Sun, Jan 1, 2012 at 6:54 PM, David GUENAULT <
reply@reply.github.com
> wrote:

> surre ! i will commit check_wmi_plus soon. I think it's time to add
> discovery rules for windows hu ? and snmp ones
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/naparuba/shinken/commit/886e873777b8e5e7dbeeb3ec6deea02035b2442c#commitcomment-830402
>
"profile" should be a const
Well, ast.literal_eval(string) complains of malformed string. Will catch it
with a ValueError.

2012/1/11 Jeff Schroeder <
reply@reply.github.com
>

> Using ```ast.literal_eval()```, you likely don't even need the try/except
> blocks. Either way, please don't use ```except:``` as it might catch
> exceptions we don't want to catch here. Find the specific exception being
> raised and catch it explicitly.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/bastichelaar/salt/commit/5b5d7dcd125296c0175292af5df4b6b3ed456097#commitcomment-857796
>
this one goes away
Just a remark. This change introduces a break in backwards compatability as the setting exchange_type is removed from the amqp input config
That makes sense, thanks, too.
I was unable to test this change, as dexy + latex blew up on me. Bad form, I know.
Yes, I think this can work. Thank you for this contribution, Pavlov! I still would be interested in a performance comparison of your solution and mine. At first glance, it looks like the change to the types can be made independently of whether one uses a mutable fallback (as in my solution) or a forwarder (as in yours). My solution has the cost of a synchronized in the construction of the partial function, but the apply is optimal in the sense that it there is no overhead if the original partial function is invoked (as opposed to the orElse argument). Your solution makes the apply go through one forwarder per orElse. Not sure what difference this will make.
when would there ever be more than one `Tag` with the same `tag_text`?
@joshsh any thoughts on this?  Now that we've pushed Kibbles into Rexster it seems that the right <groupId> should be com.tinkerpop.rexster.kibbles...maybe it should have even been that way before.  what do you think?
Would be nice to add a comment that we are overridding the default to be booklist.phtml which is outside the book directory, which makes it clear that directories go in `setTemplate()`.
@redsquirrel I wrote some tests for and fixed a bug in `Worker#restore_unprocessed_messages` on the airplane leaving #rbonales before I noticed that you dropped it.

I'm not sure if you'll go that direction again. In case it could be helpful:

https://github.com/amiel/resque/compare/d23b9e5b4f3df7db15d066c2b86a9dcd06b0c693...192749e82299005c419da2d2e8ed77426b8aaed8
Cool I will commit.
Trying to debug why all subreddit get sorts seem to be returning results from time = hour.  Is this bit of deleted code necessary to translate the _get_sorted('top', time='day') in objects.py to the proper URI key?
C'est quoi ? Un téléporteur pour retourner au Spawn ?

(Tention au `NullPointerException` si `team` est `null` ^^)
@ractive , could you give me push access so I can commit @tonit changes?
yes of corse please also take a look to the new pyload addon
Sorry, I am very poor English...

Thank you for your testing. This forked module is operating normally in my app. "Snap to their max/min positions" Is it behavior, like a iPhone's AssistiveTouch?
https://gist.github.com/1994388 and filed issue #20
No, The entry into the sql upgrade script (sql/4_1_0-to-4_1_1_upgrade.sql) should be placed at the bottom of the script. This script is used when people will upgrade from 4.1.0 (current version) to the future version 4.1.1 .

You also need to place the tables in the sql/database.sql file, which is used during installation of OpenEMR. Follow the current formatting in that file and place the tables alphabetically there.
надо проверить но мне кажется это лишнее, так как гость по умолчанию не должен управлять данными сущностями
L'appli entière mériterait bien quelques test unitaires -__- ...
Ik heb de code duplicate en de mailman hernoemingen gefixt in respectievelijk 991e1e804261ef1d73f09e7c867a475efbd480d5 en 8ccf64449b12c78eec8f3e4673475520fc2c6d74. (Zou jij naar mijn XXX in de laatste willen kijken?)

Van de jaargroepen weet ik nog niet hoe dat het beste kan. We kunnen gewoon een check doen en die renames denyen of misschien dat het werkt als we losse renames sturen voor alle losse jaargroepen en als laatste de hoofdgroep. Ik kan alleen niet goed overzien of dat zou werken.
I think Peter (perhaps) added all the SagePay files to the repository, my
push promptly removed them all again, how embarrassing!

(All I've been changing lately is LBChooks.module)

On 23 January 2012 18:27, Michael McAndrew <
reply@reply.github.com
> wrote:

> no probs - what did you mean/what to do? I'll fix it and let you know what
> I did.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/michaelmcandrew/lbc/commit/0e3e895a413e76e47d0e623c1127e3e2592ab7f7#commitcomment-893117
>
Thanks for questioning this. While all domainnames are lowercase, some scripts do not have this distinction and their characters are not considered neither upper nor lowercase. Besides, browsers convert things to lowercase when needed anyway. Fixed in next commit.
No encontre otra forma de "arreglarlo"... el spell no tiene flags raras o algo como para diferenciarlo de otros spells. Pero claramente dice RESIST spell harmful ... pero el efecto le da chance de miss... sera blizz haciendo de las suyas :p?
On Sun, Feb 5, 2012 at 2:39 AM, Martin Clausen
<reply@reply.github.com>
wrote:
> Thanks for the feedback, and sure i will give it a go.
>
> Did the test coverage look OK?

Yes, the tests look ok.
Turns out that the readme renderer actually limits images to 852px wide, not 900px. Grr, argh.
Хорошо, убирай.
`control.panel_div = element;`
No. New install should display that info too. What I see in support email, people are reinstalling the app and they will never see the problem.
Whats the sense of this?
I'm using bundle exec rake test (since without it, I get the Rake 0.9.2.2 is newer, etc warning). Yes, this is the first time I've run the test suite.
This seems to be messing up the initial zoom level as it's not filling the entire width of the timeline container.  Must have something to do with the default value of _zoom?
makes llAnimuBot's job much easier and should have been written much earlier.
git commit -a ?
I would be fine with removing support for this and adding it back in if we end up making video submission more asynchronous.
i import all sql updates files from the branch,
@fabian the routing one has an equivalent in the ChainRoutingBundle (with a different name).
and Halion
+1 thx
Closes #4838
Polinômios
it seems something got out of sync with the test repo. I'm debugging it. Also the error handling is weak. We should handle things like 422 and show the actual error message. Right now is trying to handle other things that are not 404 like valid messages and it will return a misleading exception
Would it be better to simply make these two changes:

1) change JPAPaginator constructor to:
public class JPAPaginator<K, T extends GenericModel> extends Paginator<K, T> implements Serializable

2) change ModelPaginator constructor to:
public class ModelPaginator<T extends GenericModel> extends JPAPaginator<Long, T>

It doesn't look like you really change much else, and the two lines of code above should accomodate your needs without breaking existing code.
Even HipHop's static analyzer cannot find stupid mistakes like this one. :-)
Esta linea habria que partirla por el paréntesis
I love this commit so much that I'm tempted to comment with an exclamation mark.
Would make a real patch, but then I need to figure out first how to merge original master changes to my own fork master to keep my released changes there without pushing all the other changes. No time for such right now, so here's a rough how-to-do-it walk-through:

MKNetworkOperation.m, about rows 31-37 contain MKNetworkOperationState.
Move those lines to MKNetworkOperation.h, about row 40.
Should compile now.
Nothing wrong with this at all, but I like using the `simple_form` gem to simply form generation. Something to keep in mind and look into once you're more familiar with core Rails.

https://github.com/plataformatec/simple_form
Wow, that's pretty neat!
Ahh that's a much better approach thanks, I for some reason just assumed that queues would always be available. Thanks.
I have also fixed them on Crowdin.
Great you continued it. But just a sugestion, try not to make "god-commits" make them gradually with non-breaking code like funtion stubs and comments for them. then start adding each functionality separately. After you get a working set of commits push them, but still each commit will be atomic and gradual. That's makes it easier to find flaws and buggy things.

Now I'll review it :P
It was David Priest who provided the fix.
This source not required in patch as there is no change except an empty line.
What was fixed? I didn't know it was broken. Works fine for me.
>css transition

Только начиная с версии 10.5. Оно тебе надо?
Moved changes to new fork
# Homework 3

| **Part**  | **Points** |
|-----------|------------|
| base      |  1 |
| advanced  |  0 |
| if and while | 0.25 |
| random | 0.25 |
| tracking and reusing user input | 0.25 | 
| zazz | 0.25 |
| **Total** |  2 |

I had a general moment of panic at the shutdown error message for less than half a second, well done.  Output looks good, code itself looks good.  Comments are good if not excessive in places.  Overall, really well done.

One odd part in the code:

```python
# if you lower, when would it ever be "Y"
if yule.lower() in ['y', 'Y']
```

Also, comments usually go above the line they describe, took me a bit to figure out what was going out.

Finally, to answer your question on the "real time" you can use:

```python
import time

time.sleep(2)    # sleep for 2 seconds
```

For making me think my machine had been rooted, I give you the [gangsta sysadmim](http://youtu.be/Fow7iUaKrq4) award

![award image](http://farm4.static.flickr.com/3127/2369869904_4796a850f9.jpg)


> PS Part one and many pieces of the homework will be restrictive at times to force people to use specific code, in this case raw_input with variables and string concatenation.
感谢Heping Hu <yunheyun@gmail.com> 提交的bug
Thanksです。もんだいなさそうなのでマージします。
Is the `composer.json` ready to be used?
"it can be controlled whether" => "you can control whether"
HAHA yea that is exactly what is going to happen with me.. im on android until my n900 is fixed/replaced
do you think it's user friendly? Imagine it will be on google home page...
@sjfahr @jyung @cjmaclellan 

multi-slice multi-echo imaging has been updated and ready for testing.
Python dictionaries and sets used to create a hash table of dicom header info
Also this expires 20h keys for security
Thanks man, I appreciate it :)
this is a simple implementation of the p2p dht, the algorithm is somehow similar to the the one described on http://en.wikipedia.org/wiki/Distributed_hash_table . you can read more about this project requirements here http://bit.ly/xyAkEK .
Try to create 2 or 3 autocompltes on one page.
Thanks, I forgot I had that. Will re-read...
is this change needed?
Akh, you're right. It's not as pretty! :) How about something like this:

```
StringBuilder sb = new StringBuilder(url);
sb.append(url.indexOf("?") >= 0 ? "&" : "?");
sb.append(name).append("=");
return sb.append(OAuthEncoder.encode(value)).toString();
```
Note that in our current code generation schemes, this is not just a user-defined label.  The name of the kernel /must/ match the name of the function in the code:  e.g.  Kernel("void foo(...) {...}", name="bar") will not work.
this should still respect the `maxInstancesPerNode` setting, I think
Best solution is probably to pull it out and use the same version from both the NettyConnection and the NettyRemoteTransport
A good effort, but a mutable case class would be very confusing.
Correct

On Sep 19, 2012, at 8:38 AM, Matthew Rocklin wrote:

> In project.clj:
> 
> > @@ -4,7 +4,6 @@
> >    [[org.clojure/clojure "1.4.0"]
> >     [compojure "0.6.4"]
> >     [expectations "1.3.3"]
> > -   [hiccup "0.3.6"]
> We don't need this? Is it implicit in noir?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
QPID is capitalized 6 lines up but lower case here. Pick one.
ok, this is just beautiful stuff!
Why a condition on emptyCollection?
If you're OK with the interface change, I'll whip up a changeset.
The header license has changed since 1.0.6.Final. Check the other files for an example
should be "which you need" ?
Hi wojtek-szarmacher, please always use curly braces after if statement. Could you create new pull requests with proper changes?
Here you also need to clean up your parameters
I understand why you needed to do this, but we decided to refactor the code such that all the defines of Erfurt_Store are now class constants. Please merge with the current AKSW develop and remove this line.
I'm not sure what restriction are we talking about now. It seems to me it doesn't add any:
```sh
ls -A
"aa b.c.d	.a		a 		a.a.a		cc
'a		.a a		a a		bb		nn
```
```erlang
1> filelib:wildcard("*").
["\"aa b.c.d","'a",".a",".a a","a ","a a","a.a.a","bb","cc","nn"]
```
Also add it to the comment above so it gets added to the documentation.
technically this method is not side-effecting (although it also is not referentially transparent); the parentheses are a matter of taste for this one
Roland gave me a good idea how to create only one detector and still not  
using locks. I will implement that.
Jeez, who has time to type all of these spaces?
These notifications are not used in this pull request.
`NSUInteger` values should be explicitly cast when used in format strings, because you cannot depend on them being `long`s (if you do, then you defeat the purpose of the `NSUInteger` type).
There's no real need to track the unref state, `uv_ref()` and `uv_unref()` are idempotent.
Dans tu pourrais probablement retirer les traductions pour Vote et Choice
I would use "superfluous" or "inappropriate". "Extra" is also possible, but for some reason I don't like it in this context.
Fix this..
here is how nibabel does it for siemens. 

https://github.com/nipy/nibabel/blob/master/nibabel/nicom/dicomreaders.py

and @rfdougherty just posted the ge version here:

https://github.com/cni/nims/blob/master/nimsutil/dicomutil.py

i think we could try to include bob's code in nibabel and then use that bit to do it for both siemens and ge. hopefully nolan can provide some info about philips.
missing description and doctest example
No, I'm pretty sure that would break separate compilation -- it's declared extern in __multiarray_api.h, and references in other .c files need to be resolved to this definition by the linker.
Can use `options.fetch(:year, year), ...`
There's a race condition here. Suppose `method` is `:foo`:

* Thread A checks: `::ActiveRecord::Delegation.delegated_to_scoped_klass.include?(:foo)`, it returns false
* Thread B defines the `:foo` method
* Thread A gets to to the `super` call, which now returns true. However, this is incorrect since our `@klass` does not respond to `:foo` in this case.

To fix this, we need a critical region around the check of `delegated_to_scoped_klass` and the call to `super`, and we also need a critical region when we define the methods.
Yes, seems fine
If we need to exclude something we need to fix `filter_by`, this is not the proper place
You could use `Hash === options` instead.
You could probably use `options.fetch(:coder, Hash)` here.
Yes. `typedParentType` and `internal.Trees.New` still silently convert Nil to ListOfNil, but it's their own choice, no longer imposed by the typer.
qual.tpe.typeSymbol.orElse(fun.symbol.owner) looks a bit sketchy, but I don't have anything concrete to propose.
Super long paths distract from what is going on. There is no benefit to having "scala.collection" sprinkled around. Also, consistency.
Same comment as for ``__call__``. Do you know the difference between ``bispev`` and  ``bispeu`` by the way? Docstring and code are pretty much the same.
this seems unnecessary
nit: space after the colon, I believe.
Not sure I understand the reference. Is there some history behind switch and Python?

I'm pretty sure that I've already alienated all the hardcore python syntax gurus out there :)
I've moved each of the three branches of the `if`-statement in this method into separate methods.  Looks better now (I think).
I think that trades clarity for conciseness to an extent I am not comfortable with.
Don't factor in the default options when building the post body params.
Please remove personal comments/notes.
or in other words, do we really want to give the user full control what to set here? There aren't that many options/modes available
Ok, but what are those values used for? If they are used to display the confidence interval etc. in the network meta-analysis view, those displays should be based on the quantile information directly, not on a distributional approximation of.

On the other hand, if this information is used in the benefit-risk analysis, it should be based on the *multi-variate* distribution on which the BR simulations are also based.
On 5/1/2012 2:07 PM, Jeff Balogh wrote:
>> @@ -90,7 +90,7 @@ def message_validator(request):
>>           return request.errors.add('body', 'body', msg)
>>   
>>       # These are the keys we'll accept for messages.
>> -    VALID_KEYS = 'title', 'body', 'actionUrl', 'replaceId'
>> +    VALID_KEYS = 'title', 'body', 'actionUrl', 'replaceId', 'cryptoBlock', 'site'
>> Granted, as far as I'm concerned, "site" is just an index value. I'm using the hostname of the remote service because I figured it'd be easier for folks to work with, but it could be any string value.
> If this is an index value for your use, why bother developers with it? You can use the push URL as an index and keep the protocol simpler.
>
Hmm.. I'll check to see if there's a problem with that, but I can't see 
a reason off hand not to.
Grep for UV_LOOP_PRIVATE_PLATFORM_FIELDS and use that approach instead. Duplicating the fields is bound to break later on.
:) nice. waiting for your push to your branch.
I've just seen that this is actually used in the schema-bindings module to do the actual code-gen - potentially this documentation could be updated to reflect this.
Why do we call this a limitation?  I'd say it's a feature.
they can use the index, it's just ineffective.
period at end of sentence and move """ to the next line
Unfortunately the hbase jar has incompatible method that was removed in 12.0.1.  So we can't upgrade to 13 until we can get that jar updated.
Shouldn't each var on its own be treated as an empty string, rather than nothing included at all if you don't set one of the vars?
I think title is most often used, imo.
If something is broken, file a bug for it. Don't just add some comment for it. Either this is how it should behave or this is a bug. In the former case this comment should not be here and in the latter case it should be fixed.

That said, strings like {FOO}/other/stuff were not supported in the past I believe, so this would in fact be a new feature.
Sorry for these many faults, I'm  not already conftable with your Code Layout.
Yes, it's a new-line (\n) delimited JSON.
Hrm, these look like PEP8 changes. Should `runner` be added to the list of things for the pep8 linter to check?
leave two spaces
I haven't seen the lua scripts, so I didn't consider that.  Still, I like `all`, `[]`, `[]=` and `clear`.
so do some test after `var unit = ...` to make sure we return the empty string if given a poor argument?
```
String(undefined) //=> "undefined"
String(null) //=> "null"
String({}) //=> "[object Object]"
String([]) //=> ''
String(['singleDepth']) //=> 'singleDepth'
String(['bigger', 'array', 4]) //=> 'bigger,array,4'
```
Shouldn't we use a uniform naming style for class members?
NIT: spelling error. "deleteComlain" should be "deleteComplaint"
@johnstoecker  UUID is openMRS internals, not Raxa / JSS patient # -right?
remove ```varName = queryParameter``` logic. Not needed with new uuidResource dictionary.
The ~ make it say ```Cube``` rather than ```iris.cube.Cube```.
Will keep if that's ok.
Update: g++ is fine with 9223372036854775807, or with L or LL.
I added such a test in a new PR.
[JSLint](http://www.jslint.com/)
You don't have to specify the array length explicitly. signos[] = {SIGHUP, SIGUSR1, SIGUSR2, SIGTERM, SIGINT, SIGCHLD} should be fine. While compiler will complain if the explicit length is less than initialize list size, it will keep quite if it is greater, leaving entries at the end uninitialized, causing runtime issues. By the way, since this array is immutable, you can make it const and static.
Why is this code structure like this? Why not just use an if statement?

You can get a 1/3 chance using "if (random.nextInt(3) == 0) setDownloadVersion(version_list[random.nextInt(version_list.length)])"
Only maybe on more than one line and more readable.
Mark this function as private: ``_set_in_progress(deployment)``, use the ``deployment`` object from reboot to avoid fetching it twice from the database, assert the deployment is not marked as done before rebooting.
Code style: `if (typeof v != 'object') return;`
the exception name is wrong too
I'm also fine with percentage, in the end it's just a matter of proper documentation.
See #892.
I hear you. What if we swapped the order of `numeric` and `extra` in `jQuery.css`? That way "try" parsing looks like `jQuery.css( elem, prop, <not-undefined> )`, forced parsing looks like `jQuery.css( elem, prop, undefined, true )`, and the only thing we lose is argument symmetry in the width/height dimensions methods. Then one of use can sweep away all the still-present `parseFloat`s throughout the code and _really_ reclaim some bytes.
Same here. We should have a comment explaining why negator is not defined but otherwise remove this line with dead code.
Made it so :)
Ummm, is this a bunch of test code you forgot to remove?
The suggested config is a better reason for seeing what the report looks like.  I guess that would be okay, then, but it would be nice to make it more obvious how to go back to the config.  (I know I can click on the short name, but having a extra link there seems like it might be a useful thing to have, too.)
Should remove the commented out url here.
No, it does not compile with that suggestion.
`unless (etc/'freeDiameter.conf').exist?`
Yes, I can read it, and I have thought about it..
But the only problem is the version in Gem can only accept like 1.0.0.RC1 in this format. If I use 1.0.0-RC1 or 1.0.0_RC1, it will prompt error.
So to keep the version number separated by dots, may be a good idea. In that case, I can successfully modified to read from package.json file.
Fichero y archivo es lo mismo
getCalenderObjects did not use  getCalenderObject, so check/clean in getCalenderObject will not be called. What do you mean?
Shouldn't this be print_unescaped($normalString); ?
don't hardcode environment in here. You shouldn't use the environment as the has key so you can avoid this
wiki... wiki everywhere! Style note: Part of the purpose of namespaces on imports in python is so that you **don't** have to prepend every variable name with "wiki" - you can do things like call this variable "modactions" and then do `from r2.controllers import wiki` so you can reference things as `wiki.modactions` :wink:
Would you mind adding title attribute (description field in model) so that we won't lose existing information? And maybe not  adding target="" if self.window is False would be better?
Dis<b>t</b>able?
`||` instead of `?`
Can we just use `for in` instead of forEach?
Nope gotta close over `key` for the "data-#{key}" below.
wouldn't we want to destroy the video player if the node didn't have a `videos` property, too?
Is this really meant to check that the region can have a scroller because it only has one item? We shouldn't be accessing variables that are intended to be private. If we really need that value, then we should have a getter.
Looks like there's a newer createMockEditor() API that wraps all this up neatly (along with a corresponding destroyMockEditor()).
Nit: fix indentation so the `:`s line up
filter is now search.

match is now required. My plan was to ultimately make a way for other plugins to be added that were not triggered on a character. For example have a quick open that searches all the menus and commands. It would be trigged by a different call to quick open rather than a triggering character.

I documented that resultsFormatter has a default implementation when one is not provided

itemFocus/Select -docs added about arg

-----Original Message-----
From: Peter Flynn [mailto:reply@reply.github.com] 
Sent: Wednesday, May 02, 2012 1:20 AM
To: Ty Voliter
Subject: Re: [brackets] Quick Open round two, no caching (#734)

> +     */
> +    var origSelection;
> +
> +    var dialogOpen = false;
> +
> +    /**
> +     * Defines API for new QuickOpen plug-ins
> +     * @param {string} plug-in name
> +     * @param {Array.<string>} file types array. Example: ["js", "css", "txt"]
> +     * @param {Function} called when quick open is complete. Plug-in should clear its internal state
> +     * @param {Function} filter takes a query string and returns an array of strings that match the query
> +     * @param {?Functon} match takes a query string and returns true if this plug-in wants to provide results for this query
> +     * @param {Functon} itemFocus performs an action when a result has focus
> +     * @param {Functon} itemSelect performs an action when a result is chosen
> +     * @param {?Functon} resultFormatter takes a query string and an item string and returns a <LI> item to insert into the displayed search results
> +     */

* filter -- seems like an odd name, since it's not given a list to filter down... maybe something like search() or query()?
* match -- is it really optional?  What happens if you don't pass a match() function?
* resultFormatter -- same question
* itemFocus, itemSelect -- we should document that the focused/selected item is passed as an arg

--- 
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/734/files#r760445
I don't think you need to update this comment since it is still correct. You don't close it right here directly, but you're still closing it in your _onLostContent apply call. Besides, your updated comment becomes incomplete sentence. (ie. if condition A or condition B, instead of  if condition A or condition B, take this action)
To be consistent with other "class"-like modules, we should probably use `exports` instead here.  E.g.:
`exports.ColorEditor = ColorEditor;`

And then client modules like InlineColorEditor.js would change to use `require("ColorEditor").ColorEditor`.
yes, I thought that too.
Then, I found the actual call is done from the listener to `codetools.codeclick` --> [here](https://github.com/ajaxorg/cloud9/blob/bc3a5520f2084835835d47e7791ebbc084c631a0/plugins-client/ext.colorpicker/colorpicker.js#L262)
I added the controllerPath function, but I noticed an issue with it...stay tuned...
can we call just @app.call(env) here?
Note the if statement at the top. It only *adds* 1e-4 to the diagonal elements. assert ensures it is indeed just diagonal.
what is this, java?
I'd prefer `data$y <- yintercept %||% data$yintercept`
1. 應該是 `%button` 吧? (拼錯?!  -_-")

2. 一般 value 裡面的字串可以用 symbol 就用 symbol `{ :type => :submit }`
In signals, because I test if `HTTP_IF_NONE_MATCH` is integer.
This is wrong. All classes in Python should extend `object`. Otherwise they are old-style classes.
If you're removing the *Without* code path, why keep that naming convention?  That is, why not just call it updateTrackEvent?
Externalize .js for linting + caching.
This is a piece of the old zoom bar that has not fully been killed. I am working on it bit at a time.

Basically, the old scrollbar was reverse of this new one. 90% needs to be 10%. 100 minus the _width percent would reverse it.

I'll try to kill it outright.
Why do you need to pass in a `width` at all here? It looks like it just gets set to `DEFAULT_WIDTH` in `media.js`. It's never reset to anything. What exactly it is for? Perhaps it needs a better name.
'status' a bad name for a custom attribute, use 'data-status'

i think using a class name will be a better choice here
Looks like a test variable that slipped.
FYI: OpenBSD and FreeBSD pf have diverged *significantly*. I'm not sure if it matters, but you only linked to the OBSD pf page.
Creating a Client yourself is not necessary, every TestCase has one: https://docs.djangoproject.com/en/dev/topics/testing/#default-test-client
The `show_sexpr` functionality has proven valuable enough to me that I think it should be available to any metaprogrammer. Surely, unlike the other functions, this one is mostly for interactive use by the metaprogrammer, not for use in actual released code. So it could conceivably be put in a separate `Meta` package, if we want to have both `Base.Meta` and `Meta.jl`. (But then there should be pretty clear guidelines of what goes where.)

About encouraging Lisp style: There were some discussions on julia-dev about how hard it can be to come up with the right incantation to produce some AST:s. Just like `show_sexpr` allows you to see the structure of any AST (unlike pretty printing, which hides a lot of structure), `@sexpr` seems like a generally applicable solution to parametrize any AST:
* Use `show_expr` to dump the structure of an example AST
* Put in interpolations where you like

The `show_expr` expansion can of course be very wordy. But you don't have to use a full expansion like

    @sexpr (:function, (:call, :f, (:(::), :x, :Int), (:(::), :y, :Int)), (:call, :*, :x, :y))

`@sexpr` accepts subtrees in any argument position, so you can write the ones that you know how to using plain syntax:

    @sexpr (:function, (:call, :f, x::Int, y::Int), x*y)

This hopefully means that `@sexpr` could be the standard way out when you don't know how to construct some particularly tricky part of an AST, but plain julia syntax will still be used for the rest since it's more convenient in other cases. This also has to be documented somewhere, of course.

I'm not sure if `show_sexpr/@sexpr` will make it into this module (though I'd like myself), but I included them to get the discussion started. What do others think?
Right, I saw you were doing this in your code, but I did not see all the consequences. Changing.
Just curious: what migration requires Candlepin to be up, I thought only seed script does require it.
Please use format:
"%(a)s %(b)s" % {"a": 1, "b": 2}
translators will thank you
will this still be treated as an async notice?
I'm fairly certain I changed this in some version. It should say "not is_blah" now.
This is currently wrong. Either let Pusher encode the payload, and then the formatter should be a NormalizerFormatter or set the last argument of ``$pusher->trigger()`` to ``true`` to indicate it is already encoded
As with the methods above, these should have Parameters and Returns sections.  Feel free to ask if you're not clear on how the docstrings should be formatted (or take a look at e.g. `astropy.config.configuration` - everything there I think is following the docstring format).
Maybe the constants here (``decimal.Decimal(jd_zeropoint)`` and ``decimal.Decimal(86400))``) should be defined higher up in the file
Sure, that's fine. Though autopep8 won't pick up things like the line continuation issue.
I'll fix the error message.  I was trying to say >4.  'needs more than 5' would probably do it.
Yes, that's much more clear
That was commented out as 8 lines further up in the debug.log file that information is displayed there.
It is possible to rename this, sure, so I'll count your +1 and will wait for other comments :).
I'd simply say default: -proxy 

Sure, socks4 cannot do ipv6 and tor, but that should mean that those are disabled when a v4 proxy is used. Not that those go around the proxy?
This might be a good candidate for `Celluloid` itself.
Although a real registry would be better.
Your name is now forever and always out there. (Unless you undo the commit that put it there, and I think any commits that touch the same file, and jump through a bunch of other hoops.)

While, legally speaking, works don't need any notice at all of copyright to be covered by copyright in the US (and presumably therefore don't need real names to license said copyright), I'm not sure about other countries. Even disregarding that, it makes me uncomfortable to include licensed code without the name of the person from whom it is licensed.
If was briefly considering this, too. I.e. using simple strings to indicate the "property type" instead of a full classname, but I thought it would give integrators more flexibility if they could specify their own classes here. What would you suggest for using your own classes for properties and collections? One could of course make a subclass of RdfDriverArray, but since the thing you want to override is in the middle of the longest function, you would have some copy-n-paste between the two. An alternative might be to pass the classnames you want to use as optional arguments to the constructor, but that's a bit ugly I guess
render json: pusher_response

# ...

private

def pusher_auth_response
  Pusher[params[:channel_name]].authenticate( params[:socket_id],
      { user_id: @current_user.id,  user_info: { username: @current_user.username } } )
end
OK - I'm fairly neutral on which is better, but I think I marginally prefer having the `locate` commands together, so I'd remove `ipython profile locate`. Adding a note in the help makes sense too.
Yes, currently it inserts the text or filenames at the cursor location which is updated to the current mouse position. 

I agree that it might be better to accept files anywhere, but as you describe it isn't necessarily clear what the expected behavior is.

The handling is a bit odd because our terminal seems to try to be both a text editor and a terminal... The implemented behavior here uses what I would call editor semantics, but it sounds like you would prefer terminal semantics.
This should probably be in utils - perhaps in openpy.
rename OAuthAsyncClient, OAuthClient to OAuthAsyncApi, OAuthApi

Also, the generic types are flipped.. it is sync, async vs the other way around
Instead of exiting with a goto, I would be tempted to add a `DONE` state and have terminal cases such as this just go into that state and then let the loop exit naturally.
Agreed, but maybe the name of the function should be changed to something else. :)
i'll look into it.
Same as hacking_howto. Needs the `$ `
As you have mentioned on the bug this test should better be located under utils/ because we want to directly test the `utils.saveScreenshot()` method.
`if not params.uuid: raise Hell()`
please leave the imports alone
Are all these `use` statement useful?
This is really not what the problem was.

PHP is complaining that `$data->theme_layouts` doesn't have a value, so instead of setting `$theme_layout` (which is actually overridden as a string in the foreach loop) you need to set `$data->theme_layouts = array();` in that same line.

I know I asked you to send a pull request, but that was mainly because when somebody suggests on twitter they might know a fix then link me around to errors and whatnot its very hard for me to care while im on a bus trying to inbox zero on my phone. Most of the time if somebody has found a bug they're only 2 minutes from finding a fix. You tried, and you missed but I am grateful all the same.

Also, this will only ever show up if there are no theme layouts. Why are there no theme layouts? :)
Can you add a space in here:

     if (
indeed, much simpler :)
Symfony uses single quotes everywhere (except when a special char is needed) so it should be changed here. And you don't need the extra braces around the assignment.
I'm waiting until a decision is taken about this class.
Sorry about that. It must be something with how we have samba set up. I'll fix it.
Or even `Suggestion.new`, which would be even better
Would require(n >= 0) do instead of a manual if + throw?
I don't think this is what You wanted to do: argv[0] point to executable name.
Just do a ```result = KNOWN_EPSG.get(key, None)```.
think this comment is out of date
I dont think so but @kselden don't like the idea of defaultApplication :(
I think we need some input from @wagenet, @tomdale and @wycats here to get a consensus
Maybe call it Socks5?  Again, the term query is very generic in XMPP.
Should this be an exception?  Where will this print statement show up?
Thanks, trying that now...
I'm hoping to be able to load this from the classpath, rather than the file-system. However, I don't think Byteman supports this. Waiting for a definitive answer from Andrew Dinn: 

https://community.jboss.org/thread/213698
I didn't look that far away, just fixed what was wrong on my setup of TestSwarm. Can't it be more tested before you do something like shift to ua-parser? I mean it's ok to fix bugs in ua-parser but I think the shift should take place after your fix is reviewed ;]
Use sentence case ("Reset failed jobs") like the others.
I just realized that there's a posixpath.py with a function that does a similar thing.  Why can't we use that?
The entire `capi` module will mostly be a port of the libspotify header file, e.g. enums, structs and functions, so I wouldn't move this to a `constants.py` file.

Though, I'm considering not having a flat namespace as in libspotify, but instead make `sp_error_message()` available as `spotify.capi.error.message()`, along the lines of what PyGI does with GLib/GObject/Gstreamer.
Nope: https://mxr.mozilla.org/mozilla-central/source/dom/base/BrowserElementParent.js#116

But setActive makes more sense. @daleharley?
I had some issues with the use of 'this' all along the file, it was not properly used in many places, so I stopped using it to avoid fails.

But in this case you're right, so changing it, thanks for the tip
var options = command.options;

and use that.
I think you can declare your argument here like `re_contactCallBack(item)` instead of using the `arguments` array
If you are going to auto-repeat this song, call setRepeat() here so the UI indicates that it is in repeat mode.  (If you can do that without altering the user's saved settings.)
On 03/08/2012 20:57, Andreas Gal wrote:
>> @@ -535,12 +529,12 @@ const GridManager = (function() {
>>         }
>>   
>>         if (animation) {
>> -        goToPage(index, function() {
>> -          setTimeout(function() {
>> -            pageHelper.getCurrent().
>> -              applyInstallingEffect(Applications.getOrigin(app));
>> -          }, 200);
>> -        });
>> +        goToPage(index,{callback: function() {
> same here, please avoid allocations, they cause GCs, which cause UI stalls

I'm removing some in the main panning code in 
https://github.com/mozilla-b2g/gaia/pull/3048 but it is not rebased on 
the latest changes.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mozilla-b2g/gaia/pull/3122/files#r1304526
Ah, tests. OK.
yep, lint has to be checked
@etiennesegonzac these rules are here now… they’ve been changed in order to support more than one sub-level of panels.
Did I delete the FontLIght rule? :-O

The @font-face definition for the FontLight is still in the dialer.css code. We introduced a new font-light class to set the font family of the elements holding the phone number / calling or caller name.
Is this really needed versus a 'normal' function call "this._checkDB(function() {" ?
I didn't because it's the goofy conditional comments for the <html> tag, which is usually not indented as it's not inside another tag. I'll change it if you insist.
Kill needless space.