labofsahil/github-event-dataset-2012 huggingface dataset:
_id property of polymorphic types are not handled correctly

When the @JsonTypeInfo and @JsonSubTypes annotations are used on a class that being persisted to MongoDb using mongo-jackson-mapper, the _id property is not correctly identified by the IdHandlerFactory, since the root JsonDeserializer will not be the BeanDeserializer, but a WrappedDeserializer.
Avoid error if there's no tweets

This commit fixes issue #4
capistrano task

Hello,
I just added capistrano support for sidekiq-scheduler and wiki file for it. Waiting for acceptance.

Have a nice day,
Mustafa
Service manager implemented

adds ServiceManagement by sk89q - modified/updated
adds ServiceEvents modified from zml2008
adds EconomyService as a standard for plugins to implement.

- Signed off by Sleaker
Fix to rounding errors in unix time conversion routines in greenbutton style templates

Signed-off-by: Marty Burns <marty@hypertek.us>
mesecons groups settings, detector textures
Fixes for compilation on Ubuntu 12.04

With this changes to the Makefile and source files the binaries compile fine on Ubuntu 12.04.   

I had to remove the mapnik/expression.hpp import, the file is not present, no idea why. I compiled it against libmapnik2-dev (2.0.0+ds1-3build1) from ubuntu/universe.

Note: 
The current mapnik git version doesn't seem to be compatible, there is an compiliation error in PYTHON_MAPNIK_API char* PyRenderTile.

EDIT:
Also added a error message to ogCreateLayer when invalid parameters are provided. Took me a while to figure out, that i wrote extend instead of extent :-)
リクエスト: syntax table を設定可能に

新しく `table` プロパティを導入して、 syntax table を ac-source 毎に切り替えられるようにしてみました。ただ、補完自体はうまく動作してるのですが、popupの挙動がおかしくなってしまいました。

このプロパティを使えば、例えば、ドットの入った候補リストを使って似非 omni 補完的なものが出来るようになります。


以下のコードで試すことができます。

```lisp
(require 'auto-complete-config)

(defvar my-dotty-syntax-table
  (let ((table (make-syntax-table c-mode-syntax-table)))
    (modify-syntax-entry ?. "w" table)
    (modify-syntax-entry ?_ "w" table)
    table)
  "Stolen from `python-dotty-syntax-table'.")

(ac-define-source my-dotted-source
  '((candidates . (list "Spam.Egg.Onigiri"
                        "Spam.Egg.Foo"
                        "Spam.Foo.Baz"))
    (table . my-dotty-syntax-table)))

(setq ac-sources '(ac-source-my-dotted-source))
(global-auto-complete-mode 1)
```

補完してみた結果がこれです。カーソルがなぜか popup menu の後に飛ばされてしまっています。:

<a href="http://www.flickr.com/photos/arataka/7178333104/" title="screenshot-2012-05-11-223208 by takafumi_a, on Flickr"><img src="http://farm9.staticflickr.com/8022/7178333104_7b6a52b5da.jpg" width="321" height="132" alt="screenshot-2012-05-11-223208"></a>

popwin.el は master の最新のものを使っています。
CLC-747 Toggle behavior for scheduled/unscheduled tasks

Not quite as DRY as I'd like, but was able to eliminate majority of dupe code with this approach.
Added Flash files for spec
Update cookbook with Foodcritic tests

Update the cookbook with changes from [Foodcritic Tests](http://acrmp.github.com/foodcritic/).
Get Last updates
fixing autocommit

previously could not turn off autocommit once it was started
msxml_get_definition_code_exec: added support for ie9

CVE-2012-1889: Support for ie9 added.
fix: better load_json
idk i broke the last one i gave you but now it works verified :D

adsfasdfaasdf
Rewrote proxy method to utilize work done in rack-proxy
Rewite　listener
PRE_ITEM_SPAWN event (CraftBukkit)

### Usage
Extremely useful to prevent duplication tricks, because it will also be called when when you destroy a block with a Redstone torch attached to it.

See also: https://github.com/Bukkit/Bukkit/pull/282

### Example code
```
    public void onPreItemSpawn(PreItemSpawnEvent event) {
        Material type = event.getItem().getType();
        Bukkit.getServer().broadcastMessage(type.name());

        if( type == Material.REDSTONE_TORCH_ON || 
                type == Material.SIGN ||
                type == Material.TORCH) 
        {
            if(locations.contains(event.getBlock().getLocation()))
            {
                event.setCancelled(true);

                locations.remove(event.getBlock().getLocation());

                Bukkit.getServer().broadcastMessage("Unregistered position, disallowing pickup");
            }
        }
    }
```
Fixed bug. Did this ever work?

Yo
PHP: add --with-suhosin

Adds support for the Suhosin patch to the PHP formula.
I couldn't define multiple patches (DATA would never patch when I tried this), so my custom suhosin patch contains info from DATA as well as the FreeBSD 5.3.8 Suhosin patch.

EDIT: Figured out the issue with multiple patches and cleaned it up. As it is now, there is an intermediate 'patchlist' array that I'm using, is there a better way to do this?
function name was wrong (case sensitive)

handleOpenURL instead of handleOpenUrl which is wont trigger because of case sensitive
[TIMOB-11412] add missed "enabled" property to accessibilitychanged event

Backport of #3213
Pull Request #99 had a bug when selecting distinct on a column

I noticed the tests for distinct count weren't really testing anything (since by default it was counting on the composite PK, which would be distinct anyway), so I added some tests to actually count distinct and not distinct rows. 

That lead me to find that Pull Request #99 had a bug if you specified a single column to count on, my code there assumed it would be a String, but it could be an Arel Attribute too, so it raised an exception.

Anyway, then I realized we could just remove the special-case code in favor of calling Rails' version of the method, unless CPK needed to deal with the count specifically because an Array was the parameter. 

All tests pass on mysql and postgresql.
My pad

 Hi Fabio. I tried to delete and modify some uboot stuff to get it in sync with the upstream v2009.03-rc1 branch.
 Next step would be to maybe port to a newer uboot.
 Please check if it still works OK for you.
Refactored to pull ErrorLog out into its own class.

Refactored to pull ErrorLog out into its own class.
Debug examples in IE
LPS-26396_Structure_link_goes_to_New_structure_if_structure_is_global

This one is part of support issue which gets reproduced in trunk
Fix grp_collapsible to actually call the on_toggle callback
Issue #9

Use proper name for prospectAssign function in PardotConnector.class.php
Make sure that we save the prospect and get our data back because otherwise we are out of date
Bundler changes

Merge in the enhancements to bundler.
upgraded codenarc-idea to codenarc 0.16

I upgraded the codenarc plugin to CodeNarc 0.16. It is important to update because of the performance improvements in CodeNarc. 

Can you please test this and then update the JetBrains plugin repo? 

Thanks so much, 
Hamlet
Merge pull request #14 from Psychtoolbox-3/master

"Rotten Apples" release, next try.
Dansguardian blacklist download fix for shallalist

Dansguardian fix to properly update blacklist if the downloaded blacklist has a two deep directory structure (such as shallalist). Tested with shallalist. Since shallalist has both one and two deep structure, fix will also work on the original blacklist download which was only one directory deep.
Base#git_* methods return nil on initial deploy

Fixes #37
Added support for different markup languages

I've added slim, haml, and jade tags, which are generated based on the file type that's open. I've tested it and it seems to work consistently.

I'm not a python guru, so I'm probably missing some opportunities to make the new code prettier.
Add left mouse click support & path for issue #23

see  issue #23
Fixes #320. Merit / demerit system was added.

Fixes #320. Merit / demerit system was added.
Fixes 187: updated dir in htaccess and common
Updated Twitter Icon

Updated Twitter logo is Retina display ready and Page <titles> are more descriptive–especially good for RSS readers. 

I’m sorry if I didn’t do anything correctly in sending you this Pull request, this is my first ever and I am just learning. Also, you can email me and I will send you the .svg with the new Twitter logo if you want. 

My blog (BeyondTheCo.de) has the new design with a few other tweaks.
custom jade template parse to render jade partial with socket io

custom jade template parse to render jade partial with socket io
Fixed typo in readme; Wrapped command references in code tags

Nothing big. Thanks for the great work!
Revision standardMenu
generate.py fixed
Added documentation for Graphite integration features
Include channels user was in when 'kill' is emitted

Implements #69

Would prompt a version change as the signature of the emit is changed.
Little typo on test name fixed

I assume you just copied from the other test. Not very relevant anyway.
Extra info for property_path

Added notes to inform users that in case a field is not present in the mapped class, the property_path can be set to false and thus avoiding the exceptions thrown by symfony.

This is useful for example in forms where an extra field is needed, like in registration where the "i agree to these terms" will not be persisted in the underlying user object.

This was discussed here: https://github.com/symfony/symfony/issues/1960#issuecomment-3774169
Clear the fingerprint when an attachment is removed

When an attachment is removed all the related attributes in the database are currently cleared except for the fingerprint. This comment causes the fingerprint to be cleared as well.
Remove broken iso9660 sector alignment code

This block will align the read to the closest sector boundary, even if that
means refusing to read a chunk of the requested data. That can't ever be
the right thing to do.
Adds code to guess abbreviation of the application the search templates belong to

The guessing uses the application abbreviation which is the first part of the searchable object type's fully qualified class name.
Minor version changed to 3
[AS7-3080] Add module to allow bundled hibernate 3 deployments to use

the second level cache with default settings. Clean up transient and unused
dependencies in the JPA module. Add tests for jpa and native hibernate
3 second level cache usage. Remove a bunch of duplicate classes from the
compat tests.

The default second level cache provider has a dependency on hibernate 4  and 
cannot be used for hibernate 3 deployments. Since caches now support eager 
starting JndiInfinispanRegionFactory can be used instead but it must be added to 
persistence.xml.
Super happy fun world refactor

**This branch also includes the guice_fix branch because it's necessary to get a complete build.**


This refactors the World object to live only once and be created only once when the Runtime is created. 
All tests pass, and the jruby backend has a test to ensure that the ruby files are not parsed multiple times.

Also, the guice backend would tank when it found ambiguous steps as the annotated methods were loaded multiple times. I'd assume spring has the same issue, but the same fix resolved it.

Yay!
Ejqtool

ej's improvements to the qtool
Fix (#10451) - apt::ppa fails to "apt-get update" when new PPA source is added
Mark some core.stdc functions @safe pure nothrow.

This will make it a lot easier to mark a lot of Phobos with these annotations too.
Api bump 4.0

I was not aware that you updated gem to 4.0. But when i looked to your implementation 
i found some problems.

Doc says that desired result should be namespaced to object with key == API_VERSION and desired result as value.

```json
   "desired_result" : {
      "4.0" : { "tags" : ["not_spam"] }
   },
```
I dont think we should pass '4.0' and other api versions everywhere in consumer application code.

I dont see any facility for this in your code.
Same problem with result: 

```json
{
    "4.0" : {
        "action" : "block",
        "tags" : ["spam","strong_threat","malicious","anomalous"]
    } 
}
```

In our version api version used automatically for both request and response.
[*] Version: 2.
INT 2612: Support for MongoDB Inbound/Outbound adapters

INT 2612 looks to add support for Mongo DB adapters there are two sub tasks to this
  * Outbound adapters
  * Inbound adapters

Currently only outbound adapters are developed and the PR will be subsequently updated to include the inbound adapters too.
Fix typo and improve Readme formatting
find sequences under threshold of 60 minutes
Separated the model/controller structure a bit more 

Validate method now takes two arguments, the email and password, that way you can call it easier from elsewhere, if for some reason you need to.
Use simpler tokaido install

I've been working with @wycats on getting @tokaido ready to use for Rails Girls Amsterdam. This is the result of the work. It's a binary distribution of ruby and the gems needed to run rails, so they don't have to deal with rvm, homebrew, or compiling. I spent a long time with to no avail helping a girl with linking errors at Rails Girls Krakow and I think this will fix the issue.
Load tasks method

The branch based on `root_dir`.

I added `ParallelTests.load_tasks` to have ability to load tasks in Rakefile.
Add scaleX, scaleY, scaleZ properties

Can this be merged like this, or is there anything else I should do?
Merging 2.1.0 into master
Adding .gitignore file

Just adding .gitignore file to make "git push" more easy
New italian gramatical corrections

New gramatical corrections for:

guides/getting_started_it.textile

guides/getting_started_rinse_and_repeat_it.textile
Merge branch 'master' of git@github.com:jakamph/Obscure-Reference-Python...

....git into gui_test
Attribute filter

This contains the documentation updates needed for openmicroscopy/openmicroscopy#283 as well as a general proof-reading of the LDAP pages.
Fix wrong text id in pit of saron. :D

ERROR: TSCR: DoScriptText with source entry 36658 (TypeId=3, guid=15228916) could not find text entry -1659007.

Signed-off-by: frozenarmor <venom.victorios@gmail.com>
Performance: separate evaluation of bindings

I made the code changes to support this.

In jsonExpressionRewriting.js, the binding value (val) is wrapped in a function call: function() { return (" + val + "); }

In bindingAttributeSyntax.js, the parsedBindingsAccessor (called allBindingsAccessor in the binding handlers) and valueAccesssor functions unwrap the extra function call. Since these are called within the binding handlers, the actual value is only evaluated within those handlers.

Also in bindingAttributeSyntax.js, instead of a single dependentObservable for all bindings, there's one for each binding. The init functions are not called within a dependentObservable since they only need to be called once. dependentObservables are only created for bindings that have an update function.

Possibly breaking changes:

The 'allBindingsAccessor' function takes a binding key parameter and returns the binding value. The valueAccessor function just calls the allBindingsAccessor function with the binding's key. I've changed the default bindings to use the new allBindingsAccessor syntax, but any custom bindings would have to be changed from allBindingsAccessor()[key] to allBindingsAccessor(key). I also changed a few of the tests that dealt directly with the expression re-writing code.

Because the binding values are now evaluated within the binding handler, if a value is accessed from more than one handler, it will be evaluated more than once. Before it would be evaluated only once for all bindings, but would be evaluated each time any observable binding value changed (because then all binding handlers were updated).
Make traits work after Factory() was called.

Here is a test to reproduce the issue I faced: 7b0f86cfd81f0eb

And that's the fix: 64ad5c0b8b3f


I guess the problem is Factory() call modifies an instance of factory and sets @evaluator_class.
Next time when Factory@with_traits is called it does `self.clone.tap do |factory_with_traits| .. lala .. end`. So it clones an existening factory with `@evaluator_class` already setted.
Added German translation

I have translated the HTML file to German.
FIXED error in POST & PUT request body

Hi guys

I'm really sorry there was a bug while the POST & PUT body in JSON format.

As all the specs were green I thought it was ok, but I found the bug while using the gem.

I'll double check and improve the specs but I want to send you the patch ASAP. Now I'm sure I can create and delete addresses.

Thanks and sorry for the issue.

Juan
this resolves http://jira.graylog2.org/browse/WEBINTERFACE-169

This fixes WEBINTERFACE-169 when running on Ubuntu Lucid's default ruby - ruby 1.8.7 (2010-01-10 patchlevel 249) [x86_64-linux]

I have not tested if it breaks on any other rubies though.

Mick
Fixes for NewMessage and GetMessages

https://github.com/andyedinborough/aenetmail/issues/1
Removing one extra call to GetResponse which I'm assuming is a typo.

https://github.com/andyedinborough/aenetmail/issues/16
Checking that message count > 0 before raising NewMessage. This *seems* like the correct thing to do, but I could be misreading the specs for IDLE.
SOCIAL-292: Changed 'linkedin' element to 'config' and extracted common namespace handler code to an abstract base class Edit
Adds db information lookup to defaults and library

Add node['openstack']['db'] array of db info hashes
Add ::Openstack::db() and ::Openstack::db_uri() methods
to the library.
The right definition of the 'disambiguator' parameter in the Disambiguator service

As the Disambiguate service accepts only GET requests, I think that all the params should be QueryParams, not FormParams.
Topic/cleanup

Code changes for security holes/bug fixes
Still need to update readme
I've set DEBUG and TEMPLATE_DEBUG to false in local_settings.py for the staging server, but not production, so we should be able to see if anything breaks when this is merged.

I've added the readme changes and the MIT license stuff
made sure that jasmine resource are always expired, so the browser won't cache them

Wen working with the jasmine:bdd goal, the scripts are currently cached in the browser. This makes it very awkward to do TDD. Adding an expiry time  in the past in the response header solves this.
MooseX::Types prereq missing

CPAN testers fix (see for example http://www.cpantesters.org/cpan/report/f0f18b84-e0ee-11e1-9066-3d54fb7543f5)
changes in video codec

for alpha 7
Simple account cancelation

As discussed:

* Cancel button on profile page - Triggers an email to us
* Note on profile page about cancellation taking up to 24 hours
* Link to "Update your account settings" in broadcast emails (#86)

@sandal look over the code just to sanity check me, but I think this is OK to merge.
Cleaning View Controllers

- He movido código de los .h de los controladores a los .m para mantener las interfaces limpias: las cosas que no son "públicas" no hace falta exponerlas en el .h
- Eliminado el código repetido en todos los controladores. He creado la clase ```BaseViewController``` de la que heredan todos los controladores y así se puede compartir código.
- El método para comprobar si tienes conectividad lo he borrado para usar [JSReachability](https://github.com/JaviSoto/JSReachability) que comprueba si hay conectividad **asíncronamente** sin bloquear la interfaz.
Add support for several UUID providers

Support, in this order: Ruby 1.9.2+ stdlib's SecureRandom.uuid, then guid, uuid, and uuidtools.

The major benefit for me is using a UUID gem that I can get pre-packaged with my distro.
fix splitDateTime for plain datepicker

When using a plain datepicker, getDate returns always the current date because splitDateTime puts the date in the time part of the result.
This change only splits when the input actually contains a time.
Engine extensions

Added Engine Model extension example, Updated model namespaces
Small correction on the %pre part of the rpm install phase. And %preun

We now check if it is an install or an upgrade, so we don't try to stop plex if its an install, and the rpm install fails.
Dev to master

Pull the dev branch to master - break in the GitHub software.
Added PloneIDE and collective.twitter.portlets
Allows for specifying a device via serial id

as above.  Please review.  Not sure if this was the best way.  [ More or less doing this for practice coding and getting used to github ]
In For The Kill - La Roux feat. Skrillex
Updated to pull stale flag from config, and use update_after

Update to make it easier to use the update_after stale flag for couchdb 1.1.1.  Also pulled the value out into the ini file to make it easier to update globablly
Bruno, have you tested these with multilingual content? I'm not sure, but OTOH $title should be an array, and "$title" might cast it to a string. (Same with other fields.)
(That's probably the least important comment I've ever made)
you are right this definitely shouldn't be in search. However I think these models should be in the same app, it seems to me product and deals are the basic building blocks for everything else and may be fetched by all apps. The model that's a question mark is: ProductPriceHistory. Maybe it shouldn't be in this app, but I'm gonna keep it here for now until we know exactly what we want to do with it.
ovo sam dodao
Outra coisa, este projeto é em pt-br, estamos conversando no idioma errado... :-P
Add a NOINLINE pragma
I'll look at it tomorrow, but you have a point there about the confusion that could happen.
It's not really cyclic; if no responses are sent, there isn't a callback. So when a server stops, the timeout function will be called at most once.

Also, Date isn't really instance-specific...
I don't see the point of adding those methods here... as those happens to be just shortcuts so that users can write $this->createUrl() instead of Yii::app()->createUrl()
This should be `css('Blog.common.css', null, array('plugin' => false));`, right?
你什么时候能用select...
Well, I was thinking about this instead, and it is always then an array all the time:

```js
types: mixinsValue ? [type].concat(mixinsValue.split(/\s*,\s*/)) : [type],
```

The only problem is that dealing with it as a string in _getCtor is far easier, so I have to convert it back to a string when calling it (even converting it in _getCtor is tough because of the recursion).
The coding style requires braces to be indented 2 spaces.
Added a simple test for them both and tested it under all 3 pythons.
For updating, sir, you deserve an internet cookie. Here you go :3
I believe we should remove previous version of the file right ?
something like:

@files = @files.filter (f) -> f.path isnt file.path

Or change the FileWatcher.on("change") to not simply add the file but look for an existing file and call compile()
Maybe we should make `lfold` the canonical name for this and have `reduce` just be an alias. @JeffBezanson, thoughts?
Ce serait cool de faire des commits qui compilent, et en français correct, j'ai autre chose à foutre que de bisect pour voir où ça plante.
Ce serait encore plus cool de foutre de vrais messages de commit.
Et enfin, ce serait mega-cool que tu compiles avant de push : "make", puis si ça ne amrche pas, tu regardes pourquoi (ou tu demandes à Google de le faire à ta place)
could have removed the brackets :P
Era ok pentru cine vroia doar lista email-urilor :)
Ultimately we would like to manage all this with a stylesheet.  See #44.  It'll cut down a bit on code size / complexity, simplify the API some, and make these types of customizations easier.
There's a handy method string.IsNullOrEmpty(foo) which is generally preferred over != ""
I'm itching to get this new ccall stuff into master...
this is a terrible and pointless commit
so we can't get this running with the bundles defined in `settings_local_mkt`?
Can you rework this as a (passive) pattern instead of doing it in the core? I want to whittle core down to only basic framework logic and move all logic such as this out to patterns.
It's funny that cppcheck it totally full of unmentionable hacks which you personally and deliberately put in that are far far worse than this. This is minor compared to those and just duplicates existing practice.  When do you plan to revert those?  I guess it's time for me to 'really' leave when you deliberately remove sanity checks because they work.  There is really no point in continuing under these circumstances.
alright, i submitted the changes. i did notice that you may have to change where the client loads the fonts from the cache, in the GameWindow class, also i dont know how to request to add a file to the source, but here it is. http://uppit.com/efkc8eanm6ks/jagex1.jag
Seems like a better default, maybe nil or empty string, is appropriate here.  Magic names are ... never fun.
Just as a fail-safe. Usage _inside_ jclouds that relies on JcloudsVersion should have JcloudsVersion injected (rather than using .get()) and should ensure a correctly-prepared JcloudsVersion is available to Guice
http://pastebin.com/vDVNJRr1

Reverting https://github.com/mangosR2/mangos/commit/3178cc8d18941522fe97f7dfe3c159fd4632b6a2 + [mr1631]
on my server (ytdb) it has 6s cd en this line:

INSERT INTO `spell_proc_event` (`entry`, `SchoolMask`, `SpellFamilyName`, `SpellFamilyMaskA0`, `SpellFamilyMaskA1`, `SpellFamilyMaskA2`, `SpellFamilyMaskB0`, `SpellFamilyMaskB1`, `SpellFamilyMaskB2`, `SpellFamilyMaskC0`, `SpellFamilyMaskC1`, `SpellFamilyMaskC2`, `procFlags`, `procEx`, `ppmRate`, `CustomChance`, `Cooldown`) VALUES
(53601, 127, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1048576, 0, 0, 0, 6);
in this usage check IsWithinLOS nas no sence - already in LOS always.
No, that's not how `find` works either.
Yeah, I didn't realise we already have `is_foo`, so I went looking for something to achieve that. It seems to me that `python <unicode path>` *should* work. Do we just need to do `path.encode(sys.getfilesystemencoding() or sys.getdefaultencoding())` to fix this?
Whoops, this came back to life
Thank you, Coda.
It looks like you submitted the front end resource bundle t for both the admin and front end.  

Unfortunate the back end one is the bigger of the two.

https://github.com/blueriver/MuraCMS/blob/develop/requirements/mura/resourceBundle/resources/en.properties
This line causes all components of Seamonkey to be retrieved. Is there any reason to want a particular subset that requires manual updates?
i had commandbook overriding this command, so instead of editing that plugin, i ensured that this one overrides other plugin commands
 (presuming that they enable before this one...)
Oops.  Should have been blank
const string ClassInheritFrom = "";
My fault - I commited an uncleaned file.
Nop, it seems I miss one of the last Oliver commits :p

But most of the changelog is here :)

On Thu, Feb 2, 2012 at 12:48 PM, Sbastien Coavoux <
reply@reply.github.com
> wrote:

> Finally :)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/naparuba/shinken/commit/93f53af30e5eed5fca8ff811da093c266f62312a#commitcomment-926086
>
было бы круто переписать нотификации в виде декораторов, что бы в декораторе был доступ к инстансу объекта, метод которого оборачивается
we need to reset the `padding-left`. this should be `padding: 0 22px 0 0`.
`p` allows scripts and text so it's lagit.
You're right, I didn't realized that the behavior you've pointed were in the constructor, I thought it was in the validate method.

Thanks for the patch, I'll merge it :-)
Considering RubyInstaller is ì386-mingw32` platform, perhaps you can target any possible compilation of gem to it instead? Just change it to `i386-mingw32` or `x86-mingw32` as it will be used by the gem package process.
Rather than handling the `~` character here, how about changing `rewind` to `fast_forward` and using that to move the pointer one step forward in `char_autolink_username`?
+1 on better error string names.  suggest `DISABLED` and `NOT_SUPPORTED`
Ahh , great, yea I had actually planned on working a little more rebasing into this and making a proper name-spaced class/set of classes sometime very soon, just been hammered the last few weeks. I see light at the end of the tunnel though and will jump back in this soon as I have some other ideas to get pushed too.

Thanks!
we should probably target IE 7 and 8, right?
It doesn't really matter what the event is declared as, since it is created as an EntityCombustByBlockEvent it is such. If your instanceof check doesn't work then either the event is being created somewhere else, or your check is wrong.
no ghost=deleted account on git
Потому что он новый (несозданный!), у него нет ```created_at```. для этого я тебя и просил вывести @page.comments
This is a usability regression. It will break this scenario, for example:

* User sizes his window to the size he wants
* Then opens config dialog and plays around with the font size, using "apply" button in config dialog, until the font is a correct size so the entire page fits in the window

I hate programs restricting me unnecessarily, and I don't see any benefits from this change. A user surely sees that the window is too small, so he has two choices: a) make it bigger b) use a smaller font. This change helps in neither.
I don't like assert private variable, because it is abnormal use.

If `@@configuration` should be accessible, using getter would be better.
If not, this test should assert behavior (in this case, interval of execution).
sure, my bad :(
This could be inside the if.
hmm. ok.
Slang term for please :), I would love to have this pulled in as well
Sent on the TELUS Mobility network with BlackBerry
if there's no objections, I will merge this into the master branch soon.
I don't know if csc allows it, but I can see some uses for response
files within response files. Checking that it's not recursive should
be fine, because the error will be cryptic if someone does it
accidentally.

On Fri, Mar 9, 2012 at 9:59 AM, slide
<reply@reply.github.com>
wrote:
> I thought about that as well, I guess I could just check the that response file currently being processed is not the same as the one requested to be processed. I thought it would be useful to have response files in a response file, but maybe not. Does csc allow response files in response files?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/slide/ironlanguages/commit/5b0239ab0cb4277df6c53d88a24630be696266bc#commitcomment-1066699
show post delimiter author
Change the language to 'All', if an md5 value matches multiple languages. Don't just change the default language to another type.
Nice! This has been on my todo for a while now! Thanks!
The code for prune_dead_workers gets objects of type Worker by

          all_workers = Worker.all(self.resq)

`Worker.all()` in turn calls `Worker.find()`, which used to call the Worker constructor.  By default, the 
worker constructor assigns the worker object the hostname and pid of the calling process (current 
behaviour in master, maintained in this revision if pid and hostame are omitted.  The changes in 
`Worker.__init__()` allow for any code that creates workers to set the pid and hostname for the worker object.

This is especially useful if we're creating Worker objects by reading the redis server, since they might 
possibly refer to remote objects, and so we need to set the pid/hostname externally.

That way, 

              bits = worker_id.split(':')
              worker = cls(queues, resq, pid=bits[1],
 	                    hostname=bits[0])

I'm actually not too happy about this snippet, but it works as long as the __name of the worker__ 
itself does not have any colons.  A better option would be if worker_id was a tuple (or namedtuple
) of (hostname, pid), but that would be a more involved change.  Also, the variable name `bits` is pretty
misleading, and will be changed if you think there is value in this commit.
umm what did i do?
Why store this on the window object?
#replace this value--> if type == "normal": by --> if Type == "normal":
$tpl+$prefs doesn't merge both arrays correctly

for example when enabling Prowl with apikey the user posted apikey gets overwritten bij '' from $tpl
escape $patdata['pid']
Here is the commit that ensures all the code is compatible with the new security model. All you need to do is to cherry-pick it over to your branch that your above commit is in. The changes are very extensive. Even though I've tested it a quite a bit, please test it out a bit:
http://github.com/bradymiller/openemr/commit/598539bda1fa0cbbef9da02236550c56674af8c0
"This problem statement is the exclusive and proprietary property of TopCoder, Inc. Any unauthorized use or reproduction of this information without the prior written consent of TopCoder, Inc. is strictly prohibited. (c)2010, TopCoder, Inc. All rights reserved."
Added error catching for sqs results in db29c728c2732d2c2be1e959d98e987a5e94d4d3.
Uh, problem with indentation. Have to fix it.
Both excellent points; I'm not sure.  Experimentation might be interesting, but I think what you've done here is clear enough.  (Though I admit, manipulating params may have been easier/clearer.)
Note on the password section, it's your job to encrypt and decrypt that password.  DroneD doesn't dictate policy on the configuration.
A code snippet to test with a strange set of nested concave arcs.

        ofPath shape;

        ...

        shape.clear();
        shape.setColor(ofColor(255,255,0,127));
        shape.setStrokeColor(ofColor(255,255,0,127));
        shape.setFilled(true);
        shape.setArcResolution(40);
        shape.arc(0,0,0, 300, 300, 40, -40);
        shape.arcNegative(100,100,0, 150, 100, -40,40);
        shape.close();

        ...
  
        shape.draw();
A few comments:

 - This patch obviously includes several unrelated changes all bundled into one which isn't good practice. In this case the clk changes and git ignore should be separate.
 - What's up with the renames of .ngc ?
 - You seem to have included backup files by mistage (the coregen~ file)

If you don't know it, you should look into git citool which allows to graphically pick and review changes to put in a commit.
Using git commit -v  (verbose) also allows to get a final view of the patch before commit.
yes.  

(As some background...I tend not to worry about calling getClear() in functions that aren't time critical, and I also tend to use LINK() rather than getClear() inside the body of functions to avoid the likelyhood of following code using a value that has been cleared to NULL.  )

However for returns it would probably be better to be consistent.  I'll change.
@Andorbal you mistyped ColumnModificationType see code comment :)

As a side issue could you also fix following issue Alter.Column("Label").OnTable("SomeTable").AsAnsiString(40).WithDefaultValue("em'ty"); 
As this generates syntax error on sql, this generates following sql ALTER TABLE [dbo].[SomeTable] WITH NOCHECK ADD CONSTRAINT [' + @default + '] DEFAULT('') FOR [Label]'
This would be changed from $class->name to $class->fieldMappings[$fieldName]['type']
actualiza con los comentarios, si quieres dejar la db así como esta me late, y ya para hacerle merge, muchas gracias
Cookies and Headers are also used by Response. Maybe they should get there own directory?
comment seems out of date
Thor currently uses chksum16, I don't know which is preferable,
but 1 seems enough.
I don't understand the point of the double check. In particular why not just the second one? 

Worst that can happen if get it wrong is that peak memory usage calculation assigns a block to the wrong activity (and unless it's an atomic snapshot over all active blocks it's always going to be potentially slightly inaccurate).
1.0. nice
Hmmm.   Maybe I don't understand all of this yet.  But I think you want this instead:

      if (next_target.target.all_relative)
          next_target.target.X += startpoint.X ;
      if (next_target.target.X < X_MIN * 1000.)
          next_target.target.X = X_MIN * 1000.;

Because if you run into the software endstop, you just want go to MIN, right?  Not MIN + startpoint.  In other words, I think the result of running into an endstop here should be the same regardless of whether you are in all_relative or not.  However, the initial calculation should be different.

And I'm not sure I understand what happens after SpecialMoveE, so I may still have this wrong.
Nope as I uses it from goldenleaf. 32
On Jan 14, 2012 11:12 PM, "Rockr172" <
reply@reply.github.com>
wrote:

> Mcs6000  driver updated eh?? So i guess this kernel can be used with ICS??
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/aspee/lge-kernel-msm7x27/commit/3bbcaa7acf883b6f88497ef07603700df9f45d1d#commitcomment-867508
>
only other thing here is that I think this value should be set only once (ex. the supplier should lazy create a value and only ever return that value).  I think there's a pattern for how to implement this somewhere, and you could ask on guava, etc.  but probably you need to make a field with AtomicReference<Server> and then lock and set that if not yet set.
これもっと短く書こうとすれば

```scala
 champion.exists(_ == other)
```
ってかけますね。 **え?だからなに?** っていうくらい微妙な違いですが・・・
Nil isn't a PHP constant and is throwing a notice.
ok, after 2 commits I got it right.. thanks, bye
Eh, no particular order right now.
With javascript, I am way out of my comfort zone and I my efforts are not working.   Can someone spruce up line 262 to make this work and it should be good for testing.
I think you closed the wrong issue here.
Sorry, I just removed this line. I missed to run tests. I will be pushing some fixes right away.
Sweet as, it's not critical.
page and product_post should have tests.
def blablabla(text)
.... so LG-P690 need another fix... and can i ask you one question?
How can you bring long battery life on your kernel?
One of my kernel user told that your kernel have a big battery life
okie cool, I'll whack #SILVERLIGHT defines around it tonight.
This seriously confused me, to the point where it took me a while to even realize I was thinking about it wrong. I thought that any object conforming to MAKVOKeyPath would be able to be a key path, which is to say, you could pass `@"foo.bar.baz"` or `ARRAY(@"foo", @"bar", @"baz")`. But now I see, assuming that I'm understanding it correctly, that key paths are always considered to be period-separated strings, and passing in a collection just makes an observation for all of the key paths contained within the collection. If that's so, then I think this protocol's name needs to change (not entirely sure to what, but maybe something like `MAKVOKeyPaths` or `MAKVOKeyPathSet`) and the comment also needs to change to better indicate the semantics.

Additionally, given that the returned object is only ever used in `for` loops, how about making this return `id <NSFastEnumeration>` instead? This would allow greater flexibility in the implementation of this method without any downsides (that I can see), and would make the implementation of this method on the various non-`NSSet` objects simpler, since they could all just return `self`.
Could we remove the automatic installation of jQuery and jQuery UI and leave that to the developer/designer to implement? Its a UI thing and am thinking its a bit too much for this given many ways out there to use it.

Let me know your thoughts please.
Symfony CS is camel-case and no underscores.
Besondere Preise:
good sir, this looks very interesting
That's a major API change. You're saying it needs cleanup, but I'm not quite clear on why its necessary in the first place. Do you have an example where the method is using those arguments?

getCMSFields() doesn't need to know about its controller or form context before Form->loadDataFrom() is called on the resulting FieldList (outside of this method).
That was me that made these changes. I was on a desktop computer instead of my laptop.
Is this in any version? I just upgraded and issue #80 which is the same is not fixed yet.
k ... will fix it
Sounds good, I just find it is easier to explain/use parts like this if you don't have to make a choice (and easier to write docs/examples if you don't have to explain an either/or).
Actually I have to change this. It will always fail since only one directory will ever be there.
Instead of "in a second" could we be more precise? Maybe "once you're done"?
Is array access just for 3rd party plugins who might be reading the array values? I'd prefer to not need these if possible.
No biggie but mySelf is a single word and should be 'myself' - looks ugly now. 
Where is controller defined?
leave this off: it is implied
Maybe you can add '_poissonian', as for the hazard curve calculator. In the feature we may have stochastic event set generators that do not assume a Poissonian occurrence model
transitively
You're correct, I made a silly assumption here. I've reverted the fake response locally and tests are passing after unescaping the XML. I'll push that up soon, Thanks!
No, thanks for catching it
Counter, unlike defaultdict, doesn't seem to be imported. Should I import that specially for this? What is the advantage of using that over defaultdict?
I'm confused, if we are providing a proxy to access remote resources that shouldn't affect the check of whether the plugin can preview the resource or not. Perhaps it makes more sense to put this notice on setup_template_variables, so people can check if they need to change the url.
I still advocate for using the recorder instance in the base class and checking what tags were written to the screen over the mock. May not be necessarily applicable in this case, but if there was text before the failure, the tags would let you assert that was printed as well. I also fear our tests are getting really mock-heavy and tend to shy away when we have a construct in place that would provide the checks we need.
According to an email from slagle to pulp-list on Aug 2, output strings have to be directly wrapped in the _() method, and it does not work to store them in a variable and wrap the variable as we have here. I can't find this in the gettext documentation, which is why I'm citing slagle.
Another way to achieve this is to use the **kwargs magic and then check if the keys are valid. Keeps you from having to define a global. It's not really a necessary change, just a different approach.
this needs to be on the `<body>` b/c when we run this manually it's a good idea to visually check the spinner. the test is helps, but there's a lot that it can miss.
another thought: we may be at high risk of breaking old migrations bc of these changes.  should test direct migration from 1.8 before releasing.
`#!/usr/bin/env python` is more standard and a safer bet.
Era pra ele se auto ignorar, subiu nem sei porque
throttle and unthrottle should be private[this] to prevent ant race conditions
Sorry for sacrificing the readability. Its initialised on line 143
You can use 'cenum' for all this btw...
to have a more logical order
Oops, I'll remove it right away. That was from `enclosingScrollView`.
Inserting unicode characters directly into the source [isn't](http://developer.apple.com/library/mac/#documentation/Cocoa/Conceptual/Strings/Articles/FormatStrings.html#//apple_ref/doc/uid/20000943-SW2) recommended. 

Try this:

    NSString *placeholder = [NSString stringWithFormat:@"%C", 0x2022];
il ne faudrait pas un index sur token?
I don't think we should keep this difference. If there is not strong case _for_ having a different behaviour, I suggest PlaceboTransaction should behave just like a TopLevelTransaction, in this regard.
@hanke: could you please send an updated change request (and perhaps test in your scenario with condor)?

```
if sys.version_info < (2, 7):
    from ordereddict import OrderedDict
else:
    from collections import OrderedDict
```

@moloney: the OrderedDict comes in when the config is read in config.py?
Does anyone know how to get the datasink to ignore all the folders created by a MapNode?
This inadvertently outputs the meshes inside folders (named _smoother0 to _smoother44) inside meshes.all
Out of curiosity, the last two tests look to do the same thing as the first two. What am I missing?
Other things to check:

* 0-d array (the only valid index should be `()`, scalars and non-empty tuples should be errors... I'm not sure if the code above gets this right actually)

* some mixed dtype calls. Actually I think the code right now will produce the following (wrong) results because of the FORCECAST thing:
```
>>> a = np.array([2])
>>> a[0] **= 3.5
>>> a
array([11])
>>> a = np.array([2])
>>> np.power.at(a, 0, 3.5)
>>> a
array([8])
```

* Fancy-indexing with multidimensional index arrays.

* Boolean indexing.
This line should be here
Is this `if` really necessary? I think this PR also needs tests and documentation.

Good idea BTW!
`#respond_to?` checks whether a given instance respond to given method:

```ruby
Array.method_defined?(:to_a) #=> true
Array.respond_to?(:to_a)     #=> false
Array.new.respond_to?(:to_a) #=> true
Array.respond_to?(:new)      #=> true
```

`Array` is an instance of class `Class`.
These `merge` calls in the methods above could be actually `merge!`, to avoid creating two of them on every call.
Yes. This is a nice refactoring
I guess making HashSet1 and HashTrieSet private[immutable] is out of the
question because of binary compatibility? Then I will make all three
overrides of filter0 protected.

On Mon, Aug 6, 2012 at 11:57 PM, Paul Phillips <notifications@github.com>wrote:

> In src/library/scala/collection/immutable/HashSet.scala:
>
> > @@ -247,6 +263,60 @@ object HashSet extends ImmutableSetFactory[HashSet] {
> >        }
> >      }
> >
> > +    override def filter0(p: A => Boolean): HashSet[A] = {
>
> No idea. None of those xxx0 methods should be public; so let's not create
> needless future difficulties in the name of an unimportant consistency.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/scala/scala/pull/1068/files#r1316303>.
>
>
Please, never, never, never "catch Throwable => silently discard." Never.
Indeed. If it's in the same namespace, just add the backticks.
If you want you can create unit without system, as is done at the beginning of the mks file. In principe one can wish to use these units without system, and only later chose one (I recognize it will be a rare case). Another example is the one I give in the documentation, where you first use the astronomical system and then the mks system: without the global set function you will have to change the system for each unit (at least for now I see no other way).
I think this can even be missing in the base `Operator` class as the `Tr.doit` method will not call it i it doesn't exist.
linmon -> linearmomentum
or

converter = {} # see sympyify docstring for description
Yeah.  For some reason, I did not know that at the time, so all my strings have this.  Let me go through and clean them up.
You can't use this, unfortunately, because it doesn't work in Python 2.5.  I guess you can chalk that up as another useful feature added in Python 2.6.  You could add something like it to compatibility, though like I said, I think you should actually just remove all the warnings here completely.
Are you sure it doesn't just mean apply horner's rule.
>
> Again, thank you very much. I really appreciate all the work you are
> putting into this even though you have hung up your hat.
>
I'm trying to stay out of the ring, but haven't left the gym. If I
can't keep from putting the gloves on too much then I'll have to leave
the gym :-) So if you want I can continue the review as I have been.
You got some other good review, too, I saw. I'm glad for that. I can
probably finish up this 2b in the next day or so.

/c
Here (and throughout) the doctester will normalize whitespace inside of lists and dictionaries so you can write these each on their own lines.
Yep, coercing strings to numbers.
Something's wrong with formatting here. Make sure you're using tabs.
Looks like it's incorrect in many places since method is in lowercase http://www.yiiframework.com/doc/api/1.1/CDbTransaction#rollback-detail
You cannot use tables.php to get field names. The reason is we're getting rid of tables.php. Field names are fixed in D2, there is no need to do any lookups.
How then do we avoid the compile errors we've been getting by leaving them in?
I would think so
They *should* be datetimes but in our current schema they're dates.
Removed the `$hookArg` assignment and simply call the function.
Can you hyperlink method names?

eg. instead of `grunt.file.expand` do [grunt.file.expand](api_file.md#gruntfileexpand)