labofsahil/github-event-dataset-2012 huggingface dataset:
Update MojoShader

(The API changed, so this should be pulled just before the request in MonoGame)
各种小调整
layout all cells after HeaderView's height was changed

Some times the HeaderView's height will be changed, after that I will call `[gridView setNeedsLayout]` to relayout. 

This commit fixed the problem that the `layoutSubviews` method did not notice the height of headView changed.
Fixed some stuff on flamethrowers.

Fixed an icon that was not updating.
Fixed a useless piece of UI.
Updated to RC and RavenDb 1.2 unstable due to version conflict with Json.NET
Bottom-aligned toast locations | Updated to newest jQuery version

Added the ability to select "bottom-left", "bottom-center", and "bottom-right" for the .toast-position. Also updated the jQuery version to latest.
Tickets/8496

IE8 doesn't allow reading the dropdown selected value for a hidden DOM element, which we have on the full viewer line plot when we click on the enable checkbox, so defaulting to horizontal if element isn't available
Canonical callbacks
Different approach to requiring inventor's to sublicense

My take at addressing #18
Make it work with the current version of Backbone (0.9.1)

The internal _add method was removed from Backbone.Collection so that the original approach did not work anymore. I also had to change the code concerning the model id. This patch is verified to work with the current version (0.9.1) of Backbone.
Issue 796 - Asserting a null object reference throws AssertError Failure internal\invariant.d(14) or Access Violation

Rewrite assert calls of pointers to structs with invariants and classss references to:
`assert(e);` -> `auto tmp = e, e || assert, e->invariant()`

Most of the lines changed are indentation changes due to making some code inside an `if {}` unconditional.

Add `?w=1` to the url to view the diff without whitespace changes.

http://d.puremagic.com/issues/show_bug.cgi?id=796
Implementing @AfterInject for @Enhanced components, 

@AfterInject allows methods to be called after dependency injection in @Enhanced components (the dependency injection happens after the constructor returns).

See issue #6.

@matboniface could you please review and merge that (green button)?
added "protect" method

Added "protect" method in /lib/media_wiki/gateway.rb to support protecting/unprotecting pages.  See: http://www.mediawiki.org/wiki/API:Protect
Track processed urls to prevent multiple checks
Eclipse 4 initial port
created build specific .css files this closes #24

Created the files stylesheet.html.css, stylesheet.pfd.css, etc. for all builds.
Used "stylesheet.<%= @output %>.css" in the stylesheet tag to do the magic.
Envio de correos

Agregue funcion enviar al correo
v1.0.7-b3-SNAPSHOT Created
Restructure and Consolidation

Updates include:
   - Divided all calls into clients based on the Cloudstack API documentation groupings. This makes finding the functions you are after simpler.
   - Updated General Client() class to use divided clients to prevent duplicating code.
   - Consolidated some of the code in the BaseClient class and change the name of self.request to self.call
   - Changed name of BasClient.py to common.py.
   - Moved login and logout functions into common.py
PULL NOW
Testhosts

Fixing a merge problem that happened in first pull.
Error message in Emacs mode is fixed

Error message "Error during redisplay: (error No match 2 in highlight (2 font-lock-function-name-face))" is fixed by removing second group highlight specififcation because no secodn group exists in expression
(rx "{"
          (group (or "literal" "/literal"))
          "}")
Update DeskClock/Files/res/values-ru/strings.xml

2.4.27
Sprint5 server backlog

When someone gets the chance, it would be good to start evaluating some of these commits so that before we start the bug work (sprint6) some of this could already be in the mainline.
INTSAMPLES-76 - Add additional wait time before socket connect

For more details see: https://jira.springsource.org/browse/INTSAMPLES-76
Fix silverlight build as it doesn't support System.Data.Linq
clean up and asset pipeline fixes
Fixed separators + button colors (transparencies)

Reworked the actionbar and actionbar_item layouts to have a 1px View object for separators, rather than using the background color of a parent view
GRAILS-8341 - withFormat not working correctly with */*

This is related to a pull request on grails-core, https://github.com/grails/grails-core/pull/146.  In case that gets included I also thought I should update the functional tests.
Update packages/sample_content_generator/controllers/dashboard/generate_...

...install_data.php

You can generate data for content.xml that can be used at the install.
LPS-24408
add libdrmwvmplugin libWVStreamControlAPI_L1 libwvdrm_L1 and libwvm to s...

...elf-extractors
just a small pr to clean a few issues
Упрощяем код превьюшек и избавляемся от замыканий в setTimeout

Если какой-нибудь браузер (опера) не поддерживает `setTimeout` то я сделаю прослойку наподобии той, которая используется с хидденчаном.
Фикс для панели настроек на сосаче

Тень портит вкладки, а трансформации выглядят дико некрасиво на панели настроек.
Allow siriproxy's host to use the DNS server that redirects

This pull request uses Ruby's Resolv library to request an A record for guzzoni.apple.com from Google's Public DNS servers.  Users can override the upstream DNS servers using `--upstream-dns`.
Minor bugfixes

Fixed two minor bugs:

1. Incorrect mode (absolute) when editing new buffers/files before entering insert mode.
1. Incorrect mode (relative) after gaining focus while already in insert mode.
It should not break if ActiveRecord isn't defined

Hi,

I am using devise_cas_authenticatable with mongoid and it is raising error cause ActiveRecord isn't defined.

Please pull from my repository.

Thanks!
Pull Req: blackberry.identity.uuid

Feature: blackberry-webworks/BB10-Webworks-API#9
JSDocs: blackberry-webworks/WebWorks-API-Docs#99

Implementation, unit test, and functional test are included in this pull request.

Note: You must include the read_device_identifying_information permission in config.xml in order to read the field, otherwise it will be an empty string
Pull request: standardAskAsync for blackberry.ui.dialog extension

Adding support for standardAskAsync.

Issue fixed:  2 of blackberry-webworks/BB10-Webworks-API#3
Pull request for docs:  blackberry-webworks/WebWorks-API-Docs#117
Viewable docs:  http://blackberry-webworks.github.com/WebWorks-API-Docs/WebWorks-API-Docs-next-BB10-dialog_standard_ext/view/blackberry.ui.dialog.html
signal handler and icon fix

Two commits: one to add a handler to clean up the pid file when the process is killed, and another to fix the notification icon name - so that it works across desktop environments.

Do you have any specific beer preferences?
Hoppas detta funkar...

Verkar ha glömt att pusha efter att jag hämtade det sista så nu när jag push upp det så ser mitt plus det jag hämtade ut att va nytt... Hoppas att det inte fuckar till något =S... Jag och git, vi är inte bästisar ännu...
slider chicks moved

slider chicks moved
Install using standard Perl mechanisms

This series of patches uses Module::Install to automatically install VelvetOptimiser to standard locations, removing the need to move the module and script by hand to the correct places.

Commit a762a08 also improves cross-platform portability.
Add nfc-extras + telephony permissions

Satisfies #40

Change-Id: I5a325069be6750772e678fa214f4b6af1290e73e
Update functions.php

Correct html markup according to:
https://github.com/rachelbaker/bootstrapwp-Twitter-Bootstrap-for-WordPress/issues/54
ak target dostane CC, vsetky pety by mali prestat utocit. Ak ale posles peta na niekoho, kto uz ma CC, tak by mal utocit. Momentalne ta cast nefunguje, takze pet v ziadnom pripade nezrusi CC. Stale lepsie, ako to bolo.
oops, don't mean to put a 104 there
Je vois plus le terme client comme une application qui accède à l'agent e-mail (le client mutt), voilà pourquoi j'ai préféré "boite" en sachant que le terme agent me parait pas légitime ici.
libexecinfo provides backtrace, backtrace_symbols_fd functions. rustrt needs them.
The "-z origin" is to add the "origin" flag to. If this flag is missing, the $ORIGIN in rpath has no effect.

For /usr/local/lib/gcc46 etc, because I have some problems with the default gcc (4.2.1) of FreeBSD 8.2-Release,
I use newer gcc (4.6) to compile rustrt and rustllvm.
However, the compiled objects need some symbols that do not exist in old libstdc++.
I have to add extra library paths and rpaths to link with the new one because ld only looks /usr/lib and /usr/local/lib by default.
I add gcc44 and gcc46 paths because users may install either gcc44 or gcc46.

I hope these answer your questions.
jde se slavit
Master nette je něco jako `work` větev, ne? Takže raději hezkou historii a hezké commit messages za cenu nějakého toho nezbytného force pushe na lidský faktor :)
Compared to SDL it seems like OIS has a quite small collection of recognized keys. Apart from the missing ones it doesn't seem to support non-us keys that well. But it shouldn't be too hard to add it to the OIS source; one could probably get some inspiration from how SDL does it.
I'd like to do.
Looking at the code, it seems the the code is converting a tooltip to RichText if it applies. Would it make sens to name the option `toolTipsRichTextConverted` instead of `toolTipsWordWrapped`

To push a little bit further, it seem the class `ctkToolTipTrapper` is now doing two things. Would it make sens to create a class named `ctkToolTipRichTextConverted` or some other name describing better what the class would be doing ?  

That said, it seems pretty cool to be able to add arbitrary html within tooltip :)
I thought .gitignore is project files ... like `echo .idea >> .gitignore` this way it will not delete your config but also will not track it for others
Did you encounter an issue with the `Stage()` method?
Please run multiple test suites in one run (w/o so many ENV variables) and make sure you test suite takes no more than 20 minutes. Otherwise we will have to ask you to remove zf2 from travis. Your build matrix size right now in unreasonable. Thank you.
Please make sure your .travis.yml passes http://lint.travis-ci.org.
Another question- how would I center the label to the right, halfway vertically? I can't get edge insets to work properly.
принято удалять файлы пустышки, так приложение становится более опрятным (при необходимости они могут быть созданы руками)
Could we just use 
protected double calculateHits(FacetDataCache<?> dataCache, List<String> terms) {...} method
instead of supplying the calcHits function?
NP, there's too many layers to this stuff. Though they are basically necessary.
I'd rather have the logic of checking if it is a binary inside the caller module than this module. This way this module is generic and only does what it's told.
Because people like me have been using it for a while now..... It would be unfair to just break plugins.
А, ну добавлена поддержка freedesktop.Notifications - для систем линукс.
Is this a good default?  Why not 'F'?  Ah, I think I see: because the subsequent permutation assumes a 'C' order array to start with?
Cool! But there is a similar bug with race condition on creating groups here: https://github.com/dcramer/sentry/blob/cd114bcb99593b229ed994036e2e4e2402d3d83d/sentry/manager.py#L376

Actually, every get_or_create is a source of create/create race conditions.

Already bitten by this.
Il manque la balise fermante ici, mais bon :p
Waarom deze return...?
And `/regexp/u` now works fine, thank you :)
No optimisation?
nice.. ;)
You're right, there's a ``# TODO:`` somewhere to make the ports autoselected for all the server types but honestly I'm not sure if eventlet.wsgi and wsgiref.simple_server support it. I'm +1 on this if we can make it work.
The original line was definitely problematic (only one key would be removed) but is the new approach at risk of any compiler optimizations?  Not sure.
@richarddong 这个应该给reply-comment这个分支一份的。。
Yes, using dojo/method is generally not a good way to add event listeners to widgets. At this point though I guess I am not too concerned about it.
oh...scheiß copy-paste:(
We have lags every 20-40 hours of online. Players often demand restarts when online is 24h+
@PSZ I have Debian 64Bix, Thank for replay .
curhealth = 0 in creature?
Any idea on when/what build this will be in?
You meant something like this in your first answer?

    <div class="row">
        <div class="two columns">…</div>
        <div class="ten columns">
            <div class="row">
                <div class="three columns">…</div>
            </div>
        </div>
    </div>

The problem with nested rows is that you are definitely out of the grid. Three columns inside ten columns is three-twelfth of the parent column and not a _standard_ grid column. (BTW: a nested grid with equally column widths would be an awesome addition to Foundation ;) 

Stenlyk's fix is only a _fix_ for the last column but not the columns in between and makes some simple grid layouts impossible. I’m not sure that a CSS-only fix is even possible at the moment. (And I really hope this bug will be [fixed by the webkit team in the near future](https://bugs.webkit.org/show_bug.cgi?id=5531).)

I would prefer to let Webkit solve this bug in a future update and give the Foundation users an optional class to fix it by themselves:

Fix 1: 

    <div class="row gap-fix">…</div>

    .gap-fix [class*="column"] + [class*="column"]:last-child { float: right; }

or fix 2:

    <div class="three columns float-left">…</div>

    [class*="column"] + [class*="column"]:last-child { float: right; }
    .row .column.float-left, .row .columns.float-left { float: left }

I would prefer fix 1.
yes, but then we have a problem, u cant connect the mic when u are in the game, always u should connect it before to start, and go out only for it... load game can be much time :/
I do this commit only to learn (me first commit :D), and u can see that my repo started to the beginning, now I need merge :(
Yeah, actually, you uncover a good point.

The output from fpm only shows you the filename. Perhaps it should show some data as well?

```
Created whatever.deb
name: blah
version: blah
....
```
Who made this comment o.o
Was hast du korrigiert? seh nur ne TODO! Sind die Grafiken schon ersetzt? Hast du dir auch die Taballen angeguckt?
Du darfst net vergesen im gegensatz zum Code sieht man nicht was geändert wurde, daher sind "korrekturen" als Begriff etwas zu allgemein!
I would say that in these days pthread are almost everywhere. But let's keep it ifdefed for now.
Instead*
we have #ccc and #eee in lib.less - will those work?
I stupidly deleted my build-tasks repo recently and never did merge this--it'd be great if you'd send this up to the main repo.
`:cateogry` . grammar police here `:>`
This should return 'service', not upstart. That way it fits in with the rest of the service modules.
Always nice with some constructive criticism ;)
Should this be * or #?
为啥你总是要反复做这些事情。为啥要refresh....
 :em06
why can't you build a dict?
redundant else, you can just make it return smelting.hasNext()
I think that SDL's cross platform support (for setting up a GL window) is a bit stronger/more consistent. Also, sdl's input handling is a bit stronger (joysticks, etc). Still, if glut meets your needs, then no need to change for now (or at all)
@paulmillr: I don't know of an interpreter that *disallows* writing to `Infinity` outside strict mode. They all implement ES3 15.1.1.2.

> The initial value of `Infinity` is `+∞` (section 8.5). This property has the attributes { `DontEnum`, `DontDelete` }.
You're setting this, but where is this used?  I don't see it being passed to the FileCacheStoreConfiguration's constructor in FileCacheStoreConfigurationBuilder.build() ?
In the previous version of Velocity, {} returned a HashMap, not a LinkedHashMap. Is there a guarantee that {} will continue to return an ordered map?
-- Thaddius
- Stalagg
- Feugen
-- Kelthuzad

missing sound id
ich hoffe das geht so und ich hab nichts kaputt gemacht ich hatte wieder probleme mit dem push aber scheint so zu gehen
I see what you mean, but I'm not sure if I like the idea of polluting the Framework_Process generic class with specific cases like this one... If the command line PHP executable requires a specific argument to run in Windows, that should be specified in the Administration executables tab. What are your thoughts on this?
This totally makes sense.
the file contained cr/lf line endings before, which breaks the diff function for github, the actual diff is

```diff
@@ -38,6 +38,7 @@

     protected EmbedderControls embedderControls() {
         EmbedderControls controls = new EmbedderControls();
+        controls.doIgnoreFailureInView(true);
         controls.doGenerateViewAfterStories(true);
         return controls;
     }
```
Сигнатура метода не должна была меняться. Throws должен быть только для TransferException, так как это исключение бизнес уровня.
I don't mind you working on CityWorld (I definitely need some help, too many requests and not enough time) but could I get a little heads up on what you are doing to it? Maybe an email with the changes you have planned, git makes it rather hard to see some of the changes you have made. Right now I have promised a number of folks I would give them a release tonight. Stabilization/testing is important and while I am not saying that you are doing anything to jeopardize that... I am a bit confused why you are changing what you are changing. Just makes me a bit nervous. :-)
Yea. I'm taking my first programming class this semester and am taking one
on Android next semester. I'm just trying to see how github works (but I do
use your app for my law school classes). Thanks!
On Jan 31, 2012 11:56 AM, "Nicolas Raoul" <
reply@reply.github.com>
wrote:

> Hi Kintseb!
> Is it a test?
> Cheers!
> Nicolas
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/kintseb/Anki-Android/commit/31e8aea4b11381b95b8f888c4add41abec1a1cb1#commitcomment-919016
>
*cough* if(NOT self.using_express) *cough*
In CMake. Right.
yeah, I saw the revert after commenting here
Your script loops the first page, my download songs method fixes that.
I'm not sure if this is the correct script target. I suppose there is a trigger NPC somewhere which isn't spawned yet by DB providers. We need to research this better.
My editor is showing that these lines are using tabs instead of spaces. Please convert this to spaces with 4 spaces per tab.
OK, will send a vote.
rename fileNamesSelected() into selectedFileNames()
1 space between `#` and comment text.
I think the last one should fit the best. After all we have only a few cases where this happens (4 or 5 if I'm not mistaken). The last one gives you the possibility to see all the chests in one place. But also the first version doesn't look too bad, it reminds of the old times when we were using a GUID variable for each object. :)
Yup, thx ;)
https://github.com/michalpolko/scriptdev2/commit/53d450285c8fe0da70a1a89b59510b70246d4df8
We're grown ups now, and it's our turn to decide what that means.
Not sure what this does, eclipse just put it in there, could someone explain please?
what's CH postprocessing?
scanf는 필요없습니다. *head 만큼돌면 됩니다.
Ok thanks you
canPass is false for HoleTile unless the entity is a bullet. The function could return true if the tile is an instance of wallTile. It could return once it finds something to avoid checking more tiles than needed. If it doesn't find anything, just return false at the end.
Hi Julien.

The supports submodule was removed from Foundation a while ago, as it now instead resides in Modules/Support. This commit have added supports files to the Foundation repository itself. I have removed them again.

Cheers,
Trond
Disrespecting devs by insulting them without any reason even when they are giving some tips .............,doesnt sounds nice to me.
Can you kandera also convert T10 melle bonus http://paste2.org/p/1908405 ehich is in issue tracker and not compatible after this conversion? Thank you.
Looks as though you're expecting this will have only one match, if that's so and head is not Alive, shouldn't you break the loop?

My point is that (and I may be wrong), if you don't break and find another match, you'll start Linking many heaps without information on where to Release.
there might have been something in gemfile too..
would throw an IndexOutOfBounds if the specified index was not existent..
I'm not sure what a pull request is. I will be glad to do it your way, just please explain me how to do it. Can I insert it somewhere myself?
I'm just working on applying some patches and didn't want to see a whole bunch of errors for missing files.
> > No, that's fine, @ogrisel is just a bot, everybody knows that.

> Actually that's fine too, everybody is just a bot and @ogrisel knows that.

:+1:
Can you revert this for now? It should be on a different branch until it actually works. No committing todos on develop
you can also use the @deprecated PHPDoc comment.
El problema es que esto no funciona si no añadimos esta línea, por que la modificación de service builder haría que no compilara. Hay que dejarlo
Btw, this has 2 labels, one for the input and the other as a wrapper with a class.
Did you see the changes tdm and I posted about MMS?
You'll need to crib the IHO changes to CdmaSMSDispatcher.java, and for now we are using a hack by tdm to send error to log in MMS rather than crashing the app.  It's ugly, but works for now.
Needed so I can specify a `null` generate structures value. Not that it technically matters, since the world creator will ignore if it differs from the NBT.
:) , so the priorities are in the order or have no order ?
```layer.getLayer()``` would be the API replacement for ```layer.data.layer```
Do you have the same tabs/spaces settings...?
Doh, will clean that up... thanks for going through this with the fine comb so to speak :)
my idea of how to do this is writeVirtualMemory(status-location , byte array[] ) but i have to convert the status to bytes first yea?
Nice!  I merged this in https://github.com/bassnode/ruby-echonest/commit/a0e9bbc1f0101fe7e6e259b9e7a7612cf551a8ce
this process only the first url, I don't know if that's the intention or you want to process all of them?
Argh, I have a pile of test helpers just like this sitting in my branch. *sigh*
```
$PREPROCESSOR "$DISPLAYNAME_DEF" $CLIENT_NAME.in > $CLIENT_NAME
```
Align these two setFlags calls, please.
Now, when an unavailable variant is selected, the add to cart button is disabled and the price is replaced with Unavailable.
Before, the wrong variant was added to the cart (with Ajax).
Could you post your build output on pastebin? Thanks.
I should not have pushed this: I quoted manifests which include several files. That's what necessary here.
fue una solución parche, en realidad sólo debiera estar la linea 32, porque en caso de que no haya foto está capturado por el empty de thumbnail
Sounds like `StandardMavenAuthenticationProvider.java` would be helpful for other plugins as well. I know that it would be at least for one or two of my own plugins. Maybe this is a good candidate to move it to Gradle core.
Couldn't you make it work without the "lib/"? On my setup I add only the header without folders.
path = admin_media_path or DJANGO_ADMIN_MEDIA_PATH
NameError: global name 'DJANGO_ADMIN_MEDIA_PATH' is not defined


Should be self.DJANGO_ADMIN_MEDIA_PATH there.  Sorry for not downloading it then patching it.  Curious how you even ran the code without that fixed.
I just thought about using data attribute on the filterfield form so the javascript should be able to pick up the correct gridfield from that.

    $form = new Form($this, 'filter', $fields, $actions);
    $form->setAttribute('data-gridfield', 'File');
    $form->addExtraClass('cms-filter-form');

Would this be a better solution?
Goods:
1. namespace
2. script bottom
Looks great here too nice work
I was thing more along the line of the following:

    @job(queue='Foo', connection=myconn)
    def some_work():
        pass

Would that make sense?

Of course, the `queue` arg must be required, as this is about the only added value for the decorator at all (not having to worry about which queue to enqueue to when you want to submit work). My point was to make the `connection` arg optional, not the `queue` argument.

You're right on the `timeout`. Please do leave it out there, yes.
Use `<p>` tags instead of multiple breaks.
Good remark : I don't know in fact, I'll check... except if someone give me the answer before...
Its Jasig, not JA-SIG
Thanks for fixing this. I originally wrote the DAG checker to have the same assumptions as the YAML parser: that a reference can't appear before the anchor it refers to. That is no longer the case.
No longer using FrozenDicts.
This should be encoded in the Open state and not in the breaker
transportsAndAddresses is just used in "success", move the expression in there
Yes, you're right. It actually is kind of weird looking. Should I change it?
Roland asked me to rethink that question, and yes the messages are of course handled correctly by the *real* actor later. Would have been nice with a comment explaining that when we transfer the messages.
Yes, name mangling was the reason to choose "." as a separator.
Good catch, this used to be a val as an Option. I'll fix.
I don't think we do,  I'll update this to use xml.minidom.
The code above is repeated three times in this file, and also in cvs.pm, svn.pm and point.pm. I suggest adding a new function for it to Base, something like this (untested):

	sub getURL {
		my $urlName = shift;
		my $url = shift; # read default
		if (-f "$basepath/lib/fink/URL/$urlName") {
			$url = cat "$basepath/lib/fink/URL/$urlName";
			chomp($url);
		}
		return $url;
	}

then the above could become

	my $gitrepository = getURL("git-repository", "https://github.com/danielj7/fink-dists.git");

and similar for the code below.
This is a magic little artefact of broadcasting :)  We have ``dti_params[new_axis, :] = parameters``, which turns out to be equivalent to ``dti_params[:] = parameters``.
This code was written with a lot of vectorization, I'll take a look to see if the MultiVoxel fit performs comparably  but I suspect that the vectorized version is going to be much faster.
It looks like these were put here by mistake and were only necessary for rhel5. Both are in the standard library for python >= 2.5
I didn't mean to step on anyone's toes, I just want my feature to work which has been broken since AE got merged.
if options.save:
So in the example, this will connect to `/dev/ACM`? I think this could be a little more clear.
This comment change doesn't make sense.
Only one tab instead of two. Same for license and the other files.
This partial function will be executed on some other thread, and it loses over this, is that safe?
Ooops, I will add that

On Thu, Sep 13, 2012 at 8:32 PM, Mark Bessey <notifications@github.com>wrote:

> In source/dom/Control.js:
>
> > @@ -629,7 +629,7 @@ enyo.kind({
> >  		}
> >  	},
> >  	getParentNode: function() {
> > -		return this.parentNode || (this.parent && this.parent.hasNode());
> > +		return this.parentNode || (this.parent && this.parent.hasNode()) || (this.parent.getParentNode && this.parent.getParentNode());
>
> Seems like this is missing a check for this.parent to exist before
> checking for this.parent.getParentNode.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/enyojs/enyo/pull/143/files#r1604221>.
>
>
This interval should be in a constant.
I'm glad! :hamster:
This doesn't do what you think it does. You're casting the char that `begin` points to to an unsigned int.
Dimension. Or I don`t understand what you mean by component.
I use PHP5.4-lastest and test should pass in all currently maintained version :)
@smalyshev Could you confirm that the document I am referencing here (http://docstore.mik.ua/orelly/webprog/php/ch14_08.htm) is correct?

```c
PHP_FUNCTION(test) {
    char *str = emalloc(5);
    strcpy(str, "newb");
    RETURN_STRINGL(str, strlen(str), 0);
}
```

By setting the last argument of RETURN_STRINGL to 0 anything emalloc'd will get efree'd internally or no?
если этот код не нужен, убрать его совсем лучше
I haven't looked deeper into the code, I just noticed this mutation of an outside variable inside a loop, and thought that maybe this was an oversight. If you're confident that that's how it should be, then so be it :).
No need for `else` block.
My bad. I didn't look carefully enough. I just just saw the debug at the bottom of the method.
there is alias_method_chain imlementation https://gist.github.com/614ccbbe263cd1196203 looks almost same. will it be better if I commit this one?

/cc @dhh, @tenderlove, @steveklabnik, @sstephenson, @rafaelfranca, @jeremy, @fxn
Does Scaladoc have 'styles'? In that case, shouldn't the highlighting color be configurable?
Since `HIDDEN` is a strict subset of `SYNTHETIC`, this change will let more warnings through. Is that intended?
these conditions are getting pretty unwieldy -- could you please simplify or elucidate with a comment?
think of the poor reviewer and future-selves reading this code
Why? is `map2` better?
actually I think I have found a solution which is reasonably comprehensible and does not use division, I’m currently fighting ScalaCheck in order to get it approved ;-)
i'd vote for renaming `tpeHK` to `typeConstructor` while we're at it
what if i set ?filter=destroy_all ? :)
Should add a:

        else
          flash[:notice] = t(:coupon_code_added)
        end

to let users know when they've successfully added a coupon code. Although the discount will come up in the adjustment list, this makes it a bit more 'dummy proof' so to speak.
the comment? No idea
It doesn't fails now in this pull request and in the master. since autumn
 (but when I began with this PR it failed).
Thanks for the remarks. I was using doctest for the firts time, but have allready coped with it.
If this is the sort of thing that might one day be an object or if it is very similar to other things that are objects then I think it's acceptable to keep it upper case.
With sanitization above, the ifs can be removed and the following can be written

```
format_str += ["%" + ("-" if self._alignment == "l" else "") + str(w) + "s"]
```
I shared a g+ screenshot of what I'm seeing
ok hab ich gemacht
Hm, I think we should consider renaming to `AngularCoordinates` to make it clear that this class only accepts what you list above. Then this can subclass from a skeleton class `Coordinates`. People later may want to generalize the concept to many other coordinate systems.
The URL generator is an optional feature. As such, the documentation should respect that choice.
No, it's the class name as a string
Need to update the comment to reflect reality - Oracle actually throws a java.sql.SQLException (or a subclass thereof) ... [not just java.sql.BatchUpdateException]
With is this commented out?
I wonder if the Wrapper shouldn't have a close() method if its own instead of making the people who use them know about all the things stored inside and what should and should not be closed.
is this going to work well in production where most of the migrated docs have 'MDN' as their root breadcrumb parent?
Please write comments (why are you doing this?) on #raise_if_knockout_used!, #uses_knockout?, and possibly the regular expressions above, too.
See next line below.
Because array_pop takes a reference, so you must assign a variable.
This actually seems like it should be a slash?
Get rid of inline assignment please.
I am not seeing a replacement test for a missing controller.
No message title?
Case in point. Why is post_username passed to user_delete event? What makes this particular column special?
appended to
I am still not sold on the abort in this situation.
What is the initial value? If initial value is empty and user is required to enter 0, I don't think this is acceptable.
This is not a great choice of a variable name given that we have a "limited access" role.
What do you mean by "highly suspicious"?
Here we go - 3042efb5196f766c632e1d564da05f764447ee39.
Problem is that this is happening in `run` instead of `init`
Hm, I’d love to come up with a better method name, but right now I can’t.
Good catch.

Simple enough as `if (typeof angle === 'number') return angle;` or should there be more robust handling of sorts (not sure what that would be...)
of course not. Good catch. Thank you!
Please set a hard-coded value. We already know this is Networksolution.

```ruby
property_supported :registrar do
  Record::Registrar.new(
      :name => 'Network Solutions',
      :organization => 'Network Solutions, LLC',
      :url  => 'http://www.networksolutions.com/'
  )
end
```
Un-needed whitespace
@edolstra  Does @aszlig still need to move to pythonPackages and use buildPythonPackage, or is this good enough to merge?
Not sure I understand the rationale for this ... hopefully we can come up with a less hack-y solution?
Could we state that for filenames this can include wildcards?
I wanted to mention that. Thank you!
This could be simplified as 

"and execute it with the `nosetests` program."
Thats actually a good question. I don't think @AdHoc... procs have that info now, so I didn't add it here. But it might be bad that we're not re-checking at catalog change time. I'll look into it.
@basvandervlies You could move the syntax error checks to the functions specific to the promise types, and then do a simple lookup name -> function to call. Later we can replace this lookup with a dynamic one.
Not sure I like begin, as this is quite different to SQL transactions - it's also wrapping the deferred SQL stuff. Besides, all sensible databases accept START TRANSACTION too :)
you are right, that was bad
nitpick. you should remove commented debug statements from both the code and the tests
Is there a test case which test the correctness of this calculation?
> At first glance, I'm not clear on why (a) you're using == instead of ===, and (b) this particular condition. Are we assuming that if there are no caps, then it must be snake case?

The regex can only perform changes on the string if there is the presence of capital characters.  If there are only lowecase letters, then the first check allows you to skip the expensive regular expression.  I've actually simplified this further by using ctype_lower.

> e.g., What if (for whatever reason) the $word is hyphenated?

This method is used to convert CamelCase to snake_case.  The strategy used to inflect can be changed in most places in Guzzle where needed, so if inflection needs to be handled properly for hyphens or something more generic, then a user could implement a custom strategy.
You can declare this function as STRICT, so that NULL inputs are ignored.
Hmm, SRF_IS_FIRSTCALL() and SRF_PERCALL_SETUP() could also call ERROR.  It is unsafe to use them outside of PG_TRY().
This could also potentially be extracted into accounts-oauth-helper, with another argument being the oauth version.
This part doestn change anything, if this if-clause is commented out, or did i understand sth. wrong?
Uses a list e.g.:

```rb
system "make", "smu", "PREFIX=#{prefix}"
```
Also looks like `remctl` on its own doesn't exit 0, so the test will fail.

```ruby
system "#{bin}/remctl", "-v"
```

works, though if there's something you could use to actually test the functionality that would be even better!
Emacs added the coding line. Should I remove it, even though it indicates there's UTF-8 in this formula?
Not the right place for this - it belongs in run!
Supprimé.
You can link to it here by using ```:ref:`criteria````
I know pylint suggest `_` as a dummy variable, but it conflicts with the global `_` for gettext, so it's better to use `dummy` or `unused`
@kubb, lol in the first "Fix" it totally missed your point. Was only taking care of the whitespace after commas.
Eight tests fail when I replace the body of `getTestClass` with simple `return fTestClass;`. Did not yet figure out why.

I did however notice that `getClassName` can be made simpler when `fTestClass` is defined. (It might not be a bad idea to store an explicit `fMethodName`, which would properly round-trip test names created by special tools which might include special characters like `(`.)
Yep.  Can you include this information in some comments, as well?
PayPal is tiny, but I'll drop it anyway. I was just copying Yammer's
Yeah I have another comment. This is camelCase where the other things are words separated by dash. see live-lighting as a pose to maxPlayers.
Testing. I thought I pulled it out; sorry I missed it.
I don't think we use this trick anywhere else. Is it possible to do by hitching the function call instead?
Please use `window.addEventListener`. You are using the old, Microsoft-era, deprecated, partially broken, technique.
Currently, this method crashes if you pass it a command ID that is not actually in the menu.  That seems fine, but the docs should note that you can't call it with an ID not in the menu (otherwise people might assume it's a no-op).  It also might be nice to detect this case and throw a deliberate exception with a useful string (see examples in addMenuItem()) rather than just NPE'ing in the midst of things.
The standard JSON format doesn't explicitly support file comments. [RFC 4627 application/json](http://www.ietf.org/rfc/rfc4627.txt?number=4627
Nope. The reference here is just to the w3 spec. It's similar in intent, but using the NativeFileSystem instead of the sandboxed local file system.
It's a little dangerous to use `unbind()` this way (without either a namespace or an explicit function name to remove) because someone else might have bound a mousemove handler, and this will remove all of them. Either namespace the event (e.g. `$(".main-view").on("mousemove.sidebar", ...)`) and then remove it with `$(".main-view").off("mousemove.sidebar");`).

Also, we're using the jQuery 1.7 `on()` and `off()` methods instead of `bind()` and `unbind()`.
All actual executed calls of `setFrame` is inside `dbg-node.js` and 3/4 of them is calling it to set it to null
So, what do you suggest those calls be like ?
"Map<Integer, Request> requestMap" instead of "HashMap<Integer, Request> requestMap"
column names should be 'datalocation' when it can only be that, and 'location' when it can be both? or 'datalocation' and 'location', and something else for when it can be both?
shouldn't you log the error here?
If this is needed for the test then lets add it there instead.
這裡可以這樣思考:
1. 加入或刪除的按鈕也會在其他地方出現
2. 按鈕的位置應該和按鈕的樣式獨立
3. 減少 class 的使用, 把相同性質的合併在一個 class name, 例如樣式的 class 和位置的 class
所以
1. 樣式的 class => btn-delete, btn-add
2. 位置的 class => pull-left, pull-right
3. 得到 `%button.btn-delete.pull-left` 和 `%button.btn-add.pull-right`
以此為目標去修改 css
這裡會和 css 中的繼承有關, 再教你
Perhaps this should be a parameter? Or at least a comment to explain the weird string.
Kind of a strange assertion to use, but I don't know what would be better. `assert_type_of Exception, err.new`, or something?

(nb. `assert_type_of` doesn't actually exist.)
`<button></span>Send</button>` doesn't look quite right.
To be clear, I don't mind that it is. If it's unnecessary though, it can be removed.
Same spacing thing. I think one needs to go here to make MD parsers happy. Err on the side of caution :P.
This is the right idea, but it's in the wrong spot. `Resizable` is what needs patching.
This comment doesn't seem to make sense.
this isn't a good place for a private api key
Can we call this something other than `_zIndex`? Maybe `_defaultPluginZIndex`.
@DanielFribourg fixed by #2041
function name => `CFFIndexClosure`
Good catch.  Added new line for formatting.
Well... I'm open to suggestions. As far as I know there is no real mapping between blocks and epg items? With the current implementation I think that there is no other way (look at GetNextItem and GetPrevItem). Maybe a backwards iteration would be more optimal in most cases.
you can just set the boolean here.
Diese Instanz wird ja nur einmalig erstellt, daher gibt es ein Problem, wenn man `rex_a79_textile()` mehrfach und mit unterschiedlichem doctype aufruft.

```php
rex_a79_textile($code, false, 'xhtml'); // Instanz wird mit 'xhtml' erstellt
rex_a79_textile($code, false, 'html5'); // Die vorherige Instanz wird wiederverwendet, doctype wird nicht beachtet
```

Wenn man den Parameter so haben möchte, müsste man daher pro $doctype eine Instanz erstellen (immer beim ersten Gebrauch), und in einem static Array $instances speichern.
Yes see https://issues.sonatype.org/browse/NEXUS-5302
If the video has an embed, but no video_* data, then this kicks up an IndexError. I think if there's an embed, but nothing else, this shouldn't return anything. Ditto for item_enclosure_length and item_enclosure_mime_type.
For consistency,

- Should these be part of module InstanceMethods (similar to existing delete_packages, delete_errata...etc)?
The cert tooling sets this up earlier in the install chain.
+1 on the @lzap suggestion, change in converge-ui is a good indicator of trying build rpms before merging
I'm not sure what to do here. Should it raise a new error if it's the json_encode that raised the error first?
Replace this URL with

`https://android.googlesource.com/platform/packages/inputmethods/PinyinIME`
'e' and 'b' is far from being explicit. Instead of treating confirm and emergency as 'key' I would rather call a real method with explicit name when they are clicked.
I think it's ok to require users to specify the format to write tables out. I agree that ``xml`` is too generic, and that's probably the right extension to use.
Actually I'm not so sure about that now, as transformations can be added to the master_transform_graph at runtime. Though I suppose it could work for any built-in transformations.  And maaaaybe could work by actually modifying the class at runtime, though I might prefer to discourage that.
Hmm.  Actually we should put the download URL in the setup.py.  That would mean that for development builds the download URL would be nonexistent, but I don't think that's a real problem.
We just need to know what the URL format would be for a download from github.  Any ideas?
This should probably not be an `AssertionError` but instead an `IOError` (that's typical for things that fail on the filesystem).
Thanks for the suggestions - I would prefer to avoid duplications in the API , otherwise imagine what happens if you pass it a lefterror that is different from the one stored in the class itself - that would be very counterintuitive. One could add checks, but then that's no easier than doing ``lefterror = self.parent.error.array``, etc.

I like your suggestions for renaming the variables though, and I find them much less confusing. So if that's ok with you, I'll implement that instead.
I'm pretty sure there's no point to breaking up the message into different properties.
I guess init.cpp does not need this, as we don't use message queues there.
Good question. I'm not sure. Does anyone actually have a debian bastardized kfreebsd system to test on?
Exactly. But the app folder can change once you change the namespace too.
What's the 'on' method do?
why not using `setVisibility(visibility)`?
>
> Should the quote be a colon here?
>
Yes.  I'll fix and correct the RST formatting.

A
this will still call ``__isset``, no ?
this will break if the field value is a collection instead of an array
I think the protocol (http://redis.io/topics/protocol) is OS independent. 
The client hardcodes the line ending as well: https://github.com/debasishg/scala-redis/blob/master/src/main/scala/com/redis/RedisProtocol.scala
However i will make a test on a Windows box on Monday to verify.
I will try to send a few unit tests for these changes as well.
Thks,
Krishnen
```javascript
if (context.baseWindow.document.getElementById(selector))
```
seems better.
Seconded: /usr/bin/python2 didn't work with ubuntu 11.10 either. /usr/bin/python did.
Well in that case maybe we should use ~/.cython/ipy_magic ?
I think it is better to have this condition before the `e.mimeData().hasText()` condition.   

Dropping files from external application doesn't work correctly for me (Ubuntu 12.04; KDE), due to this.
I don't see where the `user_highlight` attribute is initialized.  It should be set in the `Cell` constructor to `auto`.
No, I was just making minimal changes.  Previously it used `io.rprint`, which I replaced with critical as the logging analog of 'always print this'.
assertEquals(a,b) is better assertTrue( a.equals(b))
ex. it looks like the volume "extension" in nova, is simply a wrapper/filter for the cinder api.  In this case, there shouldn't be any cinder related Api/AsynApis in nova that aren't in cinder, right?
I added a --no-progress option, and have it printing out some progress if that option is not set.
Yes, clicking "Previous" while on the first page of section B should take you to the last page of section A. You should be able to next/prev your way through the entire Style Guide in a linear path.
Hmm. I'm not sure. `user_profile` is what was there originally, so I just brought it down unchanged. I think I would rather say `profile.user` in the template.
Yeah it's not really needed, I'll just take it out
Fair enough, personal habit of listing private/protected members before public.
The docs of this function need to be updated to reflect what it now does and describe the new argument
If I remember correctly, this is the case for launching games from the command line.
I'm sorry, but I dont follow.  Also please note this is very important that this lifetime is not confused with other lifetimes (like cookie lifetime) for example.
@lsmith77 ``This fact used to be considered an implementation detail, but it can now be relied upon``

And they added new methods on the class in 5.4 to bind the closure.
This point should be removed from the changelog as it has been modified again when uniformizing them
isn't it an array ? Otherwise, the plural is wrong in the name
Should be in the CHANGELOG instead.
IMO you should use full name here, like `deleteHeader`. Also bracet should be at new line (same below).
in the interface `$blockData` is called `$resource`?!
Well, the boundary could be that you give a 'NameServerChanges' method and the current tree to a RoutingState method, and get back a new tree. Either way is fine I guess.
This should be:

```
if ([utc isKindOfClass:[NSString class]]) {
```
I agree it does look like more than two spaces, but it appears to be correct. Or do we want the params even with the beginning of the method name?
so if I understand it correctly if you get error code different then zero you cannot get stdout?
文件路径错误，应该是 file/buaamark.eps
Your class satisfies the concept CBidirectionalSegmentComputer. But at this point, using CombinatorialDSS in GreedySegmentation or SaturatedSegmentation is a way of checking the concept itself in order to be sure that everything works fine.
I wanted the code to be runtime independent and couldn't find a non-JVM StringBuilder. Sorry for the semi :)
oh nice, is it possible to replace the usage across the specs. If it's a simple find replace, then do it, otherwise just pass.
Why haven't you gone for vertical alignment?
these three should be wiki links, as described above
Totally agree, this ModelMatcher is still WIP and when it will be done it will for sure have i18n loggers.
This is/was POC work for https://issues.jboss.org/browse/AS7-4459 that is currently not done yet.
You also need a resource level transformer stripping out the USE_AUTO_RECOVERY attribute
There is an inherent race in this function since multiple calls to get prior to the first onsuccess will result in multiple distinct creations.  This is a case where promises are awesome, and the fallback is maintaining a list of callbacks (manually or via event emitter).  If there are higher level mechanisms preventing that, please add a comment to that effect.
Also since your main class is called SimManager let's just call this method unlockSim instead of unlockSIM
We indent the "#" in comments to the same level we indent the code in comments.  Please fix this in a follow-up.
This is awesome, but ideally should conform to our coding guidelines (pass gjslint consistently use 2 spaces).
Was just adding/removing, but yes, I can add it right now. Makes perfectly sense. :)