labofsahil/github-event-dataset-2012 huggingface dataset:
Have you tested whether this works? You may actually need to call response.getContentType() here, but I'm not 100% sure
me neither. Could you document it please?
But yes, since we could fix it, may as well.
Got it. will patch
This doesn't look right to me
Add comment: // In delay slot.
Do we need these lines, since they're already being declared and defined in update.h?
Añadir función que devuelve la cuota mensual.
Okey! merge the Sh*t!
Would be happy to merge a pull request if you rebased this on the current master and made the appropriate updates. :)
Ok, then I request review from @o11c and @bjorn for that one.
In your example appVPath was not changed. As a result, appVPathLen still evaluates to 6. If we assume the following pre-condition:

current_path="/vpath/page2.aspx?a=b"

The action variable evaluates to "page2.aspx?a=b".

I think, this is exactly what we want.
Please bump the gem!
On Sat, Feb 18, 2012 at 12:08:03AM -0800, Nico Williams wrote:
>

> There's really nothing we can do here but ignore ftruncate()
> failures (and fclose() failures, but we practically always ignore
> close() errors too).  But I'd rather use a #pragma to turn off the
> warning, quite frankly, than to do something that the compiler has
> every right to complain about.  Ditto wherever you just assign a
> return value but don't check it.

I am of half a mind to just turn off all these warnings for rsh/rshd,
telnet, popper and all of the legacy code.  This is the code that is
the most difficult to fix as there are lots of cases where it is not
quite as clear what we should do on errors.  And, IMO, rsh/rshd/telnet
should probably be broken out from heimdal and maintained (or not
maintained) elsewhere.

--
    Roland Dowdeswell                      http://Imrryr.ORG/~elric/
Calling the function on the object means we can use 'this' in the function (also for update below).
Ah, I did wonder about that but it seemed to work ok on mine. It should be
easy to move it out back into the index page.
On Mar 1, 2012 10:17 PM, "Len Feehan" <
reply@reply.github.com>
wrote:

> This kinda fucked up the database read, doesnt look like a db session can
> be opened in an include header then accessed from the main file.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/peteretep/ecom/commit/64b47d011313d76ff0245a7b8a243f92dd5650b4#commitcomment-1033205
>
Is there anything wrong with using a system location like `/Library/Developer/Xcode/Templates/Application`?
What's so bad about a dict (apart from that it can contain almost any type (int, string, ...)? The reason why I made it a dict and not a list of keyword parameters is that there will be more configuration parameters in the future and they can be grouped (like in this case there's a group "notifications"). But if someone has a better idea I'm happy to adjust it if it makes someone else's life easier.
g++ 4.4 doesn't have nullptr yet :(
Salut Seb,

Il manque un '='. La ligne `if name = 'main':` devrait être remplacée par

`if name == 'main':`
This needs to go before “Make libutempter optional”.
I have a fork of the docs, just haven't worked out how it works yet :P
ClasiFIFIelds? jaja
[self bounds] is a CGRect, although compiling for Mac OS 10.7, 64bit doesn't complain about it...should we keep following NSRectFromCGRect(...) convention for its conversions?
Thanks for your comments. I feel this is no-where near a PR, though and I probably have not time to finish it until march.
When is your sprint again? Your issues are probably not *that* related but maybe I can still give you some input on my experience.
Also, add a bit at the end.

"You might ask yourself, 'why only 50% of available RAM?' and the answer is that the other physical RAM will be used by the operating system, the Erlang VM, and by the operating system's filesystem cache (the buffer pool on Linux).  That filesystem cache will translate into much faster access times (read and write) for your cluster.  A second reason for the buffer is to avoid paging memory to disk.  Virtual memory helps prevent failure due to out of memory but the costs of paging memory to/from disk are very high and cause noticeable impact on cluster performance."
versteh ich ne...

es soll doch den Dateinamen von nem Pfad-String rausfinden. Da isses doch sinnvoller einfach das File zurückzugeben (wenns keinen Pfad dazu gibt)

Ich mein das so:
/test/123/file.txt -> file.txt
file.txt -> file.txt
The options variable is not being used in this method, the arguments of this method are passed to the `all`-method in which they are transformed to a `Query`.  It can be removed - all tests still pass after this.

Of course, the same refactoring can be done in the `has_no_selector()`-method.
@michalpolko can you point me to LoS check code?
Luckily I do, [here](http://dl.dropbox.com/u/8097754/inpout32.zip) it is.
Жесть! Перенеси в класс.
@param mixed $name alternative model name to be used in froms. Set to false to use model class name.
There is an error in toon.cs in the UPDATE for toon table:
{1} is present twice, so arguments are saved in the wrong DB column.
15. satirda yazili adres sourceforge.net adresiyle degistirildi
@dorey @Topol  we should chat about whether you think the following is the right behavior
- you check timestamps+imei with other xmls to see if this is a potential duplicate (if (imei,start_time) pair matches, the instance is the same one, even if data=xml file is different)
- if (timestamps+imei) match, then we should discard one form and keep one. Keep the one that is more complete. If same completion rate, then keep the most recently submitted one.

(Edge-use-case: surveyor has data half filled out. His form gets backed up and submitted to formhub. Then he goes back out into field, changes some data fields, and submits a completed form. We should keep the later one. However, if the backup system happens to do a third submit, the second submit (more complete) should be taken.)
i've just improved [README](https://github.com/zenorocha/hub.me/commit/8f075c4d7a44f37382bd2f9cfc4bebac2be1532a)
может создавать и редактировать какие-то категории или управлять какими-то модулями. -> может создавать и редактировать определенные  категории или управлять определенными модулями.
I thought you were going to delete the org.hibernate.infinispan:3 module and let it be part of the org.hibernate:3 module.
does this need to run in AMO code?
That's really odd---I've started up probably hundreds of devices since I put that code in.  Are you using the current sage patches?  I think I may have seen this problem when I had outdated Sage patches applied.
Probably shouldn't do the development on Windows, copy it to Linux, and commit. Makes tracking changes pretty impossible.
@JMobius, what do you think of using Java properties built-in replacement mechanism, i.e. insert ``{0}`` into your descriptions where required and running them through ``MessageFormat.format``? 
It should save us some specialized code.
Feel free to send a pull request, the code really needs that kind of cleanup :)
No problem. I cloned Narrow's repository and added the save on load fix, pom fix, help file fix, and item index.  It works well with both push/pull -- so something about your fixing the potential reworking the filter code breaks push/pull. I'm fine with this build for myself, but thought I would let you know if it helps to debug.
You are right, there is no SMSG_SHOW_MAILBOX in 2.4.3. According to Schmoozerd this is only used for argentum tournament pets. Reverted in 65bf90a
line longer than 80 symbols
I don't think we should be adding CACHES into settings.py. But to the settings_local.py on the servers. I think local devs could use the python memcache lib.
如果要是更新或是讨论还是到我的那个分支去吧，等都形成一定阶段了再 并过来。以后有修改或是意见可以到我那个里面去讨论，
Great. Really great.
is this reading from attrs.xml ? Where is the test that's verifying IndexedAttributes_bogus, IndexedAttributes_dude, etc.?
Not sure we will be able to drop this. I forgot the reason why we dereference () functions, but there must have been one. So I propose to leave old version in comments to be resurrected if the problem resurfaces.
Stray .DS_Store file snuck in with this commit.
I don't believe we store the password... just the modhash right?
We know. cyanogen has a support ticket open with github as it seems to be with them.
just told you I speak engrish :p the use is supposed to be sure.
А boost'ом? ;)
Estás seguro de eso? que loco
I see. Smart!
Instead of having the rather odd double ID selector; let's instead refactor the way the HTML works so that we can use classes instead. You're free to edit it around to the ideal way and we can tidy it up later.
"Logger" makes it sound like a Logger class, rather than an instance. I'd expected the createServer option name to be "log".
Where did mhansen suggest something about `git diff`?

The reason the rebase command didn't work is that you didn't actually fetch branches from `upstream`, just added it as a remote, so `git rebase` couldn't interpret `upstream/github-master` or `upstream/master` as branch refs. Do this:

```
$ git remote update upstream
$ git checkout lfun
$ git rebase upstream/github-master --onto upstream/master
```

I'm not really worried about the patch including the README.rst, but eventually I will be deleting the commits on the `github-master` branch, and if you have permanent commits based on them they're not actually going to go away but will instead be preserved in your commit history.
I am not sure. I think Tejaswi added it.

On Wed, Feb 29, 2012 at 9:13 AM, anand kishore <
reply@reply.github.com
> wrote:

> whats this for?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/semanticvoid/reDygest/commit/da4b5aafb617e0ec29f7ff17d517cab31e1a9036#commitcomment-1027121
>
This is a glimpse into my twisted way of thinking.  I've modified this as you suggest.
What's about 
<pre>
~~~.json
{"teapot": "brown","inside":["milk","sugar",23]}
~~~
</pre>
Yep, I originally didn't do any indentation for some reason.  I've fixed this in a recent patch.
Hi Ivica:
You've changed the imports order (maybe something related with your IDE configuration?)
Please recommit with the correct imports order
Thanks!
Juan
Okay, I like the idea. But please rename generator to more common name `install`:  `rails generate imperavi_rails:install`. And just make one another commit. Thanks!
@outsideris 노드로 만들꺼 있는데 안도와 주시겠어요?ㅋㅋㅋㅋ
Warum lässt du das nicht Hardcoded? Passt doch so wie es war
See?! Told you that the ConfigureJoin method was hard to grasp ;)
それが良さそうです。よろしくお願いします。
What? Clearly should have been "Derpy Whooves". Would have generated much more interesting worlds.
Nice! 10q
There are calls in jlib, that could be used here to fetch physical RAM and use as basis here.
but they are based parsing /proc/meminfo, not sure portable
sysinfo looks like portable method to use.
Done: https://github.com/symfony-cmf/symfony-cmf/commit/126a07fabfca5b1408ab3b2e4d56695c72b36e24
Thanks!, but question, Does this fix the current issue with some mobs getting stuck in evade and following the player indefinetly?
\src\server\game\Battlegrounds\ArenaTeam.cpp(467) : error C2065: 'slot' : undeclared identifier
Ashley, 
Those freads had me baffled, but from what I could tell, We first determine number of bytes in a file/object. "size"=num_of_bytes. Then we malloc "size" bytes for buffer. So we want to read 1 byte at a time of "size" bytes from the file into the buffer. My assumption was we are transferring bytes from a file into a buffer. So we want to make sure we read "size" bytes from the file, otherwise an error occurred.
@buzzb0x
The French translation got a bit roughed up in my merge, can you have a look at it?
SHA: aa936dcc571455bfa4133ef3a621d494161b32f4
please add also the weather plugin to the plugins.conf

thx!
Refinery with dependencies is not loaded during specs ? I thought dummy app is used for testing engines now.
Looks like README hasn't been updated.
Aren't you missing a \ at the end of line 37?
Damn. Totally forgotten to apply your coding standards.
And we don't want to hide the element because we want to retain the positioning properties of the SELECT for the dynamic INPUT created.
5>D:/Patch/SkyFireEMU/src/server/game/Entities/Player\Player.h(33): fatal error C1083: Не удается открыть файл включение: NPCHandler.h: No such file or directory
5>D:/Patch/SkyFireEMU/src/server/game/Entities/Player\Player.h(33): fatal error C1083: Не удается открыть файл включение: NPCHandler.h: No such file or directory 

Cmake re-run
Checked and reviewed
I'm not sure if we are really adding a new way for them to be created - this is just a refactoring. It does the same thing it was always doing but this time in its own method. The point of this is so that one could subclass BrowserIDBackend and define your own create_user method.
Did the bot work?
oh so very true! Was really late when I finished that going to fix it immediately.
Whoops. The line above it should've been commented out or deleted. Oh well.
testomment
_                  
( \                 
 ) )                
( (  .-""-.  A.-.A  
 \ \/      \/ , , \ 
  \   \    =;  t  /=
   \   |"".  ',--'  
    / //  | ||      
   /_,))  |_,))
This is a lie; there's no builtin logic to treat 'template' as a path for template-loading. The string contents themselves get used as a template. Should we add logic to treat this as a template filename by default and load the template when needed? Or change the docstring to reflect the actual behavior?
yeah, I think you're right.  I need to package up strange_case for pip, i'll include markdown2 as a requirement.  I'm gonna rename the tags, though, to be just "markdown" instead of "markdown2".

I think I'll do that this morning.

more photos!


On Feb 25, 2012, at 2:59 AM, Rocky wrote:

> It would be nice if these were built-ins, I seem them being very common to sites.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/colinta/rockymeza.github.com/commit/91c264a5054d25f5d89aeed1e8e8990ad47d4804#commitcomment-1010565
maybe you should run this only if video_id isn't None?
> A test to confirm IncorrectZkpException is thrown when g^x^q % p = 1 would be nice :)

Filed Bug 719856 - Improved test coverage for J-PAKE.
Maybe some tests would be good considering all the crypto magic being done here :)
f+ otherwise
I think add item should have it's own test.
Hey Fosser, i'm trying to learn from your git, i want to understand what is the impact of bits_per_pixel on gtablet screen, does it looks more colorful or more dense? Sorry for grammar mistakes :P (I'm from Brasil)
I'm glad folks seem to have hammered out a roadmap for this, but the example is still broken right now.  If I can't use what currently works, can I at least remove what's broken?
ah ouais j'ai peut-être trop pensé "pour développeurs" ici et pas assez "produit utilisable". (: Je vois ça ce soir, avec entre autres un loader ajax pendant la connexion, ça peut etre relativement long étant donné qu'on dépend de scripts externes.
numintindiuqueelefalo também, :x
fixed the wrong message appear in console when a use do: `rake import_products:install`
Indeed. I never checked before, but "adb devices" is showing the proper usb serial for my device. The problem is definitely related.
Do we really need it on the top of the page?
Actually, I don't think it's a problem in this case.  The tryPutMVar is for waking up the outer thread to kill the worker threads.  If the outer thread prevents a worker thread from calling tryPutMVar, it's okay: the outer thread has already been woken up.  Only if the worker forks a thread to kill itself would this be a problem.

Do note that serveLoop itself is not async exception safe.  It shouldn't need to be, since I never kill it.  Nonetheless, I wrote an async exception safe version [in another branch][1].  Let me know if you spot any holes in it.

 [1]: https://github.com/joeyadams/haskell-chat-server-example/blob/async-safe-serveLoop/Main.hs#L126
I already pulled it :P ! If you can make an other commit that would be great.
Sounds great, just let me know if you have any questions or anything I can help out with.  Thanks!
Rather than duplicating these queries and the FIXME from the lockss() function, could you put them into functions in this handler (temporarily), or better yet, move them into the IssueDAO?
The latter is better.

I would prefere, however, to write in `main`:
```c++
ISystemOfLinearEquations* sys = new LisLinearSystem(100);
```
So `LisLinearSystem` is derived from `ISystemOfLinearEquations` interface. My question is why do we need a function, which name is starting with "create"?
I'll change them to call checkError() for each line. As MPI part is not tested and not used right now, I'll remove it.
the source of the problem is PyPIProxy setup living in the pth file.
I'll dig deeper on this though and see what I can come up with.
Okay. I don't use windows so I had no idea.
Why do you create an dialog here instead of doing it in _create method of eventHandlerView?
Missing ";" in this line
Are we actually logging anything with Inspektr?
That's because the `%s` of a type is `<type xyz>`, but it doesn't hurt having it explicit.
Maybe mention that phi is using logarithmic scale (like dB)? I know it is in the article, but I expect that most people will not read that :)
Any member in Joining or can we be more precise?
fixed, using system.eventStream instead, much better, thanks
Is there semantics required here for ready to be at the front and initialized at the back of the sequence?
I don't see that -- git and svn don't have to be essential for that, as we don't selfupdate during bootstrap. And once the bootstrap is done, the user can simple install them as normal to do a selfupdate... In fact, "fink selfupdate-git" could even offers the use rif it finds no (new enough) git to install the git package:

  $ fink selfupdate-git
   .... selfupdating ...
   no 'git' binary found. Should I install package 'git' and continue? [Y/N] 

Personally, I think this GitPath etc. is overkill. I just see no good excuse for it. There are no special optimised versions of git or svn either. If we ever find a need for GitPath, we can add it that day, until then, let's keep the code simple :)
model : obj or model : object
No way, I didn't realize you could set the side effect in the decorator. That rocks.
I can see how this would affect the 32-bit version.  In addition, I think all of the _PTR variants should add a `.toPointer()` method, which further makes clear that their value can be either integer or pointer.  `rkey.getValue().toPointer()` would make this particular expression more clear, and avoid `new Pointer()`, which implies something different.
hmm, is this an instance method because of cardname?

very impressed that you achieved this with such non-invasive code!
tests that catch any of these errors would rock!
use a val otherwise it will be computed every time
I can do this later if you don't. But, i think it would be cleaner:
````
#ifdef LIBAVFILTER_SUPPORTS_BUFFERSINK
  frames = m_dllAvFilter.av_buffersink_poll_frame(m_pFilterOut)
#else
  frames = m_dllAvFilter.avfilter_poll_frame(link)
#endif
  if(frames < 0) {
     CLog::Log(LOGERROR, "CDVDVideoCodecFFmpeg::FilterProcess - poll_frame");
    return VC_ERROR
  }
This should be bool, not boolean
Why `0.1`? Is this completely arbitrary? I'm thinking something between `0.1` and `0.5` would be good.
change to mName, mInternalName (this is not perl), and mCustom.
the original code ignores \\Draft and \\Recent, whereas your change will not. what this matters, i'm not sure.
May be move it to the prototype?
\ before Nette is not necessary imho.
STYLE: format consistancy with out_file
typedef qreal
Typedef for double on all platforms except for those using CPUs with ARM architectures. On ARM-based platforms, qreal is a typedef for float for performance reasons.

Стандартный Qt-ый typedef . Не понимаю, зачем  нужно менять название.
On second thoughts, I find this addition problematic.

As someone reading this, I don't know what does proper configuration of the email server mean. Is it the basic config to enable the server to send emails OR does rails require some special config in the email server to be set to raise delivery errors other than this config option here? 

If we go about clarifying that in detail, we may have a small paragraph here, which is not the purpose here and which should go into documentation (guides or api).
Yeah. The issue is that the local variables are issuing warnings because they're not used in most of the partials inside the tests, and since a warning is issued, this equality fails. Anyway, you're right, this is more like a workaround than a real solution. And actually, I have a branch here to stop generating some not necessary local variables, I'll check to see if that solves this issue as well and push later this week.
So I'm not seeing a clear path to implementing this. In this code the `QueuedMessage` class is namespaced under `AsyncMailer`. The only time the instance of `QueuedMessage` is being created is to interrupt creating an instance of `ActionMailer::Base`. Moving the `default_queue` method to the class of `ActionMailer::Base` makes no sense because it defeats the purpose of having multiple queues. The same goes for moving the entire queue system to the class level of `QueuedMessage`.

I think this goes into a bigger issue of the `Rails.queue` not having explicit names.
Ah OK, makes sense now. Thanks, @spastorino & @josevalim !
I've updated the code but not sure if it's the best solution in this case. I need some feedback /cc @rafaelfranca
"run" is almost always a poor choice of method name, in part because of the pervasiveness of java.lang.Runnable.  Can we pick something else.
Don't know whether it's desired. I'd assume yes. Yep let's update the docs.
Well, the issue is that the function returning a combiner really has to create a new combiner each time it is called, because each processor must get its own combiner. So, we cannot just create a combiner once and use it later - we need multiple instances.
I know that checking this that way is not the cleanest possible thing, but creating and discarding an empty builder object is a small price compared to constructing a parallel collection, which assumes there a lot of elements in there.

Yep, I saw the new `to` :) - I can refactor.
Could you please give an example of a static this check excludes?
I'm not real sure what's going on in this method, but this is a very inefficient way to say "val (c1, c2) = (o1, o2)".  Since o1 and o2 are the owners of something (seeing as that's how they arrived) neither of them is a ModuleSymbol.  Let me make that concrete:

    scala> PredefModule.moduleClass.asClassSymbol.asType.members.map(_.owner).toList contains PredefModule
    res1: Boolean = false

Every one of PredefModule's members is owned by someone else.  (Specifically, by the module class.)
Should be ``from _decomp_qz import *``.
This is going to come-up again on the NumPy list.  I'm not excited about being dependent on a slow Python-only API to get this information out of CTypes whenever we need it --- so we are going to do something like this in NumPy. 

One thought is to do this in NumPy and then have SciPy rely on the NumPy API.    This seems like a reasonable path forward.
Reason this is a class method?
Oh, my bad. I read the end of the `reset_spree_preference` call to being the end of the `before`... but I was wrong.

If this code is inside the `before` block then that'll be fine. I'm going to look at merging this one after I'm done with #1331.
In my head `condition` here is, as Ronan points out, a verb. This certainly isn't the standard usage though. I'm open to other suggestions.
You should check for the type of `g` and raise an exception if it is wrong.
so should this file be ignored?
It's not a big deal, but you could replace the second if block with the third argument to reduce().
Once this gets into master, it's not really experimental any more, so it must conform to our coding standards, assuming we have any. So yes, I'm -1 unless this is changed (or unless we explicitly decide that anything goes as far as code formatting is concerned).
Would returning S(0) here be better, to get a SymPy object? I've had other difficulties in mechanics with returning integers.
Hm, okay :-) I can vaguely remember reading something like that somewhere; nevertheless, I've only rarely seen this particular convention followed in SymPy.  That's just FYI; feel free to discard my remark.
In master `generate_schreier_sims` does not give the elements in lexicographic order either

```
>>> from sympy.combinatorics.permutations import Permutation
>>> from sympy.combinatorics.named_groups import SymmetricGroup
>>> G = SymmetricGroup(4)
>>> [p.rank() for p in G.generate()] == range(G.order())
False
```

although they are more lexically ordered than in this PR. In both versions it gives its elements in `coset_rank` order.
Hm, apparently the function works both ways in the sense that it does not matter in which order I specify the pattern and the expression, right?  Then forget my previous comment about `expr` and `pattern`, but consider the point about explaining the arguments in a docstring stressed one extra time.
Since pagination object is still of class `CPagination` or its children it's better to leave it as `CPagination|false`.
In any case no need for a changelog if there is no bug fixed or some new method/property added.
Maybe better will be "now allows to register multiple meta tags with the same set of attributes"?
@klimov-paul yes we were talking about this, for comments this issue #1725 :) But may be in 1.1.14 it must be added or if @resurtm have time for 1.1.13 it would be great.
This inClose() business does not seem to be necessary.  Can it be removed?
So do I, actually.

I copy-pasted those from the node-gyp README, which did it this way.

I'll change that.
What makes you think that? Unless there is data to back that up, I would rather keep it as is.
どうせならStringが `null` の場合考慮して、 `NullPointerException` もcatchするとか
I'm not sure it makes sense to always alert in this case.  The Strategy is there to do the right thing when this happens.  If someone wants to alert, they should put it in the strategy.  If the strategy handles it, an operator doesn't need to know about it.
No where else in the documentation do we use the second person formal "i.e. One." because it sounds stilted, and isn't necessary. It's particularly awkard in this case because you use the informal later in the document. 

In some situations you may want to copy a database from one MongoDB instance to another. For example, you may use these commands to support migrations and data warehousing, seeding test environments, or to help perform backups.
For more information on the relationship between bits and precision see the ...
django database API (i.e., `filter`) handles sql character escaping.
Will probably just factor it out to its own method. I'll keep your comment in mind.
Well, I tried it, but in fact, there always exist at least one style and at least one language, so the condition will always be true. I think current code is the best solution... Or I can use `db::sql_query_limit($sql, 2)` and check if 2 rows have been retrieved. If not, we are sure there is only 1 language/style available. But I do not consider this solution being clean, bugfree and understandable, so this is why I think current solution is the best one.
There is an unstated assumption that categories have lower id than modules. Therefore, prior to the update this query retrieves a module with the lowest id, and on repeated database updater runs this query retrieves the category for said module. The subsequent left/right comparison is supposed to determine which of these happened based on left=right-1 for category.
So, move out provider and locator out of constructor and initialize them separately, then pass as parameters to style's constructor?
Also an unecessary comment, that's exactly what the if condition says. Comments should add additional information to help understand code, not rephrase code in plain english
Indeed, the docblock should specify that it still needs to be escaped to be insertable as a string.
I noticed that but it is how it is done for phpbb_functional_url so I just kept it.
Autoloading for mocks is now available in develop. http://tracker.phpbb.com/browse/PHPBB3-10973
obtain_attach_extensions() takes the forum id as a parameter.
I don't believe <&lt;> is a valid tag in xml. This requires additional validation.
Has to be handled by the convertor.
infof needed here instead of just info?
Din nou, aici, liniile ăstea două merg interschimbate, pentru a evita check-ul.
Is this test supposed to be a superclass?
Three ... three blank lines ... ha ha ha!
Missing `private` access modifier
Unless there is a compelling need to maintain multiple versions of a package, we usually just have one. Could you rename 4.5.5.nix to default.nix?
I am not using Elmasse's library. His wasn't working properly with our design. Saptarshi had mailed me an edited version of his library and I am using the same.
Same response, a shorter copy, in keeping with the rest of the file.
Why do you do this, and then construct a tuple later on? Could you pass ```cube.coord_dims(lat)``` straight to ```iris.util.broadcast_weights``` without this added logic?
rows -> columns
%.2f
No I don't remember I changed this...
Ah, it's specifically a DataMigration, missed that. Using SchemaMigrations you have to wrap data manipulation in ``if not db.dry_run``.

So, everything's alright, carry on.
When setting sizes, don't use pixels.
Use cls instead of Asset here, otherwise inheritance will break this.
something more like `allocate a default-constructed module/agent that will auto-destruct at simulation termination`
What's the point of this query? I don't see anything in the view which could delete users (and even then, why should it delete users…)
So I check for : at the second position now.
You could get the version info from openquake/__init__.py, instead of writing it again here.
s/prfile/profile/
https://github.com/hub-cap/oizys/blob/master/generic_package_scripts/nova_diablo_scripts/debian/nova-common.postinst
You shouldn't call `to_a` inside the check_error block. The error status should be checked before any more work is done.
Doh! ;-)
Good to know. On the move today, feel free to hack in the branch ;)
Nice, this is so much better then buried var initializers between assertions.
Not a critical path, size it is ;)
style:

    if ( typeof define === "function" && define.amd ) {
        define(function() { return Sizzle; });
I think it would be better if people learn to do `console.log(Todos.findOne({}));` to inspect how a document looks like in an example.
Doesn't hurt that we implement this, but do we actually use mute from any frontends. Currently I don't even think the playback/gstreamer API we created has it, should that we fixed?
1 is fine for you. In prod we should have more than 1.
Nit: This should line up with the "not".
Blank line before install.
oui ca commencait à me souler à force de tester le client ...
Might be better to use the constants in `vumi.message.TransportEvent` (although we're not really that consistent about using them).
I think you're missing a `yield` -- even if the test passes.
Ah, je les ai carrément oubliés !
Bon, je mets *daemon* partout, comme on l'a fait pour *commit*, le nom.
I think that might be more appropriate for the SSL config guide that will be in the platform guide. When that section gets written, it will probably make sense to revisit this section and adjust for overlap.
Ok, I found a much better solution, matplotlib allows to set only the ticklabel font size. Will commit soon.
Removed unimplemented tests
nit: Interface --> interface
@stephenh yeah the GClass.addEquals() method shows that your approach has legs! I dig.
this was what required the code style change
Not all reply messages styles conform to the others used in other plugins (the bot talking about himself). This feels a little bit inconsistent, what do you think?
Ok - please add a comment to this effect, since otherwise this activity appears anomalous! Should we move to a parser which isn't broken this code would need to be removed. Can you find a reference to the relevant bug report on the parser, or else construct one?
Cheers
Please fix javadoc indentation
Please don't import *: import classes by name
Android audio fallback conditional should stick around. If we're using the video player, the other controls shouldn't be shown, as they'll never be visible when the audio is playing.
In HTML5, yes. In PhoneGap, currently none of these new functions work, so at present they should really all just bail out.
There's an extra `)` on this line that causes a JS syntax error
Typo ``reabir`` to ``reabrir``?
Probably should do the same for _getRelativeMenuItem() too
Might want to clarify that 's a _reverse_ sort (high to low)
copy-paste error: this isn't Go
Yes, this does everything... It sets up the supervisor section of ADM Reports. There are other ADM sections coming, and presumably other items other than just ADMReport objects in the ADM report sections for the future.
I missed this missing comma when I merged it.  Fixed now, but in the future, please run the gallery_manifest.json through jsonlint.com before submitting it.
This is more direct:
```python
yield kaa.inprogress(g.signals['connected'])
```
No, but not correct. `range` is again a function call, so you should indent it accordingly.
indentation is off for this block
Is it not reliable enough to just accept the object returned by the contructor above?
What about settings this as a `kDelayBeforeHidingControls` as the others in the top of the file?
Empty response is valid from some api calls.
looks like you got the wrong overlay here.
First I actually logged it as an error but than I changed my mind and changed the log level to info since it's not really a server error.

If you want though, I can log it as an error.
what does this affect?
There should be a space after the '#' character.
remove the cast!
Yes the prupos of this is to show good coding practices but we want to respond of one need too. Z-f.fr's forum is very old and we want to upgrade it with this forum.

So we must learn from what is currently on the different forum ...

Later why not doing a Stackoverflow clone ;)
The request was updated. Thanks.
not all OpenLayers Layer types have a url property, so check for this
What if instead of this change you do:

```python
def batch_permissions(request):
   try:
     return _batch_permissions(request)
   except:
     # handler code

def _batch_permissions(request):
   ...
```

It will make the diff cleaner
I've added allow-rfc2136 locally. If the value is 'no', we return RCode Refused without any message. Please let me know if Refused is the correct reply when we disable rfc2136.
For what it's worth, I don't think we should restrict the kinds of Git URL that are allowed. It's entirely possible that someone would host their own and provide public access only via SSH (that is, e.g. far easier than setting up HTTPS, and is arguably more secure). It's also possible that some services allow read-only cloning via SSH as well. A better solution, imo, is to delete .git/modules/Foo if installing Foo fails.

Regarding the PkgHub, etc. idea: yes, that's exactly what I envisioned as well. I'd like to figure out how to modularize Pkg a bit and provide hooks so that we can cleanly provide custom behaviors for different git hosts.
Thought it was used on another page, after double checking this is no longer needed and I will remove.
It is for debug, but useful debug.  Currently if the job->init fails you are unable to determine if it is due to object initialization errors (invalid ID or bad db connection) or permission errors (no user logged in or no user rights to job).  Personally I don't think the job->init should be overloaded to do two separate things that are unrelated (object creation and user validation), but I don't want to unwind the current code.
At the moment, 'language' is a plain string, and I’ve just committed a slightly different version of this API with a 'direction' property (read-only), see below.
In the three functions above, I propose adding ``.`` after the values to make them floats, e.g.

    if not -24. < hrs < 24.

because otherwise it has to always convert int -> float on the fly:

```
In [2]: %timeit a > 1
10000000 loops, best of 3: 144 ns per loop

In [3]: %timeit a > 1.
10000000 loops, best of 3: 118 ns per loop
```
Why are all these import statements commented out?
Should this be ``../../coveragerc``?
Like: "we did this, because with some OSes default style, text on the progress bar is unreadable" ;)?
Nice catch - this would indeed be much cleaner. Patch coming up.
What does changing the case of "visible" do here?
an AnnotationReader extending the parser ? The naming is really weird, especially considering the fact that Common already have readers
you should not use ``void`` as it has no meaning in PHP (if a function does not have a return statement; it will return ``null``)
@tijsverkoyen Ik kan eventueel AppKernel.php wel aanpassen dat hij ipv ApplicationRouting de Installer class aanroept, en deze dan hacken tot op 't punt dat ik een container kan doorgeven.
better rename nfds to nwatchers
This line should end in `::` and have the examples below indented.  That way, in the html docs it will be correctly rendered.
So can we just change this to `subprocess.check_call(["latex", "-halt-on-error", tmpfile], cwd=workdir)`?

Or do we need to squash any output?
```
with open(os.devnull, 'w') as devnull:
    subprocess.check_call(["latex", "-halt-on-error", tmpfile], cwd=workdir,
                          stdout=devnull, stderr=devnull)
```

Or potentially keep the output?
```
p = subprocess.Popen(["latex", "-halt-on-error", tmpfile, cwd=workdir,
                     stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
stdout = p.communicate()[0]
if p.returncode:
    raise SomeSortOfException(stdout)
```
When `new_notification_widget` is called, let's store the widget in a dict inside the `NotificationArea` class.  Then let's create a `get_notification_widget`  method that returns the widget by name.  This way, we can have the widgets encapsulated inside the `NotificationArea` class, rather than needing to have each of them be a global variable.  We could even have a `message_widget` method that posts a message to a particular widget by name.
Glad it helped :+1:
add \@PostCreate annotation to have this automatically started when needed.  p.s. you probably need a \@PreDestroy to stop?
why change to commons.io?
Did you test this? Shouldn't it be `array_get()`?
Yeah, that seems cleaner. `goto` made me feel a little dirty.
Yet another possibly unnecessary existence check.
So actually one thing we now loose is the default ordering - the years in the templates are coming out in random order...
I think we should just take out #2 so that #1 applies to search as well.
Clicking this looks like it works. But just tighten the styles a tad and we can get bram to help for further refinement if he didn't mock it up.
I would really like to see a comment explaining those values. It's hard to follow otherwise.
so, I totally hate this, but the ExternalProcessor never has had unittests.  these default contructor arguments make the ExternalProcessor parallel with the base class Processor.  It makes unit tests easier to write in the deprecated context of the Processor tests.
Why not `px`?
Well, this condition is the same than when adding a child through the builder: https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Form/FormBuilder.php#L96
Fixed. Tnx :)
missing space between ``switch`` and the brace
why this assignation ?
typo: Null
I added this functionality. I think it's a nice way of doing things, and inherently doesn't require a max limit. This method handles little hiccups fast, but doesn't DoS the server. It's the same method that Google uses in their products (e.g., gmail, app engine uploading script, google talk, etc.).
`width` or `height` gets replace with `width / height: auto !important` in `flex-direction`.

**Downsides**
-   It's a hack.
-   Possible `width` or `height` values get overwritten. But that isn't too bad, since one should not use `width` or `height` in FlexBox. `flex` is prefered.
-   You *have* to define `flex-direction`. Otherwise your page is filled with squares. Besides that you should always define `flex-direction` to be more explicit.

**Upsides**
-   It works! The page looks exactly the same in new and obsolete browsers.
-   It' transitionable, too!
oops, I thought I hat caught all of those.
footer ham hamin ghaziye barash sadeghe?
The "plain ol text" types (*.mfs, *.txt, *.xml, *.inf, *.mfold, *.mlist) aren't really needed here, especially since they're not really Plasma file types so much as types used by other plasma tools -- that's why I didn't include them in PlasmaShop 2.0...   Also, types that are configured to just launch an external program (*.prp, *.jpg, *.tga) shouldn't be in the filter either -- they only show up in the file pane for convenience, but we aren't really "opening them for editing" as an Open dialog would imply.  I'm fine with adding .pyc though, since we actually do some work on them from within PlasmaShop.
consider making a variable from this color code
How do we know what projection we are in?
Sadly, the "tolerance" is currently arbitrary for image comparison. 17 is a pretty good score, and is related to a couple of pixels moving (presumably floating point tolerances between 32bit and 64 bit).
So far we've been writing all the properties assuming that Ember.EXTEND_PROTOTYPES is true so this property should also be written in the same style.
message.message().as_string() always returns <type 'str'>. dkim.sign() raises a UnicodeError if you convert message into unicode before parsing it to dkim.sign().
same here, although her variable names could use a bit more clarity as to why they are unique. this needs to be collapseSessionsEF
I'm not sure this will work on Windows? http://en.wikipedia.org/wiki/File_URI_scheme
no need for that at all.

you could just move all the code from populate(operation,model) to this method and that would be it
Why standalon-full.xml? The OSGi integration tests should run against the default config, no?
```js
if ( this.open ) {
    this._close();
}
```
Guess we can do paddingTop, paddingBottom, borderTopWidth and borderBottomWidth
Thanks very much. Good point. Changed ... Should probably do a similar change on existing projects in the future.
yes, you are right

http://www.whatwg.org/specs/web-apps/current-work/multipage/elements.html#custom-data-attribute
Per discussion on https://bugzilla.mozilla.org/show_bug.cgi?id=755402 the reason you need that is because of a platform bug! So I'm fine for landing this file if you mention the bug number inside it. (so it will be removed once the issue is fixed).
This code is a bit weird. We are checking 2 times what is visible. I think this code can be refactored to make something cleaner. And message.keyboardHeight can be substracted after those specific cases to define the currentApp and the currentHeight.
maybe not related to this pull, but l10n missing
That would only be possible of course if we take longer then the period between syncs to sync and maybe gecko is smart enough to only allow one lock per topic (which seems sane)
Same here, use <tt>arrayCopy</tt> or <tt>copyArray</tt>
this will cause a CSS warning, we need a unit, and we can probably drop the line all together since it's being ignored right now.
we use gjslint
the comment is misleading since the function *is* invoked when screen is locked too.
we should probably add a comment below.
You can save the braces here. But it's O.K.
This should not pass lint, missing space.
groupsout.length
You don't need `strlen` check. I think nobody will do `have_` matcher :)
Mimochodem to "== true" je důležitý, protože turtle.run vrátí jinak objekt vysvětlující kde se stala chyba.
Or, for those who oppose `anything.nil?` vs `!anything`, behold: `!$!` :)
What about using `Adamanitum` and then using `memoize :name` here?
Why exactly 3?
En als ZF2 nou niet aanwezig is?
Not sure if JSLint says to put `else`s on new lines, but please put them on the same line as the preceding closing brace.
>What do you mean?

Nothing.
Use `parentNode` instead of `parentElement`.
Ohh, wait a sec, this may be another section similar to my notes  (below) where it got removed as part of my rebase.  As  below, we need to double-check this stuff is suppose to be removed (i.e. b/c of a prior commit) or if I screwed up my rebase (entirely possible).  Thanks for pointing it out.
Raise an exception here to break testing?
I think if the option is provided, you don't really need to ask the user again.
This one still skips validations: http://api.rubyonrails.org/classes/ActiveRecord/Persistence.html#method-i-update_attribute

Just do
```ruby
Notification.where(:recipient_id => current_user.id, :target_id => @post.id, :unread => true).each do |n|
  n.unread = false
  n.save!
end
```
Same observation. If we can live without `read`, it would be much better.
surround multiline body by brackets
This doesn't work, but you get the idea and yes ipfw really is that silly.
It feels great to get rid of this code...
Excluding all the error cases is worrying. The API will receive bogus requests as normal operation. Excluding them is a bias.
not sure what lineno refers to here?
From reading the code it looks like you're always getting the component actions. So this only works if there are no additional rights for items, correct? Certainly a reasonable assumption, just making sure I correctly understand this.
Let's commit to this new way of deploying artifacts.  Remove this plugin definition from the file.
this space doesn't seem necessary, and the comment style we use for one-liners is `// like so`
The problem is not weather it will be simple change or not, problem is an API breakage, that's why I prefer to start with async API from the beginning.
util/list doesn't have add/remove methods
I think there's a race condition here that can lead to erroneously reporting "offline" when a domain explicitly disabled primary support.

Since `primary.checkSupport` and `db.getIDPLastSeen` are async, couldn't `db.getIDPLastSeen` first report that an IdP was seen recently, while `primary.checkSupport` finishes later, notices an `{"enabled": false}", and reports a type of `"secondary"`?

In this case, the user will be shown the "offline" screen, even though the domain has explicitly disabled primary support and asked to use the fallback.
Another alternative: the implementation appears to already allow for passing both the `--run-metrics` and `--summary` options together in the same invocation, and it will simply do one then the other with the same timeframe options. Since the cron job passes the same timeframe options to both, why not just combine it into a single cron job - then the locking works properly as well.
missing whitespace after ','
Regarding to your comment and that the original [`updateTitlebar()`][1] doesn't have any parameter, I'm going to remove this `aBrowser` thing and use `gBrowser` instead .


[1]: https://mxr.mozilla.org/comm-central/source/suite/browser/tabbrowser.xml?rev=2ca434d7ccd6#865 "SeaMonkey's updateTitlebar()"
Yes. As in my knowledge, when we log in, the session_start() represent a new session, for another page, the session_start() represent resume the current session. SO leave it there.
I didn't actually change this is just a case of an outdated docstring
Just a small nitpick: 

I noticed that you're stubbing the release version in the tests (` Facter.fact(:kernelrelease).stubs(:value).returns("5.10")` ), but there's no confine statement to that effect here, which seems a little off. If the code isn't changing behavior based off the `:kernelversion` fact, then there shouldn't be a reason to stub it.