labofsahil/github-event-dataset-2012 huggingface dataset:
This looks nicer:
```js
return item == null ? 0 : ~item.toLowerCase().indexOf(this.query.toLowerCase())
```
yea, it's two times faster when `*` wasn't found in string

http://jsperf.com/indexof-vs-lastindexof/3
Zend Service Identifier?
add a space after the if and before the open brace
Surely you would just leave the property alone if the event is cancelled?
I don't remember having to do this when I installed homebrew - does it do it for you?
You mean false, right? But yes, the second comparison is useless.
/app_dev.php in der Javascrip route... Nicht gut... Hierfür besser das Routing verwenden.
Write short about the benefit about using classBindingProvider, and a link
Well, since it's an assignment the `;` is as necessary as in the expression `var x = 3;` It's not absolutely needed depending on the context after but it's correct to put it there.
it returns '' (similar to classic JS DOM)
@samiluukkonen "Käyttäjä ei saa laittaa valmiiksi" -ominaisuus kuuluisi olla osa "taso"-parametriä käyttäjän takana. Taso-jutut rivillä 834. 

Jatkossa aina kädet pystyyn ja paperit seis jos tulee keissi jossa halutaan samaan parametriin leipoa useampi täysin eri ominaisuus.
@samiluukkonen Mitä tarkoittaa? Missä kommentti.
This is wrong. Doctrine is using the MIT license now
Aha, sorry, then I was confused by the naming :)
I'd simply drop them both then.
Not sure why you'd bother with removing jQuery/Zepto as a requirement.
remove this dirty comment?
add docstring: """Install by copying bniaries to install dir"""
While I like the idea of provided a default filepath, we cannot rely on `__main__.__file__`. When you run an addon on the command line using `xbmcswift2 run`, the `__main__.__file__` will be the xbmcswift2 binary, not the addon's main python file.
We need to do this across the board.
A note on this: There seems to be a mismatch in kitchensink's versions in a comment:

        use version 3.0.0.Beta1-redhat-1 which is a release certified 
      to work with JBoss EAP 6. It requires you have access to the JBoss EAP 6 maven repository. -->
      <!-- 
      <jboss.bom.version>1.0.0.M5-redhat-1</jboss.bom.version>>
Why is it `required=False` ?
More [magic](http://esreality.com/files/inlineimages/2012/87469-magic.gif)?
s/new_kb_article_page/kb_article_page
Do we really want to enforce lowercase filenames? Capitalization in the window title might be important.
It seems to me that testing `d.setTimezone` on an instance is ok but testing `Date.setTimezone` fails:

    > time = require("time")
    (...)
    > time.Date.setTimezone
    undefined
    > time.Date.prototype.setTimezone
    [Function: setTimezone]
    > d = new time.Date
    Wed, 25 Apr 2012 14:44:48 GMT
    > d.setTimezone
    [Function: setTimezone]
it really should just be `name = name or create_unique_identifier()` I'll patch it in a bit.
I think I remember reading that we don't like using .html_safe this way, but this is the way it's done a few lines up for collections.
wow that is an awesome ruby nuance!  cool :+1:
path is already a String
Maybe this should be left public in case anyone wants to use the `Trade` directly instead of using the `TradeManager`.
I'm fairly certain you can just use a PUT using `nexus[url].put(File.new(n3_temp.path)).

Older versions of this gem did a similar PUT for uploading files before it was changed to a multi-part POST.
Session creation is actually a 2-step case here since we visit `/auth/shopify?...` for auth. After some good amount of confusing trials, `show` seemed like the best replacement for `finalize`.
Changes the variable name in this place too
the usual way to get the selected value is to use
```javascript
var val = select.options[select.selectedIndex];
```
I'm unsure whether `select.value` is working in legacy browsers...
а кто это будет читать?
can we make a page region object for these area...

take a look over https://github.com/mozilla/Addon-Tests/tree/master/pages/desktop/regions files
This was missing from trunk/mainline - meant the description was not displayed in the category editor
These keys should be destructured in the argument list.
`(string/replace (name %) \- \_)` could probably be extracted as a util `underscorize` function or something.
Can you document what the `GetObject` function does or at least put a reference to documentation? From what I see it's an external library but it would make the code slightly more readable.
We can probably rename groupUuids.forEach to principalUuids, as it includes the actual user now as well.
@nicolaasmatthijs With checking for content manager/admin/... do you mean check if the user is allowed to delete the comment or does this need to be stored somewhere? The former is incorporated.
oh, ok. then lets be consistent with symfony2 core.  @rande, would you mind updating the doc annotation if you agree with my proposition? then i think we are good to merge, unless @Crell has some concerns?
There is no official `gremlin` endpoint baked in, so simplifying this to only check the plugin endpoint.
This one looks really weird, should be converted to an if, with several style changes:

    if (( ! empty($tok) && $tok !== '0') && $tok !== '..')
    {
        $uris[] = $tok;
    }
Хорошо, посмотрю, надеюсь хватит на это времени. Спасибо
да, ошибся
В задании сказано, что необходимо мутировать обе ячейки и вывести hammingDistance не только до, но и после мутации
Потребуется минимальное изменение в коде.
It should be where you are adding a new student.  Should be a button called "Add new section".  The button should be located in views/students/_form.erb.html
`db/migrate/20120503232451_add_uptime_to_wheelers.rb` has:

```ruby
add_index :wheelers, :uptime
```

but the schema never got checked in
Why not `if (!$path || $path == array('home')) {`? Otherwise this patch looks fine. Just do me a favor and squash it. Thanks!
please remove the extra spaces around ``!``
Do we want to declare these constants up top maybe, before the function gets returned? Like `DEFAULT_DISPLAY_TIME`?
Can you make sure that all available packages have been added from this plugin?
Hubot will expect there to be one line per configuration. `Notes` is a good section to put longer form notes though.
What file comment at the top?  The first comment in the file is about the Add struct{}
I would use the GitCheckoutOpts class and remove this one.
Ok, bf91e21061a3cc1a28ac3dc6afe048794a06c300 is the commit that produces this change. But what should I check there?

The problem is that if there is no language variety name set, then the template will just output “— **variety:**” and nothing more.
In [17]: type(True)
Out[17]: bool

so I would say "bool"
can't you get this automatically from the connectivity matrix by splitting X using the found connected components?
I just think it's asking a lot to the user to specify this.
I would just expose a title param so users can plot whatever they want and we get rid of the is_power and many comment above with plot_tfr_imshow that is not that relevant after all.
only thing changed is an updated description
多余的空格
I wouldn't capitalize "Locale" here.
`it 'should call keys and delegate to inline store'`?
А вот тут имеет смысл один раз `const auto& resMap = res.toMap ();`, и дальше уже с `resMap` работать.
А тут зря ньюлайн (и тремя строками выше тоже).
мне так больше нравится
Это, наверное, тоже стоит обернуть в `#ifdef  HAVE_MAGIC`/`#endif`.
Como se habrá escapado esto? debe ser cosa de Trello...
this line can simply be `(= category :errata)`
ohh fuck.
Sure I just updated the code...
Nope don't need it removing it.
TxCompletionNotificationCommand is not visitable, will never get there..
IllegalArgumentException unmarshallable(String name) ?
True, could create a local cache cache ;)
It should, no additional test failures!
I know you didn't write this, but it's not really correct to check `error` against `nil`. `error` should _only_ be used if the method has returned `NO`.
because it's used in UDAFCumulateHistogram as well.
Oups... I was meaning to look into that before doing the PR, but I forgot...
Updated to use a for loop.
I think the capability of the function is the same as before. It just works around another bug in PIL, which does not seem to close all the files it opens.
See previous comment on multi-line docstrings.
I'm about to add an automatic calculation of sourceSize width and height, so that it always gets withing a certain limit, and we always make use of all the memory we have...and I need it for that too :/
That returns a primitive type - how do we work with those? `(prn (.split "asdf,asdf" ","))` gives `#<String[] [Ljava.lang.String;@70751932>`, not the actual string.


On Thu, Jul 19, 2012, at 09:38 PM, Sam Ritchie wrote:
> >  
> >  (defn parse-line
> >    "Accepts a vectorized string, as outputted by `text-map`, and
> >    returns a single hash-map with the GADM ID (integer) as the key and
> >    the ISO3 code (string) as the value."
> >    [x]
> > -  (let [[iso gadm-str] x]
> > +  (let [[iso gadm-str] (string/split x #",")]
> 
> Better to do this since we don't need regex:
> 
> ```clojure
> (.split x ",")
> ```
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/reddmetrics/forma-clj/pull/150/files#r1203066
Indeed it still breaks with 1.7.1.
Actually valuelen should be greater than conditionallen in order to have a
true rule checking - minlen.

valuelen - length of the value to be checked
conditionlen - length condition from minlen rule


On Wed, Sep 5, 2012 at 8:45 PM, Richard Rodger <notifications@github.com>wrote:

> In lib/parambulator.js:
>
> > @@ -182,6 +202,8 @@ var rulemap = {
> >      return cb();
> >    },
> >
> > +  minlen$: lenrule( function(valuelen, conditionlen){return valuelen > conditionlen} ),
> > +  maxlen$: lenrule( function(valuelen, conditionlen){return valuelen < conditionlen} ),
> >
> >    re$: function(ctxt,cb) {
>
> as per above, these should return true if ok, so use:
> minlen$: lenrule( function(vlen, len){return len <= vlen} ),
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rjrodger/parambulator/pull/9/files#r1535975>.
>
>



-- 

*malex*
Calls to Arrays.toString(...) here are not lazy here and could incur additional overhead. Would recommend having this wrapped in a if (LOGGER.isDebugEnabled()) block
We were serving up a template string without a completing ;
same here: SGVector
doesn't work for empty password - mysql prompts for it if there is -p''
for this one, I would keep it empty, so that LessphpFilter does not set it explicitly, meanign the lessphp default is used, even if it changes in the future
This part is a strict refactor, as the original code it's replacing used an inline `&nbsp;` to separate the `>` of the Publish button from the word "Publish".
You don't need to call `$.fn.css` twice, it can take a map:

```js
hintMarker.css({
  "top":  topValue + "px",
  "left": $(".CodeMirror-gutter").width()+"px"
});
```
Here you can set the database name for all php files!
Just FYI, there's a trivial macro __wt_calloc_def(session, number, pointer) that allows you to allocate some number of fixed-size structures, so this would be __wt_calloc_def(session, 1, &cp));
This link is wrong. It should be ``http://api.symfony.com/master/Symfony/Component/HttpFoundation/Session/Storage/PdoSessionStorage.html``
if there isn't anything in the href, do you either want to remove any reference to it completely or just place a # in it and return false in javascript?
node does it this way.. I'm hesitant to change much of the node 'flow' because someone could look at that project to add new features or fix bugs.
In Scala it's more common to don't use getters and setters but the direct name. So

   def size = ...

instead of 

   def getSize = ...

Scala doesn't need getters/setters, for every publicly available value can be replaced inside the class by getters/setters without the code using the class noticing it.
Understood. Just trying to avoid the addition of new tech debt. +1 to rest of it
Ah yes, you are correct. I think you are right in saying the plugin won't be able to auto-magic JSON convert. We could do something like 

``` javascript
function isOptions(options) {
var a = ['raw','expires','path','domain','secure']
for(var o in options) if(!$.inArray(o,a)) return false;
return true;
}
```
Haven't executed that code but you get the idea...

Wait a second, when is it valid to get a cookie **with** options ? Shouldn't all gets be just the key ?
@technoweenie we haven't announced this functionality yet, we want to do it as part of a much larger push. 

So there's no problem, people using it so far have found it by chance and are unlikely to be using this, at least to our knowledge.
Same with the `count` here. But `(empty? (filter (complement pred) a-seq))` is nice and cheap.
Mind just making these double quotes?
also you're using `this` here, instead of `element`.
you don't really need this, because you don't inject it in the DOM..
This looks weird. I'm just sayin'
It seems that this code has a lot in common with `PopulatorInit`, shouldn't those two classes be merged?
I didn't skip it; it's in PR 92.
We need to add an overload that takes a namespace in as well.
There should be an addAll method, no?
Was just following suit from the function @brendo created above it.
Damned, I've already made this rename (in "pdb") but I've pushed that on master instead of filter, I'm updating the pull request.
It works :)
I have no idea why, though :)
Ehm, you are doing it in opposite direction. Wouldn't it be getter if you would simply find URL from `url_name` and then check if this is the prefix of current URL? If it is, you return something, if not, not.
why is this float not working at all?
*opened_connections* -> *connections*
This is not good. There is a race condition. What if some other task (if tasks will be very long or multiple servers or whatever) changes `is_online` to `False` already: then you will send an update again. You have to set `is_online` to `False` in an atomic operation only if all condition still hold true.
From here https://bitbucket.org/mitar/django-missing/?
How do I do that from bitbucket. Do I have to clone it (hg clone ssh://hg@bitbucket.org/mitar/django-missing) or how?
So the best thing here is to completely reorganize forms, and also change registration form with registration view. Sounds like fun :). Will try do do it asap (after getting some sleep first :) )
No. `flags.py` has not been deleted. `bin/openquake` is no longer using it, but it's still used in other places.

It should eventually be phased out, but that's outside of the scope of this bug.
Evtl. Message und Title in den Konstruktor aufnehmen. Dialoge mit verschiedenen Messages werden so oder so nicht unterschieden.
That sounds fine. I thought you might be thinking along those lines. We could revisit it if it starts to be a problem, but it's fine as is
You don't need the 'i' Flag as the regex does not contain any a-z text.
what happened to this file?
We need to be able to identify the changes, but it seems you rewrote it all.
Depreciate or Deprecated?
I'm not sure "adapt to changes of the same variable" is a good way to characterise this. Perhaps @rwldrn could offer a quick hand with some alternate phrasing here?
I'l try to be clearer. :)
I can't see a *real* use case of asking a collection with a pk argument.
Supporting this option cost us some twisted alogrythm: in the __call__ method AND in the _collection one, we have some weird `if` to handle this case.
Asking for a  collection has already some limitations: for example, you cannot use as argument a field that is not indexable.
In my point of view, we also could add the fact that you cannot use a pk argument to get a collection.
This will make the code really simpler and clearer.
(Oh, it's my RER station!)
Would this also work for physical registers?
Apologies for the nit, but s/login/log in
You can see an example of this here: https://github.com/davehunt/bidpom/blob/master/tests/webdriver/base.py#L18
Not "it's", "its".
Might drop the braces in this if-else, since the one above doesn't have any.
Replaced concatenate with ``column_stack`` and ``row_stack`` everywhere in https://github.com/rgommers/statsmodels/tree/pr-408-comments.
It looks to me that this is now missing from WLS/OLS, should cause a test failure (hopefully)
add "as used by fmin_slsqp"
some explanation here or in Notes section: If the optimization stops where the gradient is not close to an optimal solution, then setting ``acc`` to a smaller value might help.
no, the URL that has to be whitelisted here is the one that will end up being the src of the iframe. which in kitchensink, is the one at mwhenry.com. The link that's in the remote page (the one pointing to mulberry.toura.com) is the one that's *not* whitelisted, and the one that should kick out to the external browser.
Use Django's JSON library.
This comment should be removed. mavensearch.net, as far as I can tell, is a spam website that is not related to maven at all. Other organizations have removed links to mavensearch.net, citing them as nsfw for their organizations.
I confirmed that the warning is gone now that you've put this fix.
This is not flux inside, rather than photons outside.
In alpha order now
now testing for assessive is right
I left this one here because `tzinfo` would be a run time dependency, would it not?
Yes, I'm not very proud about that either but I'm not sure how to do it safer. Maybe we could just add those routes into a new file inside AcmeDemoBundle and then the detection would be easier as I'd just need to find the include file part?

What do you think?
And here I think it should be `self.function_or_constant(self.predicate, *pred_args, **pred_kwargs)`
Surely double quotes are better here than having to backslash escape the apostrophe in Can't, right?
iOS change the keys when the language is changed "ABC" became "أ ب ج" :)
Weird indent. Mozilla always use two spaces.
Let's hold this off a bit until the `MPIRequirement` is fixed. https://github.com/mxcl/homebrew/issues/16678
Same answer.  The waypoints calculate a new bearing to display while being dragged (based on WPValuesChanged).  However, the flight data model is only updated when the waypoint is dropped (WPManualChanged) so there is no loop.
Creating a Data Timer for it to pass data to the timer
About effect 1 : just a mistake, I was talking about #0.

I just tested it, and this is crazy but gouge damage effect seems to break stun. With this, it doesn't ...
1. Correct spelling is `XCache`, not `xCache` (according to the official site).
2. Would be good to add here [eAccelerator](https://eaccelerator.net/) and [WinCache](http://www.iis.net/download/wincacheforphp) (meaningful for MS Windows Server users), wouldn't it?
yah, i just caught that. doing so
it's been replaced by the stuff below it

On Thu, Sep 27, 2012 at 5:48 PM, Cory Zue <notifications@github.com> wrote:

> In logistics/settings.py:
>
> > @@ -27,9 +36,9 @@
> >  LOGISTICS_NAVIGATION_MODE = "url" # "url" or "param", depending how your site navigation works
> >  LOGISTICS_USE_SPOT_CACHING = False # use spot caches in various places we've found performance hits
> >  LOGISTICS_SPOT_CACHE_TIMEOUT = 60 * 60 # spot cache timeout, in seconds, defaults to an hour
> > -class StockedBy:
>
> Assume this wasn't used anywhere?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dimagi/rapidsms-logistics/pull/3/files#r1712600>.
>
>
String.CASE_INSENSITIVE_ORDER уже является Comparator<String>
I like this much better. Only problem is, this is no longer a topLink. Should we still be using the topLink name?
This line appears 3 times. It could be moved to the top in a separate if-block.
I suggest "non-negative" instead of positive.
Then the question is why the testing framework didn't find this. Gael, any ideas?
this test could be useful in test_common.py when predict_proba is available.

I'd personally put it there
Could you explain what is improper about this docstring?  Is it that it doesn't fit in one line?
Will do

On 28/11/2012 14:15, serge-sans-paille wrote:
>
> In pythran/typing.py:
>
> > @@ -256,32 +259,33 @@ def isargument(self, node):
> >          """ checks whether node aliases to a parameter"""
> >          try:
> >              node_id, _ = node_to_id(node)
> > -            return (node_id in self.name_to_nodes
> > -                    and any(
> > -                        isinstance(n, ast.Name)
> > -                        and isinstance(n.ctx, ast.Param)
> > -                        for n in self.name_to_nodes[node_id]))
> > -        except NotImplementedError:
> > -            return False
> > -
> > -    def combine(self, node, othernode, op=None, unary_op=None,
> > -                register=False):
> > +            return node_id in self.name_to_nodes and \
>
> I really dislike this kind of trailing backslashes
> You could rewrite this as
>
> return ( node_id in self.names_to_nodes
> and any([isinstance(n, ast.Name)
> and ... )
>
> taking advantage of the parenthesis to write multi-line code
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/serge-sans-paille/pythran/pull/30/files#r2249806>.
>
This will require a schema version bump
Appcelerator coding standard requires testing for NULL, e.g.,

if (event_ != NULL)
Can this be removed, since it's commented?
@tinogomes LOL
bouNding
It's debatable whether we should give an error for exceeding specified max length, or only if exceed KEYBUILD_MAXLENGTH, especially since specifying the max length is only still done for backward compatibility reasons.
Given this name needs to match what is created for a user, shouldn't this constant be defined somewhere a little more public/global?

What's wrong with the TMPSCOPE #define we already have?
Not against backporting, just don't know why it wasn't a problem before...
Really, just as resizing the rowsToSort can cause spilling, resizing index should be allowed to too.

It doesn't matter too much in hThor at the moment, since this is the only callback, but if there were others, it would mean that other (lower priority) callbacks would be called if index sizing ran out of memory.
Is this really necessary? BTW does not merge with trunk.
I don't think this should go in the EventAdapter, as it has nothing to with adapting the event between the ORM and ODM.

See how the LoggableListener is handling the user
Why do you call it "bindings"? It's it **aliases** for certain keys?

"binding" is when you attach or connect things through cause and effect. Is it perhaps a Java term?
I can't consider this until post grizzley when it doesnt require patches to merge. in facts, all of the stuff that is doing patching should probably live in an external module that wraps nova.
I'm ok with just `read` as long as it throws on non local url. I'm also fine with `readLocal` to signify that. I don't think adding URI at the end necessary as function comes from `url/io` itself it should be pretty clear.
We should throw what's in $mysqli->error too as it's impossible to work out what's gone wrong otherwise.
I asked about this numeric -> associative on #php.pecl and their response was:

```
(07:21:03 PM) robertbas: good evening! while developing zend framework 2, we came across to a rather... interesting statement from one of our developers about how php works with arrays. namely, if one creates an array like: $arr = array('a', 'b', 'c'); and then calling unset($arr[1]); the array itself is changed to an associative array. so... is this true? seems a bit silly to me to be honest, but still want to ask it.
(07:21:55 PM) robertbas: for more context see https://github.com/zendframework/zf2/pull/880#r514820
(07:23:42 PM) nikic: robertbas: PHP does not have such a thing as an associative array
(07:24:11 PM) nikic: robertbas: In PHP arrays are array, hash tables and lists at the same time
(07:24:14 PM) Florian: how'd you call ["a" => "b"] then?
(07:24:26 PM) robertbas: nikic: so basically that comment is... not true?
(07:24:32 PM) Florian: implementation details aside, everyone calls that an associative arra<y
(07:25:00 PM) Clicky: robertbas: I can't reproduce that result on either 5.2, 5.3 or 5.4
(07:25:15 PM) Clicky: and it'd find it very counter-intuitive if that was the case
(07:25:44 PM) nikic: Florian: I probably should have said that PHP does not have any arrays that aren't associative ;)
(07:26:51 PM) nikic: ah, now I see what they mean
(07:27:00 PM) nikic: that the array isn't sequentially indexed anymore
```
agreed will update.
Can you edit the documentation to match the general style? i.e., for parameters:

    Short description

    Parameters
    ----------

    name : type
        param description...
    ...
Well, if you specify a distance but don't actually have one I want it to
fail hard, instead of making an assumption about what the user wants. But I
did add a single argument constructor that people can use that defaults to
km.

On Thu, Sep 20, 2012 at 2:42 PM, dmansen <notifications@github.com> wrote:

> In src/main/java/com/cloudmine/api/Distance.java:
>
> > @@ -0,0 +1,54 @@
> > +package com.cloudmine.api;
> > +
> > +/**
> > + * <br>
> > + * Copyright CloudMine LLC. All rights reserved<br>
> > + * See LICENSE file included with SDK for details.
> > + */
> > +public class Distance {
> > +
> > +    private final double measurement;
> > +    private final DistanceUnits units;
> > +
> > +    public Distance(double measurement, DistanceUnits units) {
>
> Sorry, in the event of null, don't think I was totally clear
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/cloudmine/cloudmine-javasdk/pull/16/files#r1654142>.
>
>
Why the thread.sleep? is there something you can wait for?
Yes, that is the intention - somehow how that top spacing had gotten lost (a change you made perhaps?)
I like the subject, but I think you should add a little more to the description. You say what Defensive Programming is not, but you do not say what Defensive Programming is. Or (alternatively) what problems is it trying to solve.
Also, wtf? why are you reimplementing something that's in the API?
This should be:

<a href="mailto:{{ leg.email }}" >...</a>

However, I don't see the reason why we should have the mailto on the browse page, since only developers should be using it :)

I'm a bit -1 on putting it under an h4, perhaps move it up with the <dt><dd> tags up above?

Other then that, it looks good! :)
@mramato is spot on.  This is a good change.  I'm not sure about the performance - and I don't really care at all in this case - because it needs to find the function in the closure.  I don't know how that compares to the prototype chain.
I hadn't thought about the similarity in names between `toBe` and `toBeIn`.  It turns out that Jasmine already has `toContain` which is more conventional, and already uses the more broad definition of equality.  I'll change that one test and get rid of this matcher.
OK.  I would say this could use a comment, but when the interface is changed to use initial width and height it should be obvious enough.
> Are you using this to convert from 64-bit to 32-bit floating point? I had the same idea a while back.

Yes, this is the only way I could come up with to do it.
The check for `projection.project` could be overkill.  Do we ever check for functions on an object passed in?
I'd probably just keep:
```
@see UniformState#getView3D
@see czm_view
```
`eliminateHoles` will throw its own exception if `outerRing` is undefined.; I'll remove that check here.
... \`pause\` and \`paused\` event_s_
Is this for block really duplicated? Does it make sense to write a function?
Ubuntu One use PlainTextSignatureService, but the service complain if the plaintext word is not uppercase, for the other hand Yammer (which is the other api using this class) works both ways

And also on the OAuth documentation http://oauth.net/core/1.0a/#anchor21 shows up on uppercase
`UserCredential.missing` sentinel.
Looks like this should get removed? Its passed to the format method below, but that only looks at the first argument.
The 'd' in the integral shouldn't be in \mathrm

See: http://mathworld.wolfram.com/Integral.html for how it is typically formatted
I think @monarchdodra is right; let's do it that way until #945 has been reviewed/merged/whatever.
I'm actually torn on these kind of changes. These are perfect examples of where the string syntax is better than the new lambda syntax. _Most_ of the examples in std.aglorithm are. It's the more complicated cases or cases where you need to call functions which std.functional doesn't have access to where the new lambda syntax is useful. So, I'd much prefer to have strings used here. However, it _does_ need to be clear that the new lambda syntax can be used with std.algorithm. So, I don't know. I'd probably only change the examples which involved making function calls and leave all of the arithmetic and pure comparison ones as strings.
Done. TY for the recommendations.
should probably be

    raise Http404
> I'm also for fast feedback. But I wouldn't like to step Checkstyle step in when doing a compile or test run. In other words, I'd prefer a quick test feedback over a quick Checkstyle feedback.

Wouldn't the instant feedback and testing not be in the IDE and hence not relevant anyways? 

> Running CS takes some seconds, and IMO that sums up over time. If we really want to bind it to compile I guess I could run with -Dcheckstyle.skip=true, but I think it makes more sense to bind it to the verify phase.

I could and I don't feel probably not too strongly about this. Personally I think that _verify_ is the phase where I validate the generated artifact (aka jar). Processing the sources has occurred by then. But if it makes you happy I can change it :-)
Indentation, indentation, indentation. It sucks like this.
# YOU'RE CRUFT
This goes to setup.py, like all other dependenc
`response.body()` returns a `String`, so you can lose the `.toString()`.
You're holding two cursor handles for the entire duration of the fetch. And all we're doing is walking them sequentially to invoke a callback. (See `FetchRunnable`.)

A `MergeCursor` seems unnecessary here. Just have the `RepositorySession` grab two cursors…
OK, but #2597 makes no mention of the need to try import 'from PIL' first. So that ticket will need to be updated with the info (and link) to #9790.
When I merged earlier it was in conflict, so by resolving it it added it as part of the diff
Could you change this to match the style of the file?
You have an enable here without a disable.  Also, you actually don't even need the DLAppHelperThreadLocal.setEnabled(true) since the disable/enable is already handled in the deletePortletFileEntry().
Yes, because it puts highest priority at the top.  Unless you think there's
a reason to do it the other way.

On Wed, Nov 7, 2012 at 7:50 PM, Jordan Moldow <notifications@github.com>wrote:

> In esp/esp/program/modules/handlers/teacherclassregmodule.py:
>
> > @@ -736,6 +736,7 @@ def makeaclass_logic(self, request, tl, one, two, module, extra, prog, newclass
> >
> >              # Now that we're done putting together multiple resource type sources, listify!
> >              resource_types = list(resource_types)
> > +            resource_types.sort(key=lambda x: -x.priority_default)
>
> This sorts in decreasing order (I think). Is this desired?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/learning-unlimited/ESP-Website/pull/435/files#r2063834>.
>
>
what about the route name? we could use the one passed as argument, but so far we also said empty string is ok. it is not used, it just needs to be what we use afterwards in generate. there is only one route in the collection anyways.
Надо бы еще пару тестов: когда вторым аргументом передают очень отрицательное число (большее по модулю, чем длина списка), и когда вторым аргументом передают не Fixnum (а, например, Float или nil)
I don't think the test need be that complex. I'd rather add a simple test that calls ```getFeatureIdFromEvent``` twice : first with the layer visible and then with the layer invisible. Only one feature would be needed for that test.
Wouldn't it be good to initialize it anyway, just in case we end up using it before it is set?
I was not sure which way it would be less misleading, as using `GC.clrAttr` requires that you know that nobody else sets `NO_MOVE` – but I guess you are right, it should be included in the example to not cause confusion due to the asymmetry.
Definitely; I prefer #pragma, you used it. I was just being lazy and letting the IDE work for me. (Fix Submitted).
Well, it works, Just tested it but I also improved (will push in a sec)

You right it was destroying TemporaryQueue right away so I wrapped this block in IF (CCF). The reason why it worked anyway is because creation of the consumer is in the try/catch and if it results in Exception JOG recreates a destination and sets in in the request. 
That try/catch is necessary to detect invalid destination die to connection loss (there is a brokenBroker test there), but it also made this work so kind of cool.

Anyway, I've polished it up and will push in a sec.
Added that assertion.
Just want to confirm that we don't want to detect/ignore an IllegalArgumentException/NumberFormatException?

I suppose this __would__ be fatal on the ID attribute, but perhaps we could fall-back to returning a String? Otherwise, there is no way to get/remove a message.
hastened...
**milliseconds**
This change is not strictly necessary since the `thead` will always appear before the `tfoot`, but I figured it couldn't hurt to be explicit just in case stuff gets moved around in future versions.
I think you're getting a response here, not a request.
I am not sure you should have the space here.
these new lines don't have to be in the config, and they are false by default
Should we do that now? How complex would it be?
It's a twig specific extension so that's why its named twig.xml
is `isStrict` really needed? Maybe I'm misunderstanding it...
Always pass strings as reference. The extra allocation is expensive.
Also should add some other tests to ensure we don't add that property when signing or when -d isn't provided.
`StatusIcon` knows how to display itself, what about the resource icons? Having status icons render themselves isn't that much useful if resource icons don't do that.
About the download or about the registering of the namespace (`app/autoload.php`)?

Register of the bundle in `app/AppKernel.php` is still needed even if using composer.
xrefs to @Indexed and programmatic API !?
does Twilio require UTF-8? If so, would be nice to specify that in the argument to `getBytes` ( http://docs.oracle.com/javase/6/docs/api/java/lang/String.html#getBytes(java.nio.charset.Charset) )
I see, so you want to use the pull request as the review mechanism to merge individual branches into the master.  This could work nicely.
Can you explicitly state that it's a directory when false is returned in the doc then?
proposing clarifying: "waiting for mutex to be unlocked" -> "external caller waiting for mutex to be unlocked" or something
Mark, this went through 3 iterations and i did sent out email to Juergen and you explaining that we may have discovered a case which is currently not supported by spring - pre-instantiation. The details are actually described in the email as well as the commit log message, so I am not sure what else can be done. I personally don't consider this as an abuse since I am using what's provided by the framework and FactoryBena is responsible for creating beans however if you have a better idea i'd love to hear it.
I think we should omit this as well due to the boilerplates inclusion of .clearfix to parent ID's and classes with floated children. See link here for an explanation of the clearfix:

http://html5boilerplate.com/docs/css/#clearfix-

Basically adding .clearfix to an element will ensure that it always fully contains its floated children by adding a few rules that allow it do degrade gracefully and work correctly in IE6/7 without throwing off the page layout.
Same logic applied to both splash and icon, no comment added...
This is for Embedded Containers like OpenEJB/Weld EE etc where the Test Lifecycle is not reexecuted incontainer, but rather client is is part of the container side. 

TomEE, GlassFish, JBossAS etc should work fine as is.
I like it, although as noted earlier in this pull request, the file is not being linted atm. Although I think using this convention we could change that.
we don't want to call clean-native because then Jake takes a long time on every subsequent call. Clean-native is only called when jake clean is explicitly called. 
This way only the first Jake will take time and then subsequent Jake calls are fast
Should have test cases for issues discovered by SV&V. For example there should be a test for invoking without a protocol...
(== 0 (mod n divisor)) palauttaa sellaisenaan booleanin