labofsahil/github-event-dataset-2012 huggingface dataset:
Editorial pass

I've taken a pass at a light overall editorial cleanup. I've fixed some grammar errors, tightened up quite a few sentences, tried to make the overal tone of voice a bit more consistent, and removed a few bits that were redundant. I've stopped at the end of "Basics", but could continue into "Advanced" if you think this is helpful.
Fixed last changes compatibility problems

See https://github.com/sonata-project/SonataAdminBundle/issues/510
Изменяем логику определения языка

Избавляет от ~150 варнингов при компиляции. Скрипт после компиляции (с использованием экстернов, которые я запощу чуть позже) уже слегка юзабелен.
Умное меню скрытия
Breath some life into this :)

* Use latest version (Add mandatory AssociateTag parameter)
* Fix search (ItemSearch instead of Search)
dz4

simplified
Major overhaul

* Directory structure:
 - Move submodules into ./libs
 - Move source files into ./src

* Update upstream:
 - Fast-forward submodules to their latest versions
 - Update package.json to the latest format and latest stable
   dependencies. Also using "~" version:
   http://npmjs.org/doc/json.html#Tilde-Version-Ranges

* Clean up
 - Consistent use of tabs or spaces (tabs in this case)
 - Validates JSHint
 - Fix EOF error: "No newline at end of file"

* Fixup
 - Update README: Fix syntax error: one ")" too many".
 - Add .gitignore file for node_modules (clean `git status`)

* Update test suite:
 - Changed sample test suite to pass by default, with
   commented out failures.
 - Demonstrating how comparing to similar objects is
   "not equal" in javascript (this is not a flaw in QUnit).
   And adding a test demonstrating use of deepEqual which compares
   by value instead of reference.
 - test/index.html: Drop redundant attributes
 - test/index.html: Use new <div id="qunit"></div> format
 - Update command .travis.yml
ajax on question screen

Sending Ajax request
Fix to change_column_null

The alias_method_chain for change_column_null generates
change_column_null_without_expression, but change_column_null was
sending change_column_null_without, instead (missing "_expression").
Automation of LD_LIBRARY_PATH for tests

Here is the POM change to avoid the need to set LD_LIBRARY_PATH to run the unit tests.

I may proceed to make it avoid the need for LD_LIBRARY_PATH altogether.
Added jamon, japid and rythm template engine
proposition to add a reminder to enable expressions 

before using @preAut.horize, maybe it will save someone some time
Remesa Actualizada

numeroOrdenes, montoTimbreMunicipal, numeroSiniestrosTitular, numeroSiniestrosFamiliar
Added in a logrotate.conf file to rotate Instagram logs

Please review! @seshness @derrickcheng
Updating README.md with latest changes made to build process

-cmake removed
-webplatform clientFiles changed
FF-Rhein service html seiten redesign

Ich habe mal den statischen htlm seiten ein bisschen neues styling gegeben. Mobiles style fehlt hier noch.

ich habe auf der robinson seite mal den debug-text entfernt. Das verwirrt otto-normal-verbinder nur, und leute die sich damit auskennen wissen, welche Daten ggf weitergegeben werden müssen.

Auch habe ich da mal die /todo links entfernt. Verwirrt auch nur, und sollte erst eingebaut werden wenn die seiten da sind
SoundJS + started on procedural buildings

I've added very basic SoundJS integration and some other bits and pieces. Procedural buildings aren't turned on at the moment due to buggy coordinates.
windows phone plugin version
Added our organization Hack de Overheid

This is pretty much how we always have done hackdays (though there are of course still things to be improved).
build.sh: better method for linux architecture detection
using token authentication

https://github.com/Islandora/islandora/pull/217
draft changes to schema discussed during 121312 meeting

Added several additional fields to the core required / basic fields to support components of the open data policy (last update, contact name/email).  Added field "Homepage URL" in the list of expanded fields to allow for Data.gov to point users to an Agency-specific website for a dataset/API if the Agency has their own catalog or other website for the dataset.
Added a check to prevent accidental refresh on scroll up.

I added a check to prevent a page refresh if you didn't start scrolling within the top 5% of the page height.
Add an `is_published()` method for Content using the SimplePublicationMixin

This is the same check that the `PublishedManager` QuerySet filter uses. But where the Manager finds published content instances, this determines published status on a content instance you already have.
Maven 6

Hello Brian,

This pull request has two new features added to the liferay-maven-support

1. addtional JSP (standard inits) and web.xml tld updates as per the MAVEN-6 issue of liferay-maven-support

2. I have created a new feature for generating the search-container using the maven 

Please let me know your comments
README.md

In example:
use FOS\OAuthServer\Entity\Client as BaseClient;

Should be:
use FOS\OAuthServerBundle\Entity\Client as BaseClient;
Added getObject

These changes add a getObject function it gets file the filecontent of the object
Refactor chassis wheel panel
update a10 camera source

this works on p76ti
Update Ukrainian/main/Thinkfree.apk/res/raw-uk/eula.txt
Mon everest !

Purée j'ai cru ne pas en voir le bout !!!
Il se peut qu'il reste une ou deux phrases à traduire à cause du contexte difficile à cerner et une ou deux expressions sur lesquelles j'ai des doutes (mais encore une fois le contexte peut aider à corriger le tir).

Mais pour le coup une relecture par des yeux neufs ne serait pas du luxe !

Bon courage ! ;)
updated speaker notes

updated speaker notes
Yeah I thought so too...
Sorry about this, guys.  I think it happened when I merged Kyle's pull request and I'm unsure how to fix it. :(
With this the warriors are nerf?
It should be, but perhaps there is some other code that sets focus on load.
Will take a look. Out of interest what is the extra button?
On Jan 25, 2012 5:09 AM, "HitcherUK" <
reply@reply.github.com>
wrote:

> One last question I promise -
>
> Is the default control hard-coded to the first 'proper' context button (id
> 1000) because I can't get it to focus on an extra button I've added to the
> top of the list? If so, could it be changed to the actual grouplist now
> that this is the official context menu setup?
>
> Thanks.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/xbmc/xbmc/commit/277fc2f53df5682081512e4e8bceeec1f6afcca0#commitcomment-896677
>
Is already fixed in mainline.
shouldn't we rpush here, otherwise we're behaving like a stack
Please set roles to empty string or maybe "user" if it is missing rather than breaking running code.
I've checked for invocations of ExternalCommand from the code. One is EditCommand which is disabled atm and other is OpenAttachmentCommand. This shouldn't reintroduce #219, atleast my mailcap has quotes around the filenames and it seems to be intended to be used this way. (Note: this approach is not perfect as it would break filenames with single quotes, but I assume this is how everyone else uses the mailcap file anyway)
Don't worry, you're right... kind of.  Your logic makes sense, and I didn't notice how large the search box was, but I did do an extensive amount of testing (that's how I got 600).  Here's what I just debugged:

count = 624

That's the count when a tab doesn't exist.  It's ALWAYS above 600, but since you pointed it out, that doesn't really make sense, does it?  Maybe there's something wrong with Simba's countColorTolerance?
Can you add some comments about what these linker flags are for, particularly -lexecinfo the two gccs and -zorigin?
cf. 30607b0c3aecaba4a994438faaef6b3fdc97906d
This breaks when child is an isolate (there's no disconnect method).
The work function W of a metal is closely related to its Fermi energy  (defined relative to the lowest energy free particle: zero in the above diagram) yet the two quantities are not exactly the same. This is due to the surface effect of a real-world solid: a real-world solid is not infinitely extended with electrons and ions repeatedly filling every primitive cell over all Bravais lattice sites. Neither can one simply take a set of Bravais lattice sites  inside the geometrical region V which the solid occupies and then fill undistorted charge distribution basis into all primitive cells of . Indeed, the charge distribution in those cells near the surface will be distorted significantly from that in a cell of an ideal infinite solid, resulting in an effective surface dipole distribution, or, sometimes both a surface dipole distribution and a surface charge distribution.
It can be proven that if we define work function as the minimum energy needed to remove an electron to a point immediately out of the solid, the effect of the surface charge distribution can be neglected, leaving only the surface dipole distribution. Let the potential energy difference across the surface due to effective surface dipole be . And let  be the Fermi energy calculated for the finite solid without considering surface distortion effect, when taking the convention that the potential at  is zero. Then, the correct formula for work function is:

Where  is negative, which means that electrons are bound in the solid.
I think you're reading too much into these examples. :)
They are only meant to demonstrate the way the code should be formatted and how naming works.
The "virtual" keyword probably came with a copy-paste operation or something similar.

But you are right; a getter or setter method should in general never be virtual; if there is need for a way to allow for subclasses to hook into it, a protected virtual "onNameChanged" method should be added (and called from setName).
Done, I reviewed other commits.  This one is more difficult, IMO these are false positives, this commit should not be applied.
When I initially design the class, I considered the word "trap" having
meaning similar to "hide".
Looking at the first commit, it mentions "used to prevent tooltips from
appearing"

Considering this last statement, I am not sure that mixing the idea of
enabling wrapping (richtext) and hiding tooltips is valid.

Thinking more about, it seems what motivated the integration of the "text
wrap" functionality with the "trapping" one is the fact the underlying
implementation deals with event filter. If the "eventfilter" capabilities
are in common, we should factor it out and still create a "wrapping" and a
"trapping" class.

To push further the analysis, the concept of "RichText" should be used
instead of "Wrap".

To conclude, what about having two classes: "ctkToolTipRichTextDecorator"
and "ctkToolTipTrapper" :)

Thanks
Jc

On Thu, Mar 8, 2012 at 10:15 AM, Julien Finet <
reply@reply.github.com
> wrote:

> The more I think about it, the more I think toolTipsWordWrapped is
> dangerous and not scalable. It has a side effect of "trapping" the tooltips.
> and the following is odd:
> setToolTipsWrapped(true)
> // here tooltips are trapped (installEventFilter), but toolTipsTrapped()
> return false
> setToolTipsWrapped(true) should internally call setToolTipsTrapped(true)
> then.
> Code is duplicated, the installEventFitler and removeEventFitler are in 2
> locations.
> but the following would be counter intuitive:
> setToolTipsTrapped(false)
> setToolTipsWrapped(true)
> -> now toolTipsTrapped() returns true, having the order of calls
> Trapped/Wrapped being important is a risk.
>
> Also what happens when we'll add more features to the tooltiptrapper ?
> each option would have to install the event filter ? does not seem scalable.
> So I would prevent toolTipsWordWrapped from installing the event filter,
> and let that work to toolTipsTrapped.
>
> Jc, I wouldn't split the class in 2 . I believe the "wrap" feature is
> generic enough to be part of the "base" tooltip class.
>
> Danielle, if you don't have time to work on the cleanup I can do it, let
> me know.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/commontk/CTK/commit/4314c15836b87f2946665bda7e442ecd4593cb16#commitcomment-1058722
>



-- 
+1 919 869 8849
awesome! Thanks!
Well, it's really two small changes, consistently applied in several
places. :)

/Ulf

Den onsdagen den 11:e januari 2012 skrev benbro<
reply@reply.github.com
>:
> It's a big change so I'll play with it for few days before using it on
production.
> Thanks
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/benbro/gproc/commit/7505b2247795ac7c0ca9607066b09080a89d5e93#commitcomment-855255
>
@atmos Thanks ! btw how do you bundle so that next time I can do it myself?
Yeah, seems like a string is easier, although probably that recursion in _getCtor() is unnecessary, you could do:

```javascript
function _getCtor(/*String*/ types){
	if(!_ctorMap[types]){
		var mixins = darray.map(types.split(","), function(t){
			return (dlang.getObject(t) || (~type.indexOf('/') && require(t));
		}, type = mixins.shift();
		_ctorMap[types] = mixins.length ? 
			(type.createSubclass ? type.createSubclass(mixins) :  type.extend.apply(type, mixins) ) : type;
	}
	return _ctorMap[types];
}
```

Hmm, maybe that's not any shorter.
Should be just: common.__file__
cse is still open
Typo in commit message
No, go ahead, but note that the newer changes are my own usage-specific hacks, so you might only want to pull the initial cleanups.
2.0.0 has identical munging - it was just harder to decompress the tarball because OS X doesn't support .tar.xz natively :-(

But, yes, I do see your point. Sadly, DADiskGetBSDName() returns "disk1" instead of a file path, and IOKit uses the short name, but easy enough to hide the munging in mediamonitor-darwin.cpp ?

(e.g. use /dev/r<BSDName> in diskInsert() so m_DevicePath is the long raw path)
This file should have been generated by clang. llvm.memcpy is an intrinsic so I don't know why it would be declaring it explicitly.
В этом массиве кажется одна строчка лишняя. Не считал? Хотя может и нужна где то
Forget what I said earlier. Blair wants you to implement it to return true if and only if all the values in the bitset are true. If a single value is false (0), getrawbutton returns false.

The update registry class is something different, don't concern yourself over that.

Sent from my android device.



-----Original Message-----
From: swarage <reply@reply.github.com>
To: Anirudh B <anidev.aelico@gmail.com>
Sent: Sat, 14 Jan 2012 11:42 AM
Subject: Re: [612-code] Added a registry in which you can register functions to be called in (9cb5a4a)

for the joystick implementation, how would the GetRawButton function work? I know it has to recognize a button being pressed, but how can we code the implementation for it?

Do we have to leave the function alone and it will work or code inside it?

--- 
Reply to this email directly or view it on GitHub:
https://github.com/rbmj/612-code/commit/9cb5a4a7c37c39e431880990a86e68a10684300b#commitcomment-867435
thank you very much for your bug fix.

i'm very sorry for the late feedback, because of my continues fever
There is a problem with cyclic dependency (try to package whole parasim project). Source code needs to be refactored.
:) absolutely. 
This commit should not to be here )). This code has been written in experimental branch 2 months ago and has been forgotten ))
2 days ago I have installed 'oh-my-zsh' with some plugins and had some practice with it.
This commit is the result :).
So I'm very surprised. I never thought that someone will be looking at this code at all :)

But ! Thank you for your advice. I'm just transferring from '1C' and 'C#' to Ruby and RoR, and of couse I need those advices and guidance.
Looks + Works fine for me
This import should be at the top of the file with the rest of the imports, not mid-function.
That wasn't the intention when I said using _GlobalSelectionService_.

Activator should have these methods save and load. And when the
bundle is started it automatically set's the selection if there is no
selection available. 

NullPointerException is thrown, when starting:

```java

java.lang.NullPointerException
	at de.lmu.ifi.dbs.medmon.medic.ui.views.HomeView.refreshPatientLink(HomeView.java:151)
	at de.lmu.ifi.dbs.medmon.medic.ui.views.HomeView.createPartControl(HomeView.java:109)
	at org.eclipse.ui.internal.ViewReference.createPartHelper(ViewReference.java:375)
	at org.eclipse.ui.internal.ViewReference.createPart(ViewReference.java:229)
	at org.eclipse.ui.internal.WorkbenchPartReference.getPart(WorkbenchPartReference.java:595)
	at org.eclipse.ui.internal.PartPane.setVisible(PartPane.java:313)
```
It already is in the Scroller trunk :-):
https://github.com/DataTables/Scroller/commit/b9ecc510dbeeedfc435311a8fc3fa874d494b70a
http://datatables.net/forums/discussion/8367/x#Item_2

The fix hasn't been packaged up in a new version yet, but it is available as the nightly on the downloads page: http://datatables.net/download .
I've just finished installing all the latest components and still the problem is there.
thanks. I felt great after tonight coding sesh.
This needs to go in the library files, rather than in the java code source.
Wie jetzt? Es gehen alle übelst ab o.O :)
Am 19.01.2012 10:45 schrieb "Sighter" <
reply@reply.github.com
>:

> Mit welchem Commando kann man das Rendern?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/brainafk/Empire/commit/6c9ce89ee5ce26ea8d4fc30ee491d5a1383c53a8#commitcomment-881370
>
I'd prefer if you create a new case here just like S_IS_LINK so its nicely sorted.
The message isn't consistent. First it complains to user that execption should be caught by reference. And next line it hopes user does so by using "could". Do you want user to do it or what?.

If there is such a recommendation then there should be a reference. Just saying something is recommended (where? by who?) is not convincing at all. Even more, it is good to tell *why* it is recommended. Users don't do it if you don't give them good reason and a motivation to do it. Otherwise you are just nagging for them without a reason.
Связано с issue #113
O cake pede $components. :)
meh i couldnt add him as author, what was the command again?
Requestを継承していたのは遥か昔のままコメントほったらかしてましたw　
ちなみにSimpleAuthの引数もRequestFlowになってます、
->user()はSimpleAuthには無いですが、ログインユーザの認識の為にユーザモデルなどをlogin_condition成功時にセットしています。
On Mon, Feb 13, 2012 at 2:20 AM, tea
<reply@reply.github.com>
wrote:
> # BeatInfoChanged slots has direct access to metronome
>
> I suggest use Qt 's signals/slots.
>
> (in this metronome case)
> If applyed this suggestion, the variable "metronomeBar" will be used only within the MainWindow constructor
> and the all tasks what MainWindow side does for metronome object are:
>
> * Create and Initialize the instance
> * Connect signals/slots
> * Add widget to layout

MetronomeBar does not need to know about bpm, only bpi.  The
BeatInfoChanged signal provides both values, so either MetronomeBar
would have to discard bpm or ClientRunThread would have to split this
signal into two signals (one for bpi change, one for bpm change).

Right now MainWindow updates the BPM/BPI status bar labels and then
forwards the new bpi on to metronomeBar.

I can try this out.

> ## Method like a "reset()" is often slot in other Qt widgets
>
> Here, the mean such like method work like reset(), e.g QTextEdit::clear().
> The reset() call can be signal/slot or called internal in metronome bar setBeatsPerInterval slot.

Yes, good idea.

> ### The behavior for BPI/BPM changed event
>
> windows client does not reset current beat position when bpi/bpm are changed.
> they count the rest of the loop, and update BPI/BPM on position at 0.
>
> Can remove the reset() on BeatInfoChanged slots ? or is that still needed for BPM changes ?

The bpm/bpi does not change immediately because the BeatInfoChanged
signal is raised only when the next interval starts.  We have the same
behavior as the Windows client.

The reason why I have reset() is because Disconnect() calls
BeatInfoChanged(0, 0).  This causes the status bar BPM/BPI labels to
say "N/A" and the metronome bar resets.

I think the reset() could be moved into the if (bpi > 0) else
statement, so it only happens when BeatInfoChanged(0, 0) is called.

> ## setBeatsPerInterval is a slot
>
> That can be connected by something bpi changed notify signal, instead of call the method manually.
> now the arguments does not match.
>
>  * the signal arguments are bpi and bpm
>  * metronome slot recieve bpi only
>
> This change may require add new signal for bpi changed notification, but it does not be much cost.

Okay, I commented on the same issue above.
For me this work:

```yaml
providers:
    administrators:
        entity: { class: AcmeBundle:User }
Changes made to metamodel for related to entityName (passing as a parameter needs to be revisited). 
Could you please elaborate more on problem you were facing on this?

I am already refactoring this multiple persistence unit issue, so will have a look later in detail. 

Could you please extract out this from current pull request and raise a different issue and add this to new one.

-Vivek
AFAIK it's been possible always, simply by using framework.GetModule("TundraLogic").StartServer(..) etc.
I see my mistake. I was working with an out-of-date version of the `pyqt` formula, which was 4.9 rather than 4.9.1. Running `brew update` fixed my problem. Thanks, @samueljohn.
what are reasons to wrap replace in `if` ?
Use `equal()` instead of `equals()` (assuming equal() exists within the older version of qunit we're using. If not, then just ignore this comment).
git add . !!!
It did, I would have needed a dedicated server, 8gig memory etc for my stats, no thanks... I went old school and compiling nginx logs at night for basic stats, real time is nice but I can just see them the next day.
Yeah, not sure how I feel yet about enlive, domina, and templates yet. Hiccup, noir, pinot feels better for some reason. Need to hack a bit more.
Shouldn't this be `_findRecent`?
ok THAT is strange -.- sry
Hmm not exactly sure what you were running into here, but based on some similar issues.  I can't completely explain it... I find it pretty subtle and hard to reason about.  

My theory is that the issue here is tying state.email and pmfromemail together via a value="{{state.email}}" field in the template. I *think* this is not quite right because the value="{{..}}" is only truly filled in as a result of completing the re-evaluation of the template -- (ie post $digest).  

I believe you get what you want if you do a self.$watch on state.email that sets self.pmfromemail = state.email.  This will execute during $digest (and watches keep executing until everything settles down)

ie ditch value="{{state.email}}" and in dashboard do a

```
self.$watch("state.email", function(scope, newVal, oldVal) {
  self.pmfromemail = self.state.email;
});
```
would changing values also increase the amount of animals/monsters spawned?
or
would animals/monsters show up faster if for example i turn day, to sudden nighttime?

thanks
You should not delete files if you just want to change a few lines. Also be a bit more verbose in your commit messages, eg. explain why you delete or update some part of the code. Let me quote from our Studio wiki:

"One commit should solve one problem. It should be an atomical unit -- before and
after the commit, the code should be in consistent state. If you can break the
problem into more subproblems, it is usually good idea to do it and split the
change into more commits. This makes the understanding and reviewing the commits
much easier. The important exceptions are:

  1. You are doing the same or very similar change on many places.

  2. You are rewriting a part of the code completely.
"
Wrong message on this error
Blank comment line between the summary (first line of RDoc) and everything else.
I missed your request to give some sort of indication about what to do.  How about "Not enough context to partially apply hunk.  Use `+' to increase context."?  This fits on an 80-column wide message line.
what kind of bug was this?
couldn't we just upgrade to a more recent version of Prototype?
Unfortunately not. There is still a little more work needed in documents regarding video thumbnails
This should throw an `IllegalArgumentException` instead of silently ignoring an invalid argument.
spaces, 4 spaces = 1 tab. I thought  I had my editor defaulting to this. I'll double-check
Man, do you check this manually? or do you have a tool for this?
El directorio nbproject contiene metadatos utilizados para que
Netbeans IDE lo reconosco como un poryecto.

Lo eliminé para que quienes se bajen mis commits no tengan conflictos con sus IDE'S
Whoops, I delete-trailing-whitespace'd our pants off.
ProcessBuilder has all that, already? What does this class add to a ProcessBuilder? 

What this class does is split a "command" into executable and arguments... but this might not always be convenient. It depends on the calling code. You could solve that by making CommandBuilder optional:

```java
process = Process.command(new CommandBuilder()
  .setExecutable("rm").
  .addArgument("-fr")
  .addArgument("/")
  .build());
```
Thanks, I always forget about that. Meaning I need to maybe create a sub-function to first check if set then check condition.
I'm like this commit, especially since I've been over-using `:before` and `:after`. However, these guys are supported in IE8 and apart of the CSS2.1 spec. So I suppose "what's our baseline?" is up for discussion.
either use ->getDate() (see http://phpcr.github.com/doc/html/phpcr/propertyinterface.html#getDate() ) or just $this->getPropertyValue('jcr:created')
Тексты http://pastebin.com/3K71FWEE
правда они уже есть в SD2 но под другими ID.
Creo que habría que mantener la URL relativa
If you have a filename with a slash in it, e.g. `Image_Big/Small/Medium.pdf` then its actual name is stored on the computer as `Image_Big:Small:Medum.pdf`.

No idea if this is related in any way, but I thought I'd mention it just incase this is what the lines used to do.

Currently, QS shows the object name to be `Image_Big/Small/Medium` and the details (path) to be `Image_Big:Small:Medium.pdf` which makes sense to me.
I'm trying to compile it.Immediately I compile it I will implement the code and get back to you.
so how should we do it then?
I don't know what happened here with git, actually.. these are just the same code.. just entered things here and there, but git almost reaplace it all '__'
I like that idea, thanks. Care to send a pull? Otherwise I'll try and take a look soon.



On Jan 26, 2012, at 10:20 PM, Oleg Korsak wrote:

> isn't it better to use variable name "use_transition", and then pass "false" into show(), also you will not need to do inversion in if() statements.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jquery/jquery-mobile/commit/bb17f146e47b0f48990fa014b3add8e8ceeb01a3#commitcomment-904426
This should be folded into the first patch since it fixes an issue introduced by that patch
Truly a work of art.

`:)`

AfC
As @fabpot backported the changes done in the bin/vendors script to the SE 2.0, people downloading Symfony 2.0.10 when it is out will also have the ``lock`` command to lock the deps to their current state.
translate and escape
We don't want to use "arial".
Once sniffs are created based on PSR-1, there is enforcing. Advices and suggestions should not be listed in PSR-1. It's a set of rules, not best practices.
了解です。マージしておきました。
Should this be:

```c#
var serverPipeStream = new NamedPipeServerStream(uri.Host, PipeDirection.InOut);
```
I still have an issue with this. I have 1 tiled layer with transparent tiles and nothing is showing on the screen.
The absolute positioning is specified in the core theme (jquery-ui/themes/base/jquery.ui.core.css: .ui-widget-overlay), but those without that css will see the overlay in the wrong place (as described in ticket).  Makes sense to always have the absolute 0, 0 positioning for an overlay.
Inconsistent behaviour: only a single warning is issued in case of duplicate import, however, two errors are issued for duplicate parameter/pattern names. The error/warning distinction looks fine, but I think, both parts of the duplicates should be reported here (and then the corresponding test case needs to be updated).
Ahh crud.  I'm still trying to figure out how all this merging/branching stuff works.  I'm not entirely sure how that got in there.  I'll reverse this commit and re-submit the push request.
I think the objective-c best practice says that the designated initializer should be the one with more arguments.
No, it's fine as-is.  I just wanted to investigate if this wasn't fixed for other slackware users.  Thanks!
`ftp://` didn't work, `http://` does
Needs regression test.
gone in the wilderness of my loneliness.
FYI: The nil test isn't necessary because calling -boolValue on nil will return NO anyway.
here you can see the door: http://img196.imageshack.us/img196/6819/immagineflg.png
Вася, ты смотришь старые коммиты, я это все подправил:)
Suggestion for method name: descriptionOfAddedThrowableShouldGetAddedToPresenter
Todavía no funcionan correctamente. Falta depurar el código y reparar el error de los accentos y ñ
As a point of style, it is generally a bad idea to have "using ..." directives in headers. I think I read that in Scott Meyer by I'll have to check the reference. It pollutes the namespace for other files which include this header, but are not concerned about the details of your implementation.

I know this can make headers verbose, but it does help keep code clean as projects get bigger as the names inside namespaces can often be very generic, and it is easy for unintended conflicts to occur.
* 1. use english,, i dont understand spanish.
* 2. 4.3 is not for public use...so no this is not for 4.3
Using `respond_with` instead of respond_to will save some lines:

``` ruby
respond_with(@locations) # this replaces the whole respond_to block
```
If you call the sniffer like this: `".../php" phpcs.php --report=xml --standard=.../htdocs/drupalcs/Drupal .../htdocs/drupalcs_TestFiles/bad.module`
and the standard uses stuff like this in the ruleset:

    <rule ref="Drupal.Commenting.FileComment">
    <!-- Do not run this sniff on txt files. -->
    <exclude-pattern>*.txt</exclude-pattern>
    </rule>

You end up having an error like this:

    Fatal error: Uncaught exception 'PHP_CodeSniffer_Exception' with message 'Referenced sniff Drupal.Commenting.FileComment does not   exist' in .../php/pear/PHP/CodeSniffer.php:814
    Stack trace:
    #0 .../php/pear/PHP/CodeSniffer.php(730): PHP_CodeSniffer->_expandRulesetReference(Object(SimpleXMLElement))
    #1 .../php/pear/PHP/CodeSniffer.php(623): PHP_CodeSniffer->getSniffFiles('.../htdocs/drupa...', 'Drupal')
    #2 .../php/pear/PHP/CodeSniffer.php(438): PHP_CodeSniffer->setTokenListeners('.../htdocs/drupa...', Array)
    #3 .../php/pear/PHP/CodeSniffer/CLI.php(551): PHP_CodeSniffer->process(Array, '.../htdocs/drupa...', Array, false)
    #4 .../php/phpcs(38): PHP_CodeSniffer_CLI->process()
    #5 {main}
      thrown in .../php/pear/PHP/CodeSniffer.php on line 814

This happens because the currently the autoloader doesn't use the path given by the parameter `--standard` for classes to load.
The change above adds code to check if a class or a path is within the path defined by the parameter.
Android patches start here!
I think now you should add in the friends data such as who's online etc.
Todas estas líneas se pasan de 80 caracteres, para la próxima corrígelo también
Ok, mislio sam to napraviti u samom templateu, no nemam django-template-fu kakav ti imaš. :)
change that line to
```
config.less.paths << "#{Rails.root}/vendor/bootstrap/lib" if config.respond_to? :less
```
I know that I set a precedent with 'plain=1' in the server, but it would be better if we use the HTTP 'Accept' header to indicate acceptable response types for the client.
2 days ago i was change it
the only different is on /sd-ext
/sd-ext 	ext4    /dev/block/mmcblk0p2	NULL	ext2

and cwm cant mount sd-ext
im not yet try remove 'ext2' string from /sd-ext
Typo here!
Should be "settings".
I get an smem error as well, but as far as I can see, you get an additional smem error in the place where the USB serial should be read. Compare your dmesg log to my own: https://gist.github.com/685757b47371d2079c35
Why no jQuery selector here?
Ha ha! Python hat Exceptions, ja. Ok mache ich! :)
Your'e declaring a variable in this method 'pretty' that is never used anywhere.  Looks like you copied code?
that's what I thought, gonna check into it more today, I'm looking at a few things I did yesterday asking myself what the heck was I thinking... It's a lot of code to add in one day and not miss a few things, anything else you see just let me know...
TODO: revert this and add a check for configuration changes in showbuildprop so we don't waste processing time on reloading something that hasn't changed during orientation change events
why remove all of this code? i personally use it.
Do we know why the matrix needs to be transposed for >1 assay?
Shall we just leave all four in the comment and remove '<-- what to call it?'
I would use the same approach used here https://github.com/gem/oq-engine/blob/nhlib-integration/qa_tests/_utils.py#L48 to check if two xml stream are "equal"
Duplicate `"description"` key. Copy/paste error.
OK, after discuss we decide to use selectedEvent and currentEvent for select and hidden input name.
If you're compiling for 1.5 compatibility though I don't think it likes it when you @Override an interface.  I'm not sure if we're requiring 1.6 yet.
My thought is it would prevent people from having to disable the ticket registry cleaner.
You changed the order of arguments in solve, any reason?
lGtm ;)
I'd probably remove all these methods and added a way to register async callbacks to the transitions.
you might want to put in a //FIXME so that you don’t forget to include the changes to that code in master (at the point when this goes live); no need to rush right now
So, create a message that you send to channelActor, that doesn't close over state, and then use AskPattern instead of creating your own Promise.
It never returned Changes. It used to return the subkeys of the current node (not used anywhere) now it returns all the subkeys of the newly integrated node.
In the context of the use case it shouldn't be a big Map, aprox 100 nodes. One could perhaps do something with takeWhile or dropWhile, which could save some tree rebalancing. Should I spend time on it? In that case I would benchmark it in the context of the router.

/Patrik

13 sep 2012 kl. 15:14 skrev Björn Antonsson <notifications@github.com>:

> In akka-actor/src/main/scala/akka/routing/ConsistentHash.scala:
> 
> >    }
> >  
> > -  private def hashFor(bytes: Array[Byte]): Long = {
> > +  /**
> > +   * Factory method to create a ConsistentHash
> > +   * JAVA API
> > +   */
> > +  def create[T](nodes: java.lang.Iterable[T], replicas: Int) = {
> > +    import scala.collection.JavaConverters._
> > +    apply(nodes.asScala, replicas)
> > +  }
> > +
> > +  private def nodeHashFor(node: Any, replica: Int): Int = {
> > +    hashFor((node + ":" + replica).getBytes("UTF-8"))
> I think this is a non-issue, I'm more scared about nodeFor.
> 
> —
> Reply to this email directly or view it on GitHub.
>
extends PossiblyHarmful
ALLOW is *nice*
On Sep 11, 2012 1:14 PM, "Patrik Nordwall" <notifications@github.com> wrote:

> In akka-cluster/src/main/resources/reference.conf:
>
> > +    # enable cluster aware router that deploys to nodes in the cluster
> > +    enabled = off
> > +
> > +    # Maximum number of routees that will be deployed on each cluster
> > +    # member node.
> > +    # Note that nr-of-instances defines total number of routees, but
> > +    # number of routees per node will not be exceeded, i.e. if you
> > +    # define nr-of-instances = 50 and max-nr-of-instances-per-node = 2
> > +    # it will deploy 2 routees per new member in the cluster, up to
> > +    # 25 members.
> > +    max-nr-of-instances-per-node = 1
> > +
> > +    # Defines if routees are allowed to be located on the same node as
> > +    # the head router actor, or only on remote nodes.
> > +    # Useful for master-worker scenario where all routees are remote.
> > +    routees-on-own-node = on
>
> I like allow-local-routees, another suggestion is use-local-routees
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/akka/akka/pull/653/files#r1575362>.
>
>
Documentation is missing.
Default 'Fluid'
The upcoming if/else block is unnecessary. Since `___reservation_to_instance_object either returns None or an object, you can just return it:

```python
return ___reservation_to_instance_object(ec2conn, res)
```
This was done intentionally because without it, it would take directories into account as well and that was resulting in unpredictable results in the IsEverythingIncluded property. The File.Exists is being used to make sure that the path in question is an actual file and not a directory.
What value does knowing the type of exception give in this case?  Either
I'll replace it with Exception, exc or list all the possible exception
types for absolutely zero gain.
On Jul 17, 2012 8:40 AM, "tobes" <
reply@reply.github.com>
wrote:

> > @@ -601,7 +601,11 @@ def _save_new(self, context, package_type=None):
> >          except DataError:
> >              abort(400, _(u'Integrity Error'))
> >          except SearchIndexError, e:
> > -            abort(500, _(u'Unable to add package to search index.'))
> > +            try:
> > +                exc_str = unicode(repr(e.args))
> > +            except:
>
> i really don't like raw excepts - and again below - line 642
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/okfn/ckan/pull/70/files#r1176276
>
The trailing punctuation gets published as "flag.:", so perhaps ditch the period.
Small typo: "list of what we already hvae".  "have" is misspelled.
Can't this comes from the constructor so the app can configure it?
i'd rather see you handle this in CPVRChannelGroup when updating
i will check if it's needed
Yep, check m_started mainly solve this. But still lost subtitle few times. In my test, pts not equal m_pClock->GetClock() even m_started is true. pts little bigger than m_pClock->GetClock(), about 50 - 100.
Да. $home = $app->homeUrl . ($app->homeUrl[strlen($app->homeUrl) - 1] != "/" ? '/' : ''); я так написал в handleLanguageBehavior, в общем нормально Можешь написать как я.
1) Только поменяй 'showScriptName' => true, мы собирались отказаться по-умолчанию от index.php
2) И добавь строчку в changelog, без нее мы теперь не принимаем
Should probably be called `_append_to_conf_paths()`. But it's a private function, so not that big of a deal.
This is to assure FFOS always gets high quality icons.
What's the point of the `{ nil }`? Won't `options[:max_locks]` default to `nil` anyways?
I think instead of a builder use copyOfRange in the vector class.
@pflorczyk Check [this comment][1].  :-)

[1]: https://github.com/dahlia/wand/pull/53#issuecomment-8351202
best to add a comment that this is needed during a restart
Is this true? I don't see this happening anywhere in these changes.
SHPObject* shp_object
@bnoordhuis 

> That shouldn't be a problem in v0.7 and v0.8, V8 supports surrogate pairs now.

I’m not sure I understand what you mean by that. 

This is about how JavaScript strings are exposed (by spec), and what JavaScript considers to be “characters” (i.e. each surrogate halve is exposed as a separate character, [UCS-2 style](http://mathiasbynens.be/notes/javascript-encoding)).

Are you referring to http://code.google.com/p/v8/issues/detail?id=761? I don’t see how that helps in the situations I described (i.e. getting the total number of symbols in a string, or generating a string based on any numeric code point (including those greater than `0xFFFF`). Would you mind clarifying?
Make sure all lines line up.
This comment is very confusing... why do we need to close handles that are already closed?
Was going to do it that way but then saw that subsample_fasta was doing it with the raise. Will change though and repush
I guess this probably came from master, I've just removed that in 9e15e572fba5afb0f7e4e011025aab1e9da863c5
You mean remove the file from commit or delete that line? Thanks.
This changelog should be in `activesupport`.
I think that the Rails core prefer to not use short-circuit `return`. Please keep the `if` block.
why was the check on warnings moved here? It reads well originally.
https://gist.github.com/4326248 this one  works too, but i don't this that this is really better than define_method
You only need the try-catch around the default block
Note:  We have issues with early initializers too.  I had to adapt the logic to be just constructors of objects.   It appears to work and allows more this.getClass to fall through than before.

NOW, if I return an EmptyTree, can I tag an error for it?
If this branch is taken by the compiler, it indicates a fatal error, because this means I misunderstood how template parsing works. What do we do in that case? `abort`?
Hmm, I can't reproduce:

```scala
scala> object T extends Throwable
defined module T

scala> val t = T
t: T.type = T$

scala> try { "okay" } catch { case T => }
res8: Any = okay

scala> try { "okay" } catch { case `t` => }
res9: Any = okay

scala> try { "okay" } catch { case x @ T => }
res10: Any = okay

scala> try { "okay" } catch { case x @ `t` => }
res11: Any = okay
```
Sadly, no. Our checkstyle needs to be much more strict it would seem!
Because the both results are correct, so I wanted to test that it gives at least one of the two.
Let me try it right here: `f: A->B`.  I've also had this idea: `f:A-->B`.

I think I like the longer arrow slightly better.  Do you think not having a space after the colon is a bad thing if we make the arrow longer?
pep8: spaces around `=` except when used to pass keywords and spaces around binary operators `+` and `-`
```
Inline comments should be separated by at least two spaces from the statement.
```

http://www.python.org/dev/peps/pep-0008/#inline-comments
and this to 
```php
* @param string extension name to be checked. Possible values are 'imagick', 'gd' and null.
```
if now preferred engine is imagick
Yep, just recognized that... Should have written tests before implementing it :)
getLastModifiedValue(), getEtagValue().

Consider this case: if someone wants to change the way LastModified is calculated (this is very likely to happen), he would have to override preFilter() and copy large chunk of code.
This is a temporary measure; the actual data is correctly populated by the importer.
This works good. Another thing you could do is

@highlight_clr = Setting.find_by_property("HighlightColor")
@dt_ring_clr = Setting.find_by_property("DrColor1")

Although the way you have it, performance is probably better.
You can close this without merging. Thanks for considering
Thanks. I was planning to ensure that little debugging statements like this were not present in the final patch--do you think there's still value in having it in at TRACE?
I wonder if we should switch the table naming to be similar to the zk path naming.  I.e. we can just specify a prefix unless we need individual overrides, at which point we use the individual overrides?
same issue with returns .. You could state "only one document in our ``things`` collection matches this criteria'
Merge markers - bad stuff here! Can't merge with this.
yeah i think it's fine to leave it w/o the _
Thanks for pointing me to the docs on creating Python objects. I was curious about how to do this with the Python linkers. 

Ah, so GpuFromHostStart + GpuFromHostWait == GpuFromHost? This sounds like a good idea. It also makes it easier to turn async gpu on or off.
I think making the content here "callback_order.inspect" suggests to the reader that it's special somehow. Why not just make it "example after restart", so it's clear that it's not important, nor will it be tested later, that it's related to callback order?
This falls under "insufficiently clear" then. !pcre_match(/^...$/) will be clearer.
Should either use a less common port (e.g. 8591) or allow the port to be configured.
Yeah i guess, so this can stay as is for now.
MIssing a verb?
should this be 3?
确认密码可以不用了，如果用户输出，登录时还可以通过「忘记密码」找回来。
var values = [];
This is much more efficient (no longer rendering to a temporary string) and it makes the IRenderContext available to the LiquidNancyFileSystem (as well as any other Nancy-aware DotLiquid tags, blocks, etc.).
using statements should be placed inside of the namespace declaration
here, call both REG and SCR versions of createList
suggest 
- moving this to line 28 / 31 (inside of the "if") statement 
- removing unmask from the else condition

there's no need for a a loading mask if form validation fails
@johnstoecker - if i had to choose, just remove it
@nundys - what do you think for JSS?

in the long run, probably all fields should be flexible, anyway, so we shouldn't have to care exactly what attributes are gathered by any one hospital
searchQuery will be a key=value style query on the GET that will be saved as lists.
So searchQuery should be ?encounterType=<uuid>&startDate=2012-05-01T00:00:00 (iso format) ...
Porquoi?
it just needs to return a Variable with type NullType. I got rid of NullVariable
Don't use that. This is not the recommanded way. Can you use
from theano import tensor 
or
from theano import tensor as tt
The java doc said we support them only, though in truth we seem to use no prefixes most places. It turned out, we supported prefixes for some types, but not others. I just made it consistent and flexible. Seems harmless to me. No?
(Please start reading my reviews from this one)

$LIBACL_* variables are lost here.
It's actually 4. Anyway I don't want these in the source, it's not hard to configure them per project in most editors, except I don't know how to do it for yours.
Please learn the alphabet, s is before t. Yep. :)
`content ,` => `content,`
This is a new function used to do bootstrap sampling  (sample with replacement). However, C4.5 doesn't use it. As it's a common function, we put it here.
This doesn't look right to me. TEXT is a collatable type, so we should use the collation associated with the argument. I'm currently trying to find out whether collations are associated with function calls or individual arguments...
This doesn't cause a problem now because we only use dev_tools_path as a string, but returning a different type of object in this case could theoretically cause problems in the future.

If we're not returning a path at all, maybe this should be a showstopper that raises an exception, rather than returning an empty string.
Comment is not necessary, one line this
Consider refactoring this function to take the query name atom as part of the arg list. This will make it more clear to the reader that this function operates on a given connection and a pre-prepared query.
please add a space after each ',' to increase readability.
Hummmm, tu veux que le serveur garde en cache certaines informations?
I do not understand this correction. You can not change file once it is committed. Si it is *after*, not before.
Maybe rephrase "put them into" into something a bit less colloquial. Even as simple as "store".
never finished this one eh? I'm fixing now...
force? Why force?
= false.

(I know it's redundant, it just makes the code more obvious).
You have to write `new URL[]` one place or another; prefer to do it here, to make it visibly clear that this is a do-nothing subclass existing just to expose `findClass`.
To me it's worth having an empty subclass for maintainability's sake... eventually you might want to pass in a custom binding class to use (and that should be a subclass of ViewBinding), and I don't want `AbstractBinding` to be something that's ever instantiated unless we change the name.
Nope, `validate` calls back with anything that has prevented validation from succeeding - namely, a validation error. IMO transport errors and validation errors are both errors, and thus both belong in the first argument of the callback.
What implementation of hashset should you use ?