labofsahil/github-event-dataset-2012 huggingface dataset:
[RFC] ini Parser

Dear, i have wrote a ini parser using range as BidirectionalRange and maybe Infinite RandomAccessRange.

section class has:
- empty, front and popFront() that InputRange requires
- save() that ForwardRange requires
- opIndex() that RandomAccessRange requires
- the value of empty to be known at compile time as false

Example of use:
```d
import std.string;
import std.stdio;
import std.ini;

void main( ){
    writeln( "0 - Starting test" );
    IniFile tester = open("myConfig.ini");
    writeln( "Ini file loaded" );
    writefln( "1 - Name: %s, level: %d", tester.name, tester.level);
    writefln( "2 - childs:\n%s,", tester.childs );
    writefln( "3 - object is null: %s", tester is null );
    writefln( "4 - number of section: %d", tester.length );
    writefln( "5 - sectionA: %s", tester.get("sectionA") );
    writefln( "6 - sectionA.param1: %s", tester.get("sectionA")["param1"] );
    writefln( "7 - first section:\n%s", tester.front );
    writefln( "8 - last section:\n%s", tester.back );
    writeln(  "9 - popfront" );
    tester.popFront();
    writefln( "10 - first section:\n%s", tester.front );
    tester.popBack();
    writefln( "11 - last section:\n%s", tester.back );
    writeln(  "12 - create a section" );
    Section toAdd = new Section( "sectionD", tester.level + 1);
    writeln(  "13 - append pair key/value to this section: D = is fun" );
    toAdd["D"] = "is fun";
    writeln(  "14 - Append the newest section to main section" );
    tester.addChild( toAdd );
    writeln(  "15 - show the section added" );
    writeln( tester.get("sectionD") );
    writefln( "16 - show all childs:\n%s,", tester.childs );
    writeln(  "17 - shrink for remove unused space");
    toAdd.shrink;
    writeln(  "18 - write current section object to a file" );
    File inisavedFile = File( "soEasy.ini", "w");
    inisavedFile.write( tester );
    inisavedFile.close();
}

```


Output

```bash
$ ./test_ini
0 - Starting test
Ini file loaded
1 - Name: root, level: 0
2 - childs:
[[sectionA]
param1=value1
param2=value2
[[subSectionA]]
param1sub=value1sub
, [sectionB]
param3=value3
param4=value4
, [sectionC]
param5=value5
param6=value6
],
3 - object is null: false
4 - number of section: 3
5 - sectionA: [sectionA]
param1=value1
param2=value2
[[subSectionA]]
param1sub=value1sub

6 - sectionA.param1: value1
7 - first section:
[sectionA]
param1=value1
param2=value2
[[subSectionA]]
param1sub=value1sub

8 - last section:
[sectionC]
param5=value5
param6=value6

9 - popfront
10 - first section:
[sectionB]
param3=value3
param4=value4

11 - last section:
[sectionB]
param3=value3
param4=value4

12 - create a section
13 - append pair key/value to this section: D = is fun
14 - Append the newest section to main section
15 - show the section added
[sectionD]
D=is fun

16 - show all childs:
[[sectionB]
param3=value3
param4=value4
, [sectionD]
D=is fun
, null, null, null, null],
17 - shrink for remove unused space
18 - write current section object to a file

$ cat soEasy.ini 
[sectionB]
param3=value3
param4=value4
[sectionD]
D=is fun
Setting the host to 0.0.0.0 still works with localhost, but also allows ...

...the server to listen on network-accessible IP's automatically.
ABI Draft v2: Le proper formatting

Seperate paragraph for second definition
+missing dot
Add error subcode to FacebookApiException

Add error subcode to FacebookApiException
Added depency org.eclipse.jdt.feature.group to work properly on Mac OS.
Bugfix in BatSubstring.compare

Fixed a bug in compare : compare (all s) (all s) would raise "index out of bounds".
Also switched the value returned to match the one of String.compare.
Using `Pervasives.compare` to compute minimum

As mentioned in [this issue](https://github.com/ocaml-batteries-team/batteries-included/issues/277), this eliminates any issues with `nan` values.
Refactored some View Helper and Redesigned New Developers view

Refactored New Users and New Modules
Added test for a further type in odbl=clean
Bugfixes & Security

Only small changes:
- facebook_config context processor crashed once. so it will now check for facebook in the session
- feincms middleware did not use the new path for the redirecter.html template
- securing redirect to app data. check for possible defacings.
Twig based mailer

Here is a proposal to implement #282

This new mailer implementation allows getting rid of the current hack using the first line of the template as subject.
I kept the old mailer by default for now because the new one has a hard requirement on Twig and people overwriting the template would need to update it.

My plans are to release 1.2 soon by keeping the compatibility with Symfony 2.0, because of #464 and to bump the requirement to Symfony 2.1 only in the next release relying on new features. The mailer used by default could then be changed in this release (which will probably be a 2.0 release as I plan some changes to clean things) to use the TwigSwiftMailer implementation.

Btw, if someone has a better idea to name the class, feel free to suggest it.
Implemented the initial control panel
Review: Allow trace() to specify optional "traceset"
Update location.html

Flightsite href typo in location.html
Extend documentation for UsageMeter return values
Added composite site for soa tools 3.3.2.Final
Update requirements.txt

Fixed problem with python package file (so we can add to the cheese shop)
analyse: work started (may not even compile :)
Gingerbread

gps is broken in the last few releases due to some missing files.
this should fix the issue.
Test Fixes - socialofficehome

notification test fixes
Add support for specifying the P4USER, P4CLIENT etc variables in the sublime Perforce.sublime-settings

If you put things like
    "P4PORT": "perforceserver.com:1666",
    "P4CLIENT": "Seb.client",
    "P4USER": "seb",
    "P4PASSWD": "password"

in the perforce sublime preferences file, the p4 info command succeeds. I was having issues with getting it to pick up my config environment variables on Linux and this should also allow people on Windows with registry issues to fix them.
Switched over to .show().

Fixed your needing help logic
fix brush name

Conflict with Cpp
Merge changes from superamerica into our team repo

Just updating our team repo with the other teams changes.
Add section to download cURL in Windows installer

@WalterBright This pull request relies on [curl-7.24.0-dmd-win32.zip](http://gnuk.net/d/curl-7.24.0-dmd-win32.zip) being uploaded to `ftp://ftp.digitalmars.com/` before it should be merged.
Extraction du nom des installations

Bonjour!

J'ai extrait le nom des 38 installations et je l'ai placé dans un fichier "fr.json". Pour afficher le nom de l'installation, il suffit de récupérer son code (exemple: "BallonPanier") et de le passer dans le dictionnaire fr.json pour obtenir "Ballon-panier".

Cela permet de faire (et de réviser!) la traduction des 38 termes très facilement: suffit de faire un fichier "en.json" semblable, et le tour est joué.

Le premier commit modifie tous les noms d'un seul coup.

Le deuxième corrige une coquille que j'ai trouvée.

Le troisième enlève un nom d'équipement dont nous ne nous servons pas: "École" n'apparaît qu'une fois, malgré que nous ayons 4 écoles. Je l'ai trouvé en faisant des tests unitaires de validation.

Voilà!

Philippe
Отключить по-умолчанию удаление exif

>Error: Image corrupt or truncated: blob:baf46e49-1182-4e4a-a233-4f4c74d4f400
Source file: blob:baf46e49-1182-4e4a-a233-4f4c74d4f400

Видимо, в некоторых случаях портит файл. Пока не исправлю, пусть будет выключен.
Employees grid

Hi, 

I implemented it in a separate project within the solution - hope it's not a big deal. Also added `.gitignore` and deleted a few unneeded files from the repo.

Play with it and let me know when you want to discuss the next steps!

Thanks,
Andrew
Failbranch

Yo I herd you like to test for failing tests so I failed your tests so you can test that they fail
Fixing small doc bug

Ref #19
Fix dashboard reload button after orientation change

The reload button on the dashboard didn't work after a screen orientation change. The syncbridge couldn't get its activity: getActivity was null in AbstractSyncBridge line 103.

I suspect this is because it was detached from its original activity since an orientation change destroys the activity. I've fixed this by removing the existing syncbridge in the ReloadableActionBarActivitiy#onCreate if it existed before and adding it again.
Jellybean

Hi again,

as promised, here are the changes for the kernel build. The new kernel source needs to be in CM10/kernel/huawei/hws7300u

mblaster
Improved context menu functional tests [manual]

+added tests for custom contexts
Adding new dispatch event to allow extensions to listen for webview crea...

...ted. This adds the proper events for Webworks and LinkedIn.
Pull req: Clean up build scripts, strip binary files for size and lint c++ code

These changes clean up the native extension build system to lint, strip and combine unit tests into a unified CMake and configure-qsk script

Also fixes issue #89
Added my files they're great
template, dynamic form

Hello, 

Following this thread https://github.com/runekaagaard/django-crispy-forms-fancy-formsets/issues/3 , I made some changes in source code to allow dynamic creation of forms using standard formset.emty_form .

Also improved a little the table rendering ( '+' icon and remove some css class for fields)

Regards,
un po' di pulizia del codice applicativo (non ho guardato quello delle spec), niente di più ;-)
Update res/values-ru/arrays.xml

fixed typo
"экстримальный" - "экстремальный"
PLFM-1425

fixes for PLFM-1425 and changes to improve migrator performance
Sb trunk oldxul

I think I finally got this to work??
cleanup: Tweaks to systemd-pndwip-image.
kapitel 5 und 7
Added duplicate checking and other stuff

* Company button is disabled rather than hidden if the user is not an admin
* If the message length is < 1, the custom format UI is hidden
* A message will warn the user if 1 or more of the SQL connections was unsuccessful
* When Importing, name and id conflicts are checked (against both company messages and the file itself)
* The refresh button will check for name conflict with local messages, remove those messages, and notify the user.
しもじまです

jQuery Mobile使ってちょっとだけスマホ対応。
途中だけど一応pushの確認用です。
various fixes

Hello! I made a few small, quick, tweaks when including these files in a project. Primarily this includes support for use with both GC and ARC. In one spot I was lazy and fixed a potential leak with GC by using CFBridgingRelease, and just removed the release instead of wrapping it in an ifdef for ARC since CFBridgingRelease makes the object it returns autorelease.

* fix header when used without pre-compiled header
* fix implicitly atomic properties to be explicitly atomic for objects and explicitly nonatomic for BOOL
* fix potential memory leak when using garbage collection
* fix usage with ARC
* fix lack of newline at end of file
* small formatting nazi changes
fixing background image sizing
FIX: Deprecated use of Object::add_extension and has_extension

After updating to the recent 3.0.3 release, the JS snippet was not being added to my pages. I tracked this down to the line 34 of `GoogleConfig.php`: the condition `Object::has_extension('SiteConfig', 'GoogleConfig')`was not being evaluated as true.

Looking at Object class I’ve seen that Object::add_extension and has_extension are being [deprecated in SS 3.1](http://doc.silverstripe.org/framework/en/trunk/changelogs/3.1.0#upgrading), so I tried changing the above code to `SiteConfig::has_extension('GoogleConfig')` and it worked.

I don’t understand why these two functions are not working as expected in 3.0.3, so I’m not sure that this commit can be applied, but it’s the only way I’ve found to make the module work.

Hope it helps,
Juan
Separated creation of new start time from creation of new assign id.

This was causing duplicate start times to be created when the create event didn't run smoothly the first time.
Dutch localization

Hi,

Thanks for creating this plugin.
I have added the dutch localization file for the attachment reminder.

Best regards,

Jaap-Andre de Hoop
Update to support Varnish 3.0 Syntax

Main change is purge -> ban. Also removed the stats command since it is no longer supported.
First pull?
Finition aliased_english

Je crois que le titre a tout dit.
Gnome3.4

Hi there

I made small changes on the extension, now it hides on overview mode.

Thanks
Include bash profile.

...exists.
Oops, sorry. Have fixed this now.
That probably doesn't work: generic method names don't contain a back tick indicating their generic arity.
Yeah thx from me too. And i will dog down murphy who leads us devs to copy pasting code and will burn his family down *grrr* :D
Fixed https://github.com/rails/rails/commit/61c5b4a9f6b953dd3f027b1a24c5526771bf8968
I doubt that Github failed at "reading the documentation". If Rails is so serious about providing the best tool to developers and it's conventions being the best practice, then it's important that the framework establish a convention of more strict security by default.
@filipeamoreira Probably just missing attr_accessible, see: https://github.com/rails/rails/issues/5228
No doubt.  The problem is only that it's not properly tracked and I don't want to pile on too many hacks.  There are any number of principled ways to do it, but it may have to wait.
So, I'm new to this, and my current problem may be unrelated, but yesterday I found that I could no longer add images to my products in my development environment.  I think it was working last week, I think I've run 'bundle update' and I think the thing that has changed is paperclip changing from 2.5.0 to 2.6.0.

I'm posting here just in case others are about to find the same thing.

The errors I get are:
[1] with gem 'aws-sdk', attaching an image through spree admin:
...lots of lines of information, ending in....

```
 1664 /Users/steve/.rvm/gems/ruby-1.9.3-p0@rkbbspree/gems/aws-sdk-1.3.4/lib/aws/core/response.rb
 1665 /Users/steve/.rvm/gems/ruby-1.9.3-p0@rkbbspree/gems/aws-sdk-1.3.4/lib/aws/core/http/response.rb

[NOTE]
You may have encountered a bug in the Ruby interpreter or extension libraries.
Bug reports are welcome.
For details: http://www.ruby-lang.org/bugreport.html

Abort trap: 6
```

[2] If I remove aws-sdk from the gemfile, and I remove the s3.yml settings from the config directory, run bundle update, I get

```
LoadError in Spree::Admin::ImagesController#create

cannot load such file -- aws-sdk (You may need to install the aws-sdk gem)
```
Tienes razón estimado George. Creo que es buen cambio para hacerlo flexible ante ambos casos
These don't totally need to be here, right? They should be available already on `butter.cornfield` as per the module system.
Please add 'echo = True' so booth directions are logged in log_file.
dont forget the innodb bit on create table
`.Clear()` and `.Dispose()` it in your destructor.
Hm. I think I lied. Very strange. It wasn't working for me earlier on Mac, but now it works fine on Ubuntu. On Mac, it's freaking out.

Video demonstrating bug here: http://www.youtube.com/watch?v=H4FwrBB-oOo
calculareBdcFromSubscription is actually not such a good name since that value may come from the account
I don't think slice is the right term for this-- boolean index I guess?
Awesome!
Do you think whether there is a way to also get also results for blocks and methods?
forgot zip.close() ?
This looks good, please make the small changes, and submit upstream.

Also, this updated commit lost the needed change to deoptimizer.h to remove the mips-specific kNumberOfEntries.

Also, I assume that the commit comments (under "Changes: ... " were for my code-review benefit, and that they will be removed when this is submitted.
most current games use the version tag correctly and most maps didn't set a version tag... so there should be very few current files where it will show this message.
masz na myśli komentarz do konkretnych linijek?
Since Greyhole is executed as root, I think it should at least make sure the file it's about to execute isn't writable by anyone but root, with something like:
```

$ok_to_execute = FALSE;
if (is_executable($file)) {
	$perms = fileperms($file);

	// Not user-writable, or owned by root
	$ok_to_execute = !($perms & 0x0080) || fileowner($file) === 0;

	// Not group-writable, or group owner is root
	$ok_to_execute &= !($perms & 0x0010) || filegroup($file) === 0;

	// Not world-writable
	$ok_to_execute &= !($perms & 0x0002);
}
$contents = $ok_to_execute ? shell_exec(escapeshellcmd($file)) : file_get_contents($file);
```
should we host the jquery here, or use the cdn hosted version?
I think, whitespace shouldn't be included in the token.
Allow *plugins* to extend permissions

Herp derp
curious about the OrgMetadata naming scheme for methods.  is this referenced in docs?
Well, test_requirements doesn't get touched unless you ``python setup.py test``, so I think we're pretty safe. :)

I was considering of getting rid of the setup() param altogether and just leaving the .txt with the README instructions.
See now this is how you get into trouble. You have changed a constant 8u/4u/2u into a function vec_step(). You are ASSUMING that AMD will do the right thing and this will get converted into a constant during compilation, which will end up being the same speed. Now given AMD's history of fucking things up, you can't trust that this is the case on all SDKs. Given that it cannot be faster than passing a constant, it cannot be of benefit and there's a distinct possibility it will actually be slower on occasions.

Moving the base to the end of the line is fine if you have evidence it helps on one platform. The other changes I cannot approve. Patch denied.
Heh. Nice catch :-)
No such thing as "related_field" anywhere in the docs, so that's why I assumed you meant to_field
> So how could GUI add any of those neat features when it is limited to the very limited "CLI is everything we need to support" -thinking.

In this case... if we add the language enum and have a "default" language as I suggested then you can add the feature if you want.

If you don't agree then I can show it.
isn't this code the same as in IsWithinDistInMap?
это как то касается вот этого момента ? 

<a target="_blank" href="http://radikal.ru/F/i069.radikal.ru/1203/ef/a46f43f8dc68.jpg.html"><img src="http://i069.radikal.ru/1203/ef/a46f43f8dc68t.jpg" ></a>
c++11 gonna prevent this mistakes with override methods
oooooooh aaaaaaaah yeeaa, got it, works nice
when updating the bugzilla whiteboard, would I have to run a fancy SVN revert command to remove the rubble this script just created when it failed?
I re-integrated this commit as it wasn't in my latest version. Strange
On a semi related matter: We might want to put the ScummVM release version in the README, by using an README.in file and let update-version handle the updates.
Why?

Delete user still has the old log format...
can't you use the nifty corners here? to avoid the browser-specific stuff?
Set Instance Data has nothing to do with the last boss I guess. It does "My encounter is done." and then can be doors opened for example.
Missed a leading space here?
Persona propietaria de la Agenda
You made changes to boost headers? Is it ok?
Hi,

is the manufacturer "Rittal" known in France? They are some kind of a de-facto datacenter-supplier when it comes to racks,environmental monitoring etc in Germany. I have a check_rittal_health which checks at least temperatire/humidity sensors
Gerhard
I think Arthur's username is @arthur-debert
Yes, the rhino jumping sound does sound completely different on level 1-1, though. I can make a recording of that with the same build as well if you'd like.
most certainly, I was suggesting that you should address the possibility of an unknown `option`. your change works for me.
Es que lo estaba escribiendo y justo me mandaste el mail por eso no lo puse... Era el tema del sleep en el assert_diff...
Now that tests are working, would you take a PR that re-enables xunit runner? Or would you prefer to leave tests disabled, like a bad person who does bad things?
I have absolutely no idea what i'm doing. Now Netbeans gives me an error saying that CommandBase is a duplicate class and because of that it wont compile.
No worries mate.
Don't forget to change both versions here.
Patrick, very funny. I added something almost identical without seeing this pull request :). I merged this in since you are also printing the message.
Most of the changes here are indentation...for some reason everything was indented 5 spaces.
honestly the mkdir and links could probably be joined together as well... so we're only pushing a single command... should i refactor?
@mbruker: The rationale here is that this can be a first step toward making the server require registration without completely shutting it down for non-reg users. Many players have a lot of issues with unreg users trolling them.

There have also been several studies showing that only about 5% of people change the defaults on anything they use, which is why this should be the default and not the other way around.
pfft. Nonsense. :-) 
Heck, I'm very happy with the code + comments as they give me enough hints to grok it rather easily. It's just that when I read code my eye (brain) barfs a hairball when it encounters some linguistic pothole while speeding along the text road. I.e. 'bablefished' manuals for electronic equipment make my eyes water.
I'm not a native speaking (American) English individual so when I write stuff my mistakes happen mostly on another plane (fubarred structure, for instance).

I just hope this edit is useful for the next anal retentive bastard that comes along. :-)
Commenting a commit
I think this bit might be incorrect.  BoundedConcurrentHashMap is not part of JDK 1.5
Thanks!  I should have done my due diligence and read the ticket, because I see that it's fully described there :)
So what do you think the value is for?
No, you can just remove that reference and link.

On Tue, Feb 21, 2012 at 10:35 AM, Omar Patel <
reply@reply.github.com
> wrote:

> I actually haven't created a lesson 7, yet. Is that required before I push
> the lesson? I just wanted it in Github, because I wanted to minimize WIP.
>
>
> ________________________________
>  From: Ron Evans <reply@reply.github.com>
> To: Omar Patel <omarpatel123@yahoo.com>
> Sent: Tuesday, February 21, 2012 8:52 AM
> Subject: Re: [kidsruby] lesson 6 (cd1ed4c)
>
> Is there a lesson 7?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/hybridgroup/kidsruby/commit/cd1ed4c18f9b148bfcb52f1a6219d1c4f781c3b8#commitcomment-992067
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/hybridgroup/kidsruby/commit/cd1ed4c18f9b148bfcb52f1a6219d1c4f781c3b8#commitcomment-992520
>



-- 
Ron Evans
The Hybrid Group
http://hybridgroup.com
p. 310-916-9924
m. 310-597-1013
> What is this serve for?
CSS uses this classname.
OS: Win7.
Called error: imagettfbbox(): Problem loading glyph
Im not sure what you mean about the shell, but good point about gettext. Is there another convention we should use here instead?
I'm talking about inline JS that you may want to insert in your fields, for fields' behavior (calendar, color-picker, whatever)
Thanks Tom. I can't quite tell the difference, but I also don't have Lion yet so maybe it's not as obvious to me. I looked at it on Zach's computer, though, and the swipe behavior does have some momentum to it that doesn't seem to be affected by tile loading. In the [current code](https://github.com/stamen/modestmaps-js/blob/master/src/mouse.js#L41) your `mouseDiv` appears to have been renamed to `_zoomDiv`. @tmcw, any idea what might've happened in the translation from the anyscale example to the `MM.MouseWheelHandler` class?
In the future, it's generally best to use all lowercase in the option_id (no need to change here, but try to keep lowercases for future coding)
Haven't tested to see what this looks like but why the hard-coded widths? Percentages are usually better than pixels.
yes, for javascript, but for xml and css and may be other languages, there should be stuff like `//{{{.my..` also if a plugin author wants to redefine `//{{{` for other c-like languages. ...
This passed with sopts is passed in.
Can L37 be [title_image, title_text].join(" ') ? Currently it is too restrictive, as i read. If someone don't want to set a title to admin page, simply sets an empty string to title_text.
Not mine.

-----Original Message-----
From: Thomas White [mailto:reply@reply.github.com] 
Sent: February-18-12 12:29 PM
To: Tony Partner
Subject: Re: [LimeSurvey] Fixed issue #05764: If enter Y-scale and X-scale at same time, and click save, only Y-scale is saved (3e868ee)

Can you also push this to the _dev  branch, please?

--- 
Reply to this email directly or view it on GitHub:
https://github.com/LimeSurvey/LimeSurvey/commit/3e868ee1357956fb6c701772536b3d24bc0c1cf1#commitcomment-981321




=======
Email scanned by PC Tools - No viruses or spyware found.
(Email Guard: 7.0.0.21, Virus/Spyware Database: 6.18900)
http://www.pctools.com/
=======
This breaks the symbolic class labels feature that I implemented some time ago. Can we remove the dtypes here? The actual fitting is going to be done on (partial) 1-of-K codings anyway.
let's call this local var what is is. also, you could do:

``` ruby
Dependency.where(unresolved_name: name).each do |dependency|
```
I think, this commit should be rethinked; possibly reverted and done in a more clean way. See inline comments for details.
You should use "let(:user) { FactoryGirl.create(:user) }" instead.
So you return nil instead of 0 on equality?
When using parameterised queries there should be no issue with inserting strings containing newlines.
The test in this commit does not fail for me on PostgreSQL 9.0 when I remove the above change.

Am I missing something? Is this a problem with earlier versions of PostgreSQL?
All right, do a "make clean" then try this:

https://github.com/evanmiller/ChicagoBoss/commit/2abba2b1db3dd068ba61206a5514caba902a4396
I think so. Adding the Selenium issue link to the comment would help, I agree.
tu es libre de supprimmer ce que j'ai fait dans Rectangle, ce n'est pas tres utile >.<
Ah et je n'ai pas mis de tête au bonhomme xDD Il en aura une, tkt
Fun fact : J'ai passé 1/4 d'heure à chercher pourquoi P4 bougeait alors qu'il était déclaré Fixe, j'avais juste laissé les coordonnées de P1 dans l'affichage... Copier/Coller POWA !
It's not too hard for custom scripts to adapt to this commit, if you can't figure out how you shouldnt probably deal with custom scripts in the first place
Delicious, delicious... something.
merge-approved
avalable? 

don't you meen: available ?
Always translate.
Please see my comment on the https://issues.jboss.org/browse/AS7-3949 , on the server side the type of background-validation-millis is actually represented by Integer. We might reach issue here once entering values > Integer.MAX_VALUE
There is a test, but it didn't test specifically for url, it tests for name's presence (it tested for absent members of DEVELOPER_KEYS). Since we only care that name is present now, the test is strangely somewhat more applicable.
Why is the writer null here?  The xml writer is specific to the schema version.  Why not use JGroupsSubsystemParser_1_0.getInstance()?
Ooh , only 1 level in vanilla ?
Could you bring out a new recommended build when you're done so I can put it up on the thread (vanilla build)
Dun dun duuun!
| was thinking of removing the tabset instead and using the existing Admin tabs?

This was carried out in https://github.com/silverstripe/sapphire/pull/159

Folder: https://github.com/silverstripe-scienceninjas/sapphire/commit/57fed26234cc2f5d26fdece954d17103c4baa63d
File & Image: https://github.com/silverstripe-scienceninjas/sapphire/commit/733b51d340f82e4f184022c724a7d4b4246a32c9
I agree I must change this code to made it clear what is happening. Thanks for pointing it out.

On 4/01/2012, at 1:11 PM, Cvg131072<reply@reply.github.com> wrote:

> Yes, whatever. But why? How about:
> return response unless response.success?
> @audit = response.audit
> 
> OK, 2 lines of code instead of one. Hmm...., I have to re-think it.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/23inhouse/active_merchant/commit/1123ee78de713fa70841f4e5254478a6b6bf1c9e#commitcomment-835823
I don't think this will work. If '\t' is not found, i+1 will overflow.
Better to introduce a bool d_isquery to replace d_ownsdata.
This is so many different plugin authors can use our libraries. If we don't do this, and let's say MV is on AllPay 6. WolfPound will also load 6, and blow up, because it wants 5.
I believe the format is dictated by the user's settings, at least when it comes to 24/12 hour time. locale on UIDatePicker has been deprecated in iOS 5, so you can't change it anymore.
No, the ATT SGS2 has exactly the same partitions, ask Entropy512 it ;)
We're generally focused on 1.9 these days anyway, so I would be fine with
expecting json to be available. Do you feel like submitting a pull request
for this?
Yes, I tried to implement that. :) I hope, it's useful. The only thing that I'm not 100% sure is the test (see comment above)
Why did you remove the db/schema.rb file from the CVS? Loading the schema.rb file is the safest way to build a new DB instance, since migrations are not reliable.

Thoughts?
TODO: change the class of the `body` tag on clicking using some JS after the AJAX request, currently you need to reload the page
Currently it's not a big deal as OSGi 4.3 is fully compliant with OSGi 4.2. So it means that we support both spec, including the newest one.
The problem will occur if a new spec is not compliant with the previous ones. At that time, we will create a branch dedicated to the spec version update.
hm after i did it i got:

Fatal error: Call to undefined method Symfony\Bundle\SecurityBundle\DependencyInjection\SecurityExtension::addUserProviderFactory() in /Users/smartius/Sites/handelsdeal/vendor/bundles/Doctrine/Bundle/DoctrineBundle/DoctrineBundle.php on line 39

here is my deps:
http://pastebin.com/JfJFwqW1
Fixed.
https://github.com/SthephanShinkufag/Dollchan-Extension-Tools/commit/93f0733217a830d741bd9736d678ac7bf071065e
Not decimal seconds. I'm glad that it works, but the returned value for the header will be an integer value.
FYI I had this error on OSX and Win7, haven't tested linux, but it seems coherent accross platforms
The missing style (I knew there were 67 lines of styles!) seems to be "aol" - it's listed in in the CSS, but not on the sample page.

How about another feature point under this one:
 - generic primary and secondary buttons
No you're right. I couldn't test all of this code until the C code was updated. Here I forgot that chunk sections count 0-15 from bottom to top, not from top to bottom.

If the bottom of the map is being cut off, that means I must have miscalculated something elsewhere.
My concern with recommending vs imposing is that BV won't be responsible for the parsing  / processing of `@ValidateGroups` and the interceptor will have to give the list of groups manually. (see method `validateParameter` and co). Likewise, if we add more info to @ValidateGroups, we will need to change the `validateParameter` and co signature

Alternatively, we can:

- add a context object to these methods
- let them accept a `ValidateGroups` annotation instance
- or let BV's engine find and read `@ValidateGroups` annotations entirely but you don't seem fan of that approach
works for me for now. just curious though - any better way to avoid the race? would be nice to help keep our test runs short
Thanks for the tip!

Won't that cause problems with recovery again, though? Or has it been fixed in bootable_recovery? Well, I vaguely remember an alternate fix for the issue being implemented in recovery_ui.c...
Yes I have thought into this direction, but I didn't really find it compelling. 

If we want to take that route we'd have a more or less empty intermediary `MethodDescriptor` object (which then must not extend `ElementDescriptor`) which would make invocations a bit more complicated/longer. Are there any particular advantages you see for `ReturnValueDescriptor`? I'm not having a really strong preference for either approach.
this is a great change.  i would suggest something other than "refactor" for the commit message for something like this -- a refactor being, by definition, something that does not change the external behavior.  i'd never have noticed the change without searching through the closed issues to find: https://github.com/bcardarella/client_side_validations/issues/260
Have you fixed your issues bill?
What is the value of indata?
The extra space before [0] is kind of bleh, no?
Ok, here's one of those newbie questions...

You updated the website code last night. I want to sync my copy on my
laptop with that update. So I've tried the following in Ubuntu terminal:

git pull git@github.com:andoverarchers/andoverarchers.github.com

It then asks me for my password, and on Ok seems to do something without
errors. But when I go to edit the files, it doesn't appear to have updated
them.

What am I missing here? :-P

Cheers, Philip.

On 19 January 2012 11:16, Marc Tamlyn <
reply@reply.github.com
> wrote:

> Largely waiting to be honest... Google will get there now your dns is
> working.
>
> M
>
> On 19 January 2012 11:09, Philip King <
> reply@reply.github.com
> > wrote:
>
> > Indeed. How do we go about raising our search rank btw, or is this a case
> > of simply waiting?
> >
> > Cheers, Philip.
> >
> > On 19 January 2012 10:59, Marc Tamlyn <
> > reply@reply.github.com
> > > wrote:
> >
> > > Hooray for google apps!
> > >
> > > ---
> > > Reply to this email directly or view it on GitHub:
> > >
> > >
> >
> https://github.com/andoverarchers/andoverarchers.github.com/commit/1467d43401d01f2f73a8ca834726e2af92e3e472#commitcomment-881667
> > >
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/andoverarchers/andoverarchers.github.com/commit/1467d43401d01f2f73a8ca834726e2af92e3e472#commitcomment-881703
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/andoverarchers/andoverarchers.github.com/commit/1467d43401d01f2f73a8ca834726e2af92e3e472#commitcomment-881724
>
I will definitely add more containers - this is just a start :)
0755 is all you'll need for write and execute for the system. 

0777 is a bit too insecure and unnecessary.
@vitalema @SparkSovereign @sayamification 
Just letting you guys know if you want to access the stored password at any time then you can call the method "giveUsername" for username and "givePass" for password
That was my doing - I'll have a look. I'm already running code that uses
spot instances with iam profiles although I don't remember off the top of
my head how I do it

On Thursday, December 27, 2012, Topper Bowers wrote:

> In lib/fog/aws/models/compute/spot_request.rb:
>
> > @@ -29,9 +29,14 @@ class SpotRequest < Fog::Compute::Server
> >          attribute :monitoring,                 :aliases => 'LaunchSpecification.Monitoring'
> >          attribute :block_device_mapping,       :aliases => 'LaunchSpecification.BlockDeviceMapping'
> >          attribute :subnet_id,                  :aliases => 'LaunchSpecification.SubnetId'
> > +        attribute :iam_instance_profile,       :aliases => 'LaunchSpecification.IamInstanceProfile'
>
> I would tend to agree if that works. However, I was copy-pasta-ing from
> https://github.com/fog/fog/blob/master/lib/fog/aws/models/compute/server.rb#L50
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/fog/fog/pull/1426/files#r2511906>.
>
>
It's a good idea to actually instantiate something significant from the library to make sure that linking is working correctly.
Ha, funny one. I will reformulate this a bit. The idea is right. :-)
I am not sure how the return value would be consumed by the callback function. Gotta double check.
This is assuming that `==` means `is`, correct? Would it then not be more expressive to actually use `is`?
I'd rather pass the actual objects to keep the WikiaDartHelper "full module"
ojoj!
for the non-physicists it might make sense to add a comment that the constant is `log(2)` (or make it a named compiler constant)
What if `currentChannel` throws? I think this should stay inside the `try {}`.
Never use "should" ;-)
I think that the state must include the unreachable members from the latest gossip, and that doesn't always reflect a converged state.
You could do:  

    import cluster._
but it wasn't really optimized before either
Should also verify that there is 1 activation and 1 deactivation per ref, and that all refs were activated and deactivated.
yes, see https://github.com/akka/akka/pull/623/files#L2R86
>       @override def toString: String = "null"

Wouldn't that be confusing?
That does not really do anything, or? It’s enough to set it once.
yuck.  let's fix this.  shouldn't be needed.
This wont be added back, if there are problems with the new code, track them down, there are large advantages to using the new code.
Replaces groups with organizations?
I've added an exception raise to scheduler.remove if the schedule doesn't exist
This will handle all such cases
Pet peeve: can we change this to "information"? We can spare the horizontal real estate.
If we default this to some value, we can skip the nil check.
probably better to use a switch / case - for better performance
line 777 at the subtitle part it's used too. i copied it from there.
let's centralize the configuration access for this
You mean as opposed to `ManyUniv`?
Missed that this happens only on head requests. Why does `GetMany` get called twice?
Why pass in `object`?
Yeah, container-padding may have been a better choice, but I don't think it's worth a change. I think of that as "padding applied to the entire grid" while this is "padding applied to columns".
Please remove the extra spaces you added around the concatenation
Create a `Twig_Loader_AdvancedInterface` would be better.
Using `layer.contents` will remove the need for this conditional.
.... what you say is indeed true, so I guess I'll make that change back, then.
Oops, I've got that.
TYPO: should be *or* not "nor"
@satra I think you had it right the first time, but the ordereddict module also needs to be added to the dependencies in setup.py if the python version is less than 2.6
I don't understand what you mean by "unit is the sample spacing unit".  

I think it would be helpful to say something like

    d : scalar, optional
        Sample spacing (inverse of the sampling frequency). Default is 1.
Array of length ``n//2 + 1`` containing the sample frequencies.

@rgommers I'm not sure about the double backquotes, but I think single backquotes are reserved for arguments.
コメントが追加されている
I don't know what documentation you are referring to, I don't see RETURN_STRINGL doing any free or efree. What documentation says it does?
Hehe, here we go again, I meant this one as well: `CentOS` ;)
Yes, sorry, I was from phone, can't write more

Something like this:

```ruby
each_with_object({}) do |(key, value), hash|
  hash[key.to_s] = value.is_a?(Hash) ? value.deep_stringify_keys! : value
end
```

It's the same as you described
As one line it will be too long - about 120 symbols and I think it's harder to read.
Maybe better with shallow indent? Something like this:
```ruby
    seconds_to_advance = (
      (options.fetch(:seconds, 0) +
      (options.fetch(:minutes, 0) * 60 +
      (options.fetch(:hours, 0) * 3600
    )
```
or this:
```ruby
    seconds_to_advance = \
      (options.fetch(:seconds, 0) +
      (options.fetch(:minutes, 0) * 60 +
      (options.fetch(:hours, 0) * 3600
```
ActiveRecord::Associations::Association#reset invokes this method and passes in the target, which is sometimes an Array. I actually don't like this behaviour and don't think that resetting an association should evict anything from the IdentityMap, but that sounds like another discussion.
master is only 1.9 compatible. So you can use `columns.map(&:freeze)`
How about min, max, and step attrs?

http://www.whatwg.org/specs/web-apps/current-work/multipage/states-of-the-type-attribute.html#date-and-time-state-(type=datetime)
This locking granularity seems too fine grained.  I would guess it's less performant and offers no more thread safety than master.
Looks fine for me :+1:
yes, that is better. The `extend ActionMailer::Async` can be called from the `async=` setter on `Base`
I never really figured out whether it was intentional or not, but ANON_CLASS_NAME is a substring of ANON_FUN_NAME, so the above is redundant.  You can just check for ANON_CLASS_NAME.  Which has obvious unintended consequences:

    package foo

    object bar {
      class anonymize { }
    }
    // scala> afterFlatten(intp.terms("foo").info.members filter (_.isAnonymousClass))
    // res0: List[$r.intp.global.Symbol] = List(anonymous class bar$anonymize)
I haven't debugged this thing but where is the error coming from? If this is from resolveClassTag then tag should be empty, if the MissingClassTagError is a problem then I don't think newArrayApp needs to be created at all.
You need spaces between your operators as PEP 8.
And I continue:

>>> and we won't be able to clear the cache every time an Add is created, the order of the args
>>> will still be effectively random, but instead of depending on the hashes, it'll depend on the content of the cache.
>>> I don't consider that an improvement.

I consider this test only as the task how to manage the objects with the predefined order of arguments manually. I can create those objects, but if I want to begin to deal with this managing then you I need to clear the cache. So the improvement is the possibility of managing itself (technical).

As regards dependency of caching on previous calculation, there are no problems with the caching of `x+y` and `y+x`. Printing system sorts the terms independently, so the user will not see the actual internal things. And the other functions in SymPy must evaluate the result independently of .args order.

On the other hands It can seems as the contradictory to the target aims of this PR.
But the target aim of PR is to remove sorting *from flatten*.

The cache system is separated too and does not influence on the other parts.

if one of the the variables `x` or `y` will be setted as non commutative or they will have another different assumption, then the expression still will be ok, (and the test will passed correctly, btw) disregarding on the cache system.

The main aim of this issue is to prepare to deal with non-commutative working of expression in natural mode, the task to separate the parts of code logical and clear: flatten nested, automatic simplification of expression, canonicalization, handle with infinities and so-on - in places where it is needed. 
I am not sure that it will be easy to do although, but it is strategic task.

Yes, Chris, and the  customizing of Add routine too.

As I understand thread in the mail list [alternatives to subclassing Expr](http://groups.google.com/group/sympy/browse_thread/thread/aca9ea914fe22b8e/) some problems which raises too:
It is hard to inherit from Add class when only one key line changes needed, It have to write the similar code and tests and maintain it, or inject the additional code to the Add.flatten itself.
No, this specific case is wanting to expand `f**n`, where f is known to be an Add and `n` a positive integer.  We only want to expand it top-level as a multinomial.  All the deep handling stuff should have already happened above.
I am making a modification that removes the decorator and makes the
kwarg argument processing a bit more pythonic. I will need some time
to finish it as I have also other stuff to do. When it is ready we can
discuss with examples whether the decorator is necessary.

I am sorry that this takes so long, however I still do not agree on
your use of decorators and the way you process kwarg arguments. I may
be wrong, so the best way to find out is for me to actually try it.
This looks like serious pattern matching stuff; I'm impressed.
Whitespace-only.
On 4/30/2012 4:14 PM, Jeff Balogh wrote:
>> @@ -58,7 +58,22 @@ class Storage(StorageBase):
>>       def __init__(self, sqluri):
>>           self.engine = create_engine(sqluri)
>>           Session.configure(bind=self.engine)
>> -
>> +        self.metadata = MetaData()
>> +        # build the tables
> sqlalchemy should be able to figure out the schema from the Model definitions. I decided not to put `metadata.create_all()` in here because that would run some unnecessary sql statements every time a process starts.
>
> Usually I would make the tradeoff for developer ergonomics, but avoiding setting up a db one time doesn't seem worth it. This is covered in the readme: https://github.com/jbalogh/push/#readme.

I missed the database creation step from the Readme and got snagged on 
the error stack. Adding the auto-create was trivial, and I figured that 
it would save folks setting up servers from hitting the same issues I did.

Plus, this reduces the number of interaction files required, since the 
schema would be defined in the file where it's used, and not potentially 
missed in future fixes/updates.
what part of the indexes are case sensitive? commenting out until we're clear here?
Why upsert:true?  If you do that, is there a need for the counters.insert above?
Agreed, I was actually thinking the same thing. Thnx! ;)
I think it would be worth commenting why you're expecting this result in restart.txt.
@EXreaction you can use index_page_header for that
git GUI prints this nice set of characters:

> [1mTrailing whitespace detected:[0m
This role sounds reasonably legitimate, however it should become the one and only role provided for "private forums" (and I still would like the naming of that to be improved).

This one role should contain the reasonable default for a user role that would be used for "private forums".
Also, I notice repeated use of request_var() (above and below this line). As far as I am aware, the $request object should be used in all new code.
Probably want to set the `$config` as class property in the constructor so it doesn't has to be globalised in all the methods. As is done in #835
https://github.com/phpbb/phpbb3/pull/835/files#diff-4
I think === is fine
It does more then just overriding the cron, just like the one in `page_header` it allows you to bypass the whole function as well modifying the stuff thrown into it.
create_* would be more appropriate.

Side note: Creation of this object really does not belong into this class, but I guess it's fine to have it here for now, since we do not have a service container (yet).
Wether an avatar driver is enabled or not is set in the ACP. The ACP settings are defined in the avatar drivers but the naming of the config settings that determine if they are enabled is the same for all drivers. I can try to move that to the avatar manager if that is ok with you.
$request->header() seems broken for Content-Type as it prepends all strings with HTTP_ but PHP reports Content-Type as just 'CONTENT_TYPE' and not 'HTTP_CONTENT_TYPE'.
Same for all variables and language keys
This wont work in lua, you need to do a = a and a or b
On Oct 15, 2012 6:09 PM, "Matt Kaniaris" <notifications@github.com> wrote:

> In agents/monitoring/default/client/connection_stream.lua:
>
> > @@ -103,8 +103,8 @@ function ConnectionStream:_setDelay(datacenter)
> >    local delay
> >
> >    if previousDelay == nil then
> > -    self._delays[datacenter] = 0
> > -    previousDelay = 0
> > +    self._delays[datacenter] = consts.DATACENTER_FIRST_RECONNECT_DELAY
>
> you can condense this code to be less verbose- something like
>
> previousDelay = self._delays[datacenter] ||
> consts.DATACENTER_FIRST_RECONNECT_DELAY
>
> then you can delete the if block.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/racker/virgo/pull/232/files#r1849732>.
>
>
At the end of this parallel the agent might not have actually downloaded anything, right?
E bine să specifici eroarea la care te aștepți. Aici e IndexError cred.
Why do you forbid to build inside project source directory? Also what is project source directory? src or root?