labofsahil/github-event-dataset-2012 huggingface dataset:
A couple of z-order fixes for Github ribbon

The search submit button and the "comment on this page" wiki link were both below the ribbon's z index.  This places them above so you can actually click them.
"Völva's Wand" requires the accent over the 'o' character.
I think that this is the wrong approach. If you call e.g. Detach(1), then the old event 2 becomes the new 1, i.e. the handles become invalid. It would be better to write this as

```go
e.handlers[handle] = nil
```

and then check for nil before calling a handler.
style wars :) I prefer the indent, it makes the code easier to read.
I think this has been fixed in a later commit.
period not comma :)
Yeah - compile warning on the unused g_curlTimeout would have been
commented out later.

On Sun, Jan 22, 2012 at 1:10 PM, Joakim Plate <
reply@reply.github.com
> wrote:

> Hmm.. so must have been some two stage commit. I did a git revert.. Would
> have expected the code before to compile, but apperently not.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/xbmc/xbmc/commit/bcbe164b25b84515d00021f22dc0093c769bc4fa#commitcomment-889229
>
cool, ok - thx! :)

Am 20.01.2012 15:58, schrieb Jupp Müller:
> Cool, noch ein Kommentar zu den commit messages:
> 
> Die erste Zeile sollte ohne '-' in maximal 50 Zeichen einen kurzen Überblick geben
> 
> Danach kommt eine Leerzeile
> 
> Danach bei Bedarf genauere Beschreibung, evtl auch mit Stichpunkten
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/ax3l/MDSim/commit/896cf76272661c9149619c941b21699d60238805#commitcomment-886266
>
Nice formatting change ;-) Should switch to the FSMDocTest approach for UntypedActorDocTest (wrt. `@Before` and `@After`), because that still leaks when tests fail.
Just curious.. why the crazy version numbers?
On Fri, Feb 24, 2012 at 06:12:12AM -0800, Jörg Thalheim wrote:
> With *setattr* the function has only to be constructed once. This way I want to reduce the dispatch overhead a little

Ok, no problem with this.

> I think the *status* wrapper could be inlined in the exception clause to return a dictionary on error.

If you add a check on the name of the command called, that's fine for
me. Otherwise, be careful, since this seems to be a result *really*
specific to ``status``.
Maybe your former change is harmless, yet I can't be sure.
It means: if not empty string. This is because you actually want the intScan to return Nothing on failure. If you enter "d2", it returns ("", "d2"). This is bad, because then the scanner seems to succeed.
Ecode should be Ecore
I would then replace `$<` with `$(makefile)` here.
```window > document``` is meant to be ```clientProp > scrollProp```. In this case, choose `clientProp`. If we didn't do this, IE6 would choose offsetProp, which includes the HTML elements default top margin of 2px.

Is that any clearer?
ah, good to know. i think the memory concern is moot because the _TSObjects get garbage collected immediately. you only start running into problems if you increase the total # of python objects by millions or something
Comment fits on one line?
Roman;
Sure, if that functionality is useful we can leave it in for the high level reporting of runs finished. I don't mind the dependency as much as the extra boilerplate code which makes it a bit harder since all the applicationbound handlers don't fit the distributed model well. I haven't finished up all of the logging revamp I wanted to do but can let you know how it looks after I get more time to work on it.
Все супер! Только вот нужно перенести это компоненты в каталог https://github.com/yupe/yupe/tree/master/protected/modules/yupe/components Модуль "yupe" - это типа как ядро =) А эти эксепшены будут частью ядра.
Won't action_node be overwritten on each loop execution, so that all the callbacks below will use data-action from the last node?
Wow - that is a pain in the ass...
Might want to add some comments in there...
oh :poop: this looks so prime for a qunit test
This was part of the original balance distribution. There are OS specific sections in the Makefile. I guess the comment would be clearer if it didn't explicitly name Cygwin.
Well, that was my first shot, so I agree. :)
We kunnen de talen uit de database laden...maar dan moeten we op de één of andere manier vlaggetjes hebben van alle landen voor het geval ze een nieuwe toevoegen

Op de manier hierboven moet Nederlands altijd ID=1 hebben anders gaat het fout
Is this really the correct way?
Just fyi, the `:smart` option is currently a bit broken in Redcarpet2. See these issues:

+ https://github.com/tanoku/redcarpet/issues/57
+ https://github.com/tanoku/redcarpet/issues/66
+ https://github.com/tanoku/redcarpet/issues/53

I'm currently using [a suggestion from one of the issues threads](https://github.com/tanoku/redcarpet/issues/57#issuecomment-2488442), but it's a temporary workaround:

    markdown = Redcarpet::Markdown.new(Redcarpet::Render::HTML)
    html = markdown.render content
    html.gsub!("&#39;", "'")
    html = Redcarpet::Render::SmartyPants.rende
Well, no better way that copy `Z.prototype.*` one-by-one :(
TODO when I've got a bit more time. Check to see if the job is active still and then return this, otherwise say it is a bogus jid.
Was already in the file
git bisect shows this commit as introduding duplicate values when posting habtm data with input('Tag') on a Post form.

request->data['Tag']['Tag'] should be array(1, 2), but now shows array(1, 2, 1, 2)
нафига?!! у меня же смартпойнтеры под оберткой.
this doesn't need to be localized?
Meta comments in the source?
MIght look bad to the purist but is at the same time an easy-to-read way to show comments about the architecture and not only about the code.
Not update ZPanel core with this line
Fixed this after I merged in the pull.  There was stuff like this, type cast mismatches, and extra closing brackets all over the place.
well the parameters should be in the order they are called out in the method, so if called remove ip from server, then param order ip, server
Well I believe when you're not author of code but you're adding it, then you commit it with `--author="Username username@email.com"` indicating *author* of code and `Signed-off-by` line indicating that *you're* committer.
неверный формат пакетов\недостающие флаги\еще всякая фигня = бэкпедал клиента.
mh weird, it fixed the graveyards correctly, didnt bug anything :<
Change this file to use AMD, since you're already making so many changes.
Ok Nate,

I will change it!
Thanks @visnup, I can see what you mean.
I'm trying to find one, believe me... but so far I haven't found one! :(
The important part here for \#90 is the broadening of the lock on the poll_mutex (lines 489, 562, 588 & 1590).  Everything else is towards connection time problems...
This is not what we want people!

I tried many ways, but could not get AJAX right.

What I've done here is:

If uname/pwd is correct "load the webtop.php into the "page" element in the index.php page itself." - Read it again.

Look at the address it is still http://localhost whereas it should've been http://localhost/webtop.php once validation is over and webtop access is given. So it has to be rectified..But what is done till now would be useful for that. Thats my intention of pushing this.
s/すいるように。$/するように。/
spelling - should be independents
Alright - can you rename _def to _wraps for readability (makes it easier for any dev to understand what that means by looking at the name)?

Also don't check len(permissions) - just checking `if permissions` is sufficient

And the big long `if` statement for checking permissions - it is more readable if you break that up into smaller bits - each check is actually a private method that returns True or False and then your conditional is cleaner to read. Try to avoid doing slashes on code as much as possible.
ben can you open a ticket with what you said here?
huh, why are you replacing the dynamic config file name with a hardcoded one ?
this doesnt change anything, but makes the code less clean
+1
Works for me, thanks niklas!
good call, wfm
Wouldn't it make more sense to make this a public final class, and use a private constructor on this? Using an abstract seems like the wrong way to prevent instantiating this class.
имя класса по стайлгайду должно начинаться с заглавной буквы
sure, looks like that was disabled in my upstream tree too. It was probably useful during development, but had bitrotted since. I'll remove it.
Behavior is unchanged.  For web sites deployed behind a firewall which re-writes port numbers, this allows a subclass to generate a correct sitemap without including the internal-only port number.
Awesome commit-hash: ``dead730`` xD
=======================================
To join Facebook, please follow the link below:
http://www.facebook.com/r.php?re=7197c7af542c7786fb7869b988c5a138&mid=5c44bcbG5af3e4dec5feG0G46&key=AQDckcSoD_5RhkJrOvxItncCnOJvXXKyT7V2juw5S01CZ_VtLdmEvxSGOYyr75GgW3jKUiSyp_JTg7PpvOG6efunVmg&invid=100002225017094
=======================================

Hi,

The following person invited you to be their friend on Facebook:

Bussiere Adrien (Invite sent: Feb 21, 2012)

Facebook is a great place to keep in touch with friends, post photos and videos, and create events. But first you need to join! Sign up today to create a profile and connect with the people you know.

Thanks,
The Facebook Team

To join Facebook, please follow the link below:
http://www.facebook.com/r.php?re=7197c7af542c7786fb7869b988c5a138&mid=5c44bcbG5af3e4dec5feG0G46&key=AQDckcSoD_5RhkJrOvxItncCnOJvXXKyT7V2juw5S01CZ_VtLdmEvxSGOYyr75GgW3jKUiSyp_JTg7PpvOG6efunVmg&invid=100002225017094

=======================================
This message was sent to reply@reply.github.com. If you don't want to receive these emails from Facebook in the future, please follow the link below to unsubscribe.
http://www.facebook.com/o.php?k=8b783b&u=100003563357694&mid=5c44bcbG5af3e4dec5feG0G46
 Facebook, Inc. Attention: Department 415 P.O Box 10005 Palo Alto CA 94303
He just said something wrong is with some variables ... I can manage that too but not solve :-)
I see logs again now and see these values:

        tx = -2147483648
        ty = -2147483648

tile = 0x0

These are repeating does this mean anything?
What's "promote()" for?  Should it be exposed to the user in the API?
I'll try rebuilding. If it does work, I may be able to get the newer SC68 to work with it.
this should be require_once at top of script, so no need to place here
Ah yeah, we weren't using it here :-)
Suggestion:

LOGGER.warn("Access to template file [{}] refused. Possible break-in attempt!", template);
there are no docs about this event being fired.
4.3.2 15211 and 4.2.2 14545 - https://gist.github.com/1717771
我們總是發現这些新概念在 Lisp 是最容易來實現的。LISP就像生物的DNA一样，永远不会过时。
Bien. Ya te anoto tu calif.
@nesquena which delayed_job should we be using? On github, there are two versions, one from tobi and another by collectiveidea. The latter seems more up to date and the wiki on tobi's version also points to that. I sent mail to both owners trying to clarify this, but havent heard back.

Do you know much about which is which?
Thanks for the tip :) So far, I've just googled isbn:s from my own books, and then added support for the top sites showing up in those searches...
Awesome, using the latest version cd17e6962f68fe7cd08e0dc9a88590a6c1019d05 works great!

> It might be worth letting Jon know that there's a use case for keeping highlight_modified_tabs around. Although the setting is still working at the moment, he may be looking at turning it off completely if it's no longer in the base settings.

Evan though Jon has removed the `highlight_modified_tabs` configuration option, isn't the ST2 theming engine designed so that a user can have settings that are not present in the default settings set that can be used in the theme. For example a theme could have some extra configuration setting checks in place specifically for the theme, and if the user wanted to use these extra settings he would simply have to add them to his user Preferences file.

In other words I don't think he can "completely turn off a setting".
In a style guide?
We reset recno in the cursor code, so this isn't possible, is it?
When you get a chance, can you check out the testable-charts branch and let me know if it creates any issues w/ your usage of the library?
I have been deliberately going out of my way to keep my changes completely separate. I'm new to this code so I am just being cautious. I also did not want to make any assumptions about where it should be put until it was a little more mature.

It does duplicate the distributor code, because it needs to do some of the same things and so that's what I started with, but it is really a replacement for the MessageRouter. This deeper integration is something that the current Distributor does not do, so I am not sure if extending the Distributor is a good fit. Although your idea may address this, I'd be worried about breaking people that are depending on it working as is.

Maybe calling my changes a Distributor isn't even right. Maybe it should be GroupMessageRouter instead. It needs some cleanup too. There are some parts that I don't think are getting used like pending message counts. These changes will make it diverge from the Distributor even more as I continue to refine it.

It also needs saga support, or at least testing. I do not have a good understanding of those yet so I have been putting that off for now. Any guidance for this feature would be appreciated!

I am also curious to know if my approach is just going down the wrong path or if you see some merit to this idea?
Or if you really love your ternary operator:
    *DstP++ = (uint8_t)((i == 1) ? (*SrcP++ >> 2) : (*SrcP++ << 6));
while #concat is entirely valid here, it's certainly more conventional to use #+= ...  [shrug]  NBD, but it does make me do a double-take.
posts 这个功能还没有写完的吧，之前我是故意去掉了的
However, I need to make sure that when you merge back to the branch with your new changes, you won't encounter any problem. So leave me a message when you are merging back to master.
This 'unless' block is similar to the one above. Can you extract it to a function?
Coding style: is_shiboken -> isShiboken
@dth1112: I noticed that the growerID gets reset each time so there isn't an easy way to save the growerID.
I disagree. In the very unlikely condition that you have multiple string literals on a single line *and* don't know what an octal escape sequence is *and* can't figure it out from the context, you can just google it or search within the es5 spec. Showing them the string won't help any more than giving them the line number. They need to go to that line to fix it anyway.
header.inc.phpだと中途半端に分かりにくいので、その路線なら思い切ってindex.phpかなあ(ぉぃ)
Probably worth being more robust, maybe this is the way to go:

> If you need to further determine if path is a package, use the isFilePackageAtPath: method of NSWorkspace.


http://developer.apple.com/library/mac/#documentation/Cocoa/Reference/ApplicationKit/Classes/NSWorkspace_Class/Reference/Reference.html#//apple_ref/occ/instm/NSWorkspace/isFilePackageAtPath:
good idea, actually we have hardcoded this in https://github.com/OpenELEC/OpenELEC.tv/blob/master/packages/sysutils/pm-utils/config.d/module with this it is user editable
I would love you to death if you figured out why the sound icon does not update when toggled....
Cool, thanks for taking criticism well. :) I was a bit worried I had ruffled some feathers as I haven't seen you for a couple of days.

I did some hacking about myself to see what I could improve. The asio headers are massive, and asio.hpp includes them all. I found that only include the ones I really needed like boost/asio/io_service.hpp reduced the compile time a bit. Also it is very rare that  you need iostream in a header file. In most cases you can include iosfwd which contains only forward declarations, and is way smaller.

Another big win you get by reducing the number of includes in headers is to cut down rebuild time. Edit one header, and the number of cpp files that have to be rebuilt should be reduced cutting down your build time, and improving the rate at which you can iterate code.
Ugh i realized this has all my previous changes as well. I am porting this into my latest pull request so this can be ignored.
These button elements aren't very DRY, but they were auto-generated, right? If so maybe we shouldn't care.
Added helper function InchesToMillimeters. Helpful when defining a pagesize.
中文commit描述测试---失败
No. The event-param is **deprecated**, we _shouldn't_ use it!
No. The reason is, something is wrong with Maven. For some reason it's not pulling the latest Bukkit builds from the repo. My bet is because ci.bukkit.org is shutdown, it's not publishing new builds.

If you want to test my new code locally, simply clone Bukkit, `mvn install`, `cd Multiverse-Core`, `mvn`

I'll beat Jenkins to a pulp tonight.
Russian

window.locale = {
    "fileupload": {
        "errors": {
            "maxFileSize": "Файл слишком большой",
            "minFileSize": "Файл слишком маленький",
            "acceptFileTypes": "Запрещенный тип файла",
            "maxNumberOfFiles": "Превышено максимальное количество файлов",
            "uploadedBytes": "Превышен размер загружаемого файла",
            "emptyResult": "Результат загрузки пуст"
        },
        "error": "Ошибка",
        "start": "Загрузить",
        "cancel": "Отмена",
        "destroy": "Удалить"
    }
};
- In paragraph 3 of the chapter opening, please change Elclipse version reference to "... Helios or greater versions."

- Installation Requirements should state the following for Java JRE: Java 5.0 JRE (or greater)
Well, I guess you're right, allthough I think adding a try .. catch clause to harvest.php, the caller in this case, would clutter it unnecessarily. There's not anything horribly wrong with the current behaviour of letting the exception bubble up. I was just a little surprised to get a seemingly complicated stacktrace when the problem I was having turned out to be simply that MongoDB wasn't running.
Haré más adelante una recopilación de los commits de TOC que hayais hecho y los meto en R2 junto con los nombres en el CommentSD:
Saves 2 nano-seconds?
Awesome. Thanks for the bug report. :)
I'll push the change to the Chrome Web Store.
Just `return new ExpectFetchDelegate(expected)`.
hi, this is still broken for iphone.
also no GL_DEPTH_COMPONENT32
Oops, sorry. Ignore my first comment, the caching in Ruby is a bit different from Java.

So the best would be to set `fetchTime` at the end of `memberCount` if the group has only one page and always at the end of `fetchMembers()`, i.e. when all members have been fetched.
Does this belong into this tool? Actually, this is very racing specific IMO.
What ist autosave? I cannot find it in the specification: http://www.w3.org/TR/html-markup/input.search.html
(fixed: https://github.com/citation-style-language/styles/commit/1f12db73cd72a08ab2258ad14ce37352036b3d1a )
Chrome Beta from the market works fine now. This change leaves the BUILD_ID set to IML74K which is what chrome checks.  Sorry we hadn't noticed your pull requests, changes to Evervolv are basically all done through our gerrit review: http://review.evervolv.com/
There was a sad bug with pagination in jqGrid, we didn't pass any offset to get_posts()
I just run this file as a view of a virtual handler, it's quick and easy. :)
So basically, this allows me to select which edits to commit? And therefore, executing
``` git add -p # some edits
    git commit
    git add .
    git commit
```
will give me two commits, one with those edits I selected in the ```git add -p``` and the other with the other edits?
http://rspec.rubyforge.org/rspec/1.1.9/classes/Spec/Matchers.html da uma olhada depois, acho que should have(turma) deve funcionar.
Another suggestion:

On CM7:
adb shell grep -rs 285F /sys
adb shell grep -rs "28:5F" /sys

etc..
Quarx2k.
i repo sync then the last problem is disappear . but new problem comes as below
hardware/ti/omap3/liboverlay/v4l2_utils.c:23:30: error: hardware/overlay.h: No such file or directory
hardware/ti/omap3/liboverlay/v4l2_utils.c: In function 'v4l2_overlay_get':
hardware/ti/omap3/liboverlay/v4l2_utils.c:55: error: 'OVERLAY_MINIFICATION_LIMIT' undeclared (first use in this function)
hardware/ti/omap3/liboverlay/v4l2_utils.c:55: error: (Each undeclared identifier is reported only once
hardware/ti/omap3/liboverlay/v4l2_utils.c:55: error: for each function it appears in.)
hardware/ti/omap3/liboverlay/v4l2_utils.c:58: error: 'OVERLAY_MAGNIFICATION_LIMIT' undeclared (first use in this function)
hardware/ti/omap3/liboverlay/v4l2_utils.c:61: error: 'OVERLAY_SCALING_FRAC_BITS' undeclared (first use in this function)
hardware/ti/omap3/liboverlay/v4l2_utils.c:64: error: 'OVERLAY_ROTATION_STEP_DEG' undeclared (first use in this function)
hardware/ti/omap3/liboverlay/v4l2_utils.c:67: error: 'OVERLAY_HORIZONTAL_ALIGNMENT' undeclared (first use in this function)
hardware/ti/omap3/liboverlay/v4l2_utils.c:70: error: 'OVERLAY_VERTICAL_ALIGNMENT' undeclared (first use in this function)
hardware/ti/omap3/liboverlay/v4l2_utils.c:73: error: 'OVERLAY_WIDTH_ALIGNMENT' undeclared (first use in this function)
hardware/ti/omap3/liboverlay/v4l2_utils.c:76: error: 'OVERLAY_HEIGHT_ALIGNMENT' undeclared (first use in this function)
hardware/ti/omap3/liboverlay/v4l2_utils.c: In function 'configure_pixfmt':
hardware/ti/omap3/liboverlay/v4l2_utils.c:196: error: 'OVERLAY_FORMAT_RGB_565' undeclared (first use in this function)
hardware/ti/omap3/liboverlay/v4l2_utils.c:199: error: 'OVERLAY_FORMAT_YCbYCr_422_I' undeclared (first use in this function)
hardware/ti/omap3/liboverlay/v4l2_utils.c:202: error: 'OVERLAY_FORMAT_CbYCrY_422_I' undeclared (first use in this function)
hardware/ti/omap3/liboverlay/v4l2_utils.c: In function 'v4l2_overlay_get_input_size_and_format':
hardware/ti/omap3/liboverlay/v4l2_utils.c:285: error: 'OVERLAY_FORMAT_CbYCrY_422_I' undeclared (first use in this function)
make: *** [out/target/product/jordan/obj/SHARED_LIBRARIES/overlay.omap3_intermediates/v4l2_utils.o] Error 1
make: *** Waiting for unfinished jobs....
hardware/ti/omap3/liboverlay/overlay.cpp:21:30: error: hardware/overlay.h: No such file or directory
hardware/ti/omap3/liboverlay/overlay.cpp:95: error: field 'device' has incomplete type
hardware/ti/omap3/liboverlay/overlay.cpp:103: error: field 'device' has incomplete type
hardware/ti/omap3/liboverlay/overlay.cpp:137: error: variable 'overlay_module_t HMI' has initializer but incomplete type
hardware/ti/omap3/liboverlay/overlay.cpp:142: error: 'OVERLAY_HARDWARE_MODULE_ID' was not declared in this scope
hardware/ti/omap3/liboverlay/overlay.cpp:168: error: ISO C++ forbids declaration of 'overlay_handle_t' with no type
hardware/ti/omap3/liboverlay/overlay.cpp:168: error: expected ',' or '...' before 'overlay'
hardware/ti/omap3/liboverlay/overlay.cpp: In function 'int handle_format(int)':
hardware/ti/omap3/liboverlay/overlay.cpp:169: error: 'overlay' was not declared in this scope

best regards,

jeff
А эту строчку не надо было перенести в слеюущий catch:
catch(OpenModuleException ex) {exception = e;}
???
This should be combined with the previous block instead of creating a new if block for every firmware we want nvram_net.txt_murata on.
If it's not used, let's remove this statement
* The test for touch support needs to be moved into a separate Feature test, registered with Y.Features. Dav and I have discussed this (we debated Y.UA vs. Y.Features).

It seemed like Y.Features was lacking the ability to register, retrieve and cache a test by name, but it may do that already. I'd need to go look.

Tilo, you're welcome to move it to a Feature test if you have the bandwidth. It's starting to proliferate, so would be good to centralize.

* The UA sniff for chrome < 6 is because older versions of desktop (mouse) chrome reported ontouchstart support, incorrectly. I don't have a bug report for it, but there may be one.
One last question, did you consider simplifying the loop? Are provides recognized and unique by their type?
line 210 - 236 : I wonder if we can't simplify this part...
This is a placeholder/reminder for future implementation.  I should make it clearer.

Lower down, we haven't done all the things necessary for MPI, but it's a step in the correct direction.
You have local variable here :)
what if buffering is already false?
it means that it would be a duplicate ack(); good point, BTW: if tries == 0 it should not poll()
This is something I want to talk about with you and Roland.
on the other hand, isn’t `null` bad, and `None` is much nicer?
The question is if REMOVED should ever be in the unreachable Set. 
Probably not. It is part of my the REMOVED status ticket. So leave it for now.
Doesn't say whether it's safe to mutate the resulting collection or not.
Yes, you may ;-) The reason—in short form—is that look-ups must work for “children” which are several path levels distant, i.e. getChild needs to be a custom implementation. And (at least at the time it was written, could be re-checked) it needs to be synchronous for creation because otherwise sending to a newly created remote-deployed child would fail.
FYI this was the actual bug that was fixed... we were reading on `client` which bypasses the wrapper, `bufio.Reader`, which might already have the data you wanted to read.
s/to/the/ ?
should add `if self.slides` to prevent `ZeroDivisionError` exception
ok, or a new request is fine too.
Sorry didn't get any email notification on your response. No the CVideoInfoTag object already exists in the CFileItemPtr from the list and both GetMovieInfo and GetStreamDetails just alter its details if necessary.
I dont think this name clearly represents what is happening.
Oops, this is an artifact of refactoring.
Test with a larger network, for example "127.0.0.1/24"
Bozo, com relação a

   #if ( $auth.AllBalances )

Pensando bem, acho que não é necessário avaliar nenhuma condição para que
qualquer usuário possa listar saldos, não é?

Transparência.

Então, consequentemente, não é necessário adicionar essa variável
$auth.AllBalances.

Vou simplesmente deixar sem nada.

Ok?

[]'s
What are these `)))` doing on a new line?
Add a space after the first `,`.
Unfortunately, the removal of POP_BLOCK here does break the 26_try_except_finally and 27_test_finally tests...  Perhaps a more elaborate solution is necessary.
Please don't change the indentation
This ivar isn't really needed – the getter can just read `cursor.backgroundColor`.
Please fix the indentation in this method.
Wtf, tohle nějaké IDE pochopí?
:+1: , maybe I'd use "superfluous" or just "extra".
add a space between "that" and "there"
Note that you do not have to export as Excel or csv. The *Plain Text (.txt, ...)* option actually downloads a QIIME-compatible tab-separated text.
Do we need these two lines?
I remember that we are changing some `inject` by alternative solutions because of performance issues. I don't know if it is still valid.
It doesn't have to be, but that's probably better.
Yes it is present on all versions of Rails 3.

OK I'll make a separate pull for 3-2-stable with a CHANGELOG entry, would that be enough?
3 fields now. And mention the now optional field after? Or just reword it somehow that makes sense.
As I suggest above, the majority of the method is a no-op.  You're not checking anything to do with companions - only the first case will match.
Oh, yeah, Ok.  I think that the only thing that is different is the calculation of A_L1 and using spsolve instead of solve.  If this is something that you and @pv want me to do, I am happy to do it.  Shouldn't take me more than 10 min.
Size could be done be the designer. It's the second heading on that page and h2 is also for SEO purposes (and it's a keyword)
I would feel better if you said 'if type(X.shape[1]) is int or isinstance(X, Integer)`.
@ellisonbg  My understanding is the Mul is doing this already.  For example:

a, b,  d, Y = symbols('a b c d Y')
A, B  =  symbols('A B C D', commutative=False)
Tr(d*a*A*B*b*2*Y)   is permuted as  2*Y*a*b*d*Tr(A*B)

Is this what was expected?
I think you can use `knit_hooks$get('plot')(c(f, dev), options)` to avoid a specific format like markdown here
What single wire support are you referring to? Or do you mean simply use one port and change its function?
Also, what if pos == path?
за это будет отвечать экшн  index в user/events_controler
there is no boolean in ruby :) only TrueClass or FalseClass
Is this still an active TODO?
Using a string to define a cache has been deprecated in order to avoid name collisions - you should use an enum instead.
Good catch.. We should just remove these entirely from the settings.py since they exist in the common..  If you tweak then resubmit I'll pull those in.
Why wouldn't you leverage `getReadMethod()`
Query should be build through `$db->sql_build_query()` as it contains joins
existent
Perhaps deleting files by memory driver is meant to clear compiled template files?

If this is the case maybe the logic can be moved to cache service somehow.
Also, stuff like:
`(some boolean thing) ? true : false`
Is wrong. Use:
(some boolean thing) 
Or, if needed:
(boolean) (some boolean thing)
Make sure the doc block format is consistent with the rest of the file. Should probably have no space before `*`.
This is very much undocumented. From what your description of $error I didn't suspect it contained language strings either. A thorough explanation of what can be in $error needs to exist somewhere.
You do not need to do this. Language variables are automatically available in the template without needing to be assigned manually. The other ones above your additions are there because we wanted to have a way to always display some language without always knowing the exact language key because it is dynamically decided.

Instead of renaming your keys to remove the "LAST_", just use L_DISPLAY_LAST_SUBJECT and L_DISPLAY_LAST_SUBJECT_EXPLAIN directly in the template.
Where is non-custom html? If there isn't any, drop "custom".
Why the empty row after definition?
> Also OpenGLWindow.h has some types from Xlib.h, Xrandr.h.

So why it is compiling with no errors then?
IKSEE this method has side-effect?
Tests for both of those as well as the unit being `%` and for when undefined is passed in.
ya `div.on_g` ro nazar ya baraye hamash bezar
What about request.get_full_path()?
Please change paltform to steam and move this to steam.xml
note that you are repeating the URL here, which means it will be harder to refactor later. Better to share the HOST in a common var
shouldnt need this <center> HTML, right?
这下面没有*在前面
Are these difficult to repair?
testNonDeterministic_RO_MP seems like a less-than-perfect name for a test that calls NonDeterministicSPProc
Get it, thanks.
Why? Anyway, fix it.
If you have it in the defaults, then there is no need to check for AttributeError in the plugin itself.
shadowing gettext
you should add a use statement for this class
you either need to apply Kevin's changes to have sample-specific styles or tell me why that doesn't apply here before I'll merge this.
maybe `App` should include `Comparable` and move sort to `<=>` method?
you never check that ``$factory`` is a closure, so you are allowing to extend an integer for instance (which will throw a fatal error)
Are these changes a linter suggestion?
e.g., What if (for whatever reason) the `$word` is hyphenated?
I appreciate what you're getting at it just seems along the same lines of questioning whether test() and replace() do the same thing when encountering a regex containing \s. We know that it does and so after testing with test() we feel safe using replace(). Being that every browser where \s matches \xA0 also matches \uFEFF reality seemed to be on my side. 

It wasn't a need, It was for the lack of need of a more strict inference. I had heard you guys were bending over backward trying to save bytes over the past few months.

I'll change the code.
Should I change the double quotes in the string to single ones or shall I escape them?
Except when we've stripped off tokens to refine context and/or generate a seed.
Why `global` and not `standard`?
Technically, the seeding happens before any iterations - so 0 iterations should return back only the seeding output.
Replaces \/ by /. Makes sure all slashes are forward slashes, because back slashes don't work in bash on Windows.
yeah, i imagine people will want to auth to an oauth api for the purpose of signing requests completely outside of any account. will need more work to get there but i think this will make sense for both oauth 1 and 2
I'm pretty sure there should be a space in between "td" and "class".  (Also on line 473.)
ah yeah I missed that one, typo. fixed!
Fair enough :) thanks!
(you have to set HOMEBREW_KEEP_INFO, as I noted before)
Please use a brace here.

> In addition we agreed on to use {} blocks on if statements with one line as well:
http://owncloud.org/dev/coding-style-guide/
I considered adding the same comments to the ```from ... import``` statements, but left it out for now.  With the discussion on removing ```from``` style it becomes sort of a moot point.

With the __all__ stuff if I go forward with a decorator for exporting a symbol the comments would be reduced to just #Constants since that would be all the __all__ ever directly exports, with everything being implicit with @export
So why would these actually ever be `None`? Stuff getting by validators? `wiki_edit_{age,karma}` both have defaults in subreddit.py
Assignment should not have the dollar sign:

    $CARTON_DIR=$(dirname $(dirname $BASH_SOURCE))

should be:

    CARTON_DIR=$(dirname $(dirname $BASH_SOURCE))
Adding a `formatDisplayName` helper method which seems straightforward; another overload of `createTestDescription` could get unwieldy.
Sneaky!


On Mon, Dec 17, 2012 at 4:32 PM, Beau Gunderson <notifications@github.com>wrote:

> In test/lib/services/linkedin.test.js:
>
> >
> >  describe("linkedin connector", function () {
> >    var pinfo;
> >
> > -  beforeEach(function (done) {
>
> Because that code is synchronous :)
>
> (mocha determines whether the code is sync by looking at the length of the
> function's arguments; 0 == sync, 1 == async)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Singly/hallway/pull/743/files#r2443241>.
>
>
I'm assuming there will be a separate script for Win? Or will this script run in git bash?
Nit: this would read better if there was a blank line above the ```} else {```
Minor consistency nit. The other _getToken* functions modify token and return a boolean. Proposals:

1) rename to _getStream
2) drop "_get" prefix from the token methods since they don't actually return a token. I can see why we don't return a token for brevity. It's especially nice not to have to check for truthy/falsy errors.
I don't see any issues. Please make the change.
Going to do this shopify side or something now?
Great idea using fixed arrays, much faster
Do we have a cached decorator? A map between `app_id` and default language for recently accessed data is totally something we could cache. And currently wrapping an app calls DocumentSchema.__init__ 16,000 times...
Completion Time is the default option. I want the user to either select completion or submission...not leave it in a state of "Select Time..."
Please make trip a symbol
same here : no double star
I am just saying that line 20 has spaces after `(` and before `)`. Not that you have to split arguments in separate lines. ;-)
It's not repeated datetime.date.today() != datetime.datetime.today().

As I tested, value is type of datetime.date. Can I assume it is always that type and never datetime.datetime?
Probably this is not needed anymore, depending how does mongodb backend do the work?
Heh, I like how https was used here, but http was used on the other requests.
Copy pasta gone wrong!
Popcorn.guid let's you pass a prefix string, so your guid could be "wikiCallback.......".
Any reason why this needs the extra security blanket? Isn't everything absolutely-positioned and completely ignoring DOM hierarchy anyway?
Yeah, I think that makes sense.
I would just check all the permission bits:

    return bool(st.st_mode & (stat.S_IXUSR | stat.S_IXGRP | stat.S_IXOTH))
Util being React\Async\Util?  No use declaration.
How is the value computed here relevant to the spec?
这里能不能也定义个ch.cmd命令？
can't locate symbol in module \"%V\"
As an API, Meta shouldn't be separate from the resource (LDPR or LDPC): whenever you get the resource, you have Meta with it.
Does not need to return any path.
I see references for this method in lib/resources/pulp.rb .. I think once all methods in pulp.rb  get moved to Runcible we can remove this.
Correction - I thought it was a reference to environment. In that case it's probably the best solution.
Should this be a scope?
@astrofrog @wkerzendorf @iguananaut  - I think I have a clean way out of this conundrum.  The `system` and `format` attributes for a `Time` object will be read-only (from the user perspective at least).  The one way to change the system or format will be the same as the current documented methods of accessing `t.<system_name>` or `t.<format_name>`.  There is actually no need to internally change the format or system.  

There is a possible performance issue if `t.<format_name>`  or `t.<system_name>` is accessed many times (since vals get generated on the fly), so as @iguananaut suggested some sort of caching might be in order.

I'm going to have a go at putting this in place and see if any problems come up.  I might not have thought of something.
What about suppressing the sysmon message entirely if it is for the sysmon process?
net.h, lines 219, 353, 388.  All three lines have nMessageStart = -1; but nMessageStart is unsigned int from line 167.  There are other warnings that pop up when I compile, but net.h seems to be in just about everything, so I see them over and over again when I build.  Does your compiler not warn on them?  Mine is a bit old.
Fixed in [73954ba544eefeb90aa4fcfd2fdaaea0a643863b]
Shouldn't be removed. This is used by GeoExt for the LegendPanel.
Sorry -- I have rewritten that one after checking the file out from master for the TokenObject rewrite so the fix got lost :(
the expectation is the first argument. Please keep them in the right order to have the right error message when it fails.

and you should probably use a dataProvider for this test
We need to write separate tests for "non-admin" and "non-member"
There is logic in some widgets that relies on things being run later. The `_send_relayout` method of `ConstraintsWidget` relies on `SheduledTask` to collapsed layout events.

Additionally, some things may test one way, and behave differently in reality if messages are not passed on a queue.
How could request be nul???
when you look at the implementation of the array driver, i build the map by reading the rdf name from the xml/array, its just in a different place. but a different driver might use a different strategy to build the list. if you insist we can also do a driver::getAllTypes method and build the list here and eventually cache this list (though the driver might be in a better position to judge if the cache is still valid)
Also disabled all "taps" as well. not sure if that is desired
move event.preventDefault(); here
again, this should use `self.log`
Prevents unacceptable formats from being rebranded as png by default.
Not fixed on this line, though!
check null, ImmutableSet.copyOf
>Hard-coding this is a bad idea. 

Fixed

>We also need to make it work if .remote is . (a dot) for branches tracking local branches.

Fixed
The ws* here is fine.  The params in "partial with params"  will includes the ws+ prefixes.
I cribbed this from https://github.com/mozilla/bedrock/blob/master/apps/mozorg/templates/mozorg/home.html#L42-79
One thing I was doing at one point, but don't think I'm doing any longer, is use uuids for these rather than random.choice. I'm not sure the result is that different, but it's shorter code:

    import uuid

    faux_title = str(uuid.uuid4())
pdb thanks you
hah, clever. I like. Not sure if it's possible to make this always work seamlessly though?
It's a bit ugly to tie the UI to RK4SimulationStepper.  Should we just allow zero value, and take a max(value, MIN_TIME_STEP) every place we use it?
Return the $conn...
"Created at: "
should be
"생성: "
if the value can't be a 0, than I would write

if (val) {
Upps, sorry then.

2012/8/9 Christophe Coevoet <notifications@github.com>

> In src/Symfony/Component/HttpFoundation/Request.php:
>
> > @@ -1306,8 +1306,11 @@ protected function prepareRequestUri()
> >      {
> >          $requestUri = '';
> >
> > -        if ($this->headers->has('X_REWRITE_URL') && false !== stripos(PHP_OS, 'WIN')) {
> > -            // check this first so IIS will catch
> > +        if ($this->headers->has('X_ORIGINAL_URL') && false !== stripos(PHP_OS, 'WIN')) {
> > +            // IIS with Microsoft Rewrite Module
> > +            $requestUri = $this->headers->get('X_ORIGINAL_URL');
> > +        } elseif ($this->headers->has('X_REWRITE_URL') && false !== stripos(PHP_OS, 'WIN')) {
> > +            // IIS with ISAPI_Rewrite
> >              $requestUri = $this->headers->get('X_REWRITE_URL');
>
> @nomack84 <https://github.com/nomack84> Symfony writes it without space
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/symfony/symfony/pull/5222/files#r1342103>.
>
>
That only works in a very specific case but not in general. Also, what's the point?
There is a BundleInterface
Missing space after the coma, after `use` and after the last closing parenthesis.
Missing space after the closing parenthesis below.
This is wrong. It cannot return null, and it cannot return a collection (an not an array of collections). It returns an entity (meaning the only possible phpdoc is ``object``)
These 2 lines seems unneeded.
Wouldn't the following code be faster ? It saves many function calls (array_filter with a custom callback does not benefit from a filtering written in C anymore)

```php
$conflicts = array();
foreach ($this->lock as $option => $lock) {
    if ($locked) {
        $conflicts[] = $option;
    }
}
```
Your'e not using the index you're incrementing anyways so get rid of it?
Love the module idea. Keeps it nice and dry. I don't think you can do private methods like that in modules, but I'm still knocking off rust. Either way I think it's a good idea.
_o之类的命名可以改下，更易懂。
It's not a work restart during initialisation
I wanted to do that, but I couldn’t get it to work.  :(

I’m *sure* it’s possible to do, though.  But I only know the basics of `str.format()` (and effectively nothing about the `%` style formatting).  

As for using `+`, I think `str.format` ends up being faster as computation cycles pile up.  If someone can figure out how to fix `%s`, then I recommend the following:

```python
'Expected topic({topic}) {method}'.format(topic=topic, method=humanized_method_name)
```
@longhengyu 同样注释和单元测试
понято, спасибо за подсказку. Сейчас сделаю фиксап, ты дальше ревьювай. Потом отребазирую бранч и приведу к нормальному виду, когда закончишь.
I have moved the urls
Still think that it's easy to read if you add the 
if () {
}

So you don't get confused with the } from the previous 'for'
let's file it :)
When and where the function is invoked? 
I guess the function is intend to handle the user cancels login page in the Browser app, so how do we know that?
Hahahaha ok! Fixed ;)
Use var overlay = this.overlay. No need to access an object property multiple times
Or, if you keep all of these controls, I think it would be better to disable them here in the HTML rather than having special code to disable them in Player.js
@sergivila @patrykdesign @authoritaire could you please check if the icons are OK with current size. Do we really need to generate the shadow via software when it could be part of the image itself?
I think these three lines of code should go directly into the init method.  No need for a special method just for this.
taggit before tower?
css path は2語なので csspath ではなく cssPath とすべきです（コーディング規約）

ちなみに、上の filesystem の方は1語の用例があるのでこのままでよいです（元の sfFilesystem クラスがそうですし）
I would better put it into MapAlgorithms because MapUtils supposed to be very simple
@snusnu wdyt? if we pass edge to every relation node that's created from that edge then we don't lose information about left/right relation that were used for join. we need that info to join "through" relation with target.
「」→ ""
原文には ** がある
Why yajl instead of json or multi_json?
Replace with:
```scala
IterableSource((0 to result.toInt by wt.toInt), name)
```
Why choose 7 for this magic number? I can't see any reason why you'd pick this number as opposed to 6 or 8 or 10.
Does this youtube_demo need to exist in /bin?

I don't think we should include other people's work in our repos unless we absolutely need it. What are we using this demo code for?
You could do the error checking on json_object here instead of in the calling code.
It should explicitly 404.
Yes, I was suggesting ``gain=None`` by default, which would mean that Poisson noise could be ignored (which it can at long wavelengths)
Does this need to be a global?  Even if so, this seems an odd place to set it, it's easy to miss when reading the code.  It would be clearer if instantiated closer (spatially) to it's use.
We should put the preferred name first (or even have just the preferred name).
This is a PHP bug or feature omission. You can't rely on the CN_match setting reliably. For getcomposer.org, the CN is actually dl.getcomposer.org (mismatch from PHP's perspective since it ignores the valid SAN entry for getcomposer.org).
I know you're just duplicating what I had originally, but I think this try(+) is unnecessary, right?
Added comment explaining.
But this is not explain the necessity to start with `;`, right? That's because I wasn't following you, I didn't understand which kind of comment I could put there to explain "the necessity".

If it's enough to you, I can just add a comment like the one you suggested.
Actually, after intentionally breaking a template and testing, it doesn't take down the server. The error gets logged normally. I imagine this is due to our `process.on('uncaughtException')` handler.
Seems this hasn't been addressed yet?
nit: space before ++
Hi!

Please review the v3 commits, not the v2! :)
https://github.com/mozilla/nightlytt/pull/19#commits-pushed-91017e7

By the way, thank you for being "r?"!

--
Szabolcs

2012/3/19 Henrik Skupin
<reply@reply.github.com>:
>
> It depends on when the code gets called. If it's during the startup phase then we can do it on shutdown. Otherwise it has to be done before possible references get invalid.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mozilla/nightlytt/pull/19/files#r573457
No, AMO doesn't support changing IDs. It's best that you stick to the one you have.
Please use the pull # instead of the issue #. Reason is that we have to be consistent and check-ins will have a pull but not necessarily an issue.
Rename to `register`
Maybe its sounds sweet by using "If this site doesn't redirect automatically please" <>Click Here<>
Sf2 Validator depends on doctrine/common. ValidatorFactory class uses doctrine code to read configuration via annotations (https://github.com/symfony/Validator/blob/master/ValidatorFactory.php). It's a feature that we don't use but I have not found a way around the problem.