labofsahil/github-event-dataset-2012 huggingface dataset:
I wanted to avoid duplication of this.dashboard.setView("#dashboard-content", view) and only have one place where we hard code the target id.
It is a constructor, isn't it?
If you make grainLength modulatable, you'll have to move this code into the gen() function. You'll probably want to check if the grainLength has changed, and if so recalculate the window.
This should not be done by default IMO. Also: This will cause 10 config packets to be send for every incoming navdata package (see `config()` implementation), so I doubt this is what you want.
The sole purpose is to avoid a LOG.warn message for a common case, correct?
That's the same as the combination on my luggage!
Sama hiukan yksinkertaisemmin:
~~~clojure
(defn permutations [a-set]
  (if (empty? a-set)
    '(())
    (mapcat (fn [p] (rotations (cons (first a-set) p)))
           (permutations (rest a-set)))))
~~~
Why do you need a NodeFilterHelper?  It just contains one loop, which is stateless.  :)  Perhaps instances of this class can be replaced with a simple, private method on DefaultExecutorService.
What's the difference between `segments` and `entriesBySegment.keySet()`?
I understand where `response2.headers` comes from, but it doesn't look like the `headers` in `headers['Cookie']` is defined anywhere.  Again, I might be missing something, though.
Don't know...I was trying to not modify the existing behavior as much as possible. I've never been very clear on what that dictionary is used for, and wasn't sure if it was OK to "merge" the dictionaries from two different artist's contains method.  (I know that the text in this case has an empty dict, but what if it didn't ;) )
The only reason that this is hard coded is because I aped the _save method
which also hard coded the dpi. In my opinion, dpi should be either a named
parameter or read from rcParams.

As far as naming schemes goes, I'm not wedded to anything. I think
print_surface kinda fits the current scheme: print_svg = print to a svg
file, print_surface = print to a cairo surface.

I also have a very basic example of using this method to mix matplotlib and
mapnik in the same document. I'm using a shapefile of California counties
as the example, so the whole package of script, data, and output weighs in
at a little over 7MB. Should I add it to the example directory and issue a
pull request on it?

--Christopher

On Thu, Sep 20, 2012 at 8:50 AM, Damon McDougall
<notifications@github.com>wrote:

> In lib/matplotlib/backends/backend_cairo.py:
>
> > @@ -435,6 +435,16 @@ def print_svg(self, fobj, *args, **kwargs):
> >
> >      def print_svgz(self, fobj, *args, **kwargs):
> >          return self._save(fobj, 'svgz', *args, **kwargs)
> > +
> > +    def print_surface(self, fobj, *args, **kwargs):
> > +        dpi = 72
>
> Is this hard-coding intentional? Is there a reason is doesn't take the
> default dpi from rcParams?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/matplotlib/matplotlib/pull/1254/files#r1652190>.
>
>
Because there is some monkey patching, this method is not called from the interactive backends, right? Perhaps this would be a good place to state this.
If this is due to truncation of app names, you should be able to use APP_NAME[:12]
Em alguns pedaços do texto dá para notar a diferença e realmente fica melhor, gostei.
When I looked at the UI5 library I was kind of shocked to see these utility methods and not jQuery.sap.containsIgnoreCase(). So RegExp it is!
even if it's commented out? i mean that line will be taken out, because nobody understands it IMO just me...
You could use SC.guidFor(nowShowing) to get a unique identifier for nowShowing, so that even uninstantiated views could be recognized.

I did some quick checks and the same object returns the a unique guid each time instantiated or not.

    a = SC.View.extend({});
    b = SC.View.extend({});
    SC.guidFor(a);
    > "sc2740"
    SC.guidFor(b);
    > "sc2741"
    SC.guidFor(a);
    > "sc2740"
`row_importer` typo
Is there still a `mock_with :rspec` line around with this change?
The download_csv function actually appends the output to a variable, see line 209, all that needs to be done is getting rid of output buffering.
Worth having a look at the fputcsv function, but probably for the second iteration of this feature, IMO.
Not sure about this. Its kind of sufficient to just have an empty <categories /> element, and this could make it a little more complicated for parsing, though probably not if using a real XML parser.
@kamaulynder thoughts? leave it as is?
I meant `You need to comment the line following line in /etc/rvmrc ...`

- it has double `line`
- it is not required for rvm to work to comment out this line, it is specific to your system/requirements
For normal eviction, assuming short transactions, a reasonable cache size and that the LRU selection is working okay, it should be unusual to choose a page with uncommitted updates.  We could optimize for that case, but I suspect if we hit it there is something else going wrong.

For forced eviction, we only get this far after we have exclusive access to the page.  There could still be uncommitted updates on the page, particularly for transactions that update many items.  In that case the page would not be clean and so forced eviction would fail.  Failing early may well be better in that case, but on the other hand if forced eviction is happening a lot with complex transactions it probably indicates that something else is going wrong.

Hmm, let me take a look at this change -- it would mean different paths through rec_write.c for eviction vs sync, which doesn't thrill me, but I agree that there are some cases it would speed up.
Hi Keith,

> I forget, why are we packing sessions to the bottom of the array as we
> create them?   Is that really useful anymore?   We run through the list of
> sessions during session open, close, eviction sync requests, and hazard
> reference checks.
> 
> What if we quit packing sessions at the bottom of the array?  That is, we
> use the first NULL slot we find for each session we create rather than copy
> the last session reference down, which should give us reasonable packing.
> Session close would decrement session_cnt to the last active slot.

Snap, I just suggested getting rid of conn->sessions entirely -- I don't see why we need it any more: we can just scan through the array of session objects directly, up to session_cnt.

Michael.
This should solve any remaining render issues. `render()` should always be invoked by the frame event, and never directly. And since we're always checking `children.length`, the render loop will only run when necessary.
This line should be indented. All contents between beginning and end tags should be indented. It looks like there's a lot in this file that isn't that way, it'd be super cool if you could fix up this file as part of this request :)
@schmittjoh looks like ``get`` is missing in your whitelist :)
You can remove the `'[*]',` and it will work the same.
Lassen wir mal so. Die Alternative würde Generics und Reflections und bla beinhalten...
this assumes that all searches will be on strings. how about numbers and regexes?
I'll have to go back and look at the dart example.  I think it used list so I was trying to keep the API like-like.
Hrm. Good point. I might ask chelsea to reword it a bit.
this seems like a merge problem?
Comment about why overloaded.
And `var i`.
Are you sure you can just concatenate like that strings without escaping?
I am barely awake. :-)
Nooo. You compile the pattern only once. And store compiled pattern in the top-level constant variable. :-) Compiling the pattern is the process we would not like to repeat again and again.
Yes will, I just thought it wasn't necessary. But it definitely doesn't hurt.
`.text`.
Mah, I would not use `fail_silently`. Better is to fail. ;-) I would also remove this flag elsewhere we are suing it.

Again, user has not provided anything.

*Confirmation e-mail has been sent to your e-mail address.*
Where do you see the problem. If user closes default connection it does not matter. All connections are still recorded in Connection document, I just would not use these for channel subscription tracking. I see we won't go far here so I will implement it your way.
Constant strings should use `'`.
This is why you need another field which will tell you when was this value fetched. Then you limit by that field (you don't want stale data) and order by it.
Please start with GS-version `1000` since the target egg-version will be a `1.1.x` version
can you look in to Jeff's suggestion of using twitter bootstrap?
also, will this work on older browsers?
static too
you could do:

```ruby
case currency
when /UKL/i ; 'GBP'
when /SID/i ; 'SGD'
else        ; currency
end
```
spellInfo->AttributesEx3
lol, yeah, that's fail -> "target->CastSpell(target, ...)" will never do its work
With change for .text_instead_input, the class for label should change to .field according to similar rows. (it is affecting padding, which is anyway wrong, but it should be at least same at all fields)
These show up in my pull requests because I haven't merged in your master yet, sorry.
@BilgeXA that's fine and allows usage of `{@inheritDoc}` also there :)
not sure this tip is needed!?
`form` is not defined. Should be `$(this).closest('form')`.
please follow the js doc conventions.


We already have a mod helper that solves this generically.

Please add tests.

We do not accept any PR without unit tests ( see the jasmine BDD for tests)
When I use None as a default the if statement at 
https://github.com/mozilla-services/circus/blob/a2d22663229e23a300e3baf0e7b37f7e2d54bd80/circus/circusctl.py#L231
evaulates to false leading to the following error:
option --ssh must not have an argument

Is there a recommended workaround?  Endpoint uses "" as a default value, presumably to avoid this problem.
If <code>action="new"</code> is default behavior of the method perhaps specify it in <code>return Bid.SignIn(self.selenium, self.timeout, action="new")</code>.  If there are other actions available can you document it in a docstring perhaps?
What's that `;`?
Yes, thanks. I'm testing in ci-368 and it only works with the unprefixed versions.
The idea is that the Resource UI and the Network Crafter can make use of that instance.
Reverting to Daniel's code. It went through review already.
Not painting issue. Without the following line it could end up with a scrollbar for the textarea. Couldn't find out a less expensive sequence with the same result.
Got this during testing, shortly after jumping into the target system:

```
Error: invalid option '%0' to 'lua_pushfstring'
stack traceback:
    [C]: in function 'DistanceTo'
    [string "modules/Scout/Scout.lua"]:209: in function <[string "modules/Scout/Scout.lua"]:207>
```

I'm not sure where its coming from. DistanceTo doesn't use lua_pushfstring internally. It could be something busted in the error handling, haven't looked closely. @johnbartholomew, any ideas off the top of your head?

Anyway my guess is that `CurBody` is becoming invalid (ie `CurBody.exists()` is false) sometime after `Timer:CallEvery` is setup. I have serious reservations about `CallEvery` that I've raised before. I haven't traced through the logic yet to understand the problem though.
Why do you think that using functions is much easier? The classes in kernels.py already have a ``__call__`` method, so you can use a class instance the same way as you would these functions. And the existing class-based kernels do a lot of other things that you are not supporting.

Note that I don't think that the current kernels are optimal, there are a number of things that could be improved (the lambda functions for initialization look odd, I'm not a fan of the bandwidth getter/setter, documentation and method/attribute naming issues, ...).
@dbu what do you think about https://github.com/symfony-cmf/CreateBundle/pull/15 ?
This applies to other tests as well.
You're right! I will move them to ```ruby-cldr```
I remember fighting with this constant scoping issue before. :(
Perhaps it would be better to have this return Set<Address>, and guarantee a non-null result.  Cleaner for clients.
Is 1 really where we want to draw the line with this warning?  I mean, 2 isn't going to look so nice either, but it's what you get if you add together two profiles that you haven't properly normalized (like an exponential + deVauc).  Perhaps could give a warning if the resulting S/N is expected to be worse than 10 for some naive calculation of S/N?
I wonder if these lists should contain some stronger way to match back to a purchase detail.
Licenses are a one time payment for a perpetual license. VMware doesn't like the subscription model.

Support is annual, I'll figure out the amount and let you know.

-- Monty

----- Original Message -----
From: "Tim Felgentreff" <reply@reply.github.com>
To: "Monty Williams" <monty.williams@gemstone.com>
Sent: Monday, August 6, 2012 12:35:51 AM
Subject: Re: [maglev.github.com] Adding Pricing table (#3)

> @@ -24,3 +24,58 @@ MagLev 1.0.0 was released on October 31, 2011.
>  
>  If you are interested in persistent distributed Ruby objects, [get started
>  with MagLev](/docs/get_started.html).
> +
> +# Licensing & Pricing
> +
> +MagLev itself (the Ruby & Smalltalk source released on Github) is free. GemStone/S is licensed based on resources. The free version below is perfectly sufficient for small to medium sized projects.
> +
> +<table id="matrix">
> +  <thead>
> +    <tr>
> +      <th></th>
> +      <th colspan="3">GemStone/S 64-bit</th>
> +    </tr>
> +  </thead>
> +  <tbody>
> +    <tr class="odd">
> +      <td>Price</td>

Please add whether this price is monthly/quarterly/yearly ...

---
Reply to this email directly or view it on GitHub:
https://github.com/MagLev/maglev.github.com/pull/3/files#r1309660
Could/should we overload the constructor so that no-one passes in producer==null? Then we could guard with checkNotNull, and catch such programming errors.
I see such overloading is done in SensorTransformingEnricher.java.
+1 let's switch to logback which allows including other log prop files from classpath
synchronize (this) ?
je l'ai fait
The extensions are part of the profile though, so it would make sense that they are added when the profile is created. I just browsed the code and found that Google Chrome extensions are already supported within the --chromeopts, which is also set in the equivalent create_chrome_options. I'd be tempted to drop support for extensions here and use the new --extension option. It would seem cleaner and better cross browser support.
`#{resource_name.to_s.humanize}` is useless here. Outside an string it is a simple comment.
were you getting StaleElementException?
can't you use `go_to_tab('location') ` here?
Must have pushed this part before I linked it.
Tiny little nit, and probably something I originally authored incorrectly myself, ```foo.bar``` vs ```foo["bar"]```
I think that is a worse name since it is less descriptive of what the implementation actually does.  This method simply makes a waypoint active and does no planning in a meaningful sense.

However go ahead, if you want to enough to push the change :)
Worth adding a comment here to explain what this is doing? (Which I presume is “stub out all `business_support_schemes` requests)
On Jun 2, 2012, at 12:20, Jorge Bernal
<reply@reply.github.com>
wrote:

> Should we ask for "friends" instead of "followers"?

Yes. That should be changed to friends or following or whichever is
the correct endpoint.
Agree with that. I did not want to argue with both the PHP manual and the RFC :)
На каждую строку ты создаёшь по BufferedReader?
You do not set NULL.
please use K_NULL or RETURN_DefaultObjectValue();
I'd like to see the error message if you still have it.
BIOFORMATS-docs-merge-4.4 fails here with
```
/home/hudson/.hudson/jobs/BIOFORMATS-docs-merge-4.4/workspace/docs/sphinx/developers/buildBF-CPP.txt:21: WARNING: Bullet list ends without a blank line; unexpected unindent.
```
This needs to be refactored to share code with the `-addButtonWithTitle:` method. Copy and paste duplication of such chunks of code is no good.
You're probably right--I've gotten in the habit of using single quotes (a holdover from my PHP days, most likely), but I'll change it to double quotes to fit with the rest of the sklearn coding practices.
Yeah didn't know what would happen and wanted to be on the safe side.
init
acitve_set = init_active_set / strong_set

iterations
a) fit on active_set
b) check kkt on strong_set (if violations, add* and got to a) )
c) check kkt on full set ( if violations, add* and RECOMPUTE strong_set then go to a) ) 

 *. add violating features to active_set

If kkt check fails on c) the active_set and the new strong_set can be different.

But I think the kkt check in c) could be reduced to checking only the features, full_set \ strong_set.
line a bit too long for pep8 :)
+1
Also: how are we going to test this beast?
Please raise appropriate errors instead of assert.
oh, I see. Misread the if a bit... sorry for the noise.
Then we would iterate twice over `self.boost_weights_`. -1 on this.
On Jul 15, 2012, at 21:10 , Olivier Grisel wrote:

>> +    ip.expose_magic('mprun', memory_profiler.magic_mprun)
>> +
>> +- **Under IPython 0.11+**, first create a configuration profile::
>> +
>> +    $ ipython profile create
>> +
>> +  Then create a file named ``~/.ipython/extensions/memory_profiler_ext.py``
>> +  with the following content::
>> +
>> +    import memory_profiler
>> +
>> +    def load_ipython_extension(ip):
>> +        ip.define_magic('memit', memory_profiler.magic.memit)
>> +        ip.define_magic('mprun', memory_profiler.magic_mprun)
>> +
>> +  Then register it in ``~/.ipython/profile_default/ipython_config.py``::
> 
> To create this file I first had to do:
> 
> ```
> python profile create default
> ```

Do you mean ``ipython profile `` etc? This is documented; practically copy-pasted from the lprun section (which was written by you if I recall correctly, right?)

Addressing the concerns now.


> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/scikit-learn/scikit-learn/pull/955/files#r1165637

------------------
Vlad N.
http://vene.ro
I wonder if we should not use explicit URLs so that docstring are still readable outside of the sphinx rendering (e.g. when using the `?` magic in a IPython session).
возьми этот файл с мастера. а то тупо тасуем порядок анотаций в докблоках
It seems like this code is testing pixels out of `dst` before writing anything there. Am I missing something?


(Probably not a bug currently since this is only called with the same `ImageBuf` for source/dest).
On Sat, Sep 29, 2012 at 01:42:40PM -0700, Hara Kenji wrote:
> You should move this test to `test/runnable/traits.d`, it's a module for the `__traits`.

OK. It isn't actually runnable so I just made ti a function in there
to compile.
Tester fails due to multiple definitions on posix: I could add an #ifdef to only use this code on win32, but I don't understand why posix and win32 seem to use different codepaths.
I don't think that the activity indicator for BB needs to be added to another window or view to be showable
This '_' suffix in functions represent private static methods and they are reflected in the Javascript world. They are treated differently to emphasize that they exist outside the scope of the TiObjects.
Probably could be added "C:\Program Files\bbndk-2.0.0" as well
should be `getV8ValueProperty`
Эта конструкция не будет работать в Xcode 4.4.
my style doesn't have whitespace around casts. I think we never had it, and I like that as it kinda highlights a casting operation as a special case.
action to ?
Right.. 

I think moving to per bucket memory, it will be simpler in this respect also.
Spilling a whole bucket + freeing it's ptr array at that point [not wait for critical] .. because the next phase may well need substantially small ptr arrays
Is this condition correct?  
Should it just check fileNameServiceDali.length() ?
or
should the second part be (!topology || !topology->getProp...)

which is the inverse of the condition guarding against the sleep.  (Or should that condition be !topology || .... instead)?
common to all engines...

that comment was originally from Roxie impl. (I suppose).

@richardkchapman - do you know why an exception wasn't/isn't thrown in checkValid implementations ?
No, there is no isAlpha() in javascript.
Looks to me as though some of your ellipsis cases have been converted to a unicode elipsis char and some haven't, which seems like it would mess up the monospaced output to me
There is no need of 'is' I think that checking NodeType should be enough
Bu kısım eskiyse, kullanmıyorsak, çalıştırmıyorsak silelim.
Just curious, shouldn't still example variable better be called $reader?
Are these blanks added intentionally?
Sorry, I don't get this. Is it a test or a helper method?
It doesn't kick in except through the command line. You have to say "rails server trinidad" or "trinidad", otherwise it does nothing. And I'd like to be able to run it in any environment. So if Travis doesn't complain, I'd prefer to keep it generic.
Is there a reason why you favor this technique to take apart a cookie than using regex?  One thing I'm a little concerned about at the line is that if the 'Set-Cookie' hash key doesn't exist, the split() function can cause an error.

I typically use scan() to capture a pattern... but that's just personal preference.
you might have missed this is C# code inside a here-document in Ruby code. The #{fingerprint} thing will be replaced by ruby, therefore generating C# code like

    return "Blahblah";

which will in turn be compiled by the XSLT processor and rendered into the result XML for verification of successful exploit.

    return fingerprint;

at that place will create a stylesheet compilation error as there is no C# variable named fingerprint in this sample code.
For the smoothing window which size is in relation with the scale factor.
these all look good thanks!
Nevermind, I read that wrong.  I thought the second false was the data but it's actually isStopIncluded.
p is global!!!
Most of these props and the animate method are supported on sections in MW, but is this something we want to communicate in the docs?
actually this line shouldnt be here :s it s part of another commit/pull request
Sorry and thank you for spotting it!
You never call this. Is it really necessary?
not easy at all. The good news is that if they are no trouble, it's ok, otherwise these lead to test failures.
It'll work; translators will just see placeholders for field and value.
stop this print insanity. generate it as a list of strings to join at the end and then print. @JensTimmerman
excellent, thanks!
Hopefully, in the "long term", all slice-able ranges will just support `r = r[n ..$]`.

At that point, the test for length would be purely assertive.
... and therefore this can be eliminated
Pretty much. Truncating temp to make it the same length just makes the code simpler. Same as in mergeHi below.
This seems unnecessary, IMO. Just use Object.defineProperty itself. It is succinct enough.
Generally I am for fail fast approach. I want to know directly when it is wrong. 

+1 for adding test sources
remove empty line above `pass`

fix indentation of `pass` line; please make sure you're not using spaces anywhere
no, this is not possible, since this method is private and only called with the property names of a class.
To avoid redundancy, can we instantiate an exclude and then toString it?
You have the correct URL now, but this line should still be placed in alphabetical order, rather than at the very top of the list of repositories.
you can create / use constants for 'ADD' and 'DEL'
Yup. Made it @default-spacing.
Should be `.on()` here and below
can we use single quotes here and in all the other files beneath?
class A {
    int a;
    int b;
}

A a = null;
a.b; // null dereference with an address > 0 .

More generally, Linux define the file /proc/sys/vm/mmap_min_addr that define how much memory is reserved by the system to handle null pointer dereferences. It seems to me that this is reasonable to use the same value here.
MINOR: Refers to logging_entry_test.h instead of log_entry_tests.h in comments.
E-h-h... You're right. Never mind.
I've not used AtomicReference before.

Everything else LGTM

Thanks for your time and for all that you taught me here
Shouldn't require any logic... `data/DoctrineMongoODMModule/Proxy` is already relative to an app skeleton's root dir (see https://github.com/zendframework/ZendSkeletonApplication/blob/master/public/index.php#L5) :)
Check for the namespace, that would be very useful :)
I think that should be in the quick start guide or a QMO page.
Should be fixed now,
Can you write the 2 commands to type to convert the current information defined inside the database ?
You should also explain how to configure the doctrine extensions
oh, without .rst here too, just @Freeaqingme said in the other diff.
No need for double namespace separators here. (and on the next line.)
One optimization I like to do is to cache array lengths to a variable called 'len' and do 'i < len'. JavaScript interpreters usually can't optimize the reference out because the array length can change inside the for loop. You can always do

```Javascript
for(i = 1, len = returnTypeName.length; i < len; i++)
```
to keep it short
Seems like this should just be:

```ruby
result.retain if options[:retain]
```
Add download tooltip
It doesn't have id, but it have resource_uri which is an empty string (that might be confusing in the output). Thats why I delete it.

On 3. apr. 2012, at 21:33, Mitar wrote:

>> +
>> +    def __init__(self, embedded, attribute, null=False, help_text=None):
>> +        '''
>> +            The ``embedded`` argument should point to a ``Resource`` class, NOT
>> +            to a ``document``. Required.
>> +        '''
>> +        super(EmbeddedDocumentField, self).__init__(
>> +                                                 to=embedded,
>> +                                                 attribute=attribute,
>> +                                                 null=null,
>> +                                                 full=True,
>> +                                                 help_text=help_text,
>> +                                                )
>> +    def dehydrate(self, obj):
>> +        out = super(EmbeddedDocumentField, self).dehydrate(obj).data
>> +        del out["resource_uri"]
> 
> Why `EmbeddedDocumentField` has `resource_uri` in the first place? Is not `EmbeddedDocumentField` just "abstract" (in the sense that it is stored embeded in the parent document and does not have its ID)?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/mitar/django-tastypie-mongoengine/pull/1/files#r641296
Should be easy to prevent it from upscaling images. All I'm saying is, move that code into a custom widget, get rid of the TODO.
Looks like the button will toggle whenever players join? So it's inactive for 1, 3, 5, 7 players and active for 2, 4, 6. I'd suggest using set_active() and set_inactive() instead for the respective cases.
remove "pair" from comment
This means register will be called multiple times, whereas I guess you only intend it to be called once.
ça -> cela
This line is significant.  Importing in Python is broken down into three groups.  System level, apps that require installation, and finally any specific to this package.  In testing, I try to break them into a fourth -- parts of the system under test.
Yes, ideally.

Note that I'm not strongly against using 3d party bundle repo, but on the other hand having our own let us control our OSGi props and avoid potential reliability issues with no-longer maintained 3d party repo (which I doubt will happen anytime soon with Spring's repo though).
this are class names for icons, for example "i-user" is icon for user :) but I think we can change them to "ico-user" for better consistancy
Actually: it would be nice to have a short rdoc here describing why AAJ's method is overloaded. Can be done post merge (or even with GH's editor).
Starting index a 1 is more intuitive when counting stuff in progress logs.

```
print("building tree %d of %d" % (i + 1, n_trees))
```
yani alan bayad baz tekrar konam :D
of course you need an else statement to handle the case where the user was logged in but can not access the page.

for you to understand a bit, after a user (a translator in this case) has logged in, the method "after_sign_in_path_for(resource)" from devise is called.  It checks if a path is stored somewhere and redirect to the latter or to a default path if not found.

Here, we put the after sign in path in session.

Not sure it will work as-is but worth trying.
You can jut call $image->paste(), without using an extra filter object
skip the if statement and do `liClass += this.className`;
I thoug in removing all these prefixes for transitions/animations but i dont know where they are going to come in b2g...
새로운 내용은 무엇인가요??
Extra closing bracket?
Туда же передаётся объект класса. То есть придётся писать что то типа 
... <..., class Comparator = std::less<T> >
...(..., compare = std::less<T>) {...}

Во-первых у меня есть сомнения по поводу того, что один параметр в шаблоне может зависить от другого и ещё что стандартный параметр функции будет согласоваться со стандартным параметром шаблона. Вы уверены, что оно будет работать?
Perhaps add an override to `SyncResourceDelegate.consumeEntity` that takes an `HttpResponse`?
you could use max(), but that's just a bit more elegant :)
Actually, the final `,` can hurt.
Pedantically, the [JSON spec](http://json.org/) disallows it, and more practically, [some](http://whereswalden.com/2010/09/08/spidermonkey-json-change-trailing-commas-no-longer-accepted/) [browsers](http://www.nczonline.net/blog/2010/02/02/how-internet-explorer-8-document-mode-affects-javascript/) will throw a syntax error with the trailing comma under the right (read: fairly common) conditions.
> Declaration **(for/of)** phpBB Social Network object
`$p_master` is the master object
`$p_master` is the creator this object
If we are going to keep collect and use it in multiple places, we should throw it in utils.js.
missed that. good point. I was going to write it something like this -

if (complete)
     createWebWorksReady()
else 
    addEventListener(...

but this works as well and is the same, I just didn't see that line :)

saw some new stuff on github -

:+1: 
:-1: 
:100: 
:109: 
:1234: