labofsahil/github-event-dataset-2012 huggingface dataset:
The error in this case is only ever going to be of the form "foo is not defined" so I chose to ignore that error and log a hopefully more informative error with the requested template.

```
> vm.runInContext('foo.bar.baz', c)
ReferenceError: foo is not defined
    at evalmachine.<anonymous>:1:1
    at repl:1:5
    at REPLServer.eval (repl.js:80:21)
    at Interface.<anonymous> (repl.js:182:12)
    at Interface.emit (events.js:67:17)
    at Interface._onLine (readline.js:162:10)
    at Interface._line (readline.js:426:8)
    at Interface._ttyWrite (readline.js:603:14)
    at ReadStream.<anonymous> (readline.js:82:12)
    at ReadStream.emit (events.js:88:20)
```
This will forbid using this handler in any existing codebase as there is no reason to have an ``event`` context key in all messages
the event in context is specific to the pusher implementation. 1 channel may have many event types.
I was copying another implementation.. my bad


On Fri, Nov 30, 2012 at 3:23 PM, Christophe Coevoet <
notifications@github.com> wrote:

> In src/Monolog/Handler/PusherHandler.php:
>
> > +     * @param int           $level
> > +     * @param bool          $bubble       Whether the messages that are handled can bubble up the stack or not
> > +     */
> > +    public function __construct(\Pusher $pusher, $level = Logger::INFO, $bubble = true)
> > +    {
> > +        $this->pusher = $pusher;
> > +
> > +        parent::__construct($level, $bubble);
> > +    }
> > +
> > +    /**
> > +     * {@inheritdoc}
> > +     */
> > +    public function close()
> > +    {
> > +        $this->pusher = null;
>
> removing the object does not make much sense IMO. It would mean that the
> handler is in a broken state after that
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Seldaek/monolog/pull/132/files#r2278130>.
>
>
Seems like you need 100 :/

This is really bad. The sms platform code should probably send this event only when it is write to the database. @philikon, do you have any opinions?
It's OK. Make sense.

Thank you for your through thought.
I think the comment is mis-indented
Well that is certainly a good reason.  I'll update.
Could this block the vnodemgr for an extended period of time for a large vnode?
I agree. All the geometry comparisons are done so. Should I open an issue?
i think that will don't work on ie. 

we must use here:

Crafty.addEvent(this, s.obj, 'ended', function(){
This is saying if url does not have scheme prefix http
you should name it ``docComment``, not ``doxygen``, to give it its propername
Also, missing `@static` here
Go ahead and take out the blank line for a 1-line function
Well both, but particularly checking the package (check should already produce a message)
2012(-nothing)
Also, why is testing the view context in the tests for the decoratedenumerableproxy?
explain single threaded?  at least in comments
а нафига?
Fixed! Using a regex to pull out the desired value, and doing a full lookup on that.
I'm wary of this. Note that `git_odb_read_header` fails most of the time: if the blob we are trying to lookup is stored in packfiles and composed through deltas, we cannot read its header exclusively so we silently fall back to loading the whole thing in memory... This kind of defeats the point of this check because the blob is in memory already, even if it's *this* big.

@peff, what does Core Git do? Is this optimization correct -- i.e. is the point to prevent running a diff on a huge file, even when the file is already in memory?
I'm glad that you added this. Would you please also add some way to get in touch with you, e.g.
`@author Uri Shaked (uri@email.com)` or `@author Uri Shaked`(https://github.com/urish)`. Whatever you feel comfortable with.
`} else {` vs `}\n else {`
So when elasticsearch (actually, it's Lucene, but whatever) analyzes text to put it in the index, it saves the text in a data structure that doesn't have enough information to reconstitute the text for excerpting. Adding the wildly technically-named 'term_vector': 'with_positions_offsets' tells the analyzer to additionally store the position of the word in the document so that the information it needs to reconstitute it for excerpting is there.

This is covered pretty well in the Lucene book in Safari. I highly recommend reading/skimming the first 4 or 5 chapters of that book. I found it really helpful.
Why did you take this out?
CORRECTION. The final correct format is supposed to be::

```
-e git://github.com/mozilla/configman@3d74ae9#egg=configman
```

Note how I switched to github.com instead. We can change this to gitmirror.mozilla.org some day when it becomes possible to clone from that repo.
Though you didn't introduce it in this PR, you may as well remove the trailing space here and on the next line.
Will this cause any problems with reviews losing their line breaks?
typo and unneeded debugging...
We should think about and add comments here on what we want to do if an error occurs. Why do you think swallowing the error is the right thing here? We should at least add some logging.
It doesn't look like the topnavigation widget has been rendered at the point where this is being added to the .on(). Is there a way to force it to load the widget before I get to here? I tried adding it to the require() at the beginning and that didn't do anything.
That `$client->request()` interface seems like the cleanest solution to me, and making the callback the last argument would make it way less ugly imo. FWIW: I'm not totally against the current solution, and I'm not trying to piss you off. I just think there is a better way. :)

Can you hop on IRC, so we can discuss it?
This wouldn't help, I run tests from a lot of different directories ;-)

Actually I think the test folder is the right place for the phpunit.xml file, but as always, this is open for discussion.
Oh, wait. You mean using `FilesystemIterator[]` as description to `array` of `FilesystemIterator`? This sounds even crazier than old good docblocks now. @fabpot @stof what do you think, guys? I'm not sure we stick to this `@... ...[]` convention in Symfony2, do we?
People might be shocked by the exception overdose in this PR then, but I don't mind adding it if it is requested by @fabpot :)
Is there any benefit in changing this file? I think we could revert all of these changes.
`if (0 !== $code)`
this should be renamed to ``controller``
This is already covered in master: https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Routing/Tests/RouteTest.php#L105
So merging 2.0 into master would duplicate the test.
is it a deprecated method kept for BC only ? It yes, you should tag it as deprecated
Is there a reason you're adding a bunch of whitespace?
Switching to `switch` is good thing in this case. But this:
```objc
case CPAfterCheckinActionNewUpdate:
case CPAfterCheckinActionNewQuestion:
    ...
```
and this
```objc
(self.afterCheckinAction == CPAfterCheckinActionNewUpdate) ? CPPostTypeUpdate : CPPostTypeQuestion
```

is making it less readable.

This is much easier for reader's brain:
```objc
case CPAfterCheckinActionNewUpdate:
    [[CPAppDelegate tabBarController] showFeedVCForNewPostWithPostType:CPPostTypeUpdate];
    break;
case CPAfterCheckinActionNewQuestion:
    [[CPAppDelegate tabBarController] showFeedVCForNewPostWithPostType:CPPostTypeQuestion];
    break;
```
there should be no space after )

no need to declare `IBAction`, Interface Builder will take it from declaration in `@implementation` section (just use `IBAction` there)
[CPUserDefaultsHandler currentUser].userID stringValue] should work on NSNumber.
I like the rename though.
那暂时就这样吧，本来LaTeX里表格和图片就容易跳来跳去，很难一次做的周全，以后有问题再说
The line endings seem to be different now.

Is it just the <script> tag which has been updated?
Why the TypeType ??

> Starting with capital letter, we are considering a type, and two more
> suffixes "Type" seem to re redundant.
>

Actually, HasNestedTypeType is indeed a type (whence the capital letter).
It checks if the template parameter has a nested type called "Type" (not
any other type name).
It's not clear what the format of data_list or data looks like. Can you add a comment describing it, with a sample value?
> Eventually, I imagine urls in comments might become exempt.

Or not.
Ne goni!
by some definition of better..
Roooh!
the emptyMap you use here is actually the headers or?
Would it make sense to only switch language if old_lang != new_lang?
When triggered through `{{action moveElsewhere context="foo"}}` then `foo` will be passed to `serialize`/`deserialize` as a context. Maybe this example is more important because it will be confusing for newcomers that the docs seem to say "serialize will get a params hash" but then it gets both kinds of objects in practice (depending on where the call came from).
I'm honestly not sure what this is even about, but I guess it doesn't hurt to leave it.
If a skill is added/removed from a cert.
I think this needs to be a deepcopy because the values are mutable lists (otherwise if the query and body both had values with the same key things would be weird)
Want to pull it out before I merge?
`isFullscreen` -> `requireFullscreen`
I'm pretty sure you can `postMessage(message, origin)` directly...
Including it right now! Thanks ;-)
and this way we don't need all the crazy prefixing. Should be just `info-primary` (in the markup and css too)
(was it not 3000ms before?)
Busybox includes lots of other utilities.  Can we please link them all in, and then override as necessary the versions in /system/bin?

   for x in `./busybox --list`; do ln -s ./busybox $x; done
nit: line too long. Feel free to fix lines 374 and 378 while you’re at it.
I have no idea what this callback function can do instead of showing a console log message, so I remove it.
I would also test that; I'm not positive it will work.
Maybe there is no need for two new lines here.
@mounirlamouri was using it for security purposes, is it not needed anymore?
Why should the content be restricted to paragraphs? E.g. what if we want to use lists?

It looks like this could be replaced by a rule relative to `form[role=dialog] section` instead.
The `ConfirmDialog` in the dialer app is != from the `ConfirmDialog` in the contact iframe, you shouldn't have any issue, only 1 `init()` call by window.
The below should be totally standalone for the date picker adapted from the BB value selector.
There are some differences as I didn't use tables for the widget.
I did the code before the device section and I fail at merging! thanks
What about adding this particular comment next to the font-size: 0 declaration in order to make it clear for external people reading the code. thanks!
same for `action1Node` of course
Because you don't want to have a empty `TO BE DONE` menu in the Settings app?

IMO this project is at the point where such thing should be acceptable...
This is pretty minor..but it's good in doc comments to surround named code elements (like options) with backticks so they stand out as code elements.  So this last line would have `:allow_episode_skipping` rather than "allow_episode_skipping" (or whatever we decide to call it).
Opps:) But I I was afraid of that. Never mind, I'll thing about it. But clearly this patch is not suited well for a fallback...
Since it's not being used by anything else, I'll strip it out.
Always add a dot at the end of sentences. ;-)
Nemeli by i ty div a name byt s @?
This is a hacky way of doing something like MarkDeploymentAsFailed().
I suspect this needs to be moved into a "<VirtualHost *:443>" block.
Do you think it should be "unable to find ~/.rspecrc" or "unable to find either ~/.rspecrc or ~/.rspec"
maybe we could make this into a set here, instead of re-creating the set each time.
My initial thought for the xhr was to do a HEAD request simply for speed. That way it didn't hold up any process for larger images. However, a 404 should return quick enough anyway.
I just put all the stopHaltedInstance logic inside of declareInstanceHalting passing an additional parameter with the flavor

Let's see new multisite redesign to improve with this "load up something based on CS config"
You should add the generate_xml_metadata() method to IOutput superclass so that people can just reuse that method when they are building an Output plugin.
that was the word I was looking for, thanks
This seems okay for us, but it can't go into CodeMirror master as-is, because it means that the browser context menu won't pop up. I think we need to make suppressing the browser context menu a CodeMirror option. (Also, if that option is set, then it seems like we'd need to skip the next line as well.)
Very specifically matches files from the jar whose full pathnames are in the form: `equipment/Item_Name_Here.item`
I mean this log msg should be like "Need to export ..."
It happened on a specific build. We have looked at it for a whole day and were not able to find it. Don't know if we can reproduce it now. Maybe it is an idea to leave a short comment after removing the hack.
What's this for exactly? Was it causing style issues?
same here. Using ``else`` is not needed as the ``if`` returns
do you use this action?
Тази черта не трябва да се маха.
I know these are rather arbitrary, but they look best, imo.
Oh, but what does your comment about ```os.path.join``` refer to? Did I accidentally manually concatenate paths somewhere?
Incuding jquery is sensible. For this 2 regex I can try to find some other solution that is not so complicated. Regex is necessary thought, as 0.0.2 does not work if you put search for term in parentheses ie.
Does "bogus" get re-set to a real etag?
read() - do not ignore result!
please fix all 7 items
this will break every linux box that does not have the `iproute2` package installed so we should provide an `ifconfig` fallback. So you can e.g. use `Puppet::Util::Resolution.exec('/sbin/ip addr')` which will return the output on success or nil if the command cannot be found.
Please drop those if they are not needed.
The url returned by `download` is escaped, so the expected value needs to be escaped as well
hmm not sure if i like this
I agree, and was thinking of it when writing the code.

I'll see if I can make use of the global colours.
Probably just need to adjust the saturation, brightness or whatever to get matching results.
Please add copyright header information.
`metatilestream` should be renamed to `tiles` for consistency with the TileStore interface.
1) Broken indentation
2) I miss an ```if( sh != NULL)``` check for this branch.
Does ```dirname()``` really return paths with backslashes on windows?
Paddy, could you please leave the old tests in there as well? That way we ensure there are no BC issues (or at least become aware were there any).
annotation value can be simply "CurlClient"; they're resolved based on imports and/or current namespace.
@cgmartin Please uncomment the data that is failing. Hide the fails goes against the priniciples of testing.
I know you're not the one who introduced this pattern but this really should be a AP_INIT_FLAG directive.
But it's so fast!
It's not really "Restore", it's more like "Reset", I would say.
Use prop and can be simplified:

`$("#toggle-all").prop('checked', allmarkt());`
I would call this event onLoginStatus. required seems a little misleading.
I think the context should be cspace.advancedSearch, since cspace.search.searchView is a little broader and might be slightly unsafe.
2 empty lines before class defintions
Do we need to add the above? Are they used?
Typo: thunderbird
this sets u to none. shouldn't it just be vouch(u)?
True. but it's just from the current site. Don't worry about it.
Great. this is much more succinct.
We can delete this comment I think
it's an optional checkbox? oO
It's really not needed, there are no heavy operations done during encoder initialization.
should this be changed to `rdoc.title = "Jammed #{version}"`
why many vs. has_one ?
@skottler i mean these lines ^^
I'm going to work on switching to that right now
This logic should be part of the GaiaApp class.
I think maybe I saw you mention you were still tweaking width, but it currently doesn't seem to always fit on a 1280px wide display (I think we recently gave up on trying to support 1024px, though most of the site still looks reasonable at 1024px). I don't recall our official standard here though, but I'm pretty sure it's at most 1280px.
should this be json encoded?
внутри функций не выводим дебаг информацию
отличное название
ты не можешь изменять то, что тебе передали --- оно const
move изменяет
Just curious, is this a python 3 requirement?
Single-quotes instead of double-quotes
Ah, snap!

You're doomed then, or you move the collector class inside the method. But I guess that's ugly.
You don't need RallyTask.
Hier mist ook de indetation
It's not a needed change, for sure. :)
Насчёт имени переменной - соглашусь, недосмотрел. 
А вот насчёт создания внутри метода... идея была именно в том чтобы сделать подобие "метода класса", а не объекта, чтобы массив выделялся в памяти только один раз, а не каждый раз для каждого представителя класса, но т.к. я не силён в obj-c то сделал через такую конструкцию типа синглтона
Метод вполне рабочий, но не будет всегда менять нужный процент последовательности, а над алгоритмом можно поработать (при мутации в 100% методу придётся тяжко).
а зачем нужно такое разделение, почему не использовать везде truncation?
дополнительный плюс - не надо конфиг менять перед каждым тестом, можно установить его только перед сьютом
Let me know if you are interested in the other commits; I think I can use some git-fu to expunge the particular commits that I included.
an MD5 signature, but *a* SHA-256 signature.
there should be spaces between operators, like ``start != undefined``
Shoot. Of course the tests passed in my side!

I'll fix it up. :)

Please excuse my brevity; message sent from a mobile device. 

On Oct 29, 2012, at 8:29, "Daniel Doubrovkine (dB.)" <notifications@github.com> wrote:

> In app/acme_app.rb:
> 
> >        # api
> > -      response = Acme::API.call(env)
> > -      # error pages
> > +      response = EventLab::API.call(env)
> What's this EventLab API? :)
> 
> \
> Reply to this email directly or view it on GitHub.
>
Should those be capitalized?
Tip is a `ICommit`
Yup, easier to let the code speak for itself
why not 64 here?
Yes, data can be an "n_epochs x n_signals x n_times" array or a list/generator with "n_signals x n_times" arrays (e.g. Epochs) or a list/generator of SourceEstimate.

I can add averaging over frequencies in each band. One problem is that it is possible that the "accumulator" function doesn't return an "n_pairs x n_freqs, .." array in which case averaging won't work. But I can check for this and throw and exception if necessary.
I would just do:

out = data.copy()
ch_std  = np.std(data, axis=1)
out /= ch_std

I would not use zscore as you don't want to remove the mean (should already be zero is you high passed)

also I would return ch_std to multply its inverse to mixiing matrix to so we can just apply it to the output of raw[:, ...]
I think we should call it "generate_sparse_stc" and add to the docstring that we randomly select a single vertex in each label.
no neeed for index
I don't think we need this. There's Mockito.eq
Don't remember what's going on here right now, will verify.
Please adjust formatting here, should align with column after`{`.
How do we know what the Global object is without reading the config?  What if the user changes the application.global setting so that the global object is now com.example.MyGlobal?
Try to avoid adding/removing big framework files as part of the patch since it makes reviewing difficult. Would be better to remove this as part of a separate commit that is not part of the patch.
you should link to the interface instead
请求超出
过速请求
!important should be document what it has to override. If your intention is to just have the same color as body text, i prefer using `inherit` in pure CSS, and a variable in SASS, as `inherit` and a variable name are explicit what you intend the color to be. Also, one less color to maintain.
The curly braces around sortAttachmentsBy are not needed here.
We already have a schema version 94! The migration is called 094_add_category_to_journal.rb.
Should these survive a Restart?
Я вообще не помню, что она делает и зачем.
Хз. Я так ниразу не делал  и мне понравилось.
link style should use global shared class : color, (font-size)
This can start matching anywhere in the string, use re.match instead (or "^(\d+_)....")?
I've just added a section called 'Varying apertures/PSFs' - could you check it out to see if it matches what you were thinking?
See the latest git version of this file. You need to wrap `attributes_table_for` in `render_arbre_component(assigns) {`.
yeah I know, this patch got old, I'll update it soon
that's a good suggestion, but we can punt on it for now.. too many changes already :)
Grumble grumble format strings over concatenation

Alternately you know that 
'''("string 1"
  "string 2"
  "string 3")'''
is concatenated for free right
I saw that :) Nice catch!
@danberindei good point, I've made it a concurrent map.
@maniksurtani here's the problem, should be:

    AdvancedCache localCache2 = cache2....

And not:
    
    AdvancedCache localCache2 = cache1...
Why duplicate what's already in https://docs.jboss.org/author/display/ISPN/Contributing+to+Infinispan#ContributingtoInfinispan-Logging ?

If something is wrong there, it should be fixed. The readme should point to that document to find out more ?
Need another blank line here.
this property does not pass through an element.
Well stop is for stoping the application. The stop implies a Unregistration
why do we need to store _w and _h if there is the tileSize property at the layer level?
Is it useful to pass an object if implementors ``MUST`` cast it to a string anyway? If someone passes an object (which I do not expect as any implementor is forced to treat it like a string anyway), then he can also cast it to a string himself.
this line should also autoload the tests/unit folder for the bootstrap
Aw sweet - makes tons o' sense! Nice catch man
Can you split this into multiple lines for readability?
Please update the postgresql driver to 9.1-901.jdbc4 the same as was done in PR #714.
I remember Sören said that for performance it is better to solve for a bunch of k values in a run. Still, could that be included in model selection?
why is this needed? if shogun uses pthread related stuff in its header files the correct fix would be to move this code into the corresponding .cpp file
we don't use anymore $p_ in parameters names
This jumps to err: with process_locked still set, after clearing the spinlock -- I think the fix is just to remove the __wt_spin_unlock() call.
"Kruisverkopen" klinkt nog slechter, of ligt dat aan mij? Alternatieve vertalingen?
Do we really need to instantiate it for every request?
You used the word "mist" 11 times. It was noticeable and distracting. I get you were trying to set a tone but repetition only goes so far in doing that.
How about we treat project as part of advanced_search? And let's make advanced_search something shorter, just 'search' would work just as well.
Fair enough! I'll change it.
confirmed on ruby 1.9.2p290
my bad. previously it was a subclass of OutOfKitchenError.
OutOfKitchenError#message is defined with a default message. 
and in the subclass, the specific error message was being ignored

self contained example here, https://gist.github.com/3004559

not needed now, will push a fix
Defaulted to 80 and parameterized in the preprocess_resume method.
can you please add minimal phpdoc for this? i guess it is used to decide if a document is published or not.

if i understand correctly, this does not help for the question whether a particular user is allowed to see the document or not (because he might see unpublished content - but this could get a lot more complicated like "i may see my own unpublished content but not anybody else's)
There is another downside of this. This means that user will automatically have enabled only those panels which existed when user object was initially created. If later on we add new panel, all users will have to manually add those panels. Is there anything we can do about it?
Was already fixed, but it is strange that new commits don't update this pull request
Let name this "profile image"?
Please see my comments from before. Attachments do not have is published flag?
I just now noticed. This should go to `accont/static/piplmesh/js/registration.js`. ;-)
This line is now a bit obsolete?
Move `10` to a top-level constant.
This one line bellow which the comment is. You should align with outer scope. Not inner scope.
Well, I wanted to prevent Pkg::SourceSaveAl call possible called 3 times. I do not know the time cost.

(And I'm almost sure that if I'd write it there, I'd see a comment criticizing it)
`"\\="` and `"\\"` would be a bit easier to read.
You probably want to remove this too, then?
I think these last two should be messages.error() ?
In fact $options['em'] have already an EntityManager at this step.

Don't check more I admit
is this supposed to create a writer? on 1.9.3 at least it does not. in any case, pleas just use `attr_{reader|writer|accessor}` which is more explicit
Please implement a `click_coordinates` method on Browser rather than calling `command` directly. Also, the command should be underscored rather than camel cased, for consistency.
Should we put the Do's before the Don'ts?
Ок, учту, сегодня подправлю
Insert here months-old criticism about traversing an array in quadratic time.
Consider: Is it helpful to add a null check too?
The comment should call out that this is an implementation detail; users should call this directly.
Should use the find_and_replace function above
Content on the next line.
We usually use snake casing rather than camel casing in ruby. Did you do this out of habit?
Indent to the open paren.
I think the .dlls are already collect by pyinstaller's dependency checker; if not, I'd make this a hook-specific change, since the .dll would need to go in the executable directory, not in the data files directory. I have not experience with Macs -- is this also true for .dylib files as well?
Rather, if we have decode Active, and we pass it an array (the subj stuff), it detonates on the fact that you cant run Decode on Arrays.  That specific Book ID refers to some Abdul guy whos name has a unicode character, of which if you /dont/ run Decode, breaks the function horribly.
cosmetic: indent array to align vertically
Thought about it. Don't know though. Right now the Cython pieces are specific to certain parts rather than common shared code. We'll be able to drop all this funky import _build business soon enough I think.
Wouldn't it be cool if `for_code_point` returned an instance of something like `TwitterCldr::Shared::UnicodeData::CodePoint`?  That way we wouldn't have to use array indices to access the code point data.  In other words, you could do `unicode_data.code_point` instead of unicode_data[0].  What do you think?
выглядит он конечно не так пиздато как мог бы(
This option needs to include the word "Point" somewhere in its name, to make the context clear.

Alternatively, there could simply be a "selected" parameter to drawPointCallback. What do you think?
Well I thought it should be in a seperate branch - a pull request I was trying to keep to minimal files changed

This it seems is a mistake
I *think* in events they usually just use lowercase?  I'm not sure about that, and I don't really care, as long as we di it consistently.
pushed a typo fix for `rais`
Again, should check several invalid instantiations to make sure they raise the appropriate exception.  e.g.
```
s = Shear(g1=0.3)
numpy.testing.assert_raises(TypeError,galsim.Ellipse,s,g2=0.3)
```
doesn't pass.  (i.e. `galsim.Ellipse(s,g2=0.3)` didn't raise an exception as it should have.)
and you can write out csv_rows as they come, too
This function is in CatalogPredicates. But perhaps better than having yet another class for holding functions etc. Will ponder more as I go through the code.
Oh, I see.  Thanks @michaelficarra.
CSS standard requires that `font-family` hast, at least, one generic font family. You can add `sans-serif` as the last alternative.
This indentation is very hard to follow. Can't it be two spaces from the start of the previous line as it is a Hash nested beneath?
Oh yeah...added `isString`
note that this does not prevent them to execute it if they use ``php app/console`` instead of ``./app/console``
Please don't do it. The master branch of symfony-standard is meant to be for 2.2.
You are currently sayign it should work from 2.0 (which is wrong) to 2.9999999 (which is obviously something you can ensure)
Digging the comments.
How does it look now? From what I can tell, it should be correct.
This seems a bit over-greedy. If this works, we should also move the regex up to the top of the file so people can find all the regexes easily for maintenance.
I think this should just be `get($uuid)`
Revert this line, mine old version was a more compact way of doing it.
This will spew a lot of noise during initial connection and should be removed or commented.
Do we need to query from tl_mail_templates? EmailTemplate will automatically do that, and you should be able to retrieve data from it for the database storage.
I had a problem when trying to modify mac_prefix to something different, it turns out that just adding '' fix the problem
`definign`
Исключение здесь, и файл останется незакрытым.
Этот код не надо копировать к себе. Используй мой.
http://en.wikipedia.org/wiki/Numeric_character_reference, both are valid in sample part of the wiki.
why not specify content type?
That code then would be superfluous because defined members default to `NULL`.
Let's use standard two space indentation for Ruby code.
we can probably move the `[index, row, col, pos]` into an args variable and pass that along... no sense in creating an array for every call.
Superfluous character.
Is an app every going to be more than one payment type? If not, I would probably use a string here to represent the payment type. If multiple is possible then I'd use a list.
We should have a new line here.
What change do you want me to make to the header then? Are you saying I'm not part of "Team XBMC" so shouldn't be saying that in the header? As far as I'm aware, I am now part of the XBMC development, and I'm donating this code to the XBMC project.
Well, don't know the weird part but, actually, the alias for these variables is not useful at all. Fixing that. Thanks for noticing.
Is there a way to add a single observer to the array (`@each.isSelected`) but have it pass a reference to the object that changed and not the array?
Add the reference on obj before releasing reference on height_ in case they point to the same object, i.e.,

obj->addRef();
if (height_ != NULL)
{
    height_->release();
}
height_ = obj;
Probably this line should be removed, since seems all platforms support that property.
EDITDATA is now SerializedED located in Edif.h with RD (rundata). This is because it no longer needs to be edited per extension, thanks to the EditData class (EditData.hpp)
Oops, fixed. I think I changed my mind about the implementation half way through and forgot :)
can you switch this to the route_helper and a link_to please
I did, just fixed that....
Use `min_value=0`
All of this needs to be replaced with something like `form_data = forms.DailyForm(data=request.GET)`
Yes, nova-network was failing for me without it.
What's this for? I'm fine having it; I'm just curious.
And yes, sorry for these white space removal ... Sublime Text 2 is doing this by default. I would tend to say that there should never be white spaces after the last character of a line
all these verifications should be done on the page.
that is redundant .. even if 'vary' isn't set .. empty will just consider it to be empty with no warning
just commit a fix for this
```defaultdict()``` avoid the need for this.
This 'end' looks odd.  Did you actually mean here, or after the ::Exception => e handling??
Would be better to use a local variable here, and not modify the datastore options.
It is possible,  more client version will be supported in the future, so it can change at the instance level.  I started the ruby programming with these modules, but I've already understand instance variables at the module level can cause problems. It will belong to the state of the module. Suggestion?
All of this code is copy-pasted.  =(

Much better would be to figure out the vscpath and move the copy/download code after that. Something like this (untested):

```ruby
     if datastore['VSCPATH'].length > 0
       print_status("#{peer} - Attempting to grab NTDS.dit from #{datastore['VSCPATH']}")
       vscpath = datastore['VSCPATH']
     else
       vscpath = check_vss(smbshare, ip, text, bat)
       # If VSC doesn't exists already then we see if we can create a new VSC
       unless vscpath
         vscpath = make_volume_shadow_copy(smbshare, ip, createvsc, text, bat)
       end
     end
     if vscpath
       n = copy_ntds(smbshare, ip, vscpath)
       s = copy_sys_hive(smbshare, ip)
       if n && s
         # If the above succeeds then we just have to download our files
         download_ntds(smbshare, (datastore['WINPATH'] + "Temp\\ntds"), ip, logdir)
         download_sys_hive(smbshare, (datastore['WINPATH'] + "Temp\\sys"), ip, logdir)
       end
     else
       print_error("#{peer} - Failed to find volume shadow copy")
     end
     cleanup_after(smbshare, ip)
     disconnect
```
Indentation for this rescue should be fixed.
Questions about the data package:

 - there is plenty of "stuff" in there - are all of those images used?
 - I couldn't figure out how to load one of the Numpy dump files with `load`?
Another contraction
I don't really like this... It surely adds flexibility, but wouldn't it just be better to put the `@SqlResultSetMapping` here?
Throughout:  In general, we should not do `else if` for each pass since, for example, we might request both depth and color passes, or - in time - color and pick passes.
It's not a toggle.  I need to be certain the state is set to a particular value.
@kring why do you render front-to-back?  For early-z, right?  Have you tested the performance numbers?  I heard (but can't actually believe) that early-z does not work in WebGL.
default implementation? We already have a default implementation of this and it is called EventHandler.bind(...). I don't see the reason for adding this code. It does not give us any benefit. The only thing that give us is more code....
Missing space here and one line below.
Is there a reason we can't just say, "measured from the right edge of the indicator to the left edge of the indicator."?
This is now defined in JavaScript (see tabgroup.js) so we don't need to expose it from Java anymore.
Undestood. So, the strings (@"starting", etc) will continue?
This is supported on Android
Damn these equations are long!!! I can see why you wanted to use numerical values. One suggestion would be to define the super long text of the equations in a file by itself, i.e. ``long_force_eq.txt`` which simply hold the text of the equation. This file may be super long. Then in your equation definition you somehow load in the equation from file and evaluate.

Another thought is that these long equations may benefit from making use of Cython or Numba to compile the code for faster runtime and allowing the equation to be in its on file. Let's talk about this. I can spend some time figuring that out.
mixing null and undefined, might as well use one.
да, в апи аплодкера должна быть функция "закрыть окошко, которое я зачем-то открыл"
coffeescript supports sexy interpolation: 

```coffeescript
$("form[id*='#{ stripe_handler.form }']").submit ->
```
I mainly added this as I was having some issues running it with the standard python 2.6 included in RHEL 6, the local copy of 2.7 I had in my path compiled from scratch was working. I will try and get around to figuring out why.
Is that the case with `union`s too? Could you e.g. write an example that has a `union { long align; char[10] payload }` allocated on the stack and leads to a misaligned object? (That would be a fairly major breakage.)
Btw, these workarounds won't be necessary anymore after https://github.com/D-Programming-Language/dmd/pull/1244 is merged.
Except that we have no idea where the `FILETIME` came from. Sure, if it really came from the file system, then there's no way that it's going to be out of range. None of this code will exist by the time that the file system would be giving times that large. But there's no guarantee that the `FILETIME` came from the file system, so we have no guarantee that `FILESYSTEM` is within a reasonable range. The code is more robust if it checks.
s/may use/may be using/
maybe replace the "if (oldEntities.size() == 0)" just with else to the last if statement? Should be the only size possibility left after you have checked for >1 and ==1 : )
``%kernel.web_dir%`` should be used. ``%kernel.root_dir%/../web`` is wrong as the RadKernel defines the root dir as being the root of the project (so the web dir is ``%kernel.root_dir%/web``
Yep, see the discussion over here: https://github.com/LockerProject/Locker/pull/849
No, `$config->default` returns the actual default or null if there is not, but won't be used in this case since `$hasDefault` would be false.
Ah, I was grepping for the full parameter name in the extension class. I see how they're set now, though :)
Actually, specifying this isn't really needed, since `ConfigureMake` is the default (and it will change like that).

So, leaving that out here is safe, EasyBuild will use it anyway unless there's a GHC easyblock (and the one you have a pull request for in https://github.com/hpcugent/easybuild-easyblocks/pull/44 can be fixed to work for both 6.x and 7.x).
remove the `patches` line, with the `configopts` set like they are, it should work

Also, get rid of the patch files (both of them), if we don't use them, they shouldn't be there. 

You can always find them back via commit history is you really need to.

If you clean that up, it should be good to merge in.
Um, I don't know.  I don't remember changing that and probably did it by accident.
dang.  fixed.
Мы теперь #pragma once юзаем :3
Here and below, should be single quotes and missing a space before the `{`
Why are you saving/restoring the error state?
You cannot continue execution at the failing instruction, i.e. sigsegv_userspace_process must not return.
That also implies you can scratch the code vs. data distinction too.
From: "David Nadlinger" 
<reply@reply.github.com>
>
> You should probably add a bunch of `/// ditto`s here.
>

Done. (Also done on #174, but I guess that's one's going to die anyway ;( )
Good evening!
I understand that it looks a bit shortly, but you've already made the check `StringUtils.hasText(directory)` before, so you should just have this one:
````java
If (hasDirectory) {
     builder.addPropertyValue("directory", directory);
}
````
Take a look how it's done about *fileNameGenerator* in this class.
IMHO: at least as done in other places too ..
There is password editing now. The page will also contain settings editing in the future.
because category cannot use setattr(position, k, v) - it's not a field in Position but is on its own separate model/table.
Could you be more explicit about *why* you're using this stack-like object here? Could you not just use a regular array?
well... `Marionette.CollectionView.extend` but otherwise, yes. :)
checking for libpthread is necessary to make certain linux distress happy.
There doesn't seem to be a type that handles our `Lots of Satellites` use case, it's `EventSource` but we need to jump around in time based on the simulation clock.  I suppose we could implement the logic to do this client side without any help from the CZML, but it seems like it might be useful to have the CZML help out here.  For example, how would we know what a good "block size", for lack of a better term, is good for the CZML?

Also, what if I need to poll a source, but that source is actually a file in EventSource format?  I want to load it incrementally, but it's not going to stay open.
add test to createUser method to evoke this response
It's not about adding default parameters in PHP for example this is supported as well it's more about the consistency in the user experience between languages. But I'd suggest to keep the current design as changing this for other REST services will be painful
把$file_type先strtolower一下，这样就不用strcasecmp，直接strcmp，能提高一点点效率
/Developer/SDKs/Research In Motion/BlackBerry 10 WebWorks SDK 1.0.4.3
Sure, then it's not really a long touch. I might as well just add it to touch.
fixed

From: Tim Windsor <notifications@github.com<mailto:notifications@github.com>>
Reply-To: blackberry/WebWorks-Community-APIs <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Monday, 5 November, 2012 3:35 PM
To: blackberry/WebWorks-Community-APIs <WebWorks-Community-APIs@noreply.github.com<mailto:WebWorks-Community-APIs@noreply.github.com>>
Cc: Brent Thornton <bthornton@rim.com<mailto:bthornton@rim.com>>
Subject: Re: [WebWorks-Community-APIs] added mail attachment extension (#114)


In Smartphone/MailAttachment/README.md:

> +## Summary
> +
> +     var result = webworks.mail.attachment.send(emailAddress, subject, body, filePath);
> +
> +
> +## Code Example
> +     <html>
> +             <head>
> +             <meta name="viewport" id="viewport" content="height=device-height,width=device-width,user-scalable=no" />
> +             <script type="text/javascript" >
> +             function doClick(){
> +                     var email = "test@test.com<mailto:test@test.com>";
> +                     var subject = "See attachment";
> +                     var body = "Some text goes here";
> +
> +                     var file = "file:///myiamge.png";


minor typo here. Can you fix that?

—
Reply to this email directly or view it on GitHub<https://github.com/blackberry/WebWorks-Community-APIs/pull/114/files#r2034794>.



---------------------------------------------------------------------
This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.
> Missing line break.
fixed

> Avoid yoda conditions.
see other code comment
About moving it to the advanced features section. I did consider it but decided to leave it here. I don't think it's really advanced. CErtainly not as advanced as opening your own `IndexReader` and messing around with Lucene indexes :)

`@ProvidedId` is not an advanced feature, it s basic feature of Infinispan Query.
`IndexingActionType.AUTO`
or
`IndexingActionType.DONT_OVERRIDE`
Do we need this test? What exactingly is this testing in Dragonfly? Or why does this need to be https?
Why there is a cache array here. The MetadataFactory already has one.
this is not settable by the end user.
(= (mod n divisor) 0) would return a boolean already
when the argument is a single lexical token
I'm not 100% sure what 'Waits for the startup script..' means. Do you mean the instance waits for the startup script to run?