labofsahil/github-event-dataset-2012 huggingface dataset:
@maguec I got DNSRR to work on OSX but had to change these lines

```
    match(expression[:expression], :A) do |dnsmatch|
      dnsmatch.respond!("#{expression[:ip]}")
    end

```

to

```
    match(expression[:expression], :A) do |dnsmatch,transaction|
      transaction.respond!("#{expression[:ip]}")
    end
```

Otherwise it crashes. Does the original code work on UNIX?
Why not calling this function 'mask' because that is what it does?
An example project to demonstrate the issue can be created as follows:

1. Create an ASP.NET Project in Mono Develop
2. Create a sub folder called vdir
3. Create a web.config file in vdir which contains the following content:
<configuration>
 <system.web>
 <authorization>
 <deny users="*" />
 <allow users="me" />
 </authorization>
 </system.web>
</configuration>
4. In Default.aspx.cs enter the following C# code:
public void Page_Load(object sender, EventArgs args)
{
	Configuration config = WebConfigurationManager.OpenWebConfiguration("~/vdir/web.config");
			
	config.Save();
}

This application will run in .Net. However, Mono stops the server process without any notification and the web.config file might be corrupted.
Welcome Dave!
because under 2.8.1, it didn't compile unless I prefixed with scala.
This is fine, but just so you know, there's handy stuff in django.utils.timesince.
Not sure how it's dangerous. If you mean that it exposes the player to some of the script, then yea, that happens, but the full source code has to be sent to the client anyway, so it's not like the client has access to the script.
It made me somewhat happy.  You could have picked a better palette though.
I think /localizers is also affected
извините...
Yeah, I'm thinking of dropping this editor and look for a more basic editor and put more of the control into the web designer.
Yeah, I was just thinking that. These are definitely the really insane ones. It's amazing how hard it is to get all the corner cases right. I wonder if Alan has taken a look at this. I feel like he might appreciate it.
And here (I know you didn't touch this part of the code, but might as well fix it while you're in the neighborhood ;-) ).
This looks useful.  I've had some requests to add support for weighted records.  This looks like a good start.  Could you send a pull request?
fuck...wieso ist mir das net aufgefallen ..sry
http://www.touchqode.com/
closes #622?
Letter g is missing. I can't pay for this shit.
Censorship man!  Not a bad idea - it'd be awkward to hit anyone's work filter for an OSS project.
Yeah but some people were also bitching about recent signs in their original state, because of the lag they cause. And some people would want to use both in different ways.
My version doesn't have the extra link about getting involved with Seamonkey that your original patch had. Could you add that?
I did find that property after I posted here, and it did hook it up nicely, but I do like the (at least apparent) flexibility / control of making my own adapter to Log4j.
but ``get_storage_class(settings.ADDON_WATERMARK_STORAGE)`` might have some benefits if we want to incrementally migrate backends and/or have the ability to revert them quickly if something goes awry
Maybe [is_readable](http://php.net/manual/en/function.is-readable.php) is useful here? I've started changing all of my file_exists usages. It depends on how you would like to raise a failure: catchable false return value or uncatchable Exception.
More thinking out loud, we should also include Flump's version number as a hash input, so upgrading will force a re-export.
should we be keeping the [old ones](https://github.com/mozilla/zamboni/blob/master/lib/settings_base.py#L1013)? can't hurt, right?
This is commit is not working yet. Give me a few minutes, I am fixing it.

On 27 February 2012 18:47, Sebastian Probst Eide <
reply@reply.github.com
> wrote:

> couldn't this be written:
> ip :: (string_list_of_json_list ips)
>

> Aren't cons cheaper than append?
> Maybe it doesn't matter since the lists are never going to be big...
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/crotsos/signpostd/commit/1f8277f5384a0a269731c7fffed466b699e2462a#commitcomment-1016434
>



-- 
Charalampos Rotsos
PhD student
The University of Cambridge
Computer Laboratory
William Gates Building
JJ Thomson Avenue
Cambridge
CB3 0FD

Phone: +44-(0) 1223 767032
Email: cr409@cl.cam.ac.uk
Ruediger Meier left as an exercise for the reader:
> Hi Nick, in case you have a shell available in your environment you could do the atemsprayer stuff also by using this one liner
> atem -s -n -f 0041000 /path/to/data | while read -r symbol fnum ;do atem --fdat $fnum /path/to/data > bla/$symbol.csv;  done
> or have a look at "man 1 csplit".

oh, and by the way, thanks for the feedback and the work on atem! you've
allowed me to replace a doddering old wine-run .exe which crashed two times
out of three.

well met! --nick

-- 
                                    nick black <nickblack@linux.com>
                 http://www.sprezzatech.com -- unix and hpc consulting
  to make an apple pie from scratch, you need first invent a universe.
This is a really useful tip! Thanks!
@jesusabdullah No need to. As I said, I fixed the regression in 44356ee04efd7ae4d109129db724b9a01684faaf
karaj

On Wed, Mar 7, 2012 at 12:01 PM, Sajjad <
reply@reply.github.com
> wrote:

> kodum shahr?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/honarkhah/Bonfire/commit/bacde686c578a6472cafc45b9a4cdd4885d781e7#commitcomment-1052291
>
Just an update - this morning on my walk to work I thought of using recursive guards to simulate a loop: https://gist.github.com/1933910

This actually works! With the caveat that we can't currently modify `.span1` into `".span@{gridColumns}"`. Hoping to work with @cloudhead on an amazing solution which uses this :)
This line may never get called by PxLoader class because it considers no tags on the loader or resource to be equal to a match. I think that within the context of this "contains" method returning false in this case makes sense, in case parent object logic changes.
I would call this an API rather than JS (reserving `js` for ES-specific functions). Other than that, this looks good!
`ready` would be a better name

http://api.jquery.com/ready/

> While JavaScript provides the load event for executing code when a page is rendered, this event does not get triggered until all assets such as images have been completely received. In most cases, the script can be run as soon as the DOM hierarchy has been fully constructed. The handler passed to .ready() is guaranteed to be executed after the DOM is ready, so this is usually the best place to attach all other event handlers and run other jQuery code.
My mind always wants to think gizmo from all my 3dstudio/max usage
!(Great Job)[http://cdn1.hark.com/images/000/003/249/3249/original.jpg]
`getContextPath()` will return "" if Jenkins is served at ROOT.

Assume that getContextPath() => "", itUrl => "iturl" and urlName => "urlName",
value returned was "/iturl/urlName", but is "iturl/urlName" (no slash at 0).
It should be removed.  Thank you!
To run arbitrary code and make conditionally load modules like in

```js
if (isCanvas) {
    modules1.splice(1, 0, 'lively/EmuDom.js');
    modules1.push('lively/CanvasExptCoreFixes.js');
    modules2.push('lively/CanvasExpt.js');
}
```
No. It is here because auto-key-retrieve makes some problem with hanging software (e.g. Mail with GPGMail).
If a software want the gpg.conf behavior it need to set this manually.
Since the above lines were taken out we have been having issues with our checkboxes which have dotted names.
Credits?
@xEnt22 You realize you are using git right?  You can revert to any previous commit right in your own private repo and do any kind of work you want, then merge it, or branch it, or do whatever you want.  Nothing needs to be done with this repo for you to get any work done. :)
Could you possibly move the whole of logging to it's own file? This is starting to be a lot of code for main.js.
Extended commit message should have been:

Move unlAlerts namespace declaration to the same location as the existing functions.
Ensure that unlAlerts.data.alert exists before attempting to check for .info
unlAlerts.server.init should not return anything, as the caller does not need it.
Ensure colorbox.js is loaded before attempting to init a colorbox.
Cleanup dynamic content generation with jQuery functions.
Add a commented reference to the alertSimulator.php that can simulate an alert.
Remove code duplication by moving cookie logic (unlAlertsA) to its own closure.
I don't buy it - why not make the up/down a parameter of the (single) UDPBurst measurement type? I want to be careful not to make the code too ugly here...
Hmm, Aco warum brauchen wir die Funktion eigentlich? Ist es richtig wenn ich davon ausgehe das wir sie nur brauchen für die Unterscheidung von addWord und newTranslation?
No french hypen file? :(
please, make over pull-request. very few people are interested in VC configs.
This has been fixed upstream for a while :)
This is really not a calendar adminstration screen (I think there is a big difference between a dated todo-list/reminder and a calendar), so do not recommend protecting it with that control. Here is a list of possible controls:
http://www.open-emr.org/wiki/index.php/Access_Controls_Listing
should these be OP_OMAP_*?
I've implemented this for events. Functions and constants come tomorrow.
Heya

compile error on linux ( debian 64 bit )

[ 78%] Building CXX object src/server/scripts/CMakeFiles/scripts.dir/Northrend/DraktharonKeep/boss_novos.cpp.o
[ 78%] Building CXX object src/server/scripts/CMakeFiles/scripts.dir/Northrend/DraktharonKeep/boss_tharon_ja.cpp.o
[ 79%] Building CXX object src/server/scripts/CMakeFiles/scripts.dir/Northrend/DraktharonKeep/boss_dred.cpp.o
Linking CXX static library libscripts.a
[ 79%] Built target scripts
make: *** [all] error 2
我认为不妥！
fixes Issue #8 ... look at all of those beautiful .meta files being deleted ...
Does this // work an all browsers?
Sorry, @giovanichiachia -- I apologize for your having done unnecessary work!  

Its not in accordance with the CVPR11 paper, b/c that paper was not doing screening.   There needs to be independence between screening and testing images ... so a new protocol needs to be established.   I might write them as two separate tasks, when I get a chance.
s/affected/effected
see latest commit, but you are right wrt. spaces...
This doesn't fix issue #1288
Please use this test case for checking: https://github.com/cocos2d/cocos2d-iphone/commit/9f70fd1c6a737a6ae644cfbd011cbfb67dcfe882#commitcomment-994129
Yes, hmm. But do you think that they will switch everything at the same time? Just because new-style exist, doesn't really have to mean that it will always throw those for all APIs?

So you don't think it should be possible to check for new-style in IndexedDB, but old-style in XHR?

Reason I did it like this is because it required very small changes. :-)
Is this being removed for ARC?
This must be a mistake?
I should have been more clear with this before, 'format24' only applies to the time picker, and not date picker
These are logged quite frequently now, as a side-effect of the DRM-sniffing code I've added.
Looking good. Can you update the steps too?

Most if not all of the recipe rules will apply to providers equally I think.
Yes I sort of agree but this is also due to the way castle is generating
ids (not using assembly qualified names). The other containers works just
fine.

I don't want to change all our persistence implementations this close to
release. But we should definitely do it for a 3.1 or 3.2

/Andreas

On Sun, Feb 5, 2012 at 10:47 PM, Johannes Gustafsson <
reply@reply.github.com
> wrote:

> Well, yes, it does work in this particular case. But it could show up again
> if an internalized component was registered as, for example, a transient.
> It does not seem to be in the saga persister though so maybe it's not
> really an issue, but I don't know about other places.
>
> I'm in the nitpick corner, I know, but I still think that internalized
> components should be hidden from the container as much as possible to avoid
> "leaks". Best way to do that would be to wrap them into another class.
> Then, components would not have to be named at all but could use the
> container default.
>
> 2012/2/5 Andreas hlund <
> reply@reply.github.com
> >
>
> > I tested and it seems to work without naming the instances. Tried
> running a
> > external raven from the pubsub sample and it worked. Can you do some
> > testing and see if there is still an issue?
> >
> > On Sun, Feb 5, 2012 at 10:20 PM, Johannes Gustafsson <
> > reply@reply.github.com
> > > wrote:
> >
> > > Components registered using IContainer.Configure() should also be
> named,
> > > right?
> > >
> > > ---
> > > Reply to this email directly or view it on GitHub:
> > >
> > >
> >
> https://github.com/NServiceBus/NServiceBus/commit/41444d10586be258d1ed40b9941eeb02206c7ca5#commitcomment-934750
> > >
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/NServiceBus/NServiceBus/commit/41444d10586be258d1ed40b9941eeb02206c7ca5#commitcomment-934751
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/NServiceBus/NServiceBus/commit/41444d10586be258d1ed40b9941eeb02206c7ca5#commitcomment-934778
>
It's a common error, at least until march/april xD.
That's exactly what I meant, actually.
Apologies for the delay in replying: I've been moving country!

I usually run PyLint over code I write/change as its meant to enforce PEP-8 standards and as a relative new comer to Python I wanted to learn the general style.  But in this case PyLint seems to have encouraged me to make modifications which are incorrect: upper case global variables and removing the _ from the function names..

Do you know of anything that more accurately analyse's code based on PEP-8?
>For krypt library hierarchy, I understood your concern about bridges like krypt-core-c and krypt-core-java(!= jruby). And I don't want to decode >something about bridge naming and design to refrain from YAGNI issue.

About krypt-core-jruby vs. krypt-core-java: I actually prefer krypt-core-java now over krypt-core-jruby. Especially since after Milestone 1, I'd like to add a pure Ruby krypt-core-ruby to compare performance stats. krypt-core-c, -java and -ruby seem to express their intent better, don't you think? If you agree, I will rename krypt-core-jruby to krypt-core-java.

PS: I'll try 'jruby -S rake coverage' and let you know the outcome.

About the hierarchy: Yes, that's my current understanding now, too. krypt-core is just a spec, while there are several implementations in different programming languages. With this in mind, I'd like to rename the current krypt-core to krypt-core-c and krypt-core-jruby to krypt-core-java. I agree that we should postpone the definition of the "bridge" interfaces until after Milestone 1. They are not really required neither by ASN.1 nor Template, so we should take the time to design them carefully. I would assume that the integration issue (how to include the bridge .so/.dll or .jar) will require some experimenting :)
Also agreed that resources for krypt-core API (especially on the Ruby level such as tests, specs and benchmarks) should go to krypt/spec/krypt-core and krypt/doc/krypt-core.
did you really mean to make the background red in IE 8, or was this accidentally left in from debugging?
Yep. I’m not sure. `Date` class actually represents date and time. Should we just agree to use word “date” everywhere regardless what the class stores?
`return isFrontSideTextCorrect() && isBackSideTextCorrect();`
Can be just `return errorMessage;` after `errorMessage = getBackSideTextErrorMessage()`. Though it is not so important here.
It was a convenience in order to not differentiate between user ivars and internal ivars that need to be reset (but present) between invocations. I'll change it so it resets the internally resettable ivars but removes the user ivars.
The conversion happens once per runtime (it's a constant), whereas the include check happens n times per runtime.
I just downloaded this fork and I am getting crashes on both the simulator and iOS device when I run the demo application.  I have Xcode 4.3 with iOS 5.1.  The demo runs for the most part and then crashes usually within a minute.  I can send you the crash logs.
This fails on google because it defines global chrome variable.
my bad. i just pasted my code over what you had in here, but apparently your code here was different then my code in OM. so this is a diff to an older version of your code.
après mise à jour ça fonctionne, par contre, dans le cas où il n'y a aucune rubrique de sélectionnée (quand on en a pas créé) ça fait une erreur ... c'est normal ?
据说是提供燃料的？是咋回事。
A jakbym podała mu assertTrue(result.accepts(""\"?\"")); to by akceptował (każdy znak, który chce w \"\" bądź \'\') ?
thanks!
I'll do it in a few days.
Super ! On a enfin des tests sur cette partie ! Bon boulot !
What I need to do is add a better error message if you attempt to use the unsupported overload of First and Single. I don't think it's OK to combine a predicate from First or Single with a Where clause. Consider how it might interact with Skip and Take and give different results if the condition is checked before are after Skip and Take are applied.
Guh, I really hate to pick nits here, but you really want to move the assignment of instance variables to between the cancel and the request. Otherwise, we run the risk of race conditions of releasing the callback in the JS thread at the same time invoking it on the main thread, or decrementing the wrong callback request count. By moving the instance variable assignment here, we can avoid threadsafety issues on callbackRequestCount and thumbnailCallback.
If only https://github.com/mozilla-services/android-sync/commit/5041e was in develop, it could have been used here :(
I suspect that we never access one of the two database fields from more than one thread at a time, but I'd rather be sure.

Could you make all three methods in SQLiteOpenHelperCache synchronized?

I think I would also prefer CachedSQLiteOpenHelper.
As lovely as Maven ever is. Please address comments.
@lsmith77 could you add all the deps, that you use with view layer? Then i'll add the deps i use with routing (i think it's only `doctrine/common`).
I'm not quite sure if using `WCF::getUser()->userID` would be better, at least we're always referring to current user.
i wonder why i wrote "Pure virtual class"...
Should KeyboardHandler.java be removed?  It doesn't look like it's being used.
OHHHHHHHHHHHHHHHHHHHHHH so that's how it works
The problem is, that to test I need the images and css on my pc, if the images and css are uploaded to the static-server, I could load it up to there and set the links to the static server immediately.
I tried exec.Error, but it gave me errors. This looks like the only way.
なるほど！廃止宣言知りませんでした、ありがとうございます。
ならば使わない方がいいですね。
なんか、rspec中途半端になってきたなぁとふと感じたりしました。
do_beefind [SEARCH [SEARCH...]] -- [SKIPLIST [SKIPLIST ...]] -- [EXCLUDES [...]}
What's wrong with attr_reader here?

or at least just

    def client
      @client
    end
'# bdc -- want to rename this also. Seems to be' - can go also
Weren't we going to make this a private instance property? I believe the only reason we had for people disabling it was because there were edge cases where it was still buggy - which we didn't have time to address? If there's a use case for folks disabling it for reasons other than bugginess, then it should probably stay an attribute.
Do they need to be public?  Is protected sufficient? (I know, I'm such a pain!!)
case Destination(_, `self`) :: Nil ⇒
I agree, suggestion of better name?
virtual nodes?
virtualNodesFactor
you mean the problem lies in their implementation using WeakReferences and their associated extra GC cost, right?
IT's definitely a required entity. An no, it works for virtually all VMs out there, atleast according to Doug Lea.
Absolutely, put in a TODO Rename to timemachine when invented
Doesn't this really imply that this Actor ought to be a FSM actor?
Then I’d propose to decouple it completely, i.e. rename it to `acceptable-heartbeat-pause` [in seconds] in the config.
Is it worth including this case here, or should we rather provide good I/O functions separately so that users can load in their data?  (I'm asking--I have no use-case experience in this regard)
Oh - ok - I wasnt sure it was limited to this use. sounds good.
This function has some problems and needs to be generally more robust:

* We need to check if the resource has url
* We need to check if the url is well formed or at least if `parsed` contains hostname and port. E.g if url is "www.foo.com" or has a typo like "http:www.foo.com" an exception will be raised when trying to set up `resource_domain`
* `parsed.scheme.lower() in request_protocol`: this will return true if CKAN runs on https and the resource on http

Perhaps it would make sense to parse `toolkit.request.environ['HTTP_REFERER']` with urlparse and compare it with the elements of the parsed resource url instead to be more consistent.

It would be great to have tests with different situations
maybe include a description trailing comma in array, object

also an array example would be good [1,2,] bad [1,2] good etc
OK. I'll revert that portion.
Personally, I don't like specifying keyword parameters by position. If okaara ever adds another positional parameter this call will stuff ['-m'] into it.
parse_func=int ?
Here again on the server.conf. I don't have a strong preference one way or another, so pick a convention and let's get the whole guide to follow it.
In fact, if you use `===` instead of `=~` (change operation order) then you naturally get lambda support in 1.9+ as well as string and regex matching.
sure, might even have written it myself once. but now that you're optimising that, this is just one more :)
здесь стабулируй не по родителю htmlOptions, а по class, тоесть class и rel на равне
Totally refactored so no image data gets lost and results in poor image quality
Why raise here -- why not in the `to_indexed_hash` where we are doing the `case` anyway?
Copy/paste error on the naming
why not 0.4.8? 0.4.2 had problems on windows
> it's immutable in master, please verify diff since you branched off to  
> another file (I don't think there should be much)

Ok, thanks!
Why was this changed? It was probably unretained because the code fails to nil it out in some cases or something.
What is this? You're not only ignoring the return value of that function (and thereby abusing it), but you're also drawing into nothing.
May be "Choose a Postgres version to remove or check the options." ?
Drop err if you're not actually using the return value of getpwnam_r().
@sblom Yes.
`static_cast`?
all of these separate var statements make me woozy.
J'ai juste regardé ce context. Avertis-moi si tu as modifié plus que cela.
op3 isn't that descriptive, maybe ind or indices?
Can't declare variable here, it isn't portable C.
Perhaps we could change this to SolrResponse.from_json() as a classmethod? -- that's generally where I store my "alternate constructors"
Yeah... maybe they'll fix this in a future version of the API/gdata library.

I don't think this is a huge issue, though, since we do not allow empty lines in the middle of mapping files in ```qiime.parse.parse_mapping_file```.
тоже сгенеренный файл
Если код не нужен, то выпиливай.
If you are going with my proposal, I believe we won't even need the setter (and the `has_index?` and `attributes_with_index` methods too).
ruby-debug specific URL
I think is better to talk About JavaScript being minimized not CoffeeScript
I think we should not write the rounded time in the database, we should only round they to compare. WDYT?
This change in particular shows how `each_with_hash` could be valuable.
Exactly, I can say `:root => false` when calling this, and with the refactoring, it'll always be using `include_root_in_json`. You could use `options.fetch` instead.

In any case, I think there's no need to change the code, the best refactoring I can think of would be to remove the `try` and use something like `options && options.key?(:root)`, avoiding the argument hash.
Your uncertainty as to whether it does what you hope does nicely underline the value of specifically targeted tests.
Note:  I wish we could use implicit class here...  That would make it simpler to do an isInstanceOf later....
What is wrong with underlying.clone? (if Clonable)
Yes, I was taking that as a given. But I agree that the comment above could also be improved.
If you just push another commit to fix this up to the branch of the pull request, it'll be added to the PR automatically. Then I'll rebase all three commits into the one and that'll be that.
This is for the download method so that we do not read the whole file into memory and then write it but do it chunked.
you say "to" twice here

The second sentence should probably "As a rule"
I never really understood how this is supposed to work. What about `cosh(1+x)`?
Oh.. My bad. I saw only the name of this file and thought it was part of the tests.
If I understand correctly, `UnitSystem_whatever_whatever` is a class having some methods to define global options, that the UnitSystem (and Unit, Quantitiy) class(es) will use?
What is the difference between `array_form` and `_array_form`?
as in?mention the algo?
Presumably the above line can be removed.
Yeah, I read this article and even recommended it on [StackOverflow](http://stackoverflow.com/questions/9735336/event-machine-for-parallel-api-requests/9735693#9735693) :-)
If we're going to have a comment here ( @bergie - I don't see a lot of comments - so not sure if you have a general standard not to include them or only include them when really needed) - then it would be a bit more accurate to read `if the link does not have "://" in it, prepend with http://`
`meck_ret_spec` specifies how to calculate a value that is returned by an expectation. `meck_expect` defines an expectation that is defined by a list of meck_args_spec/meck_ret_spec pairs where every pair is akin to a function clause/ So these are two different abstraction though the latter using the former as a building block.

In general I create these different modules not because I just like to have lots of files :-), but because I prefer to have distinct data abstractions with clean API and hidden implementation. I believe that greatly improves readability of the code..
s/Diffa/LShift Diffa/
Where does this constant get used?
elimite complex conditionals? 

potentially move an expanded and simplified version of this to the storage faq?
As commented below, this is indeed a bc break, and will be documented as such. 3.1 has various bc breaks, and we shouldn't keep old functions around which will be poorly maintained if at all, if one can easily switch to the new function.
`your` -> `you`
No need to abbreviate. The language key should use the full word ATTACHMENTS
What happens with IE10 here?
No, hence the point of having post and posts.
Yeah i guess it should be just - " There were no extensions found" ?
This is not a complete last post. Maybe "last post append" is a better description.
This looks strange. You are using the parameter depending on the type of a property on a third object? Why not use $timezone if it's not null and $this->user->tz otherwise?
There are quite a few more occurrences of <?php and ?>. I won't be touching any of them as clearly they must work fine.
Call it sql_case then?
This should work, but javascript will "hoist" every `var` declaration within a scope (ie, function body since you don't get block scopiing), so I find it more clear to do all declarations at the top of a scope, and a lot of people/linters like to do it all with one declaration. So at the top of your function you would have:

```
var i, j, k, contacts;
```

Its functionally equivalent to what you have now, it just makes clear that a single variable is actually being re-used on each iteration of the loops that it appears to be declared in (again, doesn't really matter since you're initializing each of them on every pass).
I doubt whether we should have an executable here. Only shared libraries I guess. Test application etc. should be moved to examples folder in advance.
`other` is commonly used for these types of methods since it's not known what the rhs is.
`:updated_on` and `:expires_on` parse sections that existed before I've forked a repo, worked exactly the same. Should I do something special here?
We needed to add this because the dispatch-helper does need to do a proper YUI dependency calculation.  Considering the significant differences in implementation, this should probably be split into two methods.
find picture -- else, remove the code
I'm not familiar with this syntax... what is it?
```/Pharmacy Requisition/```
or is this not yet in use?
https://raxaemr.atlassian.net/browse/TODO-23
Can we just ditch this note? I don't think it needs this prominence.
Is this a little bit of magic? Does this only make sense if the x and y coordinates are latitude and longitude?
I'm not sure I follow correctly, but idx can contain duplicate indices and can be in any order. Do your code work in that case?
english: convertable
I think extend should call attach_feature instead of ignoring the feature.
Do you want to use _abort in these methods instead of abort to get the extra message w/ file name?
api_url.rstrip('/') might be better
({ block: 1, elem: 1, mod: 1, elemMods: 1 }).hasOwnProperty(k) быстрее чем итерация в indexOf
Здесь нужно написать комментарий про то, что потенциальна возможна ситуация, когда `MagicNode.MagicNode` вернёт ссылку на «новый» класс `MagicNode`, от которого не будет унаследована нода `arch.getNode(p)`.
Is SchemaEditor the right name for this? most of the other backend things have less action oriented names.
resolveParams is not static
sounds like a good idea.

Although there is another PR with similar changes and assets-related gems not listed in `:assets` group( https://github.com/errbit/errbit/pull/323/files#L0L107 ).

Perhaps we can review these two PRs without introducing and have another PR which moves all those gems into `:assets` group afterwards? It requires a bit of configuration changes in `config/application.rb`.

I guess git history would be a bit cleaner then, what do you think?
use Transport:name() like cowboy_http_protocol?
can remove this property. Anyways its not affecting the code as the window always opens an i-frame.
What does the code look like? The type of `id` should be `llvm::Intrinsic::ID`, not `size_t`.
Sure, that works too. In that case, `parse(T, s)` is better for the single-object case.
I don't get the problem (not following your answer I'm afraid :/) but I don't think we should have a hack hidding the actual issue and I'd rather we don't go around a problem in the globals leak detection code.
@dmethvin Correct, we switched to no spaces around single string args.
Eclipse told me it was :). Looking at the signature, it should be a link to com.badlogic.gdx.graphics.Color, so I changed it along the way (and seems to work). As far as I can see, there isn't a class named Colour. Can you explain why it isn't a typo?
Slight discrepancy from the documented behavior.  

Documented:
>  @param df Degrees of freedom. This is the number of events reduced by the  degree of freedom lost by using the observed numbers for defining the expected number of observations. If this parameter is \c NULL, the degree  of freedom is taken as \f$ (k - 1) \f$.

Actual:
>  @param df Degrees of freedom. This is the number of events reduced by the  degree of freedom lost by using the observed numbers for defining the expected number of observations. If this parameter is \c NULL or < 0, the degree  of freedom is taken as \f$ (k - 1) \f$.
it seems that the get_or_create route is the best way to go, so I'm headed in that direction (unfortunately there can still be race conditions if I move the marking to the spider itself because of its parallelization)
Thanks for the heads up! Modified and tested.
Is this safe to call on an already-fired deferred?
This is one the the changes I was the most in doubt over. but i don't think this much information about the default settings is needed. By far the most people use PRAW to connect to the reddit reddit, for the few people who need to use this for other reddits the information about default reddit is clearly readable in the code just below.
This line is hard to understand and should be rephrased to:
*The result is that in case of an error, the requested component is replaced by the error component.*

Also, it does not make sense for it to be a separate paragraph and should either be included in the statement above or be given some context.
Check spacing here.
Use enumerate()?
Is this step necessary? Or do we get the same results as from getInjectedClasses ?
`terms`, `addends` or `summands`. I think those are all worse.
Could you make the indentation match the following three lines? Also, apparently JUnit style is to not use the braces if there is just one line inside the "if"
I don't even like what it's doing, much less how it does it.
I didn't know that, but I actually agree. box-shadows are quite slow, even when they're small like this.
That should work. However, if the game is too long, we will end up with an array filled mostly with `null`. You should file a followup bug to improve this.
Nah, we can do that later.
Hmmm, okay, I'll buy that.
These first 3 strings are unused, see #2252. Please remove.
toll, prima!
I wonder if we should do anything to round this to an int?  This number basically gets pushed straight onto a DOM node's scrollTop... are there any display/performance problems that could arise with a non-integer scrollTop?
May want to "expect(hintList.length > 0).toBeTruthy()" before accessing the first element.
fixed

From: Jason San Jose [mailto:notifications@github.com]
Sent: Thursday, August 16, 2012 10:49 AM
To: adobe/brackets
Cc: Ty Voliter
Subject: Re: [brackets] Fix code hinting that broke due to localizing index.html (#1378)


In src/editor/CodeHintManager.js:

> @@ -36,6 +36,11 @@ define(function (require, exports, module) {

>          PopUpManager    = require("widgets/PopUpManager"),

>          ViewUtils       = require("utils/ViewUtils");

>

> +

> +    var hintList,  /// initialized by htmlContentLoadComplete handler

Remove extra \

—
Reply to this email directly or view it on GitHub<https://github.com/adobe/brackets/pull/1378/files#r1395437>.
Isnt this call already safe?
I wonder if a ``# comment``-style comment would be more appropriate here than a triple-quoted string. (I believe triple-quoted strings are usually used for documentation, rather than code explanations like this.)
Each of your client types is implementing almost the same logic for this function. This should be factored out into a base Client class that handles the fundamental parts and then just uses overridden methods to separate out the difference in logic.
that would be nice. I used the same pattern as you had used (somewhat) with the servers on the clients.
This leads to ugly syntax highlighting in some editors, please revert it.
Maybe use guava's Objects.equals() - for this and the other equals() impls - (example here http://www.javaworld.com/community/node/8189)
And I was trying to be careful to follow your style too.  Fixed this in another commit, as well as others in the same file.
Sorry. Let it be `message`, as it is in Facebook.
We can probably just compact this spacing. `({ ... })` is fine, right?
Right, and that's accurate. But, it looks like it is (or recently was) used as a way to clear the canvas as well. So, in effect, you'd be clearing it twice. Can we set the width less often than every scroll update?
Just curious about this comment title style. Where did you get it from? Is there something else we can use that looks like something we're already using?
Remove two of these new lines.
We need a solution for when the user passes us a `image/jpeg` too. Can S3 do content-type auto-detection, so we don't need to specify this? Or do we need to modify the S3 store somehow?
new line will look better before `"https`
I would expect this patch not to change the pdf.js behavior in any way. Also, the console.error is not throwing the exception. If we are not using it, let's remove it.
That's actually intentional. The $client_ip variable is referenced before it's used elsewhere in the code. PHP doesn't consider this a fatal error, but does emit a notice, which I corrected by initializing the variable before it would be referenced by other code. This will be overwritten later when it discovers the client IP address.
Is that logger name correct?
And didn't we turn off the LogService in sonatype/restlet1x#3? So muting it here will do nothing.
The problem is that I need to set width of the container to some larger number than usual (otherwise the dependency graph looks horrible) and if I include it in the localResources the width of the other pages is also set to something large.
Yeah, this was a mistake, we should not merge this. I'll exclude that file now.
I think that better condition is !=
Caught that, and the new commit anyways gets rid of all that.
this string is used for second time.
can you  please put it in cli/src/katello/client/constants.py
Same story for the rest of the exceptions I guess...
What is the reason of this construct?
It will be in next commit :)
note, you could also use this.apply(undefined, arguments) if you didn't want to explicitly declare the arguments for callCallback. though this may be your intent.
This says the same thing as line 24
I cheated a little. With the current approach where we first identify the
best plans for the individual selects and then combine them into a best
plan for the whole union there always will be one and only one winner. I
simply call it "union". So, the final winner plan is WINNER-union.txt.
Also, as you rightly noticed, the file names for the  individual selects
are not unique and get overridden, but the individual winners though are
incorporated in the WINNER-union.txt.

This is indeed a departure from the current behavior. Probably better to
follow it as is.



On Mon, Sep 24, 2012 at 11:04 AM, Paul A Martel <notifications@github.com>wrote:

> In src/frontend/org/voltdb/planner/QueryPlanner.java:
>
> > @@ -68,6 +70,13 @@
> >      // generated by plan(..)
> >      boolean m_wasParameterizedPlan = false;
> >
> > +    class NextPlanParams {
>
> How are you then accounting for the counter that establishes unique
> diagnostic output file names?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/alexeevm/voltdb/pull/3/files#r1673859>.
>
>
why do you have to modify this CSS stylesheet?
OK. So, you want it to be like this in utils.py3

```
if sys.version_info >= (3, 0):
    encoding = 'unicode'
else:
    encoding = None
```
and it being used as 
```ET.tostring(data, encoding=encoding)```

OR
```
if sys.version_info >= (3,0):
    ...
    def tostring(data):
        return ET.tostring(data)

else:
    tostring = ET.tostring
```

(ideally the second one should be in utils.xml.py
OR, we can use a combination of both. (using 'xml_encoding' in py3) and tostring() in xml.py
This (and all the methods below) should follow the docstring standard for functions.  The main things you don't have here that would be useful would be a "Parameters" and "Returns" section (see http://astropy.readthedocs.org/en/latest/development/docguide.html#sections).
I see another pattern here :-)

    uiInterface.ThreadSafeMessageBox(str, _("Error"), CClientUIInterface::ICON_ERROR | CClientUIInterface::MODAL);
    uiInterface.ThreadSafeMessageBox(str, _("Warning"), CClientUIInterface::ICON_WARNING | CClientUIInterface::MODAL);
    ....

Wonder if we can do anything with that :)
Shouldn't this function get called if this line is going to be here?
The code is good, but we need to be more descriptive on what are you trying to do.
For example at this line you rely on PHP auto-array initialization, while you could perfectly create an array.

My suggestion:

```php
        $filename  = $this->getFilename($id);
        $filepath    = pathinfo($filename, PATHINFO_DIRNAME);

        if ( ! is_dir($filepath)) {
            mkdir($filepath, 0777, true);
        }

        $item = array(
            'lifetime' => $lifeTime,
            'data'    => $data
        );
```
At the line below. That's a leftover from a previous experiment, so I'll just inline it.
Two blank lines between top-level declarations
I'm not sure if this is actually an `int`? We should run some checks on the actual tree.
OK sounds good.

On Thu, May 31, 2012 at 12:40 PM, Thomas Kluyver
<reply@reply.github.com>
wrote:
>> +
>> +        The contents of the cell are written to a `.pyx` file in the
>> +        directory `IPYTHONDIR/cython` using a filename with the hash of the code.
>> +        This file is then cythonized and compiled. The resulting module
>> +        is imported and all of its symbols are injected into the user's
>> +        namespace. The usage is similar to that of `%%cython_pyximport` but
>> +        you don't have to pass a module name::
>> +
>> +        %%cython
>> +        def f(x):
>> +            return 2.0*x
>> +        """
>> +        args = parse_argstring(self.cython, line)
>> +        code = cell if cell.endswith('\n') else cell+'\n'
>> +        # distutils.Extension cannot handle sources that a unicode
>> +        lib_dir=str(os.path.join(self.shell.ipython_dir,'cython'))
>
> `cast_bytes_py2` casts to bytes (which is the same as str) on Python 2, and doesn't do anything on Python 3.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ipython/ipython/pull/1770/files#r910556



-- 
Brian E. Granger
Cal Poly State University, San Luis Obispo
bgranger@calpoly.edu and ellisonbg@gmail.com
I don't think I've seen it called the 'definition header' before - I'd describe it as the 'definition' or 'signature'.
No, and it seems it doesn't actually work in the notebook. I'll take it out of the list.
I like the idea of allowing the `_repr_foo` methods to return a sequence of things to display.  There really isn't any reason to not do that.  And we should be able to support that without breaking existing code because all of the existinng methods should return strings.
BTW, I left the double-space, as it was deliberate.  When text is typed in monospaced fonts without automatic line justification (as is the case in code), I like using the tow-spaces-after-period rule, as I do find it more readable. This topic is actually [hotly debated](http://www.slate.com/articles/technology/technology/2011/01/space_invaders.html), but I think that in *code* that is never flush-justified and is almost exclusively read in monospaced fonts, it is indeed an improvement. This approximates the behavior of high-quality typographic software such as LaTeX when it justifies paragraphs.  LaTeX makes the space after periods larger than the average inter-word spacing assigned to that line (by roughly a factor of 2x, as best I can tell by inspecting LaTeX-generated PDFs under high magnification).

In summary, I go with Knuth :)
correct all instances of formatting.  jclouds uses 3 spaces 120 column lines.
are these only formatting changes? is please revert if not please undo reformatting...
change to BaseOAuthModule
there shouldn't be a exception here.  none of the commands above will throw that.
You're dropping errors that checkIndex finds.

You probably want something like this:
```coffeescript
checkIndex (err) ->
  return callback? err if err
  # ...
```
Lots of unedited merge conflicts (!)
Yes, as I was `git add`ing it, I thought to myself "maybe you should rethink this." I'll fix it. :smile:
Good point. There shouldn't be a need to catch `GIT_ENOTFOUND` explicitly. Nothing other than `-1` or `GIT_ENOTFOUND` should be returned by the config function anyway.
Happy to add more. I’ll put on my “Got Tests” hat – which I really do have.

Am I correct in understanding the two existing $idx/$len tests don't belong in the grammar tests and should also be moved?

Rich

From: Veena Basavaraj [mailto:notifications@github.com]
Sent: Wednesday, August 15, 2012 2:23 PM
To: linkedin/dustjs
Cc: Ragan, Richard
Subject: Re: [dustjs] Make $idx/$len work for all array objects and not leave old values in contexts (#113)


In test/jasmine-test/spec/grammarTests.js:

> @@ -70,6 +70,20 @@ var grammarTests = [

>        message: "should test an array"

>    },

>    {

> +     name:     "array",

> +     source:   "{#names}({$idx}).{title} {.}{~n}{/names}",

> +     context:  { title: "Sir", names: [ "Moe", "Larry", "Curly" ] },

> +     expected: "(0).Sir Moe\n(1).Sir Larry\n(2).Sir Curly\n",

> +     message: "should test an array with non-object elements using $idx"

> +   },

> +  {

> +      name:     "array",

@jairodemorais<https://github.com/jairodemorais> if @rragan<https://github.com/rragan> is ok, a few more test cases makes sense here

{#.} {$idx} {/.}

more edge cases like these

empty array,

empty map

—
Reply to this email directly or view it on GitHub<https://github.com/linkedin/dustjs/pull/113/files#r1387479>.
len is needed in the loop anyway. no need to call another function.
Why do we need this? Can we narrow the scope at all?
So to use the waffle flag with the simple paginator macro we need to import it `with context`. But I didn't want to find every place it's imported and make the change so instead I just made a different macro to use for the new theme. I'll eventually get rid of simple paginator.
Is this line necessary? If yes,

```py
self.user = user(username='admin',
    is_superuser=True,
    is_staff=True,
    is_active=True,
    save=True)
```

would be better.
I think if it works, we shouldn't have these hashers here. But this is also out of my knowledge area, so maybe it doesn't matter. @jsocol - Do you have thoughts here?
That isn't correct. The ``products_list`` argument does not exist and is not needed. Also, you did not create a new service, you extended an existing one, thus you should not add a new chapter to the documentation but edit the existing one (Products).
I agree with Turaylon, "User" is much better then "Utente", it's what we call "inglesismo" :)
The only usage of this class is when you want asynchronous HTTP requests with callbacks happening on the background thread rather than some main thread. I'm not even sure if there's a use-case for it, really. Perhaps we should just simplify and remove it.
nit: Operations like this one are somewhat expensive. Should we have a per-method cache of all of our computed stuff?
Sure, i will remove it tonight.
you should probably add it as a suggestion too
@stloyd it is not a BC break: it still defaults to passing ``true`` as thrid argument
Yup. I removed it.
This is for tests of  ldap/ldap class. May be we should remove tests of that class.
yeah, but if I see ``DT`` in the code, It is absolutely not obvious that it was standing for Doctrine Type. We are not trying to make the code as short as possible, but readable
You should check for existence of this header. The user might want to specify something himself.
I don't think ~5 lines of code justify that.
Shouldn't an exception [be thrown](https://github.com/symfony/symfony/blob/master/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php#L52) instead as this is related to security ?
よくわからない親クラスをつけずに、ちゃんと用途によって明確にしてください。
That was auto-created by the IDE
in kar 2rose ke bara un image footer ke repeat mishe ye tag zadam ya bayad un image background khode tag **footer** bashe?
jan !!!
copy pase shode in code ha ?
dashboard.create
meshFromOFF
Does this need the `PlateCarree()` instance?
Is het altijd de schuld van de socket?
Whatever. We're bikeshedding on maintainability of 2-4 lines.
Style: space after `//` and use lowercase letters for inline comments (as oppose to comments describing functions etc... I know fine line in js
oups, fixed
Wouldn't it be better to explicitly have if len(self.editors) == 0 here instead of using the implicit boolean?
``mlt`` I bet ;)
although this is the result of the Activities bug, i think i like the idea of only rendering a userRead if the gravatar changed.. do you all have opinions about this?