labofsahil/github-event-dataset-2012 huggingface dataset:
MemoryUsageStress

This simple servlet is used for stressing memory.
 
The objective is to force JVM to allocate a given amount of memory (in megabytes).
After the memory is filled up, we wait for a specified number of milliseconds before dereferencing.
E.g.:
```http://localhost:8080/clusterbench/memoryusage?milliseconds=20000&megabytes=500```
will allocate 500MB and keep them for 20000ms.
Warning: Obviously, if you set *megabytes=1000* while having *-Xmx512m* you will experience an unpleasant and utterly inevitable OOM.
Note: Naturally, if -Xms < -Xmx, JVM will be allocating more system physical memory.

The results of this call is a response like:
    
    BEFORE:Physical: TOTAL: 3760MB, FREE: 474MB (12%)
    Runtime:  TOTAL: 309MB, FREE: 280MB, MAX: 1159MB, TotalOfMax: 28%
    Heap:     INIT: 256MB, USED: 28MB, COMMITTED: 309MB, MAX: 1159MB, UsedOfMax: 2%
    NonHeap:  INIT: 23MB, USED: 67MB, COMMITTED: 67MB, MAX: 304MB, UsedOfMax: 22%
    AFTER:Physical: TOTAL: 3760MB, FREE: 52MB (1%)
    Runtime:  TOTAL: 809MB, FREE: 278MB, MAX: 1159MB, TotalOfMax: 73%
    Heap:     INIT: 256MB, USED: 531MB, COMMITTED: 809MB, MAX: 1159MB, UsedOfMax: 48%
    NonHeap:  INIT: 23MB, USED: 67MB, COMMITTED: 68MB, MAX: 304MB, UsedOfMax: 22%
    Duration (including warm-up): 20283 milliseconds
Consolidating forward range requirements

Spliced from another pull I messed up beyond repair.

Basically, `save` doesn't require an actual type match, but merely implicit cast back to the original type.

This can be problematic if the save is stored in an `auto`, or forwarded straight to a template. In particular, it may instantiate the wrong template, or even fail compile if the result of save is some sort of weird non-range proxy. Things get especially funky if you pass the save to a take or similar.

Keeping this behavior means that you pretty much have to throw `auto bck = r.save;` out the window in favor of `R bck = r.save;` or `typeof(r) bck = r.save;`. More importantly, the one-liners `foo(r.save)` becomes non-valid, for the cases where foo is a template.

This pull identified a invalid forward range (ByCodePoint), which is fixed in #1006

Unfortunately, like the hasSlicing pull, this pull is still subject to 8847 http://d.puremagic.com/issues/show_bug.cgi?id=8847 ...

So I'm mostly posting this pull request to "get the ball rolling" regarding discussion, and to increase importance of 8847.
@trusted pure nothrow const in some parts of std.variant.

It's hard to go further than this because of the weird internal design of `VariantN` and the low-level calls it does.
Adding explicit dependency to DoctrineORMModule

This one was missing ;)
Show case updates

My first contribution to the Maps API.
I added a Map Widget where you can load and unload multiple kml to a map.
A preview of this map can be found at the bottom of this website.

http://1.maps-api-additional.appspot.com/

Dennis
javascript 버전 추가했습니다.

이렇게 하는거 맞는건지 ㅠㅠㅋ
Resolve foodcritic warnings

See related Opscode ticket:
http://tickets.opscode.com/browse/COOK-1428
Use ProxyInterface from Doctrine Common

This was enough to fix [Issue #255](https://github.com/doctrine/mongodb-odm/issues/255) for me.
SVN R16

Elvileg alap cuccok most kerülnek: be.
Updated reboot strings for tablets

I noticed some incorrect strings on my galaxy tab. So I decided to be proactive and fix them myself.

Apologies if I've gone about this incorrectly, I've never tried to contributed to a project like this.
Revamp todos demo to use "empty" app
Added support for maxContributors

This closes #1.

Usage:

```coffeescript
contributors.generate 'jashkenas', 'coffee-script',
  placeholder: 'coffee-script-contributors'
  maxContributors: 30
```
Changed a line

No idea
Remove superfluous type-index class

The `data-role="page"` div's on the global config test pages have a class of `type-index`.  I think these got in here by accident since the only other page div's that have that class are on the various index.html pages.  This patch removes the class from the global config test pages.

Edit: better description
3.x testing

Hi @alexbevi: project dependency updates for 3.x branch. Routes now show:

projects/:project_id/articles/:id, and projects/:project_id/categories/:id. Home page is at projects/:project_id/knowledgebase. 

I'd appreciate suggestions if you have any on getting the "knowledgebase" into the URLs for 'categories' and 'articles'.

However, these routing changes seems to break new ratings code: 

```
Started GET "/projects/cd/articles/3" for xx.xx.xx.xx at 2012-11-01 15:43:04 +0200
Processing by ArticlesController#show as HTML
  Parameters: {"project_id"=>"cd", "id"=>"3"}
  Current user: duanejevon (id=3)
  Rendered plugins/redmine_knowledgebase/app/views/articles/_rating.html.erb (127.6ms)
  Rendered plugins/redmine_knowledgebase/app/views/articles/show.html.erb within layouts/base (16025.5ms)
Completed 500 Internal Server Error in 16546ms

ActionView::Template::Error (undefined method `rate_article_url' for #<#<Class:0x000000057dd6b0>:0x000000057f4130>):
    12:     callback: function(value, link) {
    13:       $.ajax({
    14:         type: "POST",
    15:         url: "<%= rate_article_url(@article) %>",
    16:         data: {
    17:           rating: value
    18:         }
  app/helpers/application_helper.rb:1174:in `content_for'
```

Any ideas on how to fix?
Removing formatString calls and multiple function signatures.

To make this compatible with less.js, I removed the formatString calls and multiple function signatures.
LPS-31626 Add Open In MS Office button when viewing document

Resent of https://github.com/brianchandotcom/liferay-portal/pull/8196
866579: Fail fast if the user enters a bad environment.
REST interfaces corrected

...to remove address from all services
Utilize Doctrine common for mappings

@stof: Please take a look at my revisions to the test class.

The only question I have is that I had to flip the key/value pairs from the previous tests in order to work with the new locator service. Does this pose a problem for the extension class, or does the bridge take care of the change for us?
Fix #284 -- display SplashInfo statistics on dashboard
Add a begin time to AnnouncementLinks

Add to Entry as well, because even though those are unused, it will still generate errors if we don't add it.  The field is highlight_begin, in correspondence to highlight_expire.

Someone should check that the migrations work because I don't really know what I'm doing here.  Other than that I think this is relatively straightforward (famous last words).
docsprint application
Update to version 13.00 - this time with a tag

I forgot the tag in the previous request.
Improvements to ElementSelector and AttributeSelector
Add -libpath option for alternate native library location in case module/main/lib does not contain native library

Patch allows to add alternate path for System.loadLibrary resolution in case user needs a separate location for native libraries or their dependencies.
In case native library is not present at the expected location inside module/main/lib/os-arch directory and the given command line option is given that second location is checked
SAKIII-4754 - Limit the amount of JavaScript globals.

Jira: https://jira.sakaiproject.org/browse/SAKIII-4754
CLI fixes

CLI fixes: do not create OS disk in $root as it is not supported, add 'vm export', improving password entering experience
Patch to add repo name to metadata

Hi!

This is a useful module! Thanks for publishing it.

I've added a patch to include the name of the repository in the JSON metadata. It tends to be useful for us in our patch management process to know the provenance of our packages.

Best regards,
 -Bill
Adds keyboardNavigation option

Adds keyboardNavigation option which allows users to disable the arrow key navigation of the calendar datepicker has when the input has focus.

if keyboardNavigation is true, datepicker acts as normal (the arrow keys modify the date)
if keyboardNavigation is false, the arrow keys do not modify the date
trying again

developers
Json API

Hello people,

I just implemented a simple JSON API, se we can somewhat access the data from rails contributors in a easy way. Don't know about server load issues it may causes though...
Expand ~ to home directory using File::HomeDir instead of File::Glob

Using `File::Glob::bsd_glob` to expand the `~` to the home directory seems easy, but as `File::Glob` itself admits, it doesn't work on very many systems. Notably it does not work on Windows by default (it works if `HOME` is set in the environment, but `HOME` it not set by default in Windows).

This changes `~` expansion to use `File::HomeDir` which is much more portable. It also changes `_normalize_id_file` to normalize the directories to that of the system as well.

I have mainly made this change for the Windows perl users. Unless one of those users sets the `HOME` environment variable (not too likely) then the tests won't pass. The tests will also have issues on any system where `File::Glob::bsd_glob` does not expand `~`. Recently I submitted a [patch to perl](https://rt.perl.org/rt3//Public/Bug/Display.html?id=98812) to expand `~` on Windows, but of course that won't help all the old perls which don't do that. Since this module already uses `File::HomeDir` and `File::Spec` it seemed logical to use those modules for the `~` expansion.
Update ring and clutch versions
Foundation and content

Foundation was working beautifully, but the final changes caused it to
dissapear again. I put the content area on all the pages, and I will
not work on programing their specific functions.
Allow Marionette to be installed using volo

[volo](https://github.com/volojs/volo) is a JavaScript package/dependency manager. Adding a `package.json` file with some metadata allows users to install the AMD using volo using `volo add derickbailey/backbone.marionette`.
Added: base script for migration

This is the base script I used when I was responsible of the svn to github migration of eZ Publish.
Feel free to use it as a base, I would also be glad to help on this topic.
Model/Controller for Message and Chatroom

Made controllers for chatrooms and messages so you can make new
chatrooms and messages and have them displayed. No user controllers or
models are made yet, so the create controller for message just makes the
author Bob.

Haven't put in any html.erb's yet.

There's a table I can't get rid of in the schema, called comments which
I accidentaly put in. Don't know if it'll be there for you too. (Is the
database stored in the UWAnnachat folder?)
add link to close only cli related issue
regular default value + memory leak tests

attention il y a un test qui ne passe plus avec regular = true
c'est peut-être normal mais à débugger...
[feature] added pharao mob

high health and high damage
If no hostname is provided, use a hostname without the domain

This matches the behavior of syslog daemons.
Added passphrase, _key (private_key, public_key) to filter list

We store passphrases and keys in our user model, so I believe these should be filtered out as well.

Perhaps it would be best to just use pass instead of both password and passphrase?
Note about CDH

Otherwise, the user gets a client version mismatch error. Regarding the guava jar, the reason is mentioned [here](https://ccp.cloudera.com/display/CDHDOC/Incompatible+Changes)
New way of running the tests

I can't replicate your problems with running the test in Eclipse---it still works for me---but I did find a way to make nicely colored HTML output. Try running "mvn test" and then opening target/cucumber-html/index.html in a web browser.
Make manipulate callbacks consistent

When {pre,post}Manipulate is specified as an array, the callback
expects only the content, but when it is just a single handler
the callback expects an error as well as the content.

I would have preferred to use the nodejs pattern and include the error
but all connect-assetmanager-handlers only pass content so I went the
simpler route for now.
Add a couple of parameters to fastcgi_params.

A couple of parameters are missing from fastcgi_params.
Added a link to the getting started information into the README.
VS11 Beta compat changes

Hi, made some changes to make it work with VS11 beta, It is only targeted to VS11 so please check once before integrating it in the main branch!

-Fahad
Orm updates

 - add version support to definitions/objects
 - optimize orm magic __get()
 - add setting for prepared statements in pdo
Fixed a little bug

If you use the max() scale, the sample image can be smaller than the requested image.
там и так все гуд было
Fix dependencies, provide missing requires

This fixes the issues described in the issues ticket: https://github.com/thegorgon/sinatra-sprockets/issues/1
Feature/testsetup

Initial setup for unit tests
Timob 7189 Button implementation.
Add support for private repos through Github oAuth

Hi Joachim,

I saw your request for adding support for private repos, so I took a shot at it. This branch allows updates from private repositories using an oAuth token, and it provides setup assistance through Plugins > Github Updater.

Once an application is setup and and a token is given, the WPGitHubUpdaterSetup class isn't needed anymore. The token will authorize updates on all sites from then on, and will allow the plugin author to revoke privileges in the future if they wish.

While in there, I noticed that in get_github_data(), wp_remote_get was being passed ```$this->config['sslverify']``` instead of ```array( 'sslverify' => $this->config['sslverify'] )``` so I changed that as well.

Hope this helps!

Paul Clark

PS: While fishing around in the github APIs, I noticed that they allow querying a repos tags, and the response gives all versions with zipball URLS:

https://api.github.com/repos/jkudish/WordPress-GitHub-Plugin-Updater/tags

Looks like the config values for api_url, raw_url, github_url, and zip_url could all be replaced by that one URL, and the readme would no longer need to include a version number (because they're listed in the tags). Thoughts?
OpenPNE3 PHP 標準コーディング規約への準拠

OpenPNE3 PHP 標準コーディング規約
http://www.openpne.jp/coding-standards-ja/
に違反している箇所を書き換えてみました。

ただし、完全に修正したわけではなく、とくにテンプレートのPHPファイルについては修正していないところがかなり残っています。

意図的にコーディング規約に合わせずに書いているのであれば取り込みの必要はありませんが、OpenPNE開発に関わるより多くの人に読んだり書いたりしてもらいたい場合は、できるだけこのコーディング規約に沿った記述が望ましいと思います。
Refactoring validates_each method

This pull request is only to refactoring the validates_each method (and remove a lot of returns).
Write out request times to a csv if output_dir is specified

It would be nice to have all request times. This is implemented just like the ``query_count`` plugin does this.
logo suggestion

You can see a rendered version here:
https://plus.google.com/u/0/117179731755598592796/posts
[Lighthouse-59] Remove the button from the nav via the WP-admin
Pluralize

I added a small snippet to the pluralize function to take the character preceding the "y" into account when pluralizing. 

Strings that end in "ey," "ay," etc., will now correctly convert to "eys," "ays," etc.

Strings that end in "ty," etc., etc., will still correctly convert to "ties," etc., etc.

Small addition, but this will allow the plurals that display in the admin menus and on the front-end to behave in a more grammatically correct way.
Added croatian

Translated app into croatian!
Упрощение анимаций

Алсо, как там у тебя с хромом? Работают анимации появления? А то у меня появляться с анимацией не хотят нивкакую. При том все остальные анимации работают.
Переведен текст вплоть до начала первой главы

Сырой вариант, нужна вычитка.
Formatting changes to  README only
Added seed data

### Changes
* Added Loader class which implements ApplicationListener and runs once on application startup
* Added also some seed data in Loader.java
* Seeding data has no username, refactored BaseEntity to make it work, e.g username now defaults to 'system'
* Cleaned tmp files and added them to .gitignore
After applying this commit the sound goes bye bye ... :)
Yes this will go into eden...
More prominently, though, application/json requires pre-flighted cors, while text/plain goes through the much cheaper simple process, https://developer.mozilla.org/En/HTTP_access_control#Simple_requests.
ok, but do we bundle and redistribute this runtime version?
The manifest says version 9.0.30729.1 token 1fc8b3b9a1e18e3b but I can't tell from the name of the redist installer (vcredist_x86.exe)
I see. Will change later.
meh, I'm not that paranoid/enterprisey - feel free to change it though
U tohohle řádku mě napadá, že by to mohla být docela brutální feature :) Už jsem se párkrát setkal s tím, že jsem potřeboval procházet dvě pole zároveň.

    {foreach $arr1 as $item, $arr2 as $key => $value}

Co myslíš @dg ? :) Možná je to přehnané...
Just so you're aware: I'm going to change this to run the "non-destructive" `bundle install` command instead. By running `bundle update` at this point, gem dependencies *may* accidentally be updated. I don't think that we should run `bundle update` here at all.
AFAIK, it's not possible to get rid of the casts in general, but of course you're right I can avoid the ugly reinterpret_cast making it more type-safe (although making in it more runtime-burden ;-) ), I'll push the alternative solution...
There should be in comment @param vm:....
@EdGruberman
Oh, I'm sorry then. I assumed it from the <data> field, which should work, because it's short, not byte.
OK I'm now storing the build log in `/tmp/php-build.<definition>.<datetime>.log`
thx, fixed (I also found another instance of the same problem grepping! ;)
jaj, mergnu
I might agree that bool parameter is unclear and not descriptive, but I rather would not change it since that needs changes on other places in code that are not related to this patch. But , if you insist that can be done.

For Daniel: The true value is used to force li macro not to optimize immediate loading, so It always use lui/ori  pair to keep instruction sequence constant,. This is needed for code patching at some places. The same places must not be replaced with load from root.
hmm starting with this commit the password will be send from the browser to the server twice? in hashed-sha1 form and also in plain text from the origin text field?
Just a test so that I can see what GitHub does with comments.
Canbemerged!
@pmjones I guess you want to place a package folder with empty file . 

Or we want to update the https://github.com/auraphp/system/blob/master/update.php#L29 and check whether the directory exists or not else create one . Which one do you suggest ?
You're absolutely right Andreas. Will make this change.
don't understand what this is supposed to do.
json.dump(result, sys.stdout, indent="    ")
Thanks for the tip. 

This works great but actually not when the search input is present. I modified the code (in jQuery flavour) like this to only depend on the results list height :

```javascript

 Chosen.prototype.results_show = function() {
            var dd_top, drop_bottom, hidden_drop, drop_results;
            if (!this.is_multiple) {
                this.selected_item.addClass("chzn-single-with-drop");
                if (this.result_single_selected) {
                    this.result_do_highlight(this.result_single_selected);
                }
            }
            dd_top = this.is_multiple ? this.container.height() : this.container.height() - 1;
            this.dropdown.css({
                "top": dd_top + "px",
                "left": 0
            });
            this.results_showing = true;

            // See https://github.com/masylum/chosen/commit/1ba161d5511b694a10e67ceff64a318245cebcaa#diff-0
            // for initial enhancement
            drop_results = this.dropdown.find('.chzn-results');
            drop_bottom = drop_results.offset().top + drop_results.height();
            hidden_drop = drop_bottom - $(document).height();
            if (hidden_drop) {
                drop_results.css({height: drop_results.height() - hidden_drop});
            }


            this.search_field.focus();
            this.search_field.val(this.search_field.val());
            return this.winnow_results();
        };

```
this should read `#!/usr/bin/env rspec`
Do you mean add `encryptKey` and `clientSessionDuration` for 0.10.x, and then remove them again for 1.0?
I'd prefer to have the 1.0 API as clean as possible, are you opposed to this change between 0.10 and 1.0?
@MBAACC `<h3>s1asdasd</h3>`是什么玩意儿？ 
@pzi1win2go3 @Th0ar 必须创建`@newss`才能实现分页吗？不能下面这样吗？

```ruby
<%= render :partial => 'newss/news_index', :collection => @org.newss, :as => 'news' %>
<%= paginate @org.newss %>
```
 这样是不是更简单一点？
Very nice work. 
Do you do these by hand or use some program for generating SAIs?
Ah, yes! This makes the shared library file considerably smaller.
Just a test file to be deleted.
I might recommend `.merge_options` for this name. When I saw the word "update" I immediately thought maybe it was modifying the options passed in.
好的 因为我看到提示老是说 提交太多 还有请查看票状态 也有未知错误 刚看到你更新 所以想用一下
whoops, forgot to mention apostrophe in the ```after => obeschay-mne-sdelat'-shag-i-ya-sdelayu-dva```
That is thanks to the status barlet's code.  Every action in the volume barlet calls `refresh()` at the end to update the display.  If no actions occur, the display is refreshed anyways periodically (every 30 seconds in my config).
please fix indentation here.
no messages like this in R2 test and my local. i think, problem in datatype in your DB. logic of check fully equivalent default mangos, but check maked much more correct.
as far as i know count is checked in client and crash occurs because of wrong limit category sent in Player::SendEquipError

http://paste2.org/p/1887753
files typically end with 1 newline. this is 2.
Ah gotcha thanks.
Hi Petter,

13 01 2012, 22:54 Petter Urkedal <reply@reply.github.com>:
> Hi Ivan,
> 
> On 2012-01-13, Ivan Maidanski wrote:
> > The latter seems to me not logically correct as the object is marked (somehow  - it is implementation-dependent).
> 
> My take on it is that GC_set_mark_bit and GC_clear_mark_bit does not
> change the object as seen from the users.  Rather they change the
> collectors idea about its reachability, which is dependent on an
> external link to the object, rather than the content of the object.  The
> way I use it myself, objects are never changed once constructed, except
> possibly for an internal next-link.

Hmm... Looks reasonable, thank you for the explanation. I agree with you now.

Regards.

> 
> > I've committed this patch (adding const to GC_is_marked).
> 
> Thanks, I don't mind the others being non-const.  I only call this from
> low-level allocation and clean-up code, which already contain uintmax_t
> to void * casts.
> 
> Petter
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/paurkedal/bdwgc/commit/9d07ead2d0bc217a3ded434372952f7a8db6c17a#commitcomment-865805
>
в этом методе 6 строчек кода, но на самом деле можно обойтись одной.
return true/false не обязательно возвращать (просто достаточно добавить ошибку)
errors.add(:base, 'Line Items present') if line_items.any?
ok, I've removed the method.
Any chance of getting access to the Heroku TCP router private beta? :smile:. Setting up ZNC on Heroku is something I've been wanting to do for a long time.
Correct me if I am wrong, but the word "constructor" is never mentioned in the document, and the example shows a function, instead of a simple  object. Somehow I missed the connection ---- my apologies. It didn't help that I didn't get this.forEach() either --- who calls next() the first time? who resolves a promise, if not next()? Probably I need to reread the whole thing again and again.
I'm using your patch and it works almost great! One thing that is missing is when the user receives a new email (when the account is created) it references to the email_host part :S and is really misleading... I would think that the URL in the link "Click her to log in" would be built from the host_www config but it's not... but otherwise the patch is working great! I think you should send it as a pull request...
${number:defaultvalue} je kada zelis da obezbedis default value, za vrednost koja ce biti menjana.
It would be a little cleaner to do `'false' if request.args.get('autoeval', 'true')=='false' else 'true'`
Use here that your type ButtonId
It's good : always code test before real code ;)

Does it fix a view in multisite or Thruk?
Hmm, works for me(tm). What swig version are you using?
Something different than what crudify provides?  https://github.com/strangeloop/refinerycms/blob/master/core/lib/refinery/crud.rb#L205 .  I will create a ticket on the show current conference year stuff.
теперь они в виде болтающихся в воздухе строк в одинарных кавычках
If you would like to make this consistent with the MSI installers the code should be as follows.

```
WriteRegStr HKCR "tundra" "" "URL:tundra Protocol"
WriteRegStr HKCR "tundra" "URL Protocol" ""
WriteRegStr HKCR "tundra\DefaultIcon" "" "$INSTDIR\data\ui\images\icon\TundraLogo32px.ico,0"
WriteRegStr HKCR "tundra\shell\open\command" "" "$INSTDIR\tundra.exe --login %1"
```

So we overwrite each others registry keys with NSIS vs MSI and other possible clients like Adminotech Tundra. Otherwise windows might get confused or possibly suggest a list of software that are registered to tundra://? Am not sure if the difference in the first key matter or the " quotes around %1.

Dunno if this needs to be changed, just thought you might want to know the way Tundra MSI installers do it.
eh -- I guess it is again my fault confusing you and myself as "documented" in 
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=602016

probably we cannot run iptables here because process would not be running as root and initialization of the new chain is up for the admin to configure statically in his firewall,... am I right or am I missing smth again?
Awesome! Based on this great work, I guess I can implement per account browser settings and add Firefox support too. ありがとうございます。
ой, а как это сделать в данном случае? :-(
regarding your below comment, does this also need to be dropped?
OK. Fine.
Particularly given we've already release a stable version of CMake with that functionality. We can't just break it now.
Whoa whoa whoa, this is a @tbugai joint?
should this be search_query without the @?
Won't these changes break backwards compatibility of the REST API? Note that customers are using our REST API.

I am forward porting other REST changes and adjusting them to this new layout. If this commit gets reverted, those forward ported changes need to be adjusted correctly: more specifically, in that case binaryContentAttachmentFileName should come on AssetMetadata, not Asset.

Original changes on 5.3.x:
https://github.com/droolsjbpm/guvnor/compare/07bf9bf...315199c
Forward ported changes on master:
TODO
Good idea, but the indentation is scrambled and it doesn't make sense to have a socks proxy per-pool. It should just be one global variable like opt_socks_proxy, referenced in miner.h, and used to communicate with all pools. Then it won't need to be passed to json_rpc_call either.
this check needs to be *before* if (IsPeriodicRegenerateEffect(spellInfo, index)) at line 226,
IsPeriodicRegenerateEffect return true, since those spells do regenerate periodicly ...
This should be offsetFromEnd = 30 or 31, because a \n has been added now at the end of each line.
Was there a problem with using the `arrayForEach` form of exporting? The reason the code was structured that way was just to reduce the resulting code size slightly. If there's no problem with `arrayForEach`, could we change back to that style in this one file?
What do you mean? We're not deleting groups yet until the end.
parentheses aren't required
you don't need to import `AddonDeviceType`
a simpler way to test this:

    eq_(list(webapp.device_types.all()), [addondt])

coerce the QuerySet to a list and compare it to the expected objects.
why the extra checks? Do you think someone will mistakenly pass an array into this?
1) We do these changes to have less bug at updates.
2) We try to group all of them together when we do them.
3) The plugins depending on this are going to break anyways at an update.
4) Officially if a plugin decides to not build against Bukkit but against CraftBukkit it is their own risk.
I always thought that the release means that **I** do not need it anymore, but it is not deallocated until it is out of scope. Can you explain the specific issue? I also did not understand "take care not to touch and members ect". Sorry for so many questions about such trivial issue, but I am eager to learn something new.
Why we define 9p_immediate_writeout here ? If we want the default to be changed i guess we should change it in guest-hw.cfg ?
extra space between 'var' and 'surface'
This can be simplified to:

```javascript
return Q.when(connection, function (connection) {
    return _Connection(connection, local, options);
});
```
I can't find the original source of the README anywhere in the history of the gen-master branch - did it ever make it into that branch?
ah, I was planning to back it up under a Tatoos type. I must have completely forgotten it.
SecureRandom +1
Можно ещё поиграться с сериализацией объектов, чтобы делать out.write(object сериализованный)
Cant find Sange's suggestion anywhere. Did someone delete it ? Anyway, going to change it and make another attempt to close this pull :)
"test rspec test" je male uklepnuti :) ma tam byt "rspec test na vek"
input type=reset styled
It only rejects it if the referrer is sent AND it is different. So if it's disabled, it should work fine...
One comment, grounding totem vs Counterspell dont absorv 8 seconds.

Shammy gets always the silence of 4s!

Great Work!
I didn't forget. I thought about it and I thought it wasn't worth it since the history is still there. Also I think it's not correct because it means we'll duplicate (copy/paste) history which will be in 2 locations. Hence when we compute stats about history of repos (as I did recently with my new committers macro) it'll be wrong, finding twice as many commits as it should have.

IMO the only case when it's good to copy the history is when you discard the source repository and don't use it anymore.

Anyway this is a good topic to discuss on the list if you wish.
I'd prefer if you could use an easy-to read if statement.

```
if (Array.isArray(arguments[1])) {
   args = arguments[1];
   options = arguments[2] || {};
} else { 
  ... etc
```
Yay! I'm helping!
!(SpellIconID == 109)
SpellIconID != 109
is better
Should read "window" instead of "w"?
Dude, that is an ancient RC version, months out of date. Works in latest and 1.0 final.

http://code.jquery.com/mobile/latest/demos/docs/pages/page-dialogs.html#&ui-state=dialog



On Jan 23, 2012, at 6:16 AM, "dotnetwise" <reply@reply.github.com> wrote:

> Dude, this is not fixed! http://jquerymobile.com/demos/1.0rc1/docs/pages/page-dialogs.html#&ui-state=dialog
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jquery/jquery-mobile/commit/47812171f10ee1b28dcc63615cb4a764fb49d2da#commitcomment-891813
please revert this line change, its not necessary
It brought in some of the history, but not all of it.  For example, qanda.php is missing nearly all of the dev history
You are right. We could solve that by creating a subclass of DateComparator (e.g. EventListDateComparator), which has absolutely no additional code, but is used instead of the DateComparator in the above method. Then we can differentiate between those 2 use cases. What do you think of that?
This make the builder resistant to code formatters such as eclipse.
I will be honest in that I am still a little confused on the new event system as it seems like it is adding needless complexity for simple code. (Althugh I guess you would benefit if you had a larger code base to work with.)
Вот тут выдает ошибку при компиляции, и ядро не собирается...
/opt/sources/arcticcore/src/server/game/Maps/Map.cpp: In member function ‘void Map::SendInitSelf(Player*)’:
/opt/sources/arcticcore/src/server/game/Maps/Map.cpp:1915: ошибка: нет декларации ‘transData’ в этой области видимости
Fixed in #98e6cb98381b96141ff97b3d3836acb125ccc675
unfortunately, that won't lead you to anything, you can't revert back to code from sample data and sample data are ovbiously different from the sound you hear, no need to compare them

I don't know how the logs you uploaded were done but the values reported do not make any sense
If you are confident in those reported log then read audio_init & sound_init code, add logging stuff before and after each function calls and debug, mclk should be close to 53mhz ,and psg_, fm_cycles_ratio higher than 2048, anything like what you are getting (they are even negative values for available number of samples lol) is never going to produce acceptable audio
https://github.com/TrinityCore/TrinityCore/issues/4677
prepare ..., envie ... e nos envie -> preparar ..., enviar .. e nos enviar
Sry I was confused. Of course the voting was for sth else. This was discussed in #375.
Not too many voices on that though. Sry if I merged too fast.
recuerda meter esto, es exactamente lo que tengo yo en el server :S
It looks like someone posted this as issue 175.  The change I suggested above works unless you have another way you would like to handle it.
mu is really good for searching http://www.djcbsoftware.nl/code/mu/
Ticket is here: http://open.silverstripe.org/ticket/6908 nothing specific yet, but food for thought.
La bardeas, no es pro pero es divertido :)

El 06/01/2012 20:52, "MatiasArriola" <
reply@reply.github.com>
escribió:

Esto es **MUY** pro para mi papá

---
Reply to this email directly or view it on GitHub:
https://github.com/pjnovas/losColonosDeCambury/commit/e91e4e33dea941915781ba0fdc7d607d4078c97c#commitcomment-845312
I'm sorry I was mistaking, I'm using multiple node processes and one was not connected before trying to issue the first request. AH so much time lose! Thank you.
Corrigido! Obrigado
Нет. В `else` изменяется карта уже существующих постов при автоматической подгруздке. Для них генерация событий не производится (производится только для подгруженного поста).
Previously, this did not error.  I made it do so.  Can I get a review on this?
+1, I should have thought of this.
Isn't math.rint for rounding doubles to integers, not for generating random integers?
Does this commit break pmsencoder as [reported on the forum](http://www.ps3mediaserver.org/forum/viewtopic.php?f=14&t=9798&start=230#p65371)?
Because the actual public theme is only specified in one place rather then
in both files as it is now. If you make a change to one you have to
remember to make it to the other which isn't very DRY. This technique
allows for a single theme body yet still having different platform themes
as your base.
On Jan 10, 2012 5:22 PM, "Ricardo Amaral" <
reply@reply.github.com>
wrote:

> In your example they also have different parents, the difference is that
> you're talking about ```Theme.ActionBar.InternalBase``` instead of
> ```Theme.ActionBar```. But why do that? Why create
> ```Theme.ActionBar.InternalBase``` and then a "new theme" and inherit from
> that ```InternalBase```? What's the purpose of that?
>
> My point is, the InternalBase is redundant. I can't see the benefit in
> that and that's what I'm trying to understand...
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/johannilsson/android-actionbar/commit/48b124725a6364861b5f78fe40a883d345ec7032#commitcomment-855006
>
I havn't seen the infinispan example you sent. Bugged me too that I had to add the version. I will try this other approach.Thanks for the tip
Hint: You'll probably want to make the class abstract
Actually, "predicate" is a terminology frequently used in C++, where you can have both the old C callbacks and also callable objects, known as functors. In plain vanilla C you only have callbacks (simple function pointers), there are no objects.
"predicate" is a term used throughout the Standard Template Library to describe special functions (both plain or functors) that return a boolean value, receive few parameters and whose purpose are mainly to be passed as callbacks in STL algorithms. It's a narrowed subset of the "callbacks".

What I have is a C version of that, a special function (plain), that return a boolean value, receive few parameters and whose purpose is only to be passed as a callback to my algorithm: interception_set_filter.
Update with new *success* example, as per callbackOnSuccess signature the method has no parameters.
message in Spanish means too few beer: clean up package declaration
true case, will correct that
I don't think we need to change the rest of the library. YUI's CSS components tend to be used by lots of people who aren't interested in using YUI's JS components, and it'd be a shame to shove the "yui3-" prefix down their throats just for the sake of consistency.
Should be "Google's Geocoding Service", and "does not provide a JSONP API".

Also, replace "Autocomplete" with "AutoComplete" for consistency.
Consider the case where `$page` has 1 or 0 items (we don't want to generate NOTICEs). Elgg should probably pass in `$page` as an array-like object that returns null for missing keys so we don't have to worry about that.
Damn autocompletion :)
we shouldn't really be dependent on CAS-core
Some javadoc to update here...
Just made the change AbilityBookSlotToBox > AbilitySlotToBox
I could kill myself for that. Thanks Piotr for finding it and Jakub for fixing it! :)
this should be `expectMsgType[ActorRef]`
pulse?
This dance is not necessary any longer: I added the junit-interface plugin so that JUnit-tests are picked up automatically.
What if "a" is already in the map?
Is this really the desired behavior?
If what throws an exception?

If runnable.run throws it, then scheduleNext is not called. The PR doesn't change behavior here.
I liked your previous self better :(
We should make this more finegrained. Detect the diff and only publish that change, if any.
take the actor as a param
The parameter to the sample_response is supposed to take the HTTP status code, not just be empty:

:sample_response:`200`
Yeah, that's not right, will fix.

I sort of agree about not using it, but I also wanted to know if it would work.  Those probably should be reverted, but I'll let you make the call.
This needs special handling for this case. We don't want to reset players in case just a subtitle stream was added. Maybe add a parameter to OpenDefaultStreams()
Playlist window will contain only non-videodb items perhaps, as they're resolved prior to adding to the playlist?
it's even unwanted if it's not strictly needed, as it can temporarily pause playback when it's not needed
So may be it's better to remove first check: 'if (!results.GetDeviceOnLocation(deviceProperty, &prevDevice))' ?
Unfortunately I find it very confusing. Comment explains what but not why.
Since we're not using this part of the tool in production, and its unclear where the input for it comes from, lets remove it from the public repo.
Lets factor out the notion of a "Pool" into its own class, where we can describe how it's used and have these little helper functions to reset itself.
It works with HDFS too.
Ops, I forgot to fix export/import. :)
what happen if the path doesn't exists?
`onContextMenuClose` refers to non-existing function - because you refactored it to `this.onContextMenuClose`
Ok, this I am aware of! I forgot to create the ticket, so I do it now  
right away.
Shouldn't binding return buffer by default?
Requiring `identity` and `id` do the same thing.
Ping me when it's available.

I'm really impressed by your maven work and look forward to getting it into the master branch.
I may be able to make it optional, using the interface. Now It's just compatible with old behaviour.
I like empty lines.. Joking aside, I have no Idea why I keep adding them. Maybe time for a checkstyle rule ?
Not needed constructor as it will have a no-args constructor by default
Ahm, I wonder if we need the `||= false` clause here. The variable is initialized to `false` and changed later to `true` on `initialize!`, so it should never be nil. And if it's `false`, ||= doesn't make sense. Wdyt?
Yeah, way better. Missed that somehow.
I just noticed both tests are named the same `test_amount_time` :)
Update attribute(s)?
But that's exactly what's done by https://github.com/scalamacros/kepler/blob/797f4ac97087de5a59ed2f765fa9afb1757bbfba/src/compiler/scala/tools/nsc/typechecker/ContextErrors.scala#L611, right?
I'm not sure if this is the best solution. I'll ask @adriaanm when I have a chance.
As a general comment, it's a thrill for me to see comments of this quality.  Not too high-level, not too low-level, and explaining what really needs explaining.  They're certainly better than any of mine; in fact they inspire me to do better with my own.
Should toBuffer etc. use this as well instead of `seq.toXYZ` ??
nit: sort of like an extra space before new comments (except if it's the start of a new scope). also s/&/and
The order keyword should be passed to `_sort_key` and used here (`arg.sort_key(order=order)`).
Nice; I verified that doing this `__eq__`  is not necessary anymore in TensorIndex and TensorHead
why do you create a 2nd list of args rather than modifying them in place?

```
args = list(self.args)
for i, arg in enumerate(args):
   ...
   if arghit: args[i] = rearranged
```
s/dispathced/dispatched/
Let's factor this assignment out of the parens instead of doing doubled ones here.
crazy users might set width to something negative..
The instances created by this module in this global context will have two copies: one in parent...
Janktastic
No need for a conditional here; `pop` has an optional second argument, `default`, which it'll return if the key doesn't exist.
Might be worth switching this to '::' as that's what we seem to use atm.
Missing new method name
`new SerializerServiceProvider()`
So there's obviously something wrong in the doc, not following this rule : http://silex.sensiolabs.org/doc/cookbook/json_request_body.html
add a comment about what does it mean that the resource is corrupted
Weird, what is the good solution ?
No order ? Just on type ?
I assume that because this function is private that it does not have a test, however it looks like the kind of thing you might want to have tests for.
Same question with this method.  I don't understand why a NoopScalingStrategy would have to do anything at all, except maybe log that it was asked to do something?
these should be 

    .. data:: chunks

directives so they get indexed. 

we don't need to document the full use of this schema, but moving forward we will...
nope, counter to style
.. will return results assuming a spherical surface...
If I understand well, it takes the first <p> with enough parents.

We have numerous pages that start with a banner (like "This is experimental…", see https://developer-new.mozilla.org/en-US/docs/CSS/animation-duration for one). Will it be taken as the seo_summary? (It would be bad)
I did write: erikfrerejean/phpbb3@b57cd78c0eb00efaace8dee33f7f9e1b55c2b104 however due to the other comment I made I can't install this branch and test it.
This just throws an error in debug mode because the `msg_handler` handles `E_USER_NOTICE` and `E_USER_WARNING` in exactly the same fashion rather than printing debug data for the notice as it *does* do for `E_NOTICE`.
Sounds reasonable to me.
What do others think?
ROLE_DESCRIPTION_FORUM_READOWN
and delete the limited one?
Are you sure this assigning is enough? I can see `$config['site_home_url']` is blank string by default, what returns true in template system, so that condition `<!-- IF U_HOME -->` is always true, or am I wrong?
4 seconds, or 10%, over the entire test suite.

~pie/apps/git-phpbb-tools/run/test-pg  10.26s user 5.08s system 31% cpu 48.932 total
~pie/apps/git-phpbb-tools/run/test-pg  10.41s user 4.71s system 33% cpu 44.563 total

https://github.com/p/phpbb3/commit/22afbf00f1c5ff86bdc9c4f7e3c8667c5875574b
Fixed in 215402e5df1fc3a4ff7d9cc4a9e6dde6c7da14f8
yea... so I guess pcall here.
Good idea. Updated to UpgradePollEmitter in d870484
Using `Y.mojito.models` as the source isn't correct. We should be using instance.models instead.
It's a closure over the `yui` var, so I'm not sure how useful that would be.  Open to arguments of course.
AFAIK nodejitsu uses it.
Зачем это дописывать в $unit?
http://shrani.si/f/26/en/3mBZ4iFh/screen-shot-2012-04-09-a.png
Wow, you are picky :)

Btw, why isn't RootNamespaces a ConcurrentDictionary?
not sure, even a comment would be an improvement in readability

getFields()[5]  // retrieve 'x' field
Now that there's no logic inside the loop, we can get rid of the loop: `attrs = self.__dict__.items()`
this.node.style.position = "relative"; 如果只用一次。就不用另起一行了。
Oh... crap I need more sleep. So yeah, I thought the original code was wrong. Maybe it's not. Anywho, I think this does the right thing. Adding tests was helpful too.
I'm not totally sure what the right answer is. Do we only accept .sql or .ddl? Or do we reject common extensions like .xml or .jar? Do we fail or do we warn?
Why hello, print.
Moving this here is fine, but the if check is superfluous and unnecessary.
maybe eighty is a little rare and scary for the tutorial ;-)
Do you mean the :settings:`SESSION_COOKIE_HTTPONLY`? That was intentional, I didn't want to rephrase the good explanation that is found there.
reported in separated task #1673
With static_cast in `for` it is
```
cd ... && /usr/bin/c++    -g   -I/usr/local/include  -D_DEBUG -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -g -fvisibility-inlines-hidden -fno-exceptions -fno-rtti -fPIC -Woverloaded-virtual -Wcast-qual -O0 -UNDEBUG -fexceptions -o CMakeFiles/compiler.dir/env.cpp.o -c clay/compiler/env.cpp
clay/compiler/env.cpp: In function ‘void clay::getIntrinsicSymbols(clay::ModulePtr)’:
clay/compiler/env.cpp:127: error: invalid conversion from ‘size_t’ to ‘llvm::Intrinsic::ID’
clay/compiler/env.cpp:127: error:   initializing argument 3 of ‘clay::IntrinsicSymbol::IntrinsicSymbol(clay::Module*, clay::IdentifierPtr, llvm::Intrinsic::ID)’
```
It fails here even without any flags except -c and -o
For limiting the range to [0,2^15-1], use `getAs<uint16_t>()`. At the moment that requires the SQL type to be INT2 (i.e., may be less convenient to use), but I may change this in the abstraction layer that also INT4 or INT8 are accepted, but that these type just raise an exception if the value is too large to fit into the template argument given to `getAs`.
Can you please follow the style of the other CSS rules on this page? That is -- one line statements when possible.
url 'https://github.com/erkin/ponysay/tarball/2.4'
For me the main reason for adding isl is for use by GCC as well. But adding head will help more users. Will do in next commit.
This is a duplicate because line 656 already creates a log entry.
Should not be needed!
Supprime le code plutot que de le commenter
CDR prefix
We should be using .yaml as our yaml file extension. That is preferred over .yml
Move this to the first paragraph of text localization on the client side section and keep only that section.
yup.. is geplant..
Proposal: I wouldn't ignore it, but give --conf-dir higher precedence: I.e. first add CDIST_PATH then the ones with -c
I'd hope we can find a better way to do this.  Can we just create a filter that takes in a description, and returns true if it matches one of the desired method patterns?  Then we could apply that filter to the runner we're about to run.

Have you tested this with, for example, a Parameterized test?  I think it will break it.
I tried that in the beginning, but then the modules aren't aware of the promise and we'd need to create some mechanism to deliver it to them everytime a new promise is created which I didn't really like and couldn't really figure out a nice way of doing it...

What about having one unique promise for the `Resizer` module? We could append the resized element id to the parameters when notifying the deferred object. This way, `Resizer.resizing` would return always the same promise, and the modules could identify which updates refer to them by checking it. (Granted this could get messy if people using the promise aren't careful enough...)

I'm attending tomorrows Hackaton in London, and as I'm no real expert in promises, maybe someone there could pitch in with some ideas?