labofsahil/github-event-dataset-2012 huggingface dataset:
Bug265: Rewrote a bunch of code for holding objects

This should be ridiculously stable now, and easy to make new classes for. At some point we may want some sort of generic "holdable" class or mixin, but that should be pretty reasonable now that there's a better structure to that part of the code.

## Changes to player.lua

The player class now uses the variables ```self.currently_held``` and ```self.holdable``` to store, respectively, the object currently being held, and the object you would grab if you were try to pick something up. So, self.holdable is used the same as before, but self.holding is no longer used, traded off for the more descriptively-named and better-used currently_held.

It also contains some new methods, all of them documented.

### setSpriteStates(presetName)

Takes a string "preset name", such as "holding" or "default", and sets some internal variables according to that preset. At present, those variables are walk_state, crouch_state, and gaze_state, although one of the nice things about having that code all in one place is that it would be easy to add more variables to this function's purview, unlike before, where these values were being set by external code all over the place.

### pickup()

Pick up the current self.holdable. This does nothing if you're already holding something, but will set sprite state to "holding" using the above function, set currently_held to holdable, and call currently_held:pickup(self) if the object contains that callback function.

### throw()

Throws the currently held object. Works a lot like pickup(), including setting sprite state to default, and calling the thrown object's "throw" callback if available.

### throw_vertical()

Throw the currently held object straight into the air. You can do this by gazing up and throwing. Callback code is what you'd expect.

### drop()

Can be used to set fragile objects down without injury, drop a baseball to the ground half-heartedly, etc. Callback code is what you'd expect.

## Baseball and Pot

Both of these classes had some surgery on their innards, and no longer listen for keypresses. That job has now been moved to the Player class, along with setting sprite state, so all these classes have to do is provide the optional callbacks dictating their behavior when picked up, thrown, thrown vertically, or dropped. These callbacks mostly involve physics and setting some variables that track internal state.

While I was in there, I also took the liberty of incorporating the player velocity into the object's physics wherever it improved gameplay. There were a few parts where I tried it and it was lame, but those parts I undid before publishing.

## Holding and glancing

One thing that was made obvious during testing is that the wrong sprites are used when looking up or down while holding an object. The animation simply becomes "holdwalk", which doesn't make much sense when you're standing still underneath Cornelius' chin getting ready to pop one straight up at him from underneath. That's probably a fix for another day, though, since it's unrelated to the physics, and was a preexisting problem before I stuck my nose in to fix this bug.
Adds ability to navigate backwards

Adds the ability to type <shift>-c to navigate backwards.
LPS-26544 - Welcome Theme

As per Jon Neal's request I'm sending the pull request to Nate Cavanaugh for review. Jon told me that it only needed to work in IE8. From some of my testing it seems to work IE6 and IE8 but in the IE7 the call to action button doesn't work.
Bugs sprint2

major or higher bugs fixed during sprint 2. Tickets have been closed and tested on every day.
Fixes #287. Remove deprecated and silly APE stuff

- removing non-functioning ODS links
- removed tnsuarez specific stuff (youtube)
- removed "lame" user messages and functionality
- removed old "Project Mercury" toggle for wordpress
- removing old portal toggle code
Acesso a todos os controladores através de login e senha

Vale lembrar que ao criar um novo usuário é necessário marcar a opção "enabled", do contrário, não conseguirá fazer o login nos controllers.
First try at Danish translation.
Set code of HTTPException to HTTP response received

I found it odd that the Exception code wasn't being set. 

One shouldn't be forced to parse a string to find out the nature of an exception. Especially if you're going to generate exceptions on 404s when dealing with CouchDB.
rename record to subject

This gets around injecting `record` into `Blog::Presenters::PostList`, and lets us be clearer with terminology (`subject` is what the `delegate` returns). 

It also helps with users thinking all delegates have to return records.
date Field type: date format example is invalid

It is actually a documentation bug but I haven't found where to file them. So:

http://symfony.com/doc/current/reference/forms/types/date.html -- here it is specified that 

> For more information on valid formats, see Date/Time Format Syntax. For example, to render a single text box that expects the user to end YYYY-MM-dd, use the following options:

and the issue is that according specification http://userguide.icu-project.org/formatparse/datetime#TOC-Date-Time-Format-Syntax (which php's IntlDateFormatter follows) `YYYY` is **not a valid year format** and `yyyy` should be used instead.

If you specify the format as specified in documentation you'll always get `1970` as year fraction.

Reated: https://bugs.php.net/bug.php?id=52283
Renaming registration features

No need to prefix feature files in `registration` folder with `registration`
Showcase for Spring Extension Alpha 1.

Added fallowing modules:

* spring-inject
* spring-javaconfig
* spring-jsr330
* spring-testng
* spring-jdbc
* spring-hibernate
* spring-jpa
* spring-ejb
* spring-jms
LPS-29014 CacheResponseUtil.setHeaders() should use given headers to ove...

...rwrite response, but if some of the given headers share same name, them should be appended.
LPS-28437
LPS-30905 Fix Not advanced search in sites
LPS-28407 Html5 validation (input type=image can not have a value)
LPS-28407
[TECH SUPPORT] LPS-28123 Implement lock for Staging
Fix ticket #11245

 Fixed validation when creating new RSS channels. The deprecated EntitySearchFilterDOM has been dropped and the intended class FilterUtils is used instead.
Single Page Application Patterns
Fixed @import using lessphp filter and relative paths

Changed filterLoad() to use lessphp's built in loading of files. This allows @import of .less files to work correctly with paths relative to the .less file being loaded.
movement in the structure

Создал отдельную ветку, только с изменением структуры расположения файлов.
Перемещения предвидятся, так как есть "мутные" моменты и порядок в структуре это тоже важно.

продолжение #49
p/s немногим позже выложу наработку по ускорению разработки с помощью виджетов.
Fix Issue #293 and Issue #294. All credits to mirell.

Fresh pulls of rubygems.org fail on `rake gemcutter:import:process ~/.rvm/gems/ruby-1.9.3-p0/cache` as describe in Issue #293 and Issue #294. 

The fix from @mirell's (4ff7fde) fixes this if applied in `app/jobs/indexer.rb` resolve this issue and allow the rake task to complete successfully.

(Also close #293 and #294 if you pull this.)
Added information on accounts.url directive

Updated docs/frontends/FTP-and-SFTP.rst to include information regarding the accounts.url directive per ticket #1660
Don't raise error when RAILS_SECRET_TOKEN is not present.

Heroku couldn't compile assets because of this.
More information what Heroku does after received a deployment: https://devcenter.heroku.com/articles/rails3x-asset-pipeline-cedar#troubleshooting
LPS-26060
Makefile: Use -rpath instead of LD_LIBARY_PATH for xpcom_abi

Hi, please pull my commit about rpath-linking. This simplifies the invocation of the xpcom_abi and fixes cases, where multiple library directories are needed (ubuntu once again).

Makefile:
* The PLATFORM variable was set with a recursive make invocation everytime the
  Makefile was called, even when trying to do a "clean". If the compilation of
  the xpcom_abi executable failed, no make command could be executed.

  Instead, we depend on the get_abi target to set the PLATFORM variable using
  the "eval"-function.

* Instead of providing LD_LIBARY_PATH for the xpcom_abi program, we link the
  shared libraries using -rpath. This way, the executable will know where to
  find the shared libraries.
Remove unnecessary/misleading exception handling for rightscale_tools in block_device lib

Cookbooks depending on rightscale_tools should simply depend on the rightscale cookbook in metadata and include the rightscale::install_tools recipe to install the gem packages in the compile phase, ideally via the chef_gem resource.
Proccess timeout fix

Now it should honor the composer setting
Misc changes from O.S. Systems
Added Sublime Text 2 support

Hi,

I added Sublime Text 2 support. Check also readme, currently it shows my fork.
Previous and next associations - linked lists essentially

models now have next and prev associations where applicable when added to a collection.

I may have missed something, but I couldn't see how to traverse within a collection so I amended the _add function of Backbone.Collection to add model.next and model.prev where appropriate. 

If there is already an established means of doing this, please let me know.

Thanks!
Added a 'raw' flag to keep JSON response in URL obj

What I'm writing has to feed back JSON if requested. It would be rad if I could pipe back the JSON directly from you guys instead of decoding a python obj.
Fix grouping in JHtmlUser::groups

While testing joomla/joomla-cms#29, we found that the grouping for the user group selection list was broken.  As it is based on the one used in JHtmlAccess::usergroup, I compared the two and found an extra grouping field.  Removing this field fixed this selection list.
Document starting at with a percentage complete

Here's some documentation for that new feature.
6.x 398 capwiz connect updates

This commit pulls in various updates, including the capwiz connect js module.
    * capwiz_connect.admin.inc:  Added sync date reset & batch size field to admin settings; encrypting Capwiz API user name and password 
    * capwiz_connect.module: Setup alternating sync for cron job pulling users and actions
    * CapwizConnect.inc: Added sanity checking of incoming data; queries to export alerts & alert data
    * CapwizConnect.info: Added dependency on ecrypt module
    * capwiz_connect_js: Adding sub-module that enables passing of data from the Capwiz form to the Drupal and Salesforce.
SubRip file updated and updated camrip link

Sync on first minutes (line 74) and merged with last commit (SHA 371d37b1f3).
Also noticed that youcannotredo.com updated the camrip link. When I tried to access the GDrive link got an error of "unaccessible", so I think that it's down.
Add options to breadcrumb links

I recently opened an [issue](https://github.com/seyhunak/twitter-bootstrap-rails/issues/248) about this one. In a nutshell, I'd like to get more control over the links. I'm not sure that it's good idea to shove the stuff in `<a>` tag. Instead of that, I used `link_to` helper to achieve the same. I also changed `.gitignore` to blacklist backup files (I use Emacs, so there's bunch of `~` files out there, but `.swp` for vi is there too).

**DISCLAIMER**
It's almost 2AM, and I'm braindead. AFAICS, I messed up smth, and it's not working as intended. Will catch up in the morning.
Some added functionality

- support for a sort control when searching
- support for deleting an entire tree
- actions return full PDU objects with success? and failure? methods instead of true or false
Dispoing connection properly + tests + documentation for RabbitMQ

Hiya,

A bit of docs and things.

Oh, and `Using_the_handler_test_factory` fails for me! But it did that before I started adding documentation, too.

```

Test 'MassTransit.Transports.RabbitMq.Tests.Testing.Using_the_handler_test_factory.Should_have_received_a_message_of_type_a' failed:
	TestFixtureSetUp failed in Using_the_handler_test_factory

TestFixture failed: SetUp : MassTransit.Exceptions.ConfigurationException : The subscription was not received on the output bus
   at MassTransit.Testing.Subjects.HandlerTestSubjectImpl`2.Prepare(TScenario scenario) in D:\code\oss\MassTransit\src\MassTransit\Testing\Subjects\HandlerTestSubjectImpl.cs:line 53
   at MassTransit.Testing.Instances.HandlerTestInstance`2.Execute() in D:\code\oss\MassTransit\src\MassTransit\Testing\Instances\HandlerTestInstance.cs:line 40
   at MassTransit.Transports.RabbitMq.Tests.Testing.Using_the_handler_test_factory.Setup() in D:\code\oss\MassTransit\src\Transports\MassTransit.Transports.RabbitMq.Tests\Testing\HandlerTest_Specs.cs:line 34
2012-01-23 19:07:59,240 [TestRunnerThread] INFO  MassTransit.BusConfigurators.ServiceBusConfiguratorImpl - MassTransit v2.0.1.2, .NET Framework v4.0.30319.239
2012-01-23 19:08:04,927 [TestRunnerThread] DEBUG MassTransit.Subscriptions.Coordinator.BusSubscriptionEventListener - SubscribeTo: MassTransit.Transports.RabbitMq.Tests.Testing.Using_the_handler_test_factory+A, MassTransit.Transports.RabbitMq.Tests, a10024bb-5847-4134-8fef-9fe1013b5488
2012-01-23 19:08:04,929 [TestRunnerThread] DEBUG MassTransit.Subscriptions.Coordinator.BusSubscriptionCache - SubscribeTo: MassTransit.Transports.RabbitMq.Tests.Testing.Using_the_handler_test_factory+A, MassTransit.Transports.RabbitMq.Tests, a10024bb-5847-4134-8fef-9fe1013b5488
2012-01-23 19:08:04,931 [TestRunnerThread] DEBUG MassTransit.Subscriptions.Coordinator.BusSubscription - SubscribeTo: MassTransit.Transports.RabbitMq.Tests.Testing.Using_the_handler_test_factory+A, MassTransit.Transports.RabbitMq.Tests, c1a8b124-0262-4fbc-ba5a-9fe1013b5489
```
Hacketyhack may now start without an internet connection

hacketyhack does not crash any more when starting without an Internet connection. The only place where I had to check for this was the check for the latest version. 

In order to determine whether we have a connection or not an HTTP request to the hacketyhack API is made in order to see whether the response is a HTTP Success.

I went with this approach rather than just rescuing the error, because the same function could be used to deactivate other Internet related functionality (Register a new account, Upload button of the editor).

This should close issue #140 (is there a way to get this pull request directly into this issue?)
Performance degradation in content_columns: respond_to instead of rescue

`rescue` causes a slowdown for ORMs that don't have `content_columns`

See https://gist.github.com/2bea805664bfb6491d1d#gistcomment-73931

`rescue`-ing from everything is pretty weird thing to.

(Should probably consider not using it at all here).
Lots of build improvements. "ant examples" at the top-level now builds t...

Made a bunch of build improvements.
Drop multiple tables by type

I needed to do this for some unit tests and I thought these extra couple of methods might be convenient for others as well. It also brings the Drop methods more inline with their Create counterparts.
Remove runtime dependency on ActiveRecord

I was going to add this to the previous Pull Request :bomb:
[WIP] Extract http calls to client

Please review the refactoring.

I haven't test in real application(will do) but I made all tests working.
Use 401 with Authenticate header, when appropriate

This is attempt to fix Issue #187 - 401 v.s. 403 - with your recommendations from the unauthenticated_response branch.
Added scopes example to docs

The concept of Scopes in the GitHub API v3 isn't obvious, and should be included in the docs here.
API Token security fix

After an API token has been set, any attempt to user Client.token(username, password) would return the already stored token, even if the username was for a different user.  This means an API token could easily be leaked to a different session.  This pull request fixes that bug by always requesting a new token with Client.token(username, password).

Also, we've fixed the specs: no failing specs, 4 pending (Issues #43 and #36)
Corrigindo bug ao editar instituição e salvar.

O problema estava relacionado ao redirect após clicar no botão de salvar.

As mudanças eram salvas, mas o redirect apontava para uma url inexistente.

Também foram ajustados outros redirects.
Created article Browsing Blobs within a container using the Windows Azure Node SDK
Updated documentation for point2dem

...pare to MOLA, and details on the post spacing argument.
Make gem fetching not rely on a chroot, and a couple other enhancements. [2/3]

This pull request fixes up our gem handling routines in the build
system to no longer require a chroot to fetch all required gems and
their dependencies.

Instead, we add a wrapper for rubygems that allows you to specify
multiple version constraints, and use that wrapper with dependency and
fetch to fetch a gem and all of its depencencies without the
requirement of actually installing htem -- a feature that neither gem
nor bundler have on their own.

This also expands our understanding of gem declarations in crowbar.yml
files to allow specifying version constraints using rubygem-inspired syntax.


 build_curl.sh        |   16 ----------------
 build_curl_chroot.sh |   16 ----------------
 crowbar.yml          |    2 --
 3 files changed, 34 deletions(-)

Crowbar-Pull-ID: 5a7193a964dbe055d1ecdae15c758660cc205039

Crowbar-Release: development
Fix a number of issues in the guide

I'm reading the guide and fixing various issues along the way. Please let me know if there are issues in my fixes :)

I'll be adding more commits as I progress through the guide so you may wait until all of the chapters have been proof-read.
Added ability to pass options to autosignin_url helper

Example:
```ruby
auto_signin_url_for(user, :return_to => path)
```
Instance variables are a list of symbols now.

... fix the case when objects have attributes with the same names as ruby keywords.
Fixed a couple of bugs and made a few changes to Antiqua

Fixed a bug with obj/item/clothing/under/chameleon/psyche , added some small details to Antiqua and redid the arrivals arm, and removed the patient ward area. (can still be recovered via TortoiseGit, by reverting to an earlier version; I'll explain in the Antiqua topic why I removed it)

Basically, the gist of it is there wasn't enough space for the sleepers, and that the placement of the frosted windows managed to bug the line of sight when in the ward, or in a "cell".


Edit: Fixed a bug with the player panel.
global optionHeight

there was a global optionHeight i found by running jshint. there were other problems, but this is the most important one.
Remove scala from the SNAPSHOT jar

The scala library was removed from my machine and indexing is working fine. The purpose is to solved the problem from the following issue.

https://github.com/hunterhector/dbpedia-spotlight/issues/1
1.6.1 release

fixes for Eclipse plugin descriptions for checks
@OutputDirectory on downloadGaeSdk

Just put the flag on the field so that you can leverage Gradle's own dirty checking instead of hand-rolling it yourself.
use rdoc/task instead of rake/rdoctask

rake/rdoctask is deprecated in RDoc > 2.4.2+
Feedforward and windup control

I added a bit of functionality that you may like to incorporate. It seems to be working well for me; feedforward is particularly nice for temperature control, where it can (roughly) estimate the heat loss.
Numerous fixes

We had to apply fixes in many places to have our extensive DB test suite pass with this driver. You may find this fixes valuable too.
Fix dates to force correct format

The dates were being formatted in a way that google wasn't happy with. This commit uses the specced format.
Testing

-Retirada da variavel status desnecessária.
-Problema conhecido de duplicação de agendamento(corrigido)
-Otimizado o gerador de números primos
Clones read-only config prior to assignment

Fixes #1
Fix for strict alignment warnings

This is the fix for strict alignment warnings that are emitted with gcc 4.4.3 on ubuntu 10.04 x86_64
add yarddoc gem and guard-yarddoc

remove rdoc rake tasks and gem
.netbridge for OpenEngSB

.netbridge, for OpenEngSb startig from version 2.4.0
Dustjs support and feature matrix
Codecleanup

Lots of code cleanup in the codebase, including adding useful exceptions for common errors, doc strings, removing unneeded code, etc.
Switching over to use wp_enqueue_scripts method

Most scripts and styles now modified to use wp_enqueue_scripts.
fix pagination previous-9 < 0  : Page not found.

no need +9/-9
New fallback service for embedded videos and update to History Channel

I'm hoping to extend in the future to obtain a list of supported site regex's from the Framework, and then use that instead of just youtube and vimeo.

 - Update to HistoryChannel to support MP4s
Temp ics

Update init.p3.rc
Corrected fatal error in the function selector

Added the proper syntax for the function selector, program now works as
intended.
Adding a fix to the automated functional tests

This fixes issue: blackberry/BB10-WebWorks-Framework/#204
Don't show add conference buttons to logged out users
Simplified default paths

server/express: Simplified default paths, fixes the problem discussed after the close on #90
Experimental - Fix Provisioning and removed "Banning"

If speechID and assistantId are blank, it is because the phone isn't setup, not because it is banned....

Now only lets phone provision of there is Valid key in the Database. Otherwise all oobjects are blocked of there are no keys in the database.
Sure thing. I've got one more change incoming to add a bug fix I put in View yesterday.
err... so if the email is a list, you just return the whole list?!? if i tabcomple a name where this is the case i would end up with all those addresses? doesn't sound right to me.

also, why not demand a name value? afaik abook satisfies that does it not?
This will force a space after an image in browser rendering, something that doesn't matter with the other assets but may be important with images.
Why did you change this?  There doesn't seem to be a difference between GetFullPath and GetLocation if a configuration is supplied.
Wouldn't it be easier to pass the authentication object itself and have the MutableAuthentication class set all other properties?
- some fixes to make the new code compile on windows (there is no "bool" in plain C)
- fixed openGL crash when loading RGB textures (gravit was alwas assuming RGBA)
- tried to improve skybox texture rendering a bit (from tips I found on several websites)
- in preparation of using different textures for every side of the box, repeat "glBindTexture(GL_TEXTURE_2D, skyBoxTextureID...)" for each quad of the box.
I guess you should do the same in Russian then. ;)
I disagree, why do you want to change the default log level for salt-call? I set it at info because salt-call is often a diagnostic tool.
OIC, didn’t scroll down to see the zeroing of the pointer within the 
routine. I can't imagine why that would need to be restored (but don't use 
the cli much).

-----Original Message----- 
From: Nicholas Humfrey
Sent: Thursday, March 01, 2012 6:34 AM
To: David Kopf
Subject: Re: [contiki] Fix for Mac OS machines, which only send CR and not 
LF, talking to serial line diver. (974ab3f)

I released that I had accidentally also changed the behaviour - previously 
it was reseting ptr if the process was restarted.

I have created a amendment, which restores the previous behaviour:
https://github.com/njh/contiki/commit/7edcd70996e23193dbd3b2597c5e008e284232ff

--- 
Reply to this email directly or view it on GitHub:
https://github.com/njh/contiki/commit/974ab3f246bfdd8be41df8c3eca56b9016195e0b#commitcomment-1030691
@larzconwell it's dead simple to do something like that in previous rails versions, by doing for example `ActiveRecord::Base.attr_accessible nil`, the thing is that if you have a big app that does not whitelist attributes, it will all break if you turn it on. That's why there is no simple magical fix for it.
>     Integral(f(x), (x, 0, 1)) == Integral(f(y), (y, 0, 1))

Which I think should not happen (see issue 2440).  Maybe something like equals should be made to be able to compare these.
alright, hope spree 1.1 support Rails3.2 .  Rails3.1 is too slow in development mode
Is that comment correct? Can you generate a shadow hash for windows using openssl? If not please update it.
It's perfectly valid for an interface to be empty - it's only used to indicate that a filter can also serialize (though technically the serialize interface should extend the filter interface, and then any serializers implement it only.

When you say it's in the wrong file, do you mean in terms of auto loading, or?

If you ask me the Filter.php is in the wrong place, the Cache dir was meant to be a directory only for cache implementations, which is why the filter.php and filter folder were outside it.
I agree, hehe :)
Thanks! I actually removed that long ago, but forgot to commit it. Now pushed live.
Currently this seems only to be called used by the EditGameDialog, which already processed the raw array and created an Common::Array<ExtraGuiOption> out of it. So why not pass this (as a const reference) here? That would make the logic below easier.
Oops, missed a commit in there - this adds Client#site_config too.
of course, that is exactly what tweakr was designed for; perhaps you can contribute a function `tweak_css()` to this package
This is fine for now, but we might want to keep an eye on this. Since we're maintaining _position and rectangle separately, it seems possible (though probably unlikely) that they could end up differing enough that it would cause problems.
You should look at 0.15, I am pretty much sure this is some GES port obscure crazyness
There is no tests for checking default request attributes?
`case` shouldn't be necessary
I thought I replied to this, but the reply appears to have gone missing. Basically, the two methods are similar, but not similar enough to make it easy to merge the two. Note, in particular that `print_shortest` does not use proper scientific notation, but rather always uses an integer mantissa followed by an exponent. That's because this "improper" form takes one less character on account of not having a `.` in it. Also, we want these methods to be very fast for printing lots and lots of numbers — especially `print_shortest` since it's likely to be used to print huge volumes of data into CSV or TSV files. For that purpose, I suspect the grisu algorithm will be very effective at giving good compression. I'm guessing that grisu-formatted, bzipped CSV/TSV files will actually be a very compact data representation for a lot of real-world data, which tends to be more "special" when written in decimal than in binary.
@rbs90 das hier scheint der grund zu sein:

Quote: http://www.gnu.org/software/libc/manual/html_node/Directory-Entries.html#Directory-Entries

This member is a BSD extension. The symbol _DIRENT_HAVE_D_TYPE is defined if this member is available. On systems where it is used, it corresponds to the file type bits in the st_mode member of struct statbuf. If the value cannot be determine the member value is DT_UNKNOWN. These two macros convert between d_type values and st_mode values:
What you think to when set etag the methods check with CakeRequest if tye header If-None-Match was received and if match automatically response with 304. This behaviour should be optional, but enabled by default.
That way when controllers set the etag the others requests are unnecessary. Also you dont need to put that logic in app controller.
This commit solves the nightly OpenJavaDocCheck report at http://pele.farmbio.uu.se/nightly-1.4.x/ojdcheck/builder3d.html on line 82. 
 This commit is intended to go into cdk-1.4.x.
I don't think the readability is improved here.

The pattern ") {" is easier to read than hardcoded logic. Especially since the hardcoded logic will assume that tok is ')'. When you read the code you have to analyse lines 4146 - 4164 to know what tok is.
Changed redstone wire to glowstone dust. The wire is fairly useless and doesn't stack neatly.
No and yes.

It won't spin them up until needed, but yeah, it means 6 port no.s.
It could be configurable. Sometimes I just like to make decisions on some things though. I'll leave it hard-coded and see if someone ever reaches that limit. :)

On Monday, January 2, 2012 at 2:52 AM, Joseph Wynn wrote:

> Would it make sense for this to be configurable?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/laravel/laravel/commit/09d1c85e98c2a766b11f723d5a1b63bfa4f477fa#commitcomment-830924
> 
>
I think there is a typo : service.notifications_enableD
Does this make it so that any HTML not in a block is not rendered?
I see... That doppler effect page has fascinating graphics :)
Alright, cool, get the idea.
@RapotOR this is exactly what this change is about as using objects makes it impossible to switch the render tag to use ESI (the ESI request needs to pass the attributes through an url)
And world domination right after! :-D

I heard it's possible to cross-compile for Mac and there's some [evidence](http://devs.openttd.org/~truebrain/compile-farm/apple-darwin9.txt)  [in](http://easleyk.wordpress.com/2010/02/10/cross-compiling-firefox-for-mac-on-linux/) the [net](http://weblogs.mozillazine.org/bz/archives/020363.html). I would love to get that working :-D
I believe this should be the other way around.

classes/profiler shouldn't escape it, that should be done by the display code.
What does this line fix?
Ah! I banged my head a long time for that one :) Thanks!
Have a look at 63ebfc5 and let me know if that fixes it.
Note the `dev=True` on that. For these, we don't want to include the `AUTHOR_ROLE_DEV` in the permissions.
pretty sure this isn't the right URL.
You should use `home_url()` to echo out this, cause `site_url()` retrieves current path of WordPress files directory and sometimes this can be different from home/
I think you need a `mail_pending_refunds()` call here?
yes, exactly. A quick workaround is to test if `coord_transform` exists:

```
if (exists("coord_transform")) data <- coord_transform(coordinates, data, scales)
else data <- coordinates$transform(data, scales)
```

not elegant but I expect it will work.
+1 for not having these discussions in commit messages, it's really not the appropriate place, it breaks us out of our flow.
Replacing:
   '%s.%s' % (klass._meta.app_label, klass.__name__)
with:
   app_label, model_name = parts[index], parts[-1]

For example if we have Order model for app 'foo' stored in 'bar' module (that is valid code), we will  'bar.Order' as model string which is invalid.
Many of the elements with `type` or `name` attributes are handled by the `element.canHaveChildren` check, but those that aren't are checked by this regexp.

I purposely exclude `a` elements from this regexp because they can contain html5 elements. So to avoid naming issues devs can use the alternative syntax `document.createElement('<a name="foo">');`.
Onder OS X wordt activated() ook getriggered wanneer je op [return] duwt. Misschien op andere systemen niet. Helpt het wanneer je deze regel weer terugbrengt, maar dan verbonden met d_ui->filterComboBox->lineEdit() ipv d_ui->filterLineEdit?
someone has been watching other framework's goodness :P
I figured there was an easy way, just didn't around to looking/asking. The other thing I want to better understand is all the merging between branches and forks. 

Thanks for the tip though.  

Nicholas Hagen
www.znetdevelopment.com/blogs
* sent from my iPhone *

On Jan 19, 2012, at 8:46 PM, "Brian R. Jackson" <reply@reply.github.com> wrote:

> If you had written "fixes #4" it would automatically close the issue when the changes were pulled into teatrove/teatrove.  see: https://github.com/blog/831-issues-2-0-the-next-generation
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/teatrove/teatrove/commit/863f2a54e44d9f639828ba82a2d7c9af41bf6f71#commitcomment-884538
Are these colspans supposed to be here? It looks odd...?
hmm.  in retrospect, this was solving the wrong problem as a subsequence != substring
I was helping a friend here and found this line. It makes no sense to me.

I am applying "click" action to ElementList, and the action is only applied to the *first* element? Why? What if I have more than one item?

It makes more sense to apply click to all elements instead of this kind of magic choice - at least to me.
Please move those lines into the init method of gMemChaser. I will have to do the same for the console service later.
Obviously this is fixed in 0bbe749. Nothing to see here. Walk away.
Optimizations=good!
if it needs the includes, that probably means it should also "depend" on the library (vtkSlicerTractographyDisplayModuleMRML) as well. here it's ok because it depends already on the logic that depends on mrml. but i guess it wouldn't hurt (no compilation time wasted) to specify it manually.
@rkh re: language in the URL: http://www.w3.org/2001/tag/doc/alternatives-discovery.html#id2261212
I was try run rtmp mode, but have error:
Subscription failed: nascar360_1@58684
Hmm, that's odd.  There is CSS set within the js file but it shouldn't be messing with any styles not within the element you define (like ul.demo1)
`req.logId` is gone I believe. Ditto later in this file.
We should also load html5.js from the local repository for offline development. Should we use html5.js or modernizr? As far as I know, html5 is part of modernizr. We should also add the version number to the file if it is local.
Haha, perhaps you wanted `git commit --allow-empty`?
perio
+msgstr "Aktualnie przeglądasz reddita z tego IP: %(address)s."
s/IP/adresu IP/

+msgstr[0] "creddyt"	
+msgstr[1] "creddyty"	
+msgstr[2] "creddytów"
are you sure? "creddyt" looks weird

+msgstr "dodatek do chrome"
s/chrome/Chrome/

no consistency with "reddit gold" translation: "reddit gold", "konto gold", "redditowe złoto"
Yeah, something got copied in a bad way. I think I'm going to drop the legacy folder in a bit, anyway. It's doing nothing but taking up space.
Good call KingCrunch. I'll amend it to say "Use namespaced constants, not global constants" (or something to that effect).
По-моему злоупотребление AJAX-ами и всякой активщиной это как раз тоже неправильный подход к веб-программированию. Настройки расширения тоже было был лучше сделать статическим HTML с минимумом скриптов, но там уже просто всё было готовенькое...
Actually, I don't think so. Running the other shell scripts, they start off their own /bin/bash or /bin/sh process. This one actually needs to be run in the current shell (sourced), so it has to be able to be interpreted across shells.
Das ist merkwürdig ^^ ka an was das liegt aber skyfire ist ja eigentlich arkcore.
haste den rebase verwendet?
This still doesn't work. What if a language says 'bhasa 100 de volos' for '100 views'? (Completely made that one up :P)

This should be

```jinja
{% trans view_count=views %}
  {{ view_count }} views
{% endtrans %} 
```
It's still no go.

Either TinyMCE instructions are invalid here http://www.tinymce.com/wiki.php/Configuration:valid_elements or they are against different version. #p does not make p appear on the valid element list and it is treated as invalid element and thus stripped away.

Or am I missing something?

Any clues?
Isn't it better to use InnoDB for performance?
That was removed before.
안녕하세요 1학년 1반 4번 경규성입니다.
다름이 아니라 제가 고친걸 올리려고 하는데 왜인지 모르겠는데
파일에디티가 안되서 ㅠㅠ 댓글로 수정하려고합니다. 
양해부탁드립니다!!
Wow, we really fucked up
rename the file 2011_02_08_00_world_misc.sql with the correct year :P
and 38342 its trigger not for horde bank-for alliance
I've pushed it to the master branch, and soon to the touchdb branch. Thanks again!
estendida
also configured man pages to go where homebrew expects them to be
This looks familiar. I vaguely recall it used to say "if (distributedSource != distributedTarget) and crashed as a result when distributedSource was null
It's working now :) Actually it was working before this commit but using the init script seems the more 'official' way to do it, and doesn't seem to require adding a `sleep`.
We should add a test but I not sure if publish is the right place.
Oh look misspelled with :|
Hi, 

we are now using a new format for changelog.
@petro065 will write a mail for that. 

Greetings

Jotschi
There is no need for a file called blog.css. Each site/theme already has its own css file called style.css. If styles are needed in multiple sites the style needs to go in the main style.css (sometimes with a selectors specific enough to not affect all sites, or all parts of a site/page). Each page gets both the main /base/css/style.css and the specific theme's style.css loaded and those should be the only two stylesheets in the header.
Check if UIDVALIDITY is implemented and use it if it is
Oh well. ;_;
Could we add where we got this plug-in from?
https://github.com/balupton/history.js/blob/master/scripts/uncompressed/history.adapter.jquery.js

Or should we not touch these?
this could be made faster by adding a 

````coffee
for cz in [0... @chunkSize] by 1
  ...
````

coffeescript, by default, doesn't know which direction the loop will take, and compiles the code-as-it-exists to

````javascript

  var cx, cy, cz;
  for (cz = 0; 0 <= chunkSize ? cz < chunkSize : cz > chunkSize; 0 <= chunkSize ? cz++ : cz--) {
    for (cy = 0; 0 <= chunkSize ? cy < chunkSize : cy > chunkSize; 0 <= chunkSize ? cy++ : cy--) {
      for (cx = 0; 0 <= chunkSize ? cx < chunkSize : cx > chunkSize; 0 <= chunkSize ? cx++ : cx--) {
        console.log(cx, cy, cz);
      }
    }
  }

````

so, two extra if statements per loop iteration. blegh! this may not be a colossal improvement, but you might check to see where else this sort of thing is happening.
We could add something to slime-repl-hook or slime-connected-hook that checks for a remote connection and resets it to nil.
Typo? `routes`, not  `routers`
Waiting for a fix from Don on the interpret3.d failure.

I thought I had checked in the druntime fix, apparently not.

On 1/3/2012 9:45 PM, Brad Roberts wrote:
> Since this commit the druntime tests have failed on all platforms.
>
I'm not some people o.O
i will document this "feature" and send you a pull request, but not until next week. i'm very busy at the moment :(
well it wasnt supposed to make it in to an RB and it is definitely not the final fix, but it is a temporary one until we finish the new version of the launcher
kk, I'll do later tonight. thx

On 02/18/2012 08:56 PM, Ioannis Canellos wrote:
> Yes, if you think it fits best there, please do move it there.
>
This last commit seems to have broken my BBone install.

Any ideas why ?
And you were the only person that knew that, which is why I got asked to fix the formatting; this was my fix. I think that formatting tables should be the special case that require a specific sort of CSS in order to not look terrible, rather than "any old table", especially if the distinction is going to be 100% undocumented (tables used for formatting are much less common and people were actively avoiding using tables because nobody knew how to make them not look bad).

Presumably the way to do this is to revert this specific change and shuffle around the CSS to special-case the formatting tables. I'll look at it at some point, or if you'd like to fix it that'd probably be faster. However, simply reverting this alone brings us back to the previous situation, which got changed for a reason. Finally, please document these things.
And by minor changes I mean push all the files I forgot to push LOL
No complaints on my side... If you look at 
https://github.com/malixsys/watir-webdriver-rails/commit/97e38a28ab699db49927680161cd8950a8a80cad

I think that that piece of code did it for me... :)
lost it eh?
Potentially to someone like dad that doesn't really know much about exceptions
I have no idea if it fixed something. It is just the original pinmux extracted from 2.6.32....
OK i will do!

I want say that im translating to Portuguese (Brazil). 

This is a really good plugin, simply wonderfull!!

Congratulations!!!
Tabs should only ever be used for indenting -- often, they're used for aligning, which causes issues when different developers have their tab size set at different things. Ultimately it's personal preference. But tabs are evil. And they must be stopped.
I have added some comments to make your life easier :) Awesome work!
Si puedes modificarlo
I meant the if the 4s is banned then the assistaid will be empty. Banned= cannot create assistant. 
Never mind I am pulling this merge request and we will see .
Great.  Should we merge `v0.9-development` branch in EditorConfig core?
Are you on the symfony master branch ?
That's perfect for backward compatibility. Thanks!
boe!
Agree that there should be some acknowledgement of the conditional checking for existence of defined `window.foo`. May change the wording up a bit but this is fine for now.
Again, propose upstream and provide URL or sha1.
"bobbingforapples"
Completely optional, but you could reduce branching logic we need to test with this: http://commons.apache.org/lang/api/org/apache/commons/lang3/BooleanUtils.html#toInteger(boolean, int, int)
Yes, there's nothing in common between the CUDA and OpenCL solvers, so this should go in `opencl.py`.
Should go above jquery.wikia.js IMO.
sounds good, thx
Ummm, what?
I see now that this `abs` business is totally unnecessary, negative Ints shouldn't be discriminated
no need to say `remaining` here, that’s implied
methods ending in ? should be booleans.  methods ending in ! have two common meanings:
1. (ruby) calling object will be altered.  (eg, my_integer.to_s!)
2. (rails) raise error on failure (eg. save!)

this isn't either of those.

I'm thinking about fetch_trait and fetch_or_new_trait as instance methods?  

guessing they should both skip virtual regardless -- I don't think we want to get into virtual traits, do we?
I find take_step a bit misleading. I think next_step is better. 

In https://gist.github.com/3794432 you used 

    streamlines = ClosestDirectionTracker(model, interpolator, mask,
                                          stepper.take_step, angle_limit=60,
                                          seeds=seeds, max_cross=None)
I think it is a bit ugly that for the interpolator you add directly the  object as a parameter and that for the stepper you add the stepper function.
The long and short of it is that I'll fix it when I do `combiner_filter`. It has to do with escaping shell quotes. I figured I'd rather have a working implementation sooner and clean this particular wart up a little later.
I'll go with `j-KEVIN`. :-P
If there is no tileset to be loaded it will definitively break in 
```
if (!ok)
        mError += QChar('\n') + tr("Error mapping tile id ").arg(tileid);
```
which is also annotated below. Nevertheless I'll return 0 here
FYI --enable-python-bindings is not required in order to build pylibemu.
I see. Never used argparse... always just do it this way. Happy to change it if you think it's necessary.
Is there any reason not to register for this notification at initialization time? (It'll make it easier than having to manage a flag about it.)
These extensions are unnecessary. Besides the CG/NS compatibility, which is mentioned on the `NSValue` extensions, it's trivial to just serialize these types as `NSValues`.
These should read `TUI*` instead of `NS`.
Is it any improvement to do this?

```javascript
var removeRemove = false;
for (var key in this._events) {
  if (key === 'removeListener') {
    removeRemove = true;
  } else {
    this.removeAllListeners(key);
  }
}
if (removeRemove) this.removeAllListeners('removeListener');
```

Saves a function call if there are no removeListener listeners.
> As nodejs.org doesn’t work over HTTPS/SSL, the protocol check was not needed either.

It happens here: if URL starts with `//` it prepends with current page protocol.
Okay.  Let's rewrite the array methods in terms of ref float, then.
```
Support unicode characters in routes. Route will be automatically escaped, so instead of manually escaping:

    get Rack::Utils.escape('こんにちは') => 'home#index'

You just have to write the unicode route:

    get 'こんにちは' => 'home#index'
```
Ah missed that: `# If Date bits were provided but blank, then return nil`
I found `namespace_path` good enough but `namespaced_path` is also reasonable.
Too much logic for one line with non-semantic variables IMO :). And line is pretty long. That's why I decided to split this method chain
Can you take the opportunity and realign this line?
Ok. By the way, I'm leaning towards exactly `conditions` :thumbsup:
Hum, I think for some reason my mind was plugged to the **constructors are not inherited** thing from Java. It works.

    1.9.3-p194 :001 > class ActiveRecordError < StandardError
    1.9.3-p194 :002?>   end
     => nil 
    1.9.3-p194 :003 > ActiveRecordError.new "some error"
     => #<ActiveRecordError: some error> 
    1.9.3-p194 :004 > _.message
     => "some error"
I think text/rtf is not part of the defaults, right? Make sure you add backticks around the option name (`AV::...escape_whitelist`). Wdyt about this:

```
Add `ActionView::Template::Handlers::ERB.escape_whitelist`, that contains a list
of mime types where text should not be html escaped by default, preventing texts 
such as `Jack & Joe` from rendering as `Jack &amp; Joe` for the whitelisted mimes. 
The default whitelist contains text/plain. Fix #7976
```

Feel free to make more adjustments.
Either[Int, Int] is a type I would prefer to see only in my nightmares.
yes, I verified the byte code
Because when the number of elements is small, a set is slower.  You might be thinking a million is a lot of elements, but when bugs aren't shaking their bug parts, the usual number of bounds is somewhere between 0 and 2.
Logging call is must-have thing A.  Must-have thing B is to rethrow scala.util.control.NonFatal.
@xeno-by: I don't understand your comment.
second argument can simply be -1.
typo, should be ``np.exp``.
Good catch.  This was #1537 by the way.
This throws a MatrixError if the matrix is not diagonalizable. What does this error mean in the context of ode systems? It would be awesome to catch this error and throw a new one with a clearer explanation specific to this instance. This isn't necessary though, the majority of SymPy (and my code in particular) certainly isn't written to this standard. Just a thought.
Furthermore, I want to reiterate once again that the reason in the current code that `Au[:, -1]` doesn't work is **not** because of the `-1`.  It's actually because of the `:`.  I guess I should look at why `Au[:, :]` and `Au[:, -1:]` do work to fix this. It must take a completely different code path if both keys are slices.
Yeah, the imports still work.  Importing from `sympy.unify.core` bypasses `__init__.py` completely.  (Unless I am mistaken), `__init__.py` is only ran with `import sympy.unify` and `from sympy.unify import stuff` (`stuff ` being specific names or just `*`).
But the assumptions should always return a bool or None. If it did otherwise, I would consider that to be a bug.
Yes, I would use the parens to get `OuterProduct`s here.
Note: you can also write this as:

```
return abs(cos_theta + 1) <= eps
```

But this is not a big deal, use whatever style you prefer more.
I changed this method name here as it is only used used in write operations.  Confused me for a bit so I thought it might help make the code more understandable.
`$options` argument could be added if such feature request arises in futures.
I count down from max_int to 0 for assigning id's when they are not specified. (if the mesh has no id= in xml)

This to make the change cf a collision smaller. Otherwise i would need to check against a list for every item to insert.
Three ways? password/oauth and env-var/git-config, 2*2=4.

Besides, is it necessary to support *both* password and oauth token?
I don't understand why ast.literal_eval is being used here.  Could you explain this?
Looks that way, yes.

@indutny: You can check that with `pipe->ipc != 0`.
Because it was the same as the column name.
got this one