labofsahil/github-event-dataset-2012 huggingface dataset:
Retry on Error added for CloudFormation creation

CF's in Amazon are essential to our CI pipeline, and when creating one
fails a lot of useful data is lost. Added in a retry option, defaulted
to 0, which a user can specify the number of times a formation will try
to be created before it finally fails.
Update Snippets/Table for Bootstrap 2.0
Mais um CV!
LPS-24343 Unable to remove form fields translations in Web Form Portlet

Theses changes allow:
- Remove form fields translations
- Change the form fields order deleting previous translations
Fix for installing RPM

While updating the spec file I moved the stop plex on install to the pre phase without checking if plex was installed or not. Caused install error. Also, not adding exit 0 to the %pre phase caused the rpm to bail out before actually installing plex. Now has check for upgrade or install and ensures we exit out the pre phase with exit 0, so we continue with the install/upgrade.
[WFC] Update version 2.2.x

Mise à jour de Twitter Bootstrap de la version 2.0.3 à la version 2.2.1
Add Facebook and Twitter sharing links

This patch adds Facebook, Twitter, and other service sharing links with
AddThis:

http://www.addthis.com/?r=0

Signed-off-by: Pekka Enberg <penberg@kernel.org>
Создание коллекции объектов

Не знаю, как объединить объекты в коллекцию, сделала через массив. Подскажите, как исправить. Еще я не понимаю, как эти объекты связать с объектом Event, который был реализован во 2ом задании, чтобы поля заданные по умолчанию в Event нашли отражение в создаваемых объектах.
Fixing some problems with a couple tests.
Fixed bug where buttonMode on flash display objects is broken

After setting the handcursor functionality in starling sprites and buttons, using buttonMode on flash displayObjects no longer displays a hand cursor. This fix restores the buttonMode functionality.
Moved classes on the main file to their own files.

A small, code gardening, contribution :)

Now, do you think the exceptions deserve their own file too?
[Patch] Can't use Chart.get if a serie wasn't visible first

If you add a series with *visible: false*  and you do a *Chart.get(not_existing_id)* you will get an exception.

ex: http://jsfiddle.net/5eFpg/4/

The problem is located in the *Chart.get* function who search for points in a series with an matching *id*. If the series set *visible: false* when it was added there's no *points*.

The for loop who search for points with matching id didn't check if it has been initialized.

This patch check if there's no point in the serie. If it's the case it skip to the next serie.
implement 'Touch Position'
Clean-up, integration and first steps in presenting the info.

run `sh start_over.sh` and be happy
Nanbranch

Making your site prettier. more to come. Feedback please :) 

xoxo
Updated czech translation

Updated missing translations in czech .po file
Update proprietary files extraction method

This is based on jt1134's work for the galaxytab7c, I might have messed something up or missed something though, but it looks right to me
this is a test

adsfasdf
Remove duplicate preferences menu entry on Linux

Linux had two entries for accessing the preferences (in Tools and in Edit). In Linux, default is the Edit menu (see SB bug: http://bugzilla.songbirdnest.com/show_bug.cgi?id=10030).
This patch removes the entry in the Tools menu.
Will be ported to SB as well.


And I hope the indentation is ok now :-)
Corrected OS requirement and other minor changes
Secondary dn
Version 0.4.0. Added updating support.
Est-ce absolument nécessaire de remettre le brush à #00ff00 après chaque appel de DrawRectangle?
It might be worth looking to the javascript upstream that provides the 'undefined' value; if it's possible to check for "undefined" there, rather than having it cast to a string and sent into PHP, it would be nicer. "undefined" is a JS construct and it's a little strange to see it in PHP.
should be and::
Probably `app.device_manager = None`; I don't think we do a hasattr check.
в пси+ тоже был этот краш?
Looks good to me.

The advantage of making the changes in DVDAudioPlayer is that the changes will also work for the xvba branch.
In the future, maybe don't merge if you have comments that should be addressed - I won't be offended. I think it's simpler to hold pull requests pending and allow more commits to be added with fixes, rather than have a bunch of pull requests later fixing these issues - or worse having those fixes in unrelated pull requests. This way we'll also have a clearer idea of when features are "complete".
This is fine, as long as there is no matching `--` `intypeof` behaves exactly like a bool.
FWIW, if I bump the socket timeout from 500 to 550 ms, the test passes in release and debug mode.
Why? Is any other code depending on it?
Yes this is intended, even if the real encoding of the file is UTF-8 the round-trip with ISO-8859-1 will not corrupt the characters. The opposite would not be true, an invalid UTF-8 sequence would turn into the unknown character (U+FFFD) and then transformed back to ? in ISO-8859-1.
I don't think there is enough "feature" to split the class in 2. I believe 1 class that contains everything that is tooltip related is good enough. 
Every object can always be split into smaller objects, but the line has to be drawn somewhere. What if you want to control the delay, color, position, behavior, animation,... of tooltips ? You would create 1 class for each property ? You would then need a tooltip manager to control all of those properties because you don't want the application developer to manually instanciate all those classes. Why not have this ctkToolTipTrapper class be directly the "manager" of tooltip properties (display of tooltips: visible/hidden, plaintext/richtext...) and keep all of the work in 1 class ? Having separate classes sounds a bit over-engineered to me, especially when the logic of those can be written in 1 line of code (hide tooltip, convert to richtext)
The name ctkToolTipTrapper might sound a bit too specialized and it could be renamed into something more generic (ctkToolTip, ctkToolTipManager, ...).
It would definitely help to log the command to log_filename. Currently commands are in log and output in log_filename. It's a bit confusing and hard to read.
Just give me 5 minutes more, Its nor finished yet, could you use one after this one please?
Hey Mike, 

Thanks for forking the gem, and the mention on your blog.
Let me know if this works, and I'll change the dependencies. 

Just out of curiosity - what is it that 0.4.1 gives you that 0.6 does not?
Right. Sorry, obviously I was too tired when I committed the change!
Either way Arcane Explosion has wrong DBC data (What a surprise from Blizzard, jk...) or it is supposed to be handled somehow otherwise because of that flag. For example Holy Nova has no Interrupt Flags, nor most instant casts like DoTs/HoTs
0f465bfea391c7953a4a21a5ad7043601e014b06
I'm mainly concerned about memory use
Now we have no way to install dependencies (update_ve is gone), and they are not described anywhere. In practice, this means a lot of guesswork for users.
Ah okay now i understand :) i got the meaning of the isIgnoringCancelled wrong i thought it should ignore if a event is cancelled and past them through
This size adjustment is correct, but should be done before you read in the buf, immediately after you read in size.
no 2.3-dev is not included in 2.2.*, it is <2.3 (stable, but it's bigger than 2.2.999999 (which is the upper bound of 2.2.*)
Later updated unittest for this change here: https://github.com/jedie/PyLucid/commit/0d8845a12fda8a88db59acf10d6880de71516481
this should be conditional based on the rails environment, not commented out.
Fair enough, I was just thinking task would sit nicer along side the controller directory.
why do you say so? :)
Forgot to add issue-tracking number. #10.
https://bitbucket.org/mumurik/xyzzy/issue/10/set-menu
Perhaps we should mention RabbitMQ, I know we take it for granted but this might lead people to thing a minimal vumi deploy can be pure python ?
Thanks, the compliments offset the normal critique, we need to keep this up :)
Whoops. Should be fixed now.

On Mon, Jan 9, 2012 at 8:59 PM, hastebrot <
reply@reply.github.com
> wrote:

> Looks like `test_continuous_rv.py` is missing now.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/mrocklin/sympy/commit/f224fd36456e15d09f3d16040c06ee22ad550fc9#commitcomment-851872
>
I originally had this as returns retained and was going to add CGColorRelease calls after the colors' transfer into CALayers. But I couldn't tell what the retain rules in CALayer were so I didn't touch it. This looks good.
WPF rendering seemed to be working without these methods, does implementing them fix or improve anything? I was never clear on if these methods were needed.
In sb we did it to the RSS searches...

Sent from my Windows Phone
From: shypike
Sent: 2/26/2012 4:31 AM
To: thezoggy
Subject: Re: [sabnzbd] nzbmatrix changed "TV: Divx/Xvid" category
into "TV: SD". (4c259d8)
For which API call is this a required parameter?
Or is it for the RSS feeds?

---
Reply to this email directly or view it on GitHub:
https://github.com/sabnzbd/sabnzbd/commit/4c259d8d2b3a2f10585c5906f544639fb3cb6d8d#commitcomment-1012120
It's supposed to be the date the page was first published, isn't it?
I figured that printing characters directly would generally be more efficient than printing entire strings which have to be allocated somewhere. Also, it amounts to a form of explicit loop unrolling. In this code in general, I opted for printing individual characters over strings. If that's not the right choice, it's pretty easy to change.
thanks for pointing that out
You sir, just won an internet!
The tests you have are good, but they don't cover the behaviour of the external script:
 * what happens when the external script fails?
 * what happens when the external script is `sleep 900`?
 * what happens when the external script prints to STDOUT, and to STDERR?

You also don't test all the current permutations of invocation - I don't see testing of `failonfail` true and false differences in behaviour there.  (I think that feature should go, but if it stays, it should be tested.)

I don't see tests that the places invoking the alert do the right thing - the public API needs more testing.

Otherwise, with some polish and a bit more thought about how it interacts with the logging subsystem, seems like a reasonable thing to me.  We should probably run this around UX a bit and see what they think.
help whats type? make dump and publish it.
Ok, i thought i was the only one with this problem. I have dedicated servers for mysql and dedicated servers for MaNGOS cause my DB requires to keep at least 60 GB de RAM in use for fast work.
I learned something new from this, thank you! I haven't seen this syntax before.

Do you think we should be consistent though and use, for example, `edit_user_path(current_user)` here and the other places?
EricV: we have discussed the patch you have provided. The problem is that it is high risk to get a patch of this size into RC2.
On Wed, Feb 29, 2012 at 01:25:35PM -0800, goodfeli wrote:
> What rule is this an exception to? Everywhere else that we use a letter as
> an abbreviation, we make the letter capital. 

The rule that caps are reserved for class names (and constants), function
names are lowercase. At least that's the pep8 naming convention. 

> (RBM, CAE, etc) I did change the filename though because we have been keeping filenames lower case so far (rbm.py).

True of RBM, but I'd argue that that's because it's a class name. (there's no
CAE class, it's ContractiveAutoencoder, mainly because CAE is in far less
mainstream usage than RBM and the C is potentially overloadable,
"convolutional" or some other variant we haven't thought of, whereas someone
coining another "R____ Boltzmann Machine" would have decades of literature to
fight).
Isn't lib included by default?
Looks like divs that has only one characters will not be added to the text layer
thanks, I like that better too
I'm worried about ICS - Samsung is clearly janking around with weird stuff (like the secure container lockout), and they haven't fixed any widely documented bugs like the fuel_alerted wakelock bug...  I'm starting to suspect that their ICS kernel is a major hackjob.
Tenés razón, entonces sería:

```ruby
all(input).second.click
```

Aunque sería mejor identificarlo con el link o algo más absoluto, porque si en los fixtures agregamos uno nuevo puede que ya no quede más en segundo lugar y el test va a fallar por nada.
Thank you @honarkhah 

where are you from?
True. That's not a big deal for now, but it would be good to make sure there's a test.
Hey @JMobius, I've been pondering the Charm display lately and started to clean up a little.
Trying to make the footprint of the SVG-based system as small as possible.

Doing so, I came across this line. From an experiment, I reckon it is necessary to keep the current Charm tree on display when a filter is set.
Is that it, or is there anything else, something less obvious to it?

Just so I know what to watch out for when I change stuff around.
+1 zenix. I guess the baby's leaving you alone now? ;)
That one already exists in the parent class.
UI真心不错 吐槽一下Logo……
那啥 有想过支持Linux么，期待呢>3<
This comment makes sense in Python, but is really obvious in Ruby.
() ? true : false is unecessary. The line should simply be `return isset($this->extensions[$name]) && $this->extensions[$name]['ext_active'];`
A string replace for each API invocation is not efficient but it works.  There is probably a better way but I wanted to get this code in and hopefully available soon to my github-oauth-plugin.
I agree for the use case. But definitely not on the use of a static member.
This singleton enforce your app to have only one instance of the service running. What if you want several of them (multiple account for instance).
Again, can you please access the username using the service from the view model ?
So this is my fault. I did:
qmake
make
as the guy named iceburk said in http://qt-apps.org/content/show.php/PosteRazor?content=9043
Everything's ok now.
So the tst_ was compiled too. I'm making package (.ebuild) for gentoo so could you please make a download for it (a persistent one with version in filename as Gentoo will checksum soure package)
just need to delete this
sql to dbc system was made for this purpose 8P
What is the reason for referrer to be checked?
User should set it themselves if they run into this issue. I tried compiling assets locally and didn't run into any issues.
I don't know whether it's the problem, but you should use the scripts from the repository root: so it should be
C:\b\tw> scripts\make_release.py 'win32'
Oh, my God! All this stuff looks like to a fragile domino ladder.
LGTM, thanks.
Mailed you the screen shot. 
Compiler rejects __bridge because it is not transferring out of ARC to non ARC code.
Yeah ... somebody already spotted that. A fix is pending in #934.
这种屏蔽手段能成功绕过问题，不过最好还是彻底解决集火目标错的问题
That's why I think this should be in help files rather than having a special function for some users special needs. Copy paste a command line is easy too
But why not put that back, that could be part of the "standard" `dsm-control.sh`
You're absolutely right. That didn't make sense. I checked out the example again, and am now using getCookie within the WebSocketHandler. Thanks for the suggestion!
Please check more careful what you commit. I'll go and remove the ~HEAD file...
Tem certeza que precisa desse unless total < 11 ? é alguma verificação ? 
Até aonde eu analisei ele vai voltar total do multiplicador, mas ainda não estará com restante do resultado do multiplicador.

Exemplo: multiplicador = 234
(234%11) unless 234 < 11 ?
I actually wanted to check szID since I noticed it is empty when an URL is used, but I guess it doesn't matter which one gets used.
I'll commit your changes, sounds good.

As you can probably tell I don't have too much C++ experience. I wrote some a few years ago, but I was not using much of the basic functions and character arrays then as higher-level classes were used.
Да, она важна. Иначе через полгода мало кто в коде разберётся. :)
отключение помогает. правда можно еще поэксперементировать с underline при наведении на ссылки, может там как-то еще влияет..
For ID, we can not retrieve from app.session.

For all others functions they are just shortcuts
custom added to prevent break in date liniearity
Can you submit the doc changes in a separate commit please ? I am good with the code change, but will have remarks on the doc changes

Also, add your name in the contributors in the headers of the files you are changing

Thanks for your contribs !
rename to loadFiles()
please add a comment to explain what chars are you trying to extract (with an example)
OK.  You should *never* put a DCM in any piece of HDL code that is not at the top level of the design, so an #ifdef is not appropriate here.
Great fix!  I've worked around this several times and never thought to change the ordering.
Glad to see this. :)  My only complaint is that you apparently only really versioned the final result.  Use it from the get go!
thanks leak as stated this works and fixes crashing warden.
We probably did. I did the new voting logic on a new branch so I'll have to go cherry pick the none voting-related changes we made and get them in here.

Good memory sir.
thanks!
good job, just as always :)

keep up good work!
Nothing to do with this commit, but if we are to merge our engines in the future, we might start looking for better checks than ```targetThor```.

One simple way is to assign properties to all engines, and check on those properties everywhere rather than engine names. This also makes easier to create/destroy engines or refactor them.
If it's already sorted, then the lines above will convert shuffle to sort/group that will ultimately be optimized away.

Maybe this two conditionals could be re-arranged to avoid that.
I know you passed of your other plugins, are you still actively maintaining this one?
That means the command handler is outdated, but the SQL query is not wrong per se. As long as the column count equals the value count the query will be valid. In this case the `emote` column will remain at its default with every new entry using these commands.
Have you considered setting :max_retries on the AWS::S3 interface when you construct it?  

By default it retries 3 times with an exponential back off.  You can set a more aggressive retry count (e.g. 9) and it will retry with the following sleep durations: [0.3, 0.6, 1.2, 2.4, 4.8, 9.6, 19.2, 38.4, 76.8] (total of 153.3 seconds of wait).  You can construct multiple s3 objects as well:

```ruby
s3 = AWS::S3.new # default 3 retries
s3_stubborn = AWS::S3.new(:max_retries => 9) # for multi-part uploads
```
I see it's shared between a few places, so I guess may make sense
stopped code review here for now, will continue later
Gah!  So sorry, will revert this...
y vosotros lo habeis matao asi? vaya cabrones ¬¬
it does on our end I believe
Should I leave it to 0, switch back to IPPROTO_TCP ?
Fixed in 2eba479dd5b6b1e0aa1722329182dc1eeba61359
re: https://github.com/Maescool/Catacomb-Snatch/pull/574#issuecomment-4222479

As far as I can tell, this class seems to be out of sync with the main branch.
I kind of half-assed this one the first time through. ord() and chr() also work to map capital letters and other various symbols which muck this up a bit. I had to go back and work the kinks out because a problem in a later chapter requires a correct, working version of this program.

This handles cases where the amount to rotate way out of the range of lower case letters (97 to 122) in either direction and brings them back into that range. It also converts all letters to lowercase. I should probably also include an str.isalpha() test to filter out punctuation marks and other such things but that'll be later.
Hi,

By "normal", do you mean "development"? Since "add_dependency" and "add_runtime_dependency" are aliases IMO. If "ruby_parser" is required just by rake tasks, then there should be used "add_development_dependency" instead.
This renders WebSocket transport special. Also, of concern is amount of service data sent along with a bunch of short messages (which are the key ones in some setups).
I would make WebSocket transport take a payload, as other transport do. This would also help consistency.
buzzzz
ixi .. outra zica..
I think I see what you're saying.

IMO, the account GUID is the GUID of the record on the server. That will be globally unique. The profile column just indicates which profile this is for (which might be redundant, but is harmless). We know which Sync account we're connected to implicitly.
On Sat, Feb 4, 2012 at 9:24 PM, Leon Timmermans <
reply@reply.github.com
> wrote:

> Also, PERL_MB_OPT has exactly the same issue (it always overrides the
> Build.PL arguments), and no way out.
>
>
Did I do that?  That seems bad.  Command-line args should have highest
precedence, followed by Build.PL args, followed by the environment
variable.  What's the current state of this?

Although "no way out" is a bit of a dramatization, since the way out is to
do 'PERL_MB_OPT= Build install' or whatever.
Ok and what I get is this
http://subir-imagenes.es/images/sinttulml.png
Sorry for being very stupid in my last commit. I was playing with <if not ..> and <if .. is not> stuff. But that got checked in :(
@roqu3: I compiled, flash, but my phone didn't boot
Fixed: 6923730d4b3f5789c1bf380989478e20665bd686
Here's a short screencast that I made to let you see how I commit files into github using git gui (same program you are using I believe).
https://screencast.com/t/PU8XOEZfT
yea, unless you do some changes to the stock recovery_ui.c
Let's remove it
I just spent some time troubleshooting the tests to get them to work when actually testing Regal code, and after doing a "make clean"

It looks like we actually need:

1) A dependency on lib
2) $(GTEST.OBJS) instead of $^ when creating the static library as a consequence of (1)

here.
I added this test before adding the fix, and the test failed when I ran the suite. After adding the fix, the test passed. So I'm not sure why it passes for you under the current code. I could look into more, if you'd like, though I'm rather swamped at the moment.
_getDefaultSelectedNode might be a better name
Ha! How could anyone have used multi-reads before?
s @ _ ?
Partly.
Views are all merged in from packs, and if there was a module here before, it goes in  a module Model
Used for debugging messages only
and test it :)
Sorry for this, but I need a virtualenv.
Not worth saving one letter: ``sampl_start`` -> ``sample_start`` (or ``sample_radius_start``?)
Another terminology switch. It's QPID broker above, stick with that instead of referring to the daemon directly.
Hi, sorry, I was too quick on this comment. This works as advertised.. :)
Comment ça, ça « marche pas » ? Forcément, si tu testes sous Windows, ça va pas marcher, mais le site tourne sur un OS normal.
I'd write it lowercase as it is a keyword.
Can you maybe use one of the entity initialization strategies?
I noticed that too but it boils down to a single 'is NULL?' check in release builds.
@shtylman I think normally node core code stays away from too many C++isms.
1) switched dependency to netty-metrics-yammer (it brings metrics-core)

2) did not migrate yammer.Meter -> netty.YammerEventRateMonitor  etc.
since netty monitor api feels clunky and incomplete compared to the yammer.
Just in case it's unclear, it should be:

    throw new NullPointerException("newInboundMsgBuf");

to be consistent with all NPE messages in Netty.
Might as well test the astype method too.
And while we're at it, note thing one in there as well. You might reference the [GN paper](http://www.ncbi.nlm.nih.gov/pubmed/22832344) there too as an example of where we've used this.
а этот if с предыдущим нельзя объединить?
This is because this form is closer to natural language
I don't think fetch is a good idea here. You should not be required to have a tracker per handler.
I think we are not using the `Example:` label in the documentation anymore. :scissors: == "Remove this line"
I think this is better:

```ruby
@current_config ||= if ENV['DATABASE_URL']
                      database_url_config
 	                else
                      ActiveRecord::Base.configurations[env]
                    end
```
I am going to update this so that it will try to use NetAddr if the user has included it, if not, it will fall back to IPAddr.
This is gonna be Real Bad for performance, isn't it?
Do we really need to check for both string and symbol keys?
I typically maintain a single commit on top of the pull request named "pull request feedback", squashing new feedback into that commit if necessary.
I have become fuzzy on the details of the nullary-empty-parens-overriding interactions.  Let's just say we're better off not having to know.  Since clone() is inherited from java, the parens come with it; it should have parens everywhere.

    class A {
      def foo = () => "a"
    }
    class B extends A {
      override def foo() = () => "b"
    }
    object Test {
      def main(args: Array[String]): Unit = {
        val b = new B
        println(b.foo())
        println((b: A).foo())
      }
    }
It looks like `missingMessage` is created here just because `filename` is N/A at error throwing point.

If that's true, then probably `symbol.associatedFile` would help to evict this nuisance.
On 14 September 2012 01:34, Josef Perktold <notifications@github.com> wrote:

> In scipy/stats/distributions.py:
>
> > @@ -2772,17 +2763,17 @@ class exponpow_gen(rv_continuous):
> >
> >      """
> >      def _pdf(self, x, b):
> > -        xbm1 = arr(x**(b-1.0))
> > +        xbm1 = asarray(x**(b-1.0))
>
> do we need the asarray in the _xxx methods? the main method should only
> hand off arrays
> the asarray is included in the _xxx methods not consistently, see _logpdf
>

I removed the asarrays here, and it works:

 In [5]: exponpow.cdf([3,1./3], 3)
Out[5]: array([ 1.        ,  0.03702849])

 —
> Reply to this email directly or view it on GitHub<https://github.com/scipy/scipy/pull/310/files#r1603651>.
>
>
Please remove ``:func:``. It's unnecessary and looks bad in a terminal.   `` `convolve` `` works
Most sparse types go through coo_matrix as an intermediate step, but some
(mostly just lil) do not. I'll update the docstring to say "most sparse
types" require a C- or F- contiguous `out` array.
Event will never be `null` here so we can remove this section of code.
@khughitt this is too much for me, could you see if you understand it?
EVE?
Subtract and compare against a small epsilon.
I think it's fine.  The Wolfram Functions site is a solid resource.
Take it up with Guido.
Add a full-stop at the end of the sentence.
lowlevel->low-level
It is not related to DeferredVector, however I feel that MatrixSymbol and DeferredVector are related. This is offtopic so I will post is as an issue http://code.google.com/p/sympy/issues/detail?id=3287
ok, then we can edit the generator.
I'm sorry, I don't understand - what do you mean by addressing it?
@certik I missed that comment in my earlier push. Will address that soon.
you need to check for numpad keycodes too. they are different.
I have already fixed this; did you run `git pull --rebase git://github.com/yihui/knitr.git origin master`?
两个空行去掉一个空行，下同。
Yes, you right. Just did not see you wrap it in `' '`.
I think it'll be better to keep cache and cache flag separated. That was my fault at the beginning. So passing :cache should make :cache_enabled true, but passing true should be forbidden. What do you think?
Add a new line to make git happier? Files should end in newlines. Seems to be a thing.
Is any bounds checking on row needed here?
Should be renamed to `phpbb_di_pass_kernel_pass`.
I'm able to understand what it's supposed to mean. I just thought you forgot that. It's not a real bummer. :D
None of these functions belong into the base class in my opinion. They are all rather filesystem oriented and thus should probably be in the file driver.
75996a6
use PATH_MAX?
This still needs to specify to link against libzmq.

+libraries=['zmq']
those I mean
TIP: In the future, if you're making a commit against a non-beautified file (existing file has lots of tabbing and spacing problems), do it in two pull requests.

1st pull request ... beautify the existing file without any of your changes.
2nd pull request .. add your changes (beautified).

The advantage is that it will be easy to tell that everything in pull request #1 is purely aesthetic (to fix code style), and everything in pull request #2 is functionality. Right now, it's hard to tell which lines of code are new functionality, because so many lines say they've been edited in the Diff.
We should be allowed to add a decimal here.
You can combine todayPatientGrid and sevnDaysPatientGrid into one common component, "patientGrid"
Don't include this file.
lol this line's not better @adamgamble
@pelson Done.
还需要typeNames么？
fixed the pep8.
extra character added for no reason.
Don't take apart and re-assemble every plan node in every plan -- it's not very efficient.
Instead, call recursiveApply on each child and ONLY take apart the parent node if the return value is different from the input value.
Человек вообще может быть офлайн. Смысл гонять его в интернет за скриптами?
E.g., cabal-dev uses `packages-7.4.2-conf`. We probably want it to be something like `.cabal-sandbox/package.conf.d/ghc/7.4.2`.
I'm not sure if operator== has been implemented for the String object yet, but I guess if it compiles it must have been added (if the test fails, it's quite possible that it was implemented incorrectly)
What is "u" in "bbu"?
Nothing changed, this is the Linux part from above.
Seeing the number of times this pattern with futures and then double loops has come up should there perhaps be a helper for it?
Because there is not html tags in msg, so there is no need to mark it safe.
what's the difference between urlparse_cached and urlparse (I would guess that the former is the same thing but memoizes its results; just verifying). Is there a reason to not use urlparse_cached here and on the next line?
I'm still learning all this :-)
Is the version not enough here?
We don't accept `md5` hashes anymore as forgeries are too easy to create. Use `sha1` hashes instead.
This is done automatically for keg-only formulae.
The `else` part can probably be ditched in favour of something like:

```ruby
if path...
  path.sub!(..)

  path
end
```
we should be consistent and specify `infinity` timeout here as well.
when requesting /alias/owncloud/index.php, it require lib.php
``__FILE__ == '/var/www/owncloud/apps/lib.php'``

``OC::$SERVERROOT=str_replace("\\", '/', substr(__FILE__, 0, -13));``
is similar to 
``OC::$SERVERROOT=str_replace("\\", '/', substr(__FILE__, 0, strlen('/apps/lib.php')));``

after that, 
``OC::$SERVERROOT == '/var/www/owncloud'``
Supprime les unused variables plutot
I still think we should only target when filename == '3.jpg' or whatever the youtube one is.  Seems like shortfile name should be fine as long as their unique and the only one we know of that isn't is the youtube one.
Would this break if I submit `Zoë` as a value?
this achieves the same functionality basically as the mocks parameter from the loadModule functions. according to our discussion it's better that there is just one way to do things, so we should remove this function and also take care to update the tests that use it.
``chdir`` may throw an exception. please do complete error checking.
We can land this for now but we should really use the component that Honza wrote for IE to do generic parameter type checking and optional parameter handling.
Maybe this one could be simplified as follows:

``` ruby
def use_session_stack
  defined?(@use_session_stack) ? @use_session_stack : false
end
```

What do you think ?
Lost your formatting.  Can you re-do with code escapes in your comment?
我这个安装指南带有强烈的个人偏见，比如对于多个版本的R我持反对意见，不要干这种事情，永远遵守一条原则：用最新版本的R，只用一个版本的R。对于那些旧版本下能运行而新版本下不能的程序，就一条路，改，改到它适应新版本。
Stefan,

Would you prefer we pull this change in unchanged, or should we finish #361, and use that for these tests here?
@avandeursen 
thx for reminding me. I will fix it.
`super()`
Always use === operator in Brackets.
There is already an X icon used for the Working Set list. I think you should be able to re-use that one.
In the loadAgents() function, we're reading from '_enabledAgentNames'. Extension developers can put any string they want into this map (through enableAgent). That means that an agent might not actually exist with some name in _enabledAgentNames.

That's why we need the check in the loadAgents() function.

However, we only add a name to the _loadedAgentNames array if an agent with that name a.) exists and b.) has a 'load' function. All agents that have a load function should have an unload function.

So, when we're in unloadAgents and we do agents[name].unload(), an exception is appropriate if unload is undefined (or if agents[name] is undefined)
Oops, never mind, you have getMenuItem() above.
Done, though the core menu creation code doesn't use them yet since the addMenuItem impl doesn't support relativeID and position yet

-----Original Message-----
From: Narciso Jaramillo [mailto:reply@reply.github.com] 
Sent: Wednesday, May 23, 2012 11:12 AM
To: Ty Voliter
Subject: Re: [brackets] Menu, MenuItem, key bindings, and Command API (review only) (#927)

> +    MenuItem.prototype.handleCheckedChanged = function () {
> +        // TODO
> +    };
> +
> +    MenuItem.prototype.handleEnabledChanged = function () {
> +        // TODO
> +    };
> +
> +
> +    function init() {
> +
> +        /*
> +         * File menu
> +         */
> +        var menu;
> +        menu = addMenu("File", appMenuBar.FILE_MENU);

Also, it seems like we should be creating menu groups and then populating them.

---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/927/files#r869837
I think we can keep it. The modified message implies (2) changes, disk and in-memory.
Minor nit: could we just pass 1 or -1?
Fixed. Added an editor ``change`` listener to update the line count.
That's weird, moving up the bottom line works well for me.
Isn't it best to pull in those PRs and then update these?
Please, make it async!
I'm getting the error at this line
Also, the usage of the full command string and the full document contents as object keys - surely needs to change
This is already in master... not sure why it is listed here. Please ignore it.
This is a weird change. Its kinda not really a bug but sorta is. At best its pretty much error prone if client_threads every goes away because cores won't be defined and will cause an issue below. It's best to just assume client_threads exists. You might also assert that its greater than zero when you're done setting it.
drop the if clause. self.coords[1:] will be empty if len(self.coords) <= 1, so the loop will not be executed either.
Write "User Accounts"
Yes, and I also found that the type conversion happens in rb_raise, (StringValue on the symbol). I will fix that too.
It is working :). Will update in a minute.
https://github.com/hmarr/mongoengine/issues/476
This test, and the equivalent rules one, should be split up a bit. In particular, the 404 and 400 tests should be separate tests. In an ideal world we'd have ViewTest prepopulate the history tables to avoid all the set-up here, but I don't think it's worth the effort too, considering how painful it would be to maintain that data.
We actually shouldn't need this at all. I'd rather is disappear. Since there should always be a default config when we're developing, and it will be compiled into the library when the package is made, we can probably just depend on defaultConfig always being filled properly. If it's malformed JSON, it's a problem we definitely want to know about.
YYEESSSSSSS!!
whitespace req, res
var options.
           filters;

    filters = {

That doesn't look too ugly to me. Multiple var declarations looks uglier.
Add a space after the first round brace here.
Also, you probably want to make the message in this textarea disappear when you click on it. Can you use a `placeholder` here?
See above re: line length.
We can remove this and lines below
I'd do this the other way round so that people can override this at the call site. Something like:

    response_kwargs = { 'content_type': 'application/json' }
    response_kwargs.update(kwargs)

then use `**response_kwargs` instead of the last two parameters to the `super()` call.
样例程序也要同步更改：https://github.com/qiniu/php5-sdk-example/blob/master/lib/qiniu/qbox/authtoken.php
What is the scenario when multicaster == null?
Would really like to see less TODO comments... something more appropriate like:

    Subsystem for Maven group repositories

etc.
this macro is not called?
Just messing around :)
Could optimise this with Enumerable#find, so it bails out as soon as it finds a value that isn't a number, or if it gets to the end of the loop then everything must have been a number.

And [return early](http://www.themomorohoax.com/2009/02/09/ruby-tip-return-early)!
Only this line and the next were added. The comments on the M98 and M99 commands.
will do, plan to commit this:

```javascript
        var toggleButtons = function(enabled) {
            for (var i; i < buttons.length; i++) {
                enabled ? buttons[i].enable() : buttons[i].disable();
            }
        }
```
Also apparently there were some spaces checked in that vim fixed
@Pajk ok, for the future, try to keep this kind of changes in separate commits for better documentation of the changes.
Can you confirm that English string will show up when there is no translation? If not, you need to copy English string to all locales.
done (although they're actually `RA` and `Dec` objects.)
For cases of converting enormous numbers of coords, one can always do something like `gc = c.galactic; del c` if we don't care about keeping the original coords in memory.  Though a `convert_to` method isn't terrible either.  Could also have a `convert_from` classmethod.  For example `GalacticCoordinate.convert_from(c)`.  Or the like.
Better as in it doesn't tie up a global VM lock for no good reason.
What if OutputDebugStringF is being called twice simultaneously? This doesn't solve anything, unless you have a guaranteed call of this function while still in single-thread modus. That's almost certainly the case, but if it is, why not make it obvious, and have an InitLogging() function, called in init?

I'm nitpicking. ACK.
These should be `index(...).data(Qt::EditRole)`. This will get the raw, unformatted data suitable for passing to other functions (and also get rid of "(no label)")
This works on Windows?
Before merging you can also remove the heavy `hasOwnProperty` calls here.
`Number`.
就算是也注意自己改了
but it is still not fixed
`QSingleWidgetLayout` automatically removes the old widget when adding a new one. There's no need for you to keep track of it yourself.
You don't update the object's hitBox as well.  For one frame, objects are not hittable where they are drawn in the client.

In addition, obj.pos is not enough to move some objects like bullets.  Bullets work in continuous segments and teleportation breaks this continuity.  The simplest way around it would be to kill the bullet when it enters the teleporter and recreate it on the other side.
Mhh, I was about to, but reading the code I see that it does in fact have *one* configurable variable that controls behavior.  That's one use for the plugin that we can't replicate if we simply load the magics in manually.  I don't use this much myself, but there are bound to be users of the functionality, so I think we should leave it be in the interest of not causing unnecessary end-user breakage.
I used the exact same tests in stlib json, but those seem clearer, so I will do that.
I'm not sure that I like doing this via a method. Why not just mix in all the proxies when we include the module?
I think this is approach allows for more flexibility. You can have different timeouts based on the file type or on application policies.
Actually, I've removed much of the exceptions, because they were only there because of the Trilead SSH client, which is now gone.
The braces here aren't necessary
Do you want to use the new `bool` that @tanoku introduced? Then when you call this, it would also be nice to use the `true` and `false` constants instead of passing in `1` and `0` so it's more obvious that you're passing a boolean value instead of a numeric one.
Craig - good feedback - both issues are now fixed.
It just has the desires effect of jumping back to the top. Have not checked whether it exists in the header
Is 10 a special number here? I can't tell if this is arbitrarily chosen or if it has special meaning.
out of date docstring
I know I know. "Strong opinions held loosely". Optional keyword parameters should be keyed. Feel free to ignore.
Can we get rid of this? Can whoever creates instance of this class instead make this call?

Can't point to a formal PEP or Wikipedia page but my gut knows that it's dangerous to call methods of self in the `__init__`. At least it's not explicit.
Agreed, but it keeps consistency with the rest of the middleware. And it's deep in a sub-sub-module, and used only by the middleware service, so it shouldn't really be a problem... :)
I am not fan of links without texts for accessibility. It can also be an issue with future CSS changes where tooltips would be removed.
Might be better to deal with text hiding in CSS (see https://github.com/twitter/bootstrap/blob/master/less/mixins.less#L106)
Why send an empty string through? `null` is fine.
Is it intentional that the references_ vector does not receive this "internal" name of the texture? Can we implement this without the concept of "internal texture names"?
should this do something cleverer with exceptions? Usually when I call Executor.execute() I don't expect a RuntimeException to bubble up
This is not Doctrine ;). Spaces around `!` aren't needed.
Even better would be if the `render` method sets the current hostname and scheme if only a path it passed.
why adding some commented code ?
I don't think it should be a notice. The fallback from fr_FR to fr is totally normal for instance, and this would be spamming your logs.
``ObjectInitializerInterface[]``
No need to use variable here. This can be in `if`.
One could argue the same applies to almost all methods of `OptionsResolverInterface`, e.g. `setFilters`, `setRequired`. 
They don't delete non-supplied items. But only add and replace.
unused $test
Added a new param.
you should not unset the properties of the class but only reset the values in the object only
How about using the standard string::find_first_not_of() and string::find_last_not_of() member functions, which will let you replace these loops.
dieser befehl fehlte. er sorgt dafuer, dass im live-environment alle assets an die richtige stelle gedumpt werden.
No, leave it, it is serving a good purpose!
Making it a property of the Feature would seem a good fit. Then either it's up to the Feature instance whether it's hard-coded, has a default, or requires it on construction.
remove for now?
It uses the new close method in lines 253-254. sys.stderr and sys.stdout are LazyWriter objects.
This line just checks whether stdout and stderr are redirected, in the same way as in
https://github.com/chris-fren/gunicorn/blob/0728287ed069d64172ccd957050a3885cedb355a/gunicorn/glogging.py#L152
Мм, что-то избытка тестов не видать
Решил потом дописать, TDD не нравится?
sorry, but this can't actually work: the equals condition is always true. this line should be changed to:

```java
.addInt(hostName.equals(getDefaultHostLabel()) ? 0 : 1)
```
Could we return something eve simpler - e.g. " "?
Fixed in yakky/django-cms@9a151107b13bc34ccc590c4696bc20b060822cc0
The log message should report the operation it is refusing to perform because of the condition. "Not adding IPv6 subnet for X because ..."
I'd really like to see this in it's own commit with tests.
Nitpick: class name should be fixed to match the file name.
missing spaces around `=`
Since this is so similar to get_position() it seems more intuitive to put the $album param last.  I know this is nitpicky, but API consistency reduces cognitive load for developers.
This method should also add the ServiceVerificationHandler to this newly created ServiceBuilder, as a listener:

          if (verificationHandler != null) {
              sb.addListener(verificationHandler);
         }
The purpose of SVH's settleTime isn't "the maximum time for the services to reach the desired state". It's the max time to wait for a PROBLEM_TO_START_REQUESTED event to be submitted for a dependent service after a STARTING_TO_STARTED notification is sent for a parent service.

Is this because a lot of work is getting done in the handling of a STARTING_TO_STARTED notification, and until that handling is complete, this service's dependencies won't become available?
How about this
        Throwable cause = controller.getStartException();
        while(cause instanceof StartException && cause.getCause() != null) {
            cause = cause.getCause();
        }
        if (cause instanceof RuntimeException) {
            throw (RuntimeException)cause;
        }
        throw MESSAGES.cannotGetServiceValue(cause, serviceName);
Fixed, didn't see that when refactoring into a separate object.
What dialog?

Sorry but I still struggling to make this work. Any code example? I see that autocomplete and dialog use a _appendTo function but without making use of a ui-front class.
I meant that the abstraction is a bit broken if it's not safe. Shouldn't `getWithinInfo()` handle this for us? Probably just `offset: withinElement.offset() || { top: 0, left: 0 }`.
I am very convinced if you have a fullscreen FTU app you would get your keyboard.
If it's not working then, reach me anytime if you need my help on that.

I propose to close https://bugzilla.mozilla.org/show_bug.cgi?id=803028 if you agree.

BTW if activity solution of FTU is tolerable, reopen it and tell me.

z-index cases are too many to manage with, there must be something out of my consideration.
But we should keep the code clean and everyone would be happy.
This code is duplicated from line 760 ... can we ... (warning infamous word ahead) *refactor* ?
Why not remove the default case and do it outside the switch?
What I did for time interval options in the "Synchronize" field was to give each of the options a value that matched their meaning; so for "3 Minutes", the value is "3m". I think that makes things easier for us if we have to go look at the settings blob for debug purposes.
This looks like a workaround to me. Have you find out the cause of this early `onidle` filing? Is this really a expected behavior of the Idle API?
Awesome! Im so happy about sharing resources :). Could it be possible to use the 'apn.json' also from shared? Could you add this small fix in order to have all resources working from '/shared'? Thanks!
regExp.test() with the presence of "i" should be fine with my search, I don't need to use match() just like you said.
And I moved the regexp ouside the isMatching() function to avoid recreating of regexp.
Just as a side note, I am currently controlling that the UI is ready to receive postMessages because I am finding the case where the onerror is even faster than the USSD window opening and it crashes... :-) No problem, everything under control...;-) Also controlling that there is network connection to let the user send USSD messages :-)
For videos, I've found that setting src to null causes a warning in the console.
To avoid that, you can do this:

this.audio.removeAttribute('src');
this.audio.load();

This is spec-compliant and causes gecko to release all media resources.

Not required for this bug, but something you should know.
This is necessary to make the differentiation between, when we have only 1 button full witdth and 2 buttons half width
The other option is using box-flex but it is deprecated
Yeah that makes sense to me but if we do that we should move the focus to the messageInput after the user picked a message recipient. That way I can just continue typing.
Save!!!