labofsahil/github-event-dataset-2012 huggingface dataset:
This HTML page, as well as `findgames.html` and `findquarters.html`, are representations without corresponding resources. Rather than make them separate representations, you should think about how you might incorporate templated links that you've used here into your other representations. For example, it might make sense for your "all series" representation to include a templated link that allows a client to search for series that meet certain criteria.
This breaks if you didn't install MacVim from brew...e.g. if you installed it from the google code download
Traurig, aber das ist das bequemste was man machen kann.
example_el loads at startup (without renaming it to .el). I didn't know what was going on until I noticed (whitespace was not starting in prog-mode and spaces were replaced by tabs on save). I removed the file and everything's fine again.
Thanks. What about this deal: I'm fixing the typo in the release notes, and you issue a pull request for making opacity an api property?
I would suggest moving this several lines down after we create build requests. Feel free to ignore this suggestion, though.
This comment is no longer correct -- we will attempt to evict such pages now.
「ショートカットとして」と「:method:」のあいだに半角スペースを入れていただけますか？
daumeditor/js/trex/lib/utils.js 264:265는 한줄로 하는게 낫겠어요
it is in fact measurably faster than `.push`, by a few milliseconds, if you do it millions of times. In other words, it is literally nanoseconds faster, which is basically not a reason. We should just use `.push`
hopefully this fixes the unicode problems.
This looks good, although you may not always want to use `Title` as the breadcrumb title.
Why even include `.rvmrc` in the project? As a developer I want to manage my gemset-names as I see fit.

My `.rvmrc` looks much simpler, something like this

    rvm 1.9.2@gemset-name

Can somebody please clarify what the reasoning or added value would be to include a `.rvmrc`?
こういうのほんとは良くないんだろうな
This desc line should read something like "Load and execute the code from db/indexes.rb to create indexes".
I see – I hadn't properly read DEVELOP.txt; that said, there's a mention of checking out:
```
https://github.com/iElectric/tinymce.git
```
.... which doesn't exist anymore?

Would you mind getting the changes in 8e65be0 into the repository in the *right way* – and then mention the new changeset here?

Thanks!
Like I said on push note: "spawn emotes, that solve the ugly sensation of static ambient, anyway still need a little more work but I will do in the future"
OK. What should I do about the staff code?

On Sat, Feb 25, 2012 at 18:23, blackcurrant <
reply@reply.github.com
> wrote:

> Sure, we can do that. In the meantime, you can test on your own code from
> 6.005.
>
> On Sat, Feb 25, 2012 at 6:18 PM, jdhenke <
> reply@reply.github.com
> > wrote:
>
> > No. I have not run anything. This was all just fixing compiler issues.
> > Maybe on Monday we can setup some sort of testing data?
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/uid/caesar-preprocessor/commit/2f682b0f3f2a294142c31b4f6ec6073d03534e65#commitcomment-1011621
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/uid/caesar-preprocessor/commit/2f682b0f3f2a294142c31b4f6ec6073d03534e65#commitcomment-1011630
>
crashlog?
Fix for new format of TSM Accounting lua file
log.info( and log.log(Level.INFO are the same.
And if you really want to set that as release, I can try to make it so that it actually becomes a wolf.
eso te iba a decir, ademas te has cargado el esES mamona, les va a salir a to dios en inglés.
Necesito que se meta esto lo más rápido posible. Ahora mismo sindragosa no pone tumbas y la gente se aprovechará (me lo reportaron esta mañana)
Esos logros funcionan! que los tengo yo! jajaja
well, new approach seems to have more problems than advantages. I'll bring back previous script in the next release.
I believe I added that mask to fix gnome-terminal having strange colors due to the permissions.  I never use golfram in windows anyways so I might as well just move the repo to my ext partition (I rarely use windows on my laptop)
Regarding cleanliness, I'll come up with a suggestion tomorrow - for academic reasons if nothing else.

In terms of speed, websockets do come with some overhead, so SSE can theoretically be faster for short string data. That would require a hand-crafted parser, though. In ws I've pulled a heap of tricks, both with processor state and buffers reuse/pooling/stacking, in order to be as fast as possible. The same effort could be made here.
# Homework 3

| **Part**  | **Points** |
|-----------|------------|
| base      |          1 |
| advanced  |          0 |
| format and string functions  |          0.25 |
| tracking user input and conditionals | 0.25 | 
| loops | 0.25 |
| zazz | 0.25 |
| **Total** |          2 |

One  my favorite prissy bot so far.  I liked how you checked for the "sir", tracked if I solved the math problem for later, and all the other little things.  The program feels complete instead of just a "I met the requirements."  It's got the extra bit of zazz.

Here are some little places you should watch for in the future.  First, reuse variables when you are done with them.  I haven't double checked but it seems like you can just use `user` over and over instead of making `user1` etc.  That's a personal design preference, you won't be making many programs like this anyways.

The only other things are a few comments on specific lines:

```python

# line 22
user = user.lower()
if user != "4" and user != "four":

# can be rewritten
if user != "4" and user.lower() != "four
```

```python
# throughout
name = raw_input("- What is 2+2? Try not to screw up.\n>>> ")

# how I'd do it
prompt = ">>> "
...
print "- What is 2+2?  Try not to screw up."
user = raw_input(prompt)
```

```python
# sqrt is normally done like this (I believe) so it's easier to read
import math
math.sqrt(25)
````

```python
# line 73
if failed == True:

# this is a mistake EVERY compsci student makes at some point, and the remakes again.
# if's check for True
# this can just be written as
if failed:
```

Finally, for insulting my humanity, I give you the [HK-47](http://youtu.be/Vg1gTas7OAA) award 

![award image](http://old-republic.com/images/kotor/characters/kotor_hk-47.jpg)
You couldn’t guarantee that first check would be actually first, but notifications must be in order to view.
Actually seems like that exists already (it's called `graphPath`). Probably needs fixing to include the file being graphed as a module in the grapher's output.
Did this fix it? lol =/
Yes Stef, go wash your mouth with soap (not that my mother ever made me do that, but I think it fitting right now ^^)
https://github.com/rails/rails/branches
Let's not make plans in advance :)
ActiveRecord can break the interface again
Done in commit SHA: 49566072b010d217a8a6d29112e2ee89a2dcbc2c
the tab before the if should be a space ;)
nach der neuen regel soll es } else { sein
@Kresshy Egy code reviewt nyugodtan tarthatsz ám :)
Itt például látszik, hogy lehagytam a reader.close()-t :(

csak a bugokat viszem a kódba :D
This event listener should be outside the form event listener. Right now it's only being added after the form has been clicked.
Translatable strings. Public domain
This actually doesn't do the same thing as [`Enumerable#none?`](http://ruby-doc.org/core-1.9.3/Enumerable.html#method-i-none-3F), might be confusing to seemingly polyfill with a different purpose.

It seems what you want to express here is similar to [`Object#blank?`](http://api.rubyonrails.org/classes/Object.html#method-i-blank-3F) as provided by activesupport, but recursively. You could use `[[], []].all? &:blank` or maybe use:

```ruby
module Enumerable
  def all_blank?
    all? { |e| if e.respond_to? :all_blank? then e.all_blank? else e.blank? end }
  end
end
```

Maybe even a helper which lets `Enumerable#all?` work recursively in a generic way.
Are you supposed to remove chained feature? It was designed as this:

    factual.table('global').search('starbucks').filters('region' => 'ca').first()

Now it has to be written as:

    query =  factual.table('global')
    query.search = 'starbucks'
    query.filters = {'region' => 'ca'}
    query.first()

Is it on purpose?
Agreed about the skip and take, but that obviously only applies when skip/take are specified.  My example above does not have them specified. The same idea would exist if there were multiple where clauses present.
_collection.Wh
Is it me or there is no changes there ?
Animations run smoother or something apparently, I've not tried it myself - http://forum.xda-developers.com/showthread.php?t=1429539 first post, I think it was on modaco first.
I guess you skip updating the createProjects scripts until the project generator is finished? Because right now, it's broken because of the move.
I actually thought about removing the second match case, since almost all the time, you remove elements out of a non empty deque :)
You could match on {rear = [a]}, but I fail to see how it would be better.
I think the second Russian one and the French one aren't straight-up translations but rather actual posts about the language and should probably be in the other section. Thanks for updating this.
Can we set the details of this proxy to something like "Search: [name of current tab]" similar to what you've done above?

At the moment (for me), the proxy's details are '0 items' which isn't very helpful :(
You know, looking at it, I would think that it should be... what is the error message you get?
This looks quite interesting and most certainly useful for many users.

I wouldn't integrate it into the extension itself tho as long as we have no preferences GUI. I will merge this for now as an example script. If you have any good suggestion about how we could create a preferences GUI which is also easily accessible (I'd have no clue how one would e.g. open it, I don't really want to add another key binding), we could see about it again.
Yes, missed this. Thanks.
Maybe use `logger.fatal`
This could be simplified to a single line:

```js
options = options ? Y.merge(this.defaults, options) : this.defaults;
```

`Y.merge()` ignores undefined arguments, so this ends up having the same behavior.
Why dialogButtons array is used to store callbacks?
We do not put author information at the tops of our files.
While we're here, can we fix the spelling of Access?
This should probably verify that ary is the same shape as self._array.  Also, please add docstrings to these making it clear that you get device array, not a host array.
Yup, that's on my to-to list
/**
     * INTERNAL API
I think a better name would be “Single-Use Actor Trees with High-Level Error Reporting” (or some shorter variant which mentions the single-shot nature).
I still think "Settings" is a too vague name, and that it should be named AMQP
The UID should go on the individual classes
So this is in the akka private `ClusterReadView` and it's used in tests, some of which could probably be removed (awaitUpConvergence for example).

In `LargeCLusterSpec` for example, should we just check that we have the right number of nodes, that all are up, and that the leader is the expected one? That would probably do it I think.
Yeah, I have reconsidered, I think you guys are right. Lets do that.
Strange, that should not be there. Thanks!
So it's impossible to transform the AnyVals? and should be a MatchError?
But why is it a public method that returns a String? Isn't this an implementation detail?
Perhaps add that it shouldn't be called twice?
**Note here**
```php
Request::current()->uri()
```
**instead**
```php
 Request::initial()->uri()
```
On OSX it is needed as a unique criteria for a special case where we treat PCM like passthrough.
rendering is a dangerous action, this confused me that it wasn't just getting a list of them, hence the ! in errors!
All IPS and Ports should be in a configuration
I've just pushed up a fix for this - I tested against Last.FM radio and it works for me from here. I didn't check if it was broken before.

Can you test and let me know any issues? Thanks for the feedback.
Shouldn't we use the git://github.com/GPII/universal.git URL already?
Considering this is just a wrapper I'd be tempted to just make m_command public. @frankosterfeld may disagree :)
Good call. Can integrate that with the other piece in POUserFunc as well
Always use pt.save! please
Code indention should use 4 spaces as in PEP8, not tabs :)
I don't like it! Redo!
@card seems like noise, we have card in the renderers, and this is the CardController, so it should have a card attribute that you can access.  I didn't do the attr_writer case, we can think about if we want to encapsulate something in that method ...
80 characters per line. check other places too.
This is now used in enough places to warrant making it a method
Can you take this method declaration out and put it in a separate interface both classes can access? For example: `TUITextRenderer+Private.h`
That's what happens when you copy something from an Apple technote. :trollface:
Hmmm, could you take an approach more similar to `mutate`?
Performance wise, it's probably a good idea to catch these regex's.
This is hacky, but I didn't see a better way of doing it.
closing this pr in favor of one with this fixed up.
Sounds good. Thanks. Will make the changes and send another pull. Cheers.
Duplicate is valid both as a verb and an adjective (it is pronounced differently though). Hence name removeDuplicateCookies is OK.
I think you meant:
```c++
if (PyArray_FailUnlessWriteable(a, "input/output array") < 0) {
    return NULL;
}
```
As for the reals above, what happens for the longer complex types?
Can we invert this `unless..else` to `if..else` please?
I would maybe have 2 such extractors, one for NonFatals, the other for NonInternal, or smth like that, which catches both the control exceptions.
I'll try to figure something out. It won't be easy though, because I remember erroneous trees giving me hard time, essentially forcing to account for errors all over the file.
Commenting out code instead of removing it was not my intention. Will fix.
I could go either way, but I think we should pick one.  I'm fine with either `math` or `util`.  I prefer *NOT* having any packages called `util` because they become an amalgamation of junk.   How bout we keep them either in `scala.math` or move both to `scala.util.hashing` ?
paramsCount == argsCount ==> simpleMatch
Oh, i see. So problem in partial /admin/shared/_calculator_fields.html.erb
hah! embarrassing! thanks for the catch(es)!
These symbols are added using symbols and not Symbol.
Ok, I implemented the multi line def of the matrices F and Fd as suggested.
Does `MatrixBase.jacobian` work here?
what happens to as_numer_denom? is it now defined elsewhere for everything that subclasses this?
the backtic before The should be a space.
I really dislike how these are made global. Ether they should be only global or be classmethod decorated methods in Permutation. Otherwise the code gets more confusing, at least for me, maybe for others as well. Which is the better option (methods or global functions)? I tend to prefer making them global functions if they are already used like that.
The test is a bit arbitrary. You can always rewrite a polynomial into `(x - r1)(x - r2)...`, it does not matter really if the roots are real or imaginary. I changed it so that the case `is_real is None` is handled as I would expect, but one might as well remove the check on `is_real` altogether.
I still prefer having a helper:

```ruby
- allowed_to(:instructor) do
  = link_to("Add Task", new_course_task_path(@course))
```
我用引号的习惯是能用单引号的时候绝不用双引号。像这种情况最外围可以用单引号，这样内部的双引号就不必用反斜杠了。

你确定div的id那里不需要双引号吗？
remove it; this is not necessary; `./` means the current working directory
I'd see this as an enhancement, also description can be improved a bit:

`Enh #1531: CArrayDataProvider is now able to sort cutted array, where sorting column is not available in every entry (Yiivgeny)`
Random unused "Argument" variables again.
I'm not really a fan of 60-character function names. `test_exception_not_raised` would suffice here.
Can we get something that will look better on Google+/Facebook? Something with a background maybe, or that's optimised for display in a square?
Couldn't this produce huge amounts of log spam?
very small thing, but it's a typo (toobar) :)
Should be a proper doc block, such as:
```
* Database object
* @var dbal
```
php_ext
it = item_type?

item_type is too long and causes an error from the DBAL because the index is too long.
Always returns null so description isn't correct.
checkreturn here?
51d74f5
I much prefer JOINs, anyway the issue with this is it'll show a player as online in this game even when they are online in a different game - I've updated the query to avoid this.
Hmm, I wanted to measure the "use" routine, and I want to measure the whole "dispatch" process separately.
I just fixed that.  Refresh the PR.
Here's an example of how to document objects and their properties: http://code.google.com/p/jsdoc-toolkit/wiki/TagParam#Parameters_With_Properties
Это тут не нужно
Этому тоже тут не место
вообще будет намного лучше если ты напишешь как ты видишь структуру таблиц и как ты хочешь чтобы с ними работали крон и нода. имхо так будет намного понятнее, а то я сейчас логику твоего варианта понять не могу)
better to assign the variable at the start of each line e.g 

$type = ($params->get('input-number') == "1") ?  'number' : 'range';
suggest naming k, k1, and k2 more specifically to say exactly what they are counting. that way a statement like
```
k == drugs.items.length && k1 == k2
```
becomes more quickly comprehensible without reading the code all around it
Looks more like "reading from DoctorList store"
I am not sure to realy understand what you mean, but if you are talking about listing ops that have a structured grad, would not it be better the put this list in the _documentation_ instead of the _tutorial_?
I didn't review the grad formula. It seam well tested and the tests pass. Do someone want to review that part?
return [ <-- space
I understand the idea, but this includes UI code in the touchLayer plugin unnecessarily, we should devise another solution on the UI taking advantage of the trigger here.
Something along the lines of:
$.bind($.touchLayer, function(){
    if(document.body.clientWidth>=700)
        $.ui.toggleSideMenu(false);
});
For a build tree reference, we store only the path and load the corresponding .cabal file each time we create the package index (since the package id might change). A `PackageIndex` is just a map from package names to lists of package descriptions (see `Distribution.Client.PackageIndex`).
I'm not sure if either of the above are necessary as they aren't being used.
This is somewhat unrelated to the issue, but since the current dir "." was not included in my classpath, Ant/JUnit could not find the features/** files when trying to execute tests.
The reducer registered for the type `np.memmap` should prevent that.
Make sure you add a test that would've caught the bug @staabm mentioned. Thanks for your hard work.
The code changes in this file focus on the reformatting. No code optimization here.
Would be nice to remove the 2nd dashed line.
Oh, I see.  attname not in (id_col_name, cls_col_name)
Mm, I'll change to `file:search` for now. We can consider adding an URI scheme as part of replacing tag_cache/m3u with something else.
The name and the type should match.
`def patches` goes above `def install` by convention
Not quite this simple; needs to be something like

```rb
system "make", "prefix=#{prefix}", "install"
```

(or equivalent) or we don't have any control over where things are installed.
warn, not say.  Also our three exclamation point pattern makes us look crazy.
I thought that we decided that GTK was the official place for docstrings for platform-specific code.  The reason not to put it in both is then when we change things, we need to update it in 2 places and probably won't.
Oh, there is -- you added `index_match`. :)
Can we either raise a suitable Exception here or at least do:

```python
assert len(keys) < = 100, "Trying to load more than 100 keys"
```
yeah, too many array operations / too much Python-- would be better to do nanskew in C/Cython someday
"normal_user" is not a term that means anything in sakai. Can you use roles that map to CLE?
@tgalal you can check this place.
make codeDebug for newStatus before emit and after receiving
do tests with some strings. long strings with many emoji cause garbage after emiting before receiving
I liked `SUCCEEDED` better because it is in line with `succeeded` and `SuccessfulTest`.
Whoa how to do the tests pass :/
I'm thinking we should change the name of `do`. It's already not just a language keyword but overloaded in other places too, nor is it particularly descriptive in or out of the context of a state machine. Possible alternatives: `doState` or `gotoState` but I'm open to alternatives.
@quartzjer see the code comment right above, seems like we should be using the app's token, not just any token.
This comment doesn't seem to apply anymore.
this color should come from one of the variables in _settings.scss or be derived from one
This number is also magic. Maybe set up MAX_HOSTNAME_LENGTH (in fact, if this is defined somewhere in the code under test, I like to just import and use it directly) and then test with util.get_chrs(MAX_HOSTNAME_LENGTH) and util.get_chrs(MAX_HOSTNAME_LENGTH + 1).
Double-click on *collapsible* areas is very cool, but please fix spelling :)
Following package.json docs here https://npmjs.org/doc/json.html
There isn't `self` reference here - it is `_self`
It must have been for a while
The same typo also exist in [L109](https://github.com/ajaxorg/cloud9/blob/a81fc065222d2ff840c1fad76f7c74466ec33f0a/plugins-client/ext.searchinfiles/searchinfiles.js#L109)
Yup, I just like testing without relying on active_support.
please don't do this. Use the option without value (``VALUE_NONE``) so that ``build`` is not dynamic whereas ``build --dynamic`` is dynamic (see how the ``dry-run`` option of the [composer ``install`` command](https://github.com/composer/composer/blob/master/src/Composer/Command/InstallCommand.php) works for instance)
This class is going to help during styling.
It also improves layout now.
public function getTracker($provider)
why do you have a description here? This looks useless because of the github type.
Refactored and documentation added.
Can we have two tests here? One should test that a user is created if they
don't already exist (like you have) and the other should test that if you
already have the user, they're updated.

On Sat, May 12, 2012 at 7:48 PM, bayers <
reply@reply.github.com
> wrote:

> > @@ -1,7 +1,21 @@
> >  require 'test_helper'
> >
> >  class UserTest < ActiveSupport::TestCase
> > -  # test "the truth" do
> > -  #   assert true
> > -  # end
> > +
> > +  test "should create a new user if one doesn't already exist" do
> > +    auth = { "provider" => 'facebook', "uid" => '122', "info" =>
> {"name" => 'Brian Ayers', "urls" => {"Facebook" => 'facebook_link'},
> "image" =>'image_link' }}
> > +
> > +    b = users(:brian)
> > +    #make sure it doesn't already exist
> > +    database_entry = User.find_by_uid(b.uid)
> > +    database_entry.destroy if (database_entry)
> > +
> > +    User.find_by_provider_and_uid(b.provider, b.uid)
> > +    new_entry = User.create_with_omniauth(auth)
>
> Yeah my fault I forgot to take out b when I figured out how to make the
> auth.
> This should be better.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gaye/rideshare/pull/30/files#r812843
>



-- 
Best,
Gareth
File + Line numbers would be nice here too. It's near impossible to track down var `x` in a code base.
Should be `<-` not `=`
Also provide attributes here: `{'a': ['title', 'href']}`
50 should probably be pulled out into a constant. It seems to me like it's buffer distance. If you want to change the value later, you'll be able to change all of the 50's in one place.
hm you are right... so you could write it with 3x `!`
Hover and focus are not designed the same as other footer links. Decide if all footer links should have hover of focus or none. Not half.
See how we elsewhere markup the command line.
Can just use the above hash map until it becomes an issue with an increasing amount of migrators I guess.
I'm confused about this.  We have to have some kind of project at this point, no?  Won't we leave the app in a state of chaos if there is no project created (even if empty) before we dispatch ready?
Can we do this without videojs?
We're good on those dom elements -- they're actually populated by data in the CMS. The Person model has fields for github_username and twitter_username, so those aren't API-dependent at all.
Of course, you have to check whether ca_certs is given, but that's not the check you do in the code I commented on, above. In fact `SSLSocket.__init__` as called by `wrap_socket` in Python 3.2 uses following code:

```python
if ca_certs:
    self.context.load_verify_locations(ca_certs)
```

But it doesn't check for `verify_mode != CERT_NONE`, like you do above.
这行删掉
Счётчик Метрики из `ru.md` лучше убрать. В `README.md` он есть, потому что это файл рендерится сразу при входа не страницу репозитория.
It seems we sent comments around the same time. We definitely need to determine whether the one-target-per-zvol is the best approach first. But, assuming it is...
A) I'll agree with your logic on the date. You're violating a SHOULD, but not a MUST, so you're correct that it is allowed. I'll also agree that it doesn't create any uniqueness problems.
B) I was assuming you'd set an alias to provide the readable name.
C) I was thinking we needed more uniqueness than is necessary (or desirable). For example, if you `zfs send` a zvol to a backup server, I was thinking we'd want different IQNs, which your scheme does not provide. patdk said there's no need for them to be different in that case. And clearly, if they're the same, it makes failover easier.

So the question of target-or-lun per zvol remains, as does the issue of ensuring date stability.
Hi @bartvde, thanks for the feedback.  FYI, "propertyNames" and "fields" are now both contained in "getFeatureInfo", and are intended to match the same two properties of https://github.com/opengeo/gxp/blob/master/src/script/plugins/FeatureEditorGrid.js

"fields" determines which layer attributes to be displayed and in what order
"propertyNames" determines what label/title should be used for each attribute

Example here: https://github.com/opengeo/gxp/blob/master/src/script/plugins/WMSSource.js#L103
I assume tests are coming? :)
On 08/27/2012 03:50 PM, Jason E. Rist wrote:
> In src/app/views/common/_footer.haml:
>
>> @@ -1,15 +1,2 @@
>> -%p.copyright
>> -  #{_("Copyright &copy; ") + (Time.now).year.to_s + _(" Red Hat, Inc.") }
>
> I don't think we have a choice in the matter. Red Hat still owns the
> copyright up or downstream. Leave it in.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Katello/katello/pull/520/files#r1467848>.
>
the copyright is in the code... which should be good enough.

-- bk
Nice dude, like how you pulled this out of `latlons->wkt-multi-point`.
is e2 actually needed?
|parent| should not return the parent window for this type of application, but it should returns the window itself.
Also I'm fine for passing |showime|, |hideime| via postMessage from the top container until the platform bug is fixed, but child should not use postMessage like that. I have a dumb workaround until a real solution - see a few comments after.

As a result you can remove this function.
I think `get_equivalent_units` is must more useful returning a list of unit objects (from which one can get any available information about the unit) than returning a table, which only has pre-defined things separated out.  I've added a "see also" to `print_equivalent_units`.
cancelTime可以小于time() ?
OK. This started out as a cmd call to search-cmd, later changed to RPC. I'm actually not sure which one is better practice. What's your opinion?
Ditto, `io_lib:format/2`
You're right! I did the test with an invalid value.
I'm rather sure it would be better to not use such an old Boost version, but if we use that for the Linux build this just reflects it ;).
This should just cause a fatal error. It's a programming error to try to go from a shared lock to an exclusive lock, and this implementation is almost certainly never what someone wants (it releases the lock entirely).
What did you change here?
Is pszBuffer still used now?
Shouldn't this check be nLenR >= 1 (same below for S). Otherwise, you cannot do the next check for the sign bit.
I suppose CHubListener could be derived protected. No class except CWallet should be able to call RegisterWithHub and related functions. The callback functions in CHubListener can be made protected directly I suppose - they are only intended to be overridden, not to be called by anyone except via CHub's signals.
need ```@param``` directives for c and producer_names . Also, what info is it populating the producer_names with ?
yes. I was thinking about keeping it, in addition to `getObject`.
[HIGH]
NSCalendar category methods must not contain any ESDateResolution parameters.

-(NSDate*)alignMonthToPast:( NSDate* )date_;
-(NSDate*)alignMonthToFuture:( NSDate* )date_;

// Other units should follow
not crucial, but this is a sleekxmpp's leftover
Shouldn't this be `if` instead of `if not`?
ID of the menu item should be "fbDeleteAllWatches", please fix also in WatchPanel.js

Btw. a comment in WatchPanel.getContextMenuItems says:
 // find the index of "DeleteWatch" in the items: 

But the reals action uses "DeleteProperty"

Pleas fix/verify
In this method you are inconsistent about where you are putting spaces after `if` and `for`.  I think we want something like:

```javascript
if (test) {

}
```

I know the code style things probaby seem nit-picky, but they make the code much easier to read and maintain.
Yes, whoops.

On Jul 20, 2012, at 5:17 PM, Bussonnier Matthias wrote:

>>         self.format = unicode(format).lower()
>>         self.embed = embed if embed is not None else (url is None)
>> +
>> +        print ('%s %s %s' % (self.format, self._ACCEPTABLE_EMBEDDINGS, 'T' if self.embed else 'F'))
> 
> Is this a forgotten debug statement ?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ipython/ipython/pull/2178/files#r1209612
This wait is redundant with communicate above.
Actually, this is a bug in Checkstyle:

http://sourceforge.net/p/checkstyle/bugs/679/

Do we need this wildcard?
vsphere version should be a literal string.  remove the constant
Same goes for some of the other examples. I know you only want to test if the values work, but unless its an X-prop like here, its not valid.
Once again, this calls for a specific inline function because `fromstr` sets an error message. We don't want that, and we don't want to clear the error after each callback.
This arithmetic is blurry.

```c
ptr += 3;
for (i = 0; i < 38; i +=2) {
    ...
    oid->id[2 + i/2] = (unsigned char)v;
}
```
This test suite looks sexy. :+1:
Does this need to be served up with a specific mimetype?
Yeah. It was 1 character beyond where pep8 would yell at me. I can change in other ways if you prefer.
again, good call here.
As far as I can tell, it doesn't affect them. `$ python ./funfactory/funfactory/cmd.py --python=python2.6 --pkg=foo` sets up a working foo project as expected.
Is this some leftover debugging?
if you're storing the string version of the login source, why do we need the numeric constants?
Can you do `do_raise = getattr(settings, 'RAISE_ON_SIGNAL_ERROR', False)`? Seems cleaner, and also defaults to `False`.
Both links go to the current page aren't they? Breadcrumbs are not very useful in this case ...
You won't need pubkey_version, let's remove it. I was expecting this resource to return a JSON structure that "normalizes" users and clients -- I guess a "principal". Was this discussed and decided against?
SparkFilter does not implement Serializable
hi Sam,

I just checked about those protocol relative links.
It appears that so long as you use gravatar.com w/o subdomain, you can request http and https at the same address, e.g. https://gravatar.com/avatar/205e460b479e2e5b48aec07710c08d50?f=y

So, I'd use url = "//gravatar.com/avatar.php?" here, and drop the secure attr again..
Ah.. one more typo missed :-)   A **for** loopS
type can be passed into the block.
yeah, XSD files are not used to perform the actual validation as the Config system is far more powerful for this. The main goal of providing an XSD is for IDE auto-completion for guys choosing XML for their app-level config (and they are probably a minority). Thus, if an error occurs during the XSD validation, the exception thrown to the user is pretty cryptic which is generally a WTF ("what does this stuff mean ?")
I'm wondering if it makes sense to continue maintaining them.
You should add a use statement at the top of the file and just use the class name here.
arnaud-lb/symfony#6
`too` = `to`
You know, I wasn't sure how to deal with that.  Personally, I kind of _want_ the child to still be validated even if the parent skips binding.  It's not the child's fault data hasn't been bound.  This way, `$child->isValid()` would throw `You cannot call isValid() on a form that is not bound.`.

So should I make it so the graph walker skips this child?  This wouldn't make the child valid, but at least it wouldn't have any errors.
I would return ``null`` instead, as it is the value returned by localStorage when the key does not exist.

Btw, returning ``false`` for unsupported methods is absolutely not standard in JS (except for cases where the method returns ``true`` on success)
Empty line before return.
Should be "PHP-Erweiterung".
Please don't edit this file. It is auto-generated for each release
Please use a camelCased name
Ah ok, a short comment would be perhaps helpful to get the idea behind it.
It's common practice to declare cell idetifiers as static. See code for example [here](http://stackoverflow.com/questions/3567278/problem-with-cellforrowatindexpath-method-in-uitableview-iphone-indexpath-row-i).
It was on the Mocha doc in fact : http://visionmedia.github.com/mocha/
But I did not append the braces.
It looks like we're assuming that R has alpha_channel == 3 here.  We shouldn't do that (unless I'm missing something?)
das ist der neue link zum userprofil
Not yet :wink: - although it is in the pipeline (for a separate PR).
Please see [here](https://github.com/bblay/iris/blob/osmap/lib/iris/io/_wms_interrogate.py) for this functionality and confirm it is wanted.
Don't be so harsh. :smile: 
You could just warn me? (I can update the public kwargs attribute anyway...)
Should subclass a KeyError?
이건 reroll해야겠네요. 아무 생각 없이 이 부분을 올렸네요.
Add test for does not exist
normal ?
I somehow feel uneasy about using ``inspect`` here. Maybe use ``django.get_version`` to detect the Django API version? Something similar to https://github.com/KristianOellegaard/django-hvad/blob/master/hvad/admin.py#L28 could be used.
then $sirenFormat can be removed, right?
classe non instanciable
Never mind that, just saw later in this commit the changes to the LocalDescriptions.properties
ARCHIVE.resolveModelAttribute(context, operation).asString();
Please can you include the JBoss copyright notice. Just cut and paste the one in Coordinator.java and change the date to 2012
Ok but If I pin a package with '==' in my setup.py, I would never want to have 1.2.3-10 instead of 1.2.3. I guess I did break the ruby operator though, which seems to have such a meaning.
Partly reverted, otherwise ````&#160;```` will be displayed as string.
Lot of extra line breaks
Not the same, the first only creates a passive notification with no events attached. The second one creates a different notification with event onclick.

Despite this, you are right. I missed the localization and the icons are incorrect.
`var systemAppOrigin = document.location.protocol + '//' + document.location.hostname`?
Lets free memory, if it is not used, enclose this between block comment markers and leave a explanatory commentary of what to be done.
Hi, this code is critical for panning so we need to change something in order to keep the performance. The algorithm assures us that the index always exists so we can forget the conditions and furthermore we should avoid add a new method in the panning code. Please in the place where you call your new method add this:

if (isSamePage) {
 delete pages[currentPage].container.dataset.currentpage; 
 currentPage = index;
 pages[currentPage].container.dataset.currentpage;
}
No big deal, we'll follow UX lead. It will be an easy change either way.
I suggest you to replace "apps" by "aplicaciones". If you consider the full name is too long, replace by "aplics" or something like that. Some king of shorting more related with the Spanish word than with the English term.
This should be `forward-original-message`.
Can we name the anonymous function?
I don't know how to implement this but it is quirky several data && data.e && ...  in this method
Is this the best place to do the JSON parsing or is it okay that I've done it in `lib/janky/github.rb`?
This can probably be done only once.
totally confused as to why Tsv can't handle this case? What is the difference? Looks like someone didn't extend Tsv properly. Remove this and add a object Tsv { 
  def apply(paths: Seq[String])
}
1) typo
2) does not match the updated method signature in lib/fog/aws/requests/compute/associate_address.rb
From the description of this method, this does not close connections. Therefore, you should instead call this something like "release_lock".
Validate replication's type and acceptable range of values?
I think a better solution is to use `logging.disable(logging.INFO)` instead. Otherwise you have two problems:

1. The configuration file is read on startup, but you modify it afterwards (when disabling the loggers). This means the changes won't take effect until the next run of freeseer-cli -- which is why changing the logger settings in your code appears not to work.
2. You are modifying a user's logger settings without their knowledge. It's better to hide the logger (to improve the CLI experience) without changing the user's settings.
Return some sort of error code or throw an exception, you shouldn't terminate the program from within a relatively low-level function (unless there is some giant catastrophic error. This isn't catastrophic.)
You should remove this as done in others adapters (i.e. rackspace one)
```coffeescript
to: (type, o) ->
  ...
```

```coffeescript
Redirect.to 'thread',
  board: 'a'
  threadID: 123
  postID: 1234
```

```coffeescript
Redirect.to 'search',
  board: 'a'
  tripcode: '!MayhemxaEo'
```

That's a lot more comprehensible.
It isn't, but the other ways to drive chunk unload are forced, meaning they shouldn't be able to be cancelled (unloading a region, unloading a world, server shutdown, and the like).  Player logoff is just reaping - observer list goes down, and chunk is reapable based on the fact of it being empty.  Reap event allows for plugins to keep chunks loaded when they would otherwise be reaped (that is, idled out), but not when they need to be unloaded.
Awesome ... I agree that it cant happen if the tool is used correctly, but no saying that someone doesnt (even innocently) modify the tool and send something malformed/wrong... probably no vulnerabiltiy here, but if robustness can be added and handled elegantly, then always a win in my book.

Great job on this! I'm excited to use this!
It was added as a regression test for bug 2116.  Regression tests always look useless -- until they fail!
Nope. That's a debugging print.
setResourceDirectory()?
Thanks for pointing out, there was a related bug.
Shouldn't need it. The peer_cert has already been validated by openssl. See the C code. If you make sure to do the SAN check on the same CTX and make sure that verifypeer and allowselfsigned are set correctly, then you can be sure of the peer validation. Only check left to do is CN and SAN on the peer_cert. You dont need the chain (and really shouldnt look at it)
That note about recursion is wrong, please remove.
you should add the original exception as previous exception
The deck-change listener doesn't seem to work if the video is already playing. I had to add a `.pause()` or it would ignore the hashchange/click. Doing it this way allows the print view to be used to move around quickly while the video is still active.

I thought `href` was a Location object, not just a String. Oops.
Is this the `rake-pipeline` way of defining a file which simply shall be copied?
Why don't non-touch users get seamless clicking? Future friendly for when we scale up towards desktop.
And proof in the pudding, I deleted this commented out code that I did not write ;-)
This will now be 1.491 (at least). Ditto elsewhere.
just because manage.jelly used it.
Best to kill off trailing whitespace here and elsewhere in patch.
Yeah I did not intended on making that change, I'll remove it.
Could you please move this somewhere else like `tabs-shared` or `tabs-common`. Idea for `tab/utils` is to be independent of other modules and be utils for working with raw platform stuff.
I tried (and mostly failed) to follow how that field shows up, but was able to confirm at least one path that derives the domain from the email address that sean pointed to. I think that leaving the `escape()` is the right thing to do: this is a point where we convert from one type ("email address in a string") to a different type ("string of HTML which should render to something that looks like an email address"), and `escape()` is the right tool for that job. If we ever see double-escaping, we should probably fix the other thing instead.
I'm not sure how deep we want to go with this one, but I'll just mention http://blog.gnukai.com/2010/06/fqdn-regular-expression/ and https://metacpan.org/module/Data::Validate::Domain here for reference in case we want to improve this regexp.
Do we need review of the privacy policy, given that the text has changed?  This is mostly a replacement of BrowserID with Persona, so it seems low risk...
@ozten - I politely disagree.  Having a fallback mechanism for the user in case of the Persona server being unable to connect with the BigTent server is WAY better than just throwing up an error screen.  A fallback isn't as clean/easy as I initially thought because of the variety of cases, but it is workable, and I think it would offer a better overall experience.

If address-info returns primary for users who are trying to add an email to their Persona account - all is good, the user may have to set a password on their account, but nothing worse. The next time a user accesses their Persona account, if BigTent is available, it takes over.

If address-info returns primary for users who have previously registered a BigTent address and are trying to authenticate while BigTent is unavailable - if the user has a password associated with their account, perhaps let them use it.  The user *has* verified they own the account, though whether we want to honor that verification in this situation feels slightly dirty.  If the user does NOT have a password associated with the account, that's the trickiest of all.  Perhaps send them down the secondary flow to create a new account or if they have a session already, add the email to an existing account?

With the reset-password flow, we *do* have the notion of an unverified email, so we can easily send them down the "go verify your ownership of this address" flow.  Perhaps that could work here.

It's not an easy change, but there is room for improvement over a hard error.
It should not, but there was a production issue where it did have infinite recursion. 

That production failsafe `done` hasn't been cleaned up, so I reproduced the same failsafe. It's safe to leave it in and adds a second level of protection.
@sawyerh - The `overflow: hidden` causes long email addresses to show the ellipsis instead of having the email addresses run off the end of the screen.  By setting it to visible, long email addresses go off the end again.

Perhaps a better place to put the overflow: hidden and text-overflow: ellipsis is on '#selectEmail > .inputs > li > label'  Or maybe we need a new class for email address labels.
more sneaky ninjas
We don't want to mix-up require and Cu.import statements. Would you mind to quickly move this below all require lines with an additional empty line?
I don't think this does what you think it does because the callback is going to run immediately. Also, setTimeout without the 0 should also work I think.
Yay bzapi docs!
The expression should be a ModelPathExpression and it has to be evalutated on the parent entity of the node, not on the record.
Overloading to support an optional `bpb` parameter might be cool.

I know I can do it easily using `.bpb` after instancing but I'd use it **a lot** (I guess it depends on other people's usage patterns.)
The test case name here is wrong. It should be `tests.test_coffee`.
Done - see new commit.
I thought we discussed this already to become os.environ.get('USER', 'UNKNOWN') or smth like that
And of course the following if/else part is skipped as:

    Nil can be compared only for equality and non-equality
    if condition is nil.
`RECURSE_NONE` as a default? What about keeping it to `RECURSE_ARRAY | RECURSE_OBJECT` by default?
I'm a bit reluctant to add another method to the interface at this time. Any chance this can be managed via the fieldset options?
@bakura10 I think what @Ocramius is getting at is that he's wondering if the value "__index__" is a special value for forms, along the lines of __CLASS__, __NAMESPACE__, etc. in PHP.

If so, I'd argue we use a constant instead of a string here; the constant value can of course be a string, but constants make it more clear that this is a value with special meaning.
According to the new CS guidelines approved the past week, when you split the arguments then "MUST be only one argument per line."

https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md#44-method-arguments
From Esapi:

        // encode up to 256 with \\xHH
        String temp = Integer.toHexString(c);
                if ( c < 256 ) {
                String pad = "00".substring(temp.length() );
                return "\\x" + pad + temp.toUpperCase();
                }

                // otherwise encode with \\uHHHH
        String pad = "0000".substring(temp.length() );
        return "\\u" + pad + temp.toUpperCase();
        }
no space after (   and after $data.  it should be :
   
    setFromArray(array $data)
Ping @ralphschindler. Thoughts?
Doesn't there need to be a check that '*' is set as well?
maybe just add a description here of when to use this date picker.
We do, on line 100
I just found it weird to tell "deleting blah": "the deletetion of blah failed", as it seems redundant, isn't?
use @white here, pls
thanks for catching this, but is this correct for `LEFT OUTER JOIN`s as well?

maybe:

```php
$condExprConjunction = $class->isInheritanceTypeJoined() && $joinType != AST\Join::JOIN_TYPE_LEFT
                                                         && $joinType != AST\Join::JOIN_TYPE_LEFTOUTER
```
I'm not talkign about having 2 events but about having 2 identical event args classes
Was it that slow on your machine? o.O
Is legacy root dir current working dir in this case?
Why the `[` and `]` ? I use this for indicating optional arguments, which doesn't seem to be the case here.
getLogger
"Intel" can be replaced by toolkit.INTEL when pull request #38 is merged.
Add an option to specify the software name, so you can only generate new .eb files for one particular software package.
Only one emtpy line
Can you remove this from the PR? It's a different issue.
when the server has a lot of ip addresses and I want to do proxy throught special ip.
Listen on that ip didn't help, server will connect throught default ip, this pach fix it.
Typo on my part, I'll fix it
`**0.3**`
Nit, but `#555` is shorter.
can't you just remove this entirely?
Please use the cached `hasOwn`, ie. `hasOwn.call( events, i );` - this is safe from accidental property overwrites.
added third optional argument to name a particular middleware
OMG 30 seconds? can you make sure we didnt miss an index (e.g. with explain etc) ?