labofsahil/github-event-dataset-2012 huggingface dataset:
It should be noted that `\s` is "any whitespace character" not just spaces.  If you want to add just spaces simply include a space in the character class.  See: [PHP: Escape Sequences](http://php.net/manual/en/regexp.reference.escape.php)

Also note, when including a `-` in a character class it should always be the last (or first) character, otherwise it indicates a range.  I don't think in this case that it would range between `_` and `\s`, but for clarity and consistency it's best to keep the dash until last.


Having said all that, I'm not sure I'd want a space in these rules.  They would no longer do what they say "on the tin".

Perhaps we could add some new string rules like...

```
function validate_word($attribute, $value)
{
    // A "word" character is any letter or digit or the underscore character, that is, 
    // any character which can be part of a Perl "word". The definition of letters
    // and digits is controlled by PCRE's character tables, and may vary if locale-
    // specific matching is taking place. For example, in the "fr" (French) locale, 
    // some character codes greater than 128 are used for accented letters, and 
    // these are matched by \w.
    return preg_match('/^\w+$/', $value);
}

function validate_words($attribute, $value)
{
    return preg_match('/^[\w ]+$/', $value);
}
```
@Tobion only when you sent wrong data and got a PDOException. But this should not break the profiler. However, it is an edge case so it is not really useful to display more details
Then that's a problem. Using the default directives isn't correct, we need the directives that are configured for the given context of the specific function that is being processed.
and add an else { ++it; } here
is there a reason why TO< Player* > was changed to TO_PLAYER ?
The Filter class is already loaded via Cache.php, was the plan to remove this later down the line or something?
Append `cluster` to line 3
and you do it again ...
Oh, and Pathogen to manage them all, obviously.
It wasn't part of the public API, so I moved that into the C files that use them.  (Type definitions in src/libipset/bdd/internal.h, function definitions in src/libipset/bdd/basics.c, binary-operators.c, and trinary-operators.c.)
Same problem as the previous line 87 and 142 grails-app/views/planning/editPlanningEntry.gsp
The removal of this line breaks endpoint maanger
to be clear, this does not fix the bigger issue document as the gist in https://issues.scala-lang.org/browse/SI-5189?focusedCommentId=56411
working on that...
For what reason was the pull request reverted?
This should really be called specfun. specfun is a commonly used term on netlib, in matlab, octave etc. whereas special is sort of a more generic    term.Of course, the name "special functions" is a bit strange, and there could surely be a better name for this class of functions.
This should be a Duration
site offline ? : S
will make that change in the next version :)
Is this pull request in any release? It is urgent as users cannot follow the instructions for deleting their files...
Try adding `-Wl,-rpath -Wl,$(DESTDIR)/usr/share/julia/lib` to the command that links the executable. Of course this means `DESTDIR` has to be available during build and not just install. We can also add an rpath for `current_dir/lib`. And probably a different command is needed on Mac OS.
Hey James ! Thanks for using/modifying my dotfiles. I'm pulling a few things back in my fork (e.g. pathogen fix).

For export stuff you like PYTHONPATH, you may want to look at the dotfiles/sh/.sh.d folder, these get automatically sourced if you install "sh" with ```cd dotfiles/sh && ./install.sh```. It's then pretty easy to enable/disable and sort env var related stuff.

HTH
Can I suggest `protected` or even `private` for this to discourage people from trying to access `URL::$base` before it's ready - or even modifying it?  Unless you want people to be able to do this?
Dame un sec y fixeo el problema. Lo de RSA funciona ;-)
use rebase instead of merge for updating in this cases
hi,
security.lib.php is already load by default for all application
it's not necessary !
thanks
It is vestigal. Just removed it.
I'm having a hard time understanding these. It looks like three different checks against the size of the TPM's blob.  Are there testcases for these?  Maybe some comments would help.
Can you think of any downside to having celery as a mandatory dependency? Does it limit what http servers MyTardis runs on, for instance?

Other than that, tested and works fine. Merge can happen after you get back to me :)
Yeah I kinda derped, I know how to set templates somehow I managed not  to delete the default :P
Ok for me, but could you find out which nzbmatrix categories should be added to the query? I suspect we need extra values in the subcat value in the following code fragment:

def _doSearch(self, curString, quotes=False, show=None):

        term =  re.sub('[\.\-]', ' ', curString).encode('utf-8')
        if quotes:
            term = "\""+term+"\""

        params = {"term": term,
                  "age": sickbeard.USENET_RETENTION,
                  "page": "download",
                  "username": sickbeard.NZBMATRIX_USERNAME,
                  "apikey": sickbeard.NZBMATRIX_APIKEY,
                  "subcat": "6,41",
                  "english": 1,
                  "ssl": 1,
                  "scenename": 1}
After testing that it's the same as setting the value to ''

On Thu, Mar 8, 2012 at 4:31 PM, Maurizio Domba <
reply@reply.github.com
> wrote:

> What happens if someone sets this to false ?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ronaldfenner/yii/commit/b03e2d04f4a4ff9d1467ef2c186a4f267afb957f#commitcomment-1063227
>
Man beachte die Reihenfolge :)
In zwei von zwei manipulationsknoten wurde dieser Fehler gemacht, vllt sollten wir wirklich mit RBGs arbeiten
It could. If that's a general lithium standards preference it can be changed no problem.
Could be `<meta charset=utf-8>` if you want it to.
Hum, ok
But we still need a Directions Page

Otherwise, you loose a big functionality which is to list all the steps in one page.
I reverted that one 87b7eb3345ad33a2

Run py.test!
> include the "::" as part of the constant strlen("eval::") so the number at the end is less magical

I'm happy to change this, but I did it the way it is to mirror the code in the existing hp_compile_file() function (line 1714). Let me know whether you would prefer consistency or magic-reduction.
I don't understand why you have added a 0?
source.GetWidth() -> src_span
We don't like patches from gists very much.
Looks good, but I'd like to follow Linux kernel style and put the if (condition) do-something on separate lines. If you do that, I'd be happy to merge.
This seems to be referencing another theme within the Themes folder (themes/ghp). Is this required for something else to function?
remember to change the constructor name to Projectile (instead of ProjectileBirds) :)
yeeeeyip. that breaks about half of lantern. heh.  Also it double counts.
My god...did a test but forgot to change it back...thanks to git..
Howdy @spraints --  Why did you change module/exports/require to be undefined for the eval'd code?  I'm trying to require the 'sandbox' module inside of the eval'd code, but can't because require is undefined per this change.  Thanks!
Rest looks fine
Why cant you use self.fileNameDisplay?
This is confusing and unnecessarily different from generatePMLPage. Please use the appendIndent here as well.
Document/Comment the CategroyService
Excellent, I should have said '''AbcBuilder''', sorry :)

If you look at the output of "associations", and you search for a proxy_method that includes "community".  You get something like this:

    #<SugarCRM::Namespace0::Association @proxy_methods=[communities_builder], @link_field="communities_builder", @target=false, @owner=SugarCRM::Namespace0::AbcBuilder, @cardinality=:many_to_one>

'''@target=false''', means that the gem doesn't know which module is associated with this class.  This is probably because it wasn't provided in the module metadata we received from a '''get_module_fields''' call (we look for the module name in the "module" or "bean_name" entry of the returned metadata).  

Can you run:

    SugarCRM.connection.get_module_fields("AbcBuilders")

And look for a line like:

    "rt_communities"=>
      {"name"=>"communities_builders",
       "type"=>"link",
       "relationship"=>"communities_builders",
       "module"=>"",
       "bean_name"=>""},

Let me know if "module" and or "bean_name" are empty.  If so, try editing the vardefs and adding the appropriate module name.
Не забыл. Просто еще не добавил...
Nice commit! I love it when we better encapsulate stuff :)
Because session.invalidate() destroys the session, but doesn't inform the client that the old session is no longer valid. This means that the next request is going to come with the old session identifier, and if the browser is going to issue multiple requests in parallel (in the name of performance), it will actually create several sessions.

Plus, a logout action always redirects to a /view/ action, so a new session is going to be created anyway, this is just an optimization.
ye, I also think so, we will correct it, thanks
I mean, I still debate that if the *only* time you're going to call `git_threads_shutdown` is right before exit, it's kind of like worrying whether your house is clean before the construction company shows up to tear it down
This is throwing a `NullReferenceException` in the case when you create a brand new repository. Basically "git init ." and then check this property on the master branch.
No need to test $mod0 when you just set its value to a literal.
The description for this commit is wrong. I simply removed some debugging comments from the file is all. This is what happens when you hit the wrong button, lol.
Sure, will do. Thank you.
Figured as much ;-)

On 16 February 2012 22:16, Andrew Dodd <
reply@reply.github.com
> wrote:

> Change this to false once we get the image sizing right :p
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/andrewdodd13/hwkinect/commit/565380feba8a87394d9ac2dfef321112c0a14b55#commitcomment-975610
>
+1
Thanks
In general, dead/commented code should be removed.
No need to support ancient versions of Android on this project.
I'm loving the location series you're putting together. I hadn't really thought about introducing photographic images in the badges. I think this will be perfect to get people thinking. I'm just a little upset you didn't do ones for South Korea and Ireland (just kidding!) :P

If you're stuck on more ideas, you might be to do specific events. Dale is planning on white labeling the iPhone app and mentioned a few events like TED, Lift, SXSW, PopTech, ICCM, Maker Faire Africa, Pivot25, ITU, etc.
+∞
мне кажется, лучше взять два аргумента со стека тут, а не внутри каждой функции. а внутрь функций передавать аргументы (или вообще не использовать эти функции, а тут прямо все считать). а то этот код 4 раза приходится раскопировать.
The intention of this RE is not clear to me; a comment or assigning it to a specific variable name would clarify.  

It's also not clear to me why you've separated single digits and multiple digits.  It looks like you're trying to exclude "page=1", but "page=01" becomes possible, which is a bit odd.  (Though, of course, unlikely to come up.)  What's the reason for excluding "page=1", however?  I'm not sure in this context.

Also, it's conventional to use \d for [0-9], for what it's worth.

I'm also *slightly* concerned about the .* at the end; what's the purpose of that?  In a whitelist, I wouldn't expect to see many /.*/s at all.  :) 

Lastly, while (again) %r{} is certainly valid, I'm not sure why you avoided //...

All this said, I would expect to see here:

```ruby
@rel_canonical_query_pairs_whitelist = [/page=\d+/]
```
(I've not added a comment because the simpler RE is, IMO, clear enough.)

But I assume I'm just missing something.
I thought about that first, but then kind of thought that most people would want it to just be "turned on" with a click, and don't have to register at addthis, so that is the reason for the simple solution. That being said, i think you are right, it should be available at least as an option that is not required. I'll add it tomorrow.
uh, yeah lets make those imports not insane... please? ( BaseModel, UserMixin ... )
Please have `# :nodoc:` at the start of the docstring on line 228, and do the same for all other nodocs.
Not yet... planning on opening one today.
yes, other contributors might be quite displeased to see their names replaced. so it's important to find a good workflow moving forward.

what conflicts did you get when merging? it was rebased on your latest master.

http://help.github.com/send-pull-requests/
https://github.com/blog/843-the-merge-button
Kandera , great job , but spell_dru_starfall seems have no effect since the conversion :s
Why am I not surprised, Expecto...

You didn't apply all the sqls in correct order.
Idk if thats cata bullshit but you can cleary see in http://warcraftmovies.com/movieview.php?id=155295 that 1. Channeled spells will still fire their missiles even though target runs LoS and if you check @ 15:42 when the priest starts mana burning and the mage runs LoS the cast continues. LoS is only checked at start & end of cast.
Для JS и CSS существует правило: сначала грузить сторонние библиотеки, затем кастомные. Причина - кто последний тот и прав (стили/методы определенные в кастомных файлах должны переписать (конечно если такие есть) стили/методы из сторонних библиотек).
поэтому должно быть "nivo-slider", "default", "lightbox", "application", "style" (я думаю style.css это твой файл)
This change is just absurd.
Ensimmäinen query vaatii foreach-loopin tallennusta varten, sitä ei voi tehdä yhdellä queryllä ja antaa kannan hoitaa loput. Se myös rikkoo tällä hetkellä pyörimässä olevat frontendit. En edes halua kokeilla paljonko toinen rikkoo, joka pitäisi varmaankin tehdä myös foreach-loopilla.
Not quite.  Sorry for the confusion.  AMQP has a notion of auto-ack that doesn't correspond very well to the behaviour of AUTO (which in turn is inspired by Spring JMS support), and that's what this flag is for - it means the broker assumes an ack without the consumer having to do anything, whereas AUTO means that the consumer will send an ack without the user having to do anything.
:D koszi!
use DLog - for debugging purpose and RLog in case of such errors like this one, and try to put the error message in to the log so that  we are able to understand what happend there.

More over if you don't use ARC then you need to free error.
So the code should look more or less like this
RLog(@"Unabel to track a page view /app_launch, err %@", [error userInfo]);
[error release];
error = nil;
are u really quiting scripting for good?
También en LINUX Retail != retail, por lo que todo debe darse con los nombres correctos.
CAD ?
You should be able to get away with type.name().toLowerCase() or type.toString().toLowerCase() instead of this if and the other in getData()
That is a good suggestion. I am going to start doing it right away. Thanks!
Interesting. I guess the element should be empty if the function attribute is specified.
good point, i'll update.
Well so far it consists of only one method, which is to format an integer representing cents as a price in dollars. I'll fill it out later, but there isn't really anything much to document. Look at the source in grisaille anyway.
I'm working with this one but it never runs because line 15 [default.rb above] is always false.

I updated the rds.erb in the attributes folder with the proper app_name value, tried custom names in attributes [{app_name}.erb], etc but nothing seems to work.

Are there any docs for properly retrieving app attributes? Also, my goal is to have this one recipe work for the several environments setup on EY [so site dev and production use different db's and the api dev and production use the same one's as the site, respectively].

Thanks in advance for any help here.
I could accept that you may want to assert that your MobileDefense::ScanFiles class has a file_owner method and no validations, but this file does not test that.

If you want to test that but MobileDefense::ScanFiles under test.

The same goes for ScanResponseModel
still doesn't work - requires two arguments?
bemac likes apple, your argument is invalid
This is awesome! I was trying to figure out why we didn't have a Power Menu like other devices
Haha yeah I stole this one straight from your solutions. Couldn't think of an elegant solution and you had an awesome one.
wtf is hier gebeurd :o al die changed files?
Hah, I think there is a big difference between rescuing for the sake of re-raising rather than rescuing to try to restore to a normal status.
Here's where we call the "before" hook for each host, for each test.  Again, not part of the DSL--simply giving it an opportunity to set up state for the test.  The current implementation doesn't actually do anything but help simplify the management of temp files.
I didn't realize it only returns an Iterable.  Do as you like.  You can do newArrayList(iterable)

and then do list.get(0) and list.get(1)

Whatever is fine though
In Django docs it is said *In docstrings, use "action words"* : http://readthedocs.org/docs/django/en/1.3/internals/contributing.html#coding-style
> so I don't remember what changes I made

git diff

Cha-ching!
> remove this commit. I have already applied it to master.

Done. `git rebase` is great, it managed to detect that the commit already exists and removed it from the branch automatically.
Yeah, ok.  And semc stuff is maintained by defer so if he does it... then we should too. :)
Creates the ServiceUtils and move this code into this one. Makes them more easier to reuse.
Added optional config for Minimee and Low Variables
Case in point: This form behaviour will be used in many places (AssetAdmin, SecurityAdmin, ModelAdmin at least).
So it should probably move to LeftAndMain.js, and have a selector like ".cms-filter-form".
The field serialization should be its own getter so it can be overloaded without writing the whole onsubmit() again.

Maybe we'll end up attaching onsubmit handlers based on component visibility of the list/tree/gallery view
instead, so turn around the relationship (and avoid hardcoding selectors in the onsubmit()),
but that's better left until we know what the ominous "gallery field" will look like.
Здесь можно блок `else` не писать для сокращения вложенности
Done -- sorry, did not realize that this was the case in the codebase -- my fault.
Hi Jorge,

I'm afraid I don't understand what the `process_*` function call is needed for. It's not needed for `egg`, and it's not needed for `subdirectory`, and those are the only two url-fragment options that pip understands. I don't think we need all this code (especially not code which accesses `globals()`, generally a bad sign) just because hypothetically we might someday add more url fragment options that might use it.
It may be worth optimizing this for run-time performance. Basically, the lighter we can make this impl, the less hiccups we'll have during animation. For example, I'm thinking, if there's only a single axis SV, do we need to .get() and calc both X and Y bounds. If we've determined that we're above max, do we need to explicitly check if we're below min. etc.

Same kind of thing for the helper methods invoked [ due to the fact that they're on a critical path ]

The .get() will make a noticeable difference. Not sure about the rest, and I like the readability of those - but worth evaluating.

Of course, if we're moving to a non-JS anim for flick deceleration, this point is moot, but otherwise this needs to be the tightest we can get it.
Just to make sure this is captured here, I added some thoughts to the `Y.soon()` forum thread about us possibly wanting to use one of the hacks so that callbacks will be called async, but before the next DOM repaint.
As commented before, the click event definition should be in another commit, whether or not it will be included in the same pull request as this one.
uh yes you're right, not sure how that happened
Yes, compatibility with PyDOLFIN.
You mean if you had exactly ``java version "1.6"``? That wouldn't match, but I think java -version would always have 1.6.0 for the initial version.
I think we can leave it as it is.
Instinct: I'm not sure this is the best option. It means that you need to control message implementations to be able to use them for consistent hashing. Same issue as with putting equals, hashCode etc in Object.
I propose to seed the Router with either a partial function from Any => Any or a total function from  Any => Option[Any] so that you don't need to wrap messages to be able to have them hashed.
And with my above suggestion, these validations would not be necessary: simply have defaults which do not trigger them (as you did already).
```
gtab : GradientTable
    Description of parameter goes here.
```
How about `take_step` gets replaced with just `step` (ie the verb step) to be used for stepping from a location to a new location, and `_next_step` get's  `_get_direction` to be used for picking a tracking direction.

I agree this is kind of ugly. Using the "stepper" as an object is somewhat silly because it only has a "take_step" or "step" method and nothing else. We could replace the interpolate object with an interpolate callable which would take data and a location as its arguments. Would that be less ugly, any other ideas?
Small thing but why not just import urllib?
I'd look at moving this into a helper function rather than being part of the package controller so we have more flexibility eg show on home page if we wanted.

also maybe you should have just returned a dict rather than direct, url
@seanh Yeah, this felt messy when I was writing it... but I wasn't sure how to do it better.
Now that I think of it, do I even need to call preventDefault()? If I just remove the return false lines it still works
PEP-8 says 2 blank lines between top level classes and functions.
Personal preference: "to configure Pulp to connect to", structurally, this sentence sounds like the act of editing does that connecting.
does this implementation really belong to the abstract class?
No need for this to be a pointer. A normal object will work just as well and you can't forget the call to delete in that case (which you did).
Same applies to curP and the changes in CGUIWindowVideoNav.
for utilization_class in pool.keys():

Again, try to clarify what 'util' means.
Try testing your heuristic on `0.20`. :)

I think `if v_float >= 1.0` is sufficient. Hopefully by the time they reach `1.10`, `1.1` will be a distance memory.
s/short url/tinyurl/ ?
Wait what? Att byta ut "Original author" känns helt klart fel. Lägg till dig som en additional developer eller något istället
Rename for ServiceConfigurationHandler
Obrigado pela dica! Removi a referência.
We should open a ticket about making these configurable
This does not sound right, because it's parsed as `hash + (n & 2**64)` and `n` is always `< 2**64`.
It's `ruby-osdb` bug, but nevertheless a problem.
One could simply do the `&` at the end only once, but it might allocate some memory unnecessarily, so I guess the best is to use the right precedence and maybe store the `0xffffffffffffffff` in a variable or a constant.
`$ cat a<TAB>pp.l<TAB>og`… Let's go for `tree.log`.
This also seems too specific a use case to live in `TUITableViewCell` itself.
What if there's no `alternateSelectionColor`, but there is an `alternateBackgroundColor`?
Agree, but in my defense, that was just moved from ExampleView.m :panda_face: .
Superfluous. `new()` either succeeds or throws an error.
Oh, thank for explain.. but still, there is not "slice" symbol in core? I think yes (but don't verify) :-)
semi-colons should be removed.
... do you even see whitespace changes? I really have no idea why this is diff'd.
блок-схемы хорошо бы вернуть
как-то некруто это все копировать. предлагаю сделать эту структуру в классе Node/EdgeElement полем, заполнять ее в этом методе и возвращать по ссылке
@rafaelfranca this also makes the same bad assumption the current does: that target is symbolized.  the bugz being caused can be summarized as 

"we can't know if options or url_options has had string or symbol keys written into it"
This can go inside ActionDispatch::Railtie:

https://github.com/rails/rails/blob/master/actionpack/lib/action_dispatch/railtie.rb#L22
One-liner reads well here
So does this mean I can now tell new Foo from new Foo() ?
Nice find!  What kind of random numbers are we getting here?  I'm guessing not from `/dev/random`.   Just curious for my own usages :)
I believe it should be InetAddress.getByName(null).
Especially when it's purely for the side effect.
It's already done in the adjacent pull request.
Can these three lines be replaced by ``>>> data_orig = lut(theta, phi)``? The explicit copies don't seem necessary.
This used to be called `transitions`. Same reasons for renaming the `remove` method.
Return sort key of infimum (if possible) otherwise the value of `default_sort_key` of the set.

(The phrasing now makes it sound like something is sorted.)
I think it makes sense to split this function up, according to the subtests you delineated by comments.
This used to work in master until very recently.  Have we changed how MatrixExprs and MutableMatrices interact?
I would remove the import if it's not necessary.
I'm not sure but it looks like this does not match the definition in the docstring: shouldn't `(-1)**m` come in front of `Ynm(n, m, th, ph)`?
@asmeurer, @smichr, I may be wrong, but this mapping seems like it can be useful in other printers. And I think that the pprinter in unicode already does something like this. Thus, do you think that it makes sense to have this higher up in the subclass hierarchy? Whom should I contact for more information about the printer module?

Anyway, even if it is appropriate to move this somewhere else I think that we can do it later in another pull request.
Frequencies is defined just above glom. We could make it take in such a function. Is that effectively what `counts` is below?
Used showElement.
Hmm… what if we'll leave `registerMetaTag` as a deprecated synonymus to the new name?
I let isset stay there, as it will build an valid sql-string, even if `limit` is empty(Well, empty resultset, but still valid atleast on mysql)

```sql
LIMIT 0
```
Well, i know there is no syntax error here. 

But indentation like this is not recommended anyway, for more information:

http://vimeo.com/25606006

or the text version:

https://gist.github.com/1205383
Are there "tabs" at the beginning of these comment lines?
Thanks Soroosh, there's still just this extra unused variable and then it all looks good.
Where did 375 come from?  I will not sign off unless this is 376, or it is explained.
I suspect there's probably value in formatting these over more lines so that the delta is more readily apparent.
I'm going to help find my own typo's :)
`callbacs` => `callback`
@tillsc: I have no idea what that's supposed to do.
I think we should allow nil values.
actually, we have a 

  .. stats:: 

directive for this kind of thing
Yeah, I'm a big mock hater, but having tests that at least ensure things are wired up correctly that can run on Unix provide a first line of defense against accidentally breaking functionality in an unrelated change.
Put assets/javascript/plupload into includejs also.
Should be "if (file_exists(..."
This will be set to the service container name of the avatar driver by symfony. If you change you'll be screwing up some things.
You've already said this?
Yes, IF the lock is released. Hopefully you are not forgetting to release locks on apache by, say, having exceptions raised on you between acquisition and release.
Are these two lines commented out for a reason?
This sentence and the second sentence in the next doc block are both copy/pasted from the tidy_cache.php cron task and are not relevant to this one.
make this all on separate lines, this is a bit long for a one-line if
you can condense this code to be less verbose- something like

previousDelay = self._delays[datacenter] || consts.DATACENTER_FIRST_RECONNECT_DELAY

then you can delete the if block.
I think this should be:

```js
* @param {Boolean} [asDegrees=false]
```
this seems like a nice reusable function to have in utils or the test harness.
Not how Bukkit formats enums - look at ChatColor
Probably change in whitespace on the line
I think it's good now, sorry for the unused commit
Why 'OCVSample'?
Why is this commented out? If we don't need it, let's remove it.
Change to `return (T)value;`
create a JIRA ticket to track this issue. link the JIRA ticket in the code.
Unfortunately, this isn't in a Sencha-style architecture. Please see 
https://github.com/Raxa/Raxa-JSS/pull/14/files
for an example of ExtJS code that follows Sencha-style architecture. Note that there are no ".html" files, and that all views are created using exclusively JavaScript. "app.js" launches the application as a whole, and calls many other .js files to be controllers, models, and views. 

I recommend you walk through some tutorials online to familiarize yourself with this architecture.

Also, are you familiar with MVC? We are using this to help organize our code (separating the logic (controllers) from the display (views) from the storage/database (models+stores)). If you're not familiar with it, that's something we could talk about and I can try to suggest how you should structure your code.
Well, Requirement is not clear, so I added acronym to ID (as per their current system)
if it doesn't find it in localstorage, it will return -1 thus failing the
test.

On Sun, Jun 3, 2012 at 1:23 PM, nathanleiby <
reply@reply.github.com
> wrote:

> > +    });
> > +    it("saves in localstorage", function () {
> > +        var Form = Ext.getCmp('formPanel');
> > +        Record = Ext.create('RaxaEmr.Pharmacy.model.User');
> > +        Form.loadRecord(Record);
> > +        var drug = Ext.create('RaxaEmr.Pharmacy.model.User');
> > +        drug.set('drug', 'testdrug');
> > +        drug.set('mims', 'testdrug');
> > +        drug.set('form', 'testdrug');
> > +        drug.set('pack', 'testdrug');
> > +        drug.set('di', 'testdrug');
> > +        drug.set('streat', 'testdrug');
> > +        drug.set('dosage', 'testdrug');
> > +        mainController.launch(Form, drug.data)
> > +        var find = Ext.getStore('drugStore').find('drug', 'lol', 0,
> false, false, true);
> > +        expect(find).toBeGreaterThan(-1);
>
> Did you test this by searching for a drug which *doesnt* exist, to verify
> that it fails?
>
> Just want to make sure, since if getStore doesn't find a store, I think it
> will return NULL... which in javascript is "sort of" greater than -1.
> http://stackoverflow.com/questions/2910495/why-null-0-but-not-null-0
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Raxa/Raxa-JSS/pull/68/files#r919586
>
Ungainly. This would be a much nicer experience if the zip file was unpacked as part of the build/install process. Then this code could revert to its original, cleaner form, and the user could easily view the text file to see how it corresponds with the loading code.
Not much point adding an unused function just to raise an error saying it's not implemented! Please delete from here and all subclasses.
witch -> which

witch is sorcière :)
I think you need to swap node.inputs[0] and self._numop.__name__
Thanks, I did not think of the DebugMode case.
is.read() should return the actual number of bytes read. should use the return value for i instead of calling getLength(), which may not be accurate.
1. Don't catch DataCenter.DoesNotExist, this is an argument error. Let it raise if it fails.
Не уверен, что знание про `deps.js` (и `bemdecl.js` в техологии `deps.js`) должно быть в модули технологии, потому что декларация для `bem build` приходит в аргументах команды снаружи. Здесь должно быть только однозначное знание. Например, про получение `bemdecl.js` из `bemjson.js`.
Вегед про `{}, this.__base() || {}` писал, что первый пустой объект не имеет смысла.
It may behoove us to move towards smart pointers as our user base expands. The above points are perfectly valid for our current state, though. The one thing I'll mention is that the creating/casting step can be made easier through well defined typedefs.
You're correct - this risk does exist. I'm wondering if the right approach is to catch this as a validation issue, and reject any User model with certain required attributes.
This should be `TransactionTestCase` right?
missing space here. and please add the comma after ``$filter`` to avoid unnecessary changes in the diff when adding new elements
Need to fill out these tests before I'll merge this pull request.
This is used in iframe and it is read from iframe url request parameters. It cannot be fetched from properties file.
This doesn't work?

```
for (llvm::Intrinsic::ID id = static_cast<llvm::Intrinsic::ID>(1);
    /* everything else same as before */
```

If not, that sounds like a GCC bug.
Minor issue, but all our Python code uses 4 spaces for indentation.
GPDB doesn't apparently have LIKE option.
We do not use "AggCheckCallContext" because PG 8.4 does not define it. I just searched madlib's code, it seems we have a copy of this function in the file of GPCompatibility.cpp. In this way, we can use it safely. I will change accordingly.

But I still think our original implementation is correct. Its logic is the same as "AggCheckCallContext".
Is this "<@" operator?

```
haradh1=# select array['a','b'] <@ array['a','c'];
 ?column? 
----------
 f
(1 row)
```
Ah, yeah, OK, I definitely should have taken the misuse of req as a signal something wasn't awesome.

I see, yeah, I can't remember if I had a good reason (or what seemed like one at least) to stack the middleware like that... but definitely makes more sense to call to a impl. specific method. I'll beat it back into shape!

-Mike
This is meant to be a helpful alias. Currenlty, "rhc-sshkey show" does not exist. (If it is deemed unnecessary, I can remove this alias altogether.)
Probléme en cas de reception de plusieurs messages. Chelen a regler le probléme dans un de ses push
Yes, the spec mandates TLS (so as not to expose bearer tokens over cleartext), and so do the reddit servers. Any request to oauth.reddit.com that isn't https and doesn't contain a bearer token will be rejected.
Understood. That makes sense, as long as you're sure htmldiff is reliable at escaping!
`commandArgs()`这个问题先建个issue吧，我不太想放在这个位置，以后放在别的章节再写。
This whole ChildBrowser diff is just the upgrade, right?
All our other command ids have some sort of prefix: file.*, edit.*, etc.  Worth doing here too?
Omit curly braces
It was complaining about both - I put both back in.
Yep, same height.
I think it be helpful to also test with IP immediately following "<div>" and immediately preceeding "</div>" to verify that no hints are listed in either of those places, also.
Do these values really need to be stored in vars? It looks like they're only used once, so it might be more efficient to get them as needed.
Maybe these shouldn't be named 'loadScript', since we want to support non-scripting languages. Same for `continueScript`. Also, would it be possible to remove the APF layer here? [`dbg-node`](https://github.com/ajaxorg/cloud9/blob/3b3532e38af5c2b22a672aa7b9e3684eb79d7c67/plugins-client/ext.dbg-node/dbg-node.js#L422) doesn't do much with it anyway.
This seems like a bit of a misnomer now that this method might actually send to the master cause of a sticky table. I think maybe the dispatch branching where target_method returns what method to call is no longer necessary and should be unified to go through one, which would also help dry up the rescue and stats logic below
right, good point - not sure why my brain didn't think of has_key
Because typically when an app is in development, show_errors, and the like are turned way up to show everything.  So if you throw an exception, then the exception details would show to the people you are forbidding from seeing anything.  Also, where this is placed in the code is before the Debugger is loaded, so it is the raw exception that would be shown if not caught.  This can be easily remedied by moving it to the execute() method if that is what is decided.
I probably just forgot `xintercept` and `yintercept` - the other problem is that they don't currently work with `stat_identity`, but maybe that can go away as per my other comment.  The reason that position aesthetics are treated differently is that they should be scaled (i.e. you specify position in data coordinates), but all other aesthetics should be treated as set parameter values (e.g. `colour = "red"` shouldn't be scaled).  

That said, there were a couple of bug reports on the mailing list about annotations not working with various scales.
Seuil -> Threshold
Why span?

Is this XHTML valid?
or:

```erlang
    case sets:to_list(sets:from_list(string:tokens(V, " ,"))) of
        [Val] ->
            Val;
        _ ->
            undefined
    end
```
Might be worth checking if the file actually exists before attempting to load it, and display a graceful error message if it doesn't exist.
I take your point, however there are a couple of things that I'm uncomfortable with:

 * This currently adds a requirement for requests to any project using it (we could use urllib)
 * Instantiating BrowserID with None, None is not particularly pleasant

This can't be a property do to the fact that it takes a base URL argument.
As above, I think we should try to improve this message.
Please const.
Let's choose one convention for `_` and stick with it here. Normally, in Butter, it's still a bit goofy, but `_` usually refers to a private member variable. You probably don't need it on `updateFunction`, but if you insist on using it for all of these variables, let's put it on `trackEvent` as well.
I don't even know what this file is. It's jbuck's experiment, so I'm just not... touching it. Sound good for now?
Can you split this line up so it's not so huge.
Perhaps this `e` should reference something else. Maybe it should be passed in somehow, or at least be a const.
Two things to consider here:

1) I didn't rewrite this code, I only ported it, mostly through copy & paste.
2) While I do like optimizing, these are probably instances where it will just become more confusing to read, rather than add a whole lot of value, no?
Again, this is another combination of 2 steps:
```
When I refresh the Agi Database 
Then the state will change to available
```
Updated the code according to the implementation of --with-pcre
Tässä sama long-juttu
I'm not sure.  @ChristianKreuzberger, do you remember what this is for?
Isn't the ispath test redundant?
Old school stuff I am used to. Will remove :-)
Pupius says yes
removing the object does not make much sense IMO. It would mean that the handler is in a broken state after that
Specifically,

1. restore <a>
2. handle click of <a> by binding click listener on `'msg-conversations-list'`, and find the specific <a> (and it's `href`) with `evt.target`
3. `evt.preventDefault` for clicking in the edit mode.

P.S. `<li href>` doesn't make sense, does it?
Due to an issue with the scrollbars we need to add

`width: -moz-calc(100% + 8px);`

here
Please can you use byte *char_left_ptr, without the extra space?
So this still includes legacy BDB support? Means we need to keep 2 code-bases up to date.
What was the intention to keep it to be able to revert, just wanna know :).
Just want a yes or no here, as all other compiler flags in the pro-file are ``CXX`` ones and I want to avoid to add something we don't need here!
Seems like this exception could be avoided by just re-using the MissingConnectionException.
This docblock needs to match the new mixed string|array type, also maybe better not to have a default value for $conditions param?
Just a thought, but this may be a false economy -- you incur an extra database query just to correctly size a slice in memory. Probably not a big deal either way, though.
We can move this to VsCommon, and nuke the Vs2008 implementation. They're the same, except that this one has a catch.
While @deprecated to a Javadoc has some advantages, the most important thing is to add ```@Deprecated``` to the code. IDEs and all sorts of code inspection tools rely ```@Deprecated``` in the code, not in the Javadoc.
Yes, this is intentional grouping.  "removeMath/replaceMath" are not quite the right function name, as here we do :

1. removeMath: Search for MathJax/LaTeX math blocks/fragments, caching them as we go (and replacing their text with special markers indexing back into the array cache)
2. Perform MarkDown processing
3. replaceMath: Replace the special markers with the math blocks.  

It would be good to have a pair of function names that capture this process, but I couldn't come up with them.  At the least, it is probably worth a code comment before this block of code explaining exactly what's happening here.
Thanks, that looks good.
...grudgingly ;)
Is this `new_record?` check intentionally reversed?
Idd, return "show columns FROM {$table}";
These are redundant. `isfile` includes `exists`.
Documentation... but can't we use something more declarative than ints for errors? Enums? Exceptions?
It should never exit `block_` with `S.in_directives = true`. Even if the block contains nothing but directives, when the closing brace is encountered the flag will be unset on [L713](https://github.com/mal/UglifyJS/blob/5ae3187cdb139015ef727a5a86d6d1dd216c6d1a/lib/parse-js.js#L713).

Running it over the code below gives the expected result in both the AST and mangled output. :)

```js
function a(){
    "I am an important directive";
    function b(){
        "I'm a directive too";
    }
    "I am but a lowly string";
}
"just a string, doing my thing";
```
Is there any external script that we can put this in? Trying to avoid inline scripts.
Would you mind to use a local test page? That way we are independent from the underlying application.
forgot gettext for l10n ^
I don't think so. I see two possible cases:

1. You use "normal" IDs internally to identify your objects. In this case the URIs are only used in the communication to the client and needs to be parsed and verified when the client sends them to the client - that's when you would use `ResourceLinkVerifier`. There wouldn't be a case where the URI would come from some other source than the client.
2. You also use URIs to internally identify your objects. In this case you wouldn't need to parse them at all.
How is it possible to display this information? It is not shown for me.
Wouldn't be better to move this extension to the DependencyInjection component?
Please don't do this. The test namespace have nothing to do in the autoloader of all projects using symfony
Removing a class in a *maintenance* release ? Seriously ?
When the original filename contains spaces, `rawurlencode` will convert those to `%20`, which in turns raises an exception (see [ResponseHeaderBag](https://github.com/symfony/symfony/blob/master/src/Symfony/Component/HttpFoundation/ResponseHeaderBag.php#L244)). Maybe replacing spaces by underscores would be a good workaround?
add `[client registerHTTPOperationClass:[AFJSONRequestOperation class]]` and `[client setDefaultHeader:@"Accept" value:@"application/json"]` here and then remove the NSJSONSerialization below
I'm not sold on needing a uniform API across the three communication modules. While uniformity is great and all, we aren't taking advantage of it (e.g. pseudo-Duck Typing) as far as I can read.

I'd vote for removing it.
Is there an advantage to having this return a 'const int &' versus just an 'int'?
Memory leak!  Better delete iiw here.
Yes, I am waiting for the pull request for error printing to be merged,
will do the appropriate changes everywhere.
Same here with "Boots"
If in doubt, the former; it'll fail all over the place, should the ```python``` be 3.x, which should be pretty clear, or you could force a pretty fail by checking sys.version (might be worthwhile anyway, to catch unsupported 2.x versions?) - If you're running tests for the CMS or Django, one would hope you're at least vaguely aware of what's supported.
The reasoning for it is that the storage might need information about the image other than just the image. Here at globo.com, for instance, we store images with the organizational area and product code they belong to. Imagine an image that is a sports (es) image and belongs to "Globo Esporte" (ge). We would store that image in es/ge/path/to/image.jpg.

When we reach the upload storage there's no way we could discover those values just by looking at the image url. They would have to be sent alongside the image.

The problem with the date was that it was not being removed properly when storing the image (the problem Damien referred to was a bug).

That said, I do agree that your implementation is a more sensible default. I just like to have this extension point, though.
you still have a merge conflict in this file...
use the cached $this here, I also missed out on that one
Also this is not really last use, it's last aquire, which means a long held connection might get reaped even if its in demand
EmbeddedDeployableContainer should probably have it's own EmbeddedContainerConfiguration
I agree with Alberto. I was thinking about the same issue the week before. @basiclines any idea?
nit -> turnSpeakerOff
I would suggest if you plan on using an external lib to mark it clearly as such and place it in a sub directory (or entire) different directory so its clear that you will maintain/update it as changes come through upstream. Unless you plan on forking the lib it makes perfect sense not to modify the library in any way. @davidflanagan had some thoughts on this earlier.
This part is good, thank you :)
Add a space after <tt>}</tt>
Why do we have this duplicated selector, is the same as the above one right?
This has been implemented now!
does this ever block?
What if they have different slash flavors? Doesn't feel like a solid normalization. Maybe call Path.GetFullPath?
Carrier return is in the wrong spot ;). Nothing after the start of the block, but in between the require and the ActionController::Base call.
These cases should be more readable. "87" means nothing.
I really don't see what you have against the exclamation point. I'm not committing silly changes like these.
I think it's common to just use `str` for string. Also, I'm a bit uncomfortable with `dir` as the parameter name since there's the built-in python function of the same name. (It won't break anything, but it's usually best not to use it as a name.)
It seems like an Exception could cause `sysman.close()` do not get called. Shouldn't this be done with something like a `with`/`except` clause?
Lines 64-66 should be moved under the 69 line if, so this code won't run unless there's a conflict suspect
Is extending the user model now the recommended way of adding extra fields to a user? It used to be via user.profile
This and your (now removed) comment in the changelog, made me think:
How about adding a new bootjob that leverages the existing Database structure to write those items you want written?
That would take care of distribution and us inadvertently overwriting modified items in a single stroke.
I don't know when this should ever occur, it is about a rebuild of a build that was performed before. Only in a case where something has gone terribly wrong there won't be any sourcestamps. I think the best way to handle this is to log a message and to return defer.failure. Any other suggestions?
.... in that order.

Do we document anywhere, the intended method of getting a consistent picture of the universe?
i.e `startConsuming` -> query db -> updating rel. received messages
I don't think this correctly accounts for having multiple versions of a single cookbook.
No specific reason, we can change it.

This is more "lightweight" though.
I'm not sure if it's the intention, but I noticed not all global variables are listed here. For instance, the keystore related vars come later.
This prevents the script from being fully automatic. If it can work unattended it's so much better: we can put it in Jenkins, run it and go for a coffee, etc. Make this a command line option, or an environment variable (for instance, if the password is set).