labofsahil/github-event-dataset-2012 huggingface dataset:
It's because we are starting the server on multiple ports and to start them both we use engine instead of server.
I read this page where it says multiple ports: http://docs.cherrypy.org/stable/refman/process/servers.html
Also sabnzbd uses this command to start the cherrypy server too.
I'll get on it sometime this week.
Yes, Scala is fine for me.

On Mar 7, 2012, at 5:21 PM, Benoit Sigoure wrote:

> My current implementation is in Scala, would that work for you?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/ylangisc/asynchbase/commit/4bb9a364c46fcd4600c707124a0ce140979dbcf6#commitcomment-1054138
>
instead of patch the working branch commit your changes to a new branch
cd vendor/cm
git checkout github/ics
git branch vigor
git checkout vigor
<patch your source>
git commit -am "bla bla bla"

after a sync u can switch back to this branch or cherry pick the commit on
the main branch
Il giorno 03/mar/2012 09:34, "dmeadows013" <
reply@reply.github.com>
ha scritto:

> @pbeeler oh yeah lol. Could I add the vigor to the gerrit even though
> CyanogenMod hasn't created a device tree for the Vigor yet?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/CyanogenMod/android/commit/89acec784fd50305cc55d05ecb3416fcd7c3eb0e#commitcomment-1038444
>
Looks good when i get home i try to merge maybe we should think of dividing the objects into more categories for easier use
Obrigado.

O lneves continua a utilizar o repositório svn do software livre. Esta versão git está desactualizada.
Postgres and sqlite didn't worked too... 
I've changen to the Drupal OpenID Provider, the first of the 10 scripts i tried, that works instant, and maybe it's better to use an script which uses a language you spoke.

Thank you very much for your help.
Dit is een redelijk heftige query voor _elke_ pagina.
This guarantees that we return a real boolean, there might be other places this is advisable. **Note:** This isn't the bitwise `AND` operator, it is overloaded on `TrueClass`: <http://ruby-doc.org/core-1.9.3/TrueClass.html#method-i-26>.
Should be `closedir($fp)` as `closedir` closes the directory handle, not the directory path.
This file should be renamed to `config_example.ini.php` (or so), so that it can co-exist with the actual config file on the server.
I had some difficulty reconfiguring a new build with Qt-creator after this change.

The trick is not to use the flag -DCMAKE_CXX_COMPILER and -DCMAKE_CC_COMPILER , like I used to before, to tell Qt creator what to use,

but instead -DMPI_CXX_COMPILER and -DMPI_CC_COMPILER, which is recognized by the CMake FindMPI
Could we have another commit hook for banning dos line endings from the repository?
I don't think it is not important. The name is used when creating a form theme, and also when using the form type registered through DI.
I agree with your comment. :-)
I have the following use case for the new `:s3_headers` can be set as a Proc feature. Given the following models

```
class Band < ActiveRecord::Base
  has_many :songs
  attr_accessible :name
end

class Song < ActiveRecord::Base
  belongs_to :band
  attr_accessible :title, :audio
  has_attached_file :audio, :storage => :s3, :s3_credentials => ...,
    :s3_headers => Proc.new { |song| { 'Content-Disposition' => "attachment; filename=\"#{song.download_filename}\"" } }

  def download_filename
    "#{artist.name} - #{song.title}.mp3"
  end
end
```

When I create a new song like this

```
band = Band.first
band.songs.create! :title => "Test", :audio => open("test.mp3")
```

it fails with `NoMethodError: undefined method 'name' for nil:NilClass` in `Song#download_filename` because the `artist` association is not set at the time the Proc gets called by Paperclip.

In order to fix this, I would like to recommend calling the Proc at a later time. Maybe in the `flush_writes` method?
use addslashes instead here:
see http://www.open-emr.org/wiki/index.php/Codebase_Security#Plan
(STEP 4 -> EXCEPTION)
еще тут нужно исправить https на http
Perhaps depending on the current zoomlevel of the map? I checked this morning first with zommlevel 14, and there precisision was about 20m, with 18 it was almost spot-on! And with the high zoom needed to really evaluate the result it was one tile only anyway...
So we could get the central tile(s) with a variable and generally higher zoomlevel to enhance the experience.
Hi guys, I am trying to implement Pubnub on 1.8. Could you please tell me how to implement this fix on my end?
ok I found where the problem was. I had to set data encoding as CSV, not base64 in Tiled Preferences :)
Since it's a pain to clean this up later, please be sure all source files have the same header with the Apache 2.0 license at the top. Copyright can be held by UMich for UMich-contributed files, and Google for Google-contributed files.
an example to understand !!!
I don't think it's toxic to leave this in right now. It lets people to `butter.plugin.manifest` which doesn't really hurt.
We can do this check before the if else above, changing it to if !target
user-*specific*
-> do the query sorted by expire date asc

How it would be dat? :sucksatsql:
sometimes that is necessary when you are mucking around in Drupal land.
it's either qDebug() or cout, but not both
It seems the `live` and `trigger` options break the hovercard plugin.

Should be fixed in next commit: 59456cca834156251135a5dde2aa220cf1cc860c
Well, the main problem is *when*. The way the (make) macro works in
clarity, I would have to apply the stylesheet at creation time. The
stylesheet mechanism starts with the root and tests each of the
members of the component hierarchy on whether they match any of the
rules, and applies the different looks when applicable. The problem
with that is that when it's just been created, the component is empty,
so nothing happens (unless the root itself matches, in which case
itself is configured but non of its children that are added later). My
solution to that was to add a ContainerListener and listen for
componentAdded events, and apply the same stylesheet to each one of
them as they were being added.

The problem with this solution is that it has the potential to result
in the same stylesheet being applied to the component more than once
(although now that I think about it, maybe not). This should never
happen, because styles in clarity are just a bunch of "mutators",
basically a bit of code that can contain anything (for example a
"style" could add a listener that changes the background on mouse
over) and definitely has side-effects, so "applying" it twice to a
component could have unforeseen and hard to debug consequences.

Does that make sense?

Hm, I need to think about this a bit harder...

On 11 February 2012 00:14, Dave Ray
<reply@reply.github.com>
wrote:
> I'm very interested to hear what was "stupid" about it. I've thought about this problem a bunch (not recently though), but it always seemed to have a bunch of problems. I can't remember what they were though :)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/stathissideris/clarity/commit/e975b3a61130e4ce7f03e9860ab60e1e692d25b2#commitcomment-955814
OK - just ported both libsnes and PS3 to the latest version- all seems fine with zero changes to the core.

Alien 3 still doesn't work on the PS3 port whereas it does on the libsnes ports - I wonder what is up with that still.
Part | Points
------------- | -------------
base | 1
*Total* | 1

Passes tests. :sparkles:
It might be a better idea to assert that element is not contained already. If you appends something that is already a child, it is probably a programming problem in the code that calls this method.
Would seem sensible (also a GitHub issue to remind us to revisit it...)
isLeaf() -- introduce method
This gives “not used” warning. Is it ok for now and `SPREADSHEET_ITEM_OBJECT_ID` is intended to be used in future?
Many thx :)
arjunup: Added the reset function for the two counters
if I recall I was replacing tabs with spaces or visa versa.
connect, email, not-found
Before we were throwing a NotImplementedExcetion, but not our test passes. We're moving in the correct direction.
How do empty events arrays get added to the map in the first place?
I don't know... [browsers allow it](http://jashkenas.github.com/coffee-script/#try:'use%20strict'%0Adec%20%3D%2009)

I'd say you're right -- `08` and `09` should be errors as well. Though I wonder if there's a reason to allow them?
Ok then, tomorrow i will try to do so....hoping that it is the problem! ;)
For now, just a lot of thanks!!! XD
If u have any other advices then just tell me, contact me directly at
simone201@gmail.com
Il giorno 04/gen/2012 22:13, "Andrew Dodd" <
reply@reply.github.com>
ha scritto:

> Just having it enabled in the defconfig (not even the default governor!)
> was causing kinteractived to hang when in LPM.
>
> A governor causing softlockup events even though it was disabled =
> immediate "kill it with fire" reaction from me.  It might be possible to
> fix the interactive governor so it doesn't act badly, but I just removed it.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/simone201/neak-kernel-sgs2/commit/6a05eb6d5337f9e1cd21748cf4d99fcceb2f5175#commitcomment-838296
>
"NoFunctionReturnType" scheint mir hier fehl am Platz zu sein, ebenso in Zeile 28.
I made a [pull request](https://github.com/elasticsearch/elasticsearch/pull/1602) for that.
Ok for me :)
argh .. dont undo my fixes! please update your DoctrineFixturesBundle and revert this change
Please use [at] instead of @ for the e-mail.
You can also compare the complex data structure with ```Test::Deep```
Agreed on both points: It would be cool but I'd consider it a nice to have.
After this update i get a compile error on linux

src/server/game/Achievements/AchievementMgrBase.h:205: error: âChatMsgâ has not been declared
src/server/game/Achievements/AchievementMgrBase.h:211: error: âChatMsgâ does not name a type
src/server/game/Achievements/AchievementMgrBase.h: In constructor âTrinity::AchievementChatBuilder::AchievementChatBuilder(const Player&, int, int32, uint32)â:
src/server/game/Achievements/AchievementMgrBase.h:206: error: class âTrinity::AchievementChatBuilderâ does not have any field named âi_msgtypeâ
Concordo. at ia citar essa parada de mudar os forms, mas no acho que
teramos muito trabalho. Como disse, no tenho preferencia. Se acham que 
mais comodo continuar com o simple_form, continuar iremos. (mais alguma
objeo?)
-- 
Thawan dos Santos Fidelis
Graduando em Cincia da Computao - UENF
Linux User #522067
@Thawan_Fidelis
Why not use a `FloatField`?

https://docs.djangoproject.com/en/dev/ref/forms/fields/#floatfield
If I remember correctly, the RealTimePlugin catches onBeforeCommand and stops the command if it is BOLD, ITALICS, etc. and then executes its own code to apply the style on DOM and the Tree model. Why doesn't this work with STRIKE_THROUGH?
Are all these changes a fix for the hibernate workaround hack?
Pretty sure this is redundant.
OK, will use new exchange!
AttributeError: 'module' object has no attribute 'err'
nope - geht :)

jetzt zeigt der sogar bei Avg (ms) 

etwas an :)


vorher war dort nix


wann soll das display genau schwarz bleiben ?

ich hab jetzt per verknüpfung den screen gelockt & auch per power knopf

beides Mal ließe sich das Gerät wieder aufwecken
There's a requires_one method you can use, https://github.com/fog/fog/blob/master/lib/fog/core/attributes.rb#L169.
When running the units tests in IE, I noticed the use of a reserved word, so I switched this test to use dereference the property by name.
Re-reading the last sentence of the above paragraph: this is wrong, the dispatchers for head and routees are completely distinctly configured. This should be made more clear in the docs.
So what is the semantics? Can this method throw something?
… an exception
So if I have an actor-service that wants to "unload" at runtime after a while, it cannot?
Remove this sentence, it is repeated below:

"+One ``CamelExtension`` is only loaded once for every one ``ActorSystem``, which makes it safe to call the ``CamelExtension`` at any point in your code to get to the
 	 110	
+Apache Camel objects associated with it."
use explicit return types for _all_ getInstance() methods please
Adding Json versions of some type based views, and we likely need more, Pointers at least.
@MrBago This is not fundamentally broken. This is the best way we can deal with this. Indices1 and indices2 will be different and that is a good thing because they correspond to different spheres. 

You say here "We could make a method like fit._set_cache_values(sphere) and call that in gfa, qa, peak_indices, peak_values, ... but this stuff shouldn't be updated on every call to odf." but the point is that if you don't cache the odfs ( and storing the full brain odfs is a bad idea because they are too big for the full brain) then you need to recompute the odfs for gfa etc. Does that make sense?
In this case it is not possible since we are checking for the environment variable before we can import from any ckan related code.
The problem is that the unit install api docs suck
We should probably put together a page about specifying content units under conventions, if I have time at the end of the sprint, I'll tackle this
Yeah, that was dumb.  Fixed it up.
Doesn't this need to checkout output too?
$this->template = "{headline}\n" . $this->template;
Тут кстати \n нужно? Может там <br />?
просматривай пожаулйста принты
Should document `loaded`.

A name like `already_loaded` might make your code more instantly clear. :)
Actually, I think i'm thinking of it wrong. The first person to set the value should win, it's deterministic that way. 

I was thinking that the timeout would fire *just* as the event was coming in, and it'd be worth giving the event the chance to override the value between the timeout and the acting-on-result case statement. Now that I'm describing it, that sounds like a *really bad idea*. :)
Formatting: "viewMaxY - scrollViewMaxY;" (spaces around minus sign)
Why not `WCF.DelayedExecuter = Class.extend({`?
Similarly, this should be `QString::fromLatin1`.
what is a master channel?
Here's the code:

```
[_navigationController.view addLayoutConstraint:[TUILayoutConstraint constraintWithAttribute:TUILayoutConstraintAttributeWidth relativeTo:@"superview" attribute:TUILayoutConstraintAttributeWidth]];
[_navigationController.view addLayoutConstraint:[TUILayoutConstraint constraintWithAttribute:TUILayoutConstraintAttributeHeight relativeTo:@"superview" attribute:TUILayoutConstraintAttributeHeight offset:-TAB_HEIGHT]];
[_navigationController.view addLayoutConstraint:[TUILayoutConstraint constraintWithAttribute:TUILayoutConstraintAttributeMinXMinY relativeTo:@"superview" attribute:TUILayoutConstraintAttributeMinXMinY]];
```

It seems like more, but it's declarative and easy to modify individual strands. If you find any issues, let me know! I've added cool value transformer stuff in there (and blocks!)
Maybe use socket instead of server, it calls up images of clunky 2U devices.
Git should be able to handle this without gitattributes.
Umm, okay. Done.
Ok, just bear in mind that only the Order class is shown inside the belongs to text, even though the first example shows everything (but it's not related to all other associations).
yes I'll move it there. I put it in `process` because I think it communicates the intent better, than overwriting the variable later. If it produces warnings though I'll put it back.
Hey @lest. This proc always receive only one argument, the exception. So I'll change to the second option. Thanks to review.
I don't think it's necessary to use `any?`, since it'd start iterating over the collection, `present?` should suffice.
can you test running tests with https://github.com/guilleiguaran/rails, in the ``remove-manifest-option`` branch, I think is a problem with my machine
Probably, @paulp would rewrite this even more concisely with his glorious DSL. Unfortunately, I never made an effort to learn it, always assuming that it's just few weeks before quasiquotes pwn everything else :)
This might be nit-picking, but if you've got a large Scala codebase and someone generates diagrams with this set at say, 10, how meaningful is the result when you end up with a truncated batch of implicitly converted (input or output) classes? In this case, isn't it that what shows up is somewhat arbitrary? Say there are 20 related implicit conversions classes and you've truncated at 10, then you get some random 10 classes shown in the diagram, some of which might be less meaningful than others that were truncated and are not shown. I'd imagine that'd create quite a lot of frustration and confusion-- imagine someone depending on Scaladoc for this type of information, finding that some class instance they're operating on is being converted to some totally arbitrary class that's not documented, while other ones are.

It seems to me that a simple no-implicits-at-all-in-my-diagram might be preferable to some arbitrary cut-off, no? (and yes, I do realize you can also just set max-implicits to 0, but it's a bit weird IMO)

Alternatively, what might be better... There could just be a stop-list of implicit conversions you don't input or output converted classes associated with shown. Though, for this to suggestion to be widely useful, you'd probably need two stop-lists-- a local and a global one. The global could contain conversions like any2stringadd and the local could contain others that the author of a specific class might want to omit. Do you get what I mean? I think it'd be a lot more meaningful and it'd probably avoid future complaints/bugs.
Is this name clash safe? It's not immediately clear what changes caused this either.
Example would be useful. Can be the same as in tutorial.
I think MaxAge in minutes makes sense from a user perspective. I was able to test the cull_members_from_pool function by passing 0. That won't work if 0 means "never" with the scheduled version. At the end of the day, I want to be able to write a test for the scheduled culling and not have to wait too long for the test to run. So the easiest path might be to change the cull related function to be more flexible in terms of time granularity/units.

Good point about the granularity. I don't really see that as much of a problem as long as it is documented. 

Since you're interested in having the scheduled cull feature, I'm curious to understand how it will help you. We've found that there isn't a whole lot of benefit of reclaiming resources during non-peak loads since we are going to need them to handle peak load.
H1 tag breaks design here a little because it has different line-height.
Here is a counter example to your suggestion @jrioux  

`MatMul(Identity(3), 2)` would return `MatMul(2)`
The tests should have detected any clashes with the compex beam parameter formalism, but I have just checked and there are actually no tests for the conjugation of a beam parameter and a ray transfer matrix.
OK, will do; will keep this in mind.
> I really don't have the time now to wrap my brain around what is going
> on to give this a good review. I know you have some TODOs but this
> doesn't break anything with the current master, if I understand
> correctly, since pyglet is still supported (though not shipped with
> sympy). You are just providing another development path for producing
> plots. As such I would say let's get this going and improve as issues
> arise. Do you agree? Have others been in agreement?

It is a long pull request so it is hard to review, I agree. There are
definitely more issues to be ironed out, however I would prefer if
this gets merged: This way tracking the issues will be easier and it
will permit us to use the issue tracker a bit more effectively (you
can not file an issue for code that is not merged, others can not
contribute to code that is not merged).

Moreover this is already used by others for their gsoc work (it got
reviewed too, however due to the size of the pull request it is
possible that there are still hidden bugs)
I put the comment into the function docstrings and removed it from here (but left the test that now passes).
I'd prefer to have lens and file_path not publicly accessible.
`paste0()` requires R >= 2.15.0; please use `str_c()` here
See comment on line #305
After some source reading I know for sure: current colander's All validator do not support deferred validators. I've created issue in colander repo and sent pull request with bugfix.
Add a high level comment on how we estimate initial size.
What do you think of this compromise, @eric? `:only` and `:except` have two higher level tests using string matchers for clarity and a third expectation test. The later defines the protocol for the matchers and asserts the report is sending the right messages. If the implementation changes from sending `#===` to sending another message, the expectation test will fail. That's by design because it means that change may break clients coded against the original protocol.
Seems that the unknown new line in the head of the HTML output come from the new line between an and and a start php tag.
Like that : 
?>

<?php

Need to do :
?>
<php
and it's work

Or just suppress "?><?php" ;o)
We could, but the changes frontend actually needs to iterate the list multiple times (once for a validation pass, and then for submission). And we also pass the result list over an actor boundary, which could have some interesting resource cleanup semantics if we're streaming.
Please use the following styles to be consistent with `| Map | Satellite |` buttons on the right side:
```css
font-size: 13px;
font-weight: normal;
padding: 1px 6px;
```
remove this section? this is about master/slave?
16 should be a constant set at the top
2005 again. You like going back and forth in time? :P
Indentation is incorrect here.

Also, because this information isn't needed on every page, you can probably just add it in posting.php and in the ucp/ucp_pm_compose.php files.

Lastly, since you don't seem to be using $s_select_editor anywhere else, you can condense the lines like this:
`$template->assign_var('S_VISUAL_EDITOR', ($user->data['user_selecteditor']));`
Note that you don't need the == 1 or the ? 1 : 0 because in its usage here it is treated as true or false.
using as is from symfony, and wrong PR, that is events dispatcher stuff.
Why use wiki and post revision for the same kind of thing?
The event name obviously is the function name _(which is IMO the expected naming schema in these cases)_. So if you look at it you understand that the event is used to change the behavior of that function. The weird naming is caused by strange phpBB function names, and should be resolved at the api level rather than a work around in the events.
But a cache driver could potentially implement both and be both a valid SQL cache driver as well as a standard cache driver. I cannot think of any situations for this right now, but I wouldn't want to not allow that case.

The way it is, the generic driver is only used if the selected sql cache driver does not implement the sql interface.
This sounds weird, to make sure of what?
why don't we move this guy to the toggle state in "t"?
less complicated:

```js
// in most cases the name of the handler is part of the description
var handler = navMimeType.description.match(/audio|video|image/i);

// return early when the mime-type is irrelevant
if (!handler) {
  return;
}

// case insensitive, support "video" and "Video"
handler = handler[0].toLowerCase();
```
Should use a common routine from the test harness, or at least the one from utils so we are using one we write unit tests for.
That was meant more in the sense of "Does this need to be here, run every time you make a renderer? i.e. once per page view"  Though I guess configuration won't be that expensive anyway.
What's the number mean here?
Does JHC require GHC 6.12.3 to compile, or can newer versions of GHC build it, too?
On Tue, Sep 04, 2012 at 06:33:42AM -0700, maggesi wrote:
> > @@ -12,8 +10,8 @@ stdenv.mkDerivation {
> >    inherit name theories;
> >  
> >    src = fetchurl {
> > -    url = http://isabelle.in.tum.de/website-Isabelle2011/dist/Isabelle2011.tar.gz;
> > -    sha256 = "ea85eb2a859891be387f020b2e45f8c9a0bd1d8bbc3902f28a429e9c61cb0b6a";
> > +    url = http://www.cl.cam.ac.uk/research/hvg/isabelle/dist/Isabelle2012.tar.gz;
> 
> Indeed this is what I did in the past in many of the nix expressions I
> authored (including this one).  But later I noticed that it is often
> annoying when you want to cut&paste the url (e.g. to call wget or
> nix-prefetch-url).  Moreover, this make the code a little bit longer and a
> little bit less clear for little value.  I don't have a strong opinion on
> this matter and I understand your point, but if no general guidelines
> enforce this, I prefer this style.

I think the same about copy&paste, and I always doubt what to commit, too. I
dislike when there is a version attribute only for this, but I understand other
people prefers that.
Am I misunderstanding the test? it appears to 
1) verify that the local timeout limit ( Util.getTimeoutLimit() ) equals the ExtJS ajax timeout limit ( Ext.Ajax.getTimeout() ) [http://docs.sencha.com/ext-js/4-0/#!/api/Ext.Ajax]
2) calls storeUsePrivileges

I don't see anything that verifies a timeout happens or a "connection error" is shown on timeout.
Is that a vaild cube? Would that be a vaild NetCDF file if it were magically produced?
Oh, I see this was actually causing an image to have a gap in it. Ignore me.
I wonder if any of the voltlabs scripts have to change?  Or if the `voltadmin` tool contains any notion of `start`.
>azolex(should be max aura fields, no idea and dont care)

This sums up his whole contribution to the ArcEmu project.
Also, EffectBasePoints[0] should be EffectBasePoints[mod->i] or not?

https://github.com/neo-mat/arcemu/commit/4eb4e104fb4573e58858b56db946003773b9b909 (Second branch with second variation of patch on this topic)
This is abstract and meant to be used internally. Same answer still applies.
Yes we don't need this. The time stamps should be accessed through the cue or node and we are not storing any string data outside of string data in the cues and nodes.
Thanks for spotting those. I still need add a few math tests.
There are also bugs in the range reduction of sin,cos,tan for large 
arguments, which is my current priority.

On 06/04/12 17:41, Joe Groff wrote:
>> @@ -56,12 +56,12 @@ overload recip(x:T) = T(1) / x;
>>   inline overload abs(x:I) = if(x>= 0) x else -x;
>>
>>   inline overload sign(x:Long) = bitor(bitshr(x,63),Long(bitshr(ULong(-x),63)));
>> -inline overload sign(x:Int) = bitor(bitshr(x,31),Int(bitshr(UInt(-x),31)));
>> +inline overload sign(x:Int) = bitor(bitshr(x,31),wrapCast(Int32,bitshr(UwrapCast(Int32,-x),31)));
> I think you mean `wrapCast(UInt32,...)` here. Is there a test for `sign(Int)`?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jckarter/clay/pull/256/files#r655636
If we're looking for "background" at the beginning, then the condition should just be that: `name.indexOf("background") === 0`. The intention won't be mistaken and we're spared the cost of coercion (though likely negligible)
Yes, Django is BSD licensed.
Can you add a comment that notes that this patch has been merged upstream and can be removed in the next version?
Yes, it will. I need some way of signaling that xcode is not there. Any suggestions?
How to deal with formulae that have
` depends_on 'automake' => :build if MacOS.xcode_version.to_f >= 4.3` ?
Ver el glosario: https://github.com/nodejs-es/api/wiki/Glosario
This message is so long that the if condition is hard to read at the end.  Try to find a way to shorten this line so it's clearer.  One improvement would be to not throw a RestException, but instead one that is locally scoped.    We just want to end and report the error to the user as cleanly as possible, and it's not technically an exception in retrieving the data, it's a usage error.  Throwing a new, global InvalidUsage or other generic "user exception" would make more sense.
If requesting /alias/dir/index.php/pathinfo
```
OC::$SUBURI = substr('/alias/dir/index.php', 1 + strlen('/alias')).'/pathinfo'
> OC::$SUBURI = substr('/alias/dir/index.php', strlen('/alias/')).'/pathinfo'
> OC::$SUBURI = 'dir/index.php'.'/pathinfo'
> OC::$SUBURI = 'dir/index.php/pathinfo'


``OC::$SUBURI = 'dir/index.php/pathinfo'``
Ok. Will change.
Actually we're both wrong :)

The function returns a list of subreddits whose name match the query. A search for ask returns both "AskReddit" and "AskScience". I believe reddits search function is a trivial one that simply test if the query is contained within the subreddits display name. So perhaps

"""Return subreddits who's display name contain the query."""
You are right, the examples do not really fit well into the resources directory. The reason I put them there was to include them in source into the jargs-$version-tests.jar file.

I will try to come up with a better place, `src/examples` sounds good
This `block='true'` causes the template to traceback in my install.
Not as such.  Apparently some work towards it but didn't make it in in a way that "just works"
If it's exposed but not typed what is its use? I'm on mobile and can't see it in any obvious fashion.
Should we first delete all `nick_` keys here?
Should not the condition rather be (options.delay_attach_on_connect != 1 || icanhasall)
Since it's a builder-style method, I would prefer it to be called `withMissingExceptionMessage`. That way you can write

```java
thrown.expect(SomeException.class).withMissingExceptionMessage("...");
```

What do you think?
you mean fExample1One= null in @before ?
, but it should be in @after
pls confirm.
The HashSet should be sync too no ? When we remove a model to generate his mesh, should we remove in the hashset too ?
There's a good reason for this. If you'll look at the Zone class, you'll see I've already rewritten it to handle coordinates in the order they were selected rather than highest-first. (This doesn't break backwards-compatibility at all, btw... sorted coords still work fine.)

I need the coords in the order they were input in order to keep track of the orientation of a Zone object. This matters when teleporting from one VoxelPort to another of the same size, but which is rotated differently.
The semantics of mixing the platform arg with the platform property weren't immediately obvious to me.  It looks like the platform property overrides the platform arg, except that the platform property can't then use undefined to mean "all platforms" (which it _can_ if the platform arg is omitted).  Should we simply disallow passing the platform arg if keyBindings is an array?
I've been playing with this a little bit and... would it be okay to request the animation frames everytime we hit mousemove instead of recurrently within doRedraw?

I've checked and couldn't tell both cases apart. Both resizes seemed equally smooth with the only difference that calling requestAnimationFrame from inside mousemove results in way less doRedraw calls (none if we aren't moving the mouse, for instance).
As in "Editar", there's no consensus about this. Chrome and Firefox both use "Ver" ;)

Changed it to "Visualización" in line with the others.
i think this won't work over vfs, since in that case `args` are sent to remote machine where __dirname doesn't exist
`trFiles` might not be defined yet:

![Screenshot](https://img.skitch.com/20120425-di873ft2168nfegay4hq65k82y.png)
Good catch. This is probably spammy and race-prone.
You should at least be logging some sort of information here even if you just ignore it.
that class is imported at line 29. Shoul invoke simply Utils.escapeIllegalCharacterInQuery(text)
It's used in to simulate blocking for one-way synchronous calls, in SyncClientHelpers.sendSynchronousOneWayMessage (in swift).
Isn't this the same computation as in `coord_cartesian`?
Anything Opera specific should go under `lib/user_agent/browsers/opera.rb`.
Rename to A1
I figured we just have no conditional, but remove this line when (or if) rugged goes to 1.9 only.  But a conditional would be fine too.  This PR should work on 1.9 without adding `gem 'minitest'`.
Just wondering what these constants are...? :)
There are not many examples of `default_error_messages` on the web - the ones I found all override defaults(?).
Use external file. You can use one same external file and document ready event handler for all PiplMesh event handler registration.
No `__`.
Same as above. Everything works like a clock here. Could be optimized later, perhaps, but I'd rather not fiddle with it now. It works as intended, and moving stuff around now might be an unnecessary risk.

I really do appreciate the thorough review, but heed the comment here https://github.com/mozilla/butter/pull/1313/files#L0R151.
remove first blank line if not necessary (it's not here).
this tooltip only shows up if you try and click `save` in the editor ( once you've logged in ) and didn't give your project a name. It will appear in read letters telling you to do so. Once you mouseover the input element again, this will get removed.
don't know why the file name had to change, but ok
Because less indentation == better IMO, I just wanted to leave this note for readability
Line 132, E:0001: Extra space at end of line
Ah, good catch. I did not see that.
This doesn't seem to be the only place where **Ruhoh.setup** is called.  In `lib/ruhoh/client/client.rb` on **line 20**, there's another call to *Ruhoh.setup*.  Does it also need to call *setup_paths*, *setup_urls*, and *setup_plugins*?
I find it a bit inelegant to have two variables that do the same thing (i.e. hold the module ID). How about:

m_Application.EnterModule(sModuleID.IsEmpty() ? NULL : sModuleID.GetBuffer(0));
Should put if/else on separate lines for better readability, with braces
Jag tolkar det som att en assistent har många labgrupper som den har relation till eftersom den rättar deras labbar.
Rent praktiskt innebär detta inget mera än ett bekvämt sätt att komma åt relevanta labgrupper via en assistent, och är på inget sätt en viktig del av denna lösning.
Anledningen att jag lade till det var att det fanns en spec för `AssistantRegisteredToGivenCourse` som sade att den ska ha en labbgrupp. Jag ändrade den så att den letade efter flera labbgrupper och lade till relationen.
Om det stör dig så kan vi givetvis ta bort det.
I didn't even think of this. Good job!
Please note, I also discovered a bug on the updateAttribute method. 
The description says: (reference: https://github.com/1602/jugglingdb/blob/master/lib/abstract-class.js#L557)

    /**
     * Update set of attributes
     *
     * this method performs validation before updating
     *
     * @trigger `validation`, `save` and `update` hooks
     * @param {Object} data - data to update
     * @param {Function} callback - callback called with (err, instance)
     */

And the corresponding code part for the callback: (reference: https://github.com/1602/jugglingdb/blob/master/lib/abstract-class.js#L577)

So my solution is to send the instance to the callback method too.
formatting needs change
I just pushed to master.
This change was intentional.  Before the enhancement to support label & id, the user always provided the product name, so we were able to use prodName which was derived from the incoming options.  After the enhancement, we now pull the name from the product record that is retrieved from the server.  That record could be retrieved using name, id or label.
So then on year do you tell them to go back to finalize nurses??
Probably want to assert maxAge and expires aren't both set.
please use the short class name as it is in the same namespace
Where is the form action?
You might want to say here that you need the developer *command-line* tools.  I read somewhere you can install them independently from XCode despite the fact that they are the "Command Line Tools for Xcode" (although I haven't actually tested this), and this phrasing implies you need the full Xcode.

Or you could just say that what you really need is a compiler, and the easiest way to get one is via the XCode command-line tools.
I have a preference for the latter, because it requires less parsing, and is therefore less error-prone.
bugfixed.... 我错了，repeat period后加的，没测试。

可是每9分钟一次在cron里面也是*/9啊，不会给你准确的九分钟一次的选项啊，关键是需要么。。。大部分都可以用整除时间把？
Can you put a big comment either before or in this complicated nested #ifdef explaining what is going on?  I'm lost.

Something like:
"If you're building from an un-tarred git-produced tarball, then blah, if you're building from a checked out git directory then blah or blah depending on blah...."
IMO it should stay in ClientModel. Don't put such state in views, better not to rely on it that RPCConsole is created once at the beginning of the program and kept around forever. This may change later.
Good idea, will change this.
Yes, that's what I am trying to say all the time. The +1 is valid, however
it is better to do it at different places, so you can distinguish between
the two conditions that I mentioned. Currently you *can not* reliably
detect the condition that causes the error.
This should be `!isset($this->_charsets[$name])`.
It also highlights the lack of test cases for this change :)
Missing a static here.
Missing spaces here
That's spiffy! http://php.net/manual/en/language.oop5.late-static-bindings.php only comments mention that construct. (I will fix it tomorrow)
Describe text should be describing the action that is actually happening. What you've used it for here seems more like a context.

Also, in this particular case, it's probably just going to clutter up the test sentences and make them really long, so for things like this I tend to leave it blank (e.g. just `context do`) and you can put the comment that you've mentioned as an actual comment above the context block (instead of as a describe text).
I suppose this change is ok, but please adjust the unit tests accordingly. Right now, they fail for this PR
I think you should make the arguments match `install` exactly.
The module level docstring should have at least a small summary of what the extension does and the main entry points or useful things it provides.
I will limit the size of this set by waiting for futures to evaluate if the set contains more than a maximum number of elements.
Is temporary URL signing specific to CloudFiles?  We can use this generally for all Essex-based Swift providers.
Can't you just write `&amp;` directly? Saves one useless function call.
I had tested it in an application that I'm building. But instead of copying and pasting the original code, I retyped it. And you saw the result. Sorry about that.
Seems that this was *never* necessary. I'll take it out.
as we spoke, not sure I am completely comfortable with this change.

I am surprised no othr tests broke. I wonder if we had coverage for this line before
some code to test various URLs, example:
http://example.com
Regex can't find a port, path and query, and file uploads to http://example.com:undefinedundefined (port and path are undefined)
This code fixes it
This change was for debug and should be reverted.
Actually it might be a bit more complex than that because you care about the order of the suite IDs in `self.cleaned_data["suites"]`, right? You'd still be better off doing the single query for all the suites, then looping over that to construct a dictionary mapping suite IDs to suite objects, then iterating over ``self.cleaned_data["suites"]`` and doing a lookup in that dictionary rather than a db query each time.
Is it possible for the default to be ./deferredCrashStore instead of /home/socorro ? This would be nicer for running out of a checkout
epic thread is epic
This is available as `application:get_env(chef_wm, bulk_fetch_batch_size)`.
LOL.... a nice failing test.
Why did you delete the .gitignore file?
this should probably remain a global function, as it has no references to this, and only accesses global variables
I'm currently working on switching PyroCMS to use Sqlite and Postgres too, so can you make sure this is all using query builder stuff?
I was declared a wrong variable name, and has fixed it in my last commit.

Cheers
This is actually okay, in this situation only really. If you check out how the embedcontent widget constructs the div I'm manipulating, this will always work. I want to check for the presence of the width property as set by that widget, nothing more.
Can we delete these variables?
This looks like a complication to me. Also it adds additional header dependency. What is the logic behind this?
Btw, with #5187 it would be possible to generate the URL as before (no bc break) although it doesn't really make sense in this case.
Why do we need to change the order?

I believe we could avoid that either by simply removing the ``= null`` part (PHP should not complain if there are required parameters after optional ones although it does not make sense), or the other alternative keep ``= null`` and simply instantiate the default handlers if none is passed.
But look at the code in Symfony\Bridge\Propel1\TestsFixtures ItemQuery and ReadOnlyItemQuery.

Both these class are unrelated object trying to be a TableMap (check the getTableMap method) !

I am not saying your request cannot be done, I am saying that typehinting ColumnMap will break other test code that are build correct, and theses tests would also be needed to be changed, but I do not understand them, so I cannot change them !
You should separate the short and long description.
This looks wrong as you changed the id
why mixing protected and private properties ? And btw, when having both, protected properties should come before private ones
typo: relevent
typo: $value ^^
Perhaps and 'required' to these exceptions?
Not in love with this method name. Lacks a verb.
In the past the details param would return importers and distributors.  This makes it appear as if it doesn't.  If not what does it do now?
Why not, has long as your intent is clear. You could also set up a real action type in a case class, with the different fields named accordingly - it won't cost much in memory/processor, and will be clearer for the future yourself.
So here, perhaps it would be better to use a "reportLogger" field name, so that dev / debug messages are not sent to the ops logs.
What does this 0 mean here? Should it be related to z-depth instead?
Seems to be a typo here.
Please confirm the tests are passing.
Same here with "Helmet"
Don't worry, I really appreciate you spent time on this. If you have any problems and feel like we could help don't hesitate to post your questions :)
I'd love to use the plugins "insterface"!
Copy/paste leftover?
Is this step used?
<version.org.jboss.marshalling.jboss-marshalling-river>1.3.15.GA</version.org.jboss.marshalling.jboss-marshalling-river>

This looks like an unused property. The marshalling-river artifact (rightly) uses the version.org.jboss.marshalling.jboss-marshalling. I think it's better to get rid of that version.org.jboss.marshalling.jboss-marshalling-river property altogether to avoid any confusion. I'll send out a separate commit/PR for that.
Man, and you should see how I deal with those who remove my last empty line… :-p
I suggest that you move these calls to get() (and the other ones) to init() or to the localized event handler that calls init.  Just define new global variables and store the values in those.  If you're worried about the user changing language while the app is running, you could call re-localize again onmozvisibilitychange.  I think it is okay to keep the calls to get() in showOverlay() because that should only be called infrequently.  But unknown artists or titles might occur much more frequently.
Add this directly onto the container.
I think readonly permision is sufficent here, isn't it?  Please change unless the app really needs to be able to create and delete files.
I tried, but couldn't find a way. I might look again, but I think this comes from the top + bottom margin of the <ol>, bottom border of the <ol>, and top + bottom margin of the <li>. Those are each 1px, so it adds up to 5. It's kind of a chicken-and-egg problem, in that I need to have all those elements available first, and rendered, before I can inspect the styles. And, need to find a spot to find a sample instance to do that once, so I'm not doing it everywhere. (Or maybe cache it)
There has a invert parameter for getMessage. Since you use the sorting workaround to sort the messages again in message thread view, but it should return the messages with invert sequence (from oldest to latest) if you set invert  for the ThreadUI page(if the DB works correctly).
dead code :)
Maybe you could append the desktop testing js dynamically in HTML?
<script>
  if (!navigator.mozSms) { document.write('<script src="js/desktopTesting.js"'><\/script>'); }
</script>
Might be good to log the user out before initiating the forget sequence below since that is the real use case.
Rename to SetDeployer
It points to the folder that has the reference assemblies that we include. Currently it's just System.ComponentModel.DataAnnotations.dll.
I' know you love spacing: b: T
In the same spirit, if strategy is not set, set to default if available otherwise assert
nicely broken down into comment steps first.
Same as before - UserAppServer.
Document this method. Also, why define this inner method here, as opposed to outside of this method?
There should be a `Raises` section here, since `TypeError`s can be raised.
Or I just add isSearch when giving it to Redirect.to which saves the hassle of determining it.
This seems like fairly duplicate code.
Oh you're trying to override a peer validation failure too? Rather than have the users accept an insecure bundle, why not have them configure a secure bundle? (eg prompt the user to apt-get install ca-certificates ) I think manually adding CA roots might be a bad idea, especially given that they may need to be revoked.
Indentation is messy
Should I update both?

    src/Composer/Downloader/SvnDownloader.php:        if ($status == 0) {
    src/Composer/Repository/Vcs/SvnDriver.php:        if ($status == 0) {
LOL. You are picky man! ;-)
Please have a look at https://bugzilla.redhat.com/show_bug.cgi?id=862325. I have understood that rules rule2-modify and rule3-modify should fire in this case. Did I get it right?
Actually, this whole function is now unnecessary and has been replaced with spd->caps[cap_num]
Should you still be testing for prefs on line 110, as you're not passing it in to the function any more?
take out commented-out code
This shouldn't be a warning if it's actually an error, which it seems to be since we return "failed"
s/signature/rule/g
Good point I'll do that.
Not much it's just toString on it will be: `function () { [native code] }`
We both perfectly know that, but it would be helpfull for newcomers to tell that loaderURI refers to cuddlefish.js and not loader.js.
py.test --destructive --credentials=credentials.yaml --baseurl=http://www.myfavoritebeer.org --driver=chrome --chromepath=/usr/local/bin/chromedriver -q myfavoritebeer

and

    py.test --destructive --credentials=credentials.yaml --baseurl=http://www.myfavoritebeer.org --driver=firefox -q myfavoritebeer


The credentials should be okay (they worked for the 123done.org tests, at least).

Thanks!
this line is no longer needed as a result of https://github.com/mozilla/browserid/pull/2365
I'm not sure that this should be a subpage of ProfilePage. It seems to be a standalone page to me. It might make maintenance simpler if it were in its own file.
Why would we need that? We could bind the tooltip to the surrounding span element.
We do not only check if the SDK is active, but even execute commands. So what about 'sdk-run'?
All this is no configuration related code and should live in the logger module itself.
I don't think that we should care about code before bug 682996 has been fixed. It was fixed for Firefox 10 which we still support on the ESR branch. Releases before we do not support anymore. 3.6.x is an exception but will also be dropped soon.
This page is awesome. :metal:
Should be ``AMQPQueue::delete($flags = AMQP_NOPARAM)``
`if (isset($this->tablesByPhpName[$phpName])) {` would be a bit faster than `array_key_exists`
it's even easier with Composer, as it creates a `bin/` directory in your application