labofsahil/github-event-dataset-2012 huggingface dataset:
Added a delimiter to collection rendering

Added a new attribute to allow for you to specify a delimiter when rendering a collection.  The use case I had for this was I was rendering each item in a collection to a specific JSON format.  In JSON each item in a collection needs to be delimited by a comma so this was the way to do this.  

I did not want to add a comma to my view that was rendered on the collection because there would alway be one extra comma output for the last item in the collection.  

Also useful when output HTML Lists.

Usage:  

&lt;ul&gt;
&lt;li&gt;#renderView( view="item/summary", collection=prc.items, collectionAs="item", collectionDelim="&lt;/li&gt;&lt;li&gt;", prepostExempt=true )#</li&gt;
&lt;/ul&gt;
put code for keyboard firing back in

sorry, my merging skills failed me ;)
fixed #656

This is the final pull request for this fix.  It is wasting my time.
Merge with latest auto-complete-clang, and cleanup.

Your e-mail address shown on GitHub doesn't seem to work, so I'm sending this from here:

Hi, you might've noted that I forked your project.
I merged it (manually) with hara's latest version of
auto-complete-clang (which is the latest code from brianjcj plus
Objective-C support), and also did a lot of clean-up to meet
elisp-coding standards. I *might* have broken things while doing all
this, and can't reliably test it myself because I don't know how
exactly your original code is supposed to behave (I probably have a
different clang version and all). So it would be very friendly if you
could test my version and see if it behaves well. If it does, I would
ask you to merge it and continue working from there.

In case you wonder what "elisp-coding standards" I'm talking about,
the biggest issue with your code was that it defines functions and
variables with generic names like `completion-proc`, `filter-output`,
`parse-completion-results`, `send-source-code`, etc., which all end up
in the global function/variable namespaces of the emacs user. Imagine
how many different elisp packages/libraries might be using a generic
function like "filter-output"... So to avoid clashes, everyone should
adhere to the convention that all functions and variables in a
package/library are prefixed with a unique identifier of the
package. E.g. in this case we use `ac-clang-async-filter-output`.

Regards, and thanks for the code
Taylan
Korean translation by Millenium3

Korean translation
Bugfixes and additions :)

Hey guys, I tried to pull request like a month ago but, there were an error on Windows support and I preferred not to proceed with the commit.

But, before that, I've had some problems with your plugin (with it not telling why my PHPCS stopped working and so on) and decided to fix it myself and in the middle of hacking I added some stuff that I thought were necessary.

Well, I hope you enjoy those fixes/changes (changelog below), and I'll be glad to help you with anything necessary.

Those changes were tested on NB 7.0+, Ubuntu and Windows.

Best Regards,
Willy.

---
Changelog:

- Added: Shortcut, press Shift+F12 to analyse current file.
- Added: Validation to the form/panel. Validates if PHPCS is working and if there are any valid Code Standards.
- Added: Validation on preferences storage, avoids invalid settings to be saved.
- Added: Refresh button on configuration panel. This button revalidates PHPCS binary and refresh code standards list at user will.
- Added: Warnings/Errors are also added to the error stripe (column right to the editor scrollbar). Now, one won't need to scroll all the file to find CS warnings/errors.
- Added: Context menu for JS and CSS files (both also supported by PHPCS tokenizer).
- Added: Show information dialog if no error/warnings were found.
- Added: If PHPCS fails with a fatal/parse error, show an information dialog.
- Fixed: Exception handling if no CodeStandard is selected (NullPointerException).
Use force to update application layout

Instead of this:

```shell
$ rails generate bootstrap:install
      create  lib/templates/erb
      create  lib/templates/erb/controller/view.html.erb
      create  lib/templates/erb/scaffold/edit.html.erb
      create  lib/templates/erb/scaffold/index.html.erb
      create  lib/templates/erb/scaffold/new.html.erb
      create  lib/templates/erb/scaffold/show.html.erb
      create  lib/templates/erb/scaffold/_form.html.erb
       exist  config
      create  config/initializers/simple_form.rb
      create  config/locales/simple_form.en.yml
    conflict  app/views/layouts/application.html.erb
Overwrite /home/gaurish/Dropbox/code/practice/splitcast/app/views/layouts/application.html.erb? (enter "h" for help) [Ynaqdh] y
       force  app/views/layouts/application.html.erb
      create  app/assets/stylesheets/bootstrap-variables.css.scss
      create  app/assets/stylesheets/bootstrap-generators.css.scss
      insert  app/assets/javascripts/application.js
```

its better to add a -f switch, so the user is not promoted about overwriting. Because to use bootstrap, they do need to overwrite `application.html.erb`. so `-f` makes sense


Btw, Thanks for creating this gem. its quite handy & rad! :+1:
Doc annotation mongodb

Added docs to use annotation with Doctrine ODM (MongoDB)
Spotify urls

This makes the spotify resolver use http://localhost:55050/sid/spotify:track:lalalala.wav rather than a randomly generated ssid. It also fixes a couple of memory leaks to do with sp_track objects.

Hope you like it.
colspan not work in IE 7 (may be in other version to)

If use "colspan" as attribute name (attention to symbol's case) - IE show group name at first column, not expand to all row.
Fixing OpenId Strategy for the changes made to Ninjauth lately.

The commit about refresh tokens (0d87fc0fcb6f1d4214fa01c44bebbffed6fa3341) broke the database insertion of OpenId authentications at least on MySQL with PDO drivers. `secret` must be set to something otherwise there's a "Column cannot be NULL" constraint error.

$this->provider->name must also be set to something for the same reason. This problem was introduced by 3602bfa01929ea3de97c8bd2e8535a07a15c09fc

I also did a small bugfix to correctly add a space between compounds mapping from OpenId to required format for attributes.
Added whiskers and jade.

whiskersとjadeを追加しました。
jadeに関してはキャッシュを利用していません。
原因：renderメソッドをコールすれば使えるはずなのですが、exportsオブジェクトを意図する形で動作していないため、エラーが発生します。この問題が解決すれば、相当速いのでは…。
GET arguments passed to callbacks as arrays. 

Pretty much that. Any statement that ended with a ? wouldn't route properly. I thought it was a bug but it might've been a design decision. Either way to preserve backwards compatability I've made choreographer push the GET arguments (?foo=bar) to the last argument in a callback. Callbacks now look like:

function(req, res, [arg1, arg2 ...], args){ }

This way there's no static amount of arguments, there's backwards compatibility and it's easy to use. It might still be confusing for people new to choreographer though, even though it's documented. It also slightly complicates choreographer, it doesn't make it do any new tricks though. Just completes it a little more.
SOCIALFB-103: Add 'about' property to Page object.
Payload

duplicated files problem corected
Provide example of correct attribution

* Provide .png example of correct attribution on a webmap
* Copy the "where to put it" sentence from the Legal FAQ for the most common use case

(I should again point out that this is untested as I don't have a working Rails port right now, so apologies in advance for any typos.)
Small bugfix for a PHP 5.3 strlen error

Hey dude - small bug fix for a problem in PHP 5.3 you might want.
Topic/mark enhance

Fixes numbers 1, 5, 10 on the google doc bug list ( https://docs.google.com/a/azavea.com/spreadsheet/ccc?key=0AgXwHVQ2cCDZdDRmX2FybEhTX3RGaEd3LVdNYzJ1Znc#gid=0 )
Fix for issue 57

Use BIT type for boolean fields in MySQL adapter
Use env["PATH_INFO"] instead of env["REQUEST_PATH"]

env["REQUEST_PATH"] isn't covered by the rack spec, and thus not necessarily passed to the application by some web servers. This fixes issue #1.
Updated README

build what I learned in class from scratch, and then do it again.
Changed input format order.

Changed the default input format order ('before', 'label', 'between', 'input', 'after', 'error') to ('before', 'label', 'between', 'input', 'error', 'after'), so <code>&lt;span class="help-inline"&gt;ERRORMESSAGE&lt;/span&gt;</code> is before <code>&lt;p class="help-block" style=""&gt;HELPLABEL&lt;/p&gt;</code>. Otherwise the help-inline-element is not shown inline with the input-field.
Upgrade to Box2D version 2.2.1

The official dist 2.2.1 is included. It includes a number of fixes and new features. Most notably it also contains the b2ChainShape. 

The mkf files were updated.

I tested it in my own apps. The only breaking change is that the constructor of b2World now only has one argument. The second argument was removed. Additionally, the code is cleaned up so that it no longer produced GCC compiler warnings.

I didn't modify the Box2D files. They work out of the box.
Fix incorrect description of array.length assignment.

Increasing the value of array.length actually creates a sparse array (as shown by the modified example I added).
faces for rpm-spec mode
Add domain to role

Add domain config param to filter for a specific domain
fix broken tests

add urls to *all* TestCase classes
Fix typo in seattlelab_pass.rb exploit.

Also remove the $ from the end of the regex which stopped
the exploit from being executed.
Graphics and Commands
reset an rnaseq lane
Here you go

Second commit is just clarifying a comment
Implement exclude as a regular expression

The documentation says that these can be regular expressions, but it
wasn't implemented that way. Now it is.
Fix last structure, that I missed in the last patch.

Changed last structure from ( a == x or a == y or a == z) to use the /in/ statment (a in (x,y,z)). With this the code is simpler and is easier to expand.
SQL backend

* SQL backend is implemented and has unit tests
* Still need to add a test for the AuthenticationBackend
* Updated doc as needed

Note: mongoengine backend has not been implemented
SQL backend

* SQL backend is implemented and has unit tests
* Still need to add a test for the AuthenticationBackend
Fixes "Unknown column type json requested."

When you install SonataNotificationBundle and then attempt to make an update to doctrine, it fails in Symfony 2.0.12 with Unknown column type json requested.

The message entity's body attribute should be an array according to the model. If it's supposed to be json then a new doctrine type should be created for this.
Wijmo minor fixes to top border radius and dropdown item padding

Worked on issue here: https://github.com/addyosmani/jquery-ui-bootstrap/issues/12

- Removed Wijmo top-left and top-right border radius
- Fixed padding issue
LPS-24591
Use LaunchDaemons in /Library instead of /System/Library

On OS X, Apple reserves the use of /System/Library for its own use. Therefor, Jenkins should install the slave launcher in /Library/LaunchDaemons instead of /System/Library/LaunchDaemons (a full install of Jenkins already uses the correct Library directory).
Search user’s posts does not work in Statistics tab

Search user’s posts does not work in users profile statistic tab.

http://phpbbsocialnetwork.com/viewtopic.php?f=44&t=4368
fork the created process

I had an issue when starting running 'gmrun',
it had to finnish before i could use another button.

Same if i would start a fresh chromium, then i could not use the buttons until i closed chromium.
It would just stack up in the background and start right after I closed the first one.
added missing file and updated gitignore

added missing file and updated gitignore
Added option to disable the adding of event handlers

There wasn't an option to simply format the contents of a field without binding event handlers, so I added it, based on [this Stack Overflow question](http://stackoverflow.com/questions/9454191/using-jquery-priceformat-plugin-on-focusout/9454335#9454335).
Date Input Type

Support for binding when input type="date". This does not work correctly in iOS Safari.
Bootstrap 2.0 changes

Bootstrap 2.0 changed the actions class from 'actions' to 'form-actions'
Adding a link-copy shortcut (alt or ctrl+shift)+click
Standard-Wert für $page auf null gesetzt.
FLUID-4630: High-contrast external link indicators

@cindyli, if you have a minute, could you double-check this pull request, and if it's ok, merge it into the mediawiki-1.18.1 branch and pull it down to the deployment server?
誤植の訂正

「インストー」を「インストール」に訂正しました。
Develop

refactored code...separated the biz logic from the UI...this is the first commit out of 3+ commits... the plan is to have the MVP style implemented in our code after this Friday demo.....
Refactored the authentication web service to use most recent node

Along the way, removed mongoose auth, removed mongodb and everyauth, and added a mailer functionality to mail users when they try to login too many times, and to welcome them so they will remember their login. 

What this simplifies:
* almost all node is simpler now because we dont have libraries or "magic"
* no more mongodb
* no more mongodb schema, if we want to add to the users we can do it in the two sections mentioned in the authentication lib.
Modif motorinn lee lilly + env dairyexterior trappingandy

Bon, c'est un premier jet. Le tutoiement lors du combat me plait pas mal dans le sens où la violence (folie) est à son maximum sur cette séquence. J'ai essayé de garder une cohérence quand même (la mère vouvoie dans la peur, et lee repasse sur mode civilisé quand Andy pointe son arme).

A tester ingame, à mon avis.

Je ferais le deuxième un peu plus tard.

See you.
ThemeChooser: Simplified Chinese Translations

Revamped Simplified Chinese translations for ThemeChooser, quite a few
translations were inaccurate.

Patch Set 2: Some revision.

Change-Id: Ibe6cb21a065cefcd61ca132cb2d8e6fa774128a3
Good deal. I'll probably be trying to restore more S3 functionality. If you get a chance, maybe you could see what's wrong with DynamoDB. It has something to do with the session keys and the response headers and requests.
True, however this only applies to the overlay renderer, and if this function is ever called with GL_BGRA, you'll get image corruption.
viz prvni commity, imo by se to melo testovat jen u potrebnych driveru. Tady to muze pusobit zmatky.
ah crap, I think there's more shadowing going on inside of it. I'll need to check every block.
looks like above where we were talking for line 109 that you could just use this helper you already have
![](https://a248.e.akamai.net/assets.github.com/img/2ffd2eb5773d9962f79411c5395b473f789abd9c/687474703a2f2f692e696d6775722e636f6d2f7356356c612e676966)
@radekdostal Voláš to moc brzo (attached ti to přepisuje).

Téhle diskuzi by bylo lépe na fóru, ne?
Too many blocks and hash lookups. This is why Rails is slow. :trollface:
`::Rails::Rack::Logger` is making noise with `Rails.logger.info`
@mikz what happened if we don't use `+` and put them in a string `"#{t(:editing_product)} &ldquo;#{@product.name}&rdquo;"` like this?
Did you do that up there, on line 212?
There's another similar comment a few lines above, which probably wants the same fix.
prima. de manier die ik heb gemaakt heeft minder overhead (voor als je jquery verder niet nodig hebt) :)

En het is natuurlijk cooler om zelf javascript te maken :)
created DOCS-151 to address this.
Dead code? Remove if so.
Does the spec say anything about `eval` and `arguments` being restricted, or was this check added for compatibility?
Maybe. What methods should brunchplugin provide by-default?
,    
     false  before_destroy. 
        false.
  :

if line_items.any?
  errors.add(:base, 'Line items with this product present')
  false
end

 false     ,     ,
   errors.

24  2012 . 11:12  gotva <
reply@reply.github.com
> :

>     ,  ...
>      errors (return true/false  
> )
>        
> errors.add(:base, 'Line Items present') if line_items.any?
>
>        
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/flexoid/depot/commit/fc15f835be435e4d27977856c8661106a5c07819#commitcomment-1006312
>
yeah this is better - we don't need this on submissions step (hopefully) since there can't be any prior previews because this step would have to be completed for that to happen
jesus christ, credit me with some intelligence maybe... there were TWO onlyTargetCategory members, one in WEAPON *and* one in WEAPONDEF, the second became totally useless after removal of toAirWeapon and has now been removed itself, panic attack over???
I don't know why md4 has tests in the wrong place. I'm aware of #1254. Moving the tests into std is the goal, but the makefiles are not set up to actually run tests from there.
Thanks, @aL3xa!
already working on #3 :)
Actually, docrails has public write permissions, anyone can contribute directly.

That seems to be a trailing space right? We should revert that one.
Looks to me like the tarball is missing.
el binding funciona sobre un elemento (jquery object), no un selector (string)
Can you fix the typo here?
Exemple de commentaire.
@wilkie Seems like for already_posted?  (and just ```do_not_repeat_yourself``` prior to this change) you'd actually have to look at all prior statuses and compare the text?

In it's current from, whether you leave ```already_posted?``` or do as wikie suggests and go with:

```ruby
def do_not_repeat_yourself
   if feed.last_update && feed.last_update.id != id && feed.last_update.text == text && feed.last_update.author.id == author.id
    errors.add(:text, "You already posted this update.")
```

You're still only really checking the previous update.
1.2.0-3ubuntu0~ppa1~oneiric1

Not sure why the 1 is needed but libnfs cannot be found. Only libnfs1 is found on the ppa which refers to the package 1.2.0-3ubuntu0~ppa1~oneiric1.

We're trying to solve this issue on our livecd:

17:23:46 T:3006245744 ERROR: Unable to lookup host: ”
17:23:46 T:3006245744 ERROR: Unable to load libnfs.so.0, reason: libnfs.so.0: cannot open shared object file: No such file or directory
17:23:46 T:3006245744 ERROR: NFS: Error loading libnfs (HandleDyLoad).
17:23:46 T:3006245744 ERROR: Unable to load libnfs.so.0, reason: libnfs.so.0: cannot open shared object file: No such file or directory
17:23:46 T:3006245744 ERROR: NFS: Error loading libnfs (HandleDyLoad).
17:23:46 T:14115888 ERROR: GetDirectory – Error getting nfs://
17:23:46 T:14115888 ERROR: CGUIDialogFileBrowser::GetDirectory(nfs://) failed
@douglaslee 这里一没有向后台解释清楚，二是你没有在本地测试过，如此去掉class是行不通的。

正确的改法：https://github.com/PKUIRSA/pkuzone/commit/6122b7dc4710f12968b733fe1d54d6776ec1ff9e#L0R117
`&mdash;`?
Re #2 -- what about using getUtility(ISiteRoot)?
Great, congratulations!
good catch....
why 100 here?
Doppelt?
slightly shorter to do assignment on the same line.  ex. this.vmSpec = checkNotNull(vmSpec, "vmSpec");
I should probably stop commenting on throwaway code.
So, how do I "split" a commit?
Again typo, kernel, and this example's linux version is old...
As long as you restart LunaSysMgr after the change to recache it and
it works.  The "a" comes from the minimization process, but isn't in
the original source file.

On Wed, Mar 7, 2012 at 1:18 PM, Rafa Bernad
<reply@reply.github.com>
wrote:
> Tested on webOS. It works, understanding that webos uses the enyo-build.js file under root@localhost:/usr/palm/frameworks/enyo/1.0/framework/build. I've done testing by replacing that file, so if it's ok, you can pull it safely.
>
> P.S. (Just FYI, original "startdrag", "drag", and "dragdrop" methods make use of that same naming convention, with "a" instead of "e" as a parameter).
>
> El 07/03/2012, a las 18:14, Ben Combee escribió:
>
>> have you tested this on webOS?  I don't want to pull anything that affects the scrolling experience there.
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/Newness/enyo-1.0/commit/f352cdd7ad25e10fb9c2f2d978441e8f7259fb3c#commitcomment-1054472
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Newness/enyo-1.0/commit/f352cdd7ad25e10fb9c2f2d978441e8f7259fb3c#commitcomment-1055102
no output?
It is working for him with this patch reverted on a 5600 card! It fixes the colors and stuff for his configuration with and without shared surfaces.

As Lars has rebased, you could just drop this patch when rebasing?
(Kano does not want to register her for comment.)
refs #20 (typo désolé)
I wonder if I should state that it was intended to sound cool, and that the correct spelling in this regard would actually be `fillin'`. Not.

Thank you :)
oh i see... you certainly could get some false positives just testing for a `bind` function if you're going to check all the properties mixed into a widget.
Is that useful as a unit test? Probably more interesting would be a setup where we could test various components, and it'll add console.profile() and console.profileEnd() calls around the code under test.
Please enlighten me - I fail to see the how the new version is more reliable. :-(
Fixed please see commit 56640d99e906b4ceb84de41d753a71eee3b9267b in nuisanceofcats repository fork.
Does this file already exist within the openemr codebase?
tu ne peux pas utiliser les traductions en fonction du code_iso, ceci éviterai de perdre ce libellé car il y a pas mal de monnaies qui utilisent le terme dollar par exemple, le libellé en base sert juste pour savoir ce que c'est et pas pour l'affichage
Sorry about that, I'm really a noob when using Git and thought my gitignore file was perfect but, alas, it was not. So I perfected my gitignore file by adding gitignore to it and now, it should not happen anymore :)

I'll definitely look into configuring a better build environment this weekend.

JFR

On 2012-03-04, at 13:01, Daniel Scharrer<reply@reply.github.com> wrote:

> Adding a .gitignore file like this to the tree is not a good idea as the files it needs to contain will be different for everyone as it needs to include temporary and settings files used by the IDE, file browser, etc.
> 
> I'd rather recommend creating a separate build directory for yourself and then pass the path to the source instead of '.' when running cmake.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/arx/ArxLibertatis/commit/1cf0dbbdeaf39412d484dc348c9690a28f217d13#commitcomment-1040444
было бы круто, чтобы операция выполнялась не только по знаку =. т.е. если я нажму 1 + 2, а потом нажму еще раз +, то оно посчитает 3 и покажем его на экране. чтобы можно было связанные действя выполнять сразу подряд
`title` could go on this line
The `nsfw_sheild` method has gone, what to do with that line now?
Again, I used this for something else a while ago, then dusted it off for something else and didn't update it. I'm not sure if it goes with the calendar or something else.
is it really a valid usecase to ignore sourcefile dependencies when linking? after all, you might end up with an inconsistent end product (exe)...
Hey Austin, all you should have to do is:

git checkout -- resources/static/include.js

and that'll restore to the development version

or

run the scripts/cleanup_compress.sh script.

Shane

----- Original Message -----
From: "Austin King" <reply@reply.github.com>
To: "Shane Tomlinson" <stomlinson@mozilla.com>
Sent: Friday, January 13, 2012 5:36:29 PM
Subject: Re: [browserid] Bug#706572 Adding i18n support to BrowserID (a725d66)

I think I understand...
rm resources/static/include.js 
ln -s resources/static/include_js/include.js resources/static/include.js
commited in
https://github.com/mozilla/browserid/commit/f73961e151009659fff32e8b37eab65aef34cb18

If that isn't correct, please let me know.

--- 
Reply to this email directly or view it on GitHub:
https://github.com/mozilla/browserid/commit/a725d667e4ce537404006037d1d72a7c5fd6a1d7#commitcomment-865558
Interestingly enough, in my system using qemu-system-x86_64 leads to a segfault running the default test set.
Also fixed: conflicting slugs when using STI ;)
I just thought it would valid to use a 'string' to be passed rather than an array. Also with the shell i was just wondering to make it look like running through a shell command. I just pushed a new change which re-factors the codebase by moving the object code into a separate file. The idea is to try "from pbs import *" through python command line. This still needs to be fixed.
IK HEB OOK INPUT
http://i.imgur.com/6YnN1.jpg

On Wed, Feb 1, 2012 at 2:46 PM, Toon Willems
<reply@reply.github.com>
wrote:
> shit, mijn vim is wa nen held <3
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ZeusWPI/12UrenLoop/commit/dd69b1a4ea0a027dd0d41e0c3fa625d394df5a06#commitcomment-922427
Nein, auch so ist es nicht richtig. Er wiederholt es an der Stelle und dreht es nicht um. Bitte den Satz nochmal in Ruhe im Video anhören.
Moved global style back underneath  jquery ui style to overwrite necessary styles
There's barely any UI, but what little I have is functional. Except for the middle fragment that shows crap, but I'll think of something to put there eventually, I think.
There's even a better way using config - [config.action_dispatch.tld_length](http://guides.rubyonrails.org/configuring.html#configuring-action-dispatch)
Thanks. I'll merge it. Next time, try do one change per commit.
If the C api is just going to treat it as an opaque type whose only use is configuring another rados_cluster_t, maybe we should just have a create_with_config(rados_t*, rados_t) that copies the CephContext from the existing cluster handle. Then we wouldn't have this lifetime issue.
Please ALWAYS use NSLocalizedString when setting user-visible strings!
what's the use case for async start and stop?

fwiw I think making them return futures will be very counter-intuitive. 

-- 
peter royal - (on the go)

On Feb 5, 2012, at 11:56 AM, Joe Walnes <reply@reply.github.com> wrote:

> LGTM
> 
> Can you make this change very clear in the release notes, as a lot of people use the pattern: WebServer server = create(..).add(...).start();
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/aslakhellesoy/webbit/commit/15f89e85f1f27619f342f2d1099f4c9f77c17dc8#commitcomment-934499
That's what I get for a quick copy/paste!
Damnit lol
error - fix after.
in eine try rein? falls der info hash invalid is....
very good job :)

thanks!
commit log はchromeのレイアウト崩れのことしか書いてませんでしたが、リンクの飛び先の修正とかも含めちゃいました…。
Cool, I'll add t/ tomorrow and push it. Thanks for a great package!
On Feb 25, 2012 2:20 AM, "Glen Hinkle" <
reply@reply.github.com>
wrote:
and use `File.readlines(gold_filename)` here
0.upto(exp_digits.size) - 1
closes #5554
Ok, let's put it back and comment that code more. Thanks for explanation.
that looks like dead code to me. good work.
Don't worry. We are talking about neglectable speed downs in the order of microseconds per query.
@Axium no, you were able to cast through the trees even on WoTLK
I think I prefer that option
"AAttribution" typo?
Null45 commented on ProjectSkyfire/SkyFireDB a minute ago
Comment in a2af909894:

    F**k off, man.

Childish.
@KensoDev if the response is successful we return `head :ok` otherwise we return `head 409`. So there is is never anything in the response, we just return headers and an empty response.
That piece of code is something which has me puzzled since I started refactoring the cgGeo code. Would be great if someone else confirms that this didn't make sense.
Better not to use "you", "your", "me", "mine" etc. For this case it is better you use username or user_name. Also try not to use camelCaseStyle for variables, use underscore_naming_style instead :)
The updateJLPTLevels code will not be used in this app. I should remove it later.
The strcmp() wants a const char *, not just a char * and you're storing in ext everytime something new.
I'd feel safer with 55-60%
I add here:

if (Creature *pLichKing = m_pInstance->GetSingleCreatureFromStorage(NPC_LICH_KING))
            pLichKing->SetStandState(UNIT_STAND_STATE_STAND);

and delete

pLichKing->SetUInt32Value(UNIT_NPC_EMOTESTATE, EMOTE_STATE_STAND);
pLichKing->SetStandState(UNIT_STAND_STATE_STAND);

from bottom and its works ;P
Yes, I try to compile too many times under  the gentoo,ubuntu 10.04.2 LTS(2.6.32-30-server #59-Ubuntu SMP Tue Mar 1 22:46:09 UTC 2011 x86_64 GNU/Linux) with kernel 2.6.x and it's not work.
After I found that it's work with linux kernel 3.0+.

Now, I found another issue while compile below "fail.d" with segfault with dmd v2.0.59 (git repo),  dmd v2.0.57 can process ok with "fail.d(7): Error: expression callable() is void and has no value" output.
-------------- backtrace -----------
#0  0x00000000004c2c38 in Parameter::foreach(ArrayBase<Parameter>*, int (*)(void*, unsigned long, Parameter*), void*, unsigned long*) ()
#1  0x00000000004c2d5e in Parameter::dim(ArrayBase<Parameter>*) ()
#2  0x0000000000478ed3 in FuncExp::inferType(Scope*, Type*) ()
#3  0x00000000004863b8 in AssignExp::semantic(Scope*) ()
#4  0x00000000004f587b in ExpStatement::semantic(Scope*) ()
#5  0x00000000004fa15d in CompoundStatement::semantic(Scope*) ()
#6  0x0000000000491efc in FuncDeclaration::semantic3(Scope*) ()
#7  0x00000000004fdd48 in AggregateDeclaration::semantic3(Scope*) ()
#8  0x00000000004be510 in Module::semantic3() ()
#9  0x00000000004bc81c in tryMain(int, char**) ()
#10 0x00007f4d96e5fbbd in __libc_start_main () from /lib/libc.so.6
#11 0x0000000000402a09 in _start ()

-----------------------------------------------------
opsvps test # dmd fail.d 
Segmentation fault (core dumped)
opsvps test # cat fail.d 
import std.stdio;

class TestCall {
    private void delegate() callable_;

    this(long time, lazy void callable) {
        callable_ = callable;
    }
}

void main(char[][] args)
{
}
------------------
No reason.  I found it to be easier to read and type than the alternatives /at the time/. I'm open to *any* scheme as long as there is one.
fixed in fb72389
Why did you do this?????
I will do this tomorrow. I have just been thinking about the best way to test it, given mavens single class path. 

Stuart

On 10/01/2012, at 8:42 PM, Darran Lofthouse wrote:

> Stuart, can you add a unit test case for this one and I will then pull it to master as the change for https://issues.jboss.org/browse/REMJMX-5
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/stuartwdouglas/remoting-jmx/commit/2ad4e49354b3db5fc1ae05be5c73f8acce5fb3ad#commitcomment-852578
Same for this. I have a real dislike of having to split a <tt>.method</tt> call across two lines; I'd rather have two blocks, and a local variable:

<pre>
if (iPayload == null ||
    iPayload.getObject(Constants.ZKP_KEY_ZKP_A) == null) {
  Log.…
  …
  return;
}
String zkpA = iPayload.getObject(Constants.ZKP_KEY_ZKP_A);
if (zkpA == null ||
    !theirSignerId.equals(zkpA.get(Constants.ZKP_KEY_ID))) {
  …
  return;
}
</pre>
I'm also working on this to fix http://d.puremagic.com/issues/show_bug.cgi?id=4854
This commit tackles #4, CRUD logic is not abstracted yet. @danielbachhuber Let me know what you think.
This sequence of SQL queries will migrate your data to the new table layout.

    INSERT INTO USERS (id, name, lastseen) SELECT djid, djname, started FROM SONGLIST ORDER BY started asc ON DUPLICATE KEY UPDATE lastseen = SONGLIST.started;

    INSERT INTO USERS (id, name, lastseen) SELECT djid, djname, started FROM CHATLOG ORDER BY started asc ON DUPLICATE KEY UPDATE lastseen = SONGLIST.started;

    ALTER TABLE SONGLIST DROP COLUMN djname;

    ALTER TABLE CHATLOG DROP COLUMN user;
Hola Francesc,

Glad to hear from you. Thanks for the great gem !

I think the option I added works fine but I am only using it in development
for the moment.
There are a few implementations steps (outside the gem) that went along
with it to make it work smoothly.
I'll try to summarize this and maybe add some more precise docs over the
coming days.

Is it ok for you if I make the pull request at the end of the week ?

Cheers
Nico


2012/1/18 Francesc Pla <
reply@reply.github.com
>

> Hi Nico,
>
> It's seems it's a good feature for the next version. Do you think you can
> make me a pull request when you think it works ok?
>
> Thanks!
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/NicoArbogast/rails-translate-routes/commit/06d77834ea35ec17d22d05fabe962e918d544c2c#commitcomment-879161
>



-- 
Nicolas Arbogast
Weezic co-founder
+33 6 23 86 07 36 | @NicoArbogast <http://twitter.com/nicoarbogast>
weezic.com | fb:weezic<http://www.facebook.com/pages/Weezic/148256495214600>|
@weezic <http://twitter.com/weezic>

<http://weezic.com>
Ola, cool. Re: 5953, if Tues. meeting is not too late, we could do it under AOCB. Otherwise, I'm available whenever. My reasoning for having unified logs includes hat we have tools (under ```OmeroPy/src/omero/install/logs_library```) for parsing that format. If one file like _request or _error needs to be in Apache style I can certainly understand. But the internal, debugging code it would be nice to have in "OME-style".
delete this proprietary stuff
/system/lib/liblgdrmwbxml.so ../../../vendor/$VENDOR/$DEVICE/proprietary/lib/liblgdrmwbxml.so (not using after using liblgdrm from source)
/system/lib/liblgeat.so ../../../vendor/$VENDOR/$DEVICE/proprietary/lib/liblgeat.so
/system/lib/liblgdrm.so ../../../vendor/$VENDOR/$DEVICE/proprietary/lib/liblgdrm.so
/system/lib/liblgerft.so ../../../vendor/$VENDOR/$DEVICE/proprietary/lib/liblgerft.so

use source code
https://github.com/wingrime/android_device_lge_swift/tree/master/liblgdrm
https://github.com/wingrime/android_device_lge_swift/tree/master/liblgeat
https://github.com/wingrime/android_device_lge_swift/tree/master/liblgerft

WTF??? we dont need this.
vendor/__VENDOR__/__DEVICE__/proprietary/bin/linker:system/bin/linker

WTF? for what this one if we dont declare it in ramdisk?
vendor/__VENDOR__/__DEVICE__/proprietary/bin/port-bridge:system/bin/port-bridge \\

This stuff building from source code!!! 
vendor/__VENDOR__/__DEVICE__/proprietary/lib/hw/lights.swift.so:system/lib/hw/lights.swift.so \\
vendor/__VENDOR__/__DEVICE__/proprietary/lib/hw/sensors.swift.so:system/lib/hw/sensors.swift.so \\
vendor/__VENDOR__/__DEVICE__/proprietary/lib/hw/copybit.swift.so:system/lib/hw/copybit.swift.so
adb pull /system/lib/libril.so ../../../vendor/$VENDOR/$DEVICE/proprietary/lib/libril.so
Phone stop on samsung inscriptions
I believe the new commit is somewhat saner for that case.
Let's remove these comments - we cam only posit what they mean and they are now out of date.
There is not need to throw an exception. The values will just be ignored until Intermediate is set to false.
I think `Y.each()` should be deprecated.
`__eq__` methods of what? And where would those be used?
Is this "N time period" well-known? I personally prefer a plain old alpha \elem (0, 1) scaling value
Is there a class that has a different name than Deadline but still  
represents a point in time? Or the only way to do that is  
System.nano/milli Longs?

Storing the real deadline was my original plan, but this way I can extract  
also the elapsed wait duration when the Latch opens which is needed if I  
want to implement increasing backoff intervals.
great, otherwise it was a good catch Jonas
I generally prefer () to {} since () is the Unit value and {} simply infers it.
As far as my little REPL-experiments have revealed, Symbols are cached in static final fields in those classes which use them, so there should be no overhead after the class has been initialized. That is not to say that we always use case objects (for the other reasons), though.
Remove this comment, or write something real
this log message seems odd to me... I would prefer it to say "cleaning up pipelined messages" or something along those lines (rather than the technical details of *how* we're going about doing that)
i'm guessing others are doing it this way. does it strip out id's?
??? No, just the opposite, the earliest migrations are the ones I did first and got working (on xml_format, now called codenames)  This one goes first, with all the changes to use and update type_id instead of typecode.
Can we just return False as soon as we have more than one domain - that would stop us having to check them all - not a big problem here but good to try to reduce the work needing doing.

    domain = None
    for url in urls:
        if not domain:
             domain = url.domain
        if domain != url.domain:
            return False
    return True
Ah, so seeing this minimizes the impact of my suggestions above. Your call.

Overall though, I like this approach. I think even though it's an rpm specific example (which you called out as such) it will get the point across well.
above still not ok.  (will fail when card is added)
Ah stupid me you already do that further down. This is for when the OnScanFinished announcement has been sent. Forget what I said.

But the method name in the comment is wrong ;-)
This looks like this could be handled with std::sort() with a sorting operator.
Yeah that's right, this is called for every season of a tvshow. CUPnPServer::BuildObject is not currently well positioned for holding state between calls though.

Could the db call populate these instead in GetSeasonsNav perhaps? This would mean seasons could have different m_firstaired/premiered dates (I'm surprised they don't anyway).
Lots more comments to delete!
It seems there's something weird with github. I *DO HAVE* this line on my local copy.
kind of goes against go's return syntax. i'd leave these returns as-is
No worries, but we like cleaning code as we touch it in new PRs.
It works fine with the line removed, so I might have been low on caffeine when I wrote that. I'll patch that one too.
be sure to add a link to your GitHub profile at the end of the CHANGELOG. ;)
Oh, another thing - since this data structure will be read often, do it like this:

```
struct v8tags {
  char prefix[32 - sizeof(size_t)];
  size_t prelen;
};
```

Avoids a level of indirection and makes it fit in a single cache line. Presumably prelen doesn't have to be a size_t because it's never going to be very large.
Done, I think.
Changed to single quotes here and in the is_arch_armv7 function. Left it as double quotes in option parser.
To get it to fit in 80 cols had to expand to assign to a hard_float variable with standard if/else block rather than inline,
but there's probably a cleaner way.
Yes, I know. However it is written 438 /* 0666 */ in the test in node.js, too.
Besides, passing a number is faster than passing a string.
https://github.com/luvit/luvit/blob/master/lib/luvit/fs.lua#L92
private missing
This might be somewhat redundant, but ``self.excluded`` needs membership tests in ``__call__`` which could in principle be slow if a non-hashed sequence is passed.
Fixed.  Should asserts be used like this?  (I have not noticed many in the code).
I think the code would be less confusing if the listener was unregistered before calling save() again.
тоже поправила
Perhaps it should use concat with two items here, similar to other configurations? (I mean, no other place uses `key=value`, perhaps we should stick with this convention).
Can you use 1.9 hash syntax? Thanks.
Hrm. That e.g. should probably be an i.e..
Yes, that might be the way to go although it needs a little bit of tweaking. I will commit the change to this pull request once I have it working.
I'm assuming this makes life hard for Java, correct me if I'm wrong but I believe the val gets a static forwarder and the object doesn't.
specifiy(sp) for example -> specify, for example,
This else block is both messy and wrong. I think this is trying to test for an Imaginary**Imaginary case, but as I noted above, this should only test if the base is ``I`` and the exponent is imaginary.

Also, the checking of args is a terrible way to do this. First, it misses the case it's testing for (``I`` has no args), and something like this could easily go haywire with an object with one arg that isn't the common Add, Mul, etc. Basically, never check the args of something you don't know the class of.
It should stay flushed left, as that matches how it looks in the Python interactive prompt.
if you leave the ', x' out this doesn't fail.
Actually I looked it up and the guidelines state that methods of a class should be separated by one line, while those that are defined outside of any class should be separated by 2. Even @catchmrbharath pointed this out in one of his comments. So I made the changes to my portion of the code.
Indeed; fixed.
`quick` doesn't seem to be used at all in this function.
NO SIR!
United.
Оно по всем actions фильтрует? А можно указать какие-то отдельные действия, к которым фильтр применяем?
Tripping either of those options will prevent the escalation being triggered. The idea is that `escalations:*` should prevent all escalations for the given pair from triggering, whilst naming a specific escalation will prevent only that one.
I know it's a minor thing, but minimising white space changes would make the diff slightly clearer--but that said, I can see that only this file is affected.
And because var is not on this line I have to go and hunt down whether or not tr_swap is scoped correctly.
Response assertion here also.
Duplicate statement + body
Isn't that the purpose of this PR?
Space before `!==`. Same below.
nit: you could use single quotes to fix single quoting.
I don't think this is necessary. Adding something here is only required if an incoming request needs to trigger other actions besides just a standard response. Like check_schedule.changed and the upgrade calls.
cheers. ill dig into this on the weekend.
in both instances the first ~5lines are the same (maybe extract into a helper function?)
Ахаа, той е закоментиран нарочно. Помислих, че просто си го коментирала в един момент и не си забелязала (на мен ми се е случвало :)).
@silexboy Серега точно знает ответ
Use === for comparison
Normally we want these to be declared as constants, however @TarangMahajan will be replacing these as part of the i18n properties strings
Aye - this is normal. Nothing to see here - move along. :police_car:
The docstring for `dependencies` already states that it's a mapping from constructor argument names to coordinates ... is that not enough?
"resampled"
Is it worth testing that the datatype flows through properly from the dependencies?
That is what I thought - although I am still exploring the Iris code.
Did you mean ``as Theano's variables are symbolic``?
Nothing is being planned here. This timer might not be appropriate. It's certainly incorrectly named.
It's still gonna serialize the 4 byte array length even it's null. and it's still gonna take up the memory, I guess. Not sure what null saves.
can we use instanceof instead of is_a()?  please do so for other occurences of is_a() as well.
Yes, I agree with you that it's a very touch method to break down. The most obvious way is, as you say, with instance variables. I guess that would look something like this

```ruby
def paintControl(paint_event)
  calculate_coords
  paint_event.gc.setBackground(color.to_native)
  paint_event.gc.fillRectangle(x, y, width, height)
end

private
def calculate_coords
  process_opt_radius
  process_opt_width
  process_opt_height
  process_opt_top
  process_opt_bottom
  process_opt_left
  process_opt_right
end

def process_opt_radius
  ...
end

[...]
```

The obvious downside is that you would get seven extra methods (instead of three) that clutter up the file. Oh, and you should change `PaintEvent` to snake case.

I'm honestly not sure which one we should use. I think yours is aesthetically more pleasing and I like the fact that you don't have to use instance variables. 

I think that the method(s) are more coherent from a "what are we trying to do"-point of view in my example and that it's easier to read. If you've never seen the code before, the example I provided might be slightly easier to get a grip of.
The problem is that `deleted_objects` is local to `save_existing_objects` and it's gone when `save` is called. Since we need to keep track of 'commit=False' deleted objects, we need to remember it across multiple `save` calls. We can either move `deleted_objects` over to the instance or keep a new list at the instance level. The former method would break code depending on `deleted_objects`, so I chose the latter.
rspec is pretty stable afaik so I'd suggest "~> 2.0"
* We might want to specifically only target .test files here so maybe something like /test/spec/good/*.test
* We should have entries for the known-good and known-bad directories.
* From reading the git-hub tutorial on git line (https://help.github.com/articles/dealing-with-line-endings#platform-all) endings it looks like we should use the binary flag. That specified that git will not touch the line endings. I think this is the behaviour we are looking for.
I thought I git rm'd this file, will do that now
very nitpicking, but maybe this should read "window elements" instead of "windows" to make things 100% clear
If you're going by http://bugs.jquery.com/ticket/8653#comment:17 then this is wrong and should be:

`param1=`
This does look like an expensive check.  I'd need to know how badly this effects the performance of the sketch aggregates.
I usually prefer this idiom:
```
if 'numAttrs' not in kwargs:
  ...
```

But your method works, so no real complaint from me here.
The argument names (even though probably copied from binary logistic regression?) are a bad idea BTW. First, "Column" as part of the argument name is misleading, because it should also be possible to pass an expression here. Second, argument names should not use capital letters (named arguments then require quoted identifiers).
Firefox seems to be the only browser that is able to send \ to Meteor without changing it to /, and when it does so it crashes Meteor. So, to prevent anyone crashing any Meteor by sending \ to it the whole time, we fix the URL here.
Hm, this isn't what I meant. I think it makes more sense to have the context variable label_name renamed to current_label_name or active_label_name, and then keep the loop variable the way it was before.

What I'm getting at is that the if condition below is confusing, since it's not clear what label_name contains. for label in labels, label.label_name is pretty clear though.

Make sense?
Are we sure this doesn't mean the two conditions are ANDed? I.e. that searching for "apache" means both short_desc and long_desc needs to contain that substring for a log message to match. I think users expect the conditions to be ORed, i.e. that the substring only needs to be present in at least one of the two text fields.
Why not  just use ``if track.availability()`` does this ever return anything but zero or one. (On a side note pyspotify should really make this ``True`` /  ``False`` instead.
Yes, unfortunately, vamp-plugin is harder to package for osx. They changed the build system and it needs to be repatched. I think it would be better to drop it from this merge and try again later.
My pushing this feature hinges on understanding this. I'm guessing it's because Homebrew's python is first in the PATH.
I found a solution. I will update this PR after lunch ...
A small patch like this should be inlined.
`pdfsqueeze` doesn’t have `--version` or `--help`.
What's the difference between PyPy and CPython that causes the duplicate IDs to be an issue here?
This should be in your global gitignore, i think...
here you can use ``std::numeric_limits<uint8_t>::max();`` to use more the cpp way
Do we want to check err, mayhaps?
Sorry, I should've been clearer—the problem here is terminal whitespace, not the indentation.
Why do this if `_androidFixHeight` is just a stub at this point? This should be removed.
Nitpick: This should either be removed or use `mulberry.log`. It handles passing multiple arguments, which doesn't work on device, and also gets excluded from production builds.
Done. jstree uses tabs, so I copied from previous line (which may make a good Brackets feature!)
Docs for keyCmds should indicate type may be an Array & docs for platform should indicate it's optional.
Shouldn't `destroy` and `disable` also have these lines above?
Awesome, I didn't know!
We no longer treat Windows as a special case :)
If you send the parameters as a second argument, it will do a post, and in this particular case I need to do a get.
Yes. I absolutely second that sentiment. Will push a fix later today.