labofsahil/github-event-dataset-2012 huggingface dataset:
The libs should work with Adreno 200 as well as 205, but I think they're
compiled specifically for armv7 and so won't work for the Blade.

On 19 January 2012 19:25, kalltkaffe <
reply@reply.github.com
> wrote:

> Interesting. Can we use the libs from the HTC Flyer Honeycomb firmware?
> It's Qualcomm based, but Adreno205. Not sure what's different between the
> Adreno200 and the Adreno205. The files in the Flyer ROM I found does end
> with _adreno200.so so I suppose the libs are for both 200 and 205?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/TomGiordano/kernel_zte_blade/commit/c02c7dfddf7f05ff8e19c0012248507e51d1edee#commitcomment-883299
>
I've seen a few dists pushing updates to cope with the Carp change... it might be better to bonk Zefram and get that change undone.
OK sorry.
o no I didn't :)
That is the nice thing about our test suite It is a requirement that there is a user generated version, this is used in combination with the sample nif during import/export.
Indeed, seems to affect the same lines as your commit f7976c4dfac7af2dc24a21746dc08a0a50224b92.
Looks like it could be merged manually, though.
Added some missing Translations and fixed Issue #51
It is known that this commit wont compile, use the next commit.
Sorry, this is just a git oddity.  I staged my changes in chunks and overlooked this piece; it gets cleaned in a later commit.  I will rebase -i this back into better shape.
It's easier to determine what is borked if tests are separated into small methods, but this looks good.
Ruh roh, Scooby. Be careful of caching. Remember, there are only two problems in computer science: naming things, cache invalidation, and off-by-one errors.

Querying the DB for the client record doesn't seem like it would be a bottleneck. Kill this.
This should not be "prepare-package", but "initialize".

However, in Eclipse and on Windows "initialize" is not recognized as a valid execution phase mapping for this plugin, resulting in a build error.

As a result the [build information](c53f24998a96725eee253e8377430330736059b2) will not be filled correctly.
I have filed an [issue report](https://github.com/ktoso/maven-git-commit-id-plugin/issues/22), let's see how that plays out.
This is a way to restart this activity.
String should probably be formatted in the Template and not in the return here for greater flexibility later.
I just tested here and the mic doesn't turn off when the screen turns off. You should monitor the radio log during the call and see if there's any RIL activity. It could also be a baseband issue.
thanks! that makes sense.  I'm stuck though, because it seems to actually
open the file in the browser and I can't seem to make it so it just shows a
thumbnail of the file, like this
http://www.weeby.pl/blog/html5uploader/uploader.html
 I tried copying code from pages that actually do it that way and I can't
seem to get it to work.
wish me bonne chance on my french test today.
M.E.

On Fri, Mar 9, 2012 at 9:42 AM, cesine <
reply@reply.github.com
> wrote:

> Looks good! You can remove your <script> tags from elicitation.js you only
> need them if you want to put javascript inside of html, but elicitiation.js
> is js.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/mecathcart/Drag-and-Drop-FieldLinguistics/commit/06fe9e96e6e05066fa35298a9f6f735a95d751ed#commitcomment-1065820
>
$class should only be `get_called_class()`. One model can never influence the caching of another. The class param should be removed from the arguments list.
closing ...  will probably fix this myself.
Might as well put this in Y.Array. There's a ticket somewhere for this; I was supposed to add it.
The pseudo-standard for event name constants elsewhere in YUI is to use private vars like `EVT_START`, `EVT_MOVE`, etc.
Were these suppose to be a conditional assignment? i.e:

```javascript
subs        = this._subs,
resolveSubs = subs.resolve || (subs.resolve = []),
rejectSubs  = subs.reject || (subs.reject = []);
```
In devices I've tested, it works, but I was leaning towards disabling this because it hasn't been tested extensively, and it probably should. It's a nice to have, but current implementers have gotten around it fine.  Will comment it out for this release.
I dunno about "row" and "col", it doesn't make sense for nested grids (row in the same row) or cols that are 100% width (it's a row and a col).
Well, both actually. My main goals in changing the test case was to somewhat discourage shared cache manages i.e. `getInstance()`, force the test to build the context through the xml configuration file and of course, remove the need to create the cache programmatically. 

The actual deployment of clearpass (based on the docs) depends on loading the context file from the classpath, and this is just an attempt to stay in line with the deployment scenario.
Minor thing: could use `numpy.arange` and `numpy.zeros`
0 ms?
roleS && configS
Hi!

It is perhaps a bit more succinct to do this:

val stopRefresh = system.scheduler.schedule(0 millis, retryInterval) { cache.refresh() }

Cheers,
√
Since this looks to be public API it needs some ScalaDoc love
unless this is known to be completed before this actor shuts down you might want to protect against `self` being null here
Sorry, can't - some tests don't compile when I combine the apply(String) methods. Lot's of errors like
```
[error] /home/oli/Data/Source/Scala/extern/akka/akka-docs/scala/code/akka/docs/io/HTTPServer.scala:199: could not find implicit value for parameter num: Integral[java.lang.String]
[error]   val connection = ByteString("Connection: ")
```
How does `glTexImage2D` combine with `target != gl.TEXTURE_2D`?
I'd make it clear that this is essentially a read-only hashtable now.
Any changes you make to the hashtable will not be taken into account.
I'd clarify this in the docs and the ObsoleteAttribute, and check if you can actually return a read-only hashtable (that will throw exception when you attend to modify it); does such a thing exist in .NET ?
Do we ever get here? Is the new resource page ever loaded without a pkg_dict?
Good call, the task_list variable was meant to wrap it. Fixed.
Same odd formatting as I mentioned above using github's rendering. It might just be how github does it.
It is an error on my part.
well, maybe looks logical and maybe I'm oldstyle, but I think it's more readable without them and using if statements. I must confess that I'm sometimes a sinner against this too, but I usually clean it up shortly after adding it :)

it would be a quick change for you, and then this code will match the coding style of the other classes in /peripherals nicely.
I think this is the right approach. Might want to throw in a comment about what is going on here.
Replace by PUT as it does not create an additional resource.
Instead of these I'd prefer an UdpNettyTransport and a TcpNettyTransport and the user will then use the FCQN of either of them in the transport field. Then we can drop the "sealed trait Mode" + friends.
nit: foreground
Drop the conditional.
This may raise an error when <code>args</code> is actually used: <pre>SyntaxError: non-keyword arg after keyword arg</pre>
а вот doxygen-комментарии, зачем такой класс понадобился, нужны
@dalli isn't https://github.com/mperham/dalli/blob/master/lib/active_support/cache/dalli_store.rb#L198-206
Agree with @jeremy, we should keep ```config.action_dispatch.rack_cache``` to allow customisation of Rack Cache setup
I think so, because JSON for example doesn't know anything about ruby symbols and stores them like strings. So if we store attribue like somestore[:foo] we cannot access it in the same way - only somestore['foo']. If you have an idea how to do avoid this - you're welcome.
Enclose `ActionDispatch::Session::MemCacheStore` with backticks.
now use => now uses
Put the changelog entry at the top of the list
@edtsech Yeah, just a note on your single-line methods. You don't have to put semicolons when defining them.

`def parent nil end` and `def name user.full_name end` are all valid method definitions.
Probably leave it out. If we extract a rack-cache-rails plugin, can talk about config in its docs.
I'm usually the first to argue for not changing the list of inflections but we should probably remove cow => kine in master - it's an archaic word and no-one uses it.
@steveklabnik figured as much, makes sense
@radar can you send me a gist on how to decorate the Payment::Processing module? I tried module_eval on it, but couldn't override gateway_options. I'm trying not to duplicate that code.
This is a place where the data-hook has changed.
Hmm. In general I think that is a good idea, but I don't think the URL is likely to change. Can you think of another scenario where it would be useful to control?
perhaps this could be a re-write:

`# test that the patch doesn't influence the behavior of the standard library`
Maybe I am just sleepy, but this doctests seems wrong (the matrix is not diagonal for instance)
I will look it up later.
The references should come after examples. Have a look at https://github.com/sympy/sympy/wiki/Writing-documentation. Also the `<` and `>` are not needed.
How is it possible for an expression *after sympification* not to have "_eval_expand_trig"? I suppose that it may sympify to `Basic`, however does this happen anywhere?
This and `test_a` should test qapplying to numerical and symbolic states.
I've no major issue with doing it either way, or with changing every docstring to do it either way, but abc is currently standard (unless I missed the change) I don't think it should block this pull. Feel free to open a new PR changing it everywhere and we can discuss it there.
And `_smat` is not one of the args. It is obtained by harvesting after feeding the args to MutableSparseMatrix.
Shouldn't this be "query", not "path"?
If you're using `with`, you need to `from __future__ import with_statement` to maintain py2.5 backwards compat
More conflicts for me to resolve... :(
It would be nice if this could be decoupled some. I was thinking of using `Janky::Notifier.completed(last_build)` but I don't really want hubot to be notified.
Style: please split declarations and assignments (i.e. declarations first, then assignments). One declaration per line.
I tried again and this time it works with CRITICAL_SECTION.
You don't need to learn much *to* start using Geolocation ;)
Please don't change unrelated stuff in your PR, especially since this was correct, it tests that the load method loads something.
Whoops, indeed.
Consider converting this entire block to jquery.
() vs []
What is this removed for?
Looks like underscore is consistent with the rest of the file though.
When would `$topics / $config['num_topics]` be greater than 100?

I'm not a huge fan of the use of ternary here.
BaseCheck._runCheck probably shouldn't exist.  We should call self.run here instead (and emit the run event from there if we want).
@Kami 

I'm not sure why I thought this was JS, but lua seems to handle an __or__ statement just fine:

Lua 5.1.4  Copyright (C) 1994-2008 Lua.org, PUC-Rio
> a = nil
> print(a or 2)
2

a and a or b would be the equivalent to the JS ternary a : a ? b
Aș putea trimite toate informațiile formatate gata în tupluri din codul python. Ar arăta mai curat template-ul dar s-ar complica funcția din views. Aș putea scrie o funcție separata care sa pregătească tuplurile și să o apelez pe aia în funcția din views pentru a nu o complica prea mult.
昵称改为 「登录名」， nickname 改为 「login」会不会更好
а как же правило транзитивности
```php
FALSE==TRUE
```
sorry, just meant the particular two primes that are written out twice should be bound to vals to prevent the dup. This is a "magic string!" comment :8ball: 

```scala
val DEFAULT_PRIME_INT: Int = 16777619
```

then inside,

```scala
val PRIME: Int = DEFAULT_PRIME_INT
```
I missed, in case we use PHPIDS as a censor on many servers but using a central DB, MyISAM will be stuck on table lock, if we are in big wave of scanning or attacking.

Thanks Lars.
Before, commiting the code, please run your code through jsbeautifier.org. Other things to be kept in mind (like how to comment on Ticket after you push the code) while making commit  can be checked here: https://raxaemr.atlassian.net/wiki/display/RAXAJSS/Contributing+code+-+How+to+Commit+Code
Don't remember seeing this before. @rhattersley is this something we want to remove?
Not sure how this works internally, but "overrideValgrind" followed by "isValgrind" reads like it might be a bug.
!= and save the else?
staticmethods don't have "self".
Ещё я бы добавил `paths`, где можно указать массив путей для чекаута из svn
This stuff scares me, too. I don't see any reason why the altered model needs to be in any cache at all. Also, if the altered model contains foreign keys, or is pointed to by foreign keys, wouldn't the referenced/referencing model's ._meta then get changed, too? I believe those contain at least caches which could then mistakenly contain references to the temporary model. It is not enough to use a temporary cache. You would need to deep copy the models, too, to make this safe. Basically, it is going to be extremely hard to do proper cleanup.

If we want this patch into 1.5 all issues can't be solved. The migrations stuff is going to be more or less hacked in for 1.5, as other parts of the code doesn't yet have the support needed for this. Waiting for that to happen is going to be somewhat long wait if I am not mistaken...

So, at least we should document the crap out of every hack taken, and warn loudly about the potential pitfalls. I am afraid somebody is going to start using the dynamic cache in request processing, and it is 100% guaranteed that is going to end up ugly.
i mean why not just:

```
if _dummyTrans:
    return trans.response().getvalue()
else:
    return None
```

if you're using transactions at all, regardless of why or how, `None` is technically a correct response
My previous commit must not have deleted this line from the VS file when I removed the ruby tag unit test. We'll have to get someone to add it to the VS once you're code lands.
Better use $.data() instead, for performance reasons...
Don't use informal contractions "can't" in error statements, instead use the more formal "cannot".
If someone pushes a dupe onto the very public `Accounts._loginButtons.loginServices`.
That doesn't look quite right for non-local settings…  ;)
Yeah, go ahead!
Yes, I couldn't debug it, but I think it was missing parentheses, possibly `mv (lib+'mpi.mod'), include if (lib+'mpi.mod').exist?`
I do know that the conditional was evaluating to true because I tested it with an `ohai`.
Let's print this only if `--python`.
Don't strictly need a test.
Hi, of course I can't retreat from a javascript debate :-) 

In general the use of setInterval is discouraged because if the computations inside the given function take more than the interval, the executions of the same function will overlap. The common advice in this case is to use a setTimeout that at the end of the function invokes itself. An interesting discussion here: http://stackoverflow.com/questions/729921/settimeout-or-setinterval

As to wrapping ALL the things in a closure, it's considered a no-brainer, if I have the right understanding of the feelings of the community about it.

The main reasons I can tell off the top of my head to do are mostly related to the "don't make me think pattern"

- if the code is not in a file, I can't easily discover it when I look inside the javascript directory
- I can't easily track the dependencies
- I can't easily validate it (if it's in a file I can fire a JSLint run in most editors)

For example, the first thing I wonder is where the `faye_client_with_auth` comes from: of course I can grep it, but it's probably a good thing to make that dependency explicit.

From the strictly technical stand-point I agree with Chris. I would also remind that I couldn't enable strict mode, because if I do it in the global scope, then all the code in the page will be interpreted as strict, even if it isn't designed to be "strict-safe"

Generally speaking me thinks that is a good thing to expose in the global space only the reciprocal dependencies (as long AMD is not used, which is fine), not the business-specific code. 
And even in that case, putting everything inside a namespace (properties of a single global object) so I can easily explore what really available to the user of the global modules.

Let me know if this makes sense... :-)
fichero o archivo ?.
Thanks for this hint.
Alison sent a new line for this so it does not include the date. Check on the email I forwarded.
Normal que tu ais supprimé des paramêtres?
This output is rough. Maybe mention that the bind doesn't immediately take effect and requires the server to send the binding information back to the consumer.
what does `updatedValues['$set']` mean?
I wasn't considering all cases. In that case, how about this:

    @returns <type> [M [N]]

Here are some examples to illustrate:

* `@returns request ` - returns at least one request
* `@returns request 2` - returns at least two requests
* `@returns request 2 10` - returns at least two requests and no more than 10 requests
* `@returns request 0 10` - returns no more than 10 requests

Same goes for items.

Maybe make aliases (`request` -> `requests` , `item` -> `items`) so that both plurals and singulars work.
(Really I'm not arguing with you here... depending on your definition of "support.")
This suddenly seems quite complicated. .  . . I'd hate for all of our "extension" features to be implemented in one method.
The indentation looks off here, it should be 4 spaces.
You have something against split and join? ;)
It may be safer to call this in `_setupPlayer` which gets called on '/page/transition/end', so we're sure it's not doing any expensive operations before the transition is finished.
The docs for testKey() erroneously say the option is 'shift' instead of 'shiftKey'
Although it's tricky, I wonder if we could add a unit test that covers the "every 24 hours" aspect of things?  That seems like a trickier part to hand-test (more likely to go unnoticed if it breaks).

E.g. force the prefs to say the last update was > 24 h ago, launch Brackets in a test window _with_ update checks enabled, verify that the dialog does appear.  Maybe even check that the pref timestamp has been updated afterward, or launch a second time and make sure it doesn't try to ping again.

That would entail a few wrinkles though, like having createTestWindowAndRun() not always turn off update checks, and perhaps not suppressing the part where UpdateNotification sets the prefs (unit tests run in a prefs sandbox anyway, right?).  Maybe it'd be better to spin off a bug to track adding a test that complex?
Typo: execited -> executed
When we get to the localization task, we should consider changing this to a numeric format.  That would make it easier to do locale-specific date formatting without having to either (a) generate it on the server, or (b) hand-format the date in N languages each time we push an update.
This code should be replaced by a call to ExtensionUtils.loadStyleSheet()
Instead of using "document", we generally try to use "window.document", then you don't have to list it as a global.
Nice catch, I'll have a look. I just wanted it to be empty by default. The classes in symfony have little documentation about their intended usage.

Btw, the code works when I actually run `./bin/satis build --stylesheet=./path/to/my/styles.css ...` – just the test fails.
perhaps. note to self: fixed
Class names should begin with a capital letter.
rightToolbar not rigthToolbar
Is there a more Neckbeard Friendly way of formatting this switch statement? Maybe will less newlines?
I prefer here NIL_P method here. It looks nicer :)
If someone is looking for using a body in JSON format, this would be the right place to make an additional check in order to set a different Content-Type header.
Please add comment to explain to people why is this needed. Something like:

```
_ = lambda s: s # Dummy function so that makemessages finds it
```
Spacing after `function`
This is in another ticket, no?
The path location will be different if change to /Login_Form/check_login.php
In my head I was going to use this syntax as the standard form for tests:

    subject.not_if = '[ 0 -eq 0 ]'

What do you think?

I was pondering doing a ruby dsl for it, like `subject.not_if = [0, :eq, 0]` but was probably going to decide against it for simplicity.
What's up with the crazy line breaks?
don't forget to release early after the initial test cycle
Sur une autre ligne le {.
Move this else to filestatus if check
Didn't spot this one at first, foward -> forward
I believe we are undertaking an effort to remove all "rescue Exception => error" from the code base in favor of 'rescue => e' as per https://github.com/Katello/katello/pull/176.
I would probably rewrite this render to be more concise. I don't see the need to be so verbose.

```ruby
render "layouts/alchemy/layouts/login_layout", :login_form_path_string => ...
```
Or `Loader` for the constant and `LoaderImpl` for the class ?
Ugly code with side effects, there's an overload to select that passes the index so this becomes:

    return action.Parameters.Select((index, p) => Bind(parameters[index], p.Type)).ToArray()
Yes. Might as well remove it.
Good point about the line stripping.  Blank line stripping for the data parsing happens in `BaseData.process_lines()

I'm worried about changing the behavior of the core `ContinuationLinesInputter`.  It's unlikely, but maybe someone has used this for their own purposes and now it breaks.  So what about just copying your new `ContinuationLinesInputter` into `daophot.py` and making it `DAOphotLinesInputter`?  I.e. a specialized inputter that will definitely work here without potentially changing the core routine.  Then revert the `core.ContinuationLinesInputter`.
@astrofrog - agree on the relative import.  Is there a standard in astropy for dealing with compiled modules that may not be available during the build process?  I would favor doing something like:
```python
try:
    from . import sofa_time
    _has_sofa_time = True
except ImportError:
    _has_sofa_time = False
```

Then the Time class can check in `__init__` and raise an exception.  I wonder if this is even needed since the installation should fail hard if sofa_time cannot be built.  Anyway it can't hurt to have a check.
I like this, but wonder if it would be good to pass the additional API in within an optional opts object, creatgeScript(script, {trustedAPI: "source"});

It seems like with the current approach there's a risk someone might accidentally pass untrusted code in as the wrong parameter

One other super-minor thing, I might make the unit test call anotherFunction() rather than calling it through the other API, just clarifies what's going on.
Why does GetProxy have NET_IPV4 all over? do IPv6 proxies work?
In my opinion that condition is a leftover from something. 1.3 also always does the call to the header method. I think it used to be that it would only use status codes when there was an actual redirect. Hence the >=300 & < 400, but then someone decided to use redirect() as a header method that would stop execution and that is where it went sideways.

My suggestion is to merge this into 2.1+ to fix the bug and for 2.2 to perhaps fix the AuthComponent to make it use the CakeResponse object and have that stop the request instead of using a redirect which doesn't do a redirect.
Wouldn't things be more consistent if the input type was always a timestamp instead of a formatted date string?  There are probably some tricks in dealing with timezones, as currently the datetime string has no timezone information.  It seems like a workaround to skip the date() formatter.  Perhaps the existing deconstruct() logic needs to be reworked though.
*ok, I will refine it
*
On Thu, Aug 16, 2012 at 5:36 PM, Sven <notifications@github.com> wrote:

> In emesene/e3/webqq/Worker.py:
>
> > @@ -0,0 +1,564 @@
> > +import os
> > +import sys
> > +import ssl
> > +import time
> > +import Queue
> > +import base64
> > +import hashlib
> > +import e3
> > +import StringIO
> > +import logging
> > +
> > +
> > +try:
>
> You should not try/except this. If there is an importerror, emesene.py
> should handle it so we know that this session is unavailable
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/emesene/emesene/pull/1460/files#r1390970>.
>
>



-- 

*Best Regards~*


*Xiang Wang*

*CDC, TrendMicro*
Dat zou eventueel ook kunnen. Ik heb alles met die permissions afgesplitst in 1 model, dat leek mij het properste.
Won' this change the figure size of all figure types, not just svg?
ok.  you can update this branch with a clearer annotation if you like
There's no need to add any exclusions. Just check-in whatever files you need.
```
$ cd ~/git/libgit2/tests-clar/resources/testrepo.git &&
> git --git-dir . fsck --no-dangling --verbose 2>&1 | grep ^Checking | grep 'commit\|tag\|tree\|blob' | wc -l
1681
```
oops, I meants `/etc/socorro/postgres_middleware.ini`
since this returns a tuple, you could do `toc, content = _fetch_mdn_page(url)` if you want to be more explicit
what about splitting all this logic in different methods ot the BIDAuthenticationMiddleware?

We could have for instance a method in charge of returning bid_user and caching it, one for monkey patching etc. I think it would make this code easier to read..
Would `self._query().get(talk_id)` work here, too?
Looks like something wrong happened here on out - The entire functions were rewritten?

@adamfairholm  - @philsturgeon - I like the idea behind this one what do you think?
Write out package_node
This looks useless to me as you are continuing anyway here
integer|false
The previous code was not removing the extra values from ``$clientData``
Redefining the constructor doing nothing except calling the parent one is useless. Simply keep the inherited method
@stof this is called at the same level of run, so it makes sense to have it public as the run method
``is_callable`` can be an issue here as strings can be callable if you have a function with this name
I think this is fixed in 2.1
It's in the next PR.
@vicb No problem :) 

Do you have other suggestions?
this one is duplicated now. you could use this above the corresponding code generation

    if ($methods || $hostnames) {
        $gotoname = 'not_'.preg_replace('/[^A-Za-z0-9_]/', '', $name);
    }
It might help to rename `PrivateMessage` to something like `PrivateMessageTemplate`. It has a very clear responsibility right now of gluing the current class' instance variables to an ERb template in `app/private_messages`.

This test doesn't test that work. It tests the `help_out_test` logic, which nicely cohesive with a user private message.
How about just set the cookie directly?:

    cookies.signed[:yammer_user_id] = '123'
突然觉得htbp这几个参数是不是给用户来配置更好？而且只给出 h 参数会不会有些影响？(未验证)
Reference cycles. If someone uses the block this way:

self.splitView.dividerDrawBlock = ^(CGRect rect, NSInteger index) {
    NSRectFill(self.splitView.frame.size.width);
};

...they will incur a reference cycle. And sorry for the lame example. ;)
remove the (d<=n)... bit confusing since the dimension  "d" is never used
Um... yeah.  I'll fix (and that will require updating all the ref images slightly).
> The reason is, for 3 and 4 channels if you don't tell us alpha channel we can assume for you RGB and RGBA, but if you give us an image with 5, 6 or whatever channels, we can't assume anything reasonable so we fail those cases.

Ok, clearly there has been some miscommunication on the mailing list, and we've "agreed" on different things.  I thought we had agreed on some general rules for handling spectral color when alpha was missing (other than failing hard)!
Looking at SocketInput::open() suggests to me that this const_cast + open() + close() dance is a no-op, since open() returns immediately if nowait is true.  Perhaps I'm missing something, but I hope not because the const_cast stuff is making me nervous!
Yes, I was waiting for that to be confirmed/merged.
This does not compile due to a typo, it should be:
```__TBB_Pause(delay);```
```ruby
thin = case size
       when "small" then 2
       when "large" then 4
       end
```

or

```ruby
thin = (size == "small" ? 2 : 4)
```
use SERVER_SIZES
Насколько мне известно, в таких случаях принято писать всё же что-то вроде
```<% if @user.posts.any? %>```
please reference those correctly using ``:class:`menus.base.Menu```
why not make the __repr__ be Array(element_type, shape) like many of the Python builtin types?
codeis -> code is.
I like to name the test class metrhods after the tested method, and to express requirements in test method names (see http://www.phpunit.de/manual/3.0/en/writing-tests-for-phpunit.html for an example). So I guess you should split this method into four methods with a single assertion each. The new methods should be named `testSentenceWithZeroNbWordsReturnsEmptyString()`, `testSentenceWithNegativeNbWordsReturnsEmptyString()`, `testParagraphWithZeroNbSentencesReturnsEmptyString()`, `testParagraphWithNegativeNbSentencesReturnsEmptyString()`.
I see, although I'd still like to clarify a couple of things, because the documentation says the entire `Library` folder should be ignored (which is done on line 4) however files from this folder are subsequently re-included (on lines 6 to 14). So I guess I have two questions:

- In the earlier version of Unity, why are the files specified in lines 6 to 14 being re-included?
- In the later version of Unity where things have been moved elsewhere, which lines above can be removed?
Yes it is still an issue in fact restarting  AS takes a while and we probably have to wait a few seconds in the test.
I didn't find org.jboss.as.weld.WeldLogger class, is it in the jboss-as-weld project?
yes it should!
----- Original Message -----
> From: "Radoslav Husar" <reply@reply.github.com>
> To: "Pavel Janousek" <pjanouse@redhat.com>
> Sent: Monday, April 2, 2012 12:12:42 AM
> Subject: Re: [jboss-as] implementation of Exception handling TestCase (#1931)
> 
> > + */
> > +package org.jboss.as.test.integration.jaxrs.exception;
> > +
> > +import javax.ws.rs.core.Response;
> > +import javax.ws.rs.ext.ExceptionMapper;
> > +import javax.ws.rs.ext.Provider;
> > +
> > +/**
> > + * Exception mapper - occurred NPE exception remaps to HTTP 404
> > Error code
> > + *
> > + * @author Pavel Janousek
> > + */
> > +@Provider
> > +public class NPExceptionMapper implements
> > ExceptionMapper<NullPointerException> {
> > +    public Response toResponse(NullPointerException ex) {
> > +        return Response.status(404).build();
> 
> Maybe reuse HttpServletResponse.SC_NOT_FOUND for 404?

Reasonable, changing...

Pavel
All events in this switch will have `event.preventDefault()` called, unless the `var prevDef` set to false, see case for TAB or ESCAPE for example.
The case that this is trying to solve is multiple dialogs at the same level, and switching between them. You should be able to test with by creating two dialogs from divs directly in the body. As you switch focus between them, the one with focus should be on top. I think the only way to deal with this without changing the z-index is to append the dialog to its parent.
But I added 'Cost Control' so I cannot remove that colon.
Why? I think we have to keep old behaviour... if you click 'Confirm' you have to resend (TODO in line 889) and if we click 'cancel' we remove it from DOM and from pendingDB (check in device, if we change this condition as in your code behaviour is wrong)
I wish I had a better solution for this to suggest...

What about leaving a comment explaining the INT_MAX hack?
Maybe we can enclose this with a closure to avoid polluting the global namespace.
sorry, don't worry about that.
Assuming you a. have no text nodes or b. you don't care about the difference.
Same for all the transforms rules.
Where do you dismiss the dialog shown by showDialog()? Its probably in there already; I just didn't see it.  (Have you tried pulling and re-inserting your sdcard while the app is running?)
var TimeManager = {
  time: navigator.mozTime,
  
  init: function tm_init() {
    // Not sure what you have to init :)
  },
typo: `closeAlert`
Why is a separate config needed here? Just always use STORM_LOCAL_DIR in the case of default storage.
I think that this should be in another method, like `noRemoteLink(link)`
can you fix the indentation here?
@greggroth -- Can you update things so that all these formatter helper methods are declared private?
Actually the ScalaTestLaunchShortcut.getScalaTestSuites method is not used anymore, I have removed it in the following commit:-

https://github.com/skyluc/scalatest/commit/615564046ce31b0e1bd75639f92608fe29396bdd
Can we get a test for Map[Int, Double] too?
explained the reason.
Maybe instead of `strpos()` just compare ?

```php
if ('..' === $itemData['name'] || '.' === $itemData['name']) {
    continue;
}
There's also instanceof TableKunenaMessages (remove single message).
I'll try :P
Linuxの設定というより、Redmineの設定でしょうか。また、イントラネット環境だと、GmailやWindows Liveは使えないことが多いのでその補足説明もあった方がいいと思います。また、LinuxのSendmailコマンドを利用する場合は、Postfixなどが適切に設定されている必要があると思いますので、その記述もあった方がいいです。

ALMiniumはLinuxの知識がない人にでも使えるようにしているので、もう少しわかるように説明が必要です。
The code looks like ``IEquipmentDatabase.isEmpty()`` would suffice for the moment.
Feel free to add either.
These functions should probably be changed to not refer to post any. (And changed to camelCase, since if they are changing anyway).
This function seems like it would be a useful thing to have on `BaseScheduler`. Maybe `addBuildsetForSourceStamps` or something?
There is no import or definition of `delete_io` in this file? Is this meant to be implemented later?
Ok, i understand it now.
Could you add comment at this line, then we know why we use intptr_t here.
And i think we should not use stl or template if possible, because they are hard to debug, and hard to generating binding codes for lua or javascript.
the body of the ``switch`` should be indented
it is. But before executing a test, not a full testcase.
Forgy  
1974
(date of his first paper on the subject - 1979 is the year of his Ph.D. thesis)
:punch: :skull:
Might be nice to reference what these ids map to, and where to look them up?
lots of repeated code = less variables?
But the line below, return a tuple.
teams find { case (k, v) => v.url.endsWith(team) }
whats the reason for making this inline? Not performance critical code in any way.
Never checked it seems.
Why not `columnNames`? Or are these being used as column identifiers (i.e. as keys) as well?
This might be better off as:

```ruby
self.reset_password_sent_at = Time.zone.now
```

What do you think? Does devise handle time zones / UTC already?
in text-input it was defined as autofocus: {dataType: "boolean"} is that a significant change?
No in fact `none` is very useful, as all the consumer code won't have to test if `implements` exists and if it's array.
I believe this comment is not quite right. What unbind does is binds given object's method to it's `.call`, so that `this` pseudo variable to it could be
passed as a first argument:

```js
slice = unbind(Array.prototype.slice)
slice(arrayLike, 1, 5) // => Array.prototype.slice.call(arrayLike, 1, 5)
```

More precisely it's equivalent of following code:

```js
if (arrayLike.hasOwnProperty(slice)) {
  if (arrayLike.hasOwnProperty === Array.prototype.slice) {
    return arrayLike.slice(1, 5)
  } else {
    var original = arrayLike.slice
    arrayLike.slice = Array.prototype.slice
    var result = arrayLike.slice(1, 5)
    arrayLike.slice = original.slice
    return result
  }
} else {
  arrayLike.slice = Array.prototype.slice
  var result = arrayLike.slice(1, 5)
  delete arrayLike.slice
  return result
}
```
If I'm reading this right if someone creates a Loader with a custom resolve function they will have to implement relative module IDs themselves. That doesn't seem ideal and maybe should always be included.
Directly calling localStorage like this will break on IE6 or IE7 when testing to see if the unsupported_dialog works as expected.  We could use a shim or an abstraction layer.
not sure I understand why this was changed?
Our [style guide](http://mozweb.readthedocs.org/en/latest/js-style.html) prefers a space after `for`, `if`, etc.
Yes, it will be. That would be caused with the following usage.

```php
<?php
new Validate('name', 'test');
actually, move this to main.js and set the onkey listener there then call appnet.findBff or something which the bottom stuff should be set to within var self here.
:+1: we should have done this much much earlier...
Could we just take a splat and reapply it in the case of `exception.instance_of? Class`

Admittedly, I've not fully thought this through, but for completeness, can you find any holes with ...

```ruby
def and_raise(exception = RuntimeError, *args)
  if exception.instance_of? Class
    @exception_to_raise = exception.new(*args)
  else
    @exception_to_raise = exception
  end
end
```
Same thing here. The project already has matchit. I just hadn't created the bindings that you use :)
These `<script/>` tags should go in the `<head/>`, which can load them asynchronously.
If $annotation is a string you already know the $class. I suggest treat this as an else of the first if
@Maks3w The previous conditional is more specific, however, and will end execution of the method faster when matched. It makes sense.
I keep the same that Zend\Serializer\Serializer to keep consistency.
But, I don't get why this is a logic change, because if the `if ($usePeeringServiceManagers && $retrieveFromPeeringManagerFirst)` fails, the `$instance` is still `null` so, by adding an `else` instead of an `if (!$instance)` it does not change the logic.
Use setExpectedException and a dataProvider instead of foreach and try/catch
My bad, your right.
`getOneOrNullResult()` would be more useful.
I'll switch to a ugly 5.3 solution
This is the config file for Azure Diagnostics. Pretty much everything we want to do can be done here. More can be done with code. What's cool is that we can push a copy of this file to Azure Blob Storage with a special name and it will OVERRIDE the running instance's config. So if we are in a scenario where we need additional diagnostics, we don't have to redeploy to get them.
Not sure I understand this correctly, but won't this break the app in development outside of a VM?
Missing line break before and after
could we add another method here to get the PHPCR query? maybe getPhpcrQuery() ?
A new signal would be cleaner, I can try that. Maybe it would be most generally useful to put it at the top of `generate_output` or `generate_articles`. How does an `article_generator_output` signal that gets sent once with just the generator object sound? Or an `article_generate_output` signal that is sent for each article?
IMO it would be better to make tests that catch these, and change the whole Pelican into iterator-resistant. Or are there any particular reasons it needs real lists?

Of course I do realize that would be more work, but there should be at least a comment here mentioning that this is just a safeguard, a real list might not be needed. Otherwise someone in few years won't dare to change that list into an iterator, since it looks like it's intentionally like that.
What do you mean here? Is removing the ':' still valid syntax?
Fixed in 568e3aac9dcef234828762b28899e67262761b60, coming up in a future pull request.
Fix these according to the new format (one per line)
put `patches` below `sources`, leave empty line in between
So maybe add an empty `page_js` block to `base.html` and override it here?
Why was `values_list` removed from here?
I think I would prefer wording like:

10.0.7esr#1: Execute add-ons tests for ESR10 candidate builds. 
10.0.7esrpre: Execute add-ons tests for ESR10 nightly builds. 
16.0b1#1: Execute add-ons tests for Beta candidate builds.
15.0#1: Execute add-ons tests for Release candidate builds. 

...or something along these lines.
no space in-between the brackets after `Popcorn.guid`
actually the entire <code>__pycache__/</code> directory can be omitted from the pull request.
Excellent.

On Friday, December 14, 2012, Jan Klass wrote:

> In opus-src:
>
> > @@ -0,0 +1 @@
> > +Subproject commit d060dd7cbd6405c8a31510ca17adb1e051934f49
>
> Ok, if you’re not that familiar with git, the one merging should be able
> to fix it. :)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mumble-voip/mumble/pull/139/files#r2427832>.
>
>


-- 
James Sumners
http://james.roomfullofmirrors.com/

"All governments suffer a recurring problem: Power attracts pathological
personalities. It is not that power corrupts but that it is magnetic to the
corruptible. Such people have a tendency to become drunk on violence, a
condition to which they are quickly addicted."

Missionaria Protectiva, Text QIV (decto)
CH:D 59
Yeah, of course. Be brave and raise it!
Why using an env. variable to disable and not a configuration key? (curious)
I can try doing that.
But why is this dependency problematic?
It only takes effect on requests that contain the "callback=" argument, so it shouldn't have any effect on other requests.
And it sounds common enough to have it in by default. It is also very convenient to use.
mind adding a bit of docs why we are doing this join this way?
The only thing I'd make sure of here is that the / works fine in Windows. Since I know sometimes its wacky with it's /'s and \'s.
Would you mind removing spaces around keyword arguments per PEP-8 please?
Doesn't need a comment, it's obviously jQuery. Please refer to a local jQuery, as this line is too wide (80 characters maximum with code blocks).
Sure, I'm going to add an inline comment pointing to the MySQL documentation
Deleting this header causes a test failure in TestOfMapController. Was deleting it intentional, or should I restore it?
if before your changes, the spaces were allowed, then that would break people's current code, which isn;t good, give me some time so I can think of a better way to solve this.
Moved over to init.
Weird, how are all these CQL key words lower case? I could have swore we made them all upper case by now.
Since we're now using mocha+make, I think this scripts key is no longer needed.
I'll make a mental note to remove them from the other packages as well.
You can have a look here: https://wiki.scala-lang.org/display/SW/Tags+and+Annotations, the Macros section (it turns out Scaladoc had macros for years!! :))
Let's make sure I understand what's happening here: you are returning a mocked ``field`` if a field with the passed ``name`` exists in  ``parser.fields``, ``null`` otherwise. Is that it?
``$lang``, ``$INFO`` and ``$DIFF_INLINESTYLES`` are unused globals in this function (they were previously used for the mail content).
"запущена на машине" sounds good for me.
@tsetimmy 
and I don't want to modify your overall structure to avoid bugs.
btw, I modify your print as well.
Why `(.*?)` ?
Later, if it's ever needed at all. `querySingle()` is too specific and `_prep_query()` turned out to be unnecessary.
My whole point is - I don't get the whole idea of why do we have to have a `_ci_loaded_files` array, fill it, check against it, etc. when we can just use `include_once()` or `require_once()`. Those functions WILL always be faster than whatever tracking we implement on our own.

On a side note - I don't really care if it is `require` or `include`.
I've used it, so I guess other people have too. I personally wouldn't care and was actually surprised that the index started at 1 instead of 0, because naturally - that was my first guess. But we should maintain backwards compatibility.
Да, второй вариант получше. Спасибо за замечания
Не бесконечный он, будет выполнен DNA.count раз, это же обычный for
Заменяться будет строго percent элементов, т.к. 100-i всегда будет больше или равно, чем  percent-percentMutated. Если при очередной итерации вдруг 100-i становится равно percent - percentMutated, то при этой и всех последующих итерациях условие будет выполняться, и будут заменены все последующие элементы
да, проверку, что такой символ уже был на этом месте нужно добавить
На мой взгляд это делает код менее читабильным, запутывает других разработчиков и усложняет отладку приложения.
I don't know how to fix that
With changing this, you should also change the sizes for the photos in app/models/event.rb:
    :styles => { :featured => ["760x524#", :png], :large => ["290x200#", :png]},
Yeah, and limiting it to one fibier is done with a mutex.

Btw, that might explain why we get so many "unacked" messages in rabbitmq web interface sometimes (because we are processing many messages simultaneously).
Happy to change that.  I was anxious to distinguish it from the Gherkin formatter which is doing the actual formatting work.  Will push those changes up tomorrow morning.
That's your call. Both instances of that are very close to each other (so presumably hard to miss if updating one) and wrapping it in a function might even take more space/size than just leaving it. But if adhering to DRY is important, it wouldn't be a crazy thing to change.
Explicit comparison since it's not a BOOL.
As above with the backticks, although reading through this it seems you are intentionally saving that only for method names. How come?
unmixing
would not it be more natural to interpolate on the same time grid?
assert_equal ...
actually i was thinking we only mention types in the strict sensw and explain the meaning in the following lines. so should be str, as neither path nor string are names of types im Python ^^

On 02.12.2012, at 21:58, Martin Luessi <notifications@github.com> wrote:

> In mne/datasets/sample/__init__.py:
> 
> > @@ -25,13 +25,15 @@ def data_path(path='.', force_update=False):
> >      ----------
> >      dir : string
> should be path (not your mistake, but you can fix it while you are at it ;-) )
> 
> —
> Reply to this email directly or view it on GitHub.
>
There should be no space around the `.` operator
`nom_default` should be `non_default`
``Registering the listener``
I'm not sure this logic does what you want. Seems like you might want `(mimeType.startsWith("video/") && supportedVideoExtensions.contains(extension)) || X_MEDIA_BRIGHTCOVE.equals(mimeType)`. wdyt?
@phene Ouch!

@mschulkind and @phene, I am very willing to break backward compatibility. @mschulkind nailed it when he said "I fear there is code out there relying on this". Given that someone will complain, which users do we not want to complain? What are valid uses of `#reprocess!` that we want to make sure continue to work, and which should we toss? And of those valid uses, is there a better solution than `#reprocess!`?

(While we do use Paperclip, we do not use all the features of it in every app. Not sure when I last used `#reprocess!`.)
What's the deal with keeping a CopyOnWriteArrayList in this class? It seems to serve no purpose AFAICT
for(s <- connectionStatus; c <- s.right) c.cancel
@cgohlke: you're right that this is currently the common practice in NumPy, and my comment is certainly not to warrant the inclusion of your patch.

@charris: actually, we define npy_intp to Py_intptr_t if I read npy_common.h correctly. ssize_t is posix, not even in C99 IIRC.
If anyone has ideas to make this make sense, or if I missed something, I'm interested.
> Never mind; I can't send you a personal message - no email address with Github?

I do actually but now that you mention it... I haven't received a PM in ages.

> 1.) We stick with the solution above. We document that setting the honorCipherOrder option is identical to setting the SSL_OP_CIPHER_SERVER_PREFERENCE on secureOptions.

I can live with that but use this instead:

```javascript
var secureOptions = options.secureOptions || 0;
if (options.honorCipherOrder) secureOptions |= constants.SSL_OP_CIPHER_SERVER_PREFERENCE;
if (secureOptions) this.secureOptions = secureOptions;
```

It's kind of silly to set `this.secureOptions` conditionally but let's do it anyway for the sake of backwards compatibility.
Поставь тут в запросах пробелы вокруг равно. Да-да, и в SQL тоже.
using ``.git`` to store the cache seems weird. It is likely to break things when using git as it may make git consider this folder as a git repo whereas it is not one, or it could mess the git repo if it is a real one.