labofsahil/github-event-dataset-2012 huggingface dataset:
Hi vinoth,

If the completeClienteRequest() throws some exception, then the closeIntenal() is not executed. In this case the stack trace that happened was:

voldemort.store.UnreachableStoreException: Client response not read/parsed, cannot determine result
	at voldemort.store.socket.clientrequest.AbstractClientRequest.getResult(AbstractClientRequest.java:78)
	at voldemort.store.socket.SocketStore$NonblockingStoreCallbackClientRequest.complete(SocketStore.java:335)
	at voldemort.store.socket.clientrequest.ClientRequestExecutor.completeClientRequest(ClientRequestExecutor.java:273)
	at voldemort.store.socket.clientrequest.ClientRequestExecutor.close(ClientRequestExecutor.java:158)
	at voldemort.store.socket.clientrequest.ClientRequestExecutor.checkTimeout(ClientRequestExecutor.java:83)
	at voldemort.store.socket.clientrequest.ClientRequestExecutorFactory$ClientRequestSelectorManager.processEvents(ClientRequestExecutorFactory.java:381)
	at voldemort.utils.SelectorManager.run(SelectorManager.java:172)
	at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
	at java.lang.Thread.run(Thread.java:662)

I'll try to include always the stack trace. In fact I have it in my own branch, but I forgot to include it in the patch. Sorry
Add a blank line between the includes here, as they are different kinds. FIrst is a special purpose library, the other is a standard library header.
prefix with Y
Dit is onnodig, right? Deze entity is niet visible.
Lees nog eens wat ik schreef over deze regels. Stuk duidelijker de manier die ik suggereerde.
allows me to use the space removal logic to manage tabs.  I was thinking of a case where someone does "var<tab>a" or something along those lines.  It is more to make sure that replacement is not overly aggressive.   I believe the previous version just stripped tabs completely.  Basically trying to make it more bulletproof
может, стоит заменить на  arc4random_uniform? 

http://stackoverflow.com/questions/160890/generating-random-numbers-in-objective-c
не знал что к массиву NSArray можно обращаться как к обычному массиву по []. здорово!
В примере не зря есть проверка if (self)!
я правильно понимаю, что каждый раз при вызове будет создан новый объект NSString? Тоже поступил так же, но видимо, было бы лучше содержать готовые объекты (4 штуки), а оперировать указателями на них.
Просто делайте коммит, повторный пулл-реквест делать не нужно, все изменения здесь отобразятся.

Pull Request нужно делать один раз, чтобы "связать" свой проект с этим репозиторием.
Sorry I forget to change the host although I intended to do. I will add the global flag, and put it at the beginning of the section. (i.e. after line222)
очень плохая практика оставлять закомменченный код. Он или нужен - и тогда он незакомменчен, или не нужен, и тогда его нужно удалить
10 is a literal number. There's no indication here about what it means. It needs a symbolic name so I know what it is you're trying to do here.  (flag << 1 == ROOM_GODROOM) would work. Also, the double parens are superfluous.
Okay then. Let's not forget this.
Annotation for?
```objc
[[signal1 deliverOn:RACScheduler.sharedBackgroundScheduler] subscribeNext:^(id x) { /* blah */ }];

// some other place
[[signal2 deliverOn:RACScheduler.sharedBackgroundScheduler] subscribeNext:^(id x) { /* blah */ }];
```
I'd prefer to say:

> Returns a new signal consisting of the return values from each application of `combineBlock`.
Could you rename this to 'Notes' please?
I removed it, eh?
how about include_active instead of idle_only?
it would default to true
"eyes" is not defined. Please remove the whole "parser.on"
same deal as above
I was following directions from Avleen. ;)

The three submits were because I committed three times locally (kept making changes). I'll be more careful in the future.

Carolyn 

Sent using a mouse-sized keyboard with feigned autocorrect intelligence.

On Dec 25, 2012, at 11:00 AM, Mike Fiedler <notifications@github.com> wrote:

> In soft_skills.rst:
> 
> > +professionals are spreading themselves too thin on projects and
> > +deliverables that do not serve the organizations mission. One way
> > +to focus the efforts of Operations is to answer these questions and
> > +to ensure that the Operations organization, whether insourced or
> > +outsourced, is delivering services that provide the most value.
> > +
> > +Protection of Information and Services
> > +--------------------------------------
> > +
> > +Often the Operations professionals in an organization are the people
> > +who most completely understand the technical risk to organizational
> > +assets from an IT perspective. Senior management within an organization
> > +will usually understand financial risks, risks from competition,
> > +manufacturing risks, etc. but they often do not understand IT risks
> > +because of the deep-dive technical expertise required to comprehend
> > +threats to the organization.
> The top of the pull request shows the commit range:
> 
> "unpixie wants to merge 3 commits into opsschool:master from unpixie:master"
> 
> It's not a huge deal, just something to be mindful of, to prevent a bajillion more merge commits. :)
> 
> —
> Reply to this email directly or view it on GitHub.
>
Would rather see java.util.Scanner here.
when you have a scalar prototype, it should be ``<twig:extension>twig.extension.bar</twig:extension>``, otherwise you need to add a normalization to transform the array with an ``id`` key to a scalar
you should link to the page talking about the session
The format on these 2 code blocks isn't quite right - it should look like this:

    .. code-block:: bash
This should be `'' == trim($value)`
It is better to use the backwards comtable way of defining PHP arrays instead of the PHP5.4 way:

    $form_options = array(
        ...,
    );
That's not something we like to see...
why not optimize this as the one below?
Я так и не понял, зачем явно его прятать. Убери это, и метод убери.
This should be an InvalidArgumentException to keep the same behavior on failure
You could sort and then take the highest value... maybe something like

```
[@move_a, @move_b].sort {|a, b| a.ranking <=> b.ranking }.last
```

To see what this is doing, try a simple example:

```
[3,4,5].sort {|a, b| a <=> b}.last
```
Use is_callable() and explode if '::' is detected... see call_integration_hook() in Subs.php for an example.
but that's printing out the clustering assignments for each word?
I don't understand how this PACKAGE == 'all' is suppose to work. If the value is 'all' then it will fall into the first branch statement, so its inclusion in the elif parts seems unnecessary (i.e. it will always be false because if it was true the first branch would be executed).
Some places no comma and others have comma? [ie. 003_service_statuses.py]
Lets make them consistent even though i know it doesn't really matter.
Unless the class is abstract (and hence not executable), testName must be defined AFAIK.
Since a funny guy named Galder committed this: https://github.com/infinispan/infinispan/commit/7702094e5b8975638b036febfed5fd33c7302314
With the cast above, `(AnnotatedType<Callable>) ` cast is not needed.
Sure, but there are degrees :) Honestly, I'd rather return `BOOL` than rely on `error`, since checking `error` is, in every other case, bad.
:+1:

This should be `-initWithBytes:length:encoding:` then.
Sorry, got confused.  I see that it is supposed to be a string
See note about CategoryController
Inconsistent naming: Wifi/WiFI. Please use WiFi
А почему бы в этом классе вместо конструктора сразу не настроить свойство allowedPattern? А вот метод setAllowedPattern перекрыть и бросать Exception что бы не меняли эту настройку.
So that'll change the color all constructive buttons across the CMS, including the "save" buttons on "edit page" - is that coordinated with Paul?
Should the flag be cleared if val is not set?
It has to determine which type of backup file is used, in order to do this, it has to pull it from the config file.
This line is not necessary. There is no admin interface distinction anymore.
@fbricon We already updated to M4 if you take a look over here https://github.com/aerogear/as-archetypes/blob/master/jboss-html5-mobile-archetype/pom.xml#L14

Can you please check your repository? I was trying to look but github is not helping so much.

I guess that something happened during the rebase.
@stof do you really need to WTF things? I mean, this is an open source project not the end of the world.
Oh the verbosity!

One of the advantages of using jekyll/liquid templates is that you don't have to rewrite everything. Because i'm bad at typing things i almost always to something like:
```html
<td></td>
<td></td>
<td><td>
<td></td>
```
Which becomes really hard to debug if you have a bajillion of them.

What I would recommend is in the front matter (the stuff at the top of the page) define all this data and then render it into a template down here.

Top:
```
Classes:
  -
    number: SYDE 101
    link: http://coursecalendar.uwaterloo.ca/linktosyde101
    title: Introduction to Systems Design Engineering
    time: 1 hour seminar
  -
    number: SYDE 101L
    link: http://coursecalendar.uwaterloo.ca/linktosyde101L
    title: Graphics Lab
    time: 1hr lab
```

And then on the page you could have
```html
{% for course in page.classes %}
<tr>
  <td><a href="{{ course.link }}">{{ course.number }}</a></td>
  <td>{{ course.title }}</td>
  <td>{{ course.time }}</td>
</tr>
{% endfor %}
```

This also makes it way easier to separate the data structure from the page structure.

If you want a real example of how this might work I did this here: https://github.com/tallowen/tallowen.github.com/blob/master/resume.html
see my comment about the previous comment ;)
Also most of files are binary ones(png,wigetset etc...)
The user is going to have no idea what "Key has multiple values" means.
added one after merging.
`connectUser` -> `userConnected`.
No, this is already done with other similar constants, so just check how it is done.
`$($(`???
Why exactly to limit only to those in navigation? I would limit only to those published and this is it.
Ben is correct.  `gettext_lazy` is evaluated when `unicode` is called.  You should just be able to replace `_('foo')` with `gettext_lazy('foo')`.
This should be surrounded by ```$(document).ready(function(){ ... });``` just to be safe.
I think that filtering out the active entries here seems to make sense at first glance, but we might create some confusion in conjunction with the other changes to hours worked this week. For example, if a user clocks into a project that they haven't worked on yet this week, it will not appear in the list of projects under "my work this week," but the hours will appear in the "total time this week."

I'm not certain we should remove this additional filter, (line 131), but I think we need to consider the affect it has on the UX in conjunction with the change to hours worked this week and see if we can't think of something with less potential for confusion.
No, not currently
I've removed it for the next commit.
This is my newness to pig and elephantbird :)   I was trying to find the easiest place to change to support loading sequence files with json into pig maps.  The converter seemed the right choice given the eval func didn't do nesting.  So I implemented what we needed (the input side).  I think I could tackle output too.  However, if we could get nested support to work in the JsonStringToMap eval func, then this converter would be just a convenience really.  We'd just use the text converter and then run it through the JsonStringToMap and get our maps with just one json parse line.  Then these commits would amount to refactoring json parsing to a common util, and adding nesting support to the JsonStringToMap class.
you shouldn't test directly the type of something. The type for example could be another type of sequence. Test if you can do a 'join' on the returned item for instance?
What is this extra class for?   It's a bit late in the game to start changing the algorithm.
I've renamed it to `$provider`
Why are you not trimming on the left? Did you skip whitespace characters and updated $_pos before calling this method?
similar for beta kernel (distributions with lower and upper bound)

I don't think we need to get this now, but it might be down the road either in these classes or separate.
Yup.  You could add google collections if you want to get it back.
Ok, whenever it's convenient.
Left only for reference. We have a LOT of duplicated (and inconsistently duplicated) validation code in Nexus core, and we are about to clean this up. The code is left here (should be left here) only up to the point where we consolidate validation.

I think this commented code does not hurt a lot for now.
Why are we stripping the prefix here and not on the other services?  If this is needed, it should be done in the logic where the target hcid is actually being read.
Nested font tags before we even hit the text being output, my my
Tables?  I think i just threw up in my mouth .
I wrote both branches but haven't tested which if either is faster.  (Which also might be machine dependent).
Should put that #define at the top.  What is the protocol for changing code that already has a pull request pending?
I picked a large value, so it is easy to see by eye that the PSF is changing.  If you look at the resulting file in hdu 1 where the PSF is plotted, you can see the effect pretty easily.
Default to email address if user hasn't set first or last names
use nil instead?
/v1/applications/ is automatically prepended by dropwizard for the Response object.
True, but it's also used on other BSDs, so I think it'd still be useful. It means that the first time people run things, stuff _just works_ as expected. Plus, id people are developing on Linux machines, running Brooklyn agains localhost will cause packages to be installed, so why not just allow the same for OSX?
Thanks, good catch! I've fixed NetworkUtils.getInetAddressWithFixedName that I'll be pull requesting soon!
headOption?
Map\<String, Multimap\<String, NodeMetadata\>\>
What is this `2` in the table definition?

You should *never* use a generated value in the schema. If you remove the group and create it again you will have a different value (actually, there will be no group with `2` key).

This kind of thing can be resolved in a `before_save` in the Ruby side.
It's not being introduced, I just copied it over from the GET_OPTIONS macro. Yes, we shouldn't suggest users to use it though, but that doesn't mean we shouldn't support it (or remove the functionality).
@phiggins yes; if this functionality is after all required then agreed.
I Agree and it done(please, see commit below).
Why is this check being weakened? A `head 1` search should still return exactly one result.
Did we replace this test?
I've never come across this problem, but seems like -ss is faster when used in front of -i than itsoffset. You're right, it's not exactly what I tought it was. 

I'm thinking about doing it with -ss in front and if their's a error in the seeking, just having a fallback call to ffmpeg with -ss after -i. Any thought on this ?
would need to be (home/home) unless we change home to be in the :use clause
we do not ... that's not supposed to be in there.   ugh, something must
have happened with git.


On Sun, Feb 19, 2012 at 11:50 AM, James Bergstra <
reply@reply.github.com
> wrote:

> > @@ -0,0 +1,62 @@
> > +def string(s):
>
> wait a tick - why do we need genson helpers when using pyll?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jaberg/hyperopt/pull/80/files#r465451
>
made (u.email = ?1 OR u.username = ?1) and u.password = ?2 explicit.
Please add the namespace "t:MyResponseType"
Stuart Marks is the actual authro
Oh, right, forgout about that.
You probably can replace the line above by:  Collections.addAll(fields, type.getDeclaredFields());
Очень обидно будет отсортировать кучу данных и получить фейл на выводе, поэтому решил сразу получить поток, а если его нет то нет смысла тратить море ресурсов на сортировку.
this.withoutReg = withoutReg
Просто если во время sendAll что-то плохо и надо закрыть, не будет работать, т.к. итеррироваться и удалять нельзя одновременно
Зачем \\[? Экранировать надо содержимое args.toString()
For completeness's sake, I'd add a check that isLoggedIn should be true.
Unfortunatly to "conform" to the firebreath way of doing things (which has many other advantages) any non plugin libs (graphdb, graphlayout, graphrender, boost etc.) must be built seperatly (I recently reverted to this scheme).  

With newer versions of cmake there is an oportunaty to create "export" information for these libs which then form the basis of a FindGraphDB (for example), but I havn't got there yet.
okay, so to much copy and paste here itlooks like, since the js build is referencing the css files.
this is a mock file and we import urllib only if it's required
I am getting some StaleElementReferenceExceptions in these tests and I think they're caused by this step which Webdriver does not detect as a page load. I think we should put a custom wait after this.
But I think you might be safe using the one we already have: `wait_for_ajax_on_page_finish` in  base.py.

If you're curious about what to wait for, do some investigation running this test manually with the FireDiff extension open.
Please also add an "Example" bloc with a link to the example here to make it more explicit.
`basis_kernel` should be relatively small and it is dense so using plain `svd` seems like a good choice.
The header classes should have `equals()` implemented in a way that the `->toString()` calls aren't necessary here.
bad CS, should be:

    if (!is_null($parameterValue)) {
I thought your selling point for `isPOD` was consistency with the compiler ABI implementation?
Sorry, I can't do that. I know nothing about bash syntax.
Is `$this->objectClass` always a valid alias for the entity? Does ORM accept any string provided it doesn't include a period?

Also, if we're supporting custom query builder methods, can we even rely on them accepting the alias name as the first parameter? There is no longer an interface to enforce this.
Please put a comment saying that for BlackBerry it will be 'blackberry'
done.
- checked against obj nullness
- probably later we will need to check here for another object type as well, that is why didn't combine it together
There are two functions that use a `QString` reference, so yes it's needed. Plus this way we don't have to care if `QThread` changes.
Seems like this should be a member, unless it only provides a snapshot
`lit(annotation.value())`  will output the int value directly. Instead we'd want to output the corresponding R field, such as `R.layout.component` in your example. That's what we're doing with `@EActivity` when we call `setContentView()`, please check the `EActivityProcessor`
also here I don't understand why you don't invoke the _setParseContextProviderClass_ method directly
This makes more sense than no initialization. Does that make sense?

I haven't found any regression tests on BCD rtl, do we have any?
Yes, it would, I'll change
Well, it looks like just coincidence. Does the $ has a special meaning?

All I can find from the regressions (both engines and compiler) is this:

    import4.eclxml:     import * from $;
right, so maybe should parse the xml stream and check the coverage doctype
I was meaning to add this
> An array in PHP is actually an ordered map ( [reference](http://www.php.net/manual/en/language.types.array.php) )

Because there's no differentiation [in the implementation](http://www.php.net/manual/en/language.types.array.php), we could assume every array in PHP is associative (aka hash table). If we go that way, in order to define a list, or a "numerical array" in PHP we would need to borrow the term from other language. Here is how [java defines it](http://docs.oracle.com/javase/tutorial/java/nutsandbolts/arrays.html), [python calls them lists](http://docs.python.org/tutorial/datastructures.html#data-structures), and there are also [ye'olde C lists](http://www.mycplus.com/tutorials/c-programming-tutorials/arrays/). 

Lists do have length, in some langs their length is predefined. Often a list contains elements of the very same type. In all implementations though, lists are continuous (sequential), each "slot" contains exactly one element. Lists can be sometimes used as stacks, elements can be exchanged, appended, removed, etc. 

Translating this to PHP, the followin is a list (numerical array):

````php
<?php   $list = array( 'a', 'b', 'c', 'd');
````

It is continuous, each position holds exactly one element, each elements can be accessed by their respective positions from 0 up to count-1.

This example below however, does not meet any of the requirements of a list:

````php
<?php   $not_a_list = array( 100 =>  'xyz' );
````

It is a 1-element hash table that maps the number `100` to `xyz`. It is not continuous and the first element is not accessible via `$array[0]`.

Exactly the same principles apply to a modified numerical array:

````php
<?php
$list = array( 'a', 'b', 'c');
unset( $list[1] );
print_r($list);
// Array
// (
//     [0] => a
//     [2] => c
// )
````
The result is a hash table (map), that maps `0` to value `a` and `2` to value `c`.

----

A consistent understanding (and acceptation) of this is important . Simply put: we have to be on the same page :-) Because PHP does not provide a native function for this test, I believe `IsAssocArray` class is very useful and I've already used it in `Config` (that's why it sits in this PR). I'll also be using it in my applications (because it replaces my personal "utility class" i used in the past). 

@EvanDotPro I'd be grateful if you pointed me to any component where the alternate behavior is expected by a component. In such case, we could determine what happens in those arrays inside given component. Remember I didn't touch `tests/Stdlib/IsAssocArrayTest.php` - these tests still pass, which suggests that the behavior you described is not widely known and (which would be worse) relied upon.

Also:  [StackOverflow suggestions for detecting sequential arrays](http://stackoverflow.com/questions/173400/php-arrays-a-good-way-to-check-if-an-array-is-associative-or-sequential) -- notice how they call them "sequential"
Il sort d'ou le 102% ?
On va plutôt forcer l'ordre dans le back office locomotive. Comment ça se traduit ici? avec ```order_by: position``` ?
Yep I totally agree. We should probably update our formatter and remove all the trailing whitespace.
#163 で修正しました
I see, you're just using the bootstrap styles. Better would be to add your own style to the top-level modal like `payment` and then here use `.payment { .modal-body {} }`. That way it won't interfere with any other modals that we may want to use elsewhere (e.g. contact form or whatever)
Maybe "Save (B) as..."?
You can delete `list_attrs` and the `for attr in list_attrs` below
Both `sort` and `range` return sequences so the call to `seq` is not needed. Also, you can compare sequences and vectors with `=` and it returns true if they contain the same elements in the same order.
Agreed. I will make the change.
I usually add || index.row() < 0 to these sort of checks
indented this, as well as another switch in nemothumbnailprovider which was in the same style
The psexec method in htis module has been copied from the psexec method from my recent pull request that was accepted into the framework .
It is separate

On Sep 4, 2012, at 4:28 PM, Meatballs1 <notifications@github.com> wrote:

> In modules/post/windows/gather/smart_hashdump.rb:
> 
> > @@ -361,7 +361,7 @@ def inject_hashdump
> >  			# skip if it returns nil for an entry
> >  			next if h == nil
> >  			begin
> > -				user = returned_hash[0].scan(/^[a-zA-Z0-9\-$.]*/).join.gsub(/\.$/,"")
> > +				user = returned_hash[0].scan(/^[a-zA-Z0-9\s\_\-$.]*/).join.gsub(/\.$/,"")
> Has this anything to do with the powershell stuff or is it a fix for a seperate issue? :)
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
Same here.
```
metasm > push [edi]
"\xff\x37"
metasm > mov ecx, [edi]; push ecx
"\x8b\x0f\x51"
```
You could remove this file as you aren't using CDI.
If you plan on re-filling an array, setting the length to zero is better than making a new one.  In this case, since it's destroy, you're better off assigning undefined.  Or just leave it out, because destroyObject already deletes all properties.
I'll throw something together and post the results.  It should be easy to do with an isolated test case of just Cartesian2 methods.
You can get away with only passing the api_secret here. Public key is generally used for JS requests, and since we're server to server, we leave security up to the server owner (and SSL on our end).
Ok.  My question still stands though.  Why are we doing anything other than "isPersistent = TiConvert.toBoolean(newValue);" at this point in the code?  If addDialog is moved to show then the only thing this block fo code needs to do is set the property.
We could use a comment here as to why it's not setting the model listener.
There's no need to have the else's here. return breaks out of the function.
Mobile Web follows iOS's behavior here (oddly enough...not sure why we aren't consistent between option and alert dialog).
All these string constants must be defined as constants in the MapModule. Use the MAKE_SYSTEM_STR macro.
Feels a little weird to set a static member in a constructor.  Seems like it would either be a member or would use and Integer type and only set it if it was null.  

I think this is ok since we only create one thread pool.  I would feel more comfortable with threadPoolSize being non static.
Sure, I will change it to that.
So what would you suggest? How should I do this ?
If it was unclear, I meant `${name}` in terms of the vhost name -- not this particular file resource's name. When you're defining vhosts, each vhost must have a unique name, so you can use that to build unique resource names throughout.

For example:

   vhost { 'site.example.com': } 
   vhost { 'site.example.com-ssl': }

Here, `${name}` is either `site.example.com` or `site.example.com-ssl`.

Therefore, instead of having a very long `"${apache::params::vdir}/${priority}-${name}-$docroot"` or even `"${name}-${port}-${apache::params::vdir}-${docroot}"`, something as simple as `"${name}.docroot"` would be just as unique, no?

But the larger issue with these docroot and logroot blocks is that you cannot have two vhosts entries that share the same docroot or logroot -- meaning you cannot define vhost entries for http://site.example.com and https://site.example.com and have them share the same docroot. 

Even though the resource name is unique, the path that it points to is not unique and so Puppet errors out. See commit c2c75a62b7dafced8a6120a14bacfc8be7539282 for more info. IMO, this is an important issue that needs looked at.
you also need the Request for the update action.
Because "new" could be the ID as well, the slash prevents that the _show route matches.
Add a comment giving the type of 'task'
А почему не `value: value`? Можно даже `{type: 'hidden', name, value}`
Speed is my main concern, but I still have to test for wrongness... you say

> The fact that it's lazy tends to screw up performance

I take it that you are suggesting that:
```D
if (!(index + n <= str.length), throw new UTFException("Short UTF-8 sequence");
```
Would be a better alternative?

Is this performance issue a "permanent and by design issue", or just, arguably, temporary?
Please put a comment in the assert(0) explaining what's wrong if it fails.
1. This is in the wrong portion of the changelog. It needs to go in 2.061, not 2.060.
2. In general, if you change the changelog as part of a pull request like this, you're going to merge conflicts very easily. IMHO, it's generally better to do them as a separate request, though that does increase the risk of them not making it into the changelog.
If there is already a menu opened, it should be closed (See lines 63 to 68 that have been removed from main.lua).
these should be "web_components"
(with an underscore)

for both the file and the package name
wielding preset  is for compliance with future code
Maybe we should catch the NullPointerException extra and print out that the user have to insert a search string in the format ...
such reformatings pls in an own commit without a issue referenced and directly pushed (without a review). Makes my life easier without all that noise
use Utils class instead
It seems to be only a problem on linux. What exactly happens on OS X if you load a video and then set the format. What are the reasons to do this?
This actually works in 1.9, while it breaks in 1.8
I always configure my /etc/chef/client.rb with the log output path, hence no command line argument.
I think cordoba kicked them out, I didn't change it.
You need to skip the conditional so the default can kick in if the delimiter param is not set.
Yet another variance fix for FX35.
Let's stick to the more conventional

    When I click "See More"
updated the error message. However if we look within this file, definitely not all the error messages are phrased this way, it might make sense to update the error messages.
yes, this line is used for the logging message.  That's the only reason that it's there.
<created_at>
List<Asiento> asientosAdaptados = new ArrayList<Asiento>();
for (Asiento asiento : asientos)
    asientosAdaptados.add(asiento.adaptarNuevoAsientoConPrecioPara(usuario));

return asientosAdaptados;
we actually sync every time one signs in. and one can also click that button to sync manually.
Found another article which deals with indentation in different frameworks: http://www.livelywebdesign.com/blog/2011/05/tabs-versus-spaces/
In case the code doesn't tell the story -- the goal here is to have userExercise state/caching separated from khan-exercises -- userExercises will be passed in from outside.

I might not get it 100% separated, but it's heading in that direction.
Well at least on x86 it seems to be the same. Thanks! I'll use it instead.
My point is this is right:

    switch (foo)
    {
        case bar:
            baz();
    }

Or this, if you really want:

    switch (foo)
    {
        case bar:
        {
            baz();
        }
    }

But not this:

    switch (foo)
    {
        case bar:
            {
                baz();
            }
    }
I'm not sure that this is actually a valid thing to do, since it affects _everything_ in the file, including functions which take function pointers, and if those functions throw, then the function they're passed to isn't really `nothrow`.

_Most_ of what's in the file can be safely marked `nothrow`, but I question that all of it can be.
Are we saying that an HTTP header gets mapped as content-type and so we need to retain the "-" here so they are used transparently?
If you use StringUtils.hasText() instead you can avoid the code at line 122 (returns false if null or nothing but spaces.
Sorry I wrote my note at the wrong place :) I intended to comment the set template => 'template_toolkit'. But yes, I agree :)
@Ocramius just fixed it (and another one I missed also a few lines above)
Rackspace spells it as "Cloud DNS".
missing 3 =
Please use the same syntax as here for notes https://raw.github.com/zendframework/zf2-documentation/master/docs/languages/en/modules/zend.authentication.intro.rst
I said it was OK, right?  We could, of course, compare against a known correct CZML file.  Is that more trouble that it's worth?  How often will it create false failures, e.g., change in formatting, etc.  How often will implementation changes not cause such failures - and the test actually add value.
did we ever sort nodes by alphabetical? it makes no sense now for sure, as the order comes from the order they are added. new nodes always end at the bottom of the child list.
This is not goal of PR, by what about injecting helper via constructor and decline injecting container?
siehe readme comment
```dflist``` holds Linked references to the exact same files (same instance, not just same DFS file) in the actions' list, thus, when the actions' files lose the lock, ```dflist```'s references also lose them, thus no lock remains, thus the next thread to get the locks, wins.
This is the same under Linux. PATH_MAX defines 4096 as the maximum size of a full path, which might be a Kernel restriction or just common sense.

I think if we're not using this on deep loops, we might just change ALL buffers to use PATH_MAX anyway.
setRequestHeader? Please don't be so terse. It is not helping me understand what you want me to do.
Though here we should simply response an 200 http status code so as to stop further FPRNs (up to 6 times according to the document).
data-ajax="true" is redundant.
@pablodip you should consider doing some release of pagerfanta as relying on ``dev-master`` is a bad practice and forbids you to tag your bundle (a tag cannot depend on master as master is changing). Btw, the constraint for frameworkBundle is wrong. the symfony2.0 branch should restrict to 2.0.* and the master branch should restrict to 2.1.* as the are not compatible the other way.
please use camel cased names
the the (One too many)
I tried `sudo easy_install nltk` and got a libyaml missing error. I ran `brew install libyaml` and then tried again.

Please document on the wiki
@nukulb Another OSS concern. There is a github repo - https://github.com/xui/xui but then we would have to build the file ourselves.
In FindThread/SaveThread/RemoveThread, the jsonObj is deleted, but I guess I forgot to delete the thread_info itself
as above, it's an extra library, you can see it in this commit https://github.com/pgrm/openengsb-connector-msnotification/commit/4f9f7f1d388d3488c74c04c21020b0187adfa539
I have tested the iFrame related code in my real projects. They works fine, and the logic is very simple. Please merge into base as well, tks!