labofsahil/github-event-dataset-2012 huggingface dataset:
@alony идея у нас была, но зачем так?
можно же один метод для всех ролей
"from a trusted public key before executing a search."
looks like I got some white space commits in here. Opps..
Where are we checking that the right callbacks are fired? These specs could all pass and the wrong callbacks have fired for the docs (e.g. destroy + create instead of update).
As per comment description of bind, a more proper signature could be: `-(instancetype)bind:(id<RACStream>(^)(id value))block`
sb-ext:quit does support exit codes, it's :unix-status
be consistent with naming CamelCase or underscore_separated_names
As an alternative to this function, I suggest we add the attributes "lh_vertno", "rh_vertno" and "rh_data", "lh_data", which will make it easy for the user to access the data and, e.g., pass it to PySurfer. WDYT?
Note that the old way would have bombed if this hadn't been the case (also does a len check, and np.nan is also float).
It just occurred to me that we might want to specify why this is bad, if it is easy to give a simple reason. Are the decompositions better in newer versions or something? They looked darn near identical on my Ubuntu / 0.11 and OSX / 0.12 versions, at least.
@mluessi @Eric89GXL I don't like our time_to_index and index_to_time as it ignores first_samp and it has not docstrings (shame on me...)

what do you think of `time_to_index([t0, t1], use_first_samp=False)` and `index_to_time([i0, i1], use_first_samp=False)` ?

basically I want to be able to extract a time interval of data from the timings visible in mne_browse_raw

thoughts?
Why would this be better than having a function wrapper? Seems like it would achieve the same effect (since we could equivalently change the decorator behavior and the function wrapper behavior), but it seems less clear to me.

I'm also not sure how it would address the issues we've mentioned above...
# select channel 97...
nit-pick: you probably want to use a tuple for this, not a list, since we don't need it to be mutable.
Is this one still used?
Decided to leave this as is, to keep these lines as similar as possible to the original rails uniqueness validator.
What about:

```ruby
def fallbacks(locale = self.locale)
  return default_fallbacks if read_fallbacks.blank?
  if read_fallbacks.is_a?(Hash)
    Array(read_fallbacks.detect { |key, value| locale.to_s == key.to_s }) || default_fallbacks
  else
    Array(read_fallbacks)
  end
end

def default_fallbacks
  i18n_fallbacks? ? I18n.fallbacks[locale] : [locale.to_sym]
end
```

I'd still love to get rid of the .is_a? Hash part too somehow.
I did something in that spirit, hope it's ok
This need a ``.. versionadded:: 2.2`` note
This is not thread safe!
It was something like that before it was changed, besides one can reuse the field parsing in different versions of the format. Another thing is that one has to rewrite the field parsing even between the messages which is a bad idea because the field formatting is the same in all messages. Repeating it will only increase the probability of an error and will make the code more difficult to maintain.
Add //TODO docs :-)
yes, i moved the security checks to the respective controllers. 
This security check is now done at function Admin_Controller_Admin::create
Bob -> MoCo ?
這個是問過他的？
I see two options:
* Rewrite the function so it does setVisible(true) on the list given and add a setVisibleColumnsMask function that does the mask thing
* throw an Exception when both methods are used.
cool function
abarth asked in the first review what we should do about suspension.  This patch introduces the action that we abandon() the Prerenders when we are suspended, and re-add them when we resume().
remove `it` from the comment.
How would that help here?

Sent from my digital shackles. 

On Jun 12, 2012, at 11:27 AM, Ed Cranford<reply@reply.github.com> wrote:

>> @@ -44,6 +44,15 @@
>> LOG = logging.getLogger('reddwarf.common.wsgi')
>> 
>> 
>> +CUSTOM_METADATA = {'instance': {'status':'', 'hostname':'', 'id':'', 'name':'',
>> +                               'created':'', 'updated':''},
>> +                   'volume': {'size':'', 'used':''},
>> +                   'flavor': {'id':''},
>> +                   'link': {'href':'', 'rel': ''},
>> +                   'database': {'name':''},
>> +                   'user': {'name':''}}
>> +
>> +
> 
> NGHUTPR but if you like you might try out using the tree() construction I showed off earlier
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hub-cap/reddwarf_lite/pull/122/files#r968282
Should this and the next test still be disabled?

If so, wouldn't it better to solve the dependencies first and then fix this, enabling the tests at the same time?
I don't even… I think that may have been down to a cray Xcode undo. It got into a seriously weird state yesterday.
You put in an extra blank line here. None of the ones above have one.
You've fixed this so can remove the line.
I noted before in another PR (I forget the number) that I'm not a fan of four-line list comprehensions. In this case it appears to be a pretty beefy list comprehension. Perhaps:

```python
hspace_strip = hspaces[i * (cols + 1) + 1:(i + 1) * (cols + 1) - 1]])
hspace = max([sum(s) for i in range(rows)
              for s in hspace_strip])
```
`An error occured`
The existing test(s) already cover this (as a failure). Breaking this out only gives some additional information about what the particular problem is/was.
why do all your classes start with  ETM? btw, this is also an antipattern; I just can't recall it's name :-)
@kukulich: I was thinking about that phrasing, and I agree it's unclear.  It is the mixing of tabs and spaces that causes problems; I have updated the original document with language to that effect.
please maintain consistent indentation
2 spaces per indent, no tabs
Can we use the `s3d-bold` class in a span here?
Spaces around the arrow calling a method on an object is wrong, better to be consistent with the rest of the codebase, which doesn't put a space there.

We need to be gatekeepers on at least *some* of the coding conventions, otherwise we'll have an incredibly messy codebase. Although, there's minor differences per-person with things like `if()` and `if ()` I don't really care about. ;-)

To be honest, I'm not sure many people actually read the coding conventions page...
This seems fine to get started with.  Long term, it's probably worth making this a configuration parameter -- for 1TB files, 5% might be worthwhile...
``$fallback_dirs`` does not seem to conform to the naming convention (``^[a-z][a-zA-Z0-9]*$``). Please also check line 37.
Related to my comment on line 100, I'd like to keep style stuff out of the JS. Please remove the style attribute from the label.

Additionally, you can remove the span tags from within the label as they were only there as a selector for the line 160 (now 178) change below.  Now we can use the label as the selector.
@denniskeefe - There's probably no need for this fixture.  As we add tests, we'll just create data with Factories instead of fixtures.  https://github.com/thoughtbot/factory_girl_rails is probably the most popular if you're interested.
Ok, I've simplified it as you have suggested and added a comment explaining the "tricky" object.  I've committed them to my fork, but I'm not sure what I need to do (if anything) to make them part of this pull request.
Good question. IIRC, after the migrations related to making PageletsBase abstract, we had to manually manage the related sequences in 851200b4ca107554ddfa93fd9860d9e2bc7b7b24 and 97919b81a08b62179bb1ebf97bb2d53053c8d5cd. MySQL probably needs something like it too I suppose.
Can this be moved into doc/?
$model->findAll() timed-out at around 4,000 records or so. Using the CActiveRecord count() works better.
I would rather this one be a module function, no need to live in the class.
Done! :)

On Sat, Sep 22, 2012 at 4:58 PM, Jari Bakken <notifications@github.com>wrote:

> In puppet/modules/munin/manifests/master.pp:
>
> > +        package { $common_package: ensure => present }
> > +    }
> > +
> > +    define config(){
> > +      file {"/etc/munin/munin.conf":
> > +          #notice("Dette er i munin.conf tempate definisjonen"),
> > +          ensure => present,
> > +          content => template("munin/munin.conf")
> > +      }
> > +      file {"/etc/apache2/sites-available/munin":
> > +          ensure => present,
> > +          content => template("munin/apache.conf")
> > +      }
> > +      file {"/etc/apache2/sites-enabled/munin":
> > +          ensure => "../sites-available/munin",
> > +          require => File['/etc/apache2/sites-available/munin'],
>
> Framfor å gjøre dette manuelt, se på apache::vhost-regelen (f.eks. i
> hdo/manifests/apache.pp).
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/holderdeord/hdo-puppet/pull/1/files#r1667709>.
>
>
Conveniently, `every?` returns true for empty collections.
I think this line should be removed too. See the comment in the pull request.
a.k.a. use a `for` loop. ;)
That is possible. It'd be nice to have an echo for both. Wonder if we could pass in a parameter for the create logger method for the <br /> or /n (defaulting to /n.
Consider making this a top-level class or renaming to `ReceiverBehavior` and marking it as a `static` nested class.

In the latter case, code would refer to the class as `TcpInput.ReceiverBehavior`.
Is this wrapping ok?
Empty line between docstring and code. Docstring should have `"""` on a dedicated line.
Hmm, I think it comes from the previous versions of the view, which were more significantly changed because the backend model was different. I've since changed the model, so I didn't have to change the view so much, but apparently didn't remember to inherit the functions that are now the same.

You're right, of course, and it's been fixed.
You have to define dependency in Python. From this we will be able to forbid using only OpenHours panel without the map panel.

Then, on the JavaScript side, you have to make some way in map panel for other panels to get callback when map is ready.
Yes. But the idea is that if user has already set some value, then we assume that she wants that value, even if it is different than Facebook's.

Probably the right solution here is to not make gender required and to have it assigned none value for default.
See, why `horoscope.HoroscopeBase.get_supported_languages()`. Why not simply `horoscope.get_supported_languages()`?
Check how you should reference static files (there are examples already, like our logo). Add `alt` (same as above). Add space before `/`.
I would turn this around, install signal only if it is used on PostgreSQL.
I guess http://github.com/avsm/ocaml-workflow is a good place to start from.  It has the basic shared memory transport (and of course TCP). Not functorised on Async yet.

Although there is still the puzzling performance bug that makes the shmem backend slower than the TCP one, due to request pipelining issues.  Need to plot it out to figure that out.
Aquí había un detalle, verán si ejecuto:

```sh
mkdir -p ./{var,bin,htdocs}
```
Lo que me hace **sh** es que me crea un directorio llamado `{var,bin,htdocs}`, a diferencia de lo que hace **bash** o **zh**(no estoy del todo seguro de zh pero me parece que también lo hace).
Ok I see now. Then it's fine as null. 

Sent from my iPhone

On Jul 16, 2012, at 8:50 AM, Nivesh Rajbhandari<reply@reply.github.com> wrote:

>> @@ -104,6 +114,34 @@ exports.Stage = Montage.create(Component, {
>>         }
>>     },
>> 
>> +    _needsDrawSelection: { value: false },
>> +
>> +    needsDrawSelection: {
>> +        get: function() {
>> +            return this._needsDrawSelection;
>> +        },
>> +        set: function(value) {
>> +            this._needsDrawSelection = value;
>> +            if(value) {
>> +                this.needsDraw = true;
>> +            }
>> +        }
>> +    },
>> +
>> +    _needsDrawTool: { value: null },
> 
> The needsDrawTool  is not just a flag; it will hold the mouse event that gets passed to the active tool if the tool needs to redraw. Thus, I set the default value to null. If you feel strongly about this, I will make the change, but I prefer to keep the default value as null.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Motorola-Mobility/ninja/pull/390/files#r1170374
Possibly. My reasoning went something like "I already have claimed venues.
They should be given a 0 since they currently don't have a value." But the
site isn't really live yet, so even though I have claimed venues locally,
we don't really have claimed venues yet.

On Aug 23, 2012, at 8:18 AM, Dan Crosta <notifications@github.com> wrote:

In meetups/logic.py:

> @@ -121,6 +121,8 @@ def sync_user(member_id, maximum_staleness=3600):
>          query['offset'] += 1
>
>      # Set defaults on any newly created venues
> +    mongo.db[Venue.collection].update({'capacity': {'$exists': False}},
> +        {'$set': {'capacity': 0}}, multi=True)

You probably want to remove this too, then?

—
Reply to this email directly or view it on
GitHub<https://github.com/NYCPython/wheretomeetup/pull/22/files#r1444195>.
Nice catch. Copy paste from MapResolver. Will fix in there as well.
This should just be:

```ruby
if options[:ip]
```

Explicit nil checks in Ruby are generally a smell. Same below.
Because a user is currently "all or nothing", a blank user object with only an ID will be useless to the developer, and there is no way to even create one with only an ID. This is planned in the AFNetworking refactoring.

I do, however, agree that this should be renamed to ownerId.
Maybe We can "throws IOException"  on method signature to force a catch on ArquillianPlugin and ContainerCommandCompleter and them print the error.
This will crash if the resource does not exist and it doesn't in the beginning.
$site => $group_node
Should be a ui_string.
yep, that looks good.
Agree.
Il faut les virer ou faire des tests, ce qui serait bien mieux ! ;-)
should probably be `css_file`
This diff is useless.  Almost the entirety of normalizeArguments has changed.  I have code comments in the function for review.
Le problème avec l'utilisation d'un environnement personnalisé c'est que le fichier `esup.properties` va être chargée lors de l'utilisation de la tâche `ant` mais pas lors d'un appel direct via `mvn`... 

J'avais déjà proposé quelque chose de similaire à Jasig (cf Jasig/uPortal#39).
Ah, good point will add (not done yet)
I realized that my second para implied that we might actually want to keep the 0 and 1 argument constructors.  But you'd just have to implement them with the modified version of the add function, rather than relying on the non-existant SBAdd add method.  As long as the SBAdd construction gets a list, it will work (even if that list only has 0 or 1 item, although I think if it has 0 items, there may be weirdness).
You mean add SI as a dependency to spring-amqp?
why times one?
that is a good one. Let me see.
I'm not sure if the MachineSet class pulls its weight, versus using Set<NodeMetadata>, with the guava utility methods in ImmutableSet etc.
Since there are more material types this code will fail if the search includes any of them.
.sel/.selenium
This was a just a fix for chocolatebox basket_quick that I thought I would import into oscar.  Basically if you go to a flatpage on oscar - http://sandbox.oscar.tangentlabs.co.uk/about/  the basket dropdown doesn't pull any information through (total price and line items) like it does for all other pages.  This line pulls the information through correctly for flatpages and other pages.  
Yeah sorry this isn't in a branch David - it will be next time.
`notes edit <name filter>` ?
jsonMap is a List., not a Map
One of the cases I'm most interested in with this approach is how errors are handled.  Throw some tests in with null parameters, latitudes with magnitude greater than 90, etc.  The edges are always interesting, and tests with bad data often inform error handling decisions.
This is needed for the dumystream only. I will make it private.

On 03. 10. 12, at 10:02, Mitar <notifications@github.com> wrote:

> In datastream/api.py:
> 
> > @@ -272,6 +317,18 @@ def get_data(self, metric_id, granularity, start, end=None, value_downsamplers=N
> >  
> >          return self.backend.get_data(metric_id, granularity, start, end, value_downsamplers, time_downsamplers)
> >  
> > +    def remove_data(self):
> > +        """
> > +        Removes datastream data from the database.
> > +        """
> > +        self.backend.remove_data()
> > +
> > +    def last_timestamp(self):
> O no no no. This soooo introduces race-conditions. This is bad API.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
We need a way to show a wifi scan is being performed. I agree it's still a temporary hack until I come up with a proper UI.
Well, even if the "Auth-Token" name is only one used in REST, I do assume that other future transports may have a "token header name", so it would be not optimal to tie it to a "rest specific" config class.
I wasn't when I wrote this. Pretty sure it's a global. It stays within the module though, so it doesn't affect code now. It'd be neater to add a var statement. The "pairsplit" a few lines below this has the same problem :(.
Remove login from the this step and the associated method. This step is `click_write_review` and the login should not be implied. 

You can move the login into the start of the test itself.
Yes but as you may select one of the two flags in xbmc, if the addon configure supports only one, it will not work.
Post Chef 11.0.0, the client and server versions won't always be the same, so we should start on the habit of saying "Chef Client 11.0.0" instead of just "Chef 11.0.0" too.
Thanks for the reminder! Indeed we should add a version. I think it should be 0.15, though (right?)
to add a constant to a diagonal without allocating a full dense identity matrix use:

```
a = np.zeros((5, 5))
a += np.eye(5)
# or better
a.flat[::6] += 1
```
No, just the `::` marker (to be moved to the previous line).
full stop (?)
Out of curiosity, what is the motivation for this change?
whoops just saw your reply, which also got somehow lost... meh!
perhaps a link here to http://en.wikipedia.org/wiki/Singular_value_decomposition#Truncated_SVD for example
My bad, I initially calculated the incorrect value and forgot to update the assertion label when I updated it. Fixed in a485fda6aa2b0e85a448b5277d170269670849cf
Should we escalate to SIGKILL if it doesn't die after a number of tries?
Please validate that these formats do match the formats on other platforms.
A quick typo fix : ``See the vendor documentation``
hmm, indeed. Then keep it this way to be consistent
There is a `var` already...
derp, makes sense, done.
This check can be merged with this one above.
Should suggest to enter alphanumeric, not non as that would imply @#$(*&#@$ stuff.
What about:

```ruby
def column_list_for(columns, header = @header)
  sql = columns.values_at(*header)

  # ...
```
instead of blank, "must exist", "noop?"
Why do we need to do this? I'm wary of putting classes on the body element.
especially for the public methods: can you please doc comment the return value?
They are not literals as you point out, but I would interpret them as constants (I'd use const if this was C++). By using capitals we make it clear that they should not be changed.
The "Tomcat" name doesn't appear. If there is a large pentest with a huge number of machines and services discovered is difficult to know these lines are just from Tomcat, don't you think?
Add to ```Author``` instead of here.
leading whitespaces
A warning (print_status) when running the module would be useful, to warn the user which he must delete the uploaded binaries.
Whats the non-localized way to get privilege level?
also trailing comma
Remove the post conditional here.
Why do we cache `webGLContextID`?  Is a call to `getContext` expensive?  Like the rendering loop, I only want to add complexity to the initialization code when the value is justified.  Is it?
It's actually an unbounded 24-hour period.  Rather than +-12 hours from "now" I thought about making it the current local day (start at 12:00am of the current day and stop at 12:00am of the next day).  Ultimately it probably doesn't matter since people will set them if they care about them.
I like to think of the current system as `_` means private or you better know what you're doing.

With your convention above, how would we do private functions?  Define them at the file level or not on the prototype and pass `this` to them?

I'm not sure about `_` meaning private for properties and internal for functions.  Why overload it when we can make it more obvious with more explicit conventions?
Okay, I ended up fixing `||` usage throughout DynamicScene, since there weren't many.  I also merged in master and fix new tests that were still reference `../Specs` instead of `Specs`.  Should be ready to merge.
missing a space between switch and parentheses
The default behavior is only false if the activity the tab group is opened from is NOT the root activity.  If it is, then the default behavior is actually "true".
Not exactly, Mobile Web has supported gradients since 2.0 at least (can't remember 1.8.2).
By Spec NULL for Document, DocFrag,DocType,Element,Entity,EntityReference,Notation
А если аргументов будет 50? Подумай как можно сделать нагляднее.
Please add the copyright block at the top of this file.
Indent these things one more space so that the "n" in "new" lines up with the "W" in "WebGLRenderingContext".
Add also: from __future__ import unicode_literals
Thats what allows the code to consider them unicode strings without the u prefix on Python 2
I wouldn't call them mistakes, they are mostly style changes (but this one)
I forgot it to remove it before [-x, fixed in 077281a8f13dc7f45a5a0fa62ac97002682ed92b
probably nit picking, but old doesn't feel right, maybe source or input stack?
I really did a real-life LOL when I've seen how long is this line :-) Anyway, consider this as an improvement:

<pre>
('0' * 168) + salt_hex + Digest::SHA1.hexdigest([salt_hex].pack("H*") + password) + ('0' * 1024)
</pre>
We should also set `connect = false` in the `Connection` constructor.
what's wrong with 1.4? easier to release this sucker now.
Any contribtions will be covered by the BSD license in LICENSE.txt, therefore no license information is needed here,
great suggestion @zacc  I will update the pull
I really don't agree. 

Certainly it's a normal move in this refactoring to consider going from inheriting from array to encapsulating an array, and I did consider it. The reason why I didn't do it is that SectionSetPresenter is not supposed to *have* a group of sections, it's supposed to *be* a group of sections. 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.

One reason why I think the object should itself be Enumerable is because I don't see the SectionSetPresenter as a value object. I think it's reasonable for the caller to work with it, and add or remove sections. Admittedly I don't add sections to the set inside the _content_page partial, but it's also allowed to pass the SectionSetPresenter into this partial, and so other engines might want to build up this set in more than one line of code. As soon as you find yourself writing code like:

```section_set << my_new_section```

Then it's clear you're using the SectionSetPresenter as an enumerable object.

There are plenty of other similar examples from rails. What about the has_many association objects? Those are fully enumerable. 

I also don't agree with @phiggins point that it's not a good idea to inherit from ruby basic types. Why ever not? Why else would we use a real OO language? I certainly agree that it *can* be a bad idea, for the very point that he makes. Exposing a large interface doesn't always make sense. I believe that in this case it does make sense, and I think that this class should be considered to be a collection of items, not a wrapper around one.

Oh, hey, sorry about using the wrong github account (just noticed). Obviously this is @craigambrose I'll change accounts now. :)
This semantics is a bit convoluted for the sake of a very special case: input ranges that have assignable elements. I wonder whether we should better just support forward ranges with fill. Then we'd use `save` to copy the range and fill it with the classic approach.
This big block of code could be replaced by:
skipWS(p);
enforce(!p.empty, bailOut());

However it is about 1 call to empty less efficient. That is the reason the code was not changed
btw, another style that works now:

    html = '<div class="foo">'
      '<span class="bar"> ... </span>'
      '</div>';

that lets you indent the code the way you want without causing whitespace in the string. Depending on the circumstances that can be a useful pattern too.
could you use the URI class here?
http://api.dartlang.org/docs/continuous/dart_uri/Uri.html
This is in the master-eap6 branch of the https://github.com/jboss-eap/quickstart  repository. Should it be removed?

Is his one just for the community?
NB: probably time to start on the docs so someone can do ``help(scc.get_github)``. I would assume the definition here is ``use global gh_manager to ...`` etc.
No, I think people needing to run multiple instances in development is the exception rather than the norm.  Most of the time you're just using a single instance.  I'd rather make the default match the usage case 90% of the time and have the rest of the people set the property.  But if this is integrated there should be a notification on the mailing list.
I remember pulling from master and changing this to the updated version.  Maybe I did something weird when I saved it.  I will fix this again.
Hi Aliaksei, what is your reason to make it an exception. By the way, I just joined the team last week, do u have time to grab a coffee together?
--Gong 

-----Original Message-----
From: Aliaksei Baturytski [mailto:reply@reply.github.com] 
Sent: Monday, March 19, 2012 9:18 AM
To: Albert Cheng
Subject: Re: [azure-sdk-for-net] Messaging, part II. (#13)

> @@ -32,7 +32,9 @@ internal static class Constants
>  
>          internal const string MessageDestination            = "{0}/messages/";
>          internal const string UnlockedMessagePath           = "{0}/messages/head?timeout={1}";
> +        internal const string UnlockedSubscriptionMessagePath   = "{0}/subscriptions/{1}/messages/head?timeout={2}";

It's either allowing exceptions or updating each line in the file when a new constant exceeds the allotted line. I prefer exceptions.

--- 
Reply to this email directly or view it on GitHub:
https://github.com/WindowsAzure/azure-sdk-for-net/pull/13/files#r574869
I'd love to. But IE8 no likey.
:facepunch: 
:ring:
:fist:
:two:
Auch hier eine Ausnahme; Es wird kaum ein User selber entscheiden können, dass er jetzt auch Proofreader sein will... Somit kann der User nicht aus dem Context genommen werden; Eher wird der Administrator im Context sein...
createProofreader(User $user){...}
actually this flag should be dropped as the scenario in which the rollback shouldn't be send remotely is already handled by the ReplicationInterceptor:
- visitPrepareCommand registers the places where the prepare was replicated into the 'remoteLocksAcrquired' field of LocalTransaction: ((LocalTxInvocationContext) ctx).remoteLocksAcquired(rpcManager.getTransport().getMembers());
- a commit is only sent if the remoteLocksAquired is not empty:  ReplicationInterceptor.visitRollbackCommand checks 'shouldInvokeRemoteTxCommand' which does the check.
Can we make this 'getSkinURL' to be more clear as to what the function does? Also add a space between `){`
Do you mean to do `!(widget_index % 2)` here also, like above? These have different meanings:

http://jsfiddle.net/3uV4Q/
Should be single quotes here and throughout this widget
The comment is no longer needed.
Unfortunately, no. `core.memory` cannot import any part of the `gc` package due to visibility issues.
The stack is probably not aligned at the point of this call.
Why `cast(void*)` is necessary? When `rhs.type == TypeInfo.Type.Const`, `cast(TypeInfo_Const)rhs` is always non-null. Is there any exception?
We still need to do correlation inside of 'receiveReplyMessageCorrelated' method. So if someone specified 'correlation-key' but uses temporaryReplyQueue it is valid
Hello, Oleg.
I'm going to answer for all your comments, so do not rush to continue.

This your note is logica & clear, but I did not invent the bicycle.
Just look into ```BeanFactoryChannelResolver``` and his JavaDoc on constructor ;-)
might be overkill, but here you could assert that the value retrieved ('sendingTaskExecutor') is not the same as the previous instance
you should not register this namespace from the composer.json file IMO as it will clutter the autoloader in all projects using the module (thus making the autoloading less efficient). You should add this namespace only in your phpunit bootstrap file. You can see an example in Monolog: https://github.com/Seldaek/monolog/blob/master/tests/bootstrap.php
Same here, this should live in DoctrineModule
As with the other PR, if you could update the readme
The backslash was there before my changes. I'm not sure if it's necessary. I vote to remove it if it doesn't break anything.
Ah - I had never seen that bundle in the stdlib.

I just checked, and it seems that set_variable_values2() was created as a workaround to a bug that existed at the time, but has since been fixed. You should use set_variable_values() now, which does not need the filename as an argument. In fact set_variable_values2() will shortly be removed from the stdlib.

If you could please make this change, and I will merge the pull request.
I was thinking about if this will explode when `object` is not an object, but it seems that `JSON.stringify` stringifies everything: https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/JSON/stringify
it seemed reasonable because the original issue was timing rather than threading, but the old code didn't cause this test to fail. i was pretty sure it would work and i just wanted to get the smoke tests working (and indeed it made the 401 errors go away). i suppose i should have it create different threads and make one that fails without the new code, etc.
I vote for request pattern for everything, but I don't think it's a big deal.  I'll add it to my todo list.
added a class so that hte user could see the pouchname as they create their corpus. while ideally the pouchname isnt important to the user, since its the easiest thing to show theim in a corpuses view
In general, the contents of this function are very hard to follow - it's doing a lot of what looks like validation and transformation. Split this up into sub functions with good names so you can tell what's actually happening here.
Hash does not respond to as_json:

```
1.9.3p125 :001 > Hash.new.respond_to? :as_json
 => false
@wireframe From the description of plugins: "Pass in an array of plugins. Can be either the gem require or the absolute path to a ruby file."

Basically you could pass 

```bash
--plugins plugin-gem, plugin2-gem, /usr/local/socialcast/plugins/plugin-3.rb
```

Looking at this again perhaps should allow them to pass in a glob as well:

```bash
--plugins plugin-gem, plugin2-gem, /usr/local/socialcast/plugins/plugin-3.rb, /usr/local/socialcast/plugins-ext/*.rb
```
в англійських тестах перевіряй англійські тексти. я прописав в тестовий конфіг англійський по дефолту - заапдейть локальний конфіг
los ID nunca se ponen. se agrega automaticamente.
Not exactly. It is still possible and here's how to do it: http://stackoverflow.com/a/1450154/90297
This is a case where Android uses the non-standard (for us) property accessors isLandscape and isPortrait. 
(I much prefer this style for accessing boolean properties in general, but they aren't standard for us, so we need to do some manual work to document them.)

iOS supports them too, but *doesn't* support the landscape and portrait properties.

Mobile web only supports the standard property accessors.

If iOS supported the properties, and Mobile Web supported the non-standard accessors, it would make life easier for users.

If this API wasn't already in the wild, I'd recommend renaming the non-standard accessors. But I think it's too late for that now.
@arthurevans Yeah, this is the absolute URL of the location in code where the proxy was created (necessary for doing things like relative pathing)
Actually field instances passed to this method are not actually bound to a model, e.g. their `contribute_to_class` method has not necessarily been called. You could just call it this way: `add_column('mycollection', 'myfield', ListField(default=[1,2,3]))`. However, looking at how [south](https://bitbucket.org/andrewgodwin/south/src/fbc8b18d76e7/south/db/generic.py#cl-646) handle this case, I think we're better of looking if the `name` property is `None`, call [`set_attribute_from_name`](https://github.com/django/django/blob/master/django/db/models/fields/__init__.py#L235-L240) if not and then retrieve `field.column` as you suggested.
> I need to set clientDownloadDelegate to the mock version when testing which is easier to do by overriding `init()` and setting it here. Maybe 


```
protected ClientDownloadDelegate makeClientDownloadDelegate() {
  return new ClientDownloadDelegate();
}

...

Mock… {

  @Override
  protected ClientDownloadDelegate makeClientDownloadDelegate() {
    …
  }
}
```
Having to do a select to check if the lock is or is not there is practically the same than checking the value of a column.
You can change this to:

    IntegrationNamespaceUtils.setValueIfAttributeDefined(builder, element, "host");

If you need to do more specific validation of the value you can probably do that best in the setter of the *SplunkServer* class
See for the other attributes below as well.
Why was this test disabled?
No, this got in as part of the testRefactor and the status we put in. To get the overflow to work properly we must remove this. We will work on the unfolding and that will go in as it's own entire change.
Is this really a `WellNodeMixin` when it's missing the most important part of a `WellNode` -- the relationship to a `Well`?  Not sure why this needs to be abstracted.
That's for D1, so no worries.