labofsahil/github-event-dataset-2012 huggingface dataset:
Fixes session security issue

According to the Session documentation, the invalidate method expect one parameter _($lifetime, integer)_

> Sets the cookie lifetime for the session cookie. A null value will leave the system settings unchanged, 0 sets the cookie to expire with browser session. Time is in seconds, and is not a Unix timestamp.

With the current implementation, calling `->invalidate()` will keep the session alive until the browser is closed. Even if the /authorize action is a redirection, a malicious user could exploit this breach.

The fixe will reduce the open window to 1 second after issuing the authorization code (minimum allowed by the invalidate method)
smdk4210: Mali R3p0

This should be merged alongside http://review.cyanogenmod.com/#/c/23797/.
Updating next-BB10 based on changes in master
Added support for startkey and endkey

Hey Jan,
I am using your backbone-couchdb library for an application and notice it was missing support for using the startkey and endkey as allowed in the couchdb code.
It works the same as how keys work (inside the db object inside a collection)

Here is an explanation of the logic to determine what can and can't exist in a request.
<code>
if ((coll.db != null) && (coll.db.view != null) && !(coll.db.startkey != null)) {
  delete _opts.startkey; 
  delete _opts.endkey;
}
</code>
If there is no startkey, make sure startkey and endkey are null
<code>
if ((coll.db != null) && (coll.db.view != null) && !(coll.db.endkey != null)) {
    delete _opts.endkey;
}
</code>
If there is no endkey, delete it
<code>
if ((coll.db != null) && (coll.db.view != null) && (coll.db.startkey != null)) {
    delete _opts.keys;
}
</code>
If there is a startkey, delete key.

This may or may not be your intented functionality (startkey overrides key) but this could easily be changed.

Apologies if this is in the wrong format in anyway, as this is my first submission to github.

Regards,
Nick
Add 'Same as previous patient' button for contact information

With respect to Issue #31

## Comments to consider

### New dependencies in test

I went ahead and added capybara and database_cleaner as test dependencies, in order to be able to write integration tests using the capybara api. 

My reasoning for this is that there was no infrastructure in place for integration tests, and this app runs very slowly on my local machine. Integration tests were important to enable me to test javascript behaviour without having to manually click around the page and wait for each load (each test would take me about 2 minutes; it takes about as long with capybara but I do not have to do it manually).  The integration testing support file was adapted from PuzzleNode.

### Send previous patients information along with original request

There are two options I see for getting the previous patients data: 
1. Add a JSON string at the top of the file that contains all relevant information
2. Make an AJAX call when the button is pressed to fetch the patient's data

I went for the first option because it was very straightforward to implement and the asynchronous version does not add much except reduced performance and more code. We have already queried the object and loaded it into memory on each request, so I figure we might as send over all of that information with the original request, rather than query for it again if it is needed. The data overhead for adding this is minimal, especially considering the networked environment this application will be operating in. That said, if you think this is not a good idea I would be interested in hearing why.

### Hide the button if there is no previous user
I couldn't decide between disabling it or hiding it, so I just picked one to move along. Any preference?

Run the tests with:
`rake test TEST=test/integration/check_in_test.rb`
LPS-25355 Can not save the pages settings in a user group
jenkins won't resolve android site as a valid URL, so omit it; omit JRebel too as we don't actually need it here >> master
Just added "/obsolete/" to the template of "project for a tag" plugin

...tag.html

Just added "/obsolete/"
Made many changes!!

- More translations into Spanish.
- Setting the language files.
- Side panel with up to 3 diferenes realms.
- Information for each realm in the status page.
- Set of Languages ​​footer.
- Adjust to change page avatar.

If you want see a demo, go to: http://strongworld.dyndns.org
Fix for issue 9

Signed-off-by: jfederico <jesus@123it.ca>
Addresses #3

Reorganized into a plugin and proper extending. Fixes #3
merge from powmedia
Update birthday docs (and add REST doc notes)
[Patch] Can't use Chart.get if a serie wasn't visible first

If you add a series with *visible: false*  and you do a *Chart.get(not_existing_id)* you will get an exception.

ex: http://jsfiddle.net/5eFpg/3/

The problem is located in the *Chart.get* function who search for points in a series with an matching *id*. If the series set *visible: false* when it was added there's no *points*.

The for loop who search for points with matching id didn't check if it has been initialized.

This patch check if there's no point in the serie. If it's the case it skip to the next serie.
rowend/6-complete-feed

Translate
Tests pass!

Fixing #20's test inconsistencies. All tests pass now.
Spelling, grammar changes

Some spelling and grammar fixes, a few word choice changes, and one or two sentence-level changes for improved clarity/accuracy. These go to the end of chapter I.
Update PagerFanta required version to be set to Symfony2.0

dev-master version do not include management of empty route parameter that are handled in dev-symfony2.0  version
Small change to hide the API Key from being echoed

The script echoes the `user` and `api_key`, making it un-secure when
part of a deployment strategy (i.e. Capistrano recipe). I found the
most secured to be to store the `user` and the `api_key` on the
server where the application is being deployed as part of the setup
(i.e. /shared/path/rcfu/.RFUser, /shared/path/rcfu/.RFKey) and use
them as part of the deployment in a command like this:

	./scp-cloudfiles.sh.php \
		-u=`cat /shared/path/rcfu/.CFuser` \
		-k=`cat /shared/path/rcfu/.CFKey` \
		-c=<container> \
		-p=<path>
Added Branch

Please merge
Why not just use the optional second parameter $pressId to PublishedMonographDAO::getById? Would be quicker than looping.
This is wrong. Do not reference globals.
The chained function call / PHP4 thing is going to be annoying in a few cases (i.e. using the Concat function). Feel free to add a new DAO function in these case that delegates to the data source rather than adding too many of these:

$a = $o->a();
$b = $a->b();

...since they're painful to read all over the place.
Why did I do it like this before? Who knows. Less code, less switches, everyone wins.
The docstring for the library count tracker is incorrect; it doesn't have the same API. It's close, but the functions take an extra argument. That also makes me think that the call on line 1385 won't always work, since the default tracker doesn't take a file_type argument.
We have the same problem in Eternizer, too. 

----- (Sent via mobile phone) -----

Am 21.01.2012 um 04:28 schrieb Craig Heydenburg<reply@reply.github.com>:

> @Guite @planetenkiller @drak This usage of DoctrineHelper::updateSchema() completely destroys my database. I need to know if I have used it properly or if I did it wrong. If It is correct then there is a MAJOR bug in the method. Here's what I did:
> 
> 1) install fresh copy of release-1.3 core branch
> 2) install v1.0.0 of Tag (master branch)
> 3) switch to 'Sluggable' Branch and upgrade
> 
> result: All tables in the DB are removed with the exception of the tag_tag table. Good news, the tag_tag table is upgraded.
> 
> also see https://github.com/craigh/Tag/commit/25f37df8ffcddcb1bbe0b62ed56e1ccccc11a7f5 for code changes to include the Doctrine behaviors.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/craigh/Tag/commit/6e27adac8ff1042a2321af54ded169bb9bb0e471#commitcomment-887942
Here you pass with0 who had its zeros eliminated. You cau use with0.size to check the number of values stored in the sparse array. Assert that it is 9 before the theano call and the output of the theano call have 7 values. Making the comparison with todense won't test what we want.
Does this analysis contain strictly the files compiled in this run? For instance, if my project has 100 files, and I call this method with only one file, will the return value be the `Analysis` for my whole project, or just this file (and I need to compose this result with a previous `Analysis` instance)?
Actually `include_with_args` is something that I miss a lot, since there is no good way (AFAIK) to pass arguments to included module. Seems like all those `acts_as_*` plugins had a point.
Yeah, looks like it prints out that warning. My best guess is the filter never worked, and rather a different buggy filter setting somewhere else, now fixed, was masking that.
Shouldn't MiddlewareSessionTest be a subclass of Test::Unit::TestCase instead?

Ah, missed it, forgot what version of rb MRI I was running against. Can't remember but I think activesupport wasn't being loaded somehow.
@pavelvasev I meant that it works on edge rails (4.0.0.beta) from master branch at github
Basically the mouse is not grabbed ever.
Though this is not in the official OIS documentation and it may be removed in later versions.
Without these lines added, I was unable to have OpenDNP3 respond to queries sent by our NovaTech Orion 5R.  Once we added them, however, the Orion began reporting results.  It may not do exactly what is needed, but if it doesn't, something else is wrong.
I think isUniformRNG is best, isUniformRandomNumberGenerator is too long. Should the 'RNG' part really be all uppercase this time?
only the effect is desirable, could be, using some packet data to send to the client before teleporting them, im testing another solution right now, in the next days ill commit some of the testing lines, specially the good ones, improving mapusing, managing, and respawn, also there is a lot of work to do with the sqls, there are many exploits yet.
Memory leak if `!fn->IsFunction()`?
@cscott Alright. I've been trying to wrap my head around AMD and as far as I understand, this seems pretty good. Thanks a whole lot for the effort of AMD'ifying the code base by the way. There are things I like with AMD, such as that it works well with node.js with a little bit of ceremony. I dislike the increased scope-level (hidden or not) in most of the code base.

What I'd like to understand before we go all-in on AMD, is what drawbacks the alternative route, i.e. going all in on CommonJS/node.js require with or without triplets, would have for your use-case? It already works in the browser (although dependencies needs to be spelled out explicitly in form of a set of script tags), albeit not in any asynchronous loading fashion. I assume that there are tools available to combine multiple CommonJS modules for the browser more intelligently than just concatenating them as I've done. Perhaps even tools that convert CommonJS require's to AMD define's.

Does AMD fit well with the node.js community? I want to make sure that now that we drop support for d8, js and possibly rhino, we end up with something that fits the node.js culture as well as possible.
@eightysteele Would love some feedback on this line when you next have time. It seems to me to be a pretty horrible way of doing this (unicode -> float -> float -> str -> Decimal) but it works. I'd've preferred to leave it as a string, but the shapely library needs it in Decimal or float, and -- if converted back to float -- would have the same "2.0000000001" type issues we've been having on issue #4.
What is this needed for? If Homebrew isn't installed to /usr/local?
Why also put __repr__ that return __str__? By default __repr__ do that.
(...by anyone who can guess the filename, that is.)
Wouldn't work, since has_one and has_many only works for 1st-level attributes, and the space data is inside the context hash. However, I think we should create a NotificationContext model and let that that wrap the context. Then we can put has_one :space on that model.
Try it without this line; the linking step should make things that look like shell scripts executable.
Yes, you're right. I've adapted addGame() accordingly in 9476cb1
Yeh definitely still broken, same lagged out servers after few submits issue.
Since this match does double duty, it has to match white space for when it's just a regular string part.
Joke is on you!

    current_user.name # should be current_user.username

:)
Where exactly in `cm_translate.c`? Are you sure you're not looking at code that dates from before commit 6b1246e515e5f351?
This messed up channel switching! I had to revert this part.  op den kamp said "because tvheadend doesn't wait for the connection to be open before returning iirc" :)
ah yes thanks i see you've updated now
Can we add some comments for lines in this method for clarification?  The header comment for this method is excellent, but the implementation details are a tad confusing.  For example, in [L62](https://github.com/prog694/joebot/blob/develop/src/edu/stuy/subsystems/Shooter.java#L162), why does it break if distances[i]>distanceInches?
Oui mais non ... j'ai fais ça exprès sinon il faut être dans le bon rép pour lancer l'appli ... ce qui est galère quand on la lance comme un service
pdf auslesen und meta daten automatisch befüllen, hach wie geil, gute Idee!
They are nicer to read and shorter or else you get long pathnames all over. It does not seem to matter -any reason to use relative paths? Also, some tools did not like relative paths, namely the arpack configure script. But, we are not using that build process now. 

-viral


On 30-Jan-2012, at 12:40 AM, Stefan Karpinski<reply@reply.github.com> wrote:

> Why the insistence on absolute paths everywhere? I intentionally changed those all to be relative.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/JuliaLang/julia/commit/3f7f46096258f2d90e839b91de13bc9540e59897#commitcomment-912437
In JCircuitTech, XOR and related functions evolved a lot faster before this bug was fixed.

I conjecture that it's because this bug corrupted the truth-tables of compositions, and turning an OR to an XOR, for instance, requires changing only one truth table row.  Building a logic circuit for XOR is trickier, however.
Related to issue #1
It is `.travis.yml`, with a leading dot. Please use [travis-lint](http://about.travis-ci.org/docs/user/travis-lint/) like our Getting Started guide mentions.
Alternative suggestion by iocanel: try to load a different _resource_ here (as opposed to using a different _loader_. That would mean making `VERSION_RESOURCE_FILE` visible for testing, and I'd rather mess with the loader, personally.
make the 4 levels problems? Yes we have 4 levels, 0 - normal player; 1 - played characters from gm with mute rights; 2 - ingame supporter; 3 - scripter

I made the ".reload all" in mangos console.
What is the different betweens console and CLI/SOAP?
PS залез почитал про этот fastMM, заодно и на код поглядел. в 64 бит его скомпилить нельзя в принципе (половину переписать придется) только в бридж-моде, при котором естессно никакой скоростью не пахнет (зато память будет экономить). быстрой оно может быть только по сравнению с borlandMM (20 лет уже покойничку), на замену которого собственно и написано. ни с ТББ ни с VC менеджерами ее никто никогда не сравнивал (подозреваю, что в x64 будет раз в 20 медленнее VC и раз в 80 - ТББ, за счет псевдокода и переключения контекста, а в х32 - примерно те же яйца).
@mosst только не работает то что в заголовке.
@tst12 нет, это только корректирует сообщение об иммуне, не более того.
fin.eof() doesn't return false until one iteration after last line is read into datatype/data.
Thanks for the bootstrap 2.0 updates! Merged these as-is for the most part.
Slight update:

APT (says it) grabs debs with an epoch in the filename, whether or not it's actually doing so:

https://github.com/rcrowley/freight/pull/21
Raw XFL being huge is part of it, needing to parse megabytes of hairy XML at game startup should be avoided if we can. Also, having a simplified format makes it a lot easier to write parsers. All the complexity needed to get usable data out of XFL is handled once by the exporter instead of being offloaded to implementers.
I got it, no worries. If I recall it was the version of git which was the problem.
Using git 1.7.7 fixed the issue

Sincerely
That right! I missing us using jquery. And tabindex attribute must added before use focus, I will move set tabindex attribute into JavascriptManager's corresponding method
I Updated the OpenTK version to the latest one avaliable (Night Build http://sourceforge.net/projects/opentk/files/opentk/nightly/).
To be able to use new functions to handle mouse correctely. (also some to benefict from some fixes on opengl drawn functions)

Did not updated the submodule cause i cant ....
Right now it still is. 
mapEvent() is the generic input event method. mapKey() is the special case of key up/down event input.
Both output general events (mapKey() always did really)

KepMaps still only take KeyState as input as of this commit.
Mejor algo menos raro y más legible (y equivalente)

```ruby
Document.find(id_mb).tags.count
```

O también hacer el find por fuera, algo como:

```ruby
doocument = Document.find(id_mb)

assert_difference "document.tags.count", -1 do
  ...
end

```
we should have `self.addon.addon.is_free()` (like `is_premium`)
SEMICOLONS,

USE THEM MUTHAFUCKAZ!
аналогично конструктору ArrayList
@raphaelcosta the gem build system wasn't that's great until #169 was merged in- The date got missed and also the version set in JS was missed.

It should be better/fixed on the next gem. Thanks for your patience!
double getFloat?
Doesn't `application` need to be in the namespace of the .wsgi files?
The change from liking to pinning posts shifts the perspective from the recipient to the actor, i.e. the user "linking" or "pinning" posts. This makes the "Like Stream" (now Pin Stream) more useful to the user, I suppose. It does break with the original intent, though.
I don't care much for the `socorro/[type of database]_algorithms/` folders. RDBMs systems may be similar, but I don't think we can abstract NoSQL techs usefully. It looks like an added layer of complexity to me. Could we fold what you have there into the individual `socorro/externals/*` folders?

I also don't see a way to specify dependencies on other cron jobs here. We currently use the dailyMatviews job to establish dependencies between stored procedures, and its working out well enough that we'll probably introduce an hourlyMatviews job. It may be more useful to think about how dailyMatviews (https://github.com/mozilla/socorro/blob/master/socorro/cron/dailyMatviews.py) translates to configman.

Since data derivation is so dependent on what tech is being used, I think it would be good to setup a cron app in the external, such that each external could decide what needed to be done and what was dependent on which. Alternatively, we could maintain hourly and daily cron apps in socorro, using configuration pointing to particular external apps to be run to populate the data. Some convention would have to be established for a `run()` method, how to report failure, and dependencies.
Hell, you could simplify the other one to return recipes.hasNext() || smelting.hasNext() :P I just pointed out the first one I saw
Is there a particular reason you used foldl as opposed to foldl'? Foldl is generally ill-advised unless the function you're folding is lazy in its left argument and that laziness is desirable.
It's just a shortcut / boilerplate for defining a service and then subscribing that service to files and a package. You'd just install the module and then call the "easysvc::svc { 'my-service':" line with all of the options. After that, you'd have a service, a package, and files all defined you could use. This handles about 90% of the cases for my simple puppet installation right now, so I'd rather just define the services in nodes.pp with easysvc than wrangle a different module for every single service.

If any of my service management needs become more demanding, I'll look for a full blown module or write my own. Probably going to do that for mysql and apache.
It's whitespace-removal man!
Also, get friendly with a debugger. I'm not sure how good you are at pdb, but it's pretty minimalistic. PyCharm has a top-notch debugger, and they give away free copies for use in OSS projects. I used PyCharm for most of the core stuff. As an aside note, PyCharm will also jump into the .clj files. You can't set breakpoints in .clj files, but it'll let you step into and outof clojure code.
- Change Tag: IA to AI 
- Add Tag: artificial intelligence
For me is good but functionality is not complete:

1) Problem when the avatar is set to the left.
2) Whenever you open a new tab, the icon is set to hide avatar.
go ahead, i still missed one last s though
Strange thing is with the standard OE builds this does not cause a issue, what are you changing within your own builds??
¿Que te pasa? ¿Quieres que te meta? ;)
I actually wrestled with that detail of the implementation myself, but chose this approach because adding support for media-stream availability detection later on would need to be a little more responsive; Another process (such as skype) could grab the camera away from us. I can see your point though, most of these properties only need to be calculated once.
I added this null check because if the result is null from the content provider it attempts to format it rather than return null because it was invalid.
A very small nit: to dedup the StripeObject usage, I'd turn the else: into its own "if not klass:" declaration.
>>> i = u'☃'; ' %s' % str(i)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
UnicodeEncodeError: 'ascii' codec can't encode character u'\u2603' in position 0: ordinal not in range(128)
I want them to see how the internals of this class look - how `buildForm`, `buildView`, and `getDefaultOptions` all work together. You just can't get that from the API.
Don't parenthesize the return expression.
I don't understand these 3 lines of comments.
Perhaps. I have no strong feelings either way, so I'll leave it up to Arthur.
The regex wasn't working for me earlier. When saving tax we save to 4 dp (in case we need higher precision) I like to trim off the trailing zeros- thas wasn't working with this particular regex, but it was with the rtrim.
Author : Angelica Solis
See the line below. Try `Object.new.inspect` and see how the output looks like. Then, in your Rails application, try `LineItem.inspect`. See what that looks like.
Perhaps this is true, but the task doesn't exist with the deploy.rb and play_recipes.rb that this module gives. This patch SHOULD make "cap play:logs" work out of the box. Otherwise, more changes are needed.

Can you check on your apps that use capistrano? Can you see if "cap play:logs" works?
The said bit is actually inside the comment :P
Here's a commit comment: shouldn't this be the third version, since you updated README?
It shows the prompt:

    ~ 

Instead of the expected prompt:

    [ruby-1.8.7] ~ 

Using the alternative test for rvm-prompt's existence fixed it.
Now the block with MODX version number hidden, but will be visible in an upcoming update.
Hi subbu
As i was embedding the one person's UID and other persons Biometric It
should generate the erorr code as 300 but I am getting the error code as
999 may I know the reason please and Do let me know whether you will
able to generate the erorr-300 with the wrong biometric detail of the
other persons.

Thanking you
Madhu
72 hours late.
As usual: Backtrace or it didn't happen.
us say GM, crash on paladin with swap http://www.wowhead.com/item=21177
Okay, i test.
Checked out , can t find missing sql :S
Could you upload the DB u are using matching with lasted core on http://filebeam.com/ :D
With all my consideration :D
Base reef controller. Functions normally
By the way it would be useful to me if we could find some tool that shows diffs nicely like this but before I push.  Because I find that things don't stand out as well on the command line diff and I accidentally commit something which should have been removed, that I could have easily noticed using GitHub's diff.
Is this also implementing more than 9 levels in the levelselect menu ?
I opened an issue for the related problem where languages are mixed #293
Also:

We can allow get(String) for parametrized things as well, if someone wants to, and move it into Texts when we feel like it

We can also allow wrapper functions for non-paramterized things, if we feel like it

Any style will work, depending on what people want to do.
I think we should make it a Set instead of a List, to weed out duplicates.

I realize they'll be weeded out in the for loop, but why spin when we can eliminate them here?
yeah there was 3 spells of likeness and the rest were not working http://wowinstant.com/spell;546
but yer right that spell is gylph...derp
CanCastWhileWalking(uint32 spell_id);; invalid with out a spellid atm... /shrug 
i knew it looked wrong lol, 
should really be empty but.. Flamzards pre-fix for WaterWalk kinda slowed me down...
and i got impatient :S I'll clean it up later today, 
I've got more aura fixes to do anyways.. ;)
This is really sad!
This plugin was the only one that works reliably for us.
Thanks for making it!
I don't have an environment where I can test this, though I would suggest doing something like PS1='$(cat /etc/debian_chroot) etc\$'.. At least afaik the $debian_chroot is just gotten from /etc/debian_chroot.. What's the prompt when you just use PS1='...${debian_chroot:+($debian_chroot)}...'?
This patch also involved the addition of the three SSL configuration details in database.yml which weren't included due to gitignore, which are

``` yaml
sslca: ca.pem
sslcert: cert.pem
sslkey: key.pem
```

Testing works fine with or without these configuration details present, as should be expected.
https://github.com/jsiei97/FunTechCortexMX_gcc/commit/f3d192ce9b18b5129858c767438a6d94ea5bf75d
damn, how to click "nein"
Yes, you are right. Is it possible you test the new version?
Thank you
Arr, sorry, copy paste problem.... will fix (will be the same for the others)
I can't reproduce that.  If I take out the ``outer_loop`` variable I still get the alternating rows.  I'm using Jinja2 2.6, but maybe some versions have a bug?
Added in try catch here.  ran into a case where the timeout threw, and the subscribe model failed.
P500 FM Radio Bug: Sound is noisy

Open fm radio and listen for 5 minutes. After that the sound is noisy and after 10 minutes, just noise, no fm sound.
You changed the include directory to be a link to the K: drive instead of to ../Havok (a relative path). This will break compilation for anyone who tries to compile this later. Also you removed compiling Main.cpp and replaced it with "..\Game\main559.cpp" whatever that is. I'm gonna change some files around and push to master and you can take a look
I mean if you could take a look at the implemented ones (have sonme tests, etc.); I will deal with Notepad++ soon.

On Feb 27, 2012, at 23:02, Trey Hunner<reply@reply.github.com> wrote:

> Great.  I think using negative indentation to denote `indent_size = tab` should not be an issue.
> 
> I can look at Notepad++ within the next couple days.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/editorconfig/editorconfig-geany/commit/766a2581d83c95cfc5f5da6761abef79f71037dd#commitcomment-1015271
Ooops. Menade självklart dummy.pdf
I still do. Just trying to help you out :P
@pyrotechnick What's broken??
OK. I have just added the `Repo.commit(sha)` method.
I don't think this is actually needed.  We override `User.save()` to do this anyways.
Does it work for you without?

(I realize it's also there in the docs example right now.)
can you give me info about this? :)
On Tue, Feb 28, 2012 at 21:24, Michael Kaply
<reply@reply.github.com>
wrote:
> Sorry, I'm confused. Are you unhappy it was removed?
>
> I didn't remove it.
>
> But it should be removed.
>
> Kai removed it for a reason. We'll need to talk to him

In 49fa6bfbb955b5a9106d6430efed857c4ea08c4d this works fine and using
C-l does nothing.

-- 
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br
an http-request
```array('string', '_pattern' => '\d+x\d+')``` will make sure the input is in the correct format
hmm, interesting. I think yashwantsingh or edowar might know over at the huawei ideos x6 xda-forums.
Why not using a message factory? We do like that in other packages. Here Thomas didn't want to use _() so i18ndude doesn't extract this string for the atreferencebrowserwidget domain. You don't like PMF name? Maybe _p() is better?
I don't think Message('Home', domain='plone') is more readable, in the contrary. When you read the code, what is interesting is the message, not how it is internationalized.
does wlan tether work with this?

I had to set all eth0 to wlan0 in source to fix tether
Sure, that makes sense.
Could you point us out any code where this warning has been used so we can follow the same pattern?
Damn, it's already been a long week.. You're right :(
"showInProperty" might be a better name than "isShownInProperty", which mean's it has already be shown in property. showInProperty means it should be shown in property
Why deleting these lines?
Not your line, but maybe clean that up as well? :)
Mmm, I see your point. Honestly that class grew in an ad-hoc way to match what was required here. I don't know if there is something out there that we could use, but my feeling is that probably it will be something bigger that what we actually need.
I am unclear on where these exceptions are actually thrown. Could you point to the right spot please? I cant find the throw clause. 

Also, on a related note, how can an adopter customize the handling of states? If I want to not handle `InvalidLoginTimeException` and want that to be treated as a normal error, how can I easily customize this? 

I think it might be better if we inject these classes via the configuration bean. The message would then be mapped internally for each exception class, and allows one to serve new extension points via the config for new exceptions. I think we should take away from CAS the responsibility of knowing and handling these exceptions beforehand, and let the config extend that for us.
Thanks for the advice Misagh.

I'm a great supporter of using final keyword for class or method as it clearly means : "do not inherit or override",  and forbids bad usages.

For parameters, when using the final keyword, you cannot change the reference of the object, but you can still use the setters and change the value of the object. So I find it pretty useless and verbose for parameters.

That said, I feel alone on this point, I can add (let Eclipse add) the missing finals...
Isn't there a stray `"` in the line `10, TimeUnit.SECONDS)"`?
One day the king has to step down from the throne, just making sure that there'll be a new king :-)
hmm, I liked how you can run this with `sbt akka-contrib/run`, will see whether that can be done otherwise
wow, that one is ugly, but I'll change
Ah, right, toSeq will be the right thing.
this wasn't configurable, only with api, didn't thought it was important to have it as config, but now I have moved it to router config, and removed this constant
The last two sentences here are really valuable IMO.
Is "modules" the term we're using? I thought it was "units"
I think I want to deal with that issue more generally and as a different change completely.  What I have in mind is a caching refactor, and I guess it is just a continuation of the stuff I already promised I would split off.

This is about fetch and particularly fetch by id.  I think we should change so that we have just the key based cache that uses memcache (shared and external) and that the id to key caching should be strictly local and represent an inventory of the cards this process has cached.  New cards would be a little bit of a special case, but I don't see any deep problems with the idea.
@opdenkamp If no subtitle streams disappear, then we do not need this.
Actualizar comentarios, estos estan direccionados a Nacionalidad
Nitpicking, s/z/zygote/
Great docs!
Thank you for your reply.
I also consider same property ("toggle") for this issue, but it I choose that way.

You decide to add ToggleIconButton kind. That can be good for all. Thanks :)
I guess I learn something new every day! :camel:
There should be no `static` specifier here. It should just be 

    NSString *const TUITextRendererDidBecomeFirstResponder = @"TUITextRendererDidBecomeFirstResponder";
s/useing/using/
Makes sense.  I was actually mostly concerned about consistency, since you're hiding the IntPtr in your wrapper of the bind/unbind functions.
Please address TODO
I took a stab in 484f876, I wasn't going to put `_tuple_of_integers` inside there because it makes  the error-handling more complicated (another level of nesting to bubble through). Also, potentially more code could reuse this that doesn't depend on it being a tuple of all integers, but just a tuple of the right length (e.g. some dimensions being sliced, etc.)
I have had a closer look at this and where the function is used. I assume that for iteration speed the ordering of 1-dim axes does not matter (if it does it may be a point)?

Which means that it is only important in array creation with keeporder and reduce result allocation. I do not really think that there is any need for extra clean up with either of those cases. If the user gives in a badly strided array, he should expect nothing else. So maybe this could already be pulled into 1.7.? The 1.6 behavior seems to not clean up strides as well for either of the two cases. Unless there is some new functionality that really relies on this, but I somewhat doubt it.

I could do a PR against 1.7. with this especially since not cleaning seems better then failing in probably as many cases as it would clean up and its a pretty small change.
@megies they're replaced automatically (y)
please don't,   only change the codes related to the fix. thanks
It just seems more clear to me, but I'll make the change, thanks!
Do you think that we can extract these conditionals to a method with a good name? This will make easier to understand.
1) `newTypeName("")` => `tpnme.EMPTY`
2) `newTypeName("$anon")` => `tpnme.ANON_CLASS_NAME`
3) `newTermName("<init>")` => `nme.CONSTRUCTOR`
4) `newTermName("getClass")` => `sn.GetClass`
5) `Modifiers(NoFlags, newTypeName(""), List())` => `NoMods`
6) `Modifiers(FINAL, newTypeName(""), List())` => `Modifiers(FINAL)`
7) `build.emptyValDef` => `emptyValDef`
8) `build.Ident(build.selectType(rootMirror.staticPackage("scala").asModule.moduleClass, "AnyRef")` => `Ident(AnyRefClass)`
9) Also I think that generating an empty ctor DefDef is unnecessary, and subsequent typechecking will insert one automatically if not present.
Doesn't Try.get throw the exception? if so, you can avoid this patmat
Is this the right solution?
Or, even better, I think this would avoid leaks as well - change line 68 to:

    tl.last0 = if (tl.len == 0) tl.first0 else last0
This is a matter of taste, but probably it'd look more concise as `val treeInfo.Applied(typedFun @ Select(New(annTpt), _), _, _) = typedFunPart`.
Not being bound by the collections, I'd rather this had the signature

  final def contains[A1 >: A](elem: A): Boolean

The difference is invisible at the user level but significant internally.  With an Any parameter it won't infer anything; with a bounded type parameter it will infer a type, the actual type of the parameter, which can then be checked.

Also, I don't think it particularly benefits from inlining.
why different behaviour when the check type is metric? let's just keep it consistent, and leave the default of handle_flapping: false
I would probably `.strip` this too.
As per my comment in the PR, the commenting is inconsistent. I wanted to know whether Square use Javadoc to generate documentation and whether those docs are ever really used? The Google C/C++ commenting guide does not use comment markup, but we could use Doxygen for C/C++ (http://www.stack.nl/~dimitri/doxygen/docblocks.html) if it's considered useful. I can standardize the comments based on that decision.
Do you need ``is_Integer``, ``is_Float`` and ``is_Rational`` if you use ``is_Number``?
It should be '供大家参考'
Yes. You can see it also isn't used with standard `Expr`s. If we need to change it in the future it's right here.
I totally agree tha LT does a better job than the dominant for sure here. But I am not able to see how can I use LT to compute something that has been done in finite_diff in my pull request, as pointed to me by asmurer in irc.
Also I have an idea that dominant() can be used in applications where we need to find the order of growth of functions as in theta notations when we study algorithms. Here we need to, in addition to return the leading term, drop the leading constant. That was in my mind when I was thinking about dominant, but for kauers, LT will work fine. Waiting for opinions and input
OK, got it now.
I agree.  This makes more sense either in utilities somewhere or in compabitility (though the term "compatibility" has become rather broad in this context).
Done. Thought I had done it... :$
ability → feature
I tend to be a fan of docstrings if you want to be explicit about what's being tested; class/function names are just designed to somewhat-meaningful tokens.
This just includes a fix to help us debug in development, I could switch it back to the 0.2.9 version proper if you like.
I'm not talking about the whitespace, I'm talking about the whole module :)
I think we should simulate a `function_clause` error instead of throwing our own custom exception here. Creating an expectation that only accepts `[1001]` as arguments should be considered to be faking a function equivalent to `f(1001) -> ...`.
sample usage

    def post_file(method, file_field, file, params_field, params = {})
      request_hash = {}
      request_hash[:headers] = common_headers.merge(@headers)
      request_hash[:body] = {params_field => params.to_json, file_field => file}

      @rest.post_file(url + method, request_hash)
    end

      post_file("projects/#{@project_id}/codes", :file, File.new(file, 'rb'), :data, {:name => name, :runtime => runtime, :file_name => runner}.merge(options))
I can see why you wanted to put this in an array, but this is really obtuse and not really worth it, since you would only need to copy 2 HANDLEs when you had just used named struct members.
Is this the href you're setting on line 52?
Filed #8 for this
the terminology "getReservations" sounds really weird to me.  I just want to make sure we are talking about provisioned instances and not necessarily "reserved instances", right?
Remove "record".
tags tags?
Should be phpbb_user
Can these tests be (easily) moved into their own methods?
Either assertion or a comment. Currently there is no indication in the tree that those particular tags should remain uppercased.
should be "modify" instead of "alter" as we use that (modify) several times
No, the button that was named "full_editor" is now named "preview". So we don't need to look for "full_editor" because it will never be sent.
<code>&amp;#133;</code> entity could be considered as a string append.
hmm. error() is fatal.... Think we should log and return?
Vrem o linie liberă între metode. :)
Observed on ATI driver: log line contains two sequential newline characters in the end of a string which makes log printing ugly.
This is due to a nasty YUI dependency collision since Mojito already has it installed. I'd like to work with @proverma on this to figure out what is going on. Please note that developers that want to run the tests locally may have to do this same step.
OK, it should maybe be called "Y.use" instead then.
Might be clearer to provide the event name here since the function name doesn't look like a handler, i.e. it's not "on{eventName}"
Ah yes.. this little puppy. Not sure this is a good idea. A collection does not need to be an `IList`, I know you need it to Add elements, but there is a different way. Let me try and figure out how it was done.
https://raxaemr.atlassian.net/browse/RAXAJSS-370
I just realized I changed this copyright here. I actually do not know the best way to handle this in an Open Source world. Keep it the same and undo this?
typo. Are these still TODOs?
mention wildcards here too?
it shouldn't
Good point. We should make an attribute in the SparseVariable class to return directly the indices, data and indptr as we do for shape.
Is there a way to sanity check the length, pointer, etc.? Or is that done elsewhere?
Missing test case where decode contains a ? query parameter
OMG what kind of piece of art is this? Is this part of the zip call before it?
Filter it in the view.
IF CI doesn't exists, gracefully skipping
Yeah, the large warning seems to be the way to go.

There are some things we could improve relatively easily for 1.5 in the beta stage. It should be possible to prevent alteration of related models by a flag in the model's Meta class, and it should be possible to create a model class without it triggering the meta.new() code (that is, the actual model creation). This way, we could do cloned_model = type(...), temp_cache.load_model(cloned_model). That is, make the loading explicit, not rely on the global loading.cache alteration.

I can work on this in the beta stages if needed.

While the code isn't too beautiful right now, if we want this into 1.5 it is more or less what we need to go with.
Oops, yes I meant it to be a instance level variable.
Missing space between if and open parenthesis
you should keep ``integer`` which is used everywhere IMO.
Whoops, I'm stupid. I meant `[^"]`.
>> It will fail silently for cross-domain requests to non-CORS compliant servers is a big issue) but it certainly isn't helper material, not in the slightest.
>
>**+1**. The former in particular is a problem right now, and might even be big enough to drop this crap completely.

Well, we could something along the line of what we do in the ajax unit test related to cross-domain detection: use a `beforeSend` handler.

```js
beforeSend: function() {
    if ( this.crossDomain ) {
        throw "cross-domain not supported";
    }
}
```

But we'd probably need to add a context option in case it is defined inside `ajaxSettings`. You gotta love `ajaxSettings` :(

Another possibility to get around all the problems we have with those global options could be to add yet another option, let's call it ´settings´ that, if put to fall, doesn't import any option from ajaxSettings. This would mean no converter either but I doubt we care here.

Default options suck.
Actually, in our code, we never call this function as a non-aggregate function. We add this check to ensure the correctness even if it is called as a non-aggregate function.
Questionable change
Can this be called OAuth1?
Cleanest way I could think of to cause faster rewind on decrease rate in DVDs instead of slowing down as it is with normal files.
The example programs can stay. But I don't see any advantage in splitting this guide away from the others and there are disadvantages to doing it. Right now a user can go to [wiki pages](https://github.com/praw-dev/praw/wiki/_pages) and find an alphabetical listing of all wiki pages, except this one. It's much easier to make trivial changes to the wikies, such as correcting spelling mistakes and add links to other articles. The logs are clean, with the regular log only containing code changes and the wiki log only containing documentation changes.

I know there isn't an overwhelming amount of documentation right now, but it is steadily increasing and now is the time to set up a good consistent structure for the documentation.
Generally, you should put classes like this in their own module
"the message id for the **plural** translation" i think should be the text.
Removed map and filter form the bad-functions
cool, it would be good to keep some kind of progress indication if you're going to fix the code for streaming. i liked that feature.
I'm on a vacation, so sorry for reply this late. Is there any good way to get current dir of the buffer file?
Um... this was kind of needed for me to debug a case which was going to be entirely opaque if I didn't see it as wrapped.  Any suggestions about how to do better?  toString() is really supposed to be a debugging-focused method. 

Are we calling toString() in a normal production context in a way that I'm not noticing such that this would be a performance drain?
Please keep both the Java bindings and the Processing bindings disabled by default. You can enable them with the CMake GUI or with "ccmake".
the real function is called "random_alphanumeric_bytes" and actually returns random bytes.

Since I can't unit test randomness much, I just have a fake function.

I added a comment and made the function less silly looking.
Typo? In использую, I think ‘т’ at the end is missing.
I see you have split the attributes by global and non global, a comment above each section would help other devs realize this structure
What about comments inline with values?:

```
.foo {
   font-family: serif /*, sans-serif*/;
}
```
Copy the Adobe copyright header comment from any other .js file and paste it at the top.
Remove `We'll issue a fix for this shortly.`
This is kind of an ambiguous name. Could you also mark the private functions with @private? It's a bit hard to tell which ones are intended for use by the DygraphOptions class and which are part of the "public" API.
This is to get rid of chunky whitespace that was making the test suite.xml annoying.
with the first check i think the second default is unnecessary (can be replaced with len(data["licences"]) > 0 )
For a continuous scale, limits should be a numeric vector of length 2
The good way is to use the EditorManagor and request Editors like the TOC do in https://github.com/irstv/orbisgis/blob/feature/monomap/orbisgis-view/src/main/java/org/orbisgis/view/toc/Toc.java#L985
The region-list could instead be generated dynamically via javascript by calling /nav.
You should also remove *Warning Regarding Database Backend*?
No, this should not into plain template. Make your own template for search results and extend plain there, adding what you need.
Why do you strip the login button of its dignity? I know it's disappearing, so isn't this line redundant?
This can still fail if I pass in a zoom of "a", where `+"a" === NaN`.  You probably want to use the default zoom if it's totally bogus.
Why not just set `elem.style.display  = ""`
I'm assuming this is for the enter key? Can you add a comment above just to be clear.
This sounds overly dire.  I don't think it's malicious to frame-bust, but it is annoying.  I'd recommend being less technical, and simply indicate that we couldn't display the page, but you can see it at this link.
yeah, I just checked, and this can fail when the epg table couldn't be loaded or created, and it's a two line change to add the message, so add one please.
Also applies to the rest of the names. :)
Fails on Python 3 - why should we call `decode` in the first place?
I tossed all of this in favor of pushing the problem down to Flog itself. It should have been smarter.
I'll change these to just log an error instead, actually with the second two checks I  can fix the values.  If there are any invalid keys, we'll end up with Solr searches failing on them, but I suppose that's better than crashing Collins.
zvol_id fails on hardened GCC profiles removing the "@CFLAGS@" variable allows for compiling zvol_id without stack protection.

Code still needs to be cleaned up with Valgrind but I haven't had time to do this yet.
Would be great to describe params of method in belongsTo jsdoc section, with usage examples in markdown format. This is for jsdoc.info, as I can see in GA, a some users use this docs to learn jugglingdb API: http://jsdoc.info/1602/jugglingdb
Am I missing something here or should there be a `start::Number`?
I see the point now, extra parameter. Fixing.