labofsahil/github-event-dataset-2012 huggingface dataset:
Fix for Google Chrome and other apps

This fixed Google chrome and other apps that require ICS build Version.
minor correction for stdint.h conversion

This fixes some silly regex mistakes which clobbered some function names.
you right, but the code is just not readable, putting it into event would help I guess
Yeah theoretically.

In MIPS assembly, `while (0 == 0)` would look like, `BEQ $zero, $zero`, taking advantage of the registers hard-wired to zero.

It's one of my inventions I taste for, but it's theoretically possible that doing `while (1)` gives the compiler more space to optimize.
Ah, thanks for the info. I'll update the docs there with these changes.
Phasing icon :) ?
I had it at the beginning first for that reason, but then since they are only used there I moved them down. Maybe I should put them inside that function.
Probably, excepted @clone2727 is rewriting the image decoder for ScummVM and asked me not to merge it.
thrown -- caught ;)
@bcoe :+1:
How quick! Thanks!!
@j-manu Doesn't this create the log directory if one does not exist?  I would think this is desired behavior.
While using your code, I had an issue with the way fread works in read-buffered environments.  $read += $block_size increments the amount of information you've read by what you were expecting to read.  fread however can return less than the requested quantity of bytes when its waiting for packets from the network.  Additionally, fread can return a hard FALSE if it fails or disconnects.  I recommend replacing 

$response .= fread($this->__sock, $block_size);
$read += $block_size;

with

$r = fread($this->__sock, $block_size);
if ($r !== false) {
 $read += strlen($r);
 $response .= $r;
} else {
 return null;
}
Why no Rakefile? `bundle gem gearup` from the parent directory can take care of some boilerplate.
btw I updated to 2.1.3 but it looks like it doesn't work on heroku anymore getting

 Stopping process with SIGTERM
Sorry, these lines should not be removed. My mistake.
Pushed refactoring. Now every key is mandatory (which is true), so there's no checks anymore.
This looks good. Any reason not to merge it to master? AFAIK Cyan's TOS is ASCII, so this should cause no issues.

If we're concerned about compatibility for some reason, I'll just merge it over to Gehn.
This reduces coverage on `Resque#enqueue`, no longer covering hooks on this method when queue is present. Currently it is just a wrapper to `enqueue_to`, which is tested, but you might want to consider adding tests without changing existing ones.
You are indeed correct! I will make this change.
Ok. I'll keep that in mind.
Just a quick remark: the official name for the language is OCaml (and not objective caml)
Ok, nuked locally.  It'll be in my next push.
hi jannis — thanks for the reply. sorry if it came across earlier as i was just throwing code over the wall and asking you to debug my issue for me, because that certainly wasn't my intent — it was more just confusion about the implementation and how i had anticipated it working. my (incorrect) assumption was that leading-slashed urls in the static files would be resolved relative to the root of the storage, which is probably out of context a reasonable assumption, but in context doesn't really work for the reasons you noted. (maybe this is a place where a more explicit error or at least an admonition in the documentation about the method in which urls are resolved would be helpful? if you have any thoughts on how to handle this, i'd love to contribute a patch to make it clearer to others in the future because this was a major stumbling point for me, obviously.)

the point about not being able to bind the mixin to `STATIC_URL` makes 100% sense after you brought it up and would be a bad implementation overall to contribute back to core in hindsight. sorry about the confusion there — i should of looked into the implementation details before i suggested a change.

thanks again for the comments — i've otherwise really been happy working with the new cached files mixin (implementation with s3 boto storage has been a breeze, other than this issue and the other one i patched), and i think they're a great aid to what has already been very helpful with the staticfiles app.
I think it would be best to leave it as it was, because the portal will be at the end deployed in the wlan slovenija network and maintained by it. Otherwise we would have to define what or who PiplMesh is (who is the copyright holder). Can you make a commit reverting this change and add it to the pull request?
This has been documented already, see https://github.com/nvie/gitflow/commit/49305ec93d8c1bdf897df8a8e9ccee92f0d3d23d#L0R129.
Grävde lite djupare i de hela och fick fram att arvsmetodiken kallas Single Table Inheritance.
Det enda man behöver göra för att få STI att fungera är att skapa ett `type:string` attribut i `User`-klassen.

Resterande fixar Rails m.h.a lite meta-programmerings-hax.

``` ruby
class User < ActiveRecord::Base
  #
  # @type Symbol
  # @return Boolean Is user what we think he/she/it is?
  # Example:
  #   current_user.role?(:assistant)
  #
  def role?(type)
    type.to_s.classify == self.type
  end
end
```

Eller lite meta-programmering i sin läskigaste form (@Tarrasch).

``` ruby
class User < ActiveRecord::Base
  @@types = ["assistant", "student", "examiner"]
  
  #
  # @return Boolean | Unknown
  #
  # Example:
  #  current_user.assistant?
  #  current_user.stupid?
  #
  def method_missing(method, args, &block)
    if method =~ /\?$/ and @@types.include?(method.to_s.downcase)
      method.to_s.gsub(/\?$/, "").classify == self.type
    else
      super
    end
  end
end
```

I båda fallen kan vi ärva direkt från `User`.

``` ruby
class Assistant < User 
  has_many :registered_courses
end
```

``` ruby
current_user = Assistant.create!(name: "John Doe")

# First example
current_user.role?(:assistant)

# Second example
current_user.assistant?
```
Jag tror att det enklaste är att skapa en ny branch, leka lite med arven och se vart vi landar.
The continue statement is removed in GM13
With this commit Mongoid::Referenced::Bindings::Referenced::One and  Mongoid::Referenced::Bindings::Referenced::Many are exact same class now. Herit from same Class and implement same method with same implementation.

Maybe can be mix in only one class ?
Il est sérieux là?
Yikes. Thanks Yeti, I definitely need to be sure to double-check all the little things before I commit.
Why do you remove the installation of python bindings here? (I'm assuming that's what's happening)
heh sed :)

On Mon, Jan 16, 2012 at 1:11 PM, Andrew Phillips
<reply@reply.github.com>
wrote:
> Thanks! So, sed or maven-release-plugin? ;-)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jclouds/jclouds/commit/3df052565fc4493cc0ac42e6b408ac3ab1dd73c3#commitcomment-872154
can delete this line
This looks like its going to be significantly slower than the bytewise operations while still producing the same result (unless eq is changed to do something other than bytewise equality). Am I missing something?
И это нормально? обрабатывать постоянно тысячами эроров и вы хотите что б не было лагов?)
Случайно не опечатка?
jenkins doesn't run migrations - it syncs a new db. fwiw, we are in fact getting errors on -dev (`DatabaseError: (1146, "Table 'appspreviewdev_allizom_org.addons_devicetypes' doesn't exist")`)
Device address is shifted to avoid contention with the read/write bit.
@jonlemmon hope I didn't mess up you git commit or branching strategy with this one. Didn't read the button well enough and thought this would just submit a pull request like I have done for other open source projects. Guess as I am admin here it committed right away. If it is a problem let me know and I can help fix things up.

Otherwise very small fix indeed. :)
If you use multiple_clouds, you need to use :cloudfile_present? hook, right ?
Looks like @archiloque is working on version 1.3.2, which definitely includes the patch!
Good idea! Especially since new users might get turned off if they hit the GradleFx 0.4.1 vs. Gradle 1.0 m8 incompatibility.
Did you publish it to MavenCentral too? If you did, you might want to edit the 'Basic setup' wiki page. It still says 0.4
WTF, esto se fue al pasto mal... =) Cero I18n, o menos que cero... a parte de otros problemillas...
no es el 1º  es el 2º, el 1º tiene impresiones en cola y no se puede volar...
So we don't work with 3.1?
*bows*
instead you should use a M2M relationship like [this](https://github.com/mozilla/zamboni/blob/master/apps/addons/models.py)
Good work. There are quite a few instances of this around: I saw one in the speech context partial where the instance variable is a misleading `@document`, but I haven't done anything about it yet.
I just did this for the hell of it.
Looking forward to it :D
Yes he is...and if you read the comments or the forum you would have known,that now we need to find a way how to make paths on gameobject i.e. toc floor,algalon floor,frozen throne floor OR transports
Gee, thanks. Sounds cool. I'll take a look at it.

 Guy

I understand quickly when things are explained to me slowly...



On Jan 27, 2012, at 16:29 , Oliver Drobnik wrote:

> I merged the parse blocks branch now back into master and "just for you" I implemented a method that you could override in a DTHTMLAttributedStringBuilder subclass: -(void)flushCurrentTagContent:(NSString *)tagContent
> 
> this gives you a possibility to run the hyphenation on tagContent and then call super. This flushing occurs every time a tag begins or ends and aggregates tag contents until either of these. Therefore this also works if you have HTML entities inside of words.
> 
> 0d2a8b463a5555b3a9dc8b6a9e0777ad4d58b4e6
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/gshaviv/DTCoreText/commit/6e8ae6cd34b67a2a72f906508b0cd7b59fb176fa#commitcomment-908230
I would have split this up into two commits: first to upgrade the template, second to move the js around.
what's 1?
Also I forgot to mention, I also fixed a couple things with the friendship detection and added a instance variable to be used in the show action so that we can delete the friend.
Yes! You got the ps3 build draft done now! Once this is finished, the whole framework will be perfect (except for the minor fixes you'll add) ;)
I'll plan to yank this file at some point in the future (either during a future rebase or before the official sourcebase commit)
I missed this commit.
Just a copule of question:
- in ping.php is removed notification of citations, why? there was problems with it?
- if I understand right, you're adding dynamic title input box like this http://kirgroup.com/photo/5264e54694c1174259706f67cfea744c-0.jpg using javascript...
  I've added the same to quattro theme (not merged at this time because i've done some mods to other parts of theme and I have to check old themes works right, I think you can see it coming to write a post to my wall :) ), but I've done it with html5 "placeholder" attribute (has exactly the same behaviour as in mockup) and a copule of css rules... I think is more clean and easy, and enough supported by current browsers (as here http://caniuse.com/#search=placeholder )
I'm merging this jot-header to mine... I should keep this code or go with placeholder method? what do you think?
Is there a clear saving in using this local variable? If not, I'd really like to have synchronized methods instead of synchronized blocks. When you have synchronized blocks, you are asking for trouble down the road. :)
I mean "spent way *too* long on an object-oriented version"!
Pero si tenés muchos títulos de nivel 1 creo que va cambiando, asi lo ví en algun documento que hizo JD
What's up with that? Accidental commit? If you want to get rid of it, just do so. We have version control, you know :)
Specs run against Dummy app so I figured I should stub it's config but I'm curious now what's the difference between Dummy::Application and Rails.application?
Checkout this branch and you'll understand why. Guest users can destroy their own comments....
Cool, thanks.

On Jan 8, 2012, at 2:22 PM, prnewman<reply@reply.github.com> wrote:

> Hi,
> 
> I just added a bug fix from the main branch to our fork, so you're ok.
> 
> Paul
> 
> 
> On Sun, Jan 8, 2012 at 1:14 PM, Eduardo Scoz <
> reply@reply.github.com
>> wrote:
> 
>> Hi @prnewman what was this bug? How can you reproduce it? I would like to
>> fix it in the main branch if it's an issue. Thanks!
>> 
>> ---
>> Reply to this email directly or view it on GitHub:
>> 
>> https://github.com/zipmark/QuickDialog/commit/793c124f181a6932299c37e32552d42e329f7b62#commitcomment-847812
>> 
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/zipmark/QuickDialog/commit/793c124f181a6932299c37e32552d42e329f7b62#commitcomment-847952
We dont need it.
The "AssistantUtteranceView" Object set the speakableText to normal Text, if none is set!
Hi!

I can't really test this, but I assume it works. I'm going to pull it in.

Thanks!
As a dependency injection.
Thanks for the feedback. I think there is something else going here as an additional factor. I run a private fork of CGI::Uploader against PostgreSQL 9.0 in production without problems. 

Could you produce a Test::More-style test for the test suite that fails when run against PostgreSQL 9.0?
FYI, the issue I was talking about before is back.  Whenever I run play clean, Eclipse starts freaking out and throwing up all kinds of alert boxes.  I can fix it by running play eclipsify and refreshing the eclipse project, but it's pretty annoying.
you should look at the book searching site bokfynd.nu, they aggregate from several other sites. Very good for finding good prices, should be great for finding book titles as well. :)
Hi there,

I just pushed [a commit](https://github.com/shawnbot/modestmaps-js/commit/ad5f693cdb56896045f5677aaa8418ab09d7c8ac) that fixes the issue in the anyscale example, and [another](https://github.com/shawnbot/modestmaps-js/commit/38cdc166fbd7010b060ef3f36bef9556e7613f72) that adds the "precise" argument to `MM.MouseHandler`. I've kept the [anyscale example](http://prag.ma/code/modestmaps-js/examples/anyscale/) code the same to make it clear that you can pick and choose handlers rather than always using `MM.MouseHandler`.

Cool?
Look ok otherwise

So getClusterMachineList(.., "THORSPARENODES", ) 
.. is called somewhere, when you attempt to SwapNode to present the list , is that correct?
lo de el primer .cpp esta bien?
Rod,
My "high level" thought is that this code could be much cleaner if you used jQuery methods.
use $.mouseout and $.mouseenter to bind the events and use $.post to ask the server for the data.
Не разбираешься в тернарных операторах?
Ok, moved.
IMPORTANT NOTES:

A few database modifications are needed for these changes to work smoothly.

1) in myphpadmin (or however else you make changes) open the users table.
2) click 'structure' and check 'roles', click 'change'.
3) change 'Type' to ENUM. change values to 'admin','staff','basic'
4) set default to 'as defined' and select 'basic' for the default value.
5) rename the columns 'name_last' and 'name_first' to 'last_name' and 'first_name'.

If you like, you can delete any or all users in the table, because for now we are allowing a not_logged_in user to add users by going to mhsbake/users/add.  Later we will have to close this back door (maybe put a note at the end of project plan?).

These same changes will have to be made at the remote site when we upload later tonight.

General notes:

Login and Logout should work a little more smoothly.  As of now there is no functional difference between a basic and staff user (we will build out this functionality incrementally).  Only admins can edit or delete an account other than their own (those buttons wont show up unless you're an admin).  I also cleaned up some experimental code that I had forgotten about. If you see anything weird it probably means I missed it so either fix it or let me know.  :)
Typo here, should be: `null === $infos_url`
all of the above `404`'d
Actually it is "My Documents" on English Windows, if I am not mistaken.
Better to use $6 in the error message - because $5 doesn't have an associated token, so the position isn't sometimes set up correctly.

Also, better error is probably "Not expecting COUNTER for DATASET"
(FWIW, this will be supported in Chrome 19. It works in Chrome Canary 19.0.1055.0.)
What about array('127.0.0.1') for the default rather than false?
Convention over configuration?
There was a 'head' version which should have contained this change.  I put out a 0.14 version that should as well.
This commit is missing a similar change in the "else" block. I'm fixing this by replacing the duplicated code...
Typo

DELETE FROM `script_texts` WHERE `entry` BETWEEN -1810018 AND  -181008;
why not spell summon with: ID - 34400 A-52 Neuralyzer ??
+1 incredible, packet stuff always is something pretty interesting to me
Alert! Calling this method will result in ClassCastException as getFromAssociatedElementsByType return a List and it's being cast to GRETunnelService.
Depending on GRETunnelServiceConfiguration multiplicity the fix differes.
If there is only one service per config, use getFirstFromAssociatedElementByType.
página do Google Code
Why is maxtotal limited to 4Gb?  Is that going to cause problems in the future?
I wound think it isnt needed.
OK, great to see the code broken down!

Perhaps we should also put the list of types in the constants module, as if Apple decides to include some other types of videos/audio/whatever, we are ready with those (well, it seems that they do that already with epub's).
No, because the next time to use $lazyload with a different container, that will override the value for previous instances.
Can I suggest changing the way the default charset is obtained? From:

            characterEncoding = System.getProperty("file.encoding", null);

to
            characterEncoding = Charset.defaultCharset().name();
It's not executable on my end.
If it goes here, proto will not be used.
Then there's something wrong in our upgrade script
A lot of people seem to use duedate. If you want to make this change, we should consider an automatic upgrade path. Code samples we will likely produce, otherwise, won't work for "old" install.s
I think the only problem is that "Begin" is not commented out by // so changing the first line to 
<!-- // Begin
That should be the correct fix.
oh joker, you found I forgot to disable it in my commit. I pulled request already
const int is useless, int is enough
if the method is const, then it should be:
 int ...(...) const
Good catch it's meant to be `Request.requests`
arch/arm/ SHA-1 implementation properly removed in commit c1d8c5e7b17dad919bdb710953ccf0e2962167d2.
@stof: strangely, in my dev repository, a 2.0 branch appears and not in my sismo repository... Using `version=origin/2.0` is indeed the way to go I think. Thanks!
Wrong name. Perhaps `uploadRequired`? (You're not uploading commands, you're uploading because you processed commands…)
Indeed. I just build for crespo4g, and incrementals have been fine for
a while now. Every once in a while, like right now for instance, I'll
make clobber and let a fresh build run overnight. CM FTW!

On Sat, Feb 18, 2012 at 18:28, Keyan Mobli
<reply@reply.github.com>
wrote:
> As long as a device's vendor has stabilized and you're only building for
> one target, incremental builds should be fine (besides the obvious
> build.prop issue)
> On Feb 18, 2012 4:57 PM, "Justin White" <
> reply@reply.github.com>
> wrote:
>
>> Good to know. I do incremental builds for myself because my little VM takes
>> quite a while to do a fresh build.
>> On Feb 18, 2012 5:55 PM, "Keyan Mobli" <
>> reply@reply.github.com>
>> wrote:
>>
>> > All nightlies are clobber builds meaning there is no build.prop there to
>> > begin with.
>> > On Feb 18, 2012 4:52 PM, "Justin White" <
>> > reply@reply.github.com>
>> > wrote:
>> >
>> > > This is very much what I had in mind with my Build Date addition to the
>> > > Device Info screen. Will be good to keep the time info out of
>> cm.version,
>> > > though.
>> > > Will this change cause the datestamp to be updated on every build, or
>> > only
>> > > when build.prop is first created? One issue with my mod is that if
>> you're
>> > > not doing a fresh build, you must remove
>> > > out/target/product/*/system/build.prop by hand so it will be recreated
>> > and
>> > > have a current datestamp.
>> > >
>> > > ---
>> > > Reply to this email directly or view it on GitHub:
>> > >
>> > >
>> >
>> https://github.com/CyanogenMod/android_vendor_cm/commit/bc1f988afaea186c5ad508fff550189d98698bbb#commitcomment-981759
>> > >
>> >
>> > ---
>> > Reply to this email directly or view it on GitHub:
>> >
>> >
>> https://github.com/CyanogenMod/android_vendor_cm/commit/bc1f988afaea186c5ad508fff550189d98698bbb#commitcomment-981762
>> >
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>>
>> https://github.com/CyanogenMod/android_vendor_cm/commit/bc1f988afaea186c5ad508fff550189d98698bbb#commitcomment-981764
>>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/CyanogenMod/android_vendor_cm/commit/bc1f988afaea186c5ad508fff550189d98698bbb#commitcomment-981795
Nit: align =, align //.
Nit: spaces around arithmetic operators.
Thanks for including it. Btw, the responseSetupDataCall(Parcel p) method is handled by RIL class properly for v3. build.prop has setting for that and this method could be removed.
If we are not using this, we should remove it.
Yes, i know, it’s a bit ugly. :P

Before all, i thank to find the door like a lambda user (with previous text)  but it was bit complicated, so i updated the text. It was ok but the solution wasn’t efficient for me, so i learned to use Tiled. :)
Is that a Typo?
The above is a good point. It is something that I've been pursuing as part of some other writing that I'm doing, so thank you for finding a good place for the idea.
Amazing song though.  I'll try and find a shorter version
Replace `BV` with `Bean Validation`.
On Wed, Mar 14, 2012 at 2:13 AM, Eric Stonfer <
reply@reply.github.com
> wrote:

> This should work for a mock but not in a real test.  You need to create a
> VPC and then create this security group.
>

You are correct, sorry that I missed it. I have added code which creates
and destroys the a vpc around these tests and created a new pull request.

    /MaF
Hi Barbara - in the case of other locales, the key would be displayed in its untranslated state (with ## hash marks around it).
```css
/*
sadfsf
*/
a {b:'}
```

Uncaught Error: Unmatched ' at Line 2, char 6

Same line mismatch happens with `''`, `""` and `()`.

I suggest don’t count line breaks until an error encountered. Once it does, go backwards and count all `\n` using `lastIndexOf`.
Is it necessary to define a custom event here?
The number of ~ need to be the same as the function name, otherwise the documentation parser doesn't recognize it as a function.
The canonical way to do non-trivial default arguments is to set the default to `false` and test if the arg `=== false`.  Not a big deal, but it will keep our code consistent.
I think Martin Krasser made it. not sure, but it was originally in the docs.
It's those settings:

    periodic-tasks-initial-delay
    gossip-interval
    heartbeat-interval
    leader-actions-interval
    unreachable-nodes-reaper-interval

For tests we need a simple way to disable all, or one, of these. Disable all is no problem, but disabling one was not possible, because it was run "once" due to the initial-delay.

Also, I think it's rather logical to not run the first task earlier than the specific interval.
No, I meant, couldn't you use straight camel to test Akka Camel or somesuch?
rather not, because there would be confusing name clashes, for example `def join` exists in both places
onRouteDefinition closes over "this" in: (rd) ⇒ getRouteDefinitionHandler(rd) and now you're exposing "this" to another Actor.
Is very common see paginations displayed as lists, so we need this flexibility, to embrace page links into the "li" tag
Then, i would prefer an exception than a warning ? What do you think ?
I agreed that this shouldn't be the case, and we could fail correctly.
wrap = bool(...) ?
Controller should be calling logic functions here?
I can't follow you. Do you want me to tell that function's parameters in Python are persistent out of their scope?
There is no mention of the force option in these docs.
This would then cause build failures from multiply defined symbols.
I quite agree.  I was following the existing style in this file.  I'll push an update that DRYs this code up.
`X-UA-Compatible` необходимо задавать явно во входных данных (#199). На уровне проекта можно доопределить моду `meta`, чтобы задать значение по умолчанию. Что-то вроде:

```
// prj/blocks-desktop/b-page
block b-page {

  meta: {
    'x-ua-compatible' in this.ctx.meta || applyNext(this.ctx.meta['x-ua-compatible']='IE=EmulateIE7, IE=edge');
  }

}
I guess so (though I'm even newer) - as I understand it this aliases fake branch for composer dependencies (key) to real branch from git (value). 

I'd say that in your case you could even leave aliases out. As far as I understand composer it adds "dev-" prefix to each real branch and tag. Aliases are for hen you want to change the default behavior. I might be wrong though - I haven't read it thorough that well.
the setter should be after the constructor (with other setters)
I like this method of generation, I think the generate_password should be protected and you just do .reset_password and expect it to generate if that is necessary.
I added this to this file because in my last patch I put some UTF-8 characters in the `transform` test, and some versions of Python will complain if there are Unicode characters in a file but no encoding declaration.  My bad.
IMO to be consistent it should be fully named: `OPTIMIZE_INLINE_FUNCTION / OPTIMIZE_INLINE_FILTER`
@dlsniper in fact, the TwigBridge is indeed calling loadTemplate many times because it uses it to implement ``exists`` and catching the exception. You should add ``templateExists`` or ``hasTemplate`` in ``Twig_Environment``, which uses ``exists`` if the loader is advanced and catches the exception otherwise, and then send a PR to Symfony master to use it in the TwigEngine.
To tell you the truth, the older-style observers are better. If you must strip the notification object, do as you had done originally and pass it through to another method.
This function needs documentation.
Incorrect indentation throughout this method.
This header needs to be public for the OS X framework target.
If you are not going to use f anywhere in the function, you may as well do:

    s = open('config.gypi').read()
That remains to be seen. Please scrap this line. :-)
If we wanted to be clear we could say "projectScaleCenterTranslate". ;)
I'm just saying that the description of the output (and the name of it) suggest that it is a warped volume where as far as I understand it is a transformation (either a warp image or affine matrix). I guess this is just a copy/paste mistake.
Yes, I am still working on it, sorry about that. Thanks Ralf for your suggestions. Let me know if you have any other. I am going through the logs to make sure we didn't forget about anything.
Yeah, the PyArray_Descr object owns the pointer to the c_metadata, there is no reference count at this level. The PyArray_Descr is already reference counted, it seems wasteful to add another level of reference counting at the c_metadata level.
nemělo by tam být "procházející ji"?
Encouraging it in new features is fine, but this is definitely a bug fix. :-)
Updated this method
I've also cleanup up stray EOL whitespace, as in this file.
You did comment on it before - you commented, correctly, "why is this a def?" And now it is "val csym = tpe.typeSymbol" thanks to your useful feedback.
readme and tests have been updated, I've pinged @nateabele and @josegonzalez to get confirmation.
So, we must decide whether Floats are contained in the natural numbers or not.
They are not rather .

May be we  can warn the user about  the type mismatch (domain mismatch), that is about the reason why they are not.

A I understand 

```python
>>> S.Naturals.intersect(Interval(2, 10))
{2, 3, 4, 5, 6, 7, 8, 9, 10}
```

but I am not sure whether 

```python
>>> S.Naturals.intersect(Interval(2.0, 10))
```
is the allowed operation.
The class you suggest would be to `Permutation` as `MatrixExpr` is to `Matrix`.  At the simplest, it would have to know nothing about `Permutation`, except maybe how to reduce to one when given specific numeric input.
define K?
What is this here for? This doesn't seem like smart error checking. See line 490.
Why was all this deleted?
can't we write `return isinstance(s, TrigonometricFunction)`...and perhaps make this inline, not a separate function call?
I do not agree, it removes code duplication, and we have a constant here. It must be in one place.
`Enh #1345` + it should be ordered properly.
@Ragazzo, i have enough time to implement it. Will do that soon. :)
if len(sys.argv) < 3):
  ...
This is the recommended file where to require things like that. I'd like to keep `config/environment.rb` here. See https://gist.github.com/1497335.
No way, uv_async_t is declared after uv_stream_t
Plus the libeio threads would have to be recreated
They provide a context for the calls to Future below; although because system/actorSystem is passed in via the constructor, we can't declare that val as being implicit, hence the redundancy. Do you reckon that it would be saner to move it inside of #handleScanMessage?
when the write concern goes live, we'll move write-concern to point to the write-operations page, so these kinds of changes are needed.
This is a pattern I was using to prototype code intended for std outside of std, since inside std there is no `std` module. Instead I would bring each module of std into scope as needed, putting them where they would exist once the test case was transplanted to std.
https://github.com/phpbb/phpbb3/pull/608/files#r1361465
Ok, will change it, thanks.
<imkingdavid> in British english that is the proper spelling
<imkingdavid> but afaik we use the american spelling everywhere else
<imkingdavid> aka licence => license
<unknownbliss> The core is meant to be british english?
<bantu> should be british IMO
<naderman> yes license please
Because you can disable post counting for individual forums.
An event might need the topic data for its own thing. So `$row` really should be passed.
This can be simplified to params.details.target or nil

Of course, if params.details.target doesn't exist, this will eval to nil anyway.
nevermind, nu am fost atent că sunt totaluri
The regex `/[^matrix(,)]/g` also works and may be clearer to read for future maintainers.
ну показывать в админке суммарное количество просмотров/кликом по моему нужно.
может просто сделать в таблице баннеров поля shows, clicks в которые крон будет писать общие просмотры/клики, а админка будет брать инфу оттуда?
но я так понимаю ты предлагаешь крону каждый раз суммировать просмотры/клики каждого баннера?
good idea, but I'm not sure why you're forcing column names to be strings, and I really think that you need to force columns to be DataVec's if they're not AbstractDataVec's already. Also, although it looks like you're allowing 1-length columns to be recycled, which is good, you're not actually converting them to a vector, which causes problems. I'd suggest keeping anything that's an AbstractDataVec as-is, converting any other vector to a DataVec, and converting scalars to a PooledDataVec of the proper length.
Can we combine the slice operations into a single one outside of the loop, by moving this line and line 1478 out of the loop?
Great, I've just done that.
advacbedindexing? :)
Is that difficult to fix?
Switch the sense of the test and save an indent level, for clarity:

    if ( ! index.m_unique) {
        return true;
    }
It comes from the component, not from the group.
This is very ugly. Refactor.
Can't do it because of customization
Either leave *quoted price* w/o "the" or add it to *price margin* as well. IMHO, it's fine without both.
Use PaginatedMixin for pagination
It might be worthwhile "in the distant future" to explore making most of these enumerations that index an array, since most of them share the same type. It depends on how we're using it, but in this code I think there's a lot that would get quite a bit smaller and probably a bit easier to maintain.
Meaning of this value?
Does the LLVM style guide not require spaces around + and -?
Wrong method name : https://github.com/doctrine/dbal/blob/master/lib/Doctrine/DBAL/Platforms/AbstractPlatform.php#L229
It broke after some of the first changes on this branch. It deals with java junk anyway, so it can go away at some point.

Sent from my iPhone

On Jun 29, 2012, at 15:30, angri<reply@reply.github.com> wrote:

>> @@ -919,6 +919,7 @@ def test_parameterize_sites_no_site_model(self):
>>                 exp_sadigh, jsite.getParameter('Sadigh Site Type').getValue()
>>             )
>> 
>> +    @helpers.skipit
> 
> Why is this skipped?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gem/oq-engine/pull/800/files#r1075841
Exactly. And I'd also like user to see what they flattrd when in their history.
Alternatively you can fetch all the children node IDs at one time so that you don't need to EXECUTE the the end of loop repeatedly.  In addition, I don't understand why you need to check node_count every time, since the loop condition should be FOR lmc_nid IN min_lmc_nid .. max_chlid_nid.  Am I missing something?
Hmm, I see.  It would be much easier with some check function and domain type...
Yes, probabilities outside of [0,1] will raise an error. Is it worth mentioning?
Yes, state may have more than 1 row but here you filter the row by "iteration"
As I understand it, the Context is the place where the knowledge about what fields/labels will be present in the features is vested. wkb_features() is merely a convenience function to collect that information into a single function call when returning the features. You could make your own derived class which keeps the Context inside itself if you know ahead of time what the field names will be.

If the Context were placed into the PythonDatasource class itself, then you'd either have to pre-declare the fields you'll be using elsewhere when it is initialised or this convenience function would have to override you. This can of course be done but I thought collecting them together like this allows naive code to work without stopping someone doing it 'properly'.

I suppose the rationale for this is that the PythonDatasource class and wkb_features classmethod are just there to make it quick and 'Pythonic' to generate a Datasource. You could implement your own if you 'know better'.
There are powers where AI should run away and then don't use TranslateFacing() 
Where should it be defined then?.
I could write a condition to use it only by monsters, for example
if(((this.User).Field2==0x8)&&(((this.User).Attributes[GameAttribute.Summoned_By_ACDID]) !=0))
because we are currently setting Summoned_By_ACDID atrribute only by minions
It's basically map/reduce. Would a function taking a list of backends and two functions, one for the map step and one for the reduce step, really be simpler?

``` python
import itertools

def foo(backends, mapper, reducer):
    futures = map(mapper, backends)
    results = pykka.get_all(futures)
    return itertools.chain(map(reducer, results))

# or

def foo(backends, mapper, reducer):
    futures = [mapper(backend) for backend in backends]
    results = pykka.get_all(futures)
    return itertools.chain([reducer(result) for result in results])

# and usage:

return foo(
    backends,
    lambda backend: backend.library.find_exact(**query),
    lambda result: result.tracks)
```
Heh, this code doesn't work, since I forgot to update `NadTalker` accordingly. I'll fix.

I think it makes sense to include `mute(bool)` in our APIs. It is a useful feature in a media player, even though MPD does not include it. Will you open an issue for it?
*NB*: (There's no error here.) If you are using the `_lazy` versions of gettext functions, make sure to pass `unicode` objects, not byte strings. I have no idea why, but for some reason the `_lazy` functions choke on byte strings and you get very confusing errors.

![The more you know](http://jamessocol.com/temp/tmyk.gif "The more you know!")
Both the second added part is right, but I see an optimization for it, it
should read: var delegate = this == target ? this : xtag.filter(...);

We can change it tomorrow.
On Nov 1, 2012 9:24 PM, "Arron Schaar" <notifications@github.com> wrote:

> In x-tag.js:
>
> > @@ -148,11 +152,11 @@
> >      pseudos: {
> >        delegate: {
> >          listener: function(pseudo, fn, args){
> > -          var target = xtag.query(this, pseudo.value).filter(function(node){
> > -            return node == args[0].target ||
> > -              node.contains ? node.contains(args[0].target) : false;
> > -          })[0];
> > -          return target ? fn.apply(target, args) : false;
> > +		  var target = args[0].target,
> > +		      delegate = xtag.query(this.parentNode || this, pseudo.value).filter(function(node){
> > +	            return node == target || node.contains ? node.contains(target) : false;
> > +	          })[0];
> > +          return delegate ? fn.apply(delegate, args) : false;
>
> What was the purpose of this change?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mozilla/x-tag/pull/72/files#r2013270>.
>
>
`system "#{bin}"/...`
Are you sure you don't want a sync call here? Either way, would be worth a comment.
I made it more readable
if the path is already normalized in upload.php (as I suggested) then this wont be needed anymore
tadam et 6 ligne au dessus aussi
We need to return the messages here otherwise they disappear.
Do you want to keep the rest of the docstring? Because then I'm okay with "Modify relationship."
My initial implementation was very minimal and did require creating a new Reddit instance. You're right, I'll change it to have the side effect of "upgrading" the config to be OAuth authorized.
these are documented in the calling method import_unit. I can add this to the helper methods as well if it helps. It seemed redundant to me.
Another case of having to construct a *request like* object due to the tight coupling between the store and the request
Bitte passendere bezeichner
extends `Binding<Set<T>>` and doesn't need suppression.  Just a cast now.
Actually, it probably passes because it's entirely a comment.
just return early here so you don't have to do the else
Like everywhere? I cannot see any on existing method-calls or constructors. I have already put them at for ( and if (. I will happily include more but I am pretty confused where it misses them.
@Tibor17 In fact, we are not referring to Throwable#getMessage() and Throwable#getLocalizedMessage() at all. This pull request is about the failure message that is passed to Assert#fail() when no exception is thrown by the code under test but one expects an exception.
Could this line be remove?
I think this should be a function on `Batman.DOM`, no reason to prevent it from being overridden.
foreachlimit of 5-10 should be fine and make it go faster, parallelism is cool :)
Remove the `return`
I'll go ahead and remove it entirely, per your suggestion below.
Ah, got it. Good call. Done.
Hmmm. Does id need to be escapes here?
Fix indent these 2 line comments
Excuses sync on purpose, it is happening in the worker anyway.
I know we're all guilty of this, but building xml from a template is pretty sketchy—lots of escaping/formatting mistakes possible—and so as a compromise I think we should have a policy that anything that claims to be XML should be passed through an XML parser at least once. So just adding `etree.tostring(case_block)` would do the trick.
Isn't the purpose of all these changes to make things work with empty data frames? I suppose facets are a little different though.

```R
df0 <- data.frame(mpg=numeric(0), wt=numeric(0), am=numeric(0), cyl=numeric(0))

# Works
ggplot(mtcars, aes(x=mpg, y=wt)) + geom_point() + geom_point(data=df0)

# Doesn't work with facet_grid
ggplot(mtcars, aes(x=mpg, y=wt)) + geom_point() + geom_point(data=df0) + facet_grid(am ~ cyl)

# Runs (but red point doesn't show up)
ggplot(mtcars, aes(x=mpg, y=wt)) + geom_point() + geom_point(data=df0,x = 10, y = 10, colour = "red") + 
  facet_grid(am ~ cyl)
```
As this is a constructor, I guess this is not an insertion but a creation. The code does not match the javadoc.
Because it doesn't use PluginHost instance.
I like this. But I do not like the name. ;-) I would just leave it `MongoEngineBackend`. It tells everything. ;-) So:

```
class MongoEngineBackend(mongo_models.MongoEngineBackend):
```
Change this to just be `url`, and you can remove the 200 status code, that's the default
Add a comment so it's clear what this is for, and so uses of it below are clear.
Let's set these as properties of the ProgressBar in the constructor that way each time set value is called it doesn't have to find the DOM element.
日本語でページが前にあるのは、確かにちょつと変なので「{{page}} ページ」に修正しました。
oops, that should read TH{1,2,3}/!normalize/[float target_integral]/ - my angle brackets got eaten before
没有名字空间。应该是：QBOX_ACCESS_KEY 才对。
Yep, couldn't figure out how to validate it within the admin. Once that's done in the validateThis framework, the isNumeric check is no longer necessary.
Multiline `if/else` statements are preferred
Because I want the exception to be exactly the same as in Ruby. And without drop(1) you got one extra line.
Is it possible to switch to use the sprites and icon classes now that ConvergeUI is at the newer version?
This file should be removed.

The manifest is generated (for now) and should not be committed into the repo.
(If you rebase you'll have the proper gitignore)
It was setTimeout(function(), 0); before. Can you re-use that or do you really need 100?
Shouldn't `health_opts()` be of type `[health_opt()]`?
Can't say I really like this guideline. Encoding types into variable names reeks of the windows "Hungarian notation" fad, and makes little sense to me.  IMO, variable names should be clear and human readable. My IDE and compiler figure out the rest from the context.

Then again, if you think this makes the util code more consistent feel free to do it here. Please don't do it throughout the qt code though.
MAX_BLOCK_SIZE / (default -maxsendbuffer / 2)
The link simply explains why its maxsendbuffer/2.
Same comments are above. As request read the coding standards page on manual or used the code sniffer.
`FALSE` should be `false` as per the code conventions.
This is wrong. The return type is still ``object``. ``entity`` is not a PHP type
```js
format: 'url'
```
El 18/12/12 04:22, Bussonnier Matthias escribió:
>
> In IPython/frontend/html/notebook/static/js/examples/metaui.slideshow.js:
>
> > +
> > +    MetaUI.register_callback('slideshow.hslide',hslide);
> > +    MetaUI.register_callback('slideshow.vslide',vslide);
> > +    MetaUI.register_callback('slideshow.fragment',fragment);
> > +
> > +    //slideshow_preset.push('slideshow.fragment');
> > +    //slideshow_preset.push('slideshow.vslide');
> > +    //slideshow_preset.push('slideshow.hslide');
> > +
> > +    var select_type = MetaUI.utils.select_ui_generator([
> > +            ["-"            ,undefined      ],
> > +            ["Header Slide" ,"header_slide" ],
> > +            ["Slide"        ,"slide"        ],
> > +            ["Fragment"     ,"fragment"     ],
> > +            ["Skip"         ,"skip"         ],
> > +            ],
>
> The problem is when metadata does not exist, javascript return a 
> |undefined|, so even if I map nothing to undefined you will still get 
> some (and the ui will be inconsistent) if user have never touched the 
> cell.
> So I preferred to explicitly map it to undefined.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/ipython/ipython/pull/2687/files#r2445477>.
>

OK, Mathias... I read the code after send it this request... I 
understand... I have found a solution, thanks again...
Up to now, we explicitly use the Unicode versions of everything everywhere in libgit2, exactly because we don't support anything other than Unicode. If you define `UNICODE` for that one file, it might be ok; but if you define it from the build scripts, you might enable some of the Microsoft magic that we either don't want now or in the future.

i.e. `UNICODE` not being defined, forces us to write `CreateFileW` if we want to call it with `wchar_t*` parameters. `UNICODE` being defined allows us to fallback to `CreateFile` and give it a `wchar_t*`, but we should not in this case.
looking back at this line, this seems dangerous. We should flag that we're using our easyXDM ...
Thought about this too and was thinking it's not absolutely necessary. But sure, I will add it back so that we are on the safe side.
The problem with skipping validation entirely is that if a bad value is submitted somehow, it'll result in a 500 server error (when you query for that suite/case when saving) rather than a form validation error. This is probably not a big problem, since bad values should never be submitted by a normal user unless there's a bug in the JS/Ajax. The one exception I can think of is if someone else deletes a suite/case between the ajax query and the submission of the form; at the moment I think this would result in a 500 server error, which is kind of unfortunate. (The other exception is a malicious attacker, but it's probably OK if they get 500 server errors; better if they can't trigger that, though).

I would do the validation not here in the field but in a `clean_suites`/`clean_cases` method on the form. I think you can do the validation without adding any new queries at all; just do the query for actual Suites/Cases in the validation method and re-populate `self.cleaned_data["suites"]`/`self.cleaned_data["cases"]` with actual Suite/Case objects in the proper order, and raise a validation error if any of the given IDs don't match up with a real Suite/Case. So you're just moving the same query from saving to earlier in the validation step.
The extension needs sessionId field and there is no need to send `flashParams` back. Use `{url: flashParams.url, sessionId: 0}` instead.
Hmm... it's kinda messed up. We're relying on `buildlib` which will (without any choice) commit the connection. So, making this whole app transaction managed is not an option. 

But.... (see next comment)
заберу, але при збіганні імені потім може бути exception
this is not needed as it is already initialized to an empty array
good point. Done.
@stealth35 Do you mean something like https://github.com/gajdaw/symfony/commit/5b3501acdca3097331a5adf1d239d448863ddc8d ?
Since you added this, you can remove the old type cast a few line below: `$this->string = (string) $propertyPath;`;
`all()` only returns instances of Route
why not just ``&& $string`` ?
I was thinking about this, and it should probably be `this.clone().startOf()` or `moment(this)` so that the original moment isn't mutated.
What was wrong with checkin.js?
It built the bundle but failed to find the manifest.  This is something I was seeing previously, so I'm inclined to say it's not a problem with using the ruby-build ruby and is something that should be fixed outside the context of this branch.
Demeter violation and also parens not necessary here.

This leads to think maybe a nice private method might be good here, like

```ruby
private

def owner_same_as_sender
  @invitation.event.owner == @invitation.sender
end
with locals
akh sharmande meske yadam shodame h1 ro ghable p bebandam :cold_sweat:
I am relying on the fact that 2 frames are buffered (see renderupdate() in linuxrendererGL).
A new frame is ready for display. The first field from this frame is weaved with the second field from the previous frame (still available in its buffer). This is displayed. Next the new frame is displayed in its entirety.
Twice the sync period should suffice
```python
    role_list = []
    for role in  littlechef.lib.get_roles_in_node({'run_list': run_list or []}):
        if not role.startswith(REPO['EXCLUDE_ROLE_PREFIX']):
            # Only add if it doesn't start with excluded role prefixes
            role_list.append(role)
    return role_list