labofsahil/github-event-dataset-2012 huggingface dataset:
Do a pure md>html conversion when previewing in Sublime

Full HTML preview for disk and browser preview. Pure conversion without extraneous styles and markup when previewing in Sublime.

This is a modification I have been using as I find it really nice to get a styled preview in browser/disk but only want the raw MD conversion to copy out of sublime.
Added a base class for all predefined perspectives

e2wm:plugin-exec-update is moved to the :update method of this base class from e2wm:pst-update-windows.

Fixes #18.
optimize std.utf.stride

- templatize functions so that short paths can be inlined
- replace table lookup with bit scan
Some fixes and modularization

Hey Kristian,

just started to develop exactly the same extension before I noticed your gem. ;)

I fixed some minor issues with the templates and modularized the sass file. I also changed the original block comments to single-line comments so they don't show up in the compiled css.
b716804: duplicate '/' in URIs.

Whoops. Was introduced in "Fixes per quick code review" (a3b8d07ab7db48960dcc88d649d55784dfd543b6).
Openengsb 1411 2.x.y

2ter versuch
(#12475) Add Apache 2.0 License to MSI GUI Dialogs

Without this patch the MSI package displayed the default Common Public
License.  This patch adds the Apache 2.0 LICENSE file which is displayed
early in the installation process.
    
The screen looks like:
    
![Puppet Installer License](http://links.puppetlabs.com/ftw_license.png)
Don't loop forever when there is a failed seed.
We would like to sync up because we are adding WP8 support these days.
Added settings file to theme

This adds a settings file which allows users to manipulate all of the settings that were previously hard coded into the theme. It does include a random option which overrides all of the settings.
Don't re-apply the theme when pressing OK if it has already been applied by pressing Apply

I have fixed the issue where the preference page was reapplying a theme if it was already applied.
Update for multistep benchmarks
Add filter for sslverify on zip url

the zip_url never gets the sslverify setting passed from this plugin, because it's not hooked into at the point when it's run. This addition will set sslverify to the config settings when WP goes to download the ZIP. Otherwise, sslverify defaults to true and fails on certain hosts that have no idea what the GitHub SSL certificate is (lol).

Props to @pglewis for digging so deep to find this fix.
unit of work issue

Doctrine gives an error about spl_object_hash() expects parameter 1 to be object, integer given.

This is due to setting the Job using the id instead of the object.
Edited descriptions of FileBank and GoogleWeather
Onsite fixes

Miscellaneous onsite fixes.
New test case for Options nested inside of Case Class

Do not pull this request until http://jira.codehaus.org/browse/JACKSON-774 is resolved.

I'll keep an eye on it and update this issue when appropriate.
LPS-25798
LPS-24867

Added missing javadoc xml
Pull request as requested in "What do I do if I'm new?"
Added myself to member list
Added myself

Hey LearnProgramming, I'm very new to Github, so hopefully I'm doing this correctly!

Thanks!

-Scott
Fix compatibility with CloudFront API version 2010-11-01

The latest version of right_aws uses the API version 2010-11-01 of
CloudFront, so the streaming-distribution script has to be updated for
that API version.
Symbol-Tables API, first approach

API not finished nor improved, needs to be corrected.
Added README

Well, title says it all.
Scio proposal - minor language change suggestion

Details sent to hacklab mailing list.

Cheers,

N
testing out master/gh-pages branches model

Hey I went ahead and broke out mine into two branches, and I invite you to do the same :) I'm suspecting that this pull request will create the master branch on yours, and you can then merge those changes into gh-pages, and then revert to your own links on that branch. Further merges from master I think will retain your customizations unless those specific lines are changed, so it should work fairly smoothly.

I'm pretty sure that the ignore model won't work, since we actually have to commit the changes. The other thing that might work would be some sort of customization script you could run, but that would mean those changes would go back and forth with every merge, which would be obnoxious. So the two-branch model is pretty clearly the technically correct way to go, if it's usable.
INT-2399 fix MessageProcessor for service-activator

JIRA: https://jira.springsource.org/browse/INT-2399.
1. ServiceActivatingHandler: refactor for using MessageProcessor interface.
2. ServiceActivatorDefaultFrameworkMethodTests#testMessageProcessor
3. Integration test for <service-activator> with invalid Groovy inline script.
Ics

Change-Id: Ia3bb54bba447aa41b8e5f7078ad13c40cffdd979
Code Reduction

Unfortunately I didn't have time to test it, but I'm pretty sure it's solid. I assumed you needed to test it since you have access to the data, but if not, let me know!
Tested,looks good to me
Ajustes do Menu

Agora vai essa fonte.
2.2 错字

忽略var和type的什么

忽略var和type的申明
增加了2.5的关于method的两张图片，以及2.4关于type的封装表示，以及一些相关语句。
Remove double click to share & collapse by click outside

##### Need much more effective solution
added to elastic options
```js
parentElement
submitElement
```
When mouse is over ```submitElement``` than blur will not collapse.
```parentElement``` is necessary to select the ```submitElement```

Selector of the submitElement
```js
$(this).parents(parentElement).find(submitElement)
```
I did not find any better solution for create simplest DOM selector.

-
##### Here is attached diff for m.profile.js, other files have no problem with TABs vs SPACEs

```
--- a/root/socialnet/js/m.profile.js
+++ b/root/socialnet/js/m.profile.js
@@ -8,313 +8,319 @@
  */
 (function($) {
 	$.sn.up = {
+						$.sn.comments.waterMark();
+						
+						$("#sn-us-wallInput").watermark($.sn.us.watermark, {
+						useNative : false,
+								className : 'sn-us-watermark'
+						}).elastic({
+							parentElement: '.sn-us-share',
+							submitElement: '.sn-us-wallButton'
+						}).trigger('blur');
+								$.sn._resize();
+								$.sn._textExpander();
 
-					        $.sn.comments.waterMark();
-					        $("#sn-us-wallInput").watermark($.sn.us.watermark, {
-					            useNative : false,
-					            className : 'sn-us-watermark'
-					        }).elastic().trigger('focusout');
 	}
 }(jQuery))
```
Bookmark-able and Navigate-able ToC

This request addresses [Issue #7](https://github.com/addyosmani/backbone-fundamentals/issues/7) by creating a linked ToC and named anchors for headers. The ToC is quite a bit bigger than before which makes me hesitant to submit this pull request but what the heck.
1.2.x

The javadoc:aggregate xml tag in javadoc configuration has no namespace and failed the pom parsing.
Getting started guide in Polish
NXP-7954: join gadget: proper message for rejected requests

added a new method in SocialWorkspaceService to get the user status of the last subscription request
Обновление addTextPanel

Изменения:
1. Более няффный код
2. ББ-коды теперь добавляются для всего выделения, а не для каждой строки. В сосаче работает, на остальных бордах не проверял.
Updated to Cordova 1.9.

- Added example and updated the README.
- Added badge set/get methods.
- Added launch method.
- Added example code for optionally handling notifications in JS code.

Reviewer: @aogilvie
Update Adafruit_CharLCD/Adafruit_CharLCD.py - Weird Characters

In delayMicroseconds(), it's always sleep(0) in Python 2.x (get a lot of weird characters on LCD)

This is because x / y returns an integer, not a float in Python 2.x
changed to: seconds = microseconds / float(1000000)
updates for the New WebWorks SDK changes

...builtin webworks.js.
Adding "Dying on the Vine" by John Cale
Move initializer for ActsAsGmappable from Rails::Engine to Rails::Railtie

Moved initializer "include acts_as_gmappable within ORM" from Engine to Railte
To solve no method error when using RailsAdmin's history feature (config.audit_with)
Replaced occurences of app/config. with app/config/config.

I don't think app/config.yml is supposed to work, is it?
updates webplatform hash for bbm card fix
More work on listing the stenograms and generating HTML.
Provenance Activity work PLFM-1307
Fixed timing bug in NodeTreeUpdateManagerImplTest
Testing

pretty pls (:
Correct clock settings for GPU @ 300 MHz
Pretty much working plugin

Fairly tidy and robust I think
Fixed PHP errors and...

Hi,

I've just downloaded the Whiteboard-Framework and it seems really nice! I've notice some PHP errors and notices that I've fixed. I am working on WP 3.3.1 and have no more problems now. I also took the liberty to change the comment system to use the TwentyEleven one as it works with threaded comments.

Anyway, thanks for this framework,

Nicolas
Operator precedence error means query execution times are meaningless. This corrects them.
release 2012.05.16
Fix the Uncaught TypeError: Cannot read property 'firstChild' of null error

I was experiencing an error on Youtube, I wasn't able to scroble anything.

Error : Uncaught TypeError: Cannot read property 'firstChild' of null error (coming from jQuery).

So I've made a few changes, and I have it working fine now :-)

Hope I didn't break anything.

(sry for the spam, my first PR was not properly set).
Brought index.html and README up to date with most recent parameters and improved formatting

Just updating both files with the HideQRCode, Width, and Height parameters that have been recently added and improving the alignment in the pre-formatted section of the index.html. I didn't align the README stuff since Github's rendering of it would require lots of horizontal scrolling inside the div.
I wanted a simpler starting template

Hi there! Thanks for maintaining these handy templates. 

Today I went looking for a way to start a new app with Haml as the default. I don't need much more than that to get my app going at this point, so I created a simplified template that basically just uses the common recipes of the other templates.

I don't know if you are interested in including it in your repo or not, but thought I'd offer it along in case it seems useful to you.

I started with the subdomain template and removed the unneeded recipes, leaving all I needed to get a landing page built.

(additional comments blogged: http://decielo.com/articles/377/haml-by-default-in-a-new-rails-3-2-app)
Preserve foreign keychar keymap

Without that file my German Xperia Pro with a QWERTZ keyboard has a wrong keyboard layout after flashing. This file should be the same in other devices, but that should be verified.
localization: tranferred the language strings of DokuWiki core to flashuploader plugin
Small typo fix and missing words

While reading through the documentation, I found another sentence that needed a small fix.
Header fix

This should keep it from crashing....
Added DraftKings
Wrap whole words

Wrapping words at characters is seriously nasty.
ah I think I know what you mean.. something like:

this[X.properties.classname()] to just have to maintain the string at one point? this would def. make sense to me!
working here, im not coder so im asking rest to rsa https://github.com/mangosR2/mangos/issues/293
I guess this is your New Year's resolution - more GitHub!  :-)
add space after for
Shortened labels and added descriptions to each field.  Also added separator field label and description.
@fcoury :sob:
@ai you can't predict how caching looks like on user's apps
i could believe that overall it may possibly be slower and if that's the case then performance trumps everything :)
It is not written, it is not clear.. 
I believe to have seen other frameworks requiring you to write such things in a configuration file...

I think documentation should explain things in the more clear and complete way possible, there is nothing to gain keeping such things "implicit".
This was a copy from the cpu ger tests. I'll fix both.
Should the .environments be under the so the div isn't needlessly created?
и еще в корень приложения надо помещать файл .rvmrc который будет активировать нужный гемсет
чтото типа rvm use ruby-1.9.3-p0@depot_flexoid --create
I consider that we should define these constants to get a formula and return automatically the correct width
It was time to do it. Jiri will work on a second set of bitmaps now.
Need to declare `e` as a var, otherwise it's in global scope.
Well I knew there is no difference, but I thought we were replacing TO<> to TO_ ^^
Ok, tu veux que j'te mette quoi comme putain d'messages de commit ? Make j'le fais sur quoi ?
 

> Date: Mon, 30 Jan 2012 10:57:43 -0800
> From: reply@reply.github.com
> To: rikrik@hotmail.fr
> Subject: Re: [iutbm] update (b032793)
> 
> Ce serait cool de faire des commits qui compilent, et en franais correct, j'ai autre chose  foutre que de bisect pour voir o a plante.
> Ce serait encore plus cool de foutre de vrais messages de commit.
> Et enfin, ce serait mega-cool que tu compiles avant de push : "make", puis si a ne amrche pas, tu regardes pourquoi (ou tu demandes  Google de le faire  ta place)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jvoisin/iutbm/commit/b032793428713179ce89c418400d41038e8b9d76#commitcomment-915235
Actually, isn't the convention here snake_case? The parser will convert it to CamelCase IIRC.
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?
so this method will only work when someone postet a field named "javascript"? better pass in the value as a parameter to minimize dependencies.
Yes, I already discussed about this with Ablu. He will look into building a more suitable "example/demo" world, and I guess our current example should eventually migrate to an official testing ground.
Remove the code that has been commented out in this method.
Thats a big twinkie
Sorry, the working copy I was using on my localhost is 4.1.0 must still have old jquery
I keep saying Post where I mean Thread. Maybe someday we'll have a SPOT for this.
path不是page的一个属性嘛
like 

percindex(.5, [1,2,3]) => 2
percindex(.01, [1,2,3]) => 1

so, you can index the list with a float between 0 and 1.
Смысл этих методов (isOkUp, isOkDown) вообще не понял. И из названия тоже неясно.
Cool , looking forward :) .
Thanks. This fixed the issue I was having after upgrading paperclip and installing aws-sdk.
As new version 0.3 has been pushed out without the get_version method.
On Feb 6, 2012 3:27 AM, "Michael Van Veen" <
reply@reply.github.com>
wrote:
pelase fix indentation here.
At least in client esES Blizzard decide to translate it as "El Estadium" http://img402.imageshack.us/img402/3716/untittle.png
`notification-box.error` should be `notification-box error`
AFAIK after `pyrus` calls you need to call `phpenv rehash` to get that installed code working properly ;-)
I need to keep an eye on if this affects #111
You're right, doing it with a string literal will work because the literal will get converted, but doing it with a variable will cause wrong code. (currently, it should work of course)

I still think these changes are correct and should be merged in.
What the the reason to have this method here?
Had to blow away my github repo as I left my username and password in the examples/foo.rb file.

You will probably need to reclone your copy of the repo.
As far as i know when in arena matches you have a hunter (specially hunter's pet) there were a situations when the pet had an "infinite" resilience which is an unbalanced pvp , this kind of resilience numers is handled by the core.
in the template we handle the case where there is no snippet and no title and just write "(no title)".
I'll look at that, I was going to hard code it in left_nav etc, but that's a better way I think.  But I'm going to leave the database override option so you don't have to edit code.  If the entry in the globals table is blank it will use the hard coded globals.php path.  Vendors that overide the patch can and will deal with making sure their override is correct in their own fashion.
Can you please always add "{" and "}" even for "if" that have only one command. Missing brackets always make me nervous about the code getting broken in the future
Yeah, this works too.
Thanks. :)
i can look through it as well and try to discover and disparities.

are you sure you mean the iOS API or did you mean the desktop api?
Yes i agree with Hieu about the selection of body parts through the image in the main activity. It will be difficult to program such a selection method, and users can push their fingers on the wrong area and i cannot ensure that it will be accurate enough to the user's desire. Feel free to have comments about that part of the interface.
Do we want a 'slow query' statsd counter?
Thought you would push the whole theme :)
I agree it seems more correct to use `kwargs` for reversing, but it's sooooo much longer than `reverse('localizers.categories', args=['es-ES'])`
you stole my migration #, grr
Ooops, the commit message should have been "Make Table iterator work in Python 3" not Row iterator.
haha yeah it was fixed but the guy patched the wrong file so the build clobbered it
A different string for the singular version exists, so there's no need to use (s).

See MESSAGE_BANNER_PGP_ATTACHMENT_VERIFY_ERROR_TITLE vs. MESSAGE_BANNER_PGP_ATTACHMENTS_VERIFY_ERROR_TITLE
omg locknes what u have done ?  seems like one of your custom char updates  like  jail kills the server on login :P
Thanks for this update Fabian !
Please note: due to a new lifecycle mapping, tycho 0.13 is not fully compliant with M2E1.0 :(
2 possible workaround:
- installation of the nighty build of m2eclipse-tycho (see https://github.com/sonatype/m2eclipse-tycho/pull/1)
- updates the erreonuous pom, and, ignores this new cycle.
Regards,
Philippe
This breaks the build on my linux, Mandriva-2010.2, x86_64 if I don't specify DEFAULT_QUEST on the cmake line.

/home/stevebr/devel/games/zelda-solarus-dx/solarus-20120219a/src/lowlevel/Main.cc: In function ‘void print_help(int, char**)’:                                                                                     
/home/stevebr/devel/games/zelda-solarus-dx/solarus-20120219a/src/lowlevel/Main.cc:82: error: ‘DEFAULT_QUEST’ was not declared in this scope
make[2]: *** [src/CMakeFiles/solarus.dir/lowlevel/Main.cc.o] Error 1
make[1]: *** [src/CMakeFiles/solarus.dir/all] Error 2
make: *** [all] Error 2
Command exited with non-zero status 2
Nope, the idea here is to get boottool to be able to check if a recent enough grubby is available. Boottool itself will decide what "recent enough" means.

A RFC here is: should boottool attempt to silently and automatically install grubby if a "recent enough" version is not found?
ASSERT(_me->SetCharmedBy(unit, CHARM_TYPE_VEHICLE));
Is the best way :)
verifyShuffle should be inlined: Reply = State#state.cardsLeft =< 0
Please, write in english.
Testing attachment of file.

On Sat, Feb 11, 2012 at 1:01 PM, Lucian Pacurar <
reply@reply.github.com
> wrote:

> Hi Stephen. No extra config is needed for this. You just need to "require"
> the module in the app.
>
> ```javascript
> var PUBNUB = require('pubnub');
> var myPubNub = PUBNUB.init({
>    publish_key   : 'demo',
>    subscribe_key : 'demo',
>    ssl           : false,
>    origin        : 'pubsub.pubnub.com'
> });
> ```
> This works for me, but it might not be the best way to do it :)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/lucassp/pubnub-api/commit/57aeedf60e3aa8f8c4539fa7e6f995637fdd6674#commitcomment-956989
>



-- 
Cheers!
Stephen Blum
http://twitter.com/PubNub
https://github.com/pubnub/pubnub-api - GitHub
www.pubnub.com
+1-425-830-6711
Yup,
just pushed @tonit's changes ... thanks! Let us know if this works
better. Is there a SDN test somewhere?

On Tue, Jan 3, 2012 at 5:04 PM, Jean-Pierre Bergamin
<reply@reply.github.com>
wrote:
> @peterneubauer and @tonit are collaborators now and should be able to push
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ractive/neo4j-osgi/commit/57461ae4b338e05f5d18c75d60731b4691eb7361#commitcomment-833930
@sajjad-ser Great thanks for the feedback and the pull request - keep them coming :-)
This commit probably deserves mention in the release notes..
Decrementing the 1, 2, and 3 and removing the -1 would reduce the number of steps, but I think the -1 is clearer, because it serves it's own goal (changing 1-indexed user input into 0-indexed values for the array)
Hi I'm sorry I'm a noob with all this stuff, just wondering...how do I go about this. I got the proxy server up and running but I only have an ipad 2 and no 4s. So would I just copy paste all this in the command line after I start the server. Thanks in advance.
472c59ee9791c181e38708e67337969b30327d14 改好了。
Doesn't tempts you to select it? I am always tempted. lol

for right menu, look:

https://github.com/lfottaviano/emesene/commit/c4db1a7461ce7ee33f98eacae3c8e812ca13f269
You haven't documented your `rel=image` attribute value. Other than that, looks great. Nice work!
Sorry, I tried this but it seems I'm not proficient enough with Git. :-/
I thought I did it right this time. Maybe you can review and apply the changes, I don't need the credits.

-----Ursprüngliche Nachricht-----
Von: Dean Michael Berris [mailto:reply@reply.github.com] 
Gesendet: Mittwoch, 15. Februar 2012 04:59
An: Michael Firbach
Betreff: Re: [cpp-netlib] Fixed an issue with chunked content - client was expecting too many bytes and could block forever. (098299c)

Mike, sorry I wasn't able to pull this as you don't have it in your 0.9-devel branch. Can you merge this into your fork's 0.9-devel branch and send another pull request that includes just this? I want to attribute this one properly to you as well.

--- 
Reply to this email directly or view it on GitHub:
https://github.com/Mike-ECT/cpp-netlib/commit/098299ce9a64ac3d134aa9276d35f97802a98613#commitcomment-967826
No,
2>..\..\..\..\src\server\scripts\Northrend\ChamberOfAspects\RubySanctum\boss_halion.cpp(1073): error C2039: 'Talk' : not member of 'UnitAI'
Oh well, I'd not tried without libevent.
Spoof!


On Mon, Dec 26, 2011 at 1:44 PM, Jille Timmermans <
reply@reply.github.com
> wrote:

> Dit heb ik niet gemaakt :(
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/karpenoktem/kninfra/commit/9d0069645dceda104ba1a40aacd624377d3a41d7#commitcomment-820787
>
Does this cover the case that someone dies with the impaled debuff. The debuff should be removed when the player dies.
其实也不然，Qt 的 container 是 COW   (Copy On Write) 的
In other words you are missing a `master` branch? OK, then this should do it:

```
$ git checkout lfun
$ git rebase --onto f583b1b9f384e3be442c39b35e0c51a9e4815cba github-master
```
We shouldn't be checking in the Gemfile.lock, because this is a gem, not an app: http://yehudakatz.com/2010/12/16/clarifying-the-roles-of-the-gemspec-and-gemfile/
Please pass wikidictID here by reference: `string const & wikidictID`. Also, you could name it here shorter: `dictId`.
Yep. Would surely be faster though. I need to add a system to only update once in a while, or the network thread will get overloaded for big images.
This commit is hard to read, but basically, i created a new def called `args_split` that is passed an `args` string, splits it up, and returns an array in format `[multiple?, chan, args, reason]`.

I then rewrote `fpart_command` to use this new system, which you can see.

Don't pay attention to the diff, its a huge flustercluck, pay attention to the final file. If you use a viewer like Kaleidoscope, with side-by-side diffage, it makes a lot more sense
FYI: We need to fix this so it does not assume a google.com account. No need to do this now, but letting you know that it needs to happen.
its just liek escort outside cave, not much info for it http://www.youtube.com/watch?v=y5AMXINUv3Q
Fair enough! Good luck with things :)
J'ai commenté les qType pour décrire un peu ce qu'ils font.
isn't it possible to put the HandlerList stuff of the events into an abstract class?
Is this call guaranteed to be called on the main thread? I thought it wasn't…
yes, I think your update is not required!
Modifying the doc would be one thing.
Could we provide a default log4j.properties generated by "play capify" or
something like that?


On Fri, Jan 6, 2012 at 5:59 PM, David Y. Ross <
reply@reply.github.com
> wrote:

> Coming into focus now :)
>
> Would you prefer to abandon this change and update documentation or get
> this in for an out-of-the-box solution?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/dyross/play-capistrano/commit/97154f1c9ea0d7d276875b2f0c92fd6ac40ecdc8#commitcomment-844187
>
I think the answer is to create the user here, but leave it unattached to a dropbox account. If someone comes back later they can connect it then. Otherwise it gets a bit strange.
Haha, this seemed surprising to me, but I am fairly certain the source I used said this. Bad source most likely
gcc compile fixed on 1e246cb
I am not aware of a simple way to do this.
Is this something you'd be interested in contributing back to AWS? If so, please send us a pull request. :)
Thank you, now I see it. Epic fail)
Kealima pls look at this https://github.com/TrinityCore/TrinityCore/issues/2254 there is a crash fix fro MapInstanced::CreateBattleground crash
This commit is the cause of a vicious EmuHacker crash exploit: https://github.com/TrinityCore/TrinityCore/issues/4433
Thank you for the bugfix!
Reviewed, split out of TCK and implementation looks good
I fixed that in the following commit and added the case-ignorant check, so
you use the last commit.

But thanks for noting anyway.

2012/1/10 Diep Pham Van <
reply@reply.github.com
>

> I think this must be: Don't show program if NoDisplay attribute is true.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/alexander-yakushev/menubar/commit/015fda485cd2a3fe762cb5ed587764533e5745c1#commitcomment-852492
>
r+wc

pep8 and pyflakes find a bunch of whitespace and unused import errors that need to be fixed.

Also, I don't think we should be committing the product_details stuff. IT can add an import to our push script so that the servers update to the latest data whenever we push. Might be useful to just add product_details_json itself to the .gitignore.

Fix those two issues and make a pull request and we can merge it. You should probably rebase this commit on master as well, so it will show up as a fast forward merge.
I got yahoo working yesterday on my fork. It was tough because of all the double encoded hmacsha1  signatures and additional token requests. The code just needs a little cleaning up and needs a review with how it is itergrating with the config.yml For some reason, phpstorm isn't helping me cleanly update version control might have to do with my git settings... how ever its doing it is fine for me but it makes it tough for mazen if he wants to merge my updates.
2. agree, could be bounded to creating form element, not whole document
This if statement would allow an empty username because a length of 0 characters is still less than 25.
Also it would make sense to *not* have any assignment to private attributes in the `get_renderers` method. There is no indication that other methods rely on `self._renderers` to be set and that this has to happen in `get_renderers`. It's a bit counter intuitive. 

I'd rather have this method *only* return / yield a list of renderers. Another private method can then take care of assigning the renderers to the private attribute.

I've also now seen that further below you rely on index based lookups for eg the default renderer. Yielding won't happen then.

Anyway.

I suggest:

```python
def get_renderers(self):
    return [cls(self) for cls in self.renderers]
    
def _get_renderers(self):
    if not hasattr(self, '_renderers'):
        self._renderers = self.get_renderers()
    return self._renderers
```

And all private methods in the mixin to call `_get_renderers`
The val() function is a jQuery pluggin (as I'm sure you are aware).  What it does for text/password/textfield is simply return element.value, or if called with arguments, it does element.value = the argument.

the valHooks are get/set methods, keyed by tagName (lowercase) that it optionally calls to get or set values of form elements (eg select/option is complex enough to need such a thing).

I'm planning to add a get hook for text / textarea  that return's empty string if the element is in placeholder state.

Please help me test, as IMHO, this is a better approach to this rather then the wack-a-mole route.
In general, this looks pretty good.  The only comment I would make is that we've made it harder to replicate this functionality for other places where we want to create an external form for GridField.

One possible refactoring is to allow for a search context to be passed into a GridField, and for the GridField to provide a `SearchForm` accessor that would provide a Form object with the form action set appropriately.  Of course, a search context might also be an appropriate way of specifying more complex options for the filter controls at the top of the form.  The goal with GridField was to allow for different components that provide similar functionality in different ways, so an external-form-component would fit that model.

However, this refactoring isn't critical, and it might be best to wait until ModelAdmin is being implemented before going down that road.
I think there's a little misunderstanding here because of a tiny mistake @johannilsson did initially. Or maybe I'm the one at fault, but in all honesty, I don't see the problem and the need for the 'InternalBase' thing. Let me try to explain...

Currently, we have this:

https://github.com/johannilsson/android-actionbar/blob/mimic-native-api/actionbar/res/values/themes.xml#L17
https://github.com/johannilsson/android-actionbar/blob/mimic-native-api/actionbar/res/values-v14/themes.xml#L17

I think this is what @JakeWharton ment by the mirroring? If so, this is wrong and this is the mistake I was talking about.

The theme names shouldn't be different like they are now (ie: ```Theme.ActionBar``` and ```Theme.ActionBar.Holo```), they are supposed to be the same: ```Theme.ActionBar```

The **only** difference will be the parent, that's the way it's supposed to work. Which means that the library itself will only have one theme (again, ignoring light) which will inherit from a different parent depending on the Android version running. And that's the idea. Then, like I said above, developers can choose to override the theme and use the same name, or create custom ones with whatever names they want by inhering ```Theme.ActionBar```.

Makes sense?
I will take these out, I agree these ones are over the top unnecessary
after this upodate 

ERROR: Cannot connect to world database

after sql 

https://github.com/NorthStrider/ProjectRisingDarkness/blob/6d74a92b4906b2e1cee98bd9cd5fd9d08b635767/sql/updates/world/2012_03_01_02_world_locales_creature_text.sql
I know -- that's how you've implemented it.  I'm just debating whether it makes more sense to leave the code alone and just recommend this in the documentation...

[Pinto::Add]
repos=/path/a

[Pinto::Add]
repos=/path/b

...if Dist::Zilla even supports that at all.
qwerty.kl is needed to control phone, msm_touchscreen.kl and fw is needed to haptic feedback, so it's better to keep qwerty.kl (and maybe swift_keypad.kl - current kernel need that).
Ok i finally moved everything to eclipse and it all works perfectly
should continue using >= 3.1 since this plugin works fine with rails 3.2
I assume it is, and the code just dates from when there was no final attribute.  But then again, judging by Walter's commit above...?
No bug report I've ever seen.  I only ran into it with my c++ linking experiments.  The compiler always uses a direct call on final functions, so it's strictly an abi problem. (and __traits(getVirtualFunctions) is correct in returning final functions)
This commit causes strange bug.
In example: Given a media file named: _01. Smoke On The Water (Deep Purple Cover).flac_, this code will try to treat it as an album art image! ( _"SkImageDecoder::Factory returned null"_ in logcat).
Moreover, the media file is copied to albumthumbs cache directory...
I suggest a small change to the condition - file extension check:
```java
(...)
if (entry.equals("albumart.jpg")
  || ((entry.contains("cover") || entry.contains("front")) && entry.endsWith(".jpg")) {
(...)
```
Yep, not the most elegant of all solutions, but it works. That would be fine for me.
If we're going to split this into two commands, then I'd say the initial setup one should point to the new application form, while the auth command should point to the main apps list for them to click into the app for which they want to use the key/secret for. Does that make sense to you?
I don't think document.all is a good way to detect OS. navigator.appVersion might be a better choice.
Similar suggestion here. Do any other frameworks implement a class we can extend to get most of this functionality so that we don't have to maintain it?
I would gladly nest whole 'default config' stuff into the object or even dropping completely. In general we should always decide if we need 'read' or 'write' replica (or even 'readbeforewrite'). AFAIK, only "cache handlers" code would need to be changed outside of the core. In the meantime automatic selection of default config in bootstrap sounds great as backward compatibility only.
You're right and I just pulled your pull request on master and it generates an error on my desktop : 

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.12.4:test (default-test) on project cas-server-integration-ehcache: There are test failures.

I'd really like to know if I'm the only one who can't support the 2.12.4 version plugin...
Are these values available from the OpenSAML API anywhere?
This probably wants something in addition to mention that you shouldn't instantiate access objects directly.  Instead use the op2 variables.  Perhaps:

.. warning:: Don't instantiate ``Access`` objects directly, instead use pyop2.READ, pyop2.WRITE, ...

?
I’d rather invert the lines above and below and remove the sleep
A Fatal is a Fatal...
dummy arguments are best named `_`, because then it is completely clear that they are not used
Ok. I'm not using that ctor arg anyway. Will fix.
yes, I have no opinion, in that case, should I name it `smoothing-factor`?
move "this" down
that is a bug, and a actually a very stupid one. It is duplicated further  
down as well. Will fix it.
This should most definitely not be a list, since multiple entries which are equal should be discarded, and since it's only used for lookups it should most naturally be a Set, which usually have a very fast test for inclusion.
Shouldn't this be caught earlier? Make systemGuardian: LocalActorRef
If I understand correctly (which may well not be the case), then it should be possible to have actors created automatically by referencing them in the XML file. If so, please add a test case which demonstrates this.
Is the additional clearState on purpose?
a minor style thing - I think you should extract `publishDone` to a method to make it similar to the other cases
as mentioned before, I think #deny will currently fail.  may be others.
``set(('vs30'))`` creates a set containing letters 'v', 's', '3' and '0', since ``('vs30')`` is equivalent to ``'vs30'``. It should be either ``set(['vs30'])`` or ``set(('vs30', ))`` (note the comma).
This seemed a reasonable change based on first intuition when we discussed it; is there any way we can test to make sure?
I think that lib.helpers should be for template helpers only.  Also this is only used by ckan/lib/email_notifications.py (and tests) so it could live in that file.  If we find we want it used elsewhere we could move it to a shared file then.
Reduced to:

        if kwargs.get('str-eq', None) is None:
            kwargs['str-eq'] = []
forget the ```-d``` part. just found that the daemon option isn't released yet.
I will take it out and we can add and evaluate that separately.
Maybe I refactored them out and didn't notice?  Will check.
Docstring, where art thou docstring?
Can you think of a better name than util.py? Maybe sandbox.py?
`flush()`?

Probably better practice to do:

```python
if isinstance((group, unicode) or isinstance(counter, unicode):
    ...
    stderr = codecs.getwriter('utf-8')(self.stderr)
else:
    stderr = self.stderr
```

(and do something similar in `set_status()`)

I also don't think you need `_wrapped_stream()`; `self._wrapped_stream` and `codecs.getwriter('utf-8')` are almost the same number of characters, and the latter is much quicker to understand.
Just for backwards compatibility with existing UDFs? So that it is easy to move existing code to generated tuples?
also indentation here
what if this line throws an exception?
Yeah, got that :]
Should each of these be wrapped in a try/catch in case one fails?
nodes which -> nodes whose
I think there is actually a function for this somewhere.
The use of format_parser is pretty pointless here, it's equivalent to just writing `a = np.recarray((2,), "i4, i4")` or `a = np.recarray((2,), [("", "i4"), ("", "i4")])`... doesn't really matter much though.
Just tuple, the 2 ints is explained in the description.
This object will be serialized: wouldn't vsv id be enough?
лучше заменить на один большой список append'ов друг к другу
Wait, are we no longer storing the cache?
I think that adding an `else` condition here could make it clearer, wdyt?
Same comment of https://github.com/rails/rails/pull/6282/files#r1780034
Ok. Ping me when you are done.
Do we really need to use `.to_s`?
Why this change? I think we don't need `protected`, `private` is fine
What about line breaks, or should i format the text to a fixed width?  Also, does it look ok to type everything flushed left?
You meant SI-6640 here, right?
This change LGTM, it's overridden where necessary.
Done removed keepDocComments.
I think it'd be better if base_url were a parameter to the function, like

def get_voevent(self, as_dict=True, base_url="http://www.lmsal.com/hek/her?"):
```py
args = list(self.args)
```

That creates a new list doesn't it.
spaces added here
I am a bit worried that some outputs switches from the public api to the internal array based api. Is there a general rule (documented somewhere) about which should do what or are the changes just happening as we go and find the need?
I wrote a [blog post](http://asmeurersympy.wordpress.com/2009/07/20/modifying-a-list-while-looping-through-it-in-python/) about this once.
strlen is unsafety here.
use strnlen(3)
Can I change to add a check for $ options at CJuiAutoComplete.php and CJuiDatePicker.php?
Because there are no preview and new icon in jquerymobile. But I will remove the comments.
I think you don't need these anymore.
Good thing we're returning a boolean here.
Typo: /faled/failed/
As part of the de-hibernatization, the Endpoint type has been deprecated - please use the EndpointDef instead.
These redirections don't make sense. One does not "see a text to understand." 

There are patterns that we use throughout the documentation to signpost these cross-references rhetorically, that are both simple and light weight. Rewrite these as needed.
actually, isn't distances redundant here?
document_form.parent_topic is a form field and we're using old playdoh/funfactory code before the monkeypatch, IIRC. so that's why it was there at first.

in this case, parent_slug isn't user-submitted and it isn't html, so we can drop the | safe from it anyway.
Nit: This commented-out code can get nixed too
Extra space: `display_name` should be under `'`
The avatar manager will throw an error.
In that case this point is withdrawn.
I think this is more useful if the text also links to the most recently unread post, rather than just displaying the title.
Why don't you do something like if 1=1 then 'foo' else 'bar'? Should break a lot less.
K, fixed (will appear in the next push)
I don't think we shouldn't even allow absolute paths - we should just allow file names.
we could move the timeout logic for all the checks to this higher level function
in ele-agent-packages the sig is sha256
good idea. fixed in 2c858cc
不赞成过早的涉及后面的功能，我们尽量保持 PL 最小化。
CppUnit::TestFixture is inherited from TestCase and provides some conveniences (can't recall the tight now).

http://cppunit.sourceforge.net/doc/lastest/class_test_fixture.html
The sentence ending here is a bit long and doesn't read well.
See above.  I no longer convert, and now do getCanonicalName();
Inconsistent with others in file.
Should probably be in utils. Also, needs a comment.
avoid committing commented code
nit: should be 'RaxaEmr.Screener' to be consistent w existing registration app
the thing you are passing here is not a query, but a Q object
Bug filed: https://bugs.launchpad.net/openquake/+bug/979916
I thought this might cause confusion with the enum WEBVTT_SUCCESS
Likewise, the top level directory in the zip file should have the same name (without the .zip suffix) as the zip file, right?
This offset is because of the toolbar at the top right?
Have you tested this change on Android? I think it used to be like this and it was possible to get moved events without a pressed event of the same ID. This occurred when moved events occurred in the same frame as their pressed event. The `&&  _events[j].State == loc.State` clause was added to ensure only the multiple moved events per frame were filtered out.
Hah. That's exactly the behavior of `list.index()`, thus it works if we simply remove the assert.
Use ``urlparse`` to pull out the scheme, might be overkill but should ensure stuff is done right if there are any corner cases  split around ``:`` will not cover.
find and replace error?  'doc.documentElement'
Can we use the source tarball from http://packages.debian.org/sid/debianutils (http://ftp.de.debian.org/debian/pool/main/d/debianutils/debianutils_4.3.tar.gz) so that we don't need to bootstrap the build with autotools?
With a correct ACL implementation, the ACL plugin will intercept the requests before they happen and handle the correct situation. In that case you don't even have to change the current implementation, as the exception never gets triggered.
J'ai remplacé ligne de commande par console, mentionnant "shell" sur la première utilisation.
Согласен, речь шла именно о разработчиках git, хотя описанный подход популярен среди гораздо более широкой аудитории.
Заменил весь абзац на предложенный ранее вариант, сузив круг разработчиков до создателей git.
Prawn's docs seem to indicate it already defaults to the Letter page size (the global standard is A4 but that's a different matter entirely :)). Is there a particular reason you specify them again as defaults here?
Why not just call `initial-start` `start-watchers`?
what the crazy replacement? :D
Yes, 4 times, I know that.
But in official whatsapp clients, if you send a message to someone who blocked you, you will never get the "delivered" notification
xbuild reported that it was an unavailable task.
Ok, no wonder you're confused, the test involving https://github.com/colinbdclark/universal/blob/d38c51a60d42282c6a45a4a4d3ea6d2e0558d538/gpii/node_modules/transformer/test/js/TransformerTests.js#L164-L189 is testing exactly what I'm mentioning above! 

This is a special case though, where we're top level, meaning line 32-35 (of transformer) are involved in making things work. If this block { applications: { ... }, foo: { applications: {...} } takes place anywhere else in the settings, it'll break.

Wrote an test illustrating the issue: https://github.com/colinbdclark/universal/pull/5
With the simplified `getTestClass`, I can get some (not yet all) tests to pass only by making more extensive changes to at least `ParentRunner`, `JUnit38ClassRunner`, and `JUnit4ClassRunner`. While these changes might be independently desirable (they retain the `Class` object), they also introduce their own compatibility issues: for example, if a subclass of `JUnit4ClassRunner` overrides `getName` for some reason, what do we do? Really the APIs should have been designed from the start to retain original `Class`es—retrofitting this is tricky.

Since such a set of changes would be much more controversial, I would rather keep it out of this pull request. If you want it to happen, I can file a follow-up pull request where the details can be hashed out.
The variable naming is a bit odd IMHO. Could you please rename them?
While freestanding @ irks me a little bit (not as much as trailing ::), the word `this` in CoffeeScript I find even more foreign. So in this MOST IMPORTANT battle of vernacular, I prefer:

```coffeescript
@foo
f.apply(@, arguments)
Klass::method
@::method
doSomethingWith Klass.prototype
```
Being verbose wouldn't hurt here, especially given the number of params.
It appears that this block isn't needed currently. Menus and CodeHintsManager both insert the popup into the DOM before calling addPopUp. Originally, I thought this would be helpful in the future for simple popups that would be appended to the end of the DOM and be displayed immediately. The check here though is probably bad. Would you keep this in if we change it to check for ``$popUp.parent()``?
@giannis for normal copy keep the normal way to copy the selection
I like this functionality. Though I think it should have a more direct name such as `djautoskip` or something and move it into mod.BotDJ. I like the `noska` name, but I feel clever names should be left for independent plugins ;)
Is it in hash form for faster lookups?
History goes into commit messages not source code.
this could use a comma after Sorry
Again, this should be `block js-inline`.
Separate out date/time parsing. Probably should share code with the other date/time parsing you did above
So? Why are you asking me then this question if it is documentation?
Alright cool if this is just being done so it doesn't block the main thread then +1. After changing it to 0 this can go to SR
The problem with moving the two lines here is that if they're in closeDialog, when the dialog closes itself via the escape key, closeDialog is never called and the listeners won't be removed
Attempted to use npm module `url` here, but was a bit wonky. File new ticket for follow up if desired, please.
As Marc points out, this probably doesn't warrant `SingleObjectTemplateResponseMixin`. I'm not sure what it should extend though; possibly just `object` with a note that it has to be used with some sort of form view.
现在都用80端口了，这里的 :10100 port 可以去掉了
This is cool.  It would be nice if this was not feature-specific and was a more generic ``config.s3_access_enabled``` option that defines what links with external are used.  But I can imagine that might be a lot more work to implement, and this is the first time it's been used.  I'm content for it to be feature-specific for now.  It's good to see this pattern used.
`README.md` and relatives (bash completion and man page) are auto-generated by `make`
Whoops! Thank you for fixing this!
Tu m'as enlevé l'alignement.
I'm confused by this. It looks like you're targeting the #tandem_page_links inside the body tab inside the #tandem_page_links. Is that right?
what about doing the check when adding the template into a changeset?
I thought that this one was useful.
So you're re-using the existing repos from the old version? Good thinking.
this call is useless as the context will always be an array and you can iterate over an empty array
How about making the `Lockscreen.update` itself to handle the timing of calling the callback by using `MozAfterPaint` event?
nit: please add spaces after all colons
nit: the rule is to keep them on the same line
nit: `for ( ... ) {`
30 would be too large for the algorithm from JSZhuyin unless you'd made some modification to it.

It is a O(2^N) algorithm. 2^30 = 1073741824 !!