labofsahil/github-event-dataset-2012 huggingface dataset:
Maybe we should have a quick conversation because this is behaving as I expect

<pre>
razor -d -w --bad_arg policy add
</pre>

This will raise an exception --bad_args and never call parse_slice!

<pre>
razor -d -w web_arg policy add
</pre>

web_arg is not a slice so will generate an error: Invalid Slice web_arg

<pre>
razor -d -w policy add
</pre>

invokes Project_Razor::Slice::Policy.new(['add'])

I've added debug in the old version:
<pre>
def call_razor_slice
  begin
    if is_slice?
      puts @args
      razor_module = Object.full_const_get(SLICE_PREFIX + @namespace.capitalize).new(@args)
</pre>

And this outputs:
<pre>
razor -d -w policy add
add

 [Policy] <-can't convert nil into String 
</pre>

So it should be also invoking which is really the same as the new version:
Object.full_const_get(SLICE_PREFIX + @namespace.capitalize).new(['add'])

Is this what you expect? Did I miss something?
missing a space on this line

/pedantic
`SCREEN_WIDTH / 2` with spaces
I'd bias to the tiny if I were you. For instance, if we default to 1/10,000 probability, than a key has to show up 10k times to be expected to survive.

If there are 10^10 rows initially, then you have 10^6 after sampling, and presumably, some of those are repeated. In any case, that is only about 1 million rows, so I think it is safe. We can tune later anyway.

Finally, I think ultimately we should move to CMS anyway, so I wouldn't sweat it now. In that case, we would do a groupAll to create the CMS object and just make sure that object's size is not too big.
``ports_in_use`` is already a ``set``
This error message should be different from the one above it, otherwise someone debugging the system won't be able to find out where it came from.
Move to hook_field_access().
There is already a function called KunenaUser::getUrl() in this class.
better message / auto retry added
this issue also exists in other parts of this patchset
The static analyzer did not complain, but you may be correct.  I think once the property is synthesized, direct assignment is aliased; but I will recheck the documentation. Thanks(!)
The purpose of setResourcePath is different with setResourceDirectory

The method setResourceDirectory can specify the relative  path for the resource root, It's enough under IOS，android;
However under windows I want set the absolute path too.
So that's why I add class CCFileUtilsWin32 to provide the extention interface to do the thing;
I fixed it. I amended my commit.
If we're going to specify a version at all, it should probably be '>= 1.0.17' since we're using Rails Admin. It requires Jquery 1.7 and jquery_rails 1.0.17 was the first version to include 1.7. :grin:
is this something that is beneficial for non __native_client__ builds?  If not, please ifdef this somehow.
that's just a guard assertion. I can show you same examples in symfony. That did the same
Again, I'm not sure why this is here instead of just usign `term`...what am I missing?
There isn't an emoji for :douglas-crockford-hates-you: but if there was, it'd be here.
I think this would be much more DRY if inside this callback you only work out the direction of swipe and then pass that to view.advanceGallery(); to avoid duplication.
Can we rename `cs-err` to `js-err` (or `js-cs-err` if the CS bit is needed)? I'm trying to use the convention of using a `js-` prefix for any identifiers which are purely JS hooks rather than CSS so there's no confusion trying to work out which classes/IDs are for styling and which are for interaction.
Repeated in line 157+?
good point - no need to leave test code lying around. Will fix and re-submit
I would wrap this one into promised, so that following could work as well:

```js
var pid = installer.install(path);
if (isFriday)
  installer.disable(pid);
```

So I don't have to wait for install to succeed to chain.
All these shortcuts are ok except this one. Its name is misleading.
What about `hasOwnProperty`, `hasOwn`, `hasProperty` ?
Also with suggested change, there will be no need to expose a namespace function.
I removed attributes that weren't used, and removed the use of getters.
It would make sense to use getters if we expose this. `tab` and `browser` never change, whereas `uri` and `content` can, that's why getter end up being important if usecode keep a reference to such tab object.
But as it isn't usefull for page-mod code, I removed them.
I'm not sure that such paragraph is relevant in documentation.
jQuery $ slipped in, instead of bid.DOM?
Thanks for adding the test @ozten!
Oh yeah, I didn't notice that the next query took the limit, too, so what I said above is currently not true. What do you think? Seems to me that's a pretty neat (and harmless) way to handle the problem of ghosts. Worst case I can think of is if somehow tons of ghosts get generated for the same connection ID, the next worker with that ID will have a big pile of objects to chew on - but that seems unlikely and probably ok. @jeads do you have a thought here?
Again, stuff to be overridden by subclasses is effectively public API (in Java/C++ it'd be "protected" as opposed to "private" or "public", but Python doesn't make that distinction explicitly); I wouldn't use a leading underscore for it.
I /think/ this should unconditionally create the "LoadedFile" object.  If we are reloading configuration files, we want to do the right thing if, eg, `~/.puppet/puppet.conf` is created while there is a running daemon.
Same with these - public API; shouldn't be removed.
Oh I see it copy over test/*root but not other py
Untainting is performed for security reasons. Blindly untainting removes these security protections.
hmm I think personally I would rather just ignore any kind of progress bar when canvas isnt available just so we dont clutter this reporter too much more for legacy browsers
I think, it would be good to clean the code a bit when you touch it already. E.g.
```
string logdir = String::Quote( Directory::logdir + "/internet-test/");
```
and replace
```
String::Quote (logdir)
```
with
```
logdir
```
then
```
string log_ip_addr = "ip_addr.log"
```
and modify rest of code using those variables (use log_ip_addr instead of "ip_addr.log" and so on)
It's not part of this PR
I will take care of all exceptions, never mind. This subcomponent is not ready yet :-)
Missing ScrollingStyle namespace
`dei concetti` -> `delle responsabilità`
а это где используется
Yes, it's convoluted (taken from man strtol) but the more succint version you suggest is clearly sufficient.
This must be reverted...
`alt="Brunch">`
I think we could probably use a stronger selector here; this will match any other link that happens to use the organisation name.
I was calling them AsciiDoc olists, ulists, etc., because they're not limited to Asciidoctor; they're actually elements from the AsciiDoc world.  I should have camel-cased 'AsciiDoc', though.
To  be honest, @tobyS , while I agree, I also want to ditch the custom UrlHandler, and use the one symfony provides, so that we don't duplicate routes...
`return status in (TransactionState.Running, TransactionState.Committing)`
Extra comma at the end is superfluous
log.rasieException()
This is about VSC-tools, NOT EasyBuild.
Hmm... Let me frown here... Conventional according to what conventions? I personally have  never seen _ in package names, except for my own code :)  And some archaic projects. how about wicketear or wicketEar? I've not seen the latter either though :)
Wrap at 80 columns.
It's just a note to remind ourselves when we do support RTL globally to make them float the opposite way.
Good catch on the localization - whats the reason for the `legend` -> `h2` change?
if we're doing this, why do we need the set_ircname method?
I don't think this needed to change at all except for creating users so that you didn't redirect to places you didn't expect.
Your indents have had too much to drink.
This method has missed the migration
Okay, thanks for pointing it out. I will revert it now.

Thanks,
Swarbhanu.
From: Dave Foster <notifications@github.com<mailto:notifications@github.com>>
Reply-To: ooici/coi-services <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Tuesday, October 23, 2012 10:38 AM
To: ooici/coi-services <coi-services@noreply.github.com<mailto:coi-services@noreply.github.com>>
Cc: Swarbhanu Chatterjee <SChatterjee@asascience.com<mailto:SChatterjee@asascience.com>>
Subject: Re: [coi-services] Removes transforma. Fixes on_quit() of a couple of transforms by calling the base class. (#321)


In ion/processes/data/simple_granule_logger.py:

> @@ -20,6 +20,7 @@ def on_start(self):
>          self.log       = self.CFG.get_safe('process.log', False)
>
>      def on_quit(self):
> +        super(SimpleGranuleLogger, self).on_quit()
>          self.subscriber.stop()


This line is called by on_quit already, so calling stop again is likely going to error out. Also, this class doesn't call on_start's base method. Luke was saying that some of these are intentionally not calling their baseclass, this might be one of them.

—
Reply to this email directly or view it on GitHub<https://github.com/ooici/coi-services/pull/321/files#r1916664>.
Not that I have any suggestions but the name doesn't feel very Nodey, `approximateSize()` seems a bit verbose even though it matches the LevelDB API.

Needs a space between `function` and the paren for the declaration, i.e. `function (start, end, callback) {`.
Really don't want to maintain hacks like this. The mustang guys need to fix this themselves.
I think this better move into the openstack compute resource itself, surely CompureResource.first wont always be openstack :)

something like

def avail_images
   client.list_images.body.....
end

probably better to extend fog to provide back images instead of going to fog requests directly.
When kinetic settings is overriden scroll directly.

Note that using the ensureVisible base class call introduces some flickering, I'll look into that, it probably happens when the viewport height is even.
Oops! I guess you're right.  I've changed it.
Maybe a better approach for this problem is to just run unit tests to see that all the variables are set with the correct values. I see you added a regex to squish whitespace, but I'd still be worried about it being a bit brittle.
not sure what the "right" way to format this is. maybe

    <p><code>
        \gcd(<var>A</var>, <var>B</var>) =
        <var>GCD</var>
    </code></p>
Caused by a "leap second" added Dec,31 or Jun,30. Actualy may be not so important in fact...
I'd rather keep the classes different. A child theme that would want to, say, float entry-meta to the left would need to add extra `footer.entry-meta{float:none}`to override the footer. Just because they are styled the same in thematic doesn't mean child themes also would style them the same.
In general I would agree with Isart. Dependencies on external libraries should be placed on the modules that need them. Otherwise you get into a mess of figuring out which code actually uses the dependency (a problem you will get with big modules too, so be careful with just how big you make them).

On the other hand I consider Guava a special case. It is such a generic library that you will end up using it everywhere anyway. So in this particular case I suggest you follow Pau's advice and make it a top-level dependency. Consider it a business decision to build our code on Guava (a wise decision if you ask me).
He, it's copy/paste from Property.php, but you're right.
зачем хранить isRoot? можно сразу определять.
this method:
- order/sort an array
- and check existence too
=> single responsibility principle

conceptinal there is no failing path in this either. i never seen a sorting algo failing. there are no exceptional cases on sorting at all i think.
Is this a reply?
And this is a `<span>` still..? Also watch your whitespace please
We should rename this to <code>test_breadcrumb_menu_in_extensions_page</code> for better readability and consistency among the other tests.
I've made a request for addition markup for the summary and I'll update the pull as soon as that is solved.
I suspect that the test is failing because right now on staging it's returning a 500; is that correct?  If so, this is still a valid test...
FWIW, _.keys is just an alias for Object.keys so we might as well use that one.
What do you think about splitting this out into a separate file?
This makes sense, but a comment about the created value, why we take off the | and what we construct (and why) would be helpful.
Some for these.
The role is there, it just never gets set without that change.

Sent from my iPhone

On 12 jul. 2012, at 08:51, Ryan Bigg
<reply@reply.github.com>
wrote:

>> @@ -0,0 +1,7 @@
>> +Spree::User.class_eval do
>> +  before_create :set_role
>> +
>> +  def set_role
>> +    self.role_ids = Spree::Role.find_by_name("user").id
>
> There is no user role I thought?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/spree/spree_reviews/pull/15/files#r1147340
we should use the constant provided by RouteObjectInterface, to make this explicit and avoid further errors like the one you fix here
Можно было бы и написать как вы хотели -- [_dna AddObject...], но тогда перед циклом этот объект тоже надо инициализировать, например: _dna1 = [[NSMutableArray alloc] init], одного объявления в head-файле не достаточно)
(Cell *) для hammingDistance можно опустить.
Так же хочу заметить, что если метод никоим образом не влияет на состояние объекта - лучше делать его методом класса.
Right now i'm making a new partial for things in <head> tags that are called often.
Jason already caught that and is working on it.
ah will patch it up.
I was thinking of that, but then the write_string... line would be too long and would have to be broken up, so it would not actually save space (and I don't like breaking up lines :) ) ... and the current way doees not need to repeat the 'working_dir' string. Do you want me to change it anyways?
I know that it might sound crazy but we always call epochs.average() evoked (and not avg) to be consistent and explicit about the fact that it is an instance of Evoked. Can you name it evoked please?
when trying request the documentation using http://localhost:8080/news/!! there was not result. Please modify the path correctly to only return the documentation when the previous link is passed.
Yup! From PEP-8:

```
single_trailing_underscore_: used by convention to avoid conflicts with Python keyword, e.g. Tkinter.Toplevel(master, class_='ClassName')
```
Sorry, I think I wasn't clear when I said to invert :)

I would rewrite this line this way:

```ruby
if (!confirmed? || pending_reconfirmation?) && !confirmation_period_expired?
```
Use `EnumMap`
Putting this into a single sentence might be better. A comment to translators helps too.
Observer?
"in event listeners".
should it be just except your web server?
You should remove the space between `types` and `<...>`
该地址可以是域名或者IP地址，端口是可选的
地址可以是域名或者IP地址，指定端口或者不指定端口

如果端口没有指定，就认为是使用80端口。
如果没有指定端口，就使用80端口。
This property access is pretty weird, but it looks like that's how sinon-chai works, I would definitely prefer a function call. Can you please double check, that this assertion actually fails if you break the code ?
Ок. Через час сделаю

2012/5/11 Georg Rudoy <
reply@reply.github.com
>

> > @@ -30,6 +30,7 @@
> >       : Resolver_ (new LocalFileResolver)
> >       , Collection_ (new LocalCollection)
> >       , PLManager_ (new PlaylistManager)
> > +     , PlayerTab_ (0)
>
> А, ну и пусть вообще без шареда иконка будет.
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/0xd34df00d/leechcraft/pull/114/files#r807840
>
Nice! I knew there had to be a better way. That's awesome array support. :-D

Just looked up shuffle. Basically it shuffles and since there are only two it would be them in 1 and 2. If there were 3 in the array, would it throw an error or just stuff @winner and @loser properly but ignore anything beyond index 0 and 1?
Do we need this require?
I'd suggest removing the getter and setter methods in favour of transparent, open access to the model. External users should feel free to access your internal state via a reference to the model. This is somewhat contrary to the typical way object orientation is taught, but I think it's an important way to enable open, extensible architectures. 

Since your accessors don't actually do anything other than return a value, you might as well save the cost of the function calls by removing them.
I don't think that you need to check in Gemfile.lock.
This mapping device needs to be configurable. and we may have to do a little more trickery here. We don't know what's the next device available on the guest. Also in cases for ovz where this is mounted on the host, that's not going to work for us... Sorry didn't point out this earlier, just wanted to get that one volume working I guess :)
What do this changes have to do with the PR at hand?
Should this parameter be a choice between 'xelatex', 'lualatex' and 'pdflatex' so that the rc parser will find errors early?
And another print here
A agreed with @WeatherGod.
> more ugly in many cases (because it cannot be chained)

I agree completely with what you are saying:

```
data = numpy.arange(12).reshape((3, 4))
```

is better than

```
data = numpy.arange(12)
data.shape = (3, 4)
```

But in general, I find reading highly chained code very hard to read (```d3.js``` being the very obvious example).



In this case, despite the fact that it is *possible* to chain with ```reshape```, it hasn't actually been used. So whenever I see 

```
data = data.reshape(new_shape)
```

I would *tend* to write:

```
data = data.shape = new_shape
```

Cheers! :smiley:
You don't.  Whenever you want to display a code block, you use just double-colons and indent the code block.  You might also have to have a blank line as well.  The word "example" before the double-colons is not special and can be any word.  Sometimes, we even just use double colons on a line all by itself.
don't think this is necessary, it appears to be unused. It should be `root_element` if it is used.
Tests like these would be best as a test generator:

https://github.com/Sancus/amo-validator/blob/5bd11709d9a8e664b654c1870248bd9be06fcf59/tests/compat/test_gecko15.py#L28
I'm not going to do this because it is a bit of effort. It's more risky to change it than just to leave it as it is.
Я понимаю.
Под концом самого верхнего элемента я имел ввиду view.
Выразился непонятно, да.
Actually, this is not needed. Will remove.
In den Dateien selbst sind sie da, Github Diff fail?
The "Me" service and servlet covers much more than content searches. Could we call this class something more specific, such as "AccessScopedContentQueryHandler"?
white space fix:

for (int i=0; i<xxx; i++)

and for single line for statements brackets are not needed
You can probably guess what I'll say here ;) I know a lot of magic px values existed before in this stylesheet as well, but particularly for non-rounded numbers like 408px it would be nice to get some explanation on why its exactly that value, and not 400px, or 409px.
yep my fault, it doesn't need to be set as a fixed amount but I wanted it to be consistent with the preview actions panel. As far as I know we don't really need it to be set to "auto". I'll change it back to "auto" - visually it doesn't make a difference at the moment but if in the future someone adjusts padding/margins (because there is a lot of inherited styles) in the panel it will be unbalanced with the other side.
Thinking about future and implementing something for future aren't same.

Thinking about future is necessary to avoid total rewriting. Unfortunately, I have to rewrite large subsystems based on wrong architecture too. Usually their authors had implemented orignal task as simple as possible without any chance to scaling or growing and later other developers added dozens of new features using crazy hacks and tricks, so at one day rewriting from blank became easier than supporting existing code. Assuredly, it was wrong way. My approach is to think about possible change requests and features, but not to implement them prematurely. It's enough for me to be sure that possible changes will be applied easily, so I just pretend in my mind how I'll apply something without actually doing anything prematurely. In our case every possible continuation needs some minor changes.

Buying a car and software development are very different things. Imagine that you can upgrade your car by installing needed options one by one instead of purchasing all available options for every possible future necessity at first day. There are zillion cities on the planet and <i>zillion! / (zillion - 2)!</i> directions between them. Are you still want to buy a car with all available options? :)
Too many possible continuations and we don't know which will happen. From my practice most of prematurely implemented software “options” have never been used but someone initially had paid for them and others later paid for maintenance. Why not to ensure that you can install any option at any moment?
please make it private in this case to allow optimizing the container
I don't think so (according to: http://www.w3.org/TR/html401/interact/forms.html#h-17.5).
also, the differences when minified will look like this:

    c.push.apply(c, r)

    // vs
    c = c.concat(r)
I would think that this and other App.Properties calls should be working with the work that DavidL did.  Please verify with him if this should be working and whether it's a bug.
I know what the value of `key` is, but I disagree about it being "expected and intuitive". Quite the contrary. More like unexpected, undocumented, inconsistent and encouraging bad coding patterns.

```js
$.cookie('a', 'aaa', options); // returns undefined

$.cookie({a: 'aaa', b: 'bbb', options); // returns pairs object by reference
```

Aside from it making the return value inconsistent based on invocation signature, it is somewhat counter-intuitive. Why would it return the pairs object? I'd expect maybe a boolean return on whether the action was successful, or maybe the expiration date of the set cookies, or perhaps the newly formed cookie-syntax string. But not the pairs object.

All that just to save a literal assignment?

```js
var values = $.cookie({a: 'aaa', b: 'bbb'}, options);
// Instead of:

var values = {a: 'aaa', b: 'bbb'};
$.cookie(values, options);
```
I think so, again also you should made these changes in Scala IDE code
hmm good idear.
Great! I like the reference to the Javadoc'd util class. And the use of the since tag is good for readers to know.
Missed this one. Fixing now.
please fix CS. the indentation should use 4 spaces per level, not tabs. And curly braces are on their own lines only for class and methods, not for control structures.
This seems really round-about. Inject true was a little silly to begin with and the tap really drives it over the top. I think something like this would be clearer:

     missing_requirements = []
     KitchenCommand.all_requirements.each do |r|
       if !File.exists?(f)
         warn
         missing_requirements << f
      end
    end
    missing_requirements.empty?
could we do `src/lib/troopjs-*/src/**/*.js`
This is vulnerable to SQL injection. See http://guides.rubyonrails.org/active_record_querying.html#conditions for how to use an array or a hash -- it's never OK to use string interpolation when building SQL queries.
I ended up putting too much commas and it stopped working silently :)
Hm. Recommended (by Django documentation) approach is to put it into views or models. I don't really agree with that, but it is what is common. I would put it into views. So at the end of the views file you just register signals.
```
:param dict layout: mapping of count of displayed columns (string) to PanelState() objects for that count
```
Don't duplicate constants. Use existing one.
This will probably be removed, yes? :-)
Please rename `id` to `name`.
Load tags all together at the beginning of the file.
Check the case on the method call
We have a module exceptions. Define a proper exception into that module.
typo, "exposes"
Should use 

    throw log.unabledToGetKeyspace( ... )

?
ah, thats right.
same remark than tornado
That sounds great, I've never used celery to add elements to the cache! I'd look into this.
I'd agree about using call() here, but only because it looks nicer IMO. How is `arguments` "basically deprecated"? Are you referring to performance?
+1 starting the template with this.
`while (rc == -1 && errno == EINTR)` - if the pipe is closed and errno is already EINTR, you'll spin indefinitely.
I wonder if we need to decouple the proxies from CG lib or if this is desired, even in Doctrine, i think they are generating by hand the proxies?
The blind copying and pasting of code without comprehension is why OMZ Robby is in such a mess.
A usage / brief tutorial style example would be greatly helpful. Ie., a standalone file. I don't have Owen's book because I can't afford it right now ;), and I'm not really sure what I'm supposed to do with this. Is the intention of this class just to be a univariate / multivariate location and scale estimator based on optimizing the empirical likelihood? If that's the case, I might suggest some changes. Otherwise I'm going to keep posting some comments on just the code.

Are there any papers that cover this outside of the book? I don't need all the details.
raise NotImplementedError and open an issue ? as TODO
does this really predict? params includes estimates of the endogs that are on the RHS of the equations.
??
are there tests compared to sysfit
I think that would be fine.
I can do either.  Currently, it is sky+object variance, since I thought that was the normal thing.  I can easily switch it though.
Poor oceanographers... :unamused:
i.e. can I make a previous child a parent?
It's what is accepted by PS parameters...
Changed them all to assertEventuallyTrue.
If the name is ambiguous, a `ValueError` will be thrown that will not be caught here.

Recommend raising a specific exception class for that case (perhaps make it a subclass of `ValueError` for backward compatibility) and catch it here.
Ignore me, just saw your comment on line 53 as to why you do this. Wonder why this happens...
We had the same discussion in the crash storage discussion and it bothered me. 

The prefix is "json" here but it's not actually a "json" string. It's a dict or something like `collection.Mapping`.
All in order now, just pushed a commit fixing that.
I would call it ``view_directories`` instead to make it clearer
I think all "Prompts" on forms should have their own style class so we can change the globally if we want (like a header 2)?

Having styles called "bold" or "italic" defeat the purpose IMO.
没关系，互相学习:)
Looks good. I did a quick search and didn't find anything that assigned a "default route." This combined with the default root assignment up top should catch all cases that aren't matched. However, can you append a comment to remind people to ALWAYS keep this rule at the bottom of the list?
What if the user wants to run a syncTest?
I think that the error message could be somehow misleading, as you can't create any contains (even root) by passing empty string/`NULL`. Passing such thing doesn't indicate that root container is created as the user could pass empty string as first time when he uses the service. I think the proper message is to tell user that container name can not be `NULL` or empty.
If there's no `ordinal` property, let's no set the `ordinal` attribute.
Ah right... This change is not needed then...
I've removed this. It wasn't needed. It was a hang over from the mediaportal addon which I used as a starting point.
This test works for me (I removed the metric change to allow working with sparse matrices). Was there a reason behind this deletion?
Не очень понятен code-style: где-то ты описываешь один `var` на N-переменных (`var a, b, c`), тут — на каждую переменную.
Почему вообще `var`, а не `function scrollToItem()`?
Could you please add an analysis of the results and why the PA style of updates can lead to this behavior?
> OK. How much of a performance difference does it make?

for one fit / one target not much but for dict learning it was
critical and you fit many
![charlie-sheen-winning](https://f.cloud.github.com/assets/449558/28525/200e6ac2-4c47-11e2-9718-cb80ef580c59.png)

**this is so awesome!!!!**
We should add this to the other ensemble estimators NOW!
I think I just misunderstand your comments: May be I should write as

*callable : affinity can be passed in as a function that takes in a ... and outputs a ... ? *

I am not sure I have gotten your idea this time :)
> or maybe better intercepthidden and interceptvisible

intercept_hidden and intercept_visible
Honestly, I did this so long ago that I don't remember, but I don't think there was a particular reason for it, in a declaration it doesn't make any difference really, is just cosmetic. I'll leave the ap.
do you mean int ch = fgetc(stream) ?
@adamdruppe , at least stop using hard tab.
`ForeaachStatement::func` holds the actual function which enclosing it.
See here: https://github.com/D-Programming-Language/dmd/blob/master/src/statement.c#L1535
This function is most likely going to be needed by the following from Anvil:
```
util.js Line 41: driverSocket.write(Ti.createBuffer({value: JSON.stringify(data)}));
```
Copier le fichier des règles de ré-écriture
you can make this package private  with @VisibleForTesting
See [this comment](https://github.com/neocotic/oauth2-extensions/commit/4b8d217755f0d05d4ebb1409073bcfab5233697b#commitcomment-1101436).
ok, i think i could change that to 'It seems I'm not...'
I got rid of the for loop, just to avoid 2 sets of nested for loops inside a nested for loop. Since you're still in your original nested loop, this will hit all your values except for the borders.
maybe, but 1sec, is a bit arbitrary..
Why would this be important to an importer?
The existing code would fail on packed too, I suspect.
The syntax to get GNUC's version is horrendous in CMake. And most other compilers do understand those attributes (armcc certainly does - http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.dui0491c/CHDFIJAE.html).
Should we only remove the oldest one? If logfile_max_files changes, it would be handy to immediately have all old files removed.
Why is the `limit` parameter a string? It should be possible to pass an int.
Because it's part of "formatting". In the same vain that you don't decide what should be bold in the python code but you do that in HTML.
Suppose the view provides an ORM queryset to the template that it loops over in the template like this:
```
<table>
{% for each in entries %}
  <tr>
    <td>{{ each.name }}</td>
    <td>{{ each.date_of_birth }}</td>
  </tr>
{% endfor %}
</table>
```
Then if you do the date formatting in the views.py you'd have to unwind that whole thing into a list first and replace all datetime instances into strings. E.g.

```
entries = []
for each in ORM('select name, date_of_birth from table'):
    each.date_of_birth = each.date_of_birth.strftime('%B %d')
    entries.append(each)
```
Pretty clumsy. And it would make it impossible to only use generators. 

Suppose also that the formatting depends on other things. For example you might have something like this in the template:
```
{% if request.user.is_authenticated() and request.user.get_profile().country == 'US' %}
{{ each.date_of_birth|date:"%m/%d/%Y" %}
{% else %}
{{ each.date_of_birth|date:"%d/%m/%Y" %}
{% endif %}
```
Then, that might only be available for such logic in the template. 

So, it's merely a matter of purity that you wait with presentation as long as you possibly can. The closer to the data the more solid and plain it should be. If it's super performance you want, the fastest way would be to do it in SQL as part of the select statement. 
Also, it's a matter of best practice based on shear experience that is hard to put a finger on. It's more of a gut feeling of what works best.
What do you think about naming this option just "Bean"? None of the collected metrics is actually a *counter* ...
Why `ServiceManagmentService` is not using mixed approach between connection strings and explicit credentials like `ServiceBusService`, `QueuService`, `BlobService ...?
The is-a-valid-ID check should be part of the UserService. It should just delegate to UserDao.get(userid).
`(<= 2 (count (:authors book)))` returns a boolean by it self.
@branan can you be sure to update the provider so that privileges are case insensitive
Also, the current implementation for adjusting sigma is broken: if `param` is set by the user, `sigma` never gets set.
Why did you remove this? This function is deprecated but it still exists.
Can this be moved in to a method that explains what it does?  Its a bit much to grep quickly.
`camera.frustum` is an `OrthographicFrustum`. `frustumCV` is a `PerspectiveFrustum`.
typo at the end: setColum
This can be simplified, since you only have one type of scope, "course" you just need id, not kind.

current_user.find_scoped_classes(id) raises an exception... you could handle that more gracefully with something like this:

> course =  current_user.courses.where(:id => id).first

And then if `course` is nil, add a message to flash[:alert] that says they don't have access to that course (or something like that) before redirecting.
Nit pick away! Where were you when I was writing these darned things in the first place? ;-)
Yeah. Technically we should say "encoding numbers and strings to buffers and decoding numbers and strings from buffers."

But that just bites. So I'm going in for a rewrite here ;-).
No need for value variable.
Comments above, not on the same line.
Yes, that is mostly the same as the comment above. The duplicate configuration is just there so I could have some test code for deleting the cache. It will be just one settings file when I am done.
should be in the query itself,
check metasearch
使うプラグインだけを動的に読みこむようにリファクタしました
As I already wrote it ( *just write `{ f(); }` instead of `destruct(f());`* ) I still do not understand what do you want.
Probably, you misunderstand the situation.

If you need a statement, you do this:
```D
// statements
destruct(owner.relinquishOwnership(key)); // your variant

{ owner.relinquishOwnership(key); } // variant without `destruct`
// statements
```

If you need an expression, you do this:
```D
// your variant
(/* expressions */ , destruct(owner.relinquishOwnership(key)), /* expressions */ ) ;

// variant without `destruct`
(/* expressions */, { owner.relinquishOwnership(key); }(), /* expressions */ ) ;
```
It would be better to declare the function with the ''function'' keyword rather than setting an anonymous function inside a variable.
yup, that would work too
Remove this line ;-)
shouldn't this be marked for removal then?
nit: I don't like it much when you rely on a function definition that comes later...
You mentioned that timestamp comparing helps for large projects... This for loop won't delay things up in large projects?
Cool. Have added the settings variable and set the default upper limit to 100 (I'm not sure if this is maybe too conservative?). 

Some thoughts:

1.) It might be nice to be able to set a custom limit in the model itself. It feels like this is not likely to be a one size fits all number. For example, if you're just looking for a name/value lookup list (e.g.: country), you might be happy for a high limit. But if you're dealing with an object with lots of data (e.g.: a list of forum posts), you might want the limit to be low. It might be nice to be able to overwrite the setting variable on a more granular level (of course, one _could_ just overwrite the list method which would be fine). What do you think? 

Finally, gonna push back on the offset/limit: While that maps to the SQL better, the page/items_per_page maps better to Django's Paginator class. Depending on the level of abstraction we're looking for: is it better to map to the underlying SQL, or the underlying Django abstraction? I'm happy to use offset/limit if you prefer - just wanted to throw that out there for consideration. Also, in your opinion, if we do go with offset/limit, is it better to simply slice the QuerySet, or should we still use Django's paginator?
This is the only instance of DateTime.Now usage I could find in this project. You may want to do a pass yourself to be sure.
That's @jcookems functional tests, he maintain them
Ты предлагаешь засунуть -1 в кот?
Тогда лучше назвать его типа `askUserKey()`
Не очень понятно по названию, что это такое. Лучше переименуй в `hookMouseReleasedAfterTabMoved`. И почему здесь только один индекс, а не два — откуда и куда?
Yes, regressions is possible here, but I hope everything will be fine.

I reworked serialization a little, and checked various scenarios on unit-tests.

```
private string SerializeRequestMessage(Dictionary<string, object> messageParams)
{
    var serializedParams = messageParams.Where(param => param.Value != null).Select(param =>
    {
        string serializedValue = _serializer.Serialize(param.Value);
        return string.Format("\"{0}\": {1}", param.Key, serializedValue);
    });

    return string.Format("{{{0}}}", string.Join(", ", serializedParams));
}
```
Another face-palm. Turns out my issue was puppet version. The puppet version installed on my workstation is 2.7.6. I use envpuppet for all of my development/testing, rspec doesn't so I didn't catch this. 

Since 2.7.6 is the basis of PE 2.0.0, I still want commit 7c0d10b (thanks blkperl!) and will work towards getting this module in a CI testing environment so we can automatically test against all the puppet versions Puppet Labs cares about without putting that burden on you.
Incorporated
It's good that that's true.  At some point in the past, I recall delegate literals allocating a stack copy regardless of whether they were passed to scope parameters.
same for rssChannel
We have no control over what code is in TX synchronization so I believe what we aged on is to document it . . . together with what can't be done based on POP3 vs IMAP etc. 
Or were you saying we should ignore those attribute all together if TX synchronization is defined?
Actually that is a good point.
Let me restructure the code a bit so the method we would recommend to override would only have to do this:
````
List<Message<?>> messages = jdbcTemplate.query(getQuery(POLL_FROM_GROUP), new Object[] { key, region }, mapper);
Assert.isTrue(messages.size() == 0 || messages.size() == 1);
return messages.get(0);
```
This way all we'd be asking to do to write your own method that return Message and we'll take care of he rest of the things
Agree?
Amol, you need to consult with @ghillert about Enum's setters: now he has new vision for 'ignore-case' in the config case.
See our discussion here #559
We need to change the test for edit ‘AlbumControllerTest’ in module/Album/test/AlbumTest/Controller :
<?php
...
public function testAddActionCanBeAccessed()
{
...
} 
This test is testing Edit Action in this content and Add action is wrong way/
@weierophinney Thank you for the clarification :)
Actually, better would be to do ``$form->bind($album)``, as this will bind the input filter as well, and ensure we get the data bound to the model.
We are looking into different ways to specify representations. I don't like using an api_formats configuration option, so I think this should be removed.
This  should probably be @applicant.has_all_teachers?  and move this method into app/models/applicants.rb so that it knows what has_all_teachers means rather than the controller, I was just in a hurry.
I'd think we have a better URL for OMERO.server than the trac.
I would try to merge this into Python (or link to it).
Please use double brackets and quote variables: "${identities}". Maintain the coding style.

Looking at ssh-agent and gpg-agent, I have not cleaned them up as much as I would have liked. It's not a good idea to use full paths to binaries since it ignores user installed ones. I or you should change `/usr/bin/ssh-add` to just `ssh-add` and do the same to the rest of the commands.
This is not needed -- it exists in BaseOpenmrsObject and is inherited.  Likewise remove the setter and getter.
You should not post to the API. Instead this should be directing to `UsersController`
We are not doing that for other elements like orientation, icon, etc so also here it would be case sensitive.
如果临时文件不可避免，那么干脆就别避免，全都用临时文件。textlib.php完全不修改。但要使用$file->copy_content_to($pathname)来创建临时文件
the usual wonkiness with any of the nova clients...... You need a degree in "Rocket Science" to figure it out :)
te quedaron unos espacios al final dps del paréntesis puede ser?
It is used only in sc*.yaml files. Since we are not including them, I use this statement.

I'll add a comment, too.
> Is this intended for human consumption?

Currently we only use this to output single line curl commands.  As such newlines and indentation are not needed.

>

Yes.  Addressed in 718ceb75a9d0f17d9bfbad856422ad48cd910f2f.
do you mean I could do the query @ forms.py for the orderedgroups of the question together inside -> handle_next_questiongroup_form() along with the orderedgroups of questiongroups(through questiongroup_order)

and then just pass the object around the make_question_group_form()

but I'm still in doubt of the advantage of passing objects around :D
I'm not sure whether the use of table here is correct. What's others opinion on this? @fabi1cazenave what do you think?
all of these comment out lines should be removed.
would be cleaner if we just add the synchronized keyword to the method definition?
This looks really wrong. What's the test case for it?
FAIL here
Hi. I will check, but I tested this on sonar - and I think it is not possible currently use emma with sonar plugin. 
When I enabled sonar in this project: Sonar just call mvn emma:emma but because Android project are quite different from standard emma project - I got errors. Because Test project are in different project and project for test is in a different project. When sonar call "mvn emma:emma" command emma plugin all emma proces i.e: instrument code, generate meta-data file and run tests and try generate ec file- but because this is not standard project emma plugin doesn't find test sources in ${project}/src/test/ 
I should test this again. But maybe my sonar configuration is wrong :( Could you check or provide better configuration for sonar ?
Connection so needs to be rewritten to work more like ReplicaSetConnection. A task for a future commit...
override. Fixed.
We should also warn that on iOS, we still have the Kroll issue. That is, for a view foo, doing "foo.margin.top = 5" is undefined. It may work, it may do nothing, or it may take effect at any random time later on depending on implementation.
Since we already have `hasListeners`, this method name is a little confusing. It would be better to rename to something more explicit like `eventWillPropagate` or `hierarchyHasListener` -- something along those lines :)
Nothing has changed other than removal of this block of code. We shouldn't be unregistering the account watcher on low memory.
It's cake!  You can't argue with cake.

I dunno...these exist as part of the callback-android repo.  And the windows generation script covers them, because it has to cover all symlinks to work properly on Windows.
this should only be done when serializing to XML. Otherwise, you will have an exception when serializing to JSON because of a missing XML encoder if it is not registered
This should be mocked and the file removed.
Goes back to issue #60. Good enough for now
use webworks namespace
Pull Request is in for the web platform
You can just call window.webworks.event directly that is all blackberry.event does anyways.
@nukulb @erikj54 That is the purpose of calling onReadyStateChanged() immediately
Tuo on oikein kaunis tapa tehdä tuo. Tehtävänantoa on sittemmin muutettu, map funktiota kun ei tarvitse vielä osata.
`compensated`