labofsahil/github-event-dataset-2012 huggingface dataset:
Patch 1

Hi

I have changed some of the code in the controllers/auth.php. 
Specifically, I fixed the create_user() function and made 2 changes.

1. There was a weird conditional check in the start of create_user() method that was going like this: 

if (!$this->ion_auth->logged_in() || !$this->ion_auth->is_admin())
{
redirect('auth', 'refresh');
}

Which actually says: if user is not logged in OR is not admin redirect him to auth (auth/index).

No need for this kind of check. create_user() runs every time a guest wants to register himself into the system, so it is normal that he will NOT be logged in and will NOT be an admin either. For this reason, this check is useless and breaks the program flow so I removed it.

2. After the user is successfully registered into the system, the controller redirects him to auth (auth/index).

//check to see if we are creating the user
//redirect them back to the admin page
$this->session->set_flashdata('message', $this->ion_auth->messages());
redirect("auth", 'refresh');

If the user is redirect to auth, he will be redirected to auth/login which is the correct place we want him to go BUT there will be no informational message about his registration, e.g. success message. 

Solution: To fix this I changed the redirect to auth/login. 
 
Please let me know if I'm missing something. 
This is my first commit on github and in the open source community. 
Thanks again for a nice script.

Best Regards,
Thanasis Emmanouilidis
SOS-838-795

Language keys
Changing the layout and css to look more like the official cakephp site ...

...:)
add a Descriptor for the FilePath to nuke warning in the logs.

Follow-up to https://github.com/jenkinsci/gerrit-trigger-plugin/pull/13

That one caught me after upgrading to 2.5.0.

`ack-grep DataBoundConstructor` showed that this was the last class with that annotation that didn't implement `Describable` so I guess we're done now.

Ran `mvn clean verify` to proof-check my changes, haven't tried in an actual Jenkins instance.
Test Fixes

 - portal/controlpanel/bookmarks
 - portal/controlpanel/calendar
adminURL may not be available, prevent crash

If there's no adminURL endpoint in the serviceCatalog (ie. in a objtect-store,
in Rackspace identity service there's onlu publicURL and internalURL), unconditionally
accessing the first element of management_url property of AccesInfo may raise:

  Authorization Failed: 'NoneType' object has no attribute '__getitem__'

This change verifies that management_url is a tuple, otherwise management_url will be None.
Fix validator for rerenderprob

Make the rerenderprob value print a decent error since it does have to be between 0.0 and 1.0 instead of just that it is a float.
Release 12 h1 logging change for port

Allow customer to specify not only the server, but the port for syslog.
Updated es_AR (major) and en_US (minor)
css bug

make Section "group" and have fun

fix it, but you must check all such places

admin.css
can load perl configuration

you can load xxx.perl or xxx.pl configuration file.
Certification + compilation
GitHubified the readme

Made a new readme with markdown markup to render nicely on GitHub.
fixed Module implements BootstrapListenerInterface
* Allow ident:: to mean ident.prototype.

This allows for:

    (obj::)[b]

  Or, more importantly:

    ClassName = -> ...
    ClassName:: = { ... }

  Which is frequently seen in js2coffee's output.
Merge the unstable changes
merge changes from jm23
Wrong detection of request method causes that deleting an object does not work

`CRUDController:deleteAction` is checking that HTTP method is `DELETE` but instead `POST` is given, so an object never gets deleted.

The pull request fix this checking for `POST` instead than for `DELETE`, but surely this should be a temporary workaround because I think that the `getMethod()`  method should be returning `DELETE`. The form has the hidden `_method` element set to `DELETE` so I don't know where it is the problem. Maybe is it a Symfony bug?
Version check bug fixes

- Fixed a bug in the version check, that occured if you have updated zf2 but still had a ZDT_ZF_Version.cache file.
- Added an option to disable the version check.
- Improved the robustness of the version check (better cache handling for people without internet).
- Updated config dist.

**Checking the version will still fail (once an hour) with warinigs without internet!**
Added ExpectNotSend and ExpectNotSendLocal to test DSL

Hi there,

we hit a problem when we tried to validate that a message was not sent using the test DSL.

The DSL has a ExpectNotPublish but there is no corresponding ExpectNotSend. This commit adds those.

It's already been mentioned in a previous pull https://github.com/NServiceBus/NServiceBus/pull/30

Cheers,
Alex
Paypal: Validate generated XML against paypal xsd's

Paypal isn't always very exacting about the order of it's elements, but some features won't work unless the order is correct.  This change verifies the xml generated is valid against the paypal's xsd's.   I'm not sure where to put the xsd's in the file structure, let me know if there is another preferred place for them.
One addition and one fix.

- add TP-Link TL-MR3420 support to button handler patch
- remove patch to enable inotify kernel options because it's already added to the backfire branch (r30028)
LPS-25272 Web Content Display portlet should be hidden for users without...

... showSelectArticleIcon permission
Sage URL typo in index.rst

Fixed and committed. Please pull.

Thanks.
onBegin callback

Added onBegin callback in rangeinput
Add "aliases" option to Fabricator definition

Allows lazy declaration of class aliases, e.g.

    Fabricator(:user, aliases: [:reader, :blogger]) do
      name "Dave"
    end

    Fabricator(:blog_post) do
      blogger!
      text
    end
fix vc2010 build
Immutable request

When I did the dotnet45 branch I had to rework a lot of the http handling code. One of the things that I did was to make the Response class immutable after construction, since this data comes back from the a remote server and it doesn't make sense to change it afterwards.

I did not push this change up to master. However the syntactic differences in tests that dummy up responses (new Response(...) vs. New Response {... }) have caused a lot of merge conflicts recently. So here is the pull request to make Response immutable and get them back into synch.
Lithuanian intial relese
add air assets to generation script

good luck!
Fix for Nodejs-0.6.12

I had to change this http.get call in order for it not to throw

```
Caught exception in Main process: Error: connect ECONNREFUSED
Error: connect ECONNREFUSED
    at errnoException (net.js:646:11)
    at Object.afterConnect [as oncomplete] (net.js:637:18)

```

Nodejs version is 0.6.12 installed from Ubuntu 12.04 package repository.

```
$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=12.04
DISTRIB_CODENAME=precise
DISTRIB_DESCRIPTION="Ubuntu 12.04.1 LTS"
```

```
$ node --version
v0.6.12
```
CAPEDWARF-23 Fill Environment attributes
added unicode to url's
fix quotes on category
fixes for find_callback() + specs

Hi,

I forked your branch, rather than the original, as I too needed a version compatible with the latest rspec. Then I found some cases where find_callback() returns callbacks when it shouldn't (if they refer to tasks not loaded in the configuration being tested) so I've added a fix for that, together with some specs.

Ben
Adicionado busca na lista de Journals Ticket #19

Adicionado busca na lista de Journals Ticket #19
We can see multiple users in the team activity feed, finally it has a purpose :)
SECOAUTH-311: AbstractJaxbMessageConverter creates new Marshaller and Unmarshaller each read/write

This is a fix for SECOAUTH-311. The second commit also contains some javadoc for AbstractJaxbMessageConverter.
BVAL-307 CDI Integration

* Created new Integration chapter
* Adding Java EE and CDI sections
* Moving the JSF and JPA appendices into the Integration chapter
* Removing obsolete method validation appendix
radio button support
Add the subscription ID in aTopic for every SIM
RHEL6.3 support
[NEXUS-5341] Decode HTML entitites in status messages

Depends on sonatype/nexus#581
fixed text formatting bugs
Silverlight Client Enhancements

1. Fixedup ServiceClientBase to merge sl and non-sl sections with inline conditional statements.  This is better than splitting the whole class, as it keeps future changes in sync.

2. Removed synchronous non-implemented method calls.  Silverlight cannot make synchronous requests, so the methods should not exist, rather than throwing NotImplementedExceptions.  For silverlight, removed the IRestClient interface and took IReplyClient away from IServiceClient.  This leaves just IServiceClientAsync and IOneWayClient as defined in IServiceClient.

3. Added three boolean options to ServiceClientBase only for silverlight:

 + _HandleCallbackOnUIThread - defaults to true._  This is the desired behavior in most cases, but there could be some cases where the client specifically wants a callback on the background thread and then handles their own marshalling to the UI thread.  In that case, set this to false.

 + _UseBrowserHttpHandling - defaults to false._  The previous client implicitly used browser handling, which is non-optimal for service calls.  Now we specifically use client handling, unless asked otherwise by setting this flag to false.

 + _ShareCookiesWithBrowser - defaults to true._  Because we switched to client http handling, we now need to handle cookies ourself.  This flag sets the default behavior to share cookies witht the browser via Silverlight's System.Windows.Browser.HtmlPage.Document.Cookies interface.  Alternatively, you can set this flag to false and then provide your own CookieContainer.  Note, if StoreCookies is set false, or UseBrowserHttpHandling is set true, then this behavior is always disabled.
if clause has been removed at line 73. (when application has no request ...

...filter causes null exception)
Fluid 4545, 4549, 4550: A few improvements for the video player

1. Replaced decorators with sub-components;
2. Removed the unnecessary event binder component or re-place the event binder component to be a sub-component of the binder component in order to reduce the number of components to be dealt with at the top level;
3. Got rid of the un-needed demands blocks.
Tweaks to the HTML head section

Minor amends to the HTML head section of your theme.
Added the forcedefault option

I added the forcedefault or f option. This allows you to force a gravatar to any of the available defaults even if the user has a defined gravatar image--it will be ingored.

I don't really know what I'm doing, and I've only been doing Ruby for 3 days... but the commit is fairly small, and it works for me.
launcher base icons
Image output fixes

Update stock during run.
Don't output initial blank image (although user can still change before first image ready to see it).
ICS style Pattern
do not validate 'decorative' inputs

Hi I faced with problems when my form has some decorative inputs (without names).

When validate, your script implies that input should have name.

So I write patch, that filters such inputs from validation (and also write test).

Don't know in which branch send this patch, so I send in master, I can resend in 3-2-stable, if needed.
I created an ec plugin for the ec sensor in the link below.  Could you add this into your project?

http://atlas-scientific.com/product_pages/kits/ec-kit.html
OU-4129 Fix for problem with sci notation with 1 sig fig

Fix for http://ltsredmine.open.ac.uk/issues/4129

Difference in behavious of variable numeric and variable numeric sets when non-numeric entered.

With unit tests.
Add firmware version IML74K-ZSLPG for galaxy-s2

Add a firmware version IML74K-ZSLPG in extract-files.sh to support galaxy-s2
proactively set the hb interval

to be a few seconds early to account for stack latency.
New check AvoidDefaultSerializableInInnerClasses was created.
repair damage from pep8 commit

2aff81c1903657529ef551c014e569c57f97c3b1 dropped the entire "def postRequests(...)" line.  This restores it, and corrects the use of an unsafe mutable default argument.

I've actually posted to alpha.app.net with this version, though I also have a bunch of s/r.text/r.content/ because the version of requests in Ubuntu Precise is old.
fix routing error when pagination exists

Closes #21
[pt_BR][Book][Validation] Fix typo. First review

[pt_BR][Book][Validation] Fix typo. First review.
More improvement for lambda inference

Fix bug:
<a href="http://d.puremagic.com/issues/show_bug.cgi?id=7193">Issue 7193</a> - Regression(2.058head): ICE: delete lambda expression crashes dmd

Improvements:
- Improve inference behavior on arguments of normal/template function.
- Add escape check to nested function symbol.
A CouchDB, Neo4j, Redis, and Node.JS Circus
In the :javascript filter in Ruby (HAML) code, the bang equals at the start of the line is not needed

Since we're primarily showing javascript code with Ruby interpolated, the equals sign is inappropriate. :)
Update to your build files to build scalamodules for 2.9.0 and 2.9.0-1

I ran all of the tests and they seem to work, i based the changes off what you did for slf4s.
new css, formatting to contrib.js records, reformatting of README.md to display lists, 

Primarily a lot of styling and formatting
- webfonts
- reduced max body width
- cleaned up all-caps entries in contrib.js
- styles to map markers
- tooltip/table
- block quotes now display correctly
- added about "occupy data" section
Max and Ethan added Huff Po UIPicker selection ability

added picker view that appears when user clicks button on an action sheet, and a close button for the action sheet
Attempt to fix endpoints and nova.  Add nova-volume. [1/8]

Nova still has issues, but it is better.
Dashboard talks to keystone (try alot).
Dashboard talks to glance.
Dashboard talks to nova not so much.o


 chef/cookbooks/nova/recipes/api.rb                 |   32 +++--
 chef/cookbooks/nova/recipes/volume.rb              |   93 +++++++++++--
 .../nova/templates/default/api-paste.ini.erb       |  148 +++++++++++--------
 .../cookbooks/nova/templates/default/nova.conf.erb |   16 ++-
 chef/data_bags/crowbar/bc-template-nova.json       |    6 +
 chef/data_bags/crowbar/bc-template-nova.schema     |   10 ++
 chef/roles/nova-multi-controller.rb                |    1 +
 crowbar.yml                                        |    3 +
 8 files changed, 219 insertions(+), 90 deletions(-)
PYTHON-317  Fixed typo in docs
Remove the tab characters

There were two files which had tab characters (\t) instead of spaces.  I changed them to spaces and made sure they were indented properly.
Jellybean

For testing only.

You'll need
http://review.cyanogenmod.com/#/c/19186/
http://review.cyanogenmod.com/#/c/19187/
http://review.cyanogenmod.com/#/c/19188/
for the framework stuff as well.

Library has been rewritten to be independent of AutoParts.
Merge master in ngale
Avatar

Slowed players slightly, Increased rocket speed, reduced grenade lifetime, increased grenade velocity.  Fixed spawning issue for late joiners.  Increased player jump height.  Added Kill Streaks and required cout functionality.  Added Rifle.
Increased player gravity. Other changes and fixes..
Initial entry of the device API docs to Git
Clarifications on how to build application

This sample contains a minimal set of assets required to build the application. This update explains why and provided detailed instructions.
Solarsystem

One more :-)
Added check that number of fastq reads matches iRODS metadata.

Added check that number of reads in imported fastq files matches the number of reads in iRODS metadata for Import_RODS_Fastq pipeline.
Update plupload to 1.5.2

Plupload updated to 1.5.2
Test commit + howToUse.html tests included
Making team twitter links open in a new window. Updating the dr4goonis a...

...vatar image.

We don't want them to leave our website for external links.
Updating sponsor section.
--sdk-version command line argument should accept a parameter

--sdk-version was interpreted just as a flag argument and any parameter didn't have any effect. I guess this is how it's supposed to be.
Various fixes and updates

* Usual plist changes: version number, display name, documetnation
* 64 bit-ify
* Be more graceful to the user for what they type. E.g. allow "500px x 50px" or "500x50", or for file extensions allow ".jpg" or "jpg"
* Show a helpful example in the 3rd pane for the scale to... action
* Fix a load of warnings
Minor changes to compatibility actionbar

- changed portrait version of compatibility action bar so that it sits
in a horizontal scroll view - this looked really weird when there were
too many tabs especially that all the auto scrolling text was moving
that the same time - requires more that 4 tabs to test
- changed action bar view so that when a tab is selected (without user
input) it scrolls into view if it was not visible - this matches
behaviour in ICS (not sure about HC)
Backport android and MSM parts from caf

This brings the android drivers from caf and cyanogenmod 3.x kernels, tested on device.

Also includes some fixes in msm code along with some code style from codeaurora for easier porting of leo board files to later kernels from mainline.
LPS-24958 Fix source formating
Updated icon, addon and changelog

How do you stop a commit ? Don't need the 2 for the lang files
I believe this to be a more accurate representation of battery level bas...

...ed on some math I did and reference in http://www.madteam.co/forum/development/cyanogenmod-vs-froyo-battery-drains/msg45597/#msg45597
New set of derby & mssql fixes

Here's my updated set of derby & mssql fixes, which should also now work in Rails 3.

All the tests pass except for the "test_count_with_limit" test, which was broken before and is still broken (and I don't know a good way to generate the SQL needed to make it work).

Cheers,
- Dwayne
Merge improvements to django-multisite

Improvements have been made to django-multisite:

* New template loader
* Fixes to broken unit tests
Test Fixes - 02/08/2012

Test Fixes - socialofficehome/activities
Added output transform selection and registration CLI call

...ration module
play sound code only once

Now is play sound code use twice.
Need to be written only once
a minor but useful change

added rvm 'use' command to .rvmrc so gemset in use will show when directory is entered
loosen activeadmin version requirement

Didn't find any compatibility issues with Active Admin 0.4.2
LocationDiagnostics - From Shadid Haque

Sample that demonstrates the C++ Location APIs of BlackBerry 10; it can be used as a reference usage of the Location API from a Cascades application.
Return the XHR when sending a request
Using App-ID for whole App

I added three new methods with rspec testing:

RestfulMetrics::Client.set_app_id(your-api-key) 
add_simple_metric(name, value, distinct_id = nil)
add_simple_compound_metric(name, values, distinct_id = nil) 

Environment-Handling: (to send also in staging)

from: "only sending in production" -> to: "not sending in development"
Resource's get_bound_form() populates instance's form on GET

Simplified the argument's logic when ```.get_bound_form()``` returns the new ```Form``` by building a ```**form_kwargs``` dict, as ```Form``` accepts ```file``` or ```data``` arguments set to ```None``` by default: https://code.djangoproject.com/browser/django/trunk/django/forms/forms.py#L75

Declare model's instance in the ```Form``` if present, and not only if ``data``` or ```file``` are declared, then the form is populated with the instance's data on GET, much more convenient when editing, IMHO.
Fix DynamoDBReflector to treat empty strings like null strings

Dynamo doesn't like empty strings, so the reflector should set empty strings to null, so that they get handled the same way as null strings, rather than resulting in a error.
rowend/10-admin-localization

translate admin localisation folder
This will fix the following issue

the two lines will fix a strange issue (found the error in nginx logs): 

``` upstream sent too big header while reading response header from upstream ```
Toggles

Hi Pete,

I've finally added the toggles the cleanest possible i could, please review it.

Hope it helps

Cheers
Updated Mms/Files/res/raw/sms_frequently_used_phrase

Updated sms answers because it's russian translation
Equiplet node

The equipletnode contains a equipletNode that is used to start new nodes for hardware modules and change their state. There is a StateMachine super class that should be implemented by every hardware module. As an example of how to do this there is a refactored DeltaRobotNode that is a class now and implements a stateMachine.
New table 48_genome_ref.sql

This should match the spec you gave me:

        * id - a simple auto-increment identifier, primary key
	* uuid - used outside of the database to identify each reference genome
	* path - the path to the directory containing the reference genome files
	* deleted - true if the reference genome has been marked as deleted
	* created_by - the ID of the user who created the entry - FK to the users table, can be null
	* created_on - the creation date for the entry.
	* last_modified_by - the ID of the user who most recently modified the entry - FK to the users table, can be null
	* last_modified_on - the most recent modification date for the entry.
quick addition that reminds beginners to add the spark version
移植到Visual Studio 2008 Express

Signed-off-by: Vdragon <pika1021@gmail.com>
Moved editor sources for build script compatibility
Modifications sur l'épisode 1

Salut,

J'ai fait un peu le tour et essayé de corriger les fautes de frappe, d'orthographes et parfois modifié un peu le sens.

En tout cas merci pour la traduction.
Can't be null with our last commits, and isEmpty() is better than size() > 0
No need to quote numeric constants. (Not that it matters.)
Explicitly document this attribute as being type `{HTML}` since its value is passed to `innerHTML` without being escaped.
Yeah, that's better, I'll switch it.
s/adapter/reader/
OMG GitHub has turned into Reddit/2+2...
> Idea: What about moving the admin code to `admin_.py` (or something, not too happy about this name) and only leave the `admin.site.register` call in `admin.py`? This would still allow us to separate the model and admin code.

  Well it wouldn't really change much since the admin.py would then need to include admin_.py. Okay, so what if we rename admin.py -> admin_late.py, add an empty admin.py and put an "from admin_late.py import *" at the end of models.py? Still not totally clean but at least we can get rid of those monster files.
riešil by som to rovnako. Prepisovať commit spred pár týždňov, to je svinstvo. Prepísať zlý commit, kým je ešte teplý, to je využitie možností gitu.

Commity si hádam kontrolujeme všetci, všetkým chybám sa ale vyhnúť nedá...
Heads up: the `email_topic_subscribers` appears to have hard tabs indentation. Though it doesn't seem to show in this commit, but it does if you look at https://github.com/radar/forem/blob/471d1585bd/app/models/forem/post.rb.
@trevor you're welcome :)
I think the other way was right. At least that's how I always do it.
Logic like this really belongs in the controller. Just assign ` @open_mic.posts.order(:updated_at).reverse_order.limit(5)` to `@posts` in the controller.
Cool, just please document these parameters, especially the data types/formats.    IIUC, something like:

```javascript
function _getCtor(/*String*/ type){
   // summary:
   //        Returns (possibly cached) constructor for class defined by type,
   //        which is a comma separated list of class names, ex: "dijit/formButton,dijit/Mixin1"
```

and then for function extend(types){

```javascript
function extend(/*String[]*/ types){
   // summary:
   //     Construct class based on specified types (first one is main class, followed by mixins).
```
In VLC, when I got into playing an optical disk, the value preset is /dev/rdisk2
not "disk2"

So they may be doing the munging, but you can certainly add the real device name without having to worry about what crap myth will prepend 

I'm not sure VLC 1.1.9 is the best example on what to do either :) how old is that stuff?
elseif ?
@jonnii when you get a chance, can you post the routes you have, along with the URL you're hitting?  Thanks.
This should also count how often the event is called, so you can calculate the time for each call.
classe !
Ok, I didn't understand, that's all. I thought comments starting with @name
were usually directed to the attention of that person.  "Done by @jaberg"
would have been clearer to me.

On Tue, Jan 17, 2012 at 3:18 PM, nouiz <
reply@reply.github.com
> wrote:

> Yes they are. Those comments was for me to mark witch comment you ansered.
> As there is many comment, I just marked those you did so I know witch one
> you didn't finish.
>
> So by "@jaberg done it" I mean you did it. Did my small sentence had a
> problem? was telling something else?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/jaberg/Theano/commit/a019900862d2e76ccb4e868634087e325fe594e0#commitcomment-875699
>
Credit to @jswarren for this functionality; his commits got lost in the merge process.
When you use the term "master" here, do you mean the matrix parent, or do you mean the Jenkins master?
What's wrong with `deploy:finalize_update` ?
What kind of idea was clicked?
why not use ``super()`` ? it makes refactoring easier and handles multiple inheritance properly
putz... é mesmo... :P
That LoS check which was used in r2 for dalaran and orgri arenas wasnt working (i got such report) so I implemented this check, which does work ;P

Afaik GO LoS check for all GOs is very cpu consuming so I think that this check must be implemented anyway if somebody wants to use the script.

But if los check will be working good then no problem ;D
тем, что в корку падает. под линуксями всегда падал (у меня так даже ядро пару раз клал, было дело), под виндой недавно начал. обновить бы его по хорошему, да некому этим заняться, да и не факт что поможет. по форумам девским если пошарить, пишут что крайние версии еще сильнее падать начали... наверное в винде что-то поменялось.
I'm going to go through the codebase and convert all sentences separated by one space to be separated by two spaces. I know @mokolabs especially likes that!
Don't worry, we'll try to integrate it as soon as we have a good solution for cryptokit. How did you install it?
To be consistent with the test just above, maybe add:
assert isinstance(Jx, list)
修改了。。  没IE悲剧啊， 没得测...
Out of curiosity, what made this necessary? I checked the zipped size of the xfl vs xfl sans authoring tool specific information ie this. The xfl was only about 10% bigger. Admittedly this is much smaller in a non-compressed form, but does that really matter?
as for large trees, generate a height then generate 2-4 points around the tree, create nodes of leaves at these points, then create a line of wood from the tree trunk to these points, generate a node at the top of the tree (like normal tree) and generate the trunk.
cccc

There is more of the insanity below too. :sob:
didn't we have another ticket on this already?
It seems weird to change the definition of `ParseError` from being something that mimicked a `V8::Error` to now be something that just holds a reference to one. Shouldn't there be other changes elsewhere in the code to deal with this switch up?
It looks good. I will verify that it is actually working once I can merge this - unfortunately, it depends upon #71, which breaks Anathema at this moment.
I've taught you well!
#ddd is the border color all over the main content in marketplace- we should define it and rescue all the rules from header.less
looks good but where's the tests for the `with self.activate(app='mobile')` ?
is this what we want to call it?
What if we assigned the callable's return value to the Response body? This avoids the long-term need for output buffering (not that ob is a problem), and may also solve what you are trying to do. Just tossing around some ideas to hash this out.
Is there any particular browser that supports features required by reveal.js (transitions, etc) and doesn't implement `classList`?
Good call, yes, it is expensive. I didn't see a better place to put it and could use help optimizing this.

If we re-organize when EJS gets instantiated, gettext should move with it.

I don't think this is a blocker for landing, and we can investigate during this release:
Filed https://github.com/mozilla/browserid/issues/903
oh sorry - i didn't mean to tread on your work - I was just scratching an itch!
Credo di sì, però così va bene lo stesso :D
Please can you commit file move and other thing separatly. 
By sample, the removal of -f is important but the move of bumblebee-indicator is something different and uneeded according to me : my first goal is to separate those two apps and make them as independant as possible. 
Maybe we should create a separate directory for each one.
Thank you for confirming i am not imagining things. This is exactly what has happened to my heroku apps when I db:push after updating to Ruby 1.9.3. I however am able to db:pull. Going back to 1.9.2 for now.
why don't use finally instead of catch
I do not understand this. This just saves the itemid into the set. But don't a set disallow same values? So you have one item in the set or none?
Probably you wanted to save the slots here?
Please add parens to make the order of operations clear. Also, please use `dup` to copy arrays, it's more obvious than Python-style slices.

```ruby
@d = (no_copy and digits.length == size) ? digits : digits.dup
```
The 'shutdown' case was originally intended for WEBrick, but since WEBrick also supports 'stop', I removed the case.
It bothers me a bit that a constructor would start a new process. 
You'll tell me that it's actually done by the command exec() on the Builder, but package local isn't much of a protection. 

I would be safer if either:
 - this class and its builder were put together into one file, so you could have a private constructor. The builder could call start()
 - the constructor was made private and callable only from a static method that would build the RunningProcess and start it

In both case, you prevent code even in this package from creating new instances or subclassing, also it's clearer that something happens.
Coding style: Star should hug the variable name
@nevertattoo That's more or less correct, see your issue comments for more info.
Foi apenas um ctrl + shift + f, se isso nao for aplicado um dia, nunca poderemos usar o ctrl+shift+f...
I will not discuss this, I moved to .rspec and ignored the file.
I don't think it's sab issue. I guess it is subprocess.Popen issue.
I revert back to old default.py(without subprocess, with os.system()) everything working. Then I just download maraschino and XBMC freeze it again.

Here is my xbmc.log: http://pastebin.com/raw.php?i=zFpT2Kti
I can't see any clue here.
Проверих "документацията" на Draper и си права, че с `decorates_association` би трябвало да стане. И да, ще ти трябва още един декоратор — за коментарите, но не мисля, че това е проблем. Още, понеже май имаш `PhotoDecorator`, може и в него да му кажеш `decorates_association :comments` :)

За второто — по-скоро да, наистина. Ако имаш декоратор `UserDecorator` и подаваш `@user` към view-то, е добре да му го подаваш декорирано, защото ти, като контролер, не знаеш и не трябва да знаеш дали view-то ще се възползва или няма да се възползва от декорираните методи. Абе, с две думи — права си... :)
build seems to be failing because of this http://ci.angularjs.org/job/angular.js-igor/391/console
貌似是你以前改的忘了关了……捂脸
No need to , i'm fixing this , instead i'll add MODULE_VERSION , i took this from nobodyAtall xperiafeature , we should give him thanks
Yes, once they're ready.
Junge schreib mal bitte bessere commit messages. danke.
@stream = Stream.new
Not your fault, but this is the third implementation of this function.  Probably time we merged them and moved it into the stonith library.
Can you do that in a followup patch please?
Add multi-valued support here also
Looks nice to me.
Both with x and \* for that matter.
"stament"? ;-)
「ネストした匿名クラス」ではなくて、「匿名関数」ですね。for 式が foreach/flatMap/map を呼び出す糖衣構文で、map の argument として匿名関数が自動的に生成されて渡されるので、return 式のようなものを書くと、通常意図している文脈から外れた意味になってしまうということだと思います。

「このコードは、`container`が計算を遅延させた場合、ランタイムエラーが発生させ <code>return</code> が非局所的(nonlocal)に評価されてしまうこともある!」と訳せますが、ちょっとこれだけだと意味が読み取れないかもしれないので、return 式が例外の throw で実装されいることを補足したほうがいいかもしれません。特に、yield と組み合わせると確かにコンテナによって return 文の挙動が変わるコードを書くことができました:

```scala
scala> def foo: Int = { for (i <- List(1, 2, 3)) yield { if (i == 2) return i; 0 }; -1 } // returns 2

scala> def foo: Int = { for (i <- List(1, 2, 3).view) yield { if (i == 2) return i; 0 }; -1 } // returns -1
```
This was wrong previously: look into org.hibernate.search.impl.SearchMappingBuilder.getSearchMapping(SearchConfiguration)

It is expected to be able to return other kinds of mapping, including a typeless Object which has a @Factory annotation on a method.
No, the difference is that I don't "build" a new type from each `MethodMirror`, but I "collect" those that have been built from the `Method.type`. This way I don't get the #333 bug of type parameters not being found.
I'm pretty sure that you don't need a do after the while
which is... the entire commit.

nice fix man...
BTW... you're welcome...
deployed on EGATS
Corrections have been made. There were some errors due to copying and pasting I did not catch originally-thank you Christina for pointing that our. 
Any feedback is welcome.
Break into SvgNodeEventHandler and NodeInteractionManager (should also implement ChooselEventHandler).
Dont you trust me? :P
(please dont answer)
npc and players buffs and casts not work

http://www.imageup.ru/img221/878006/wowscrnshot_020412_233205.jpg
Sure, I'll fix it tonight
Destruct won't be set yet if it does need a destructor, but we can avoid a call if it is a plain data row.
Similarly this should probably be a call to poor_compiler_memory_barrier() (which may be able to do nothing as a function call is going to be enough - if not inlined - on all but the smartest of compilers)
Same question for this line?
We don't have any specific requirements regarding the OBO relations, and as you mentioned they probably need to be treated differently anyways. I therefore think we can ignore them for now.
So, you are saying that `self.limit` might be `nil` (which would be the case when `initialize` has not completed). In that case we would see an exception (NoMethodError: undefined method `>' for nil:NilClass), not `true` ... with the previous implementation, so I would argue that your change can't fix anything.

Apart from that, if we had a race condition here we would have seen issues with that before because the class isn't new. (Which btw applies to your dislike towards lazy initialization as well.)
This seems backwards...is the task supposed to fail when no warnings are reported?
Love the License!
Thanks. Thought I had fixed that ) missing before commit but must not of worked. Sorry bro
You could remove Ferocious Bite..it's already coded in spell_druid :-/
Naturally, I could have just removed these lines. But I wanted to be able to easily put them back in if needed.
ここsetStyleNameにしたほうがいいかも
普通のボタン→｛エラー｝→赤ボタンみたいに変更するときにaddStyleNameだとちゃんと動かないかもです。

それか
public void setType(ButtonType type) {
    assert type != null : "ほげほげ"
    for(ButtonType bt  : ButtonType.values()) {
        removeStyleName(bt.getValue());
    }
    addStyleNames(type.getValue());
}
とか?
I don't understand. What test case does 707 fix if the supplied test case works without pulling 707?

If there is a regression, it has nothing to do with 707. Please file a bugzilla report for it with an example that works with 2.057 and fails with 2.058.
This way it is hidden in the resulting DOM tree.
Metasploit already has a encode_base64() routine, please use that instead.
Thanks for the comment.

The reason is a bit arbitrary but has to do with how I use key chords. When I originally picked 'super+k, super+m, super+...' I wasn't quite happy with it but didn't think too hard about why at the time.

After using it more, I realized that what bothered me about it was that it didn't mesh with the convention I'd been using in emacs, which is usually 'C-<something>, C-<something>, <something>'.

That, and I felt maybe it was a little less likely that I'd clobber someone elses keymaps with less supers (I was assuming I'd add more functionality in the future).

If you find the original way faster though, there's no harm and changing your local keymap back to the old way.
Part | Points
------------- | -------------
base | 1
adv | 1
*Total* | 2

Passes tests* :sparkles:

your tree function was broken though.

```
======================================================================
FAIL: test_tree (__main__.TreeTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "hw08.py", line 114, in test_tree
    self.assertEqual(result, outp, msg)
AssertionError: Error: tree(1)
Expected:
*
^
|
|
Actual:
    *
    ^
   ^-^
  ^-^-^
 ^-^-^-^
^-^-^-^-^
   | |
   | |

----------------------------------------------------------------------
Ran 13 tests in 0.002s
```
You needed to fix it a bit more.
While it might seem like the GPLv3 would be the obvious choice, I didn't want to commit any future code that I published here to the GPL without first going over it with Matt and Spenser. The MIT License seemed a good temporary solution that's open to change following more discussions about licensing.
Great, thanks. I'd checkout out Heroku but hadn't looked much past that. I'll see what nodester offers.
@bodepd For nova we have 2 packages 
python-nova : contains all of the nova python files required to run the nova services
openstack-nova : contains the other requirements for all of the nova services e.g. startup scripts, config files ...

glance is setup in a similar fashion

so its fairly different from ubuntu where there is a package per service
Here, too, collapse.
Is there any reason for doing this, other than a bit of a tidy-up? (Not that I'm against tidying up code - I just wanted to double-check I'd not missed anything here)
Hi Douglas,

I wanted to reply to your email but i keep getting delivery error (Domain name not found) so I'm replying here.

Thank you for your feedback. I have added the following license to the code: Creative Commons Attribution-ShareAlike 3.0 Unported License. It should be available from now on in the README file and the install.sh script’s header. I see you did the same here.
I also build upon and improved your update php and apc scripts. They are tested and working now in my repo. Some more testing wouldn't hurt though.

Regards,
Vlad.
Hey, i'm not comfortable with calling it isSubscription, it's too specific for a domain.  I see that you need something to indicate whether a cart item is a good (typically billed once) or a service(recurring billing).  Maybe we should consider having specific subclasses for specialized products.  Eg  CartItemService / CartItemProduct and in your case for subscriptions CartItemSubscription.
This code will be moved in the specific JS-class later, thus we don't mind this, it's for debugging purpose only.
These are great, thanks for adding them! To make it _even better_, I'd suggest adding some JSDoc annotations to each. Something like:

    /**
     * Permission has been explicitly denied by the user. Go away!
     * @const
     * @type {number}
     */
Good catch. I had observed the failure but didn't understand what was happening.
Das ist ok so.
This line hav a bug, the fix is this:
                      <h4 class="subcategory"><a href="forum/category/view-topic/?t='.$row['id'].'">'.$row['name'].'</a><span class="small"> ('.$row['replies'].' '.$Forum['Forum41']. ')</span></h4>
Is missing a dot...
satisfiesthe => satisfies the
Hehe, I simply mean that, on my device, the contents of ` /sys/module/g_android/parameters/serial_number` are [directly derived from the kernel](https://github.com/alanorth/kernel_huawei_u8160/blob/cm7/arch/arm/mach-msm/board-msm7x25.c#L462):

``static struct msm_hsusb_platform_data msm_hsusb_pdata = {
	.version	= 0x0100,
	.phy_info	= (USB_PHY_INTEGRATED | USB_PHY_MODEL_65NM),
	.vendor_id          = 0x5c6,
	.product_name       = "Qualcomm HSUSB Device",
	.serial_number      = "1234567890ABCDEF",
	.manufacturer_name  = "Qualcomm Incorporated",
	.compositions	= usb_func_composition,
	.num_compositions = ARRAY_SIZE(usb_func_composition),
	.function_map   = usb_functions_map,
	.num_functions	= ARRAY_SIZE(usb_functions_map),
	.config_gpio    = NULL,
``

That `123456...` results in a MAC of `12:34:56:78:90:ac`.  Sure it's a MAC address, but it's not any more "unique" than what we already have statically in nvram.txt.
Du brauchst pyGtk. Es soll wohl funktionieren:

"PyGTK applications are truly multiplatform and they're able to run, unmodified, on Linux, Windows, MacOS X and other platforms."

http://www.pygtk.org/about.html
My english seems to be bad.
Kaywa says: "Non-Commercial Use Only" which might be incompatible with the GPL license. Maybe switch to the Google Chart API instead?
What's the purpose of th bcmdhd changes? I've applied all the kernel, p4-common and p4wifi changes to my p5wifi port and while wifi works with the new driver, wifi p2p still does not. Is it suppose to fix p2p? Or does it only fix hotspot (which I cannot test as you only have the wifi-only model)

This is was logcat shows when trying to enable p2p:
W/CommandListener(  112): Failed to retrieve HW addr for wlan0 (No such device)
D/CommandListener(  112): Setting iface cfg
E/WifiP2pService(  180): Failed to reload p2p firmware java.lang.IllegalStateException: Error communicating to native daemon
D/NetworkManagementService(  180): rsp <213 00:00:00:00:00:00 0.0.0.0 0 [down]>
D/NetworkManagementService(  180): flags <[down]>
E/WifiHW  (  180): Unable to open connection to supplicant on "wlan0": No such f
So the possible values of self.multiple are True and None? Why not True and False?

I think this entire block could be rewritten as:
self.multiple = any([getattr(prop, 'multiple', False) for prop in props])
Shouldn't this be a title, not a header category?
As in, where did you take them from?
Vladimir, leave out the leading slash, and .DS_Store will get ignored. Also, you'll have to delete it out of the repository to get rid of it there.
I'm pretty sure we will want the ability to customize the interfaces available on promises.  The addMethod method may be good enough but I kind of like this prototype inheritance idea.

Also note that the Promises/D proposal defines the promiseSend method.  It achieves something similar to addMethod, but uses a concept of message passing rather than method invocation.  I personally don't really like the functionality of promiseSend, but I do like this interoperability side-effect: In the context of a promise library, the existence of a "promiseSend" method must be sufficient to distinguish a promise from any other value.

Although it only promotes interoperability if other libraries also implement it which I doubt many have.
Why are you deleting this function? Is there any code calling it?
"In Scala" in the Java rst?
You're welcome, sorry to not being able to keep up with your PR rate today.
I saw in the versioning document that the term Experimental Features is used for Scala. Should we also say Features instead of Modules?
This is just another of the things I noticed and fixed while this pull request was waiting. I don't believe I can create multiple pull requests with the different bits so it got included here.

The reason for this change is that the cookie_expiration_period isn't required as part of these calls. Leaving the value as None (and not sending to the API) allows you to create a stickiness policy with a non-expiring cookie.
It is needed for initial osx implementation, let it in so we can move on to injecting AE, same goes for any other quirks for osx. We can resolve and refine it once it's in mainline. Not looking for perfection here, else AE will never get into mainline and getting it into mainline is the important thing.
Whats this for? AE_FMT_FLT, etc... are LPCM
`searchsorted(.., side='left')`` should help
for (parentSpanId <- context.get.parentSpanId) message.metadata(TraceHeader.ParentSpanId.toString) = parentSpanId

(And I would remove that .get from context if I had more time for this pull request :-) )
hoho, missing a space ;)
The problem is that the Copy into m_vecItems also copies the path stored in items which might not be the original path but a filtered path (for xsp's the original path points to the xsp file but the path in items is "videodb://foo/bar/" so this is necessary.
I suspect this can break things. But from what i can tell from specs it should be correct. Might be worth getting in.
no need for using ``else`` here as the ``if`` returns
Please remove this useless code snippet.
This might have been a remnant of using Rumi's old channel stuff, but now that I am using the normal endpoints, it should always be there. I was just adding the hasattr(self.channel, '_send_name') to be safe"
Fixed. I didn't realized that I only modified eclipse to use spaces in text files and not in Java files. Now is configured properly.
Some of these fail, and I think they will fail in develop/master too.  The test needs to be a little more detailed, find an appropriate error, etc.
that is one of the ways--missing from this highlighter at the moment!

But you can also indicate javascript with `-` as a prefix; see:

https://github.com/miksago/jade-tmbundle/blob/master/Syntaxes/Jade.tmLanguage

or

https://github.com/visionmedia/jade#code

I'm not sure what `!=` is for but it was in the original.
I'd like BSD license better.
Hmmm... it should look similar to: https://github.com/TooTallNate/node-vorbis/blob/master/deps/libvorbis/libvorbis.gyp#L50-L58
Same comment as above - `no_errors` seems to be a cleaner name since we already know this is a TestFile object.
Depends on how you define "pretty much the same". Using `delete` we use less memory and don't have to loop over keys for which we aren't sending values anyways. So we are also saving some amount of CPU. In the `= undefined` variant we just set the value to be the same as if the key didn't exist but have to store it and loop over the key. So I'd say let's go with the `delete` version.
I don't understand how it can work as `$this->cache` is only set by the `findTemplate()` method.
split different backend code into different files.
hmm. just saw your email to the mailing list... I'm fine with keeping it as numbers
Given the amount of time we took to reach this conclusion can you add a comment?
I modified this slightly for clarity since it's actually the decomposed *transform functions* that are interpolated numerically.
The whitespace change highlighted the typo here. "temporarily unavailable" would be correct. Do you mind changing it?
The %a modifier is only for the total number of days between two _dates_. Since there is only 1 date we don't have a difference.
вынес какие-то куски функций. там, где посчитал, что будет много if-ов, не стал.
сдвинуть на +1 таб
Ditto re `keys` and short-circuit `return`
This feels.. dangerous :)
This looks a bit cluttered, can we make it prettier it somehow? Perhaps it could be part of line 115 with #slice?
Fixed in https://github.com/rails/rails/pull/6067 :)
@carlosantoniodasilva : fixed.
space after `,` in both
Besides the slash that @fxn has talked about, I think the quote at the end is not necessary anymore :)
...if the +debugger+ gem has been loaded...
do you mean add on rails doc or add comments??
because the comments of this two methods are above cache method
I think you can refactor it for something like:

```ruby
records = relation.where(table[primary_key].gt(primary_key_offset))
if remaining_record_limit && remaining_record_limit < batch_size
  records = records.limit(remaining_record_limit)
end
records = records.to_a 
```