labofsahil/github-event-dataset-2012 huggingface dataset:
This was a roundabout way of doing things, so I handle everything on a single line.
that makes sense.  +1 to waiting.
From easy to hard:

1) We can make a read before making a write.

2) We could change `db.useOwnsEmail` to return the lastUsedAs value... which seems like a hack.

3) We could stop using `db.emailIsVerified` and use `db.emailInfo`. We can enhance the metadata returned from emailInfo to add `verified`.

4) We could introduce caching layer like memcached to make the read before the write cheap.

I don't see any issues with doing #3, I'll put that on my TODO list.
ngettext RIP
We already have that code in the other test. We should reduce the duplication.
Seems like that a lot of requirements have been changed since the XHTML days. So HTML5 actually allows to put a span element directly under the body.
float64
`before do` is a bit harder to read and immediately comprehend than `before :each do`.  For this reason, we should use `before :each do` and avoid `before do`.
@daniel-pittman yes, I went to change it, but then realized my use of pending is wrong (it's missing the do-end block to wrap what's in the example. But then I realized I can't actually use pending, because rspec still tries to run the test, expecting a failure or exception, but ruby segfaults. So the only thing I can do is skip the test entirely on Windows. Updated the pull request accordingly.
don't need return type.
this comment should have been moved too
I am unsure about this line. I didn't want to copy all the method over to the `DependencyInjectorProxy`. Ideas @ralphschindler  ?
In this file yes, as for the rest of `Zend\Authentication` it requires lot more CS fixes
Exception is not required here as message is valid with raw body. Converting it to Mime\Message might be needed for further message manipulations. End user should check if body is string or object anyway.
Trigger MvcEvent::EVENT_DISPATCH_ERROR have no sense here for me. Dispatch is finished here :)
Accord your implementation after call setQuery, getQuery always will return a value. So you can transform this if(!) in a else from the above if
First, you store the value of `strtolower($className)` -- as such, this condition will always fail.

Instead of doing the work inside the conditional, have a conditional test if the array key exists, and return the value. Then do the work in the main body.
```php
$normalizedClassName = strtolower($className);
$normalizedType             = strtolower($type);
if (array_key_exists($normalizedClassName, $isSubclassFuncCache)) {
    return isset($isSubclassFuncCache[$normalizedClassName][$normalizedType];
}
$parents = class_parents($className, true) + class_implements($className, true);
/* ... */
```
It's not actually the first either.
Any reason for Win8 as opposed to win8, since this goes into a URL?
`"change" : function() {` -> `'change': function() {`

No space after object keys.
This applies in multiple places.
@samiluukkonen Kaikkialla muualla käytetään arvoa M tässä arova K ?
@harjis Ennakkoja voi jatkossa toimittaa vain jos $automaaginen != ''. Ei kai nyt sentään?
this is wrong. ``$oc`` is the connection, not the platform
It clarifies the "prepare config" comment
setUp => setup
http:/hpc.ugent.be
just use ''.join([x for x in [prefix, installversion, suffix] if x]) here
I think each module name should be enclosed in backticks
I think you are missing the index.html to bounce you to the home.jsp?
FYI:  This line needs a leading semicolon.  A quick integration into [Stylie](https://github.com/jeremyckahn/stylie) broke the compiled version because rekapi.js gets thrown into the Require.js optimization tool and actually invoked the library IIFE that came before it.

I'll add the semicolon after merging.
Wonky indenting
Yes. Should be light. More correctly, it should be 300. I didn’t check this line. Sorry.
I realize this is in the original version, but is this a case where we should use 252 (the number of trading days in a year.) instead?
Hmm... if the buffer reaches the max number of traces, and the tracer sends all of the traces, then `_lastSentTs` gets updated, and when this runs it won't actually send the traces.  But it will reschedule itself for a time quite some time past when `_maxIdleTime` passes.  Seems like maybe it should be scheduled for `this._maxIdleTime - (now - this._lastSentTs)` seconds in the future?  Alternately, perhaps the timeout should be reset when `this._sentTraces` is called?

E.g:  say _maxTraces is 5 and _maxIdleTime is 10 seconds.
* At t=0 seconds, `_lastSentTs` is 0.
* At time t=5 seconds, `_maxTraces` is hit, so everything is flushed in the next tick (assuming almost instantaneous).  `_lastSentTs` gets set to 5.
* At time t=10 seconds, `_periodicSendFunction` gets called, but `(now - this._maxIdleTime) > this._lastSentTs` is false, since `10-10 = ` is not greater than 5.  `_periodicSendFunction` reschedules itself to run again at t=20 seconds
* At time t=20 seconds, it's been 15 seconds (5 more than `_maxIdleTime`) since the last batch was sent.
Hrmm... let me check... otherwise I'll close that pull request and create a
new one... fixing both the uri type mapper and reverting the protection
stuff

Weston


On Wed, Dec 19, 2012 at 2:45 PM, Ross Bamford <notifications@github.com>wrote:

> In src/com/roscopeco/ormdroid/UriTypeMapping.java:
>
> > @@ -0,0 +1,39 @@
> > +package com.roscopeco.ormdroid;
>
> I believe you'd have to use
> DatabaseUtils.sqlEscapeString(value.toString()) here, in case the URI
> itself contains quotes. This is used by StringTypeMapping, and will
> properly quote the string as well as escaping any embedded quotes in the
> string.
>
> As I understand GitHub you can make further commits on your fork and
> they'll be included in the pull request?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/roscopeco/ormdroid/pull/3/files#r2471428>.
>
>
I have to check, because ghc complained when I tried to use hoistEither. If the dependency is automatically handled by errors just as well, I'll update the build manifest :)
Why not use the host status directly?
Added cancel() method to bindToRoute return value and made bindToRoute fail early as it did before.
Not that I know of; we could maybe arrange to forward the events from the text objects floating on top of the SVG but an extra layer for the interactive stuff seems good.
yaaa rightttt
you should make this $fID = 0, and then later check if it is a valid object, otherwise I will be able to do things like site.com/download_file/force_download/cookies which will produce errors,
Pointer to what?
vertex_index --- очень длинно. можно просто vertex.
index понятно из типа
а тип лучше size_t
Can't we just use `.where(func)` here?
get_version_rows
small type here.
fixed, passing it around like the town bicycle.
If possible, this should be parametrized. Ideally, the AJDT repo and this version number should be set together (maybe a profile?).
Maybe in this case a ``getOrElse`` would be better, i.e., 

```scala
Bootstrap.virtualMachineManager().attachingConnectors().asScala.find(_.name() == SocketAttachName) getOrElse {
  throw ScalaDebugPlugin.wrapInCoreException("Unable to find JDI AttachingConnector", null)
}
```

But it's up to you.
effect or affect? :D
Nee, juist op 2 regels zetten:

```js
other.receiveDamage( 10, this );
this.kill();
```
DOH!!
У NSArray есть полезный метод componentsJoinedByString:(NSString *), уверен, он вам больше понравится.
В принципе можно, разве что алгоритм не оптимальный
Назначать переменной несущей ранее один смысл, другой - не хорошо. В Вашем случае переменная perc сперва проценты потом кол-во элементов - смысл разный. Была бы программа сложнее можно было бы запутаться.
а тут разве не будет при каждом создании экземпляра класса создаваться новый массив? Замучался да, не думал что это такая проблема )
well i had some problems with installing it under freebsd until i've changed 'make' into 'gmake' in build.js
@straydogstudio may I ask why you took out the var keyword here?
I assume this is for ensuring that shard keys aren't modified, so you'll end up replacing this with an exception?

Can we simply read the shard key fields from the class metadata here? Calling `getDocumentQuery()` seems like unnecessary overhead just to get some field names.
That example could just use `+backgroundScheduler`, since those two signals don't need to be serial relative to each other. Avoiding new queue creation sounds like premature optimization.
Yeah, I'll have to make sure the lines really do execute in an equivalent way. Maybe the thing to do is revert the changes, then make a series of commits that make it easier to see that nothing has changed from a functional perspective. That's probably safer anyway. Do we have a good set of tests we can use to validate that the inverse is working equivalently? I haven't looked yet. If not, what I can do is compute inverses for, say, two time points using the old code for each of the types of apply_inverse's. Then, make the changes, and ensure that the output arrays are identical. Would there be other tests that would be good?
Can this be a manual colormap? My guess is 'no' if it goes through mayavi, which is too bad since we have code for making an MNE-equivalent colormap...
Please encapsulate this logic in another method.
There's a slight formatting "preference" we have. Instead of having the `.. code-block:: php` on the next line, you can omit it, and instead use 2 colons at the end of this line:

>
that's what we will do in this tutorial::
OK, copy-pasted your code :blush: Thanks for your help!
please at least describe the scenario of actions of what is going to happen in the code snippet :) you are good explaining things very clearly :+1:
From what I've read its the convention in Rails for generators
Bah, I should have cleaned this up. Reusing the same spider but by reinitializing with a new source seems to work. I'm writing tests for the second phase that will tell me which is more efficient / easier to test.
please change this to <testSourceDirectory>src/test/java/</testSourceDirectory> (i think that's the right location)
Here, `g:SingleCompile_showquickfixiferror` may be `g:SingleCompile_showquickfixifwarning`? Thanks.
I agree. It just isn't implemented yet.
Why is this back again?
That warrants some further investigation then. It's the standard way to deprecate functions, and I've never seen it not work.
Porque tiene el prefijo `CS\`, y yo lo que quiero hacer es buscarlo en la base de datos de Employees donde están sin el prefijo.
it SHOULD be set to spaces instead of tabs (i use 2) and linux style \n for newlines
"1 sekundę temu" => "sekundę temu"
please remove extralines
typo: then -> than
I like this solution in general, but wouldn't it be better to branch on the version of the directory package instead of the GHC version?  That's the real underlying issue.
Are you sure that these are the only special path characters? Why not "whitelist" valid paths with a regex: <code>unless file_name.match(/\A[-_/A-Za-z0-9]+\z/)</code>
Google Analytics tracking! Sorry, this was something I added to my page because I asked some people to have a go at the mobile site to let me know what they think. You can leave it out for the purposes of our project.
I was following libgit2's lead on this one. We can go with type… although I like that it alludes to `git status`.
I guess I disagree with going lowercase because it's a loss of information, but I'm outnumbered, so I'll change it back :)
So ```pwd``` is the root of the repository, but if your in the root of the repository, when importing matplotlib via python3, you would actually be importing the source rather than the built version? 
Perhaps it would be better to run ```python setup.py build``` which will make a clean ```build/lib*``` directory, or is it advised by travis to actually install the code that is being tested?

At the very least, this mkdir should have a comment to explain the reasoning, and perhaps rename the folder to ```test_results``` (I don't have much feeling for a good folder name either).
this looks like it should just `return result[index].text`
Right. So I investigated what work we would need to do is order to achieve a more "true" object model even for this patch - we're going to have to do the following:

- Refactor the helper method function logic out of GaiaTestCase into a separate class - something like MarionetteHelper that acts as a class for common utility functions seen currently in GaiaTestCase. This is needed to be able to reference those helper functions from the app objects
- For existing patches to continue working that are not being refactored in this patch, we would keep the existing functions for now, but all they would do is call a MarionetteHelper function equivalent off of object created during the setUp phase of the test
- We would create an appobjects directory under gaiatest that would contain gaia_system.py for the system app, gaia_homescreen.py for the homescreen app, marketplace_home.py for the marketplace home page, and marketplace_search_results.py for the search results page in marketplace. Each file respectively does the part of the test seen in test_search_marketplace_and_install_app.py broken down by functional logic.
- We would split out the two files for testing only a search at marketplace and a search and install.

A larger refactoring would need to be taken care of in follow-ups that would remove the functions from GaiaTest all-together, migrate the specific UI logic in the tests out to app objects, and refactor those tests to call into those app objects.

I could either do one of two things with this patch for this comment:

1) Do no refactoring at all but file followups to clean up the areas cited above 
2) Strike a compromise and do the partial refactoring in the list above. Followups would be filed for the larger refactoring

@zacc Do you have an opinion here?
"Dans le doute",
These "artworks" just clutter a good log. In a demo-app you can consider printing this stuff to stdout directly.
Please make it the same as other files.  
It doesnt matter to me if my name us on the copyright.

Eric

Sent from my Verizon Wireless Droid

-----Original message-----
From: Joshua Timberman  
<reply+p-992818-edeacdfa3333cdfb22c2e920c29ec5f356b03969-361167@reply.github. 
com>
To: Eric Edgar <eedgar@zenoss.com>
Sent: Fri, Jun 15, 2012 14:06:55 GMT+00:00
Subject: Re: [yum] Add Includepkgs support,  fixes for x86_64, and a  
repoforge recipe. (#11)

> @@ -0,0 +1,48 @@
> +#
> +# Author:: Eric Edgar (<rocketman110@gmail.com>)
> +# Cookbook Name:: yum
> +# Recipe:: repoforge
> +#
> +# Copyright:: Copyright (c) 2011

Did you want to put your name on the Copyright? :)

---
Reply to this email directly or view it on GitHub:
https://github.com/opscode-cookbooks/yum/pull/11/files#r992818
Ahhh I shall turn of 3.3 for now then.
Banho de lua corrigido
where is `modalContainer` defined?
You're right. Thanks, I wanted a better to do this. New commit shortly.
do you want me to include a function in math.h which can make a subset matrix/vector out of a matrix/vector given the indices of rows to be kept in final ?
can you please for dynamic_cast not catch the exception but just check whether lf == NULL afterwards and then directly do the SG_ERROR here? this saves one indention level.
we select for "li" in menu on line 35 already, can we capture in a var and reuse?
All the Kohana_Exception class does is translate exceptions using the I18n class
Yep, needs no added space between = and ', and add a space between . and $
No need, $phpbb_extension_manager->available() takes care of that. Don't need to duplicate the rules for extension names here.
The error refactoring looks good. 422 is a special case, tho. We should still raise errors here.
same comment here (and elsewhere) ... I'd make methods "public" if the Dart one was public, or "internal"/"private" if the Dart one uses _ prefix. (Types are a bit more tricky, but I think leaving them default internal is fine--that's a good default for applications who shouldn't be exposing public APIs outside their assembly)
Is this import separated out for a reason?
Because multiple notification types can be requested.
to keep B/C, we could use:

    $options['directory'] = defined('JPATH_THEMES') ? JPATH_THEMES : (defined('JPATH_BASE') ? JPATH_BASE : __DIR__) . '/themes';
You're right, I accidentally removed the onclick attribute. I'll add it back.
Why we don't move this to jdroid?
Ditto, "value" is kinda technical - "setting" is better.
isOpen is always checked (later on) so it's clearer to always define isOpen.
Yikes how embarrassing... :-(
Try it. ;-) Hardcode something and try it. Or go to `./manage.py shell` and run code there manually. Twice. And you will see. ;-)
*Returns provider's source URL.*
Those should be market as translatable JavaScript strings. And be in English.
At mongoengine.. http://mongoengine-odm.readthedocs.org/en/latest/apireference.html?highlight=exception#mongoengine.queryset.QuerySet.get
I hope you forgive me because of a quick fix :)
I would link to the OpenHours? Or you propose something else? We could display a list as collapsed list and when you click on it, it expands with the content. Maybe:

http://jqueryui.com/demos/accordion/
Давай этот файл, и логику про его чтение, пока уберём. Этой функциональности пока нет, а не используемый код будет пока только с толку сбивать.
*(Correction: Constructor on type 'FooProxy' not found.)*
With this being a ZF2 (5.3.3+) module, I'd say it's safe to assume fileinfo is on.
Nope ... 180 if I'm not wrong
Shouldn't this be context.getApplicationContext()?
Docstrings for classes and a general description for the approach you used.
Using `pf_run` is recommended because it handles OS errors that are rare but when they happen and we don't log them it's a pain to troubleshoot. I know you can't use it because `pf::util` is not imported in `pf::config` (circular dependency problem) but it should at least me documented in a TODO task.

```
# TODO: once we can import pf::util in here, we should run this through pf_run instead of backticks
```
I find it weird to have it configured like that. I think I would use something like `hooks.before_start.handle_exception` or something like that.
Also, do we support multiple hooks here?
Shouldn't this default be "OpenShift guest"?
Thanks!
Fixed.
Should I change it in the "interface" section at the top also?
node['nginx']['version'] will always return the same value as node.automatic_attrs['nginx']['version']

http://wiki.opscode.com/display/chef/Attributes#Attributes-Precedence

But we can solve that in another ticket.
What's `num` for?
I agree, this will be better suited as options.
If you override `method_missing`, you also want to override `respond_to?`.
This method (and a couple that follow) seem to be repeat code from earlier in the diff.  Can they be extracted into a common class?
Is this here as instructions for how to properly add it to the application context? If so, can that be explicit? Otherwise this can be removed.
This relates to a comment Mike made in `simReal()`, but I think it would be good to allow the user to supply the uniform deviate for this random selection, as an option.
Minor, but it would be good to align the RHS of these - there's space within the 100 chars to do this.
Updated dox for sinc and lanczos to clarify this.

On Jul 4, 2012, at 12:01 PM, Mike Jarvis wrote:

>> @@ -404,6 +401,12 @@
>> 
>>     /** 
>>      *@brief Sinc interpolation: inverse of Nearest-neighbor
>> +     *
>> +     * The Sinc interpolant (K(x) = sin(pi x)/(pi x)) is mathematically perfect for band-limited
>> +     * data, introducing no spurious frequency content beyond kmax = pi/dx for input data with pixel
>> +     * scale dx.  However, it is formally infinite in extent and, even with reasonable trunction, is
>> +     * still quite large.  It will give exact results in SBInterpolatedImage::kValue() when it is
>> +     * used as a k-space interpolant, but is extremely slow.
> 
> Maybe should add that Lanczos is the usual compromise.  It is a Sinc interpolant multiplied by an envelope that cuts off the high-x values relatively smoothly.  [Wikipedia](http://en.wikipedia.org/wiki/Lanczos_resampling) claims that:
>> The Lanczos filter has been compared with other filters, particularly other windowing of the sinc filter. Some have found the Lanczos (with a = 2) the "best compromise in terms of reduction of aliasing, sharpness, and minimal ringing", compared with truncated sinc and the Barlett, cosine, and Hann windowed sinc, for decimation and interpolation of 2-dimensional image data.[2]
> 
> This might be worth putting into our documentation either here or for the Lanczos class.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/GalSim-developers/GalSim/pull/213/files#r1100953


-------------------------------

Rachel Mandelbaum
http://www.astro.princeton.edu/~rmandelb
rmandelb@astro.princeton.edu
Maybe remove the stepK and maxK paragraphs.  There are relevant in the C++ docs (from which they were copied), but this isn't anything the end user needs to wade through.  Most likely, they will say "What's alias_threshold?"
That took two seconds, so I've pushed an updated version.  Please edit as you see fit!
Make sure key is set back
Why libs-milestone?  It appears you're depending only on released libs.
these kinds of bugs are troubling--need better testing.
This row's width is only 10, when a row's width should be 12
# more stuff = to test
It might be worth having a section in the docs about different logging configurations, and including this log4j.properties along with a few lines about how to configure it (given that brooklyn is a library, and may be used by customers who use log4j or java.util.logging or a.n.other as a standard).

But we don't have to do it in this pull request.
I prefer making these private so that logging always comes from the correct class + package name (for when configuring logging).
Again, should check response code.
i will in the update i will make.
It's a reminder for me to know where i have stopped the day before.
Makes sense now. Do you think that deserves to be mentioned in a comment in the actual file?
We generally use `expect { }.to raise_error` rather than `lambda { }.should raise_error`  since the `expect` syntax reads nicer and is the syntax we recommend now for all expectations.

Also, it's best to use `raise_error` with an argument; it's prone to false positives on its own, since it passes for any error, even if you fat fingered something and got a different error than you expect.  I tend to use `raise_error /some sub-expression I expect/`.
Fixed. But what part of it isn't valid Markdown? I can't keep all the various lightweight markup languages straight anymore.
Fixed. I think I had unrolled it for debugging purposes.
Ah, wait with a non-None timeout it will loop forever for real time searches.
Agreed.   So the things that SHOULD be copied are: 
    exp_key
    owner
    version (?) 
    book_time
    refresh_time
    state (must == JOB_STATE_DONE anyhow)
And then what about misc:
    everything except tid, idx, vals?
apologies for being really nitty here, but 'return super.isPublished vs 2 lines' :P
of course you are right! (sorry was thinking about computeservice)

here's what you need.

private static enum IsPrivateAddress implements Predicate<Address> {
INSTANCE;
public String apply (Address in) {
return IsPrivateIPAddress.INSTANCE.apply(in.getAddr());
}
}

returnMapBuilder.putAll(Address.Type.PUBLIC, Iterables.filter(addresses, Predicates.not(IsPrivateAddress.INSTANCE)));
returnMapBuilder.putAll(Address.Type.PRIVATE, Iterables.filter(addresses, IsPrivateAddress.INSTANCE));
Please split multiple declarations on separate lines.
В интерактивном режиме не надо завершать работу, если произошло исключение.
Ресурсы надо закрывать в finally
А где at?
Зачем эти статические поля?
infinispan and prettyfaces aren't provided.
This will change with the rebranding. The user/email should be unique and so we should expect just one email, so we can safely get it without caring who the sender is. You can also use a regular expression provided by BIDPOM for extracting the verification URL. See https://github.com/davehunt/bidpom/blob/master/tests/webdriver/check_sign_in.py#L84
i don't quite grok why both of them exist. they are logging in as different users (paypal vs sandbox), to different pages (developer.paypal.com vs www.sandbox.paypal.com) and the test definitely doesn't succeed without both of them. perhaps @bebe or @krupa can loan us a clue.
What about: "Inherently incompatible device detected."?
If a `socketPath` is specified http-proxy uses it for key creation.
We might want to use this later to calculate project popularity...
Maybe make this a separate test named `test_eccentricity_zero`?
I find `ap` too short a name too. `average_precision` would be much more explicit.
np.diag raises a red flag for me as you allocate a full dense matrix. You can usually use broadcasting
This seems redundant with ``get_class_weights`` in sklearn.utils.
If this test fails, perhaps the module needs a warning for the parameter set that causes the errors? Otherwise, people could run it, somehow fluke their way through without an error, but get the wrong result at the end...
Do you perhaps have any papers I can reference for this ``theory says`` ?
no brackets; could you add a comment that describes the semantics of ``y_range``; you could drop the continue stmt if you negate the if expression and break if it evaluates to true.
This works, but you can get fancy with Python list comprehensions if you want :)

```python
active_apids = len([x for x in apid_json if x])
```

For more fun: http://docs.python.org/tutorial/datastructures.html#list-comprehensions
Already implemented. Message created
These should likely also take into account the layout rules as set out by appcel.  These props (top, bottom, left, right) have relative priority and depending on the cases, some may need to be ignored.
synchronized class can be used instead of lock() and unlock()
Just return it.
We have a class ".hidden" to hide any elements instead of writing them inline.
It's not unclear with change IMO.

Use cases, not sure, you might I suppose attempt to allocate less and sort and merge in pieces, although you'd need some mechanism to do that of course.

Main thinking was, that if you are known to have a relatively small table that grows to say a few MB, it's a shame to constantly reallocate the stable tmp table, when you are not a big confumer and in all likelihood something else can and will be freed to make way for your few MB stable ptr table when you need it.
do these also need to be mutable?
If you end up making a semantic change, it should not be in a 'remove Clang warnings' commit....

I would just delete the return, and open a separate issue pointing out that the switch at present does not do anything.
pedantic I know, even though I prefer the block begin on new line, you don't and inconsistent with try { above.
This whole file shouldnt be here.
then be more explicit and do: `valid = value == 0 or value`
I would make the wording just "No data is available for this signature"
If we don't want people to have to use sudo, then why not document installing it locally (./node_modules)?

@ossreleasefeed I'm not sure it's worth the complexity to explain all this in our install doc, we just want people to get started as quickly as possible, without polluting their system ideally..
Slightly above this paragraph there is a bulleted list of all contexts. In that list I think ManagedWebSessionContext should be moved to be the last bullet. Also there should be some comment about it being removed in NHibernate 4.0.
I just want to make sure -- this method doesn't need a `super()` call does it?
hmm IMO we should try and stick mostly (for now) to the engines implementation so those docs are still relevant otherwise we'll have to document everything we do. Seems kinda annoying though as far as a template engine goes to have to pass some html fragments, but that's up to hogan I guess
What about `require('url/resource').read` ? `require('url/bundled').read` or maybe `require('url/io').read` ?
Data uri stuff could go to `url/data` or maybe we could have just `uri` for that.

P.S.: Also I'd say `url.toFileName` belongs wherever `read` will be.
No it looks like I just had duplicated text on 109-110. So example is not missing either I just have to remove duplicated  text.
In truth, I think semantic versioning would dictate 3.3.0, since it's a behavior change, right?
Stop dicking around w/ the readme! It shows up on the repo dude.
Why function returning function?
cacheWarmer.enabled is already set to false earlier in this file. That was on purpose so cache warming doesn't happen in test contexts.
What's the point of this structure? You don't actually use 90% of this information anywhere else, and you creating a whole new struct when all you really need is two variables that you will be parsing back out for report_auth_info later.
I don't think it's necessary to list that `tuple` is acceptable (shorter is usually better), but I'm fine either way.
Now, that this subject was brought up, then I will add a TODO to this explaining why this should be fixed.
parenthesis around typeof are not necessary. Typeof is a prefix operator like ++ or !
I think you have an extra space after the comma here
As far as I can tell, this *should* be returning "oldChild", not this node.

From: http://www.w3.org/TR/DOM-Level-2-Core/core.html#ID-1950641247:

"Removes the child node indicated by oldChild from the list of children, and returns it."

Is that not what we do in Android?
Is there any reason not to use a one liner here?
I still  see the same text.
not `exists` but `mincrby`. What do you think about creating a helper that takes a command name and an int referring to the type of error message to use? This function could then return the proper error. This way, we can DRY up the error throwing.
You can move this down to where you're passing the object to render.

But instead change it to:
var tagMatches = CSS_NO_MATCHES;
if(data && data.length){
tagMatches = '';
}
All loop constructs (for's, if's, whiles, etc) should have a space after them.
This is what was actually added...
I statically typed the variable so it knows which one to use.
Этот кусок перенесён verbatim. Виджет работал в IE последний раз, когда я смотрел.

А есть подозрения?
> I don't know what effect it would have if you wrote it as a class, or stored the "state" in a struct. But I originally tried using nested functions, and the performance was awful.

I'd try it as a struct and see what's the net effect is.
they are not provided on top of the interface. They are implementing the interface
補足ありがとうございます！
全然分かりにくくは無かったです。
実は自分もここら辺の用語の使い方はどうしよう…と思っていたものでしたんで、参考になりました！
あと、そのmixin使うなら、Gemfileに依存追加しないとならないですね。
(being picky) i'm assuming you've tested performance other formats? in my experience with iOS and `kCVPixelFormatType_*`, BGRA was significantly faster than all other options, but i am officially ignorant re: OSX.
Yes, I know. But I recalled a rule on that, but obviously I'm way ahead off everyone with the coding guidelines that I've got in mind ;) :P
Rather not. They are going to drop PHP versions so there is a pretty big chance that `5.3.2` will be on that list. Also there are some issues with that version on travis due to compatibility issues on ubuntu 11.10. So testing on `5.3.10` should be sufficient. 

Sent from my iPod

On May 3, 2012, at 4:53 PM, "Michael C."<reply@reply.github.com> wrote:

>> @@ -1,6 +1,5 @@
>> language: php
>> php:
>> -  - 5.2
>>   - 5.3
> 
> Could you change this to 5.3.2?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/phpBB-Blog/phpBB-Blog-for-3.1/pull/66/files#r768393
Don't auto-include jQuery UI.  Not every app will need it.

Side note: when including jQuery UI, do it like this:

````
{psu_js src="jquery-ui"}
````
Are we 100% we want to create a directory? 

Are you having a variable directory output?

Maybe it's out of this task's scope to create a directory and we should throw an error instead?
Nit: There are some blank lines preceeding the end "}"s in this file
missing `)` at the end (sorry ;-))
Тип Date
Was ist die Motivation für 30 freie Zeilen?
think there has to be a newline before every ```
can you verify that this is the case?
Вот так смотришь на функцию и не понимаешь, почему у нее имя `inputKey()`, ничего не возвращает, и внутри всего лишь дергает геттер. А также что кто вообще и кто все эти люди :3
Yeah I also noticed that .first works when its loaded in Rails but .first is not defined when running compass. Need to use [0] in this case.

I'll rewrite with that.
yeah, just do the double-colons.  Nothing fancy.  See the link  I provided elsewhere on to how to document matplotlib.
Should be `===`, and this file should use single quotes
I imagine this involves the following format:

.selector {
    atttr:val;
}

that is, expanded, four spaces, all lines end in semi-colon, space before "{",  "{" is in same line as selector, anything else? Thanks.
@schmittjoh the code using it is not in ``setScss`` :)
I don't think this is right
Just tried this and we have exactly the same issue (the data is not defined error)

http://jsfiddle.net/NBPEr/4/ 
and
http://jsfiddle.net/NBPEr/3/

`with` is tricky...
It seems to me just passing in the empty object `{}` is the cleanest solution.
On Sep 28, 2012, at 11:02 AM, Alex Rønne Petersen <notifications@github.com> wrote:
> 
> The problem is that you can't do this:
> 
> union { ... } __data;
> 
> So even if you did:
> 
> union { ... }
> 
> You wouldn't completely conform to the C API since the union members can't be accessed through __data.

You can put an extern (D) function or functions in the struct to do this if needed.  I think somewhere in the Posix API I've done this.
This seems to be pre-XP. Do we care?
issure 43 is filed to track this feature request.
I have these in my global gitignore file as you end up adding these lines to every .gitignore file
функциональность?
nice.
Did not know that one!
Something's wrong with the indentation here...
Whitespace looks off here
Just to clarify: you're saying this test should be calling a different method, but since you're not set up to run the remote tests you're not comfortable making the change? If so, fair enough; I guess we need someone who's set up to run remote tests against the CIM.
`reason` will need to be a property -- use `this.setProperty("reason", REASON_CLINICAL);`
whats all this stuff?
One of these makes the other redundant, I think.
Now it's going to return False every time the session is None.
Changed it to getMessage for blocking calls and pollMessage for non-blocking calls
Can you use the live scope??

```ruby
live.limit(count)
```
minor typo here. Can you fix that?
Why did you delete this test?
why do we use this awful conf/common pattern?
My comment about this got stuck on the wrong line but it's important so I'll repeat it here.

Using good libraries well is an important skill to have. I want to see what besides jquery people know about and think this limits the information we can gain from their submissions.
yes
odd it did not cause a compile error
over to you james


On 9 November 2012 22:40, Sean T Allen <notifications@github.com> wrote:

> In src/main/java/st/redline/core/DynamicJavaClassAdaptor.java:
>
> > @@ -1,28 +1,30 @@
> >  package st.redline.core;
> >
> > -import st.redline.core.reflector.Reflector;
> > +//import st.redline.core.reflector.Reflector;
>
> is this supposed to be commented out?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/redline-smalltalk/redline-smalltalk/pull/150/files#r2082043>.
>
>
This should be calling ``order_by_distance`` & the extra space should go away.
That's the version specified in org.jboss:jboss-parent:pom:9.

Regards
Doug


On 5 Nov 2012, at 09:45, Geoffrey De Smet <notifications@github.com> wrote:

> In pom.xml:
> 
> > @@ -590,7 +529,7 @@
> >        <plugin>
> >          <groupId>org.codehaus.mojo</groupId>
> >          <artifactId>findbugs-maven-plugin</artifactId>
> > -        <version>2.3.2</version>
> > +        <version>2.2</version>
> Why downgrade findbugs? This might cause trouble for our jenkins findbugs integration
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
The description and `@return` can be merged into one line here
Any particular reason for the wanton mutability?  Can this not be done with a for comprehension?
Technically this interface isn't invoking anything, it's just an interface :) The API that does invoke implementations of this interface is TiActivitySupportHelper
`if ( true ... ` looks a bit weird...

seems like this whole statement could be simplified with:

`if ( ! empty( $version )`
This should be baked into your BBWPIgnore module. It should automatically ignore the .bbwpignore file by default.
Ty.