labofsahil/github-event-dataset-2012 huggingface dataset:
Fix Typo in german translation for boss moorabi 29305

<code>UPDATE `script_texts` SET `content_loc3` = 'Jetzt seid Ihr nicht mehr so hart, was?' WHERE `npc_entry` =29305 AND `entry` = -1604016;
-- was UPDATE `script_texts` SET `content_loc3` = 'Jetzt seid Ihr nich tmehr so hart, was?' WHERE `npc_entry` =29305 AND `entry` = -1604016;</code>
fix firewall to answer all login entry points in v1.0 branch

...ing for twitter auth
Added bootstrap button style for context links.

If you think it looks nicer :)
Introduce window system to handle dialogs better

Nothing to merge, but a place for my TODOs, error reporting etc. Please test.
Added responses to Mikesn code of smalltalk.py

I added the majority of 4S multiple responses and a 2-3 new inputs. The new responses/inputs need German language coded into it. This is the only way I know how to paste code to show "differences" so if I did this wrong let me know, and I apologize ahead of time.
Bug hide substitution

Fix a potential SED error and offer a cheap additional hook for customization.
Feature/i18n add Japanese Translation file
Implementation of sb namespace create

Issue #52 - implementation of azure sb namespace create.
DynamoDB Batch Writes involving sets

Passed in to this function is: {"SS" => ["a", "b", "c"]}, the to_a.flatten flattens too much and the variable assignment throws out all but the first item in the set. 

Also, as a note for clarification, I am passing sets in to the original batch write calls. The conversion to arrays (which causes this problem) happens implicitly when rebuilding the request from JSON. This means, to replicate the issue, you can (as this is what I'm doing) set your throttle to 5 write units and send in a batch with 10 writes. This will send back some unprocessed items, and the request will be rebuilt, and this error will occur.

I hope this helps.
Thanks for the awesome tools (not just the library, but AWS in general). 

Peace.
Fix issues in Chapter 3 of the guide

These are mostly grammar issues. I'm not a native speaker myself, but I'm very cautiously checking with google if my corrections actually make sense.

---

I'm not sure if the term `lexical` is common in the Erlang community, but it just doesn't feel right to me. By itself, it has a very general meaning. Since you mean lexical scoping, it may be more appropriate to use `scoped` or `lexically scoped` instead, as in:

> As we are going to see below, they are called directives because they are the only functions in Elixir that **have lexical scope**.

as opposed to the current

> As we are going to see below, they are called directives because they are the only functions in Elixir that are lexical.
don't remove volume shadow copies by default

Per 
http://answers.microsoft.com/en-us/windows/forum/windows_vista-performance/understanding-and-getting-rid-of-device-listing-of/9a39e843-2749-4f27-a64a-09f50eecf736
it might be undesirable to remove these in case of later resizing of
restore space, etc.
Add Shugo Maeda's vimeo link
Fixed Cobertura

The Cobertura build is now fully functional. Cool!
TestYearMonth

Annotated
Added the possibility to skip ssl certificate validation process if needed.

Added a mandatory bool parameter skipSslValidation in TextClient Connect method, specifying if, when using ssl, the certificate validation step should be skipped.
Added a corresponding bool parameter in ImapClient and Pop3Client as an optional parameter so that previous codes won't change.
The default behavior is not to skip the certificate validation step. If the validation should be skipped, it has to be explicitly specified when creating the ImapClient or Pop3Client.

See issue #40 for details.
Review Issue/153

#153
RTL support css files and Persian(Farsi) i18n

To support Right-to-Left, I'm just including the rtl css file after main
css files to overwrite the Left-to-Right css(default), but it'd be
better if we have an rtl option in the widget. it's just working fine ;)
physics work, laser work
Adds the paging library code

This branch adds in the paging library code:
- Default page choice function
- Function to set the current page choice function
- page_alloc, which is a wrapper around sys_page_alloc and should be used instead of it.
GitHubのデザインが変更されたので修正しました。(グーペとプチ)

反映お願いします！！
update AI components, bomb overlapping

- update AI to keep track of latest move for graphics display purposes
- bombs no longer allowed to overlap
- got rid of "BOMB" component in Tile struct
- added checkforBomb function to check if a bomb occupies a space on board
- more commenting added
Removed trace that was added to track TravisCI issue. Will tackle later.
jabber and column-markers

I thought I'd add faces for these two, I hope they're ok. Suggestions/complaints welcome, of course.
return 0 when count does not make sense
Issue173 ff4 regression

* Adding the 3rd parameter to addEventListener (Mandatory on Firefox 4 and below).
Fixed a bug when the cells were not reusable

In case if there are many cell, when the tableview reuses a cell the
new label was put on top of the old one
Add addressable gem dependency to pusher-client.gemspec

found one more
Here goes the brazilian translation

thanks for the module! :-)
fix #9, operators start lines indent

    foo()
      .bar()
      .bar()

    if (a == b
      && c == d
      || d == e) {
    }
1.0.2 version adds support for ie8 cross windwos postMessage

1.0.2 version adds support for ie8 cross windwos postMessage
Recipe commented by default in `main`

I think all lines should be commented by default, right?
Ship aims to where it is moving towards

Also added a few methods to your Math.m file.
Prueba icono dentro de todos los indices
Imageviewer enhancements

* Added support for imgur mulit-image links. Note: It just occured to me that we should ask the imgur guys for a batch api for image info. We could cover an entire link page with 1 request.

* Established a minimum width that an image will zoom to: the smaller of 100 pixels, or the image's initial width.

* I cleaned up the image scaling, and improved the automatic sidebar hiding so it is better behaved.

* I redesigned the gonewild feature so that subreddit moderators can enable and customize the feature for their own subreddits. If you do not want to extend it to other subs then would be really easy to remove the check for the enable link.

* Added an optional filter to prevent the IIV from adding URLs for NSFW links to the browser history. (Behavior is not altered by default, to do otherwise would cause a support nightmare)

GoneWild mode documentation:

> To enable go gonewild mode for a subreddit add `[](/RESGoneWildEnable)` to the markdown code for the sidebar.
> This should not have any visible effect on the HTML.
> When gonewild mode is enabled, by default tabs 'm' and 'f' will be displayed.
> If the default choices are not desired, then use `[](/RESGoneWildEnable-nodefault)` instead.
> If custom tabs are desired append `?` to the link followed by up to 8 label/taglist pairs separated by `&`
> A label/taglist pair takes the form `label=taglist`
> A taglist has can contain up to 8 tags separated by `,`
> Labels and tags can be up to 32 characters long and may contain upper and lowercase letters, numbers, underscores, hyphens, and spaces.
> Labels appear to the right of the "view images" button and are surrounded by `[]` brackets.
> Post titles are searched for any place that an entry in the tag list appears surrounded by any kind of bracket `<>`, `[]`, `()`, `{}`.
> Tags are not case sensitive.
>
> Example:
> To duplicate the behavior originally used for /r/gonewild you would use `[](/RESGoneWildEnable-nodefault?m=m,man,male&f=f,fem,female)`
TIMOB-7281 Added documentation explaining the remove property, and provide example of the new code snippet.

Documentation change. Was tested before sending, but might want to verify the new example works as expected.
Update docs/collections.md
Black dev

Made changes for Wednesday's lecture. Added the clicker quizzes.
Add ability to specify a template function instead of a template selector

This is inline with features in Marionette core. If you provide a template
function to your view instead of a template selector, the renderer will call
the function directly instead of attempting a load from the template cache.
adding some features and examples

+ pointPickerExample
+ ofBackgroundGradient
+ ofDrawBitmapStringHighlight
Added 2 more sites
Added Can i Use Plugin
Added new hooks and changed some forms around

I added new hooks to allow form associations to be added by modules in a hook. In order to achieve this some of the display code had to be changed. I also did a little code cleanup while I was at it.
tweaks to the preference page UI

I added some notes in the pull request, but basically this patch is some tweaks to the pref page UI. Thanks for considering it!
change nuspec to win8/wp71

#199
fixed delegating @Template() to SensioFrameworkBundle when view_response_listener: true is used

When disabling SensioFrameworkExtraBundle's annotations handling, SensioFrameworkExtraBundle's `ViewResponseListener::onKernelView` is never called, and `@Template()` annotations aren't working.

When enabling SensioFrameworkExtraBundle's annotations handling, SensioFrameworkExtraBundle's `ViewResponseListener::onKernelController` is called twice and this results in [this error](https://github.com/FriendsOfSymfony/FOSRestBundle/issues/140).

So it's not possible to delegate `@Template()` annotations to SensioFrameworkExtraBundle currently.

This PR calls SensioFrameworkExtraBundle's `onKernelView` from FOSRestBundle's `ViewResponseListener::onKernelView()` when no `@View` annotation is present on the controller and `view_response_listener != force`, so that delegating to SensioFrameworkExtraBundle works.
Create output dir

It is better to create output dir as with 'logs' and 'boundary' dirs.
Relations refactor: Regain test coverage
fix two parentheses and _

There were 2 error in the readme example. I fixed it.
Added myself to the mentors list

And added in the LPMC Casino to the projects list.
Handling mismatched log levels

Okay, after a lot of fiddling this is what I decided:

*F-- it* with making the code handle this.  It would require mapping log levels *twice* just to go through the psgix.logger coderef. It's easier to simply do this:

    my $logger = Log::Dispatch->new(...);
    enable 'LogDispatch', logger => $logger;

    enable 'Log::Contextual', logger => $logger;

...which would mean that code buried in the $app that does log_critical { $message }; doesn't go through psgix.logger *at all*, therefore nothing is being broken in the PSGI standard.  Of course the $app is also free to do $env->{'psgix.logger'}->({level => 'fatal', message => $message}), if it wants to avoid knowledge of what type of logger is being used.

So, the patches are limited to doc updates. I've split the wording improvements out as a separate commit in case you want to reject the other one.

Sorry again for the confusion. :/
2

done some documentation for set_hook and trigger_events, also updated the part for update_user which is now update
TESTFIX polished tests
Move version to setup file

I dont know if its useful to set __version__ in the __init__ of the package, but it crash when used in a buildout.
some nifty improvements

Hi,

I've seen we can add a id but it only applies to .modal-body DOM element. I often use ajax loaded forms and when modal is created from scratch I can't set a style (width for example).

So I have added data-dialog-class markup which add a class to .modal DOM element.

Another thing :
I have Added default option : autoAddCancelButton, true by default for backward compatibility. When setted to false cancel button is not added into ajax loaded modal.
Pull request FINALE

Ecco prof. Salsano,
la pull request finale in cui c'è un README dettagliato e preciso.

Se accetta questa riceverà 6 commits del Readme, l'ultimo dei quali è quello preciso e completo di fork + pull request appunto.

Grazie.
Luca Marcheggiani.
Overhaul

Finished the UI overhaul. I used the SDK to test things, so there's no XPI in this branch. Logging in to steam every time I wanted to test was a pain, but the work you've done is brilliant!

One thing I noticed but didn't know how to change was that Firefox was growling the notifications, but using the icon_connecting.png image instead of icon_extension.png. Any ideas?
Remove vim Recipie

There appears to be some sort of bug it MacVim won't open and regular
vim exits with non-zero exit code when editing a git commit message
PULL PLZ

can u? plz. thanx.
Allow update statements to set a column to null (DBNull)
fixed problem with test_by_paradata_schema

test_by_paradata_schema was sporadically not passing, as the key be using for schema name was unreliable. it is now "namespace protected" (in a sense) in the same way that other key tests are.
Add table_name_prefix support

This allows the plugin to be used on Redmine installations that have the ActiveRecord `table_name_prefix` option set.
Handle nil spatial values

This should fix both my previous issues #1 and #2.

There is also a small change to gracefully handle the case where enable_load_extension is not defined.
Acceleration fix

New way of dealing with acceleration, allowing use of rad/s². MotorRotation is now no longer a template class, it is now full of doubles. Also edited some comments here and there. Also added another parameter to StepperMotor's WriteRotationData that allows us to choose whether or not to use deviation. This parameter defaults to true, so it should not affect anything old.
Added two modules that I have been working on.

mwGearman - gearman support; currently haven't done too much with it other than getting some initial pecl gearman support baked in that I needed.  Putting in some support for workers soon.
TwitterBootstrapFormDecorators - this is based on the twitter bootstrap 2.0 edition; been using this for a bit now and we may or may not expand too much more on the suite; really just waiting for Zend\Form 2.0
Ch13 relecture

Relecture du chapitre 13.
Comme d'habitude, une autre relecture est toujours utile.
Thanks for your useful and interesting program!

I've tried this version of psmc_plot.pl . I find the history earlier than about 2 Myr disappeared while plot in other regions remain the same. 
Does the "last time parameter" mean the "excessively large ancient population sizes" in the PSMC paper?

I use this with gnuplot Version 4.4 patchlevel 3 (March 2011, System: Linux 2.6.18-194.blc). 
There is an error message like this "line 0: ';' expected". 
I fixed this error by change line224:  
qq("$prefix.$i.txt" u 1:2 w st not 2, ); 
to 
qq("$prefix.$i.txt" u 1:2 w st ls 20, ); # 20 is an undefined line style in here.
I think this error is caused by the version of gnuplot..
typo (is has)
May have been because I pushed a fix to my repo. that was already pushed to main by someone else so the changes cancelled each other out.
All arguments are under direct programmatic control inside this class except for $attachments which contains XML code generated from the DOM which should therefore already be correctly escaped.
Ah, in all Google code (including Chrome) there's a macro to do just that.  I should probably copy it over eventually.
http://codesearch.google.com/codesearch#search/&exact_package=chromium&q=disallow_implicit&type=cs
it seems ppc is handled like am (which is good)
--unitsize=1 for TEG binaries
Cool, noch ein Kommentar zu den commit messages:

Die erste Zeile sollte ohne '-' in maximal 50 Zeichen einen kurzen Überblick geben

Danach kommt eine Leerzeile

Danach bei Bedarf genauere Beschreibung, evtl auch mit Stichpunkten
New version of the patch.
I do not know the effect of the code, try to download and use it.
Hi, why is minitest needed? I think this is only 1.9.2 specific?
Díky, nechal si tu spatny autory ;)
Agreed, this lock is really not necessary.  I'll remove it.
@tenderlove cool, I will backport it today
@Tie-fighter I'm not so much concerned over violating the ToS, I'm concerned over all the innocent people that got hurt, albeit ever so slightly :p I think he should just have created a second account and tested everything that way. I'm sure they would've forgiven him for creating a second account had he just reported his findings afterward.

@MeachisM Ah you're right, he did show 2 days ago that he had found at least one vulnerability. There's no indication he told GitHub about it though, he was trying to get the attention of the Rails core team to change the default for mass-assignment attribute protection, which I definitely agree with.
My commit doesn't have anything to do with the breakage, reverting my commit indicates so. I am confused too why it passed with the commit before it..
I guess I did something wrong with my changes, I only added dojo/mouse to line 1 and changed line 62 in _handleSelect to 
if(mode == "extended" && !ctrlKey && !(this.selection[this.row(row).id] && mouse.isRight(event))){
:) About /proc/stat:

All of the numbers reported in this file are aggregates since the system first booted.
thatch45: Will do!  I was holding off before requesting a pull since I wasnt completely sure if a rehash was needed.  However, I definitely comes in handy if you are already SSH'd into your minions and have to manual rehash.

SEJeff: Thanks for the correction I really appreciate it!
Hmm... if we had a generic AR guide it would make sense to have this there. On the other side, a Rails application may not use Active Record (this is the price of choice for documentation, where do we put this stuff?).

As a compromise I think we could leave it in the configuring guide by now, in the end it belongs to the config directory of the majority of applications, many people may think of database.yml as part of the application configuration.
Avoid nulls, rather match against the option.
Typo spotted: GeneratEor?
Hm but patch already got merged - submit a new one either way?
gettimeofday does not do what its name suggests, so everything's alright, sorry.
if you get bored, you could add a test that ensures only admins can access this page.
It is my repo test, this works to the perfection if these in weapon not crash but if these in fury it throws it
i would rather insert a random word (to be consistent with other DS)
require('ember-data/adapters/rest_adapter');
require('ember-data/adapters/tastypie-adapter');

diego me peta... tu ultima version pq falta requerir estos ficheros..., te vuelvo a sugerir q separes en 2 proyectos ( dejar ember-data ) normal y ( tu taskpie-adapter en otro )... siempre puedes desde nuestro proyecto de data... sobrescribir las clases de ember-data de la siguiente manera:

```js
Em.View.reopen({

});
```
If the ball travels more than its radius before subsequent frames, we would not be able to track it.
Wow, dude, you didn't waste any time!

Looking over it right now.
Looks awesome!  Excited to see the new results from this.  Just a few small comments.
feature/database-grades blev mergat före felen blev fixade
@richarddong 
ruby.taobao.org
是什么。。
Thanks for the feedback. It's fair enough, I guess I'll try to provide more explanation in the future.

I recommend to bring the discussion in the issue tracker (filing an issue is easy, cheap, and fast). This GitHub commenting system is very badly searchable, I can't cross-ref easily and not everyone has photographic memory.
This needs a test for maintenance since it's an override of default Django behavior.
I is not the exact price of the spell, feel free to edit this to put the right price
lol no comit it's line modified on my last commit (this post start ligne 10)
行尾加个句号吧 : )
numbers of strings approx from 2 weeks old commits.
В логе http://pastebin.com/Q9NWty6M
ok, i'll put this on a issue
is it necessary to filter out events associated with contexts which are not in view?  perhaps this will just result in extra refreshes of the same data?
Why do we need these here anyway?  I don't think we should hardcode attributes at all.  Maybe the model can pull from widget metadata?

Please open a bug on M6 to track this.

@waynevicknair
Also this is not needed.
>  Albeit, for some reason, when I fly around the audio speeds up/down.

That's a feature  ;)

http://en.wikipedia.org/wiki/Doppler_effect

(albeit I may have set it up too strong to see it working)

> Yeah, we can probably upgrade misc_sound.html. Or duplicate it and do misc_sound_chrome.html.

Yup, something like this.
Does this work for you?
http://forums.silverlight.net/t/242033.aspx/1
I thought that when calling layers = iface.mapCanvas().layers() you get the current set of selected layers . I guess you mean that the text in the dialog should be updated dynamically when layers change?
You're right, thanks. I've to find some workaround but I don't see any obvious one…
oui, je commente ton code.
Yeah, sorry about that. I think making sure that it's 0 to start with is important because it makes sure we know the state of things before we make changes to the state of things and test. I tried to mark all the places to make it easier to just throw it together.
Ctrl+R not working.
If you're impatient, an outdated version of the information I'm going to be throwing up here is available at: http://opensource.osu.edu/~paradigm/bedrock

Since then I've changed quite a bit - for example, rather than schroot I'm using capchroot.

It will take me a while to get things on here, as this is a relatively low priority and I've got my plate pretty full.  I expect progress to trickle in until the summer hits, when I should have much more time to get this into full swing.
Still not happy with the oemr.org site. Lots of outdated links, horrid download page, etc. Does this info need to be on the login page. Why don't we just make a standard 'About' page (next to the Manual link) that provides this link along with other needed disclaimers etc. Could even move the copyright link there and make the login page a "clean" page with non of these detractors.
Ben c'est toi l'admin du projet XD Il n'y a que toi qui peux les ajouter au projet
You're probably right. I changed it.
Yeah, but there are a couple of things that I know need to be done. I'd love to get the conversation going though.

It still needs to generate an `ember-runtime.js` if that's important, and the license is only added during the rake task which I could see being a problem. And the way I faked out minispade so it doesn't complain about missing modules when not testing the spade distribution is pretty lame. Otherwise, I've been using it for a couple days now with no problems. I'll send a pull request tomorrow morning.
Works for me too as I was using the static reply before. Sorry for the confusion.
Use unicode objects rather than str objects. I added a text containing an "ű" character and I got this error: 

    UnicodeEncodeError at /conversation/1/
    'ascii' codec can't encode character u'\u0151' in position 7: ordinal not in range(128)

The error disappeared when I modified "str" to "unicode" in this line.
i have done it witout commit beacuse i use it in my server xD if you need it i can add a config :)
@farmisen Is this an intentional commit that should be made part of this "pull request" or something that the GitHub system tacked on by mistake because the pull request hasn't been closed?
>
> The Seesaw dep in project.clj was wrong. I fixed that so maybe that will
> get it working for you?


Thanks Dave. That fixed it for me. The piano roll looks nice Jon! I'm
looking forward to using it.

Something I would like to see in both the piano roll and the sequencer is a
per note/trigger volume control. It would be nice to Alt-click a note and
drag up or down to set the volume or velocity of that note. The notes could
be color coded to indicate their volume with red being the the loudest and
blue/green being the softest.

I'm not sure what the best way will be to use this information, since not
all synths will have an amp, vol, or volume parameter and not all synths
will use the same name for this parameter. Using replace-out would be less
than ideal, since a synth may want adjust it's behavior according to volume
of the note/trigger. Any thoughts?

Kevin

On Sun, Jan 15, 2012 at 11:17 AM, Dave Ray <
reply@reply.github.com
> wrote:

> Cool. Looking forward to trying it out.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/overtone/overtone/commit/063996cfaad9441b479f793ba1b57ea4d93e75b5#commitcomment-868830
>
Sorry my mistake, that is because actually a ```string a = something;``` calls the std::string constructor, not the ```operator=()```. 
Because of this you have to do ```std::string a; a = something;```.
The rest is $@*! ;)
@bougyman @manveru @swk http://travis-ci.org/#!/rubyists/fs_specs/builds/631669 shows what I mean. Compare this with fs_cli with LOG 7 both when the test is running and immediately after. Yes, we're pushing out to the socket on L175 and L201 but I don't see in fs_cli where vm-enter_pass.wav is ever presented. If our extension was being accepted we wouldn't get re-prompted with vm-enter_id.wav 4 times, then vm-abort.wav.
For me, vim replaces each character in the selection with a ^M. This is with vim 7.3 on Ubuntu with no .vimrc (same thing for me on OS X with vim 7.3, but there are a few entries in my .vimrc there)
I had actually thought that was the case, but hadn't got round to fixing it. Thanks
That is part of my testing plan and I am concerned about that, however, ``containsBounds`` is called immediately before the ``initSingleTile`` function to determine if the ``initSingleTile`` function needs to be called.
shouldn't you configure the result cache driver through the config and simply use ``useResultCache`` for the query needing to be cached ? This would avoid hardcoding the use of the APC cache in the controller
thanks. I've used them in my next commits 3fb383c70cbddc56256984555d3eda27071d4ee2 and de4b7a1acffebbbd207675b95690f0ad28031a56
I've taken out the question mark option - results with questions marks need a different replace, see e.g. here: http://www.worldcat.org/search?q=isbn%3A0313304483&=Search&qt=results_page 
Lots of other fixes there, too.
db is comming
That's a draft prefix used by Opera which is a "vendor". So yes it's missing :)
Then lets not!! niceness has nothing to do with it. Lets just do the right thing.
Perhaps no one was there to tell them better. There is now.

Never be nice, be right (If right is also nice that's that then)

Cheers
Boaz

On 01/30/2012 02:25 PM, Eitan Ben-Amos wrote:
> i usually see first how people work & then adapt to the environment - 
> maybe i'm too nice :)
> but thats the way it used to go so thats how i continued - nothing more 
> than that
> 
> Eitan
> 
> 
> 
> From:
> Boaz Harrosh 
> <reply@reply.github.com>
> To:
> Eitan Ben-Amos/Israel/IBM@IBMIL
> Date:
> 30/01/2012 02:20 PM
> Subject:
> Re: [nfs-ganesha] RQUOTA RPC program number was wrong, (285d22d)
> 
> 
> 
> I understand, which answers my "register" keyword question.
> 
> But since you are touching this code - adding __attribute__ ((__unused__)) 
> - 
> why not just remove that line? it's unused, no?
> 
> Boaz
> 
> On 01/30/2012 02:15 PM, Eitan Ben-Amos wrote:
>> HI,
>>
>> this is machine generated code so i usually dont touch the code itslef
>> that's how previous patches to XDR generated code were made
>>
>> Eitan
>>
>>
>>
>> From:
>> Boaz Harrosh 
>> <reply@reply.github.com>
>> To:
>> Eitan Ben-Amos/Israel/IBM@IBMIL
>> Date:
>> 30/01/2012 02:06 PM
>> Subject:
>> Re: [nfs-ganesha] RQUOTA RPC program number was wrong, (285d22d)
>>
>>
>>
>> If *buf is not used why declare it __attribute__ ((__unused__)) ? why 
> not 
>> remove it's definition?
>>
>> Also I hate the use of "register" keyword. It's totally dumb. The 
>> optimizer just ignores it and does what's right. And the -o 0 ignores it 
> 
>> as well. If it would help even with a 0.00001% we'd use it in the 
> Kernel, 
>> we don't believe me it's stupid.
>>
>> Cheers
>> Boaz
>>
>> --- 
>> Reply to this email directly or view it on GitHub:
>>
> https://github.com/eitanb/nfs-ganesha/commit/285d22dd5c57b4b52dcad3298c5d4ec9dacc57f1#commitcomment-913987
> 
>>
>> --- 
>> Reply to this email directly or view it on GitHub:
>>
> https://github.com/eitanb/nfs-ganesha/commit/285d22dd5c57b4b52dcad3298c5d4ec9dacc57f1#commitcomment-914013
> 
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/eitanb/nfs-ganesha/commit/285d22dd5c57b4b52dcad3298c5d4ec9dacc57f1#commitcomment-914022
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/eitanb/nfs-ganesha/commit/285d22dd5c57b4b52dcad3298c5d4ec9dacc57f1#commitcomment-914030
The functional part is already covered by several tests. The logic for logging is straight forward, I don't think an UT is needed for that.
Isn't code review fun?!
It's nothing to do with loading, its to do with reference access. How do you get to $this in a library without referencing get_instance()?

This does not need to use a class, it's a globally available server variable...
why not "pong"? or renaming the command to marco and let the server respond with "polo!"
changing SendMessageToSet(&data2, true) to
SendMessageToSet(&data2, false)
is enough-at least for me
Try this:
        var serviceName = req.params.id;

Also, home not used :)

And I think it's safer to delete service.auth; ?
Good point, I'll fix. Thanks
@zalun: r?
#commitsicantfave
This assumes that everything will use Messaging.send
・通知関係の挙動変更
・投稿周りの処理変更
・エラーチェック類の改善
・画像投稿に向けての準備
Just wondering, why not use [`_x()`](http://codex.wordpress.org/Function_Reference/_x)?
What don't you get? The official north was changed, so I updated the map (output by /f map) to comply.
a little weird for `map` to map a Range to an Array, no? Shouldn't it map a Range to a Range, if anything?

is there a reason you wanted a `map`? I've never wished for it or I would've written it, the only use I can think of for it is `MathBlock.prototype.latex = function(){ this.map(function(e){ return e.latex(); }).join('');`, but I think `fold` is actually more readable because `map` returning an Array is so weird.

the `return false` is like `break` is following jQuery; `each` doesn't have any other use for the return value of the function, so it's fine, but I see why you'd want to make it consistent with `map`. I feel like the most important point here is that we've never, and I don't foresee ever, actually having a reason to `return false` in an `each` iteration, but I get exceptions thrown inside `each` iteration due to `undefined` property access etc all the time, and this would make debugging those a good deal more annoying.
bad idea to handle such things in the **view**. By the way,  Symfony2 already registers `app.session` as a global twig variable, why not use only it ?
I don't know if it works with THIS commit, but scaffolding stills works with the latest commit ;-)
Doing a `git clone` here is probably not ideal. As a user, you end up with the entire git history of the base12 project.

The alternatives are kind of a pain though:

- You can use `git archive` to get a gzipped version of the repo without the .git dirs. This sucks because you really want it uncompressed, so you essentially need to pipe it back to unzip.
- You can use `git checkout-index`, but that's only for the current repo, so that's pretty much out

It looks like the way Rails does this right now is with a templates directory and a script. https://github.com/rails/rails/tree/master/railties/lib/rails/generators/rails/app
Thanks.  I think it just stops me from messing up an excellent blog :-)
Lazy ppl is lazy https://github.com/TrinityCore/TrinityCore/downloads
@evertontrindade try to maintain this addAttribute on your code, to see if it still works, plz
Working on that at the moment. Just booted the MacBook.
How could a enum rename fix anything but people's confusion?:)
closes #4908
omg, first DLOS now Warden! 2012 is definetly Trinity's year! thank you soooo much!

So anyone alredy tested?? please report your success stories asap!!
As groundspeak does not publish documentation on their objects (to my knowledge) I can only presume the meaning of 'li' :)
My tests showed that it switches to false alongside the pm flag and that the coordinates returned started to get scrambled at that point.
For basic members the behaviour of 'li' and the exactness of the coordinates was the same.
Just a test for github.
un consejo mamona, aquí pon un "DELETE FROM", con la id o guid (si la especificas) del creature que vas a añadir a continuación, así se permite meter este codigo varias veces, que sino, si lo metes 3 veces te salen 3 barrets.
Criando método para enviar o email de contato.
Is there some way we can reuse (a lot of) code from stream-parser.js?
This is not needed - those resource paths are the default values. All this is doing is to inflate the pom files, making it harder to find the stuff specific to this bundle.
It's black magic, miss. You need to go to Faraway Mount on a snowy New Year Eve with an Orange Pegasus, find the great Dragon inside the white castle on top of it, defeat the dragon and rescue Twilight Sparkle. She will teach you white magic. Yah, so after that you still don't have black magic. I know it sad, don't cry. :P
Please confirm if this also fixes data for you:
https://github.com/marc1706/desire_kernel_35/commit/4b1737cdc19379fecfd19b192dbcad7f9d684428

I already tested it on my phone and it works flawlessly.
As long as PHP_VERSION doesn't get reported as "5.4" it's ok, however:

    php -r "var_dump(version_compare('5.4.0', '5.4.0', '>'));"
    bool(false)

:(
I'm going to try to fix this now. Read the comment for why.
For From Scratch, I feel like we'd want to keep the custom.php file and the introductory comment explaining what it is.
They were 'auto-generated' based on my work in the gui layout tool. You're absolutely right, though. If we want this to be more generic, we should be able to create these programatically.
right, thx ;p
ok I will merge this
This change is also needed in TypeStruct::dotExp().
So, could we merge this?
Need to pass through `*args, **kwargs`, in case subclassed `.final` needs to see them.
Ca ne revient pas au même ?

Sinon j'avais mis {1,2} pour pouvoir matcher à la fois %05d ou %5d, qui sont tous les 2 acceptés par printf et également dans nuke.
Change XHTML to HTML5 commit for valid HTML from W3C
#done splitting in to two modules. please refer to http://dev.metasploit.com/redmine/issues/6144
more normal to combine this with the line above.
Excellent!

I've been wanting to write a mock server myself, so that's great.
I'll look tomorrow at the first part of the commit (data->sampled_data). Looks like a mistake, yes.

Thanks!
thanks for the fix!
It's not a big priority for me right now, so if you feeling like putting together a pull request for some specs, I'd definitely be game.
Please either kill all of these lines, or move <tt>createReport</tt> up and block-comment the rest.
C'est ce que j'avais essayé au début, mais du coup, la boucle qui suit ne fonctionne plus pour l'url qui est quand même définie comme absolue. A voir.
Im Zuge der Umstellund der Unternehmensrichtlinie "mehr Dokumentation, weniger Sinnhaftigkeit" ist dieser Beitrag ein unverzichtbarer Bestandteil der Unternehmensstrategie 2012!
Found an extra hyphen.
Test for "ithub for mac"
I have no idea what happened here, but it's hilarious. I think eclipse's formatting engine is a little broken.
Why have you removed the code property ?
Broken indentation. ;)
I am fine with leaving it as is for now and address it for the 5.0 release. I created https://hibernate.onjira.com/browse/HV-556 to keep track of this
Just noting a spelling mistake in my commit comment (I really need to stop coding at night).

"chown" not "shown"
You also need this frameworks/base commit:
https://github.com/aospX/platform_frameworks_base/commit/fb84fc989f9bd4c883747caa77c526658f9d6f73
Also needed for Mac Lion + RVM + 1.9.3p0:

BUNDLE_BUILD__SQLITE3: --with-sqlite3-dir=/opt/local
wait but this can happen if the 4s is banned! and cannot create an assistant.
Sorry, sorry.  The pb.cc and pb.h files got included with it so that's why it's so massive.  Just ignore those, they're autogenerated by ProtoBuf.
haha, I am here!
siteLastModifiedTime > DateTime.UtcNow.AddSeconds(30)  is allways false.

maybe  siteLastModifiedTime.AddSeconds(30) < DateTime.UtcNow) is correct.

Is it any relation of next issue ?
https://github.com/WindowsAzure-Accelerators/wa-accelerator-webroles/wiki/known-issues
I wonder if some non-default value of the white-space CSS property (applied to *.rib-remove selector matches) would help resolve this in the layout view (added to composer.css)?  See: http://www.w3schools.com/cssref/pr_text_white-space.asp
change event will refresh property view so that click event of datalist is not triggered..
A whitelist of the `.*` files we want to track seems more maintainable than a blacklist of the ones we want to ignore. Can we change this to block all `.*` files except `.travis.yml`?
This call to canWriteToContainer doesn't match with the function signature. I think you need a 0 as the first argument.
same comment about the value of the ```serialVersionUID``` ?
I may be missing the point, but you replace ```credentials == null``` by ```hasCredentials(credentials)```. I was expected the opposite : ```!hasCredentials(credentials)```, that's you did indeed line 221 - 224 !
Why an abs here and nowhere else? What if it was -1 which is considered undefined above?
ok, will do, it's simple possible flow control
the scheduled SendBatch messages will be ignored if acks not enough acks received
no retry is performed, see L869
this is in done state, when we have received one ack, it lives for one idle period of ReceiveTimeout to collect other InitJoinAck (to not pollute deadletters)
OTOH it looks like this is a left-over
I thought the standard was to use case-object if it is an ADT
why stripMargin?
this "and" is not needed (and is misleading).  path_setting always returns a path.  There MUST be a redirect here, even if there isn't a *thanks card.
No, I'll run through and add tests.
hmm the ignore_missing in the schema should help -what problem where you getting?  was it that somewhere this was getting updated?
You removed it from the call to the constructor, yes. But not from StatusMixin's __init__ itself.
hehe i was torn between American vs British ;) fixed!
In a few cases, I found we needed to define the class and it's parent before loading other things (typically automatically based on constant loading).  In this case, I think we want all the stuff that the 'Card' reference above does complete first before doing any other loading.

It may also be that this is no longer needed with the re-org in sets.rb.
le get va pas pete si c'est None...ca serait dommage sur un toString
Necessary ?
This looks very similiar to other types of upload. Some helper function might be in order to do this for a given texture id.
actually this whole area of the code might be duplication?
Do you really need to throw an exception here ? How do I change a transaction attribute in a method using this plugin ?
still no good. You use tabs right?
make the file end with a newline?
Tweaked! Let me know what you think.
commodities (spelling). Also, what are the results of the comparison? What does it mean for a commodity name to be less than another commodity name? (is this a stupid question?)
reStructuredText provides [definition lists][1], so you can format this using that feature.  Also, you can build the docs in local:

```console
$ cd docs/
$ make html
```

and then built docs can be found in `docs/_build/html/` directory.

[1]: http://docutils.sourceforge.net/docs/user/rst/quickref.html#definition-lists
It also makes it easier to document how the retries functionality works
need a removeClass call here in case someone turns off this property at runtime
This should do an early return if false, instead of putting the whole method body in an `if`.
This enum should be `TUIActivityIndicatorViewStyle`.
Kind of superfluous, isn't it?
s/getid/getgid/
@TooTallNate Sure thing.  [215](https://github.com/seebees/node-1/blob/13e47b299dfcd2685aadc7d6fffb8de79efc1f61/lib/repl.js#L215) and [216](https://github.com/seebees/node-1/blob/13e47b299dfcd2685aadc7d6fffb8de79efc1f61/lib/repl.js#L216) as well?  And do you want me to update [this](https://github.com/seebees/node-1/blob/13e47b299dfcd2685aadc7d6fffb8de79efc1f61/lib/repl.js#L240) while I'm about it?
this URL is invalid. the slashes got dashed.
I am somewhat uncomfortable with this line as creating a symlink seems like a huge hack. Ideally, we could make this work without this hack by passing the right options to `./configure`, or failing that, modifying `configure.in` to be better about discovering `libcrypto.so.$VERSION`.
Yeah, I guess either you have to do the implicits all the way where the types are still known, or use the classes. Using the classes seems a lot simpler.
I added it so that in the future we can support decommissioning nodes. Actually adding a deccomission option was out of scope for this JIRA.

Can remove it if you think it's confusing to have here.
`if key not in all_keys:`
may be add --no-install-recommends  ?
OK, I see the following text clarifies this.
yes i think so but  i tried the most clear... i'm investigating...
Hmm... Not sure `tr` is going to work in this case:

```ruby
>> 'password_confirmation'.tr('_confirmation', '')
=> "psswd"
```

It'll replace all matching chars, including the ones in `password`.
"Short docs do not need an explicit "Examples" label to introduce snippets; they just follow paragraph <http://edgeguides.rubyonrails.org/api_documentation_guidelines.html#example-code>
May be the naming is bad, but this is one of the key points of this commit.
@exviva it's just additional check. Someday we can change class attribute to another value and then it will rewrite even if it's nil and it can cause mess. So, it's not really necessary but we need it :)

I will provide 'em!
Is there a need for both skip and the migration_error equals to :none? They seem redundant, i would keep just migration_error.
But ofc described problems of Selects with NO_NAME's are still need to be fixed. But now I don't know whether this is still a good place for them.
where do we use PIL?
I do. `iterdicteq` handles this. This is around because it's hard to put dicts inside of sets due to hashability
these are no longer in lexical order...I wonder if that matters.
Seeing as we eventually don't even want auto expansion, I would just remove this.
you should run your code through gjslint. it will tell you that jsdocs should end with a punctuation mark :)
can you pls add the issue ID to the fixme then? (that we don't loose the connection here)
thread.name != link.writer and thread.name == link.reader

no need for parens around ==/!= comparisons due to operator precedence
Wie bekomme ich es hin, das es nach dem erfolgreichen speichern in die DB wieder zurück auf die parent_group_view geht?
You also need to wrap long lines (80 chars).
Remove before each execution?
What happens when you already run daemons before it and have them running?
Removing pid file you can't control it then.
Gotcha. I will change the files back to the original format. I also did not find any logger.fatal or logger.warn functions being called, so if you think you put them somewhere, tell me.
What theory do these data points get executed in?
I think "with dot notation" rather than "by applying the dot notation" 

also dot notation should be in the glossary.
Suggested addition:
If you know who's responsible for the component, be sure to mention him in the request by his username (e.g. `@github-id`), so he'll be notified.
Should probably at least log an error if it cannot get a driver and therefore cannot delete the physical avatar file.
Please add a comment explaining when it is going to be available and not displayed.
`$cache_dir` is a string, not "mixed"
Put `$search->get_common_words()` result in a variable?
@param missing.
It should probably be like reCAPTCHA - Disabled until a key is entered.
This should be done under allow_avatars check. What is the purpose of obtaining an avatar manager if avatars are disabled?
clean_row does not belong in avatar manager. If you want to put it in a class, put it in a phpbb_avatar_helper.
It seems htmlspecialchars is applied once the variable is requested by way of phpbb_request_type_cast_helper::set_var. This is already tested for in the phpbb_type_cast_helper_test.
[duplicated note] I don't like the name, specially because we are overloading YUI.use() with this definition. I probably prefer something like libutils.yuifactory or libutils.getYInstance() or something like that. And it is ok to reuse the same Y under the hood, YUI does that as well.
что за класс?
you can have multiple values returned with a custom "extractor" as described in the example, but you can't have multiple constrast, without multiple calls (which can be done in a custom extractor to). Actually, the main API of the adaptor is the callable extractor -- all the other bits are convenience, because they will probably be used a lot and avoid the need for a custom extractor callable in some cases.

I do not immediately see how the changes you are suggesting would work without imposing restrictions on the design matrix, i.e. it would need to come from sa's in the dataset. If you want to do feature selection based on tvalues for a particular
regression, you can just ask for res='tvalues' and select based on the corresponding row in the result dataset.
Nit: style on the previous three lines (end with commas) is perhaps more readable but can sometimes lead to silly errors. perhaps better to be explicit
var search = value.split(' ');
var regexps = [];
var i;
imports are out of order.
Actually, we might decide that this now becomes ```'v1.1.0-dev```. I will leave that to @rhattersley to decide.
Merge the effect of combinePlanNode into this function and just have it return AbstractPlanNode.

Determine whether you have 1 or 2 List elements from aide.compile -- BTW, these are called "fragments".
If you have 2, assert that the second is a SendPlanNode and attach it to the other with the new method.
catalog version is sized but not included?
verifyt that the form adds empty serial number
I think the more action-oriented name fits with the fact that you have to get an instance of it and call start/commit - the other backend things are single instance. Suggestions welcome for new names, though.
btw, I think you can remove the case for 0 as it should never happen.
not sure if it's the same but maybe something like this should work:

```ruby
field :request, :type => Hash, :default => {}
```
Thanks, I liked that, I'm going to update the code
* _calendar使いましょう
Yes, META.json is necessary in the zip file.  I think we could automate PGXN deployment including this file, but the "how" is still not clear.
Sweet, this means we can get rid of the crazy sys.argv munging! :-)
You didn't bring over the `--delete` option vs non destructive reindex?
We have it on test_add.py file line 62
I read the thread in question, this gist of which appears to be  "I might consider something like it in the future, but for now the Homebrew people can be responsible for it."

Sigh. This is not how we do things.

It does appear that there are one or two OS X users willing to help ensure this functionality works, but the parties involved in that are you and those users, and not Homebrew.
It won't really work properly if you do that unfortunately; pretty sure it will download and think it's the newer version in that case.
You can't have context arguments.  We pulled that out.  We could add it back but this time only allow them at the end of the argument list similar to how :type => :list works.
i have forgetten to put it into gitignore... it's an eclipse project file abolutely useless for other
Ah! Now I remember you telling me. Thanks for the reminder.
@daniel-uzunu: use a bulleted list here too
@rmocanu: weren't we using the one var convention?
Jo passt erstmal findich
Yep sorry, I’ll fix this!