labofsahil/github-event-dataset-2012 huggingface dataset:
OK, thanks a lot. :)
Putting the options at the end would allow you to omit them for most tests as they don't need them, instead of having to pass an empty array each time :)
+1 @stof 
The double parenthesis are supposed to show other coders "look I know what I'm doing here so I tell you in an indirect way"
but in [Symfony2 coding standard](http://symfony.com/doc/current/contributing/code/standards.html)  (which composer borrowed) we already reversed assertion so the variable is never on the left precisely to avoid confusion with assignement so it shouldn't be a problem.
```
       if (true === $dummy) {
            return;
        }
        if ('string' === $dummy) {
            if ('values' === $mergedOptions['some_default']) {
```
it is not a template anymore
I'm not saying you shouldn't filter, but the filtering happens on every load now, even though the hidden properties may never change.
Viewsはまずカスタムビュークラスを作成することから始めます。[render](http://backbonejs.org/#View-render)関数をオーバーライドし、宣言するイベントを指定し、`tagName`や`className`、Viewのルート要素の`id`もおそらく指定するとよいでしょう。
`("key", value)`と`{key: value}`の両スタイルの引数を制御する。
Never mind, read through the Rails pull request, which answered this question. It's only the initial leading newline.
Documented it and made it more common instead.
nit: there is two way to see this simplication: avoid repeating `source.` in many places or having hard time knowing from where comes a variable. Is it a function argument, a variable or a request's "private" attribute? Not a big deal, just to let you know what I felt while reading the code the first time.
do you mean 'descendant' instead of 'decedent'?
@lloyd, @benadida - storing the list of emails that has been seen on the current computer seems to me to be a larger data leak than a numeric user id.
these are the various types that can come in through our api.  please review carefully here (you too @benadida) to see if the constraints are as tight as they can be, and no tighter.
sure. done.
That's a left-over we don't need. I can remove it.
It has been named `obs` across the Firefox codebase. I would propose we do the same here, which makes it easier for us later to replace the instances with Services.obs.
Same remark as for the i18n behavior: the cohabitation of the validation and the concrete_inheritance behaviors should be handled.
I am quite happy with that.
@ahpook good point.

The solution to the upgrade problem should be an update to the spec file that stops the service before an upgrade and restarts it after an upgrade if it was started to begin with (thus clearing out any pids that would cause problems after the upgrade). Otherwise upgrades from previous 2.7 versions will report puppet is stopped even if it was started using the old init script. That might be enough of a change to warrant moving the merge target to 3.x.
Added \n s
`os.killpg` does not work on Windows: http://docs.python.org/library/os.html#os.killpg, but `os.kill` works on Windows in Python 2.7+: http://docs.python.org/library/os.html#os.kill
And, actually, I'd change this entirely to:
```
if (!$events) {
    return;
}
````

and then continue on with the event registration. Nothing else in this method is necessary if we don't have the shared event manager instance.
Don't need the implements clause, as AbstractDate already does so.
Use the component-level exceptions, please -- not global exceptions.
You are using tabs for indentation. This is technically correct, but isn't recommended. See [this](http://www.python.org/dev/peps/pep-0008/#tabs-or-spaces) and [this](http://www.python.org/dev/peps/pep-0008/#indentation). You can set your editor to insert the correct # of spaces when you hit the tab key, depending on which editor you are using. If you are using vim, see [here](http://www.vex.net/~x/python_and_vim.html) for some helpful tab-related settings.
No the first enum value. "FORM_CAUSE"
why assign it to a variable?
"get" was reserved for actually getting a property with no logic right? Wouldn't that mean that this is not allowed?
Note that a viable alternative may be:

    build_ops << 'USECLANG=1' if ENV.compiler == :clang
A nice way to avoid having to escape double-quotes in an interpolation string is to use the `%{...}` form, e.g. `%{Latest activity on "#{@policy.title}"}`
yes. but logging is not used in the module, so does it worth importing the module ? 
and where do I log ? I don't know that much logging module
hmmm, maybe. Seems like it's not as useful when you're entirely within your own lib directory. Like I wouldn't want files in the same directory to use "package:", or if I just move a file to "src" it would be a shame to need to add the "package:mylibname" business. We could ask @munificent for an opinion.
Voit miusta jättää ton kommentoimisen kokonaan pois, jos ne ovat luokkaan tuota.
:+1: for this method
You can probably just remove this file.
Changing Location to RestLocation inside of RestContent was not neccessary. It is only used for pathString property.
@bbinet - No objection. What's the "right way" to do it -- should I just push a commit with the change and live with the extra history, or is there a better sequence of steps so that the history is clean?
I already spent too much time on Flickr, it's a real headache to get it working due the lack of documentation.
Minor rewrite

    If you need to debug JBoss Application Server sources then normally you can use _JBoss Sources Lookup_ in JBoss Tools as demonstrated in this [screencast](http://screencast.com/t/i6k9hzkjViOs).
I don't understand why there are two different versions. Could you explain it to me?
we can nuke this line
What about win32?
you can use .
   first.try(:value)

this will return nil if first is nil by itself.
could you a description above? I'm guessing most people won't understand what it means :)
delegate to super.postStop or not?
We didn't go with that PR, but thanks for spotting :)
That's true. Also, this check is a little un-necessary since there's already a test at the beginning of the function.
Thanks, I change to And.
Is using bold in a code block possible? If so, good find! xD
Fixed for older versions of IE -- css classes need to be in quotes.
To avoid the raise of a PHP notice maybe we could declare $this->writeFlag attribute as : protected $writeFlag;
Whitespace before the return statement
Fixed with commit 6b81f9469928c79d4548ca7f4b7d12c746b96194.
I don't really see the point of this if statement. I think it would be better to simply check all of the collections on the page as is done on line 53.
I wouldn't make it special to Windows but would allow to make use of a special file type on each platform. That means we could call the parameter file_ext and default to None. Later we can check if it has been set, if not we can set the default file extension for the selected platform. That way we can make sure to also be able to download e.g. the tests and not only the binary later.
You've left a comment hanging here...
You might be able to use `fs.realpath(path, [cache], callback)` for this, so we don't have to hardcode the number of levels it has to go back.
Might be worth specifying what happens if the user can't be found in Redis.
Using this as a property is a nice idea, but you need to run the test suite before you issue pulls like this- code throughout the rest of the project depends on calling '_type_node' as a method.  The reason `@property` doesn't work is that you actually need something like `@classproperty` (which doesn't exist in Python, though it could be written).
Retain spacing
It's now in core (for this branch and pull-request)
This can be left in.
I'm pretty sure this test already exists - the functionality is already in place so there would be a test for it.  I'd suggest the existing one be refactored and this one be removed
entity enemy met entity spawner vervangen
return ($status) ? TRUE : $result;

... makes it a bit more readable.
Поле DNA - внутреннее состояние объекта и предоставление его наружу нарушает принцип инкапсуляции.
Теперь я начал что-то понимать, спасибо!  Вы все очень понятно объясняете.

Но теперь у меня совсем ничего не работает, подозреваю дело в сеттере свойства, но как это исправить совершенно не понимаю.

Добавляю в класс новое свойство так
https://www.dropbox.com/s/bkheufmk7amia2l/%D0%92%D1%8B%D0%B4%D0%B5%D0%BB%D0%B5%D0%BD%D0%B8%D0%B5_009.png

Пишу метод, но в итоге получается следующее:
https://www.dropbox.com/s/2b2rfacb0kw7i5s/%D0%92%D1%8B%D0%B4%D0%B5%D0%BB%D0%B5%D0%BD%D0%B8%D0%B5_010.png

Что-то я совсем сделал не так :(

В лекции немного по другому объявляют свойство, вот так
@interface Cell () {
 NSMutableArray *idx;
}
@end

В таком случае, похоже, почему-то к этому свойству IDE предлагает обращаться так self->idx 
Но и если делать так, то ошибка будет так же.
Судя по всему есть вероятность (1/4), что текущий элемент не будет заменен. По условиям задачи он должен быть заменен обязательно. И давайте переменным осмысленные названия. И решение с while очень плохое, так как на 100000-элементном массиве со значением prc=100 ваш алгоритм будет отрабатывать очень долгое время.

И, кстати, почему я не могу загрузить ваш проект? Загружается все время пустой.
тут все ясно конечно, если метод void то и вернуть нечего, 
но не плохо было бы сделать тогда какой-нибудь NSLog(@"wrong percent value");
иначе оно просто тихо будет пропускать процесс мутации и радостно рапортовать о том, что мутация выполнена
Possibly.

On Thu, Nov 15, 2012 at 9:40 PM, Brian Eoff <be@bit.ly> wrote:

> Not sure. Can check the documentation tomorrow.
>
>
> On Thu, Nov 15, 2012 at 6:34 PM, Jeff McGee <notifications@github.com>wrote:
>
>> In crawl.py:
>>
>> >
>> >          cpid = os.fork()
>> >          if cpid == 0:
>> >              # child
>> >              print "starting on %s"%filename
>> > -            with open(filename,'w',1) as f:
>> > -                store_tweets(config,f)
>> > +
>> > +            if compress:
>> > +                with gzip.open(filename,'wb') as f:
>>
>> The uncompressed version uses line buffering so that when the SIGTERM
>> happens, you don't lose a bunch of data. Does this lose tweets when SIGTERM
>> kills the child process? Will you end up with a broken JSON file?
>>
>> —
>> Reply to this email directly or view it on GitHub<https://github.com/JeffAMcGee/twitwatch/pull/1/files#r2147327>.
>>
>>
>
>
Since you only check if the equipped item is a wrench you could have passed the player instead of the item.
а чего belongs_to, а не has_and_belongs_to_many?
один вопрос может иметь несколько тегов, и к одному тегу может относиться несколько вопросов
Are you ever using DataManager in these tests? If not, I wouldn't include this nor the memory adapter until we need it.
Your code reviewer humbly reminds you of his disdain for { braceless } statements.
This was a fixed that I applied when trying to serialize an object that contained a null association   (ie.  a belongs_to that has not yet been connected or assigned) 

PHP Catchable fatal error:  Argument 1 passed to ActiveRecord\Serialization::__construct() must be an instance of ActiveRecord\Model, null given
I tu to samo.
> The function returns basically what surfer.io.read_stc() returns, hence "stc" in the name. There should be a better way to make plotting in PySurfer easier... I think ideal would be if an mne.SourceEstimate object could be directly submitted to a PySurfer plotting function, e.g. if Brain.add_data() would accept a SourceEstimate as first argument, of of there was a separate function for that, say, Brain.add_stc. Is that feasible, or is closer integration between the packages not desired?

then I think we should have as_surfer_stc(hemi='lh') that returns a
dict object for pysurfer viz.
I feel like there must be a more correct way of doing this. Ideas?
@wluyima why does this @link show QLocale; instead of Locale, etc?
Add an @since 1.10 here and on the setStrike method
How about "replaced by more advanced database systems, automation, and other roles increasing their database administration skills" ?
The poor Network Engineers seem to have the least amount of explanation. Can we give them a little more? Maybe some of the skills involved?
I understand that, but this basically just moves the match statement into the user execution context, which is not much.

Also when I updated the pull request, my comment about parser(rh) was deleted by GitHub since the line of code changed.  What I had said was that parser(rh) is already being executed in the user execution context, by this line of code:

https://github.com/playframework/Play20/blob/master/framework/src/play/src/main/scala/play/core/server/netty/PlayDefaultUpstreamHandler.scala#L331

The test I added in the last commit proves this.
You should change the previous 3 lines into:

    With Symfony2.1.5 with the Composer autoloader, you can use::
转换 => 变换
“被代理服务器”比较拗口，改成这样：
代理的时候，开启或关闭缓冲后端服务器的响应。
"principles"
Can we clean these lines up using a format string? They are also all over 80 chars
I don't like the looks of this change. `:back` is the "right way"
It should be turned on by default *in isympy*, but otherwise, it will use the default Python behavior (off by default in 2.6+ if I remember correctly).
Форматирование у этого if-блока поехало, +1 таб ему.
Мне не нравится это. Так можно вообще все функции вынести в namespace.
Нахуй так жить?
view-split-top-bottom
yea, that was just for me testing.  I've already removed it on my local
change item to itemType.equals("Books")
Name function more specifically (e.g. add-output-node).  Could be confused with test tree.
You should use the URL helpers here -- this would become `redirect_to stories_url`.
The import statement should be ``ìmport scilifelab.google.project_metadata as pm```
you should use a ``use`` statement per class to be consistent
Please note! This has nothing to due with mcmod.info error.  I am just new to github and don't know how to do pull requests correctly.  This is 'permission denied' bug.
When you need an else: target = 0 ??
Since there was a pexpect bug I guess this is what we have to live with... geeze... :(
You'd trade {} for [] so I guess it wouldn't really help, hence the NGHUTPR
why is preload required?
@asciidisco why expect a Windows path here in the first place? You shouldn't ever put those in a grunt file. Unless there's a good reason, let's drop that stuff.
Actually, come to think of it, what is the point of importing pyplot in a pylab example?  Either use pylab or pyplot, don't mix them up.
Well PEP8 says *not* to align code for readability, but I think common usage disagrees.  http://www.python.org/dev/peps/pep-0008/#pet-peeves
Just to double check, is the print statement going to break older python versions in the 1.1.x branch?
"an" --> "am"
Given @davehunt's comment on my past pull request, we really should start moving to an app object model. Lockscreen I don't think is going to be overly complex, so this might be a good time to refactor this for this patch to separate UI away from test logic. I have a strong hunch the lockscreen logic will be reused in other tests, given that it's the bare bones starting point for flipping the phone on.
No need now, if this fails then the test will timeout at this point.
What happens if we're already locked here? Does this still work?
think this should say Zoom or PanZoom
Hm, didn't know that – should be fixed in ad44044862730448343f85892e8e38f14f40cbbe.
s/normes/standards
yeah dawg, they take up 64-16 bytes more per number!

On Mon, May 14, 2012 at 6:06 PM, robinkraft <
reply@reply.github.com
> wrote:

> > @@ -30,18 +30,26 @@ struct NeighborValue {
> >
> >  # Collection Wrappers
> >
> > -struct DoubleArray {
> > -  1: list<double> doubles
> > +struct ShortArray {
> > +  1: list<i16> shorts
>
> Yeah, was trying to keep storage down. Do longs actually take up more
> space when serialized with thrift?
>
>
> On May 8, 2012, at 7:45 PM, Sam Ritchie <reply@reply.github.com> wrote:
>
> >> @@ -30,18 +30,26 @@ struct NeighborValue {
> >>
> >> # Collection Wrappers
> >>
> >> -struct DoubleArray {
> >> -  1: list<double> doubles
> >> +struct ShortArray {
> >> +  1: list<i16> shorts
> >
> > Hey, are we using shorts just to get the storage down? We're going to
> have to be careful with the casting, as Clojure defaults everything to
> longs.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> > https://github.com/reddmetrics/forma-clj/pull/17/files#r793062
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/reddmetrics/forma-clj/pull/17/files#r820718
>



-- 
Sam Ritchie, Twitter Inc
703.662.1337
@sritchie09

(Too brief? Here's why! http://emailcharter.org)
the name & description is french
What happens if the language isn't supported in tinymce? I assume it'll default to english?
Maybe we can remove the sakai.conf.Editor.tinymceLanguagePacks property (this was added in SAKIII-3836)
SNE probably ;)
please use as name set_use_covertree, same for variables m_use_covertree
Link is broken, and should this go to ss-dev instead of the forums?
Make sure that you don't go over 80 characters per line
- forced reset of splash img. left margin was inherited from unknown -
nothing from base that I saw.
- the quick fix was to force reset of margin/padding


On Wed, Dec 5, 2012 at 4:07 PM, Philip Schatz <notifications@github.com>wrote:

> In css/ccap-biology.less:
>
> > @@ -39,13 +41,13 @@ div.part > div.titlepage {display:none;}
> >   ******************************************************/
> >  .introduction {
> >    /*pulled from Soc*/
> > -  .splash {
> > +  .splash img{
> > +    margin: 0 !important;
>
> Why are margin and padding !important? Can you add it into the comments?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Connexions/oer.exports/pull/33/files#r2328009>.
>
>
- Are you sure ``$listLen`` has the correct type as "integer" is not a sub-type of "double"?
- Are you sure ``2`` has the correct type as "integer" is not a sub-type of "double|null"?
Well, plugins can't exactly modify scripts. How about user events on the plugin objects (e.g. 11, 12 and 13 for respectively start game, start client and start server)?
Because I want to be sure that even if a phase returns no results, that the empty array for that phase exists.
Habs kommentiert.
This is a change in behavior that will affect B/C.  I think what we'll have to do is this:

<pre>if (class_exists('JError'))
{
    return JError::raiseNotice(500, JText::_('JLIB_MAIL_FUNCTION_DISABLED'));
}
else
{
    throw new RuntimeException('Send function cancelled because mail is disabled.');
}</pre>

If the legacy platform is loaded, it will still return a JError.  If it's not loaded, it will throw an exception.  @return can change to "JError only  if the legacy platform is loaded, otherwise an exception is thrown.".
spaces around all operators
Why can't we make this work? You'd simply concat all the linked lists together.
"For real-time search" -> "Only applies to real-time searches"?
Is any example in piplmesh projects?
You cannot use embedded documents here?
Indent inside the block.
Allow 'blank' value and set that as a default.
OK, I was thinking. Lets limit this pull request in its scope. So that also others can do something. So please just do basic panel, define dependency in Python, and create JavaScript API for map event handler. And just a simple list of venues with links to OpenHours. And then we will create a ticket for somebody else to implement accordion and stuff.

I think this is also better for you, to concentrate on hard and interesting problems and leave simple user interface tweaking to others. ;-)
You still have hard-coded HTML here!
This is not good. You do not really want document to be automatically saved on initialization. So normal MongoEngine way is to do:

```
object = Document()
object.foo = 123
# nothing is saved yet
object.save()
```

In your case, document is automatically saved on initialization.
Well, the same problem can happen for the other frame types.
Hmm, no. I think the problem here is all variadic args are expected to 
be of the same type. Not sure of the best way to fix this . .

On 19/02/12 21:10, Joe Groff wrote:
>> @@ -79,7 +79,7 @@ record DequeCoordinate[T] (
>>   }
>>
>>   [T] overload subtract(i: DequeCoordinate[T], j: DequeCoordinate[T]) : PtrInt {
>> -    var mapDistance = PtrInt(i.node - j.node - 1);
>> +    var mapDistance = PtrInt((i.node - j.node) - 1);
> Can this change be backed out now?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jckarter/clay/pull/222/files#r465663
It seems like this should take a boolean, defaulting to false, i.e.:

```ruby
def balance(return_all_currencies = false)
```

Since passing ```0``` is the same as the default behavior, I think this would make the API simpler.
This is Reward.
"Wake locks are reference counted by default."
setDefaultUncaughtExceptionHandler is a static method that, I believe, applies to all threads.  This appears to be written as if it expects it to only apply to the local thread.
I've moved the upload logic to it's own method, but I'm not sure how to implement the `Action<ProgressStatus> progressCallback` -  although it sounds like a good idea. Any recommendations?
I assume this traverses up the inheritance hierarchy, correct?
I've created a ticket #29
"it" is ambiguous
However, expect that readers will have no clue why in case of success, the decorated functions do not return a promise.
Add todo about removing hardcoding.
I ended up creating l1_solvers_common.py
Awesome. I'm going to have the console error/warn overlays done in a an
hour or so. The network at willow went down. But I got your email so my
guess is it is back up.

Aaron

On 2012/10/07, at 15:24, Ze'ev Klapow <notifications@github.com> wrote:

In src/robot_dashboard/widgets.py:

>      def _show_console(self):
> -        if not self._console:
> -            self._console = ConsoleWidget(self._proxymodel)
> +        if self._console is None:
> +            self._console = ConsoleWidget(self._proxymodel, True)

and it works!

—
Reply to this email directly or view it on
GitHub<https://github.com/zklapow/robot_dashboard/pull/9/files#r1781223>.
you'll have to view the RAW output. I did a git mv on the original file, then added this new one, and git got confused :(
I don't think so. glib take care of null pointer. I will check anyway.

On Thu, Oct 25, 2012 at 2:07 PM, markdryan <notifications@github.com> wrote:

> In src/device.c:
>
> > +		g_object_unref(feature);
> > +		item = g_list_next(item);
> > +	}
> > +
> > +	g_list_free(list);
> > +
> > +	device->feature_list = g_variant_ref_sink(g_variant_builder_end(&vb));
> > +
> > +#if MSU_LOG_LEVEL & MSU_LOG_LEVEL_DEBUG
> > +	str = g_variant_print(device->feature_list, FALSE);
> > +	MSU_LOG_DEBUG("%s = %s", MSU_INTERFACE_PROP_SV_FEATURE_LIST, str);
> > +	g_free(str);
> > +#endif
> > +
> > +on_exit:
> > +	g_error_free(error);
>
> Can you check to see if error is !NULL before calling g_error_free?
> Otherwise, I think we will get a warning in the logs.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/01org/media-service-upnp/pull/87/files#r1939783>.
>
>



-- 
Ludovic Ferrandis
Open Source Technology Center

---------------------------------------------------------------------
Intel Corporation SAS (French simplified joint stock company)
Registered headquarters: "Les Montalets"- 2, rue de Paris,
92196 Meudon Cedex, France
Registration Number:  302 456 199 R.C.S. NANTERRE
Capital: 4,572,000 Euros
please read > https://github.com/everplays/Archipel/commit/d1156c181376fccfeac008519a3d077c3c8948d0#-P0
может тогда вообще сокращенные варианты не подсвечивать? пусть пишут полностью?
I'd just use @$('input'), to be consistent.
How much hand holding do we really need to do?  This guards against someone purely using the defaults and not realizing that those won't work for drawShoot.  

But I can easily imagine drawing things with low S/N either as a continuum of objects with S/N ranging from something large down to something undetectable (< 1).  Or adding low S/N artifacts to existing images.  Or drawing many images of the same object where each has low S/N, but the stacked image is reasonable.  etc.  

So I'm pretty reticent about emitting warnings for anything other than flux exactly = 1, which is plausibly a naive mistake.  Any other value of the flux is most likely explicitly set, and I feel like we should let the user do that.  If it's a mistake, they will probably notice it when they look at the image and figure out what they did wrong.
OK, I'll add an issue now to discuss and do it if we decide that...
thanks, I've pushed a revert commit to this, so this series should be ok to merge now
Is this copied from somewhere, or does it require careful review?
This done inconsistently in e.g. HttpSensorAdapter, but i'd lean toward calling ``return registry.register(newAdapter)`` and always trying to register
Style is to use goog.dom.TagName.DIV (and etc) instead of strings
Why not use MongoLog::setCallback() ?
Can you please remove the .all (which I know you didn't add) because it's not needed
este código funciona ok. Mas por convenção, identificadores em caixa alta sinalizam constantes e devem ser evitados na declaração de parâmetros de funções e variáveis locais/globais.
what assertions do you need that are not provided already by minitest?

if there are any, can you just use `require 'minitest/unit'` ?
indenting is actually proper per PEP8
SUPERMINOR, NITPICKY THING: I would change the , in "on their production environment, a Raspberry" to a :

:smile:
More details! This a recv, we must be able to fetch a more detailed error string.
Is there a reason to keep this line around? This file is an upstream source, so any changes we make we have to track, but also if we're commenting out stuff - why not just remove it?
Why does manualcontrol have specific knowledge of pathDesired?  Should this (eventually) be an input event into the FSM sitting above path planner?
скобки не ставим вокруг условий в if
Oops. Thanks for the catch! I'll get that fixed as well. 

-Pete

On Apr 7, 2012, at 8:10 AM, mack-t wrote:

>>         // Important to recognize this async request is doing a get and the user / password is passed in the search url
>>         // TODO: should most likely just use the xhr object and doing basic authentication.
>>         Tomahawk.asyncRequest(search_url, function(xhr) {
>>             var dom_parser = new DOMParser();
>>             xmlDoc = dom_parser.parseFromString(xhr.responseText, "text/xml");
>> -            Tomahawk.log(xhr.responseText);
>> +            //Tomahawk.log(xhr.responseText);
> 
> If you think this should be removed, please delete rather than comment out.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/mack-t/tomahawk-resolvers/pull/1/files#r657866
They are from Funfactory, and come from the `from funfactory.settings_base import *` line at the top.
it looks pytest-mozwebqa will not run destructive tests on prod period.

    if (sensitive and destructive):
        first_match = matches[next(i for i, match in enumerate(matches) if match)]

        # skip the test with an appropriate message
        py.test.skip('This test is destructive and the target URL is ' \
                     'considered a sensitive environment. If this test is ' \
                     'not destructive, add the \'nondestructive\' marker to ' \
                     'it. Sensitive URL: %s' % first_match.string)

 group._addoption('--sensitiveurl',
                     action='store',
                     dest='sensitive_url',
                     default='mozilla\.(com|org)',
                     metavar='str',
                     help='regular expression for identifying sensitive urls. (default: %default)')
i personally would like to see a line break in there, if it can be done in a python-happy way
Are we now forced to create codes via the Django admin?
Эту логику возмо имеет смысл унести в реакцию на модификатор, например `_opened_yes`
Space before ":"
you don't need a \ between parethesis
Sorry, my mistake!
заюзай elseIf, щоб точно
Ok, that sounds a lot better.  It would be nice to run doc generation _after_ the symbols have been resolved, but I guess that's not really possible in dmd at the moment.
Make this: export RESULTS_DIR=.$(DSEP)test_results

And move it below the if wi32 block just below this line and I believe that'll make it work on win32.
I didn't use `IO.popen`, but I've narrowed it down to a single `fork` call.
this only needs to be forward declared
Be aware that Harut already made most of these in a patch to update apidoc that also got merged.  Since you didn't put platform as the last element for events you'll need to be careful when merging not to duplicate the platforms lines
Although this is more idiomatic than above, if there's an error assigning any of the attributes in a hash this way, the stack trace will only give you line 49, and you won't know which assignment caused the error. That's why I typically use the style above.

In this case it shouldn't be a big deal either way, but it's something to keep in mind when you might be debugging code that's tough to iterate on (gem code that has to be published, then updated in an app that has to be pushed, that then has to be packaged and deployed).
It's a weird to see whitespace at opening parenthesis, and not on close.. pick one ?
bouding > bounding
No, could wipe here, but createRowWriter will remove/ignore flag if rw_compress
Should this release the exception if it catches an IException?
I'm wrong, it can point to a const start. I'll add a curUnescaped, just like the curUnquoted.
What happens when there are more than 1024 files in a directory?
I'm not sure it's critical, but am wondering whether these checkLogicalName calls in constructor are too early.. 
So an auth. failure on an any (e.g. the last) action would fail 1st. before the transaction gets going..
that may be ok...
I'm not sure this is going to help much.  It doesn't take into account the likely level of duplication (which you don't know).  I still suspect a sensible default might be better @richardkchapman what do you think?  There should be a hint to override it for the first iteration.
Why subtract  -1 at all.  You could just use a < instead of <= comparison later (as it was in the original code)
Possibly rename to maxRow instead of lastRow.
Does it all break if `'404'` is an integer instead?
I've added this spec because current Rubinius version considers variables defined in Proc body as parameters. For example it returns ```[[:opt, :a]]``` and ```[[:block, :a]]``` evaluating ```Proc.new{ a=1 }.parameters``` in 1.8 and 1.9 modes respectively.
Death to useless parens!
for string testing, better to use ! and .eql?
I'll accept for now.
@christianv surprised that travis hasn't balked yet. JRUBY_OPTS="--client -Xcext.enabled=false"
@shacker can we fix the indentation?
@shacker remove the span, new class with 14px and remove the data-notreal
may be just 
bb[i] = (byte)i;

?
Seems a little excessive ;-) Perhaps a short ```time.sleep()``` between attempts to give the process a chance to get going.
yo dawg, I heard you like trailing commas
Yeah! thanks @corelanc0d3r !
can't we do just hash = p1 + p2?
This does not seem to correspond to an attached package that we install.  Not sure why the code is uninstalling it, but what is the reason for removing the line?
No license conflicts with this image?
`<Leader>f` is already CommandT.  Is there another mapping that would work for you?
Since PolylineCollection calls this "hidden" method, there's no reason it just can't access _collection directly.  There's no reason for this method to exist.
jsperf says it's very fast, almost twice as fast as checking a boolean property on my system in Chrome 22:
http://jsperf.com/instanceof-vs-property-access/4
I'm not opposed to a property (or maybe a method?) instead, though.  In fact, I considered doing that, but I couldn't think of another example other than clear where a bounding volume doesn't make sense.  I looked at overlays, but I think they need to be handled separately even with such a property.  So post-processing effects are the other example I was looking for.  So what's a general term to describe this property of a command: `existsInWorldSpace` maybe?  We can always add this later in any case, such as when we have post-processing effects.
> This might not be a big deal (and could be reserved for future cleanup) but we can probably cache the result of `getScreenSpaceCameraController()` instead of calling the function every time, right? Same for other calls to this function throughout this file.

`DynamicObjectView` apparently doesn't require that the scene be provided to the constructor, as long as it is set before update is called.  So Dan's change as a result of this comment broke two tests in `DynamicObjectViewSpec` that were not providing a scene.  I was just going to fix it myself, but it's not obvious to me what should change.  If scene is optional at construction time and then the user can set it later, then we can't cache the result of screenSpaceCameraController in the constructor or maybe even at all.
This should be `[[self backgroundImageLayer] setFrame:newBounds];`
Does not ensure that the proxy is in the children array
It turns out that `i` is fairly unhelpful here, since there is just one item, and it's at position `0`.

Also, each fetch from the param array probably has a little overhead (admittedly insignificant).  I think it makes the code more readable and probably more efficient if you just do `ImageArgs imageArgs = params[0]` and then use `imageArgs` everywhere that you are currently using `params[i]`.
Since these are constants, they should be UPPER_CASE style.
Add braces to if
OK, well, you don't NEED to change this `__init__` if you want to add a new mpi implementation, you define these constant wherever you like, it doesn't even have to be publically available, I can just declare it insdie the mpich2 or intelmpi modules if I want to, with or without the TC_CONSTANT prefix...

These constants are here for the string that is in them, not to diferentiate, that's what the family is for.
If you like I could also just define a mapping from family to type somewhere, but then we have to use some magic to not break modularity.

I prefered to group them in the `__init__` for now, since it's easyer to find them,
I can also define them in the `__init__` of toolchain...

The magic does not show up in the epydoc generated documentation, So there is no place to look for these TC_CONSTANT's
So please please document how and where you're using this magic...
add this to rosdep.yaml
That's correct. Fixed.
You could use `course( 1 )` from `word-problems` instead.
> . I think it's quite expected for it to implement the full Enumerable interface, and if I encapsulated the array I'd need to have a bunch of methods to calling through to the internal Array.

Nope, to implement` Enumerable`, just implement `#each`, and `include Enumerable`.
Shouldn't that work with `top` returning a reference?
OK, posted pull request.
https://github.com/D-Programming-Language/d-programming-language.org/pull/218
Just an extra note regarding front traversal's save behavior: Might surprise someone...
Ah, you posted while I was responding. Yeah. We should probably go with a shorter version.
Thought about _@Hibernate_ as well, but discarded it. It is so much more :-)
I don't think that auto completion is such a big issue though. One is an Annotation the other not.
How about meaningful names?
Excelent. Thank you
Space. (Same throughout.)
Correct.  It isn't.
How about: "**Re-design and enhance the existing `SageInterface` API**"?
this line also needs a wrap
What if dividers were simply another element in the list? It seems simpler to have a list structured 

    [a, b, c, DIVIDER('YESTERDAY'), d, e, f]

instead of the current

    [[a, false], [b, false], [c, ''YESTERDAY]], [d false], [e, false], [f, false]]
Where is the cmdbs variable coming from?
This should be `https://github.com/gruntjs/grunt/wiki/Frequently-Asked-Questions#wiki-faq-devel` to link to the correct anchor.
I don't believe this case is implemented, is it ?
Well, this *is* an API/ABI break, but I don't know how much we want to care.
Is there any valid reason to provide a correlationKey if relying upon TemporaryQueues? (now that we do not even use a MessageSelector at all for these?)
Good point - I will change the code accordingly so that BOTH Message-Header-Provided AND the Stored-Procedure-Name-Expression are only valid IF `allowDynamicStoredProcedureNames=true`
I actually think it is consistent since this is really derives form HTTP's content-type
misspelled: optimizeCorreltion
In this context, "list messages" is the same as "dequeue a number of messages". The API name is "ListMessages", so I think this is the best way to get the point across.
Worth linking to the pytest-mozwebqa project's README?
While testing Application server tomcat , the resource type datasource is not able insert the value. Datasource typeis  require only incase of tomcat and Provider action "setup_db_connection"  is passing only common parameter to "db_mysql_connect_app" and "db_postgres_connect_app".    We have set attirbute  "set_unless[:app_tomcat][:datasource_name]=" and getting the value of it in template context_xml.erb here.
Seems like a lot of duplication between here and admin_manages_escalation_policies.feature. Why does the feature need 3 those cases?
Just saw I munged this header too.  Please feel free to revert back!
I would move everything from this line down to line 38 out of this function. The projectProcessingBegin event can fire multiple times if any of the plugins make changes to code, and these values won't change any.
INSERT は一度に複数行レコードを追加できるはずです。
http://freestyle.nvo.jp/archives/260
Change console.log to log.debug
Shouldn't we be doing something with the attributes?
this is a matter of personal preference, but I'd like to keep each method tidied up with no blank spaces at the end.
move all programmatic constants to WebConstants
### THAT'S A LOT OF MEMS!
Yes, remains of debug, can be removed.
This comment on its own would not help me understand how the returned data looks like.
adding wrappers to activity feed calls in User and in Corpus as these are now saved outside the corpus dashboard, and can be saved in the user dashboard (where there are no active avtivity feeeds)
It'd be nice if you also put links to show in the browser for a quick comparison.
not sure here. you either declare it false or dont add the option, so i think it's ok as is
Use .one() instead of the on() of combination..
He was wondering if the original variable name was `$translationFiles`or `$translationFileList`.
If you can change to `$translationFileList`, it's gonna be better.
seems like the mock object hierarchy is pretty complicated - might make sense to create mock helper classes
If you use a less common path, like `y:\a.txt`, I think you'll find that this test is hitting the disk currently.
Can you remove enforce_ua entirely?
here

gevent.select.select(rlist, wlist, xlist)

might lead to a block in some conditions.
blog_controller ?

what's the point ?  to remove ?
There's a misspelling in the subject.

New translationS
Is this effectively the only size we ask for? If it is then we should use 100 as that is the largest size we normally try and display.
I agree with you on the valid use case argument. I more and more like what Steve is saying regarding the difference, between what's possible and what makes sense
So, it's good to know that IMAP CONDSTORE is superior to ActiveSync then!

The soft-deletion of messages is interesting!  I was wondering if they did something like that.  I see a few possible strategies for us:

1) Do things exactly like IMAP, which is to say, generally ignore the synckey stuff and leverage the fact that the GUID's should be stable, just like UIDs are stable on IMAP.  Use searches to find the GUIDs of messages in a time range, look up the headers from that same time range, and apply delta's to figure out who went away.  This might be more chatty than IMAP to detect changes in the meta-data.

2) Use sync like you say to restrict the number of messages we retrieve on a per-folder basis, and maintain an optimizable map that tells us what block the messages are probably in.  I think the general idea would be that for each folder we have a "target" number of synchronized messages in each folder, and the "actual" number we are currently synchronizing.  The target number would be what goes in the settings UI.

So let's say the target is 30, and we set our actual to 30 as well.  Then if the user hits "get more messages", we'll bump our actual up 15 and you hopefully do something clever to get us 15 more messages without us re-hearing about the first 30 we already know about too much.  As we hear about deleted messages from that range, we can decrement actual.  When we hear about new messages, we can increment actual, since the overhead of us hearing deltas for already-synchronized messages is pretty low.

Just like IMAP will have a process that periodically goes through and reaps old messages when storage is needed (or according to some other heuristic), we would go through and discard old messages from ActiveSync and decrement actual.

In terms of an "optimizable map", I think the pie-in-the-sky cool/awesome option is to use something like bloom filters to have a probabilistic indication of the likelihood of the guid being in a block.  HBase can do tricks like that with rows, for example.  Now, I don't think we want that distraction right now, so I think a reasonable, if wasteful, solution, is just to maintain a mapping from all guid's to the block they live in right now.  A slight optimization we could do would be to truncate the guid's to make them shorter, and that would also give us the probabilistic case, since we might end up mapping a single truncated guid to multiple blocks.


I think the second option sounds like a pretty good option.  The IMAP strategy was chosen because of particular IMAP weaknesses like lack of a LIMIT-esque option.  Since ActiveSync has a LIMIT option, that seems like the right strategy...
пробелы вокруг операторов
Fair enough.  There is actually a computeHorizontalLayoutPosition() which does something similar to computeVerticalLayoutPosition() for the horizontal position so I thought it had the same miscalculation.
This isn't correct since it will force the test to pass before the async callback runs.
You don't need to call callback.complete() when using this.async()..
Estoy de acuerdo con lo que decís de que el hecho de cómo recorrerla no hace al concepto de pila/cola.  Los nombres sí, son poco friendly.  Igualmente, no se me ocurren mejores alternativas =S.
why is this one required
yhnx
We can go for Java 1.6 support and thus add a *@Override* annotation.
That is a good question. Don't know. Should be removed.
Weird implementation, can you explain why it's needed?
I'll add them in tests for window namespace.
Seems to have an extra space
You're missing a test that checks the alarm stays on even if the tyre pressure returns to normal
protected $conditional_inverse = false;

Like that you can remove the else in the setCondition