labofsahil/github-event-dataset-2012 huggingface dataset:
Nit, but this isn't a great way to indent `<p>` tags.

```html
<p>Your profile is waiting to be vouched. Send the link below to someone who
    is familiar with your contributions and ask them to vouch for you.</p>
<p>...
```

or

```
<p>
  Your profile is waiting to be vouched. Send the the link below...
</p>
<p>
  If you are...
</p>
```

Overall I prefer the former, but `</p><p>` on one line is confusing.
Line this up with class?
Why would moving the var up make a difference?
"Screencast URL" would be better label text.
Maybe this should say "item" instead of "whatever you are editing," like Line 14.
The "Feedback Greeting" h3 is redundant, since you could just jump to the next h3.
Too strict? Maybe we can relax it a bit.
add ,x..
repalce general.x with x
lol - I think I cut-n-pasted that comment
Markdown baby!!!
if(newAuthor.lastName.match(/^[0-9]/))   Detection author's Date of birth 
Author text include Data of brith .like ( Howe, James, 1946-    Donner, Andrea K., 1967-   Grant, Joan, 1931-curtis neitl, 1950-)
And the text contains multiple Author
I purposely removed a bunch of the animals the first time because I didn't like how many times it had the "four legs" answer. Is that reasonable to have less of these?
This doesnt help, it resets it right away, instead of after using the temp value
Intentional newline?
In constructors you should write "@return self" (or better omit the @return):
http://www.phpdoc.org/docs/latest/for-users/tags/return.html
Don't put the version number here. If you don't define it in the mantychore/pom.xml file once then you are bound to repeat this everywhere you use it.
Feel free to add strict warnings here, see: https://github.com/ezsystems/ezpublish/commit/f69b2fa442e4a1df23b58cd11c39cf6bef9ebede#L2L1521
Still need some approval on this section about the readability.
I think that an approach like the one in scheduleList.js:350 is less taxing on the DB: doing all the reads (`SELECT`, `Activity.getUpdatedOrNew(activities);`) at once and then doing all the writes in a single transaction, instead of having reads mixed with writes. I haven't measured this, so I can't say for sure.
i don't think this import is used.
This one I can definitely get rid of.
Please document the callback. Is it necessary to have it? It makes functions confusing if specifying something makes the function finish in a completely different way.
> This is not a parse error, but a type error. :) Just nitpicking.

True, I was copy pasting a bit too eagerly :)

> But why do you expect to find the test even if there are errors? Since you can't run it, I wonder if it's useful.

I've been wondering about this for a while, and then decided it was worth. Read the whys here https://github.com/scala-ide/scala-ide/pull/260/files#L7R13
True, sorry for using the wrong term. So, I need the full qualified name followed by the method signature, yes. And I'll try to use ``e.getDeclaringType.getFullyQualifiedName`` (or similar) to avoid the cast. Thanks!
this shouldn't be needed as it is in the core gemspec
should've just used <msg> in the html?
Не нужно запутывать человека, она не обязана быть прямо вне класса. Категорию можно объявить хоть в самом main.m, главное, чтобы файл класса был импортирован.

Другое дело, что есть некие правила, по которым делаются проекты, фреймворки, классы... Если вы посмотрите в Xcode, то увидите слева, что там есть определённая структура и иерархия. Классы в своих файлах с соответствующим классу названиям, категории - тоже в соответствующих файлах, протоколы - то же самое. Это довольно-таки удобно, так как у нас слева, по сути, получается оглавление, и можно быстро попасть туда, куда нужно.
в главной программе ты вызываешь [myCell mutate:10] - т.е.  x=10;   и этот x передается как входной параметр для метода mutate, а потом ты пишешь: randomCount = arc4random() % x; - эта строчка генерирует случайное число от 0 до 9.
I suggest doing type hinting here `array $fields` instead of the `is_array()` check below
As this method is private, the default value might be removed
Btw, I would just use the lowercase name for the Lapack routines. The uppercase is a remainder of f77 in my opinion.
why we don't use api.twitter.com/1.1
Basically as written this means that any exceptions raised in the default middleware will _not_ be caught. That seems very subpar to me.
This was here to handle the case when the login fragment was placed on the backstack and we needed to pop it off once the user logged in. The login is now a dialog and this will be removed.
I think we should try to keep the method signature the same for overridden methods.  If you want to set more values, perhaps make a new constructor that takes those parameters and sets them after creating the object with the unchanged new method.
will be one
True, to avoid adding additional complexity I thought: if we hack into the scikit PCA to make the re-attach trick possible keep everything else the same. To we can still say, we've used the sklearn...  makes sense?
Fair enough, I'll change it.
mhm. What would it tell. The time of the first epoch's first sample? 
Implementation-wise the only obvious way to relate epochs to  overall time is to get the first event's index and then call self.raw.index_to_time.
t0 is mandatory for the init, but doesn't hurt, I'm fine with this.
I'm not sure if I understand your question. With pairwise, do you mean pairs of epochs instead of pairs of signals?
In the `konf` we support loading partials using both:

    {! Resolve 'base/api/ark.js' relative to the mobifyjs version !}
    {>"/base/api/ark.js"/}

    {! Resolve 'tmpl/*.tmpl' relative to the project directory !}
    {>"tmpl/*.tmpl"/}

Starting at the top:

A project has a `base_directory` and a `source_directory`. The `base_directory` is the location of the Mobify.js API. The `source_directory` is the current project.

Both of these are passed to the environment. It stores the `base_directory` as `base_path` while the `source_directory` is stored as `paths`.

The environment's `base_path` is only used in the `KonfHandler`. The handler passes the concatenated `paths` and `base_paths` to `compile` which uses these to create a fetcher.

This is a lot of code for functionality we don't make use of.

Longer term, I'd like to refactor this spaghetti.

----

The problem with your suggestion is that as it exists, this instance of the fetcher also needs to deal with resolving `site.json` and `mobify.konf`. The paths for those files may be absolute at this point:

    /Users/john/git/mobify-client/test/fixtures-preview/src/mobify.konf

So we can't resolve that as an absolute path from the any of the roots.

One possibility is to use a seperate fetcher to resolve these files. They should be resolved using 'regular' logic :)
Right. Your suggestion is better.

Do you want me to send a pull request again, or can you change it yourself?
Well, it does work but isn't the way to go. A single backtick is to indicate a link:

    Go to the `symfony website<http://symfony.com/>`

And double backtick is for inline code block:

    Create a ``Page`` entity...

More information on the Symfony docs: http://symfony.com/doc/master/contributing/documentation/format.html#restructuredtext
why duplicating the YAML example, once alone and once in the configuration block ?
I think we can just remove the word `let's` here, then it becomes the second-person "command" form that we like :)
Thank you, I've fixed this
I *think* this section - while nice - is really extra that probably shouldn't be here
Something similar to:

parallelExecution in GlobalScope := System.getProperty("akka.parallelExecution", "false").toBoolean,
Please leave a blank line between properties (Coding standards)
```
coords = r, theta, phi = symbols('r theta phi')
```

Here and everywhere, there is no need to wrap the lhs in ()...I don't think we do this anywhere else in SymPy. (At least I don't see it often enough to remember it.)
Должно работать лямбдой. Ты что-то делал не так.
Впрочем, если упорно лямбдой не получается — пофиг, и так сойдет.
Главное, дай ей вменяемое имя.
Then you'll get it all! :)
Is this a PyCharm-specific tag?
I think Cocoa handle this, but yes, it couldn't hurt.
Why not make the persistence key an additional property of the form? If I had multiple forms in my app, then only one could get saved =\
That auto normalizes all of the files to the rest of the repo
I'd like to keep this as is to fit into the Clustering interface and provide an alternate implementation using matrices, which have different guarantees than the graph version.
I'll go ahead and fix that now.
good call, done.
Apart from the duplication that I highlighted above, the type of a key can be obtained via key.getClass(), so redundant info here.
Looks like the BiFun instance could be reused
This method looks very familiar :)
Couldn't you change it to reuse `DistributedTaskPart.cancel()`?

If not, you should change it to return false here as well...
Mircea, it should be possible for a tx to not go remotely during prepare and yet be transferred to another node during state transfer: if numOwners = 1, the local node was the primary owner before state transfer, and another node is the primary owner after state transfer.

Your solution also doesn't address the situation where the prepare was sent remotely to node A and then it was moved by node A to node B during state transfer.
Forgot to update form the copy-paste. Gotcha. Will be fixed in the next commit
This can use `@(path)`.
Space after the comment symbol
don't think is necessary with the `sort_type` property on line 261.
I liked it better with the return inside the for-loop. Try to extract the other part into a separete method rather than declaring this odd local variable.
@christianv Is this something we weren't using?
Well, we could leave it as is, and end up using this helper for all downloads in general?
I don't understand why we're calling __wt_evict_list_clr_page here.   Sync isn't evicting, so we shouldn't have to check for it on the list, should we?   Can we please talk through this line?

Second, why call this function for leaf pages or clean page, both of which we're going to skip?
Yes, I see why you bind, I meant why not use the built-in forEach method on the array? I don't see what async is adding here... I suppose it might be useful in the forEachSeries case, and therefore it makes sense to support it here too?
why not `children = children.concat(rest)` -- is `.push.apply()` faster?
not tried but

``` php
<?php
array_walk($request->attributes->get('_before', array()), 'call_user_func', $request);
```

wouldnt that work?
This change seems unrelated. The least you can do is put it into its own commit. ;)
"initializers"
Im Javadoc mein ich :) Kann auch ganz kurz sein. Aber ich hätts nie kapiert, wenn ich die Doku nicht gekannt hätte. Und auch so hab ichs erst mit dem späteren Code durchlesen kapiert.
`var attribute `
i take it back, the `_configure` overriding is still useful. checkout https://github.com/documentcloud/backbone/pull/1446

it becomes really useful when you want to add configuration logic to your view that doesn't need to always be called in `initialize`
`count` on raskas operaatio, joten `(not (empty? ...))` olisi parempi vaihtoehto.
Pass 4D texture coords.
Please use spaces to align spaces, not tabs.
The only change here is removing the unescaping for "%", correct?
Again... not used anymore. You have a talent to find those ones ^^
I'll add a manual test case and document that is a downpayment with respect to the expected JSON format in the future.
I am not really sure what you want me to do.
Then combine this into one line. ;-)
Log files in the repository? Or in the MongoDB data directory?
This could blow. Use:

```
foursquare_data.get('contact', {}).get('email')
```

`None` value, which is default, does not have `get` method. Same elsewhere.
`else` and `{` should be on the same line. So:

```
if () {
    ...
}
else {
    ...
}
```
And especially you should not hard-code URLs. What if we move registration page to some other URL?
[Here are lists of all supported query terms](https://github.com/hmarr/mongoengine/blob/master/mongoengine/queryset.py#L672) for MongoEngine. Sadly, they are defined as local variables. So, you can move them out into top-level constants (similar to how [Django has them](https://github.com/django/django/blob/master/django/db/models/sql/constants.py#L5)) in a pull request for MongoEngine, explaining to them why is this needed. Then you can test for that in django-tastypie-mongoengine, and if that constants do not exist, output hard-coded list in django-tastypie-mongoengine, otherwise output those constants from MongoEngine. So the idea is, that we send a fix upstream, but make our code in django-tastypie-mongoengine so that it works in both cases. In this way once upstream will merge the pull request we will use that. And it useful not to have those hard-coded into  django-tastypie-mongoengine so that if later on MongoEngine adds new terms, they will be automatically supported.

Yes, it is getting complicated. But this is life. :-)
I'm not even sure if this really is a bug or is a feature? Because maybe you want your date localized? 

And I think waiting for the Tastypie to approve this can take a while, so maybe this should be in another issue. And for now we should do a workaround somewhere in the API, but I have no idea where?
Oh, this is completely wrong. I just noticed. You should never access settings directly, but through `django.conf.settings`.
Isn't multiple queries slower than one query? 
Or is slower to compare data in python then in database?
Is there something I should change it to now?

On Fri, Mar 9, 2012 at 9:25 AM, Justin Mazzi
<reply@reply.github.com>
wrote:
>> -        'Content-Type' => 'text/xml',
>> -        'X-Username' => user,
>> -        'X-Signature' => signature(body, key),
>> -        'Content-Length' => body.size.to_s
>> -      }
>> -
>> -      http = Net::HTTP.new(URI.encode(host), port)
>> -      http.use_ssl = true
>> -      http.verify_mode = OpenSSL::SSL::VERIFY_NONE
>> -      #http.ca_file = File.join(File.dirname(__FILE__), "../..", "cacert.pem")
>> -      res = http.post(URI.encode("/"), body, headers)
>> -
>> -      # Checks for invalid http status
>> -      unless (200..299).include?(res.code.to_i)
>> -        raise OpenSRSException, "Bad HTTP Status: #{res.code}"
>> +      conn = Faraday.new(:url => "https://#{host}:#{port}", :ssl => {:verify_mode => OpenSSL::SSL::VERIFY_NONE}) do |c|
>
> I know you're just porting this, but VERIFY_NONE is bad.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/site5/moo_moo/pull/8/files#r539255
Strips out everything in `$o_arr` from the group by
The annoying part about this function is that is can't use any jQuery helpers. We need to at least get `version` and `path` before we can fetch jquery.
```code
if (Creature* tornado = me->SummonCreature(NPC_SAND_VORTEX, Point))
    tornado->CastSpell(tornado, SPELL_SAND_STORM, true);
```
or move this spell to creature_template_addon

use uint8/16/32 instead of int
One thing I would suggest is adding a `.limit(10)` to the end of each of these scopes, or the alternative would be to paginate.
so even if you're not allowed to reformat the whole file, next step is to apply the formatting rules on the new code.
this should be

`cleanUp( session );`

(with the spaces)
as fast
> the only reason a key would be duplicated would be out of human error

Not necessarily in this case. I've seen code like this:

    let {document, document: {body, links}} = window

which is valid and useful in SpiderMonkey 1.8+.

See also: https://mail.mozilla.org/pipermail/es-discuss/2012-April/022395.html
would be good to mention that the default is `[1024, 768]`.
Yep, I meant watcher. Did you mean watcher_hooks? :)
let's explicitely use sys,stdout.write('\n') then.
"This guarantees that the functions always return, even on exception."
Oops, leftover code from refactoring, will fix.
Bind with `this._on_response` I guess.
AFAICS the queue gets only emptied if there is a crafter. Should it not get emptied without a crafter too?
This has the effect of adding to the shadow brightness for each light source. If that was not intentional, this should be enough:

```
	// Bits of ring in shadow!
	vec4 col = gl_Color * 0.3;

	for (int i=0; i<NUM_LIGHTS; ++i) {
		float l = findSphereEyeRayEntryDistance(-vec3(gl_TexCoord[0]), vec3(gl_ModelViewMatrixInverse * gl_LightSource[i].position), 1.0);
		if (l <= 0.0) {
			col = col + gl_Color*gl_LightSource[i].diffuse;
		}
	}
```
Typo "summaruized"
Ah ok.  Just so I understand completely:  this method does not handle compatibility mapping as NFKD does, so it simply returns nil if a compatibility mapping is required.
...seems more readable, done.
There is default special rejected. It is normal behaviour.
... But the former is slightly more flexible, at the expense of requiring a little more code to cover the occasions when trying to access the methods of the `SBProfile`
Might it make sense for the default fluxes to be 0.5 in each case?  This is motivated by the typical use of the `DoubleGaussian` being as a PSF model, which might not be what people want it for, but I think this would make more sense for a default PSF-type setting.
On Jul 4, 2012, at 12:16 PM, Mike Jarvis wrote:

>>      * kInterp = Quintic(tol=kvalue_accuracy)
>>      *
>> +     * The ideal k-space interpolant is a sinc function; however, the quintic interpolant is the
>> +     * default, based on detailed investigations on the tradeoffs between accuracy and speed.
>> +     *
>>      * There are also optional arguments for the pixel size (default is to get it from
>>      * the image), and a factor by which to pad the image (default = 4).
> 
> This padding is part of the reason that Quintic works for the interpolation.  Apparently, the extra padding is what lets you get away with a more compact interpolant than the "mathematically optimal" sinc function.  So maybe we should add a comment that lowering this value should be coupled with using a larger interpolant like Lanczos.  But that this is not recommended.

Yes, that's a very important point that the recommendation for quintic assumes that one will not change the padding factor to something substantially lower.  Updated.
Perhaps we can just give a reference, `e.g. Nakajima & Bernstein (2007)`?  I think that would be sufficient...
That's how I installed it on my Mac -- python and math only.  No problems.
+1 for capturing and testing stderr.
you've still claimed them, it just is no longer detected, currently
This function shouldn't return anything, the signature is not correct. `stretAddr` works like an out parameter.
There are no translations for `general.manage`. The English versions has it.
:thumbsup: translation should take care of interpolating.
Most *gui* diffs use green, from for the CLI I chose yellow (like git), because lots of people use green as their default terminal color.
That makes some sense...or we could use `Array()` to convert array.  Although, `Array` is pretty lenient...`Array(any_object)` returns `[any_object]`...is that what we want?
it is still what it used to be before, so "m_icon.isNull();"  will always be false
Done (see 1ed76ac).
I'll inform karishma of this (in case it wasn't on the menu yet). Future problem, so merging this now.
Assuming categories is a dict, then items() won't always return the things in the same order. You could do:

    for slug, name in sorted(categories.items())

then it'll always be ordered by slug. I don't know if there's another sort that is better here.
why is the "Auth-Token" header name specific to rest (yes, I know no other transport there, yet)?
the default ```init``` is not needed on the header file
Any chance that we could just include all the languages in 1 (compressed) js instead of doing this loading of additional stuff for a locale?
I think Chase or Serve consumer webs should be allowed to alias these things however they want.  Promoting the aliases here forces all consumer webs to live with the alias decisions of Serve or Chase consumer webs.

However, I think no alias at all would be preferable to aliases here.
Сделай метод closeOrExit
Не компилировал перед отправкой.
Ну, не 100%, а какое-то подобие проверки надо сделать.
the settings are for unit test only, so it is really up to our test code on how to use it, for example, the unit test can decide to load from an XML file, etc.
the multiple tokens are designed for multiple scopes. When the media service comprises multiple scopes, this will be valuable. This has been shown in service bus.
There is a new test called DeleteBlobSnapshotSuccess below.
why? it ends up in the constructor either way, but it looks neater and uses less code if you just do it the way everything else is.
Just delete it if it's not being used.
I would rather say the optimal **subtree**.
Could you explain that to me just out of curiosity?
Would a ``memcpy`` be faster?
You should not overwrite parameters to ``__init__`` (which would mean that one call to ``fit`` would influence the next call).
So instead of ``self.class_prior = class_prior`` do
```python
else:
    class_prior = self.class_prior
```
and then use the local variable ``class_prior`` in the following - unless it is used also in other placed than fit. Please check that. If it is also used in ``predict``, then you need to store it in ``self.class_prior_``.

Please also add the version in which it will be removed to the deprecation warning, which should be 0.15 I guess.
\ell_t in the paper corresponds to loss, not dloss.
Do we need this? are we going to force git users to install the packages just to get a vhost?
Might as well change this to 2 now
The method name is slightly off, it does allow interruption, it just doesn't throw throw a checked exception when that happens.
Added empty line to separate them
Yep, need to add in EBS volumes. Another tutorial option is to add them explicitly (e.g. https://gist.github.com/724912/116f35d0b7ab30db765b858faea123919592d067).
Can probably remove this line if you want to explicitly pass through the `keypair_name` as an argument?
actullay, no =)

The coordinates space for longitude is from -180 excluded to 180 included in order to have only encoding for the prime meridian
Not new to this commit I think, but if addcontent=true and it's not a super... should it be an error?
Again, following the same style, and would clash with doRename(), where "fromName" is not the same as "logicalname" (since the rename can go both ways in commit/rollback).
This is not good. You are passing files even if form had no fields for them! So somebody can add attachments even when form has no support for them. This could lead to some security issues (there are also no checks on attachment size).

`form` instance should be enough for attachments. If files were attached, they can be found in corresponding `FileField`.
```list``` is already taken by a built-in. Avoid it if you can.
@AdrianGaudebert basically the reason I am using this is to base it on the syntax for the coming variables in the CSS spec here: http://dev.w3.org/csswg/css-variables/#defining-variables
:+1:

I add these Lint wrappers to the rack test suite some time ago and am a big fan of complying to specs! :)
I also was suggested by https://github.com/rack/rack/blob/master/lib/rack/response.rb#L24
Logs again :P
Not a big fan of this `_skip_exist_tests` kwarg.
I suggest a method instead, something like a quick factory.
This will ask some refactor of the `model.__init__.py` method to add some hook there.
But I prefer no to have this kind of specific behaviour in the `__init__.py`
It looks like this is not valid anymore - the point of this regex is to remove any old lines from syslog-ng.conf. However, since you have changed the "syslog_ng_suffix", the regexp doesn't match old lines (ie , the ones without "flags(final);").

I see a solution here: revert "syslog_ng_suffix" to it's previous value, and add another variable "syslog_ng_suffix_final", which you will use in "syslog_ng_conf", but not in "syslog_ng_conf_regex", where you will append ".*" to the regex.
yes, you are right.
I think all the initialization, in order to be compatible with JSB, should be done in the "init" methods.
eg, CCScene.js should look more or less like this:
`
cc.Scene = cc.Node.extend(/** @lends cc.Scene# */{
    /**
     * Constructor
     */
    ctor:function () {
        this._super();
        this.init();
    },

    /**
     * Initialize
     * @return {Boolean}
     */
    init:function () {
        this.setContentSize(cc.Director.getInstance().getWinSize());
        this._ignoreAnchorPointForPosition = true;
        this.setAnchorPoint(cc.p(0.5, 0.5));

        this.setContentSize(cc.Director.getInstance().getWinSize());

        return true;
    }
});
Хорошо. Так и делай ;-)
@shacker Can we make this the calcentral emitter? If wou also have the site_url for the others, we should be able to remove all the if-else statements.
`first` returns `nil` for a empty sequence, so `(first matches)` would be sufficient.
How about,

```ruby
notifies :run, "execute[apt-get update]", :immediately
```
ok, nevermind then.
I think this makes more sense as a note.
Give some indication of what larger vs smaller sigma would imply.
I don't see how the above code can produce any nan.
Ok I should be more explicit. I meant colors + xy image coordinates.
When would a ``ValueError`` ever be raised?

``kwargs`` is always a ``dict``, so ``.items()`` will always return a list. If the list is empty, this will raise an ``IndexError``. If the list isn't empty, this line will always succeed.
`indexBuffer` can be passed to the second argument of `createVertexArray`.
Get some other opinions if you want (CC @kring and @shunter), but I think it is the way to go.
`concat` returns a new array.  `push` does not.
Whichever is fine with me, I've just been using lhs/rhs out of habit for over a decade now.  After this merge, I plan on doing a pass on static methods and coordinates, so I'll make sure everything gets converted to left/right as part of that.  I can change spherical right away though if we are sure left/right is what we want.  I just did a quick check of other libraries out there but didn't find any sort of consistency.
I guess you still don't understand the problem in #32, despite I spent a lot of time explaining it to you on 8/16/2012. I feel really frustrated about my communication skills.

Anyway, I though you understood it and were aware of it, but just didn't have time to fix it yet, so I comment out that test just because I don't want to see the test failing every time I run all tests. But I didn't expect that would give you a sense that it was no longer needed:( 

So lesson learned: never comment out any useful test, although they will fail because of a known bug
This might not work well with views that host children. Will come up with a test case
To be more precise with respect to Android: if you supply a color value and it's not recognized/supported, indeed the result is "yellow" (we've hard-coded that).  If you don't supply any value, then the native defaults take over.  According to a comment in our code, i believe the yellow was put there to "stand out", i.e., make it easy to find mistaken color names/codes.
FYI, had to make a small tweak to the astwalker. The first arg can be either a filename or an AST tree. No sense creating the AST tree twice.
It's different than, say Ti.Omniture. This may just be a definitions thing, but my understanding is that a module is something that is included in <modules> section of tiapp.xml.

"A regular ol' Ti object (say, Ti.UI.Window) isn't guaranteed to be accessible using a global identifier." That's not true in Mobile Web.

Namespace is a better term IMO, but if the term modules is used a lot in the docs, then I guess I can live with it.
move var out of for loop and cache the length to a var for performance
that's not actually my local file structure either ;)  Next time I'll name it /home/wonkauser for you
In the build, minified banner is different from non-minified one. But, `cssHeader` generated with `banner()` above is being used for non-minified banner as well. Shall we keep it that way or fix it? PS: same for js
["democrats", "d"].include? party で partyを１回にまとめられます
There were 2 goals:

1. Make sure the paths printed were absolute
2. Resolve symlinks in the paths

Neither of those is probably as important as printing the method names like yours does :)
That's alot nicer
Goes both ways.
Pro: it's clearer that way and doesn't require knowledge of that particular detail of the language.
Con: adds clutter and makes people who do know that particular detail worry they don't have it right.
dynamic arrays?
Try to search [std.range](http://dlang.org/phobos/std_range.html) for "with a length" string and it will give 0 results. Now try to search "with length". 3 results.

Fixed anyway.
Added TODO. Fixed input/item.html

We should follow up with them about this. A more general question I have is if we want to do fine-grain specification within an attribute. In this example, for instance, I want to be able to add/remove 'editing' or 'completed' depending on 2 independent flags.  Currently we have to do a complex expression or a property in the viewmodel that merges them together in a single string.

In the Closure templates syntax, you could do something like this:
<div {if foo} class="a" {/if}  {if bar} class="b" {/if}> </div>
https://github.com/symfony/symfony/issues/5920
Fix this indenting, please.
wrap into a method or use a useful constant name for null
hm the definiton:
```
handler ( int $errno , string $errstr [, string $errfile [, int $errline [, array $errcontext ]]] )
```
so the last three arguments should be optional.
Why not make the context a dedicated object? I could imagine that we want to add more information to it, like the current path that we are at.
I apologize. I misunderstood where you were adding the longclick. I just checked out your code and ran it to see where the option is added. I agree that it is an icon not menu.
I still request you reword the second sentence. Either of these are preferred.
Here you can select your second preferred navigation tool. It\'s activated by long clicking the navigation icon.
or
Here you can select your second preferred navigation tool. It\'s activated by long clicking the navigation icon in the action bar.
+1. Othewise the reader would need to look into what txInfo.markForPreparing is returning ...
Thanks. Extra spaces has been removed.
@bp323: In the latest code, this has been centralized into sakai.api.Util.Draggable. All places using this iframe fix are now using that function.
please just use CMath::random(0.0, 0.5) ; for that
Should there be a rule to enforce an empty line at the end of the file?
@alexrp Actually accessing a field off the null pointer will trigger that. It's really same thing.
So, if want ot have an unit of work with `TransactionInterceptor` we still should configure some upstream components: poller, gateway, adviced handler etc.
I understand this solution is more around **message process**, but if we plan to introduce advice-chain for entire `AbstractMessageHandler#handleMessageInternal` (e.g. https://jira.springsource.org/browse/INT-2481), won't a component's configuration much more difficult?
If that's the case (that we accept we might break some existing app); then I'd rather fail fast (thrown an exception) than just swallow the argument and do nothing with it.
No, If you don't add "content-type" you should return just **transformedPayload**
Also keep in mind: **transformPayload()** may return **null**.
So, it should be something like this:
````java
 if(setContentType && transformedPayload != null){
      return MessageBuilder.withPayload(transformedPayload)
            .copyHeaders(message.getHeaders())
            .setHeader("content-type", JSON_CONTENT_TYPE)
             .build();
}
return transformedPayload;
````
Aha, ok. I tried to do:

`view.sidebar_results = True`

which seemed to work, but then wasn't there in the event listener.

On 22/03/2012, at 11:30, tito wrote:

>> +	##       ##     ##    ##          ##     ##   ##  ##     ##    ##    
>> +	######   ########      ######     ##    ##     ## ########     ##    
>> +	##       ##     ##          ##    ##    ######### ##   ##      ##    
>> +	##       ##     ##    ##    ##    ##    ##     ## ##    ##     ##    
>> +	##       ########      ######     ##    ##     ## ##     ##    ##    
>> +	
>> +	# Hi Tito!  
>> +	
>> +	# So, I've had a go with it. I had a couple of problems:
>> +	
>> +	# - The big one: I don't understand how to turn on and off the event listener,
>> +	#		so it is currently on all the time.
>> +	
>> +	# - I couldn't set a sidebar_results variable that I could then access
>> +	#		from the FindResultsListener, so I went off the name of the view.
>> +	
> 
> You can set and get the value via:
> `view.settings().set('sidebar_results', True)`
> `view.settings().get('sidebar_results')`
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/titoBouzout/SideBarEnhancements/pull/11/files#r590250

--
Stephen Horne
I'm not sure If you want split this in two paragraphs. If so is needed add a blank line between the paragraphs.
No comma here
Cite who said this quote.
«Основной принцип работы:» можно удалить.
show incorrect client error message for JSONObjectAdapterException
@return tag
Instead of this deferred-creation style, just setup the comments and posts in a `.setUp` method.
Done in new pull request.
'and its children'
I would name this X-AUTH-TOKEN to be consistent with the URL parameter. This is not the API key, it's a token :)
@willdurand that was already there, but i can remove it if you want.
Hmm, _DefaultOrInheritedActionInterceptor_ and _NoActionInterceptor_ are even identical!?
Transactions are guaranteed to exist here
It's happening whether with your patch applied or not. So it's an external issue. Just wanted to mention it here because that will not exit our script and doesn't report an error. It requires manual interaction. So probably worth filing an issue against Python.
change -> changing (here and in `hide` below)
I deleted this, because it is no longer used. This bind happens in plugin.coffee
I hope so, but the server would also need a property, so it knows where the process / workflow is currently.
According to the JavaDocs, `getStatusLine()` can return `null`. Best to handle that.
No key for encryption? ;-)
can you remove the ``routing_key: send.some.mail``
I could add quotes, to improve formatting of the test output, I guess.  printf("%s",Ci) really does print nothing if Ci is completely empty.
Can you move this check to the GraphNavigator instead?
Yes we can use lang files in there, but we have never used them for umil files. However you are right, we should use it.
I was just matching the implementation for orientation.  Is there a reason it wasn't done there and should be here?
Do you have a preference?
Um, there is no "block" parameter in the AggregateResolver, and attach() should not be a _required_ injection point. Additionally, there is a definition for the AggregateResolver in Zend\Mvc\Bootstrap already defining "attach", which allows you to do this:

```php
'instance' => array(
    'Zend\View\Resolver\AggregateResolver' => array('injections' => array(
        $firstResolverClass,
        $secondResolverClass,
    )),
)
```

As such, I don't think this patch is needed.
No space before the bracket
[divmod(index, columns) for i in random.shuffle(xrange(rows*columns)]