labofsahil/github-event-dataset-2012 huggingface dataset:
This line fills the array with items that are in the array already. Why recreate the enter array every time you hide one more field?
It`s needed for headers installation.
Yeah please add the directory check/creation here. (ie: http://stackoverflow.com/a/5021245/1811918)
Could look at history.replaceState() - but then again, I think that if you're updating the URL, then there's an expectation that back takes you to the previous one. So I'm not sure I agree with @commuterjoy
You should mention *under what circumstances* it would be useful to provide a custom subclass instead of using the default ones.
I'm not quite sure how useful this test is, but I'm ok with keeping it.
@shane-tomlinson for you: 3cd8065
Great catch. Changed it.
A bit overhead here. I would suggest to directly replace the element and make not use of appendToList.
Should be ``AMQPQueue::unbind($exchange_name, $routing_key = null)``
@Evan-Pierce @jeffweiss Patrick added a shared context in puppet for exercising tests as a specific platform. That way we can stub all of the appropriate bits, even FILE::SEPARATOR, in one place. Checkout `spec/shared_contexts/platform.rb` in puppet (at least 3.x). It would be great to do something similar in facter.
Also we introduced "log_and_raise" recently to try to remove the redundancy in this pattern... worth checking to see if that's appropriate here.
This block of tests is very slow for unit tests (the pluginsync, load_path, and "standard files" tests take about 0.4 seconds each on my system). Is there a way to test the puppet side here without having to really interact with the underlying system?
We should probably enforce the proper function naming convention here - that the function is named `function_${whatever}`, not just an arbitrary name.
"set up". Also - instead of saying "not to run Webrick", I'd say "is set up to run Thin".
Where does `top` come from?
But 0x0A is a character! More specifically a box drawing character!
Are you sure it shouldn't be
- if colspan > 0
i.e. without #{}?

Please also use space after -.
The same in the next function.
starts above the return return a(i, j, k, false);?
"Select pr.Id from Packages p 
Join PackageRegistrations pr on pr.[Key] = p.PackageRegistrationKey and " 
+ includePrerelease ? " p.IsLatest = 1 " : "pr.IsLatestStable = 1"                     
+ " order by pr.DownloadCount desc"
Rebase killed the comments but this was changed to guard against Tags being null
Could you revert this line? We want to be targeting .Net 4.0, since we use the Task Parallel library.
This call to localtime() is problematic because the return value is not checked for NULL, and it will not link when attempting to build the application on other platforms. Perhaps you should include a preprocessor check on the platform, and throw an "#error" on other platforms than "native" to provide an error message that better explains why the build failed.

Is there any plan to port this to other platforms in the near future? I would like to point out that on several platforms, clock_seconds() returns the time (as unsigned long) since starting the system rather than the wall time. This inconsistency could be fixed by making native's clock_seconds() more like the other platforms function, and adding a new interface for obtaining and setting the wall time.
Personal stylistic preference. I prefer your version, but do what you feel.
Oops.  I think I know what is wrong: `__file__` is simply never set for clojure namespaces to the right value.  Will have to work on that as well (or you can have a look at `namespace.py` and `main.py` (the import hook part), where the culprit is (I think)).
as the comment incomprehensibly states, all the gettattr can be promoted to command line options by defining them in the RUNTIMEOPTION dict (and changing eg self.scoop_size to self.options.scoop_size in the code)
however, it's safer if they can't be controlled command line for the moment (and i'm not yet sure if the all make sense as commandline option)
try block does more than changing dir
fixed in FSL branch
used os.path.join in the list passed to for
i have checked the doc, and it says 
When [!] is omitted, and {file} exists, this command fails.
but when i remove the '!', it's works correctly. 

confused, any clues?
Let me guess -- Steve wrote this. ;>
Yes -- please do omit the <code>results/</code> directory.
This test should see if the iptools support is available (eg "if slogging.access_processor.CIDR_support is None: raise SkipTest". don't forget to import SkipTest). Right now, the module "works" without iptools, but the tests fail.
Thanks, I replaced it with it by `%s/content/script/` I named it in the sense of a file of content, but the word script feels better for me because the reality of the content is a ruby script. :ant:
ce77fa4
Out of scope of this pull request.
本当にこの時点で all 呼ぶのが必要なのかな? scoped で十分だったりしない?
nit: Can we move the conditional to the arg to prevent dupe of the call? i.e. `return this.scrollTo(0, ($.os && $.os.android) ? 1 : 0);`
did you test this? The second curly bracket should be escaped, no?
Part of the generator. Looks ok to me, nothing particularly objectionable about it.
This will raise an exception if there is nothing stored for given key
```
Memcached::NotFound
```

To not raise error you could've used 
```
@connection.get([key])[key]
```


Also didn't you like log?
```
Integrity.logger.info("Read from memcached for key: #{key}")
```
No, this is correct. I'm setting the property (line 16) to null. But I should remove the parameter of the function, it isn't necessary.
I guess the point is that this is mostly an internal function that should be called by addAngles and intersect, and not be called by an outsider, but I didn't want to embed it in something else because it's too big.

As for the ordering, it follows the form of "starting point" and then "angle from", "angle to" from the previous version. I think you're right though, that passing in a string makes more sense.
song.lrd can return None
I'd prefer it if you didn't nick this from the documentation!
Please don't change this part, this method is used in other sections, the changes will be in the javascript side
if I remember correctly, there are several existing tests that use `+1` instead of `|1`, that's why I use it also.
Y.eZ.Expandable seems to not be used anywhere, could this file be removed ?
это как сюда попало? напилось что-ли?
it seems that it is done to add Hessian dependency. Please remove it.
but https://github.com/lusis/chef-logstash/blob/master/recipes/agent.rb#L93  references shipper.conf not logstash.conf
I'd definitely use `Pathname()` here instead of the is_a. and isn't path always a String after this? No need to call .to_s on the next line.
Nuke debugging print?
See comment above. I don't think it should be up to the user to define this reject.
we should use single quotes here
Oh, nevermind. I'm wrong. :)
Instead of ===? Nope, just habit. I can fix that.
Should we just use the regexp directly from the 2.7 reference manual here? http://docs.puppetlabs.com/puppet/2.7/reference/lang_tags.html#tag-names
In fact, this is not used anymore. It is rendered here: https://github.com/resolve/refinerycms/pull/1739/files#L1R12
This file should be moved out of root into the tests folder as well.
Hm, not necessarily I think. We might either want to put a wrapper around this or put the check in the business logic. I'll create a new ticket for this.
I'll keep the tenant config around as it still needs to have the `stop a tenant server` functionality.
Owwwwww my head. Touche!
That makes a lot of sense, I'll incorporate the changes.
I don't think we should check for 'null' anymore, especially not with the new wrapper around encodeURIComponent. There is a validator function that checks for a valid id format, which would probably be better to use here. That would also remove the check for null.
Be careful that the outer instance (i.e., ``ScalaDebugTarget.this``) can escape here. The actor should really be started only *after* the ScalaDebugTarget has properly contructed.
Here have some appeal to authority:
```
Ilya Grigorik writes:

The best way to get started with ZeroMQ is to work through some hands-on examples - the concepts are not new, but the ease with which you can compose them takes some getting use to.
```

Anyway, I just think of that as an implementation detail. Make a common interface that you can program to can be a first step.
I know that Stevie G contains a mental git repo, but I prefer to hack on code to understand it.
remove the andyb2000 stuff
This test appeared twice. I removed one instance.
Actually, page.start (and thus page()) doesn't really make any sense on the server, so I removed the short circuit check that I added so that page() wouldn't break on the server.
Single quotes must be used to comply with the [coding style guidelines](http://codeigniter.com/user_guide/general/styleguide.html#strings).
Some brackets around `$str` might be good for readability as well:

    return ($str) ? 'TRUE' : 'FALSE';
для чисел есть замечательный NSNum
На данном этапе хватит того, что вы насильно приравняете в этом случае length к 0. Что-то типа

`if (length<0) length=0;`
Верю, но скажите мне - что вы будете делать на 3-й лекции в модуле о Foundation? (-: Вам не кажется что вы забегаете наперёд?
В ТЗ указаны именно проценты, а то, что символов 100 – просто удачное совпадение.
Вы как-то синхронно напали, думал и правда не верно.
Да, если разобраться это не арифметическое округление ка в школе учили, в моем случае дробная часть просто отбрасывается.
round - по идее банковское округление (половина округляется к ближайшему четному)

В задании не сказано как именно нужно - вариантов много ( в большую сторону, арифметическое, попеременное).
По этому и не стал ничего пока выдумывать, пока не уточнили.
Please remove this dummy test. Thanks!
Ah, yea, I forgot, I normally do RegExp with `ig` so it replaces everything. Got it.

Couldn't we remove the dependency on indexOf with just

```javascript
resource.url.replace(new RegExp( '^' + path), yn.paths[path]);
```

or something?
Consistency nit. Use unsigned or size_t  for 'i'.
"MUST verify that its hostname appears in this list."
I see duplication here: https://github.com/codegram/hyperclient/pull/3/files#L5R36
Heh.  =)
I was thinking a line addition, don't like how it adds 10 lines. Maybe a silent failure is better (as you originally had). I do like the testing of the wait number though. I would comment the unary plus since it's not obvious what it does (for everyone)

```js
// +wait -> Unary Plus,  evaluates operand and attempts to converts it into a number
var timer = (wait && !isNaN(wait)) ? +wait : delay
```
Good point. I will add that in.
0 or 1 is not very pythonic. How about out_type='mask' | 'indices' ?
hmm.. after looking at them as well, I agree that there is definitely a strong component at around 300ms, so the values you use are probably more sane
Ahh, I see. However, given that we're essentially hiding the pysurfer calls from the user, I think it makes sense for us to use 'subjects' so users of our package have the most consistent usage patterns in our functions. (Imagine if in the future we add functionality from another package that had a different name for 'subjects', users would have to remember a third convention...)
Wyclif, it's a good first pass. I'm done reviewing. I can do another round when you clean it up a bit. Please try to use static imports for Mockito methods to make tests more readable. See http://stackoverflow.com/questions/288861/eclipse-optimize-imports-to-include-static-imports how to configure Eclipse to do that for you.
Why do you comment this out instead of simply changing?
I did not comment. This Java class is full of commented out code.
This is not correct. startDate and endDate may not be in a format that parse expects. I'd validate it on the server side.
hi David, i implemented a wrapper class for event monitoring logic. could you kindly review it? Thanks
good question. addressed.
Worth mentioning to translators that the sentence continues with *Log in* and *Sign up* precedes.
Yes, I prefer the backticking of the classes as it makes the code more readable as instantly recognisable as something special. making then `::class::` links initially and then backticks is :+1:
To setup the next PHP code block, use 2 colons, like this:

    filesystem operations::
should say "Queue this participant is a part of"?
Don't you want default broker etc here?
This can be too small for a slow box, write the tests so that the timeouts won't be giving me a headache when we run them on a slow integration test box.
Not recommended. Avoid creating tons of top-level actors (just as you wouldn't create tens of thousands of top-level files on your harddrive)
I'd do:

val connectionFactory = {
  val c = new ConnectionFactory
  c setUsername username
  c setPassword password
  c setVirtualHost virtualHost
  c
}
Classes need to start with one and only one capital letter.
"1 dzień temu" => "wczoraj" lub "dzień temu"
Do we have the download URL for git?
``null`` as all other php keywords should be used lowercase
Please keep first line of docstring on the first line (no blank line after quotes). Most doc processors expect this.
I don't think that setting sizes to 0 is good idea, any reason for that? :)
I think the whole point here was to not let users delete "root", so it's worth to have that in the conf to test. This is just the default conf, as long as the default in code in [] it's fine.
Should be removed really, forgot to cleanup.
Should save the return value and call get() or cancel() at some point to ensure the thread doesn't continue running after the test has finished.
I was basically just taking the lead from the old code. I didn't want to change too much jazz.
Stylistically, throughout this PR it would be preferable to not put spaces between function/method keyword argument definitions. e.g.

```
cset = ax.contour(X, Y, Z, zdir='z', offset=-100, cmap = cm.coolwarm)
```

becomes

```
cset = ax.contour(X, Y, Z, zdir='z', offset=-100, cmap=cm.coolwarm)
```
I'm OK fixing that, but I'm not sure what ``o`` is yet. I'll do it when I have a better understanding of this code.
self.axes is used all over the place in Figure; it is a read-only list of axes.  It is being tested for emptiness.  I could enclose it in len(), but it would not add anything.
This will have a global effect on the tests and will need to be undone at the end of execution. Perhaps: 

```
try:
    mpl.rc_file(fname)
    assert mpl.rcParams['lines.linewidth'] == 33
finally:
    linewidth = mpl.rcParams['lines.linewidth']
```
Is there any need to modulo the result incase start angle is greater than 360? Does everything still work if a negative angle was given?
Why not try a wait for condition here for something you'd expect to see on the page?
@dooremont Can you translate this to French as well? You won't see this on the page but screenreaders might read this
For anyone else who wonders why this is `accesskey="6"`:
http://en.wikipedia.org/wiki/Access_key#UK_Government_recommendation_for_access_keys
@arnodeclercq We should use 4 spaces for indentation.
Note: I will be reviewing what client code calls createActivity. It is unclear to me where this happens and whether it is even realistic that a non-Admin user would do this.
For brevity, please change the block below to `SparseUtils.logoutQuietly(adminSession)`?
Why is there no spaces around the "="
One char varriable is evil (except iterators).
Shouldn't these be `v2.0.14`?
This is OK for now but we should have the acceptable certs read in via file that comes in the zip file.
Are you sure ``array(__NAMESPACE__ . '\\Non\\Existing')`` has the correct type as "array<integer,string>" is not a sub-type of "array"?
Arg, sorry I forgot, I'll pull it in ASAP.
needs a hasOwnProperty test
Can I remove the commented code? It looks useless...
Sorry, I checked and it looks like the content negotiation was removed recently. The explicit `.diff` and `.patch` URLs are the only way to access this content.
Aren't we using pytest-mozwebqa==1.0 now?
I think that it would be more efficient to have this on the document, rather than every `.postbody`. Doesn't matter that much, though.
I will just remove this from subscribe signal.
I thought of linking to it for every venue, for additional information.
That's the funny thing. I did but I probably opened those that aren't seperated :)
No, use [startswith](http://django-missing.readthedocs.org/en/latest/templatetags.html#missing.templatetags.string_tags.startswith) tag. No need for complex regex.
Please split this into multiple lines. You already have if-block. So just continue using it.
Is really `box` class name the most informative name for this div possible?
true. Removed it in an update.
Should the else case be the default '', instead of '_parent'?
It doesnt matter for a single string I think
Can we create a URI from $agency_class to make it resolvable?
the reason why you needed to override `returnRadiusAccessAccept` should be documented more thoroughly in the POD above.
that could be useful to issue a warning in case we're not able to listen, nope?
That's right will remove it
Check ``return``
The just NetworkLogger sounds much better.
open question: make wexog and pinv_wexog sparse ?
If the number of observations becomes close to the number of features, we may get a singular covariance matrix np.dot(X.T, X).
Leftover from the merge, I still need to update the docs.
这里改成中文：“安装nginx”
Super minor, but this could be simplified further to be:

```ruby
  SYMBOL_TO_STATUS_CODE = Hash[HTTP_STATUS_CODES.map { |code, msg| [msg.downcase.gsub(/\s|-/, '_').to_sym, code] }
```
new System.out added?
I guess numpy must be using a more optimised library on my laptop install, since my output for your test script is:
```
time for append method =  0.626485109329
time for meshgrid method =  0.1158618927
time for comprehension method =  0.667164087296
time for double comprehension method =  0.343770980835
```
(funny how we're both nerdy enough to go and write the timing script).

But I'm happy with using the double comprehension for the code if the speed increase available between the two choices is at the level of being only platform dependent.
This ```__init__``` suggests that a valid keyword parameter is ```half_light_radius```, yet this is not listed as a ```@param``` in the documentation.  Is this a case of out-of-date dox?
drop a sample in here.  "en-US" or whatever you are expecting.
sure, that's fine.  it can be part of #1576
See updated push, I pulled in 2 checks from the repave, the check of hostname and against the databases.

Is there any reason we *need* the default database name to be class2go? If not I propose one more change, making the default dbname on jenkins be something else, (e.g. c2g-jenkins).
I agree. I will update the javadoc to explain how the command is handled downstream.
I prefer not to do import java.util.*
I don't think this should include `!findUrlMappings`. If a url mapping matches just /foo, then a request for /bar should result in a 404 - which I believe it did before.

Also, the 404 matcher accepted any domain, where as the serverPoolAddresses is only for the specified domain (e.g. localhost will be routed to the server pool, but localhost1 would not be).
The question is if we should change the declared type to `objc.Class` or not.
Missing "broken" test
And who suppose to fill this stub?
Side effect. If you don't have any user service here, it's possibly to use genesis with header, for example. X-On-behalf-of: genesis. If you remove check for user service, you should at least make user service as not option to not start application where user service is absent.
the issue is that there are one of two choices for someone who already installed the app, both of which are not good.

1) fake the initial migration, and not have the new column 'blurb' and have a mismatched state and manually fix it.

2) attempt to run the initial migration (while having the original schema this app had) and migration fails because the table exists.

What I'm saying is that you can redo the migration files--start from the original app w/o schema changes, create an initial migration like that, and then make  your model changes and create 2nd migration.

This way, someone who already has the app installed can fake the initial migration w/o losing any new columns you put in.

Does this make sense?  It's about backward compatibility, that's all.
The body of the conditional comment is empty. I guess that you want to include the `html5shim` thing only for IE. That line should be indented in.
This is dangerous, the function signature is interpreted once. You need to put None and then in the function set to {} if it's None.
Actually: make sure this supports live bindings so if someone dynamically adds a new `<a href="#login" class="browserid-login">Login</a>` tag after `document.onReady` it still works.

Usually that's done with `live('click', [...]` instead of `bind('click', [...]` -- unless newer versions of jQuery are smarter than I know about.
added a new file sorry for the confusion
`instanceof Array` isn't reliable.  Use the underscore isArray function.
Curious - why do we need both constructors?
Changing this search from realtime to non-realtime significantly changes the nature of this test. You might not be testing the same thing anymore.
тут було б логічно зразу список проектів в змінну витягнути
going-back sounds a strange name for a class. Sounds like a transitional state.
Before @stof mentions it: trailing newline?
SSL contexts shouldn't be reused across connections, see ed7efcd.

So it's probably more appropriate to pass in factories directly.
Pffft xD
Expanding on the referenced approach, was able to remove instantiate_collection from models/base.rb.  https://github.com/cwolferh/thor-cli/commit/98149e26bc971130bf80f4928c025d4354798efc
Лучше закрыть `<option value="5">*****</option>`
nope. currently validate() is only called within ghana's app, nowhere else.

On Thu, Sep 27, 2012 at 5:42 PM, Cory Zue <notifications@github.com> wrote:

> In logistics/models.py:
>
> > @@ -1172,6 +1306,13 @@ def __init__(self, sdp, report_type, message=None, timestamp=None):
> >          self.report_type = report_type
> >          self.timestamp = timestamp if timestamp else datetime.utcnow()
> >          self.errors = []
> > +        self.validator = validator
> > +
> > +    def validate(self):
>
> does this get called anywhere in the framework? would just need to make
> sure to override if so
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dimagi/rapidsms-logistics/pull/3/files#r1712533>.
>
>
messageCount надо проверять
Опять же, формат нарушен.
Again, English with the trans tag.
One'd have to change this file as well, at least to update the option description and `info_template` method to add new integration info, so I think we can have a check around here as well.
If it is set to True, you can pass params unrelated to the kernel and you won't get an error. That saves you a few if statements.
This comment is confusing if there's no support for sparse matrix.
this is dangerous not to bound computation time a maximum of iterations
no point conflicting with itself - this should be puppet3
Just interesting from where you get those strings?
Also I think it will be useful to have these string variable in format to somehow show us, that they are exception strings, so to identify them from other string constants we could have in future.
Do we plan to have/use error codes for each strings?
uncomment this gems
Better to have DUMP_OP
or CASE_OP_TO_TEXT(x)
In my current implementation, I have this line commented out. I figured having a configurable feature is a better approach.
I don't see what `versions` has to do with it. 

Either way, what you have there would cause a compilation error in statically typed languages. A possible solution would be something like this:


```python
def ...:

    version = None
    for release in request.currentversions:
        if release['product'] == product:
            # For crash trends we only want the latest, featured Nightly
            if release['release'] == 'Nightly' and release['featured']:
                version = release['version']
                break
    if version is None:
        raise NotImplementedError("request.currentversions did not contain a featured 'Nightly' product")
```
As above, we can just reference this by ID
Let's set this to False, so we can keep paginate()'s default at per_page=None, honoring other paginator's defaults, while avoiding forced pagination-with-no-pages.
I am sorry. It is the pull request for cocos2d-html5-tests.

Before we decide  to remove ctor or not, I think it is better to keep this._super() in this.ctor() function.

```js
var TestScene = cc.Scene.extend({
		
     ctor:function (bPortrait) {
	  	
        this._super();
```
do you need to pass the driver? isn't it already init in the custompromotionpage class?
@bodepd ok, will do
I don't think the default is true. So you probably don't need this option anyway?
There is no difference between this and AffineTransform()?
This can be written as `return this.gut.ExitCode == 0;`
Matt, what do you mean by keep two different structures?  You should totally be able to use an array here.  You also have to be careful about using 'for in' because you'll iterate all prototype members, so you need to call "hasOwnProperty" as well.  In general, the whole thing is slower, and we should definitely be using arrays.
Damn it, I'm better than that.  If you were my high-school English teacher you would refuse to review any more code until this was corrected ;)
I probably didn't adjust the line length for comments, you will have to format this manually
This exact same check (and the matching `!legacyModeActivy && ..`) are performed at many times in this method. can we simplify these two states as either saved variables or method calls?
A general comment: if we really care about (in this case source code space) saving a handful of characters (by using `", i"` instead of `"var i"`), why do we adorn the code with all these `"\n{\n"` and `"\n}\n"` in the case of single-line bodies? I.e., we should then have `"for (...) foo();"` instead of `"for (...) { foo(); }"`.
Actually, on rereview, we could shortcut a lot of this. If we have no listeners, is any of this logic necessary or used?
Аналогично

```
$.post('/dz-7-jquery/current-event.json', result.serialise())
    .success(function() { alert("success"); })
    .error(function() { alert("error"); })
    .complete(function() { alert("complete"); });
```
http://api.jquery.com/jQuery.post/
Should probably comment that this is doen in the event that document does not have an initialized object at the current_key
Because this code was never run before I suppose.
Initializing as `[]` might probably also  work.
make_random_graph
That `&` should probably be `&amp;` -- unless that ampersand get converted to an `&amp;` when this is (presumably) inserted into the DOM.
The whole point of this branch is to avoid decoding. It compares the code units, not the code points but doesn't return a slice including partial code units if not all of the code units in a code point match. I could actually (and probably should) just change this to use `==`, because this branch is only hit of the predicate is `==` (anything else _would_ require decoding). The branch after this one decodes if the string types don't match or if the predicate isn't the default one.
not necessarily. eg code id'like 
```python
ignore_seek="-DMPICH_IGNORE_CXX_SEEK"
cflags+=' ' +ignore_seek
```
<b>キャッシュ：</b>どちらかというと副作用的な利点となりますが、場合によってはとても便利です。理由は、Promise API(例えば、.done(...)や.fail(...)など)は非同期呼び出しが実行された前でも後でも呼び出せるからです。Deferredオジェクトは非同期呼び出しのキャッシュ処理として使えるのです。例えば、与えられたリクエストに対するDeferredを追跡しつづけ、無効でなければマッチしたDeferredのPromiseを返します。これの優れた点は呼び出し元は既に解決されたのか、解決の途中なのかを知る必要が無いという点です。コールバック関数は全く同じ方法で呼ばれます。
Access to this method needs to be synchronized.
I think it could be pretty interesting if you could replace the annotation with something in the manifest.mf
pls check if we can remove that line
I'd like to see this be made an option. If you like, I can work on such a patch... I guess the thing to do would be submit that as a pull request against your branch (in your fork) you're using here? Or wait until this is merged and submit a pull request against this repo?
I don't think we should install it on the Jenkins agent
I agree that there is a problem, but can you clarify what you mean under conditional dependency - do you mean just make it configurable whether to depend on it or not?
Isn't the above `storageManagerCache` @ line 87 a duplicate of what's at line 81 just with a different scope?
Do the test refs need to be in the sample project?
what parameters?
what happens in browsers when you click an a without an href? honest question i've not tested that in ... over a decade
This still does not account for namespaces correctly, e.g. `<m4:metalink xmlns:m4="...">`.
I propose the following regexp: `/^\s*<\?xml(?:.|\r|\n)*?xmlns(?::.+?)?=('|")(?:http:\/\/www\.metalinker\.org\/|urn:ietf:params:xml:ns:metalink)\1/im`
Typo in the test name
OK, how about now?
I think there was something in AJAX that required Prototype that I wasn't able to remove entirely. I'm not sure why it would be involved here, or if my explanation of the situation is actually correct. The "indexOf" at the end of the arrays is definitely the problem.

Also, I'm not sure that using standard index iteration would work, since the length of the array would still factor this in, right?
Ja da stimme ich dir schon voll und ganz zu. 
Da ist nur mein Unwissen schuld dran: Ich ging davon aus, dass wir bereits vorher ein Order erstellen müssen, um die vom Kunden eingetragenen Daten in den Feldern gleich direkt reinzuschreiben, bevor er "Submit" gedrückt hat. (aber hat sich erledigt) :-) Dementsprechend würde ich nun Felder wie Pricing, Address, Document oder offeredPrice gleich im Konstruktor setzen, richtig?
why is this imported?
sounds good.  I must have gotten excited about the freedom to use single quotes, whereas java always required double unless it's a char...  :)  I like your approach.
I'm trying to avoid relative imports deeper than two dots, as they're hard to follow. Might still be some around, but I'm removing them as I find them.  I'd just use `from cc.model.library.case_importer` here.
The possibility to become the very big size exists. However, it is predicted by the general use that it is settled within 520 bytes (MAX_PATH * wchar.sizeof) because the string that shows path that is expected to exist.
I do not have a clear criterion about size of the input that I should permit. 
Simple replacing is implementation by the combinations of malloc and scope(exit).
Which way should I use, malloc or alloca? Please judge it.
MINOR: .h
Its a very good question. One I'd love to answer, but unfortunately I'll have to refer you to my copy and paste buffer... sorry.
see comment on the client pom.
Should this really be the value of remoteFileSeparator as with the outbound adapter. I mean it will work with linux based systems but as I mentioned before we had couple of JIRAs related to FTP operations on the Mainframe where there is no remoteFileSeparator ("") and files are always fully qualified (e.g., foo.bar.baz).
We should do if(logger.isDebugEnabled)
No; I can live with it; if there's a precedent.
Is this needed? removeMessageFromGroup() calss it again.
Rephrase it to:
````
* If this component is defined as the top-level element in the Spring application context it will produce
* an Endpoint as {@link org.springframework.integration.endpoint.EventDrivenConsumer} or 
* {@link org.springframework.integration.endpoint.PollingConsumer} depending on the channel type.
* If this component is defined as nested element (e.g., inside of the chain) it will produce 
* {@link org.springframework.integration.handler.AbstractMessageHandler} to be included in the chain.
````
Wherever persisteMode is being used, it is declared as e.g.:

	<xsd:attribute name="persist-mode" use="optional" default="MERGE">
		<xsd:simpleType>
			<xsd:union memberTypes="persistMode xsd:string"/>
		</xsd:simpleType>
	</xsd:attribute>
What does this edit have to do with removing the `nearby_place` stuff?
Constant string, `'`.
Could you add a link to the SMTP Transport class?
Please use the fully namespaced name here
The same observation as above
Run `npm install` after these changes
Updated. It was also under "My Info" for me.
If we change the url here to `http://webchat.freenode.net/?channels=sakai` it will pre-fill the channel selection box on the client.
We should still be using the `scss` format in here to make it consistant with the code that's already in here.

```css
.api_function_header {
  font-style: italic; } }
```

Same goes for the CSS changes in widgetsdk.css.scss
and the column should be called location_id
That rational is perfectly reasonable, but I'd still prefer the concise method.
So long as the tests cover it I'm happy we'll be able to track any internal API changes to Django, if and when needed. (There's all sorts of places REST framework needs to work with private Django APIs by necessity.) 



On 13 Nov 2012, at 18:28, ludwigkraatz <notifications@github.com> wrote:

> In rest_framework/response.py:
> 
> > @@ -20,9 +20,12 @@ def __init__(self, data=None, status=200,
> >          """
> >          super(Response, self).__init__(None, status=status)
> >          self.data = data
> > -        self.headers = headers and headers[:] or []
> I preferred using the setattr method of Response, because that its more or less the official accessor. 
> https://docs.djangoproject.com/en/dev/ref/request-response/#setting-headers
> 
> Thats the thing that wont change. The _headers attribute.. who knows. And because django and django rest framework are totally different products i'd prefere using this official way - using the
> 
> for key, value in headers.iteritems(): set those
> —
> Reply to this email directly or view it on GitHub.
>
this can cause some conflicts.. 
best thing to do is put this after that the maxHealth is upgraded, and then just health = maxHealth (like stated before by @Borsty)
This isn't a general odata client, it's a community vm repository client.
You've got one major problem here - you're potentially calling callback multiple times. This is a violation of the contract - callbacks expect to be called once. You should either:

1. Return immediately after invoking a callback (I often do ```return callback(message);```)
2. Collect all the errors up into a collection, and return them back to the callback as a single big string all at once.
You can't assume parent is a sitetree object. Comments can be attached to *any* dataobject.
You can use this _event_ to...
a use statement should be added
Also be really careful in your function argument names. You are suing "self" here that is normally reserved for the first argument in a function defined within a class, this is a module function, so it is inappropriate to use it, and you should use something more descriptive like "form" in this case
Done. Modified to provided scope, even for the wls-remote-10.3 module.
data-state=active seems to be old, right?
WHY THE RETURN TRUE OR FALSE?

Just do this:

    return isset($_SERVER['HTTPS']) && $_SERVER['HTTPS'] === "on";
jamesladd, not for you
Why the comment here?
Sorry about that, I'm watching Tosca on youtube with my daughter, since we have tickets coming up. (It's over now, they're all dead.)  I remember now from my experiment, CBF.apply(from) creates a new builder and can't reuse it; that's where that comment came from.  The first thing padTo does is copy, i.e. builder ++= this, which in this case is an unrepentent foreach +=.  Just for fun, it happens to be one of those one-liners.

def ++=(xs: TraversableOnce[A]): this.type = { xs.seq foreach += ; this }

You're right that the result is always toString.  I'll come up with a comment that even I can understand the next day.
use libs-milestone (see INTSAMPLES-84)
Maybe you could reformulate the function like this:


turn_into_nop(int a, int b, const char* why, const char* who); ?

Then do the format only if debugging is on (it assumes we'd only have a single `%s` in there but that might be a reasonable assumption?
Should we init the filter by calling ```pathStart```? (https://github.com/nguyenhuy/Mobile-Feather-SDK-for-Android/blob/046ab27c508d80820bcce355ea6e5dea9fef2684/aviaryfeather/src/com/aviary/android/feather/effects/DelayedSpotDrawPanel.java#L543)
Here and below let's replace your username with `<your github username here>`
Shouldn't need this anymore. The has was already updated.
I mean: please, add constructors for each constructor that java.lang.Exception provides
Typo, nitpicky "getElemeById"
Now that I think about this, shouldn't this have to wait until after the webkit "Created" event is fired?
This is leftover from the browser code.  I think it should just be removed for now.  It looks it is it used to apply a different css class when a menu item is clicked.
Actually probably something more like Array.some would make more sense since all he wants is true/false and he wants it to stop iterating when it finds a positive result.
oh, indeed. looks like somebody just copy-pasted from the find method.
A new useful method
TODO : toute nouveau build => tout nouveau build