labofsahil/github-event-dataset-2012 huggingface dataset:
right,
fileext.cpp does not exit in 3.0 ... apply only 3.1

2012/3/10 Paolo Ventafridda
<reply@reply.github.com>:
> Are you sure.. UTF8 in logbook is not used in 3.0. Since it was untested, I
> left it out for 3.1.
>
> ----- Original Message -----
> From: "Bruno de Lacheisserie"
> <reply@reply.github.com>
> To: "Paolo Ventafridda" <venta@bware.it>
> Sent: Saturday, March 10, 2012 2:07 AM
> Subject: Re: [LK8000] Path Not Initalised And _tsncat -> Stack corruption
> (8aa5e6b)
>
>
>> yes I confirm, utf8File is Used by LogBook and it cause unpredictible
>> buffer overrun ...
>>
>>
>> 2012/3/10 Paolo Ventafridda
>> <reply@reply.github.com>:
>>> Please Bruno confirm this should apply also to 3.0b and not only to 3.1
>>>
>>> ---
>>> Reply to this email directly or view it on GitHub:
>>> https://github.com/brunotl/LK8000/commit/8aa5e6b92aa015785c0e480c4dd4ccf8ea0317cf#commitcomment-1068194
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/brunotl/LK8000/commit/8aa5e6b92aa015785c0e480c4dd4ccf8ea0317cf#commitcomment-1068211
>>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/brunotl/LK8000/commit/8aa5e6b92aa015785c0e480c4dd4ccf8ea0317cf#commitcomment-1068217
Should be `#{bin}/flickcurl` for the path
Eh eh. Sincerely, if there is something simpler, I'd would be glad to be hinted (when I thought about the time I spent making a concept about the need) and try to figure out how to nicely add it ;])

Now, as you may know, the dynamic sprite replacement system is here to replace a sprite from a given "equipment" (including hair and race sprites) to another when wearing another incompatible equipment.
For instance:
- Hair with pikes when wearing an helmet,
- Certain equipment with others with unsolvable incompatibilities.

I had to think about a generic solution and I do think it's answering the need.
I thought of M then.
Доброго времени суток, походу v3.2_master.diff.gz обновили... 
!!! Fetched file: v3.2-r2_master.diff.gz VERIFY FAILED!
!!! Reason: Failed on RMD160 verification
!!! Got:      f6cb904e0426d99d366f4ed89ca9e18cc59d5f37
!!! Expected: 7e65f96092657d876386f3396000265a513664a4
ya but los check was deprecated in mangosr2...and it's always enabled... any hints to what can I do?
дЫк не реально ж найти такую ауру..
this is fantastic, thanks!
Nope, sloppy of me :(
Using your own buffer to read chunks is more efficient than using an buffered wrapper, even if the buffered wrapper's internal buffer size is equal to your own buffer. Also, if you do use a buffered wrapper around an input stream and you use the read(buf[]) method it is  more or less the same as just using the inputstream's read(buf[]) method performance wise.
nobody can see the fix in this mess!
why don't just do indentation-change-OMG (w/o touching any code!) commit followed by an actual bugfix commit?

also, nobody would be able to find some one-symbol-crazy-creepy-silly-bug in such a change... probably that's why people hate indentation changes and most projects would never accept such patches.

it's a pity to see this craziness in every piece of code in every piece of software...
and why do people touch indentation at all?! totally broken editors? then get a better one! :P last line (absent LF) proves this.
Min and Max Threads count would be nice feature ;-)
Да я не против насчет куда направлен прицел. Вот только если он вдруг направлен ниже ватерлинии, то с вероятностью 10% имеем краш всех клиентов кто видит взрыв, а 1% - краш сервера. это кому-то надо?
неа. дистанция разная на разных картах.
For backwards compatibility. Perhaps a macro would have been better than a wrapper function.
Don't worry! It was a pre-commit because I need to switch branches to check something. The final commit, is based on the DAO object.

(That code was copied from here : https://github.com/osclass/OSClass/issues/151#issuecomment-3994663 )
@dbykadorov it should work
lol, I didn't even pay attention to what table it was.  Same comment applies though.
Tu dois pouvoir faire plus simple :
        Member _arg = Member.findByLogin(login);
        boolean _short = true;
        render("tags/member.html", _arg, _short);
    }
Actually, the `<>` operator is binary concatenation, so it would be more appropriate if we called `E.handle_text("", "foo")`
Nice work!

Only one minor think.
You use variable _ in place where it shouldn't be used. It makes your code harder to read.
this makes sense - after a fresh install I always set the zoom to mouse cursor first! Thanks.
I don't know how season banners are done in the skin. Are they not just picked up by XBMC? I cannot get season banners with artwork downloader (option greyed out), but how do you save these? As tbn or jpg?
it didn't break any apps/users tests, still running the rest of the suite. The one thing it might affect is logging into Django Admin. I still need to manually test that. It would only apply to new accounts of course.
Normally we should have seen it . Anyway, it work now and there is a test.
What you're describing is impossible. The resampler changes occur after the channels have been mixed, which occurs earlier in the DSP. There's no way it could completely remove a sound effect like that.
Give it a try on bsnes and see if you can hear it there. I still notice in level 1-1 that the bird noises will overwrite the effect, which coincides with my suggestion that the level background music prevents it from playing. 
If you're still insistent, I'll look for my cartridge and test it on the SNES to see if it's there.
Shouldn't this.p._index have the same number of rows as the data?  Or am I not understanding what this is doing?

I've got 12 rows of data.  (which is observable at this.p.data)  and the _index looks like this in ie dev tools: {...} , when expanded looks like this: undefined | 11

ind is then undefined, so I never get to line 3250.

Is this a bug?
Ruediger Meier left as an exercise for the reader:
> Hi Nick, in case you have a shell available in your environment you could do the atemsprayer stuff also by using this one liner
> atem -s -n -f 0041000 /path/to/data | while read -r symbol fnum ;do atem --fdat $fnum /path/to/data > bla/$symbol.csv;  done
> or have a look at "man 1 csplit".

wouldn't that overwrite the output csv with each line? i need to collect
them all. i'd actually used a read bourne loop originally, >>'ing on each
line as multiplexed by the first column, but it was horrifically slow.

i'm down with csplit :) i use it to paste together my web pages as
sprezzatech.com :).

my work on this fork is going to be ghastly surgery, and i'd recommend not
really following it :). i considered updating your -o option to take a
directory, but it didn't really fit into the scheme of things well, and i
had no idea whether you keep atem up-to-date or accept merge requests.

if you'd like, i can definitely make my slashing changes here with an eye
toward feeding things back upstream.

-- 
                                    nick black <nickblack@linux.com>
                 http://www.sprezzatech.com -- unix and hpc consulting
  to make an apple pie from scratch, you need first invent a universe.
actually, just make a pull request for this and I'll merge
why do you remove cocos2dx_loc variable in eclipse
without it, how do we configure project in other folder outside of cocos2dx base folder?
Indentation is 4 spaces (no tabs) for java files (that's one of the code style we're enforcing 100% at this time, others are file encoding must be UTF-8 and line endings \n). See README for more information.
this was done differently in commit ef4b39354eba63079582251a4e95568c1efc65eb

now all magit buffers set truncate-lines to t by default, and this can be changed in magit-mode-hook
https://gist.github.com/1739782
Same thing, with macros to ignore locking if it's not wanted.  Sound good?
Why do you need the package file in the source?
This isn't thread-safe.  Use a static initialization block instead.
Add STRATEGY_MEDIA_SONIFICATION, after line 133 as the compile borks out.
same question :D
You can see how I am doing on the cluster branch.. ;)
はじめまして。
scala-mode.lを使用させていただいたのですが、テストをpassしなくて、
実際にインデントがくずれてしまったのですが、調べてみたところ
indent-tabs-modeがnilでないと、スペースとタブのインデントが混ざって、
インデント数の計算上は正しくても、見た目が崩れてしまうことがあるようです。
ためしに、
https://github.com/pocari/scala-mode.l/tree/spike
のように変更した上でまっさらのxyzzy上でscala-mode.l, test-scala-mode.lだけsite-lispにいれて
テストするとpassするのは確認しました。
Я про другое — про initialization list.
`Foo::Foo () : A_ (smth), B_ (smthOther) {}`
Exactly. There are cases where you are compiling code that you will call several times, even after `main()` exits.
масло масляное в названии, переделать
It's actually fine; we're not starting up services for outbound sockets during RunningMode.ADMIN_ONLY. If we find a use case we might change that.
Was about to commit the same. Enables us to use cases inside the order by clause. Thanks.
This is only for the loader currently I'm Core. Usually you dont need the
IDs, because the loader can get them from the filename. When using MooTools
the old way with <script> tags, the loader doesn't know that. However I
didn't want to use the ID parameter because other loaders and compilers
will use that too, while they usually prepend something to each module ID,
like mootools or Core, depending on the project it is used in.

But yes, this solution is a bit hacky.
On Jan 24, 2012 2:11 AM, "Sean McArthur" <
reply@reply.github.com>
wrote:

> I believe AMD figures out the id if you don't specify one in `define`
> based on the filename.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/arian/mootools-core/commit/cae2b916971042f49160465884a67ee46b5ae6fb#commitcomment-894538
>
it's your choice :)
This branch should also be merged onto develop
Looks fine now, but there's should be a “parent’s parent” check (it may be undefined). I’ll merge this update by hands, because new ZC version is based on different module structure.
It should not be merged with origin (It was pushed to config branch by accident).
O rly? :D

Anyway I completely agree with rationale as it doesn't make sense to have settings on it's own.
sorry! Must be "a" ... instead of "an"
This security manager may cause some trouble. I learned it the hard way on my System Rules project :-) What you should use is something lke https://github.com/stefanbirkner/system-rules/blob/master/src/main/java/org/junit/contrib/java/lang/system/internal/NoExitSecurityManager.java
You can copy the code. It's mine.
I think #L0R75 is a bug, render result not returned
Personal preference.
You could also put only bootstrap.less into assets/stylesheets, leave all the other modules in a subfolder. Modify the code in bootstrap.less to include the subfolder in all the imports.
I'd prefer to test this code and not put in an "experimental" warning that will never get reviewed / removed even if the code proves to be stable.
Getestet?
@klaussilveira: I think having *something* related to docs/commenting, as a matter of good style, is reasonable.  You mention that each project has its own doc generator, convention, and rules for commenting code.  I'm familiar with only 2 specific conventions/generators in PHP land: the PHPDocumentor/DocBlox convention, and the Doxygen convention.  Are there other major systems that you can make me aware of here?
Sphinx が正しくパースできるように「代わりに」と「 ``」、「``」と「クラス」のあいだに半角スペースを入れていただけるでしょうか？
This should fix the merge history
I think Mojolicious must be listed too :)!
I don't actually see this. My results look fine. Screenshot?
To support this, we could provide an update-state GridField action; my suggestion on line 21 could provide the scaffold for it.
I miss the Picard development, too :D Once I have put my personal live in order again I will hopefully be back. But I am not sure how soon this will be :/
most people i know just use the english 'commit' - any of the git verbs are generally just used in english, not directly translated.
Ka falls ihr mal euer nächstes chaoscore repack veröffentlicht?
It shouldn't be possible to have question without a question and an answer without an answer. It should be required otherwise we end up with lists of questions and answers that are empty.
Most everything looks okay to me. Though:
- In the changing of reStructuredText files to txt, are we subverting/invalidating Sphinx's involvement and perhaps Read the Docs (see http://readthedocs.org/docs/mytardis/en/latest/)? I haven't been involved in the Sphinx/Read the Docs work so I'm just raising the question.
- As you probably know, the contributors file isn't complete but we can complete it on January 4ths "Move to Github" day.
See http://tools.ietf.org/html/rfc2616#section-10.4.6 "The response MUST include an Allow header"
If you want to check for permissions, you use hasPermission. It's as simple as that.
ok now again :D where can i get new map,vmaps extractor for windows ?
read the commit message ;P
The compare can't return < 0.  I don't think it is worth asserting.  (Is there a clean way of adding an assert with the guard condition?)
nope that wouldnt cause any db issues
@Ussmaan here
https://github.com/SignFinder/FaceCore/downloads
wouldn't `Mysql2Adapter,class_eval` be nicer?
Oops. Talking to myself. :) I was asking to @emboss.
@uwej711 any idea why this collection does not behave as an array? toArray or also reset($this->routes) gives me instances of proxies for the democlasscontent rather than the target document. and in the router i check for the class and of course it fails.
We discussed licensing over issue #26, but the main reasons are (IMO):

* We don't know Mojang's work license, so only our patches would be allowed for commercial use, that doesn't make sense
* I'm not sure every contributor here would like to see someone release some Android app from everyone's work, and make money out of it. A few hours ago, a simple adf.ly link to this project's builds in an MCForums thread started a sort of flame war.
A cleaner way to use namespaces is to do something like this:

(in the initialiser? Include all the namespaces you know about...)
static const NSString *ocfns = @"urn:oasis:names:tc:opendocument:xmlns:container";
static const NSString *opfns = @"http://www.idpf.org/2007/opf";
static const NSString *opsns = @"http://www.idpf.org/2007/ops";
static const NSString *dcns = @"http://purl.org/dc/elements/1.1/";
NSDictionary *xmlns = [NSDictionary dictionaryWithObjects:[NSArray arrayWithObjects:ocfns,opfns,opsns,dcns,nil]
    forKeys:[NSArray arrayWithObjects:@"ocf",@"opf",@"ops",@"dc",nil]];

Then use nodesForXPath:namespaces:error: like so:

NSArray *nodes = [container nodesForXPath:@"//ocf:rootfile[@media-type='application/oebps-package+xml']/@full-path"
    namespaces:xmlns
    error:&error];
somebody translated it to perl... great!!!!
JavaDoc for setAttribute recommends qualifying the attribute name to avoid collisions.

Collision is unlikely, I know, but perhaps 'integration_channelNames' ?
Do you think this makes it more readable?
Really? But I thought string is sanitized if it is passed as a hash, according to this section in RoR guides http://guides.rubyonrails.org/security.html#sql-injection
There are few missing views & events that should be tracked:
/b2m/init (when the setup screen is displayed)
/b2m/urls ( when the list of urls is displayed)
/b2m/urls/open-prompt (when a question if the url should be opened is displayed)
events:
- reloadUrls (when urls are reloaded)
- cancelOpen (when user decided to cancel opening a url)
Utf-8 is supported!!! Can you see the word '企鹅', which means penguins?
Alas, this is a faster option, to cleaning the vote code ^_^ It seems to look the same (minus the few cleanups did by people over the time) as to what it was in the goon code.
The docstring for `Memory` still says that `compress` should be a boolean. Is it still supported or should it always be an integer?
Changed end of header keyword regex to allow for keywords with spaces in them.
The reason for the ceiling in the first place was that, while running when quickly resumed, backgroundTimeRemaining was giving a huge number.
One constant as opposed to another? During pause, we call wait twice, once before and once after the garbage collection. So I wanted to give enough time to process, but not starve later calls.
I also got this error then tried to create weather.cod file in cmd:
C:\Program Files\Research In Motion\BlackBerry WebWorks SDK 2.3.0.9>bbwp C:\mypr
ojects\weather\weather.zip -o C:\myprojects\weather\output
[INFO]                  Parsing command line options
[INFO]                  Parsing bbwp.properties
[INFO]                  Validating application archive
[INFO]                  Parsing config.xml
[INFO]                  Populating application source
[ERROR]                 Feature cannot be found in any extension(blackberry.ui.Spinner)
@cordoval The bundle allows you to configure a PHPCR implementation without configuring the ODM if you want to use only PHPCR. This is the same than DoctrineBundle allowing you to configure only DBAL if you don't want the ORM. So the current way is fine.

The only point that could be discussed is the requirement on ``dev-master``
heh ignore this pull, i left my intentional red spec in by accident when i committed
sorry, forgot to take my comments out again before submitting
Ok I will make this change
what do you mean? Do you mean just warnings?
`SDK_INT` was introduced in 1.6 so it was available here. The reference to the superclass method was the line causing issues. There's no way to call the superclass implementation of newer methods on Android 1.6 when you are overriding them. It's quite annoying.
Ok, I have to guess how to exclude them in the gemspec
dirty birdy
thanks to spyder's fix, it helped notice what we left out.
Só acho interessante como decisões que influenciam todo o projeto são tomadas isoladamente num trabalho "em grupo". 
ps: Não estou falando apenas do formtastic, que talvez possa ser utilizado junto com o simple_form.
k, fixed, figured i was reinventing the wheel by using my own podcast/feed scraping when i can just use other addons to do it for me :D
This should be a string: "0.1" or an integer: 1
Why is it not compatible with Chrome and IE?  could you tell me what could I do to achieve that? thanks
А вот, що свойства объектов массива Posts передается через поиск ноды поста по DOMу, до ентого я не догадывался.
I thought we set a max at 120, and this is more iirc.
0 or min? or super.sessionCount() ?
I just realized that technically these controllers should not b accessing the registry directly.  I won't stop this merge for it, but we should re-investigate this.  Only in extreme cases (typically for stats purposes, do we access the registry directly)
unless I’m wrong, `either` would also work from Java and make it a bit shorter.
transition directly, no need for `busy`
Nice catch! Sorry about the broken merge.
yes

In the log there was:

    [JVM-Node2] [WARN] [10/17/2012 16:11:57.604] [StatsSampleSingleMasterSpec-akka.actor.default-dispatcher-3] [akka://StatsSampleSingleMasterSpec@scalable2-03:46158/user/statsFacade] received dead letter from Actor[akka://StatsSampleSingleMasterSpec/system/testActor1]: StatsJob(this is the text that will be analyzed)


and the test verifies with:

      // eventually the service should be ok,
      // worker nodes might not be up yet
      awaitCond {
        facade ! StatsJob("this is the text that will be analyzed")
        expectMsgPF() {
          case unavailble: JobFailed ⇒ false
          case StatsResult(meanWordLength) ⇒
            meanWordLength must be(3.875 plusOrMinus 0.001)
            true
        }
      }

so the test expect to eventually receive a StatsResult, but since the StatsJob (the request) is lost there will be nothing by timeout

this is a sample, so I don't want to complicate it too much, therefore I use a **local** facade as entry point
The ticket states: In akka-mailboxes-common and akka-remote, the import for com.google.protobuf should have a version # e.g. ;version=[2.4.0,2.5.0)".
You changed zeromq?
Hmm, you’re placing part of the failure handling inside the actor, while all other failure handling is done in the supervisor. Probably better to just keep it as it is and do `failBarrier()` in the supervisorStrategy for this case.
im not sure we even need to care about updating these variables here... just read until you hit a default select case (we're closing this connection and its going to be garbage collected)
Added a comment as you suggest. About the test -- there is one: https://github.com/angri/nhlib/blob/vectorized-gmpes/tests/site_test.py#L81
As we discussed, this could be refactored a bit so it is only implemented in the base class, and this class would simply have to implement `_compute_mean` and `_compute_term_3`.
CDVDAudioCodecPassthrough is/was buggy. We can remove DVDAudioCodecPassthroughFFmpeg at a later time when CDVDAudioCodecPassthrough is stable enough.
Sorry I copy/pasted some of this, is there a reference for writhing docs that render well? I'm kindof learning this one thing at a time, it would be nice to have something to refer too.
There's something wrong with the file DSI203_license.txt. It's full of numbers. Looks like it might be a copy of the bvecs.
@icmurray @rgrp I haven't been around during the time the decision about naming was made. Can you comment on that?
I kind of like that strategy, but we have a caching problem here again.  We are storing the cards as a class variable but there is nothing to reset the variable when a card gets renamed.  Perhaps the method should be "setting_codenames_by_group", and then we should fetch the cards at render time?
This can also be used to schedule only when the collector is started so that no check to active.get is necessary
gc-collected-records ?
Added more comments ;)
Could do with some indentation as well (see my previous comment). Actually mabe this could be factored out into a helper function as this is an exact c&p from code further up (line 149).
Bad indentation (only 2 spaces per level), same in the else-block.
We should probably make a `combine_opts_with_priority()` function; this is getting hard to understand. This priority stuff was patched together to solve the `ec2_instance_type` vs. `ec2_*_instance_type` issue, and could use a bit of a refactor.

Also, I forget if we check for specific values of priority (like if `self._opt_priority[...] == 1` means something specific), do you mind auditing the code?
In apache at least, we do not allow author tags.
this function is defined already above.
ya know?
`There are no registered paths for namespace "%s"` I think namespace is better than `path name` otherwise this may be confusing.
This is unnecessary. KVO will automatically generate notifications for overridden setters too.
Note: usually we use a struct field to store all the methods the delegate implements as we set the delegate, to quicken it up, but shouldn't be too bad without it. (Don't change this if you don't want to.)
Can we name this a bit more clearly and/or add some documentation as to its purpose? Right now, it's impossible to know what it's for without looking at the code below.
This will only break out of the `switch`, not the loop.
about closure yes i made a mistake when i read through the doc between the 2 chapters. 
for the array i needed it because of createentrypoint.cmd.php
Where's our favorite place to get that? Looks like it's currently coming in by way of uv.h--are you suggesting that I explicitly include uv.h around line 22? I'd include stdint.h directly, but it looks like uv.h is working around it for VS 2008. Do we even support VS 2008 any more?
@TooTallNate is right, `err.code` is preferred. The dns docs are outdated, a lot of the error codes it mentions are wrong too.
Hm...  Grammar?
https://github.com/luvit/luvit/pull/382
This is way too small, needs to be at least 3000. Otherwise GC on the master can easily cause it to fail.
<pre>
for k, v in ("ROSE_ORIG_HOST", socket.get_hostname()):
</pre>
Better?
indeed... habitude
Oh! Fancy stuff.
We don't want to ignore the `itemConfig`!
I'm aware of the arguments logic, I wrote it ;)

From Zend/zend_API.h:
`#define ZEND_BEGIN_ARG_INFO_EX(name, pass_rest_by_reference, return_reference, required_num_args)`

It doesn't seem to break anything though.
Not sure we should keep TODOs, FIXMEs and commented code. As we're in an in-progress PR, not a real problem for now though :)
I prefer "expires" in place of "duration". Duration is too vague, and there will be other time intervals in future.
It would be good to include some additional tests for the following:

* Key normalization
* Keys longer than the cache can support
* Values larger than the cache can support
Есть возможность запихать эти константы статическими константами того класса, на работу которого они влияют? Если есть, надо это сделать.
сделано. осталось только на вики
Please use the new hash style.
cool ! I agree.
Test  is still green :-)
Hm, not sure how to deprecate a `class_attribute`, this does not seem to work and seems to go into a loop and leak memory.
This is not the same when sym is NoSymbol, and name is NO_NAME.
Annotation reification fails with this (tests confirm this), and this problem was the reason to do NoSymbol check.
this all can be;

```
else:
    uni = kwargs.get('unimodular', True)
    if uni not in (True, False):
        raise ValueError...
    kwargs['unimodular'] = bool(uni)
    res = function(*args, **kwargs)
```
Maybe add a check 

```py
if not expr.is_polynomial: 
    raise ValueError("expr must be a polynomial")
```

at the top here.  Otherwise a non-polynomial would just loop forever.
This should probably be a new style class.
Yes, it depends on the prism, but it is just the product of the `cos` of the two arguments. You may create a function for this if you want but a whole class that does not add any functionality seems excessive.
Space near `==`
s.as_dummy() will convert a symbol to a dummy with the same name.
I don't think it would make much of a difference. Computing ecart is fairly cheap, but T could be *very* big. I think I'll just leave it as is. I'll have to go over these functions again to squeeze out all performance I can get later, anyway.
Textual representation => Full name representation
Filtering on the WEB-INF folder works in intelliJ now?
Look at the cache logic in Container.
This type of access is not BC and won't be deprecated/removed.  Its actually the most proper way to access these values.
Loads and loads of noisy whitespace changes throughout.
you don't need to assign 'f' or 'meta' here, _ them?
Style: `newSuperview != nil`
My earlier comment was smoked when I re-pushed....I agree that this feels *awkward*.

First things first, I'm not a Ruby guy ;0

So yeah, I wanted to just add a property that `job_config_path` could use.  From what I could determine, adding attributes to ActiveRecord derived classes, where the values aren't stored in the database, is frowned on?  It also appears to be deprecated in current Ruby versions per https://github.com/rails/rails/commit/b2955edc?

I originally used `repo[:job_template] = template` in `self.setup` .. but I got warnings in the logs for writing arbitrary attributes.  I assumed that to do this without adding an attribute, I needed to find a way that I might do this in other languages -- with a closure over the passed value of `template`.

Maybe the ActiveRecord thing isn't a big deal (not sure if this is something being changed in a future Ruby version)?  FWIW, the code still worked fine, just generated warnings.   I could have also totally misinterpreted what I dug up.. 

Let me know either way.
Add the corresponding assert_not_equals as well
Glad you like it! It's a little rough around the edges (as you can tell), and if I found time, I might clean things up a bit. Didn't really put any effort in to multitouch. Right now it only cares about the first finger you touch the screen with. Are you seeing other errors related to multitouch rather than just support or lack thereof?
This separation of the initial load of the "runningTasks" and the watcher actually introduces a race condition in which a task could be added after runningTasks is populated, but before the PathChildrenCache is fired up.  It's much safer to build all of the logic around the state of PathChildrenCache.
When what AMI is ready?
literals around hostname. also subdomain of example.net.
A spammer isn't going to request deletion. This is a queue for users to request to be deleted. IMO it is a good idea to let them know when their request has been acted upon, either approved or denied.
It is running once every thirty seconds. It's a single DOM operation. It won't make any difference.
Fail right here.
I could create a variable with the selectors in it or I could also remove the dl selector. There shouldn't be a case where we have a class, i.e. .forum_unread, inside a li.row which is not a <dl>.
This stuff is all copy-pasted. It should probably be a separate test case with a data provider.
These should also be attached to the `phpbb` object - it prevents pollution of the global namespace.
Please document the parameter ($style_id).
Is this todo still relevant?
That would make it sql_injection, not exactly great sounding so I changed it to sql_modification as it can modify as well as inject.
fixed in 57c122c
The option are Copy or Freeze, and we are expecting Object.freeze to become more performance in the upcoming V8. We can't allow the original object to be modified otherwise we will be bleeding between requests, or even within the same request between mojits.
I tried, but the tests failed to run when using `mojito test`.
Example Service Code.
no problem! I added the pull request then reappled the (array) cast in the following commit
It would have been fine to simply do some type checking here (i.e. assert it is a tuple).
I think for real-valued arrays, there should also be support for an approximate equality check.... there should be such a thing in TensorType
I see, just made the changes, tks!

On Thu, Aug 23, 2012 at 5:00 PM, Paul A Martel <notifications@github.com>wrote:

> In src/frontend/org/voltdb/ClientInterface.java:
>
> > @@ -289,6 +289,8 @@ public boolean queue(int queued) {
> >      private boolean m_hasGlobalClientBackPressure = false;
> >      private final boolean m_isConfiguredForHSQL;
> >
> > +    private boolean m_isExplain = false;
>
> Why is this a member, and not an argument to the one function that uses it?
> I don't think that we want the ClientInterface to "remember" this value
> between function calls.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/VoltDB/voltdb/pull/152/files#r1449203>.
>
>
Because unique together key is only on device and mount_point.
>600 seconds = 10 minutes - 1 second = 0.0166666667 minutes

I read this as "600 seconds are equal to 10 minutes minus 1 second, which is equal to 0.017 minutes"
There's something wrong in that syntax
Building on what I said above.  If the value is bogus then `riak_core_capability:query_capability/4` will _silently_ return the default.  I feel like we should still log something to indicate that the user provided an invalid value.
This seems like a poor idea, and probably should not exist.
Hmm, which do you think is better. I guess *class based*.
maybe useful for the regex lines to append a comment showing # ex: /polls/5/vote/
the space should be removed when using the short version of the ternary operator: ``?:``
Oh, if gtest implements the comparison then it might work, not sure. They technically aren't "wchar_t" though, they're uint16s, which is only equivalent to wchar_t on windows.
Worth doing (require clojure.set :refer [union subset?])
Can reference and value captures be treated as instances of "overloaded" capture rather than as special cases?
the statement is wrong, Jan 31 2012 add a month should be Feb 29 2012 not the Feb 29 2013
https://docs.djangoproject.com/en/dev/ref/settings/#installed-apps

These are supposed to be immutable. They really shouldn't be lists. If they're mutable, then it's much more likely the get changed by something when passing them around.
Given that we've kind of sucked at fuzzing passes, it's worth adding a test to make sure we don't throw an http 500 on junk for these two new fields.
Remove the `print`, and `install` does mkpath automatically, so this can just be `bin.install 'gitcrypt'`
Clearly stated.

I will find a different solution for you and release 1.3.1. Homebrew should skip 1.3.
code commenté ...
Unless I'm misunderstanding how this dispatcher works, you really want the application name here rather than the transport name. Unfortunately, I don't think there's an easy way to get at it. I'll have to dig around and see what we can do.
All this does is put the message in the `message_payload` optional parameter instead of the `short_message` parameter in the PDU if the content is longer than the maximum allowed in `short_message`. Existing behaviour was to try stuff it all in anyway, and that's still the case if the config value is set to `False`.

We need some more generic way of handling this, but that's beyond the scope of this branch.
Make this public, so it's easier to extend, use in other places, etc.
Could lines 130-133 be replaced just be checking the length of PGs, and if it has a size of 1, than the name already exists in the vault?
This inserts LI tags with no list container. This needs to be fixed.
The assumption for the repo location should be documented. Perhaps we can stuff this in a config file later. New issue?
If I remove this the hand cursor goes away -- it looks like Bootstrap turns it off by default on links.  I've added a comment to clarify.
Nope :-(

I added an OS check to the script.
Ok, I'll remove the unused vars.
Both conditions check the hintProviders.all flag, so this should be checked only once. This also makes code more clear:

```
    if (hintProviders.all) {
        if (enabledProviders.length > 0) {
            enabledProviders = enabledProviders.concat(hintProviders.all);
            enabledProviders.sort(_providerSort);
        } else if (hintProviders.all) {
            enabledProviders = hintProviders.all;
        }
    }
```
Ditto with this cluster of tests that all share the -unittest2 menu.
superclass.templatesが謎っぽい
I was trying to make it shorter, but I'm not attached to the change; I put it back.
yes, please remove it
Not sure what you're trying to say here but I wouldn't worry about Windows, Freevo isn't targeted at it and we can't try to support everything.
Add a TODO (into comment block of template? or maybe into Facebook view handling the Facebook login? maybe better later) that we want also correct redirect after Facebook login.
Done. Watch next push.
Again, same as above. There's no `order` option here though.
Yeah, that'd be nice. Even the word `styles` is redundant, so let's just name it something more general.
Covered by https://webmademovies.lighthouseapp.com/projects/65733/tickets/523-decouple-timeline-from-dom-widthheight maybe?
I wonder if it makes more sense to put this if inside the trackEvent.update. And if ghost, return early.
Random double slash.
`params.headers` is used often, you can create `headers` variable and use it.

Please maintain proper spacing between `if` and `(` or `)` and `{'. Also, look at indentation and line width (80 chars).
Why wouldn't page respond to :should ?
What is a designator?
这段描述英文版里没看到？
对齐，was -> is
only copy the templates if really requested with -t
maybe a brace at the start of the function, and the `Future.join` inside?
I say that we just do not allow ref/assign for now - since they are likely to be really slow anyways, and then you have to worry about the whole general indexing stuff.

This facility should only be reserved for passing to solvers, Factorizations and such, where the property can be used for efficiency.
It is recommended to declare all variables at the top of the function due to hoisting and to prevent the illusion of block scoping - for example, https://fedorahosted.org/katello/wiki/JavascriptCodeConventions#VariableDeclaration
Yea - seemed like it was since there is an info action.
does/should this information also get displayed to the user?
Please revert this change as seniority is an internal representations that should not be displayed in the views (the client doesn't know about our seniority counts).
Since this line is commented out, it can just be removed.
These assertions are not testting the behavior of the handler
It forces the longPolling's ajax connection to error and pass "foo" as the reason, the default error (empty) is "abort" which we handle as do not attempt to reconnect.  So by passing foo we mimic the behavior of an unintended error occurring, forcing the transport into reconnecting.  I'll add a comment for that.
Some rewording: Fixed template method in MultiStateAction to return the default template when none has been supplied to the object.
Here you can use

```perl
my $row = $domainsModel->find(domain => $hostDomain);
```
to fetch the domain row and avoid the loop. If it does not exists, undef is returned.
True, forgot that method.

Thanks!
This would set ALL images on the page to these dimensions.  Not a good idea
Shouldn't be in the pull request
Don't need any of this.
Cool, let's also remove it from bitcoin-qt.pro, then
Shouldn't we do the same for QMenu?
ACK, the naming should be more clear StartShutdown or InitShutdown()?
It's needed since the AskForBlocks code is now outside the code for the version message. It runs immediatelly after it, so for all the elses to work this line is needed, otherwise the unknown command section is executed.

Sent from my Nokia phone
-----Original Message-----
From: Philip Kaufmann
Sent:  17/05/2012 16:36:21
Subject:  Re: [bitcoin] Parallel block download (#1326)

> @@ -2396,6 +2393,21 @@ bool static ProcessMessage(CNode* pfrom, string strCommand, CDataStream& vRecv)
>      }
>  
>  
> +    // Ask a connected node for block updates
> +    if (!pfrom->fClient && !pfrom->fOneShot &&
> +        //(pfrom->nVersion < NOBLKS_VERSION_START ||
> +        // pfrom->nVersion >= NOBLKS_VERSION_END) &&
> +         nAskedForBlocks < 8 && !pfrom->fAskedForBlocks && !CaughtUp()) // TODO - tune
> +    {
> +        nAskedForBlocks++;
> +        pfrom->fAskedForBlocks = true;
> +        printf("initial getblocks to %s\n", pfrom->addr.ToString().c_str());
> +        pfrom->PushGetBlocks(pindexBest, uint256(0));
> +        NodeSummary();
> +    }
> +
> +
> +    if (strCommand == "version") ;

What use has this line?

---
Reply to this email directly or view it on GitHub:
https://github.com/bitcoin/bitcoin/pull/1326/files#r839048
I'd add:

```
Only used if ``maxIsSky`` is ``true``.
```
Is the goal of this to make the facility's parent a market? The comment for this function makes it sound that way.
So can You test it out with malicious content? And trying to encode it or something?
We do not suffix with Interface because we do not describe data structure type on class name. It doesn't make sense.
Otherwise, we'd be forced to have FooAbstract or FooClass, all debatable.
SVG is missing
I think git URLs usually have `.git` on the end of them, and it might be nice to strip that part off as well.
Is `line.replace(',', ' ').split()` more readable?
"effect of" or "effect from" probably, right?
in general the code is riddled with sleep()'s. This makes things break a lot.

I'm partially at fault for that as some of them were actually mine. I think we need to change a lot of stuff into retryables.
added a test that asserts that the jclouds base 64 impl isn't url safe and that the one used is
if you'd like to abstract out the impl of providing time, you could use
Ticker, and in joda, bind it to Instant.now()

:)

http://docs.guava-libraries.googlecode.com/git/javadoc/com/google/common/base/Ticker.html


On Tue, Sep 11, 2012 at 1:07 PM, Andrei Savu <notifications@github.com>wrote:

> In apis/cloudfiles/pom.xml:
>
> > @@ -54,6 +54,11 @@
> >        <version>${project.version}</version>
> >      </dependency>
> >      <dependency>
> > +      <groupId>org.jclouds.driver</groupId>
> > +      <artifactId>jclouds-joda</artifactId>
> > +      <version>${project.version}</version>
> > +    </dependency>
>
> No. But I think I can replace Instant.now() with System. currentTimeMillis
> and remove it completely.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/jclouds/jclouds/pull/829/files#r1580146>.
>
>
Hi @rchampourlier,

This looks really useful!

And, yes you can update the pull request.

Finally, using map and join:

    matches.map do |match| 
      "  - \"#{match.expression}\" (#{match.block.source_location.join(':')})"
    end.unshift('Ambiguous step definitions').join("\r\n")

Not really sure there is much difference, although with map/unshift/join the operations are semantically distinct.
This of course was the actual problem. Thunderbird does not have gBrowser.
Formatting of this could be a bit better. I think JSHint throws an error about this. You should move this string to the same line as `var content` and surround the string with parenthesis.
I wonder if these and the ones below should just be in settings/base, so they could be overwritten easily (then again we'll probably push code to overwrite these anyway).
i should've tried that! thanks
I think we should reduce logic branching in this doc wherever possible - if someone edits the config an it's already set to UTC and they are unsure whether or not to restart postgres, it's certainly not going to harm anything to do so.
I assume that savegames in ScummVM are not compatible with the original ones, right? If this field is added, there needs to be a save game versioning mechanisn, so that older saved games work
My bundle is complete... hmmm, don't know what could be happening
Sadly binder.bind(...).toProvider(...) only accepts a Guice provider. We're going to stop using this method very shortly.
`@throw` -> `@throws`?
@vicb Shouldn't it be a `DirectoryReource` instead?
DS_Store has not been changed
draggable是由tangramDom对象和draggable对象组成，draggable对象的组成方式是否可以考虑采用baidu.base.Class来创建，这样地事件的处理和对象销毁会带来很大的便利。可以评估一下
Тут тоже отступы поправь
could you comment on the intent of the following methods ?
very minor issue: There's some whitespace at the end of the above two lines (and perhaps elsewhere?) - can you remove that please :)
```fo``` should be ```for```
Ember.identifyNamespaces();

Then search Ember.Namespace.NAMESPACES for instanceof Ember.Application, then search props of the Application instance for the view or the view's Class.
This description doesn't match what the test is actually doing.
this flips the icons meaning on data lists, because both are visible they dont meen change the state from that , but rahter chagne the state to that.
Those are internal - but we can add the deprecated ConnectorService class
These do allow null, if invocation stats are disabled, right?
There is going to be a much bigger task to overhaul this to properly separate the required interfaces from the access and supplied implementation(s) of the interface - however as it stands the ejb subsytem currently depends on the security subsystem  so if you add an interface to the ejb subsystem the security subsystem will not be able to implement it.
I ended up using --deb-ignore-iteration-in-dependencies because it was already there (and used nowhere). Your documentation says: "(deb target only) For = dependencies, allow iterations on the specified  version.  Default is to be specific." so I left it at that. If you want the default behavior to be NOT specific like you say above. Well, then this is wrong ...
Since the default for the `appendTo` option, which is used append the `menuWrap` element, is `"body"`, we shouldn't have to set a z-index. If the selectmenu is used inside a dialog, it needs to be appended to the dialog, and the z-index on the dialog takes care of stacking. Is that correct @scottgonzalez?
This wasn't being checked for `that.close` previously, but you're checking now. Is that the correct behavior?
Im keeping it in order to ensure that 'adding' a log in our DB it's done and the DB is closed after doing it. I will keep it until the rest of code related to 'recents.js' will be working.
we can have a variable extract only for the `filter` function (using it 2 times)
I believe that if DEBUG is active you don't want to push install-media-samples by default, otherwise you want. We need to make DEBUg=1 works on the phone.
Weird code path: calling `hide()` from `show()`.

If you really need to do this, create another function called `toggle()`
Sorry! :S
@etiennesegonzac Is this going to conflict with your latest path that use manifestURL instead of Origin?
Util.add_trace- -> Util.addTrace
Here, before changing the style, we should add the following:
`if(!messageDOM) {
   ThreadUI.appendMessage(message);
   MessageManager.getMessages(ThreadListUI.renderThreads);
   return;
}`

This is due to in flight mode the messages status goes directly to 'failed' without going through 'sending', so in this cases `messageDOM` does not exist. That's why only in this scenario we create directly the DOM element with 'failed' status.
@borjasalguero  here you have some code for preventing the flashing empty images and also removes the unnecessary space inside the lists when there is no image
Need brackets for this segment,
```js
if (type === 'text') {
  return 'latin';
else {	
  return 'latin-prose';
}
```

or maybe 
```js
 return (type === 'text') ? 'latin' : 'latin-prose';
```
I don't think we need the -moz- prefix on transitions anymore
When we nuke all the cards at once, this leaves us with activeCardIndex === -1.  Our sentinel value for not really having an activeCardIndex is null right now, so this does not get normalized and we can end up sad.  This manifests when creating the first account or adding a new account.  Besides blowing away your database, you can get to add a new account by going to the settings (gear icon from folder list).  Since null was an arbitrary value anyways, we could change the sentinel value to -1 or whatever seems good to you.

also, s/affet/affect/ in this comment
Because it has moved to connectivity.js ?
Is there any reason to not use a regular JS object and then do `JSON.stringify` on it?
Great! I'll change that
> Please remember that browser is string frozen and any changes to this string will need to go through a bugzilla bug and communicated to the localizers. Thanks!

Absolutely :)
What is the expected format of the deliveryTime if it isn't an instance of Date?
which is internal and should not be used by code ;-) I looked it up and found this solution on stack overflow
Shouldn't this be a joinWithTiny? You're doing a joinWithTiny later on with the full sampledCounts, so presumably sampledRight is also small enough...
OK.  So then given this, I guess it could be written to be significantly more efficient, but I suppose as a quick and dirty test helper, that's irrelevant.
This is the literal thing it returns, but as a caller of this function, this isn't useful. If I'm calling `get_entity`, presumably this function returns an entity. But I don't see an entity being returned, so this needs to be explained better.