labofsahil/github-event-dataset-2012 huggingface dataset:
hmm we usually don't use description in hint but instead use a technical id. In this case: "csrf-token-invalidator" for example
Nuuuuub! :D
ouch... yes, I've found the problem... it's in WINVER (it mast be at least  0x0500, that mean windows 2000), thanks.
How about calling this 'filterParameters', to make it clear it's associated with the filter?
For any future viewers of this commit, I apologize.  It has a whole bunch of bugs and bullshit, and a terrible commit message.
vsfilter supports \clip(n,<clip object>) notation, where n is a scaling factor that divides all points in the clip object by 2^(n-1). This *potentially* allows for subpixel clips, as the clip object itself can only contain integers. It may or may not actually help in practice when the clip is being scaled down or is panning slowly (situations where rounding errors would become more obvious). Unfortunately libass doesn't support this, so it's only applied if the initial clip contains a scaling factor (which is why it's wrapped in an if).
I don't recommend trying to test git merge yet. Dev may not be a true branch of master, as far as Git is aware, since it started as dev, then went to dev_tms, then to _ci, then back to _dev; and somewhere in there was overlayed with content from master.

So, if we tried a git merge at this point, there is a very large risk that it would totally mess up master, dev, or both.

I think we'll need to very carefully merge dev back into master when we release 1.92 stable.  After we have master at 1.92 release, then we could re-branch dev from it, and take advantage of Git merge functions from that point on.

-Tom
I had some very fuzzy rememberings about that. In case we experience those problems again, I will revert this commit.
Will be removed until the next release XD 
So after next release it is permanent?
DO NOT REMOVE
去点点注册了个号。我们这里，点点的张贴海报非常非常多，但是感觉好像没什么人感兴趣。

又是Twitter风格的DHTML啊。。
I'd love to use `'left'` and `'right'` instead of having to deal with these "alternate universes", but how? I feel like I'd hate both typing or reading `[0]` and `[1]`'s. Maybe `left` and `right` instead of `prev` and `next`, and `leftmost` and `rightmost` instead of `first`/`firstChild` and `last`/`lastChild`? Then it could just be `[dir]` and `[dir+'most']` or something.
`append` and `prepend` are bad names for this, the jQuery method names `before` and `after` come closer.

However, I don't believe these methods would be used at all, the only interstices there are gonna be are `Cursor`s and anticursors. `Cursor`s are only `insert*`ed and `*pend`ed; the idea I had when I put them in `tree.js` was that instead of how currently `Cursor` has all four of these methods, instead `Cursor` could just override `insertAt` and it'd be known that `insert*` and `*pend` call them, so `Cursor` would just get those for free. Anticursors will in fact only ever be `insertAt` the position of the current cursor (they're for selections, remember?)
Any chance of you coming back to Picard dev, by the way? We miss you ;-)
Looks like you have a copy-and-paste error there: the last example about string interpolation where it says "//compile error!" is identical to the example above that, which does compile.
Thanks Mario. What do you think Andy?
I don't think this is enough, the platform still supports the legacy error handling and this doesn't account for it.
its not a fix. seems like your crash appears randomly. this change doesnt solve anything
Am 21.02.2012 12:45, schrieb Martin Zibricky:
> Have you tried compiling it on windows?

I can't. I don't have a windows development environment. Sorry.

> Do we need it for all platforms?
It should not harm on any platform as it forces Python for search it's 
installation in sys._MEIPASS only.

-- 
Schönen Gruß - Regards
Hartmut Goebel
Dipl.-Informatiker (univ.), CISSP, CSSLP

Goebel Consult
Spezialist für IT-Sicherheit in komplexen Umgebungen
http://www.goebel-consult.de

Monatliche Kolumne: http://www.cissp-gefluester.de/
Goebel Consult ist Mitglied bei http://www.7-it.de
I think the soft endstops will be easier to implement in the dda, but that may be misguided.  I like the direction this work is going.  I also agree the axes should be changed to an array.   So then you will have, for example, target.position[X_AXIS], soft_min[X_AXIS], soft_max[X_AXIS], steps_per_m[X_AXIS].  With a union{...}, you could even efficiently set up an alias so that &target.X = &target.position[X_AXIS].  But I would hope that almost all of the references to X, Y, Z and E would vanish anyway.  Even the gcode_parser could have axis_code[X_AXIS]='X'.

This presumes I also get the other thing I wanted to do, that is, to replace um_to_steps_x() with um_to_steps(steps, um, steps_per_m[X_AXIS]).  But this requires careful planning and testing since it replaces the overflow avoidance macros.

More later...
Enig i det, så lenge det ikke er snakk om user-agent-sniffing! Hva med å bare sjekke om browseren støtter document.querySelectorAll ?
Anyone experiencing problems with this? For me 

```sql
(28519,0,0,0,4,0,100,0,0,0,0,0,42,1,0,0,0,0,0,1,0,0,0,0,0,0,0,'Withered Troll - On aggro - Set invincibility level'),
(28519,0,1,0,2,0,100,0,0,1,60000,60000,11,51869,0,0,0,0,0,1,0,0,0,0,0,0,0,'Withered Troll - On % HP - Spellcast'),
```

doesn't work. The troll neither casts the spell nor does it get immune to damage at 1% HP...
```SQL
+DELETE FROM `smart_scripts` WHERE `entryorguid` IN (@ENTRY,@ENTRY*100);
```

Delete statement incorrect.
Missing semicolon on end.
I think if you add those memory barriers into the lock() code then you don't need volatile on sobj.
Previously it used to call clone() and not call allocate.  I've changed that clone call into a call to allocate.

I'm tempted to kill this code anyway - I'm not sure it is at all useful, and it is ugly.  I suspect it is legacy - More useful would be to use an internal number e.g., 1 to indicate it is coming from here.
Is this virtual ?
incorrect usage of static function.
The comment on line 800 makes no sense.
It says to set ```true``` for both verbose and silent.
@kalgynirae experimenting.py was marked executable when I downloaded it, but it didn't have a hashbang, just to fyi.
Sean, by default the sources are included in the shaded jar, and they are accessible via Eclipse. The sources are half-baked only in that the packages in the code still reference their old names. They are good for reference but at first they are a bit misleading.
I think that we can remove the entire rule.
If they're all set to never it defaults to assistant at the moment, but I like the idea of them all being never = random.
Multiple snippet port, and hand fulls of wtf's...

thanks to: cataproject, migcore, and anyone that had helped research and develop some of the codez used from our old core.
An option is to automatically append the '.' if it's not on the user-defined prefix and pass that to everything.  That would also allow a user to use a prefix of 'something.' and have it work correctly instead of ending up as 'something..propertyName'.  Does that make sense?
nginx.conf rewrite removed.
> Yes, Clay used features of numpy 1.6 that are not in 1.5. Jenkins also
> complained. I am on it.

Jenkins is a blessing. Thanks @ogrisel.
# 成功

**结论: 还是使用Gui吧**
Thrift asynchronous calls only work with a TFramedTransport - and we really needed to be able to make asynchronous calls (and we're not going to use indextank-service). In case your wondering, you'll just get a protocol exception (with something about a wrong version) if you try to use asynchronous calls.

I've only just started using Thrift so I'm not sure if there's a better way to achieve this without having to fork the project. Cassandra seems to offer command line constructors that determine which transport is generated so I may make a patch which does that in the future so it can be merged with the main repo.
I like how you kept the Evervolv wallpaper, haha
yeah it's made in Adobe Air and some things do not work properly, but for a php developer it's quite good :)
you're welcome!
that looks fine to me
you should rename 2012_02_09_00_world_creature_templates to creature_template. lol
I made a mistake here, thanks
Aw bro

I just got this email and I already went through the spec with small commits and thorough notes

So I'm about to push that, and just read all my commits after yours.

You did great.

Cheers! 

On Friday, February 10, 2012 at 11:02 PM, schneiderik wrote:

> Seriously, what the fuck am I missing here. I've tried to write this action for like 3 hours now. Hit me up when you have a second.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/joemsak/devcause_alpha_project/commit/30aca2fd510cf67e66cbe83eeee1b0a8c92b0f2d#commitcomment-956136
> 
>
Is there a reason that this is a form submission rather than a link?

Also, is there a reason why the action is to /teach/, rather than /manage/?

Relatedly, when I manually tried to go to /manage/{{program.url}}/editclass/{{cls.id}} (to see if we can make this change), I got an error about bool objects not being iterable.
And here's where we call the "after" hook.
Ok that makes sense!
On Jan 7, 2012 5:23 PM, "Steve Loeppky" <
reply@reply.github.com>
wrote:

> CSE 143 is probably printing off the assignments so they can grade it.
>
> I meant "It's also fine to wrap it to *two* lines, right?"
>
> 2 longer lines is better than 3 tiny lines I think.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/BigLep/ChuckanutBayFoodsInternal/commit/1fc37369a88561934954c8ce0098b30a1261a930#commitcomment-846966
>
C'est le code qu'il faut insérer dans le lien que tu glisses / déposes dans ta barre de bookmark =p
The spacing here is wrong
=======================================
To sign up for Facebook, follow the link below:
http://www.facebook.com/r.php?re=9a3dfaa8f85bafc99bf80dfe5b5063b6&mid=5c23e8aG5af3e155cf20G0G46&key=AQAicjZhZwSZkfwRQzvQQ4fnub3yjRcL46rf_WxzT1grbLHtcvLQ7S05baCW7AGybkDBnayD6saucSiI&invid=1363206074
=======================================

Hi,

The following person invited you to be their friend on Facebook:

Travis RonPaul Rowland (Invite sent: Feb 20, 2012)

Facebook is a great place to keep in touch with friends, post photos, videos and create events. But first you need to join! Sign up today to create a profile and connect with the people you know.

Thanks,
The Facebook Team

To sign up for Facebook, follow the link below:
http://www.facebook.com/r.php?re=9a3dfaa8f85bafc99bf80dfe5b5063b6&mid=5c23e8aG5af3e155cf20G0G46&key=AQAicjZhZwSZkfwRQzvQQ4fnub3yjRcL46rf_WxzT1grbLHtcvLQ7S05baCW7AGybkDBnayD6saucSiI&invid=1363206074

=======================================
This message was sent to reply@reply.github.com. If you don't want to receive these emails from Facebook in the future, please follow the link below to unsubscribe.
http://www.facebook.com/o.php?k=09386b&u=100003504049952&mid=5c23e8aG5af3e155cf20G0G46
 Facebook, Inc. Attention: Department 415 P.O Box 10005 Palo Alto CA 94303
It's redundant here. It may be generated in javascript code.
fix indent of this line down..
@gasche 

Hum, no, there was no reason whatsoever behind that ugly shadowing. Thanks for catching it.


Apart from that particular case, I rather like to use `let...and...` when there is no dependency between variables, to make it clearer where the dependencies *are*. For instance I'd keep an `and` between `extended_name` and `lnumdir`. I'm not sure what *else* `and` could be useful for with non-recursive `let`. Are there good reasons *not* to use it that way ? 

Put another way, I wonder what else would be a "special reason to be simultaneous" for non-rec definitions ?
JS: 1,699,800 B (00.22%) -- compartment(http://iichan.ru/b/res/2003279.html)
layout: 3,492,928 B (00.45%) -- shell(http://iichan.ru/b/res/2003279.html)
DOM+style: 2,629,152 B (00.34%) -- inner-window(id=952, uri=http://iichan.ru/b/res/2003279.html)
В больших тредах будет примерно так. Лиса.
the image should have copyright, so please replace it with some other
Yah, we should migrate docs to the new syntax.
'(renamed -- used to be called 'varLabel').' can go
It is useless to have getMatEntry(std::size_t rowId, std::size_t colId) to get the value of one entry of a matrix.

Three comments:
1. Since the solver is included as a member, I suggest to change the name to LinearEquationInterface
2. Use const modifier to all arguments of functions that are not changeable.
3. Leave a room for external preconditioner. This can be done in future if we really need an external preconditioner.
not really. I'll change it to use std::string.
Não precisa dizer que é o fim do arquivo. Para que você fez isso?
OK, the thing is that after a 2to3 run, I get this:

  File "/usr/local/lib/python3.2/dist-packages/Volt-0.0.2-py3.2.egg/volt/plugin/builtins/atomic/__init__.py", line 78, in run
    target.write(rendered.encode('utf-8'))
TypeError: must be str, not bytes

so the below line that does:   
  target.write(rendered.encode('utf-8'))

seems wrong on python3 as it passes a bytes and not a string object to write().
As in, I agree. I went back and forth about where to put this and settled on here but putting it as a method of `TestPipEnvironment` seems more sensible.
I don't even know the purpose of this relationship. I added it here only because it was in the Google doc.
Yikes, that's a lot of dirnames :). Can we use $CONFIG->path here instead?
`UntypedProducerActor`
This: 643ms
Master: 937
I'd use a PF instead:

    applyRoute(s, message) foreach {
      case Destination(sender, `self`) => super.tell(msg, sender)
      case Destination(sender, recipient) =>  recipient.tell(msg, sender)
    }
Ok, but comment that so no one cleans it up ;-)
Don't we normally use ConfigExceptions for this?
yes, using the `Future` API for callbacks sounds like a good plan!
No, the problematic case is when you set read-timeout to nonzero but no reads will never happen guaranteeing the connection to close after read-timeout seconds.
import java.util.concurrent.TimeoutException
True. And now netty4 uses the same terminology for handlers instead of  
Upstream/Downstream.
yes, this looks like a good one to me!
No, I meant more like caching away the TimeUnit to be used.

val pollTimeUnit = if (version.major >= 3) TimeUnit.MILLIS else TimeUnit.MICROSECONDS

And then in ConcurrentSocketActor:

val ext = ZeroMQExtension(context.system)
val duration = (fromConfig getOrElse ext.DefaultPollTimeout)
val pollLength = duration.toUnit(ext.pollTimeUnit).toLong
and then use: poller.poll(pollLength)
do we need this convergence flag any more? The state should always be based on converged gossip
You aren't using it as a PartialFunction so what's the purpose of having it return a partialfunction?
since there's not a simple way to mark a conversation as resolved:

RESOLVED
Add the +*account cards and link the users table with it.  the login field update is something I was playing with, but we don't need it.
I'm not sure I'm following your point.  For all of those requests, it was unfreeziing it before, so there is no change in that, right?  I guess the issue is if you are only reading, you may never unfreeze, so if your point is about that case then I agree.
Well, we haven't really reviewed any of it and it really has been a long time since I started it and we have done a lot of work in the meantime.

Could be worth restarting parts of it, but I think it is worth understanding what I was trying to do  before the restart.

The menu_options stuff, I'm sure I agree (below) even more.  I think we really want to use views or settings or something for this idea, and we didn't have any of it when I first wrote the.
No problem!  Just thought I'd point it out.
well we have genshi only for 1.8, 1.9 should have jinja2 but I'd not spend too much effort on this depends how long we continue to support the i suppose
There's a unit test for this class (test_extensions_modules) that should be updated to test both the case where the metadata flag is specified and not specified.
Adding this new attribute seemed like an elegant solution, but I fear others that have potentially overridden this function. What would happen to such a code? I know not. . .
I was thinking about a grabbing the first item (video stream) from a JOIN with streamdetails where idfile=idfile etc, but probably the best option is to wait until the db can return the duration of a file without any extra lookups.

It's pretty weird that duration and filesize are unavailable from the db, maybe in the future we can add those during the scan.

After switching to using ThumbLoader for thumbs/fanart the speedup is incredible, so I'm going to ditch this call and be done with it.
I changed it so that PreventSortAndRenumber() locks to read the value. Once it's checked, there is a lock, too.
So, while the sorting happens, it is locked. What do you mean?
Using _vscprintf is not necessary to fix the current problem. It can still be added later if really necessary (as it's only an optimization).
$this->module->getUploadPath() лучше заменить на уже полученный $path
это ж уже не нужно, раз есть отдельная страница
@MustOverride == abstract ?
I realised I wasn't actually expanding the tabs into spaces in VIM, I
saw it after pushing, sorry for that.

On 29 Nov 2012 14:09, xeor wrote:
>    In taggo:
> > @@ -176,6 +177,9 @@ class Taggo:
> >              count = 0
> >
> >          for root, dirs, files in os.walk(self.content_folder):
> > +         if self.ignore_hidden:
> > +                     dirs[:] = [d for d in dirs if not d.startswith('.')]
> > +            dirs[:] = [d for d in dirs if not d in self.tags_folder]
> 
>    Code indention should use 4 spaces as in PEP8, not tabs :)
> 
>    —
>    Reply to this email directly or [1]view it on GitHub.
>    [INLINE]
> 
> References
> 
>    1. https://github.com/xeor/taggo/pull/1/files#r2271742

-- 
Enric Caussa Morales
<me@enric.me>
Filters added for not to take into account temporary files. Those filters will be removed when fixing the native code creation process.
This iterator is just for the chunks, it skips the texts.
also rename this file.
`columns()` is also called by `space()` (in _functions.scss) - so this change should happen there as well, I think?
I ended up submitting a change that wasn't the version I'd tested (on a different box).  Fixed in the next commit.  I can close this and rebase if you'd prefer.
easy enough. Would you want the filename to follow the same naming?
I don't have a better name but this is just for the transition. When 2.0 will be released, this class will be remaned to just `Twig_Filter`. And `Twig_SimpleFilter` will just be an alias for it.
These need to be prefixed.
Here:

```
[_navigationController.view addLayoutConstraint:[TUILayoutConstraint constraintWithAttribute:TUILayoutConstraintAttributeWidth relativeTo:@"superview" attribute:TUILayoutConstraintAttributeWidth]];
[_navigationController.view addLayoutConstraint:[TUILayoutConstraint constraintWithAttribute:TUILayoutConstraintAttributeHeight relativeTo:@"superview" attribute:TUILayoutConstraintAttributeHeight offset:-TAB_HEIGHT]];
[_navigationController.view addLayoutConstraint:[TUILayoutConstraint constraintWithAttribute:TUILayoutConstraintAttributeMinX relativeTo:@"superview" attribute:TUILayoutConstraintAttributeMinX]];
[_navigationController.view addLayoutConstraint:[TUILayoutConstraint constraintWithAttribute:TUILayoutConstraintAttributeMinY relativeTo:@"superview" attribute:TUILayoutConstraintAttributeMinY offset:TAB_HEIGHT]];
```
The timer should be invalidated any time it's going to be set to `nil`. This can be part of the behavior of a `setHideKnobTimer:` setter.
I would expect `toBuf()` to handle that.
..., builds libpd.so/dylib/dll
This is an old pull request, Josh -- if you want to change stuff you should submit a new one. I agree that making this configurable (and lower by default!) would be good.
possibly: "decrements the verbosity *of the output" by 1.
It's exist in old code too http://api.nette.org/2.1/source-Config.Compiler.php.html#148
Why not use `System.nanoTime`? Its much cheaper, and also have have `SelectorUtil.SELECT_WAIT_TIME_NANOS`.
I mean, put the or'ed values on one line.
Yep, that's no longer needed.
This could be `zend_bool` too.
New hash styles, please.
Thanks, let's ask in the mailing list.
Certainly do not want an inconsistent database. I was following the pattern of the other tests in this file. I am happy to add an ensure to the new test method.

Possibly a DRYer solution for these tests would be a teardown method:

```ruby
def teardown
  rename_table :octopi, :test_models if connection.table_exists? :octopi
  super
end
```

I also notice that the use of the ActiveRecord::Base.connection object is handled differently in each of the pre-existing tests. Should I not just use ```connection```, the attr_reader defined in ```ActiveRecord::Migration::TestHelper```?

I've got another branch with these two changes (add teardown method and just use ```connection```) ready and tested.
Split it into ... do / body / end lines, or back to 4 explicit 'ignored_sql.concat lines (fewer code lines overall, but need to add comments back in)?
Thank you, I'll add assertion.
I want capture all errors occurred during rake task, and `system` should be in a block but result should be captured output.
I don't see ability to use result of system instead of $?.
Here is my attempt to deprecate the arity of  `instantiate_fixtures`.
`whether`
I had `parWith` in mind, but didn't include it in my proposal because it was already a bit lengthy.
Imho, if there are any compatibility problems with using `par`, use `parWith` or something else.
I don't see how a comment is the best which can be done.

    // a few lines up, it's not much of a chase
    final val maxNanos   = Long.MaxValue
    final val maxMicros  = maxNanos / 1000
    final val maxMillis  = maxMicros / 1000
    final val maxSeconds = maxMillis / 1000
    final val maxMinutes = maxSeconds / 60
    final val maxHours   = maxMinutes / 60
    final val maxDays    = maxHours / 24

    import FiniteDuration._

    require(unit match {
        /*
         * enforce the 2^63-1 ns limit, must be pos/neg symmetrical because of unary_-
         */
        case NANOSECONDS  ⇒   -maxNanos <= length && length <= maxNanos
        case MICROSECONDS ⇒  -maxMicros <= length && length <= maxMicros
        case MILLISECONDS ⇒  -maxMillis <= length && length <= maxMillis
        case SECONDS      ⇒ -maxSeconds <= length && length <= maxSeconds
        case MINUTES      ⇒ -maxMinutes <= length && length <= maxMinutes
        case HOURS        ⇒   -maxHours <= length && length <= maxHours
        case DAYS         ⇒    -maxDays <= length && length <= maxDays
        case _ ⇒
          val v = DAYS.convert(length, unit)
          -maxDays <= v && v <= maxDays
      }, "Duration is limited to +-(2^63-1)ns (ca. 292 years)")

You don't have to hardcode long strings of digits.
Based on the python docs, it looks like we should be able to get away with just calling "os.path.expanduser('~')"; the thread referenced in from 2005, so perhaps there was an issue that has since been resolved.
again, don't test for equality, just test the printed output. [I'll stop pointing this out now :).]
Well double-backticks are for "code" and single-backticks for "mathematics". There is no clear distinction between the two in this case. I have pushed another commit that consistently uses double-backticks in the first line.
Oh it doesn't :)  I just noticed it because I was running the examples to test the colors.
thank for that!
Also, I ran the series test suite and there is no test that benefits from this change, i.e. these lines are not covered (they are not traversed for any test). Did you have a test that needed this change?
do we have the chance to remove these?
No I did not mean to commit to those changes. What's the easiest way to remove them?
I like id ;-)
It's an `if` statement. They're really, really standard for executing code conditionally.
And mark it internal.
Calling `ev_loop_fork()` is not enough. libeio for instance has a couple of mutexes that would need to be reinitialized (never mind the work queue and other state).

Fork safety has been discussed in the past. The consensus was that it's not worth the trouble.
Cool trick you can do in Rails:

    attachable_assets.select{|aa| aa.local_name == local_name}.try(:first).asset

Does the same thing in one line.
workgroups => list

I guess I missed this when I renamed workgroup to project.
we should have full explain output in some other form.
instead of at subscript, did you mean array index?
If one was to flip the priority of `ENV["PWD"]` vs. `Dir.pwd` then the expectations in the test example and also the assertion about the return of `chef_config_dir` would all fail. Hopefully the text descriptions of the behavior and comments explain what the test is testing for, so it shouldn't be too confusing.
@p thoughts?
`$extension_manager` is never set
Something went wrong in the wording there.
Put `$config_name` in curly braces, it is quite hard to see that there is a variable in the template file name.
If new avatar types implement the class naming scheme (phpbb_avatar_driver_...) they will also get the shortened filename. If they don't do it this function will return a full length filename as written in the above comment.
Copied the behaviour now
Is it an out or in/out parameter? What is its layout in either case?
This should be called "SQLite3" or "SQLite3 3.X+" if we have a minimum version requirement above 3.0, with X replaced by the actual minimum version we require.
`staging.monitoring.api.rackspacecloud.com`, not cmstaging.
who wants this event?
parantezel nu sunt necesare :) e Python.
spațiu după prima virgulă, re rog. Și cel mai bine mută importul sus.
This file looks good to me, modulo docs. But, I would be happy to do the docs just to get more playtime with the new Hails/Web stuff.
The way it works in the python library is that if the hiredis module's available, then redis-py will use it, but it falls back to a pure python implementation. I believe redis-rb works the same way. Is there a way in gemspecs to describe optional dependencies?
So multiplies(a,b) with int64 or w/e else works. multiplies between two unrelated types also still generate the same error as usual.
I don't want to defend any of my typing choices in this commit :). A few of them are probably suspect.
In progress?
@pascanur can you check those modification? I don't want to dig inside scan.grad...

thanks
I just meant the four lines 1506:1510 

Currently you build a list, extend it and then turn it into a tuple. I'm just suggesting that this could be the addition of two tuples.
Check whether the downloaded processor is the same you added in `setUp()`
``if 'device' in instance.dirty_fields:`` looks more idiomatic.
Is there a better name for this?  Maybe a comment explaining what's going on would help too.
Code style:

```
var items1 = this.items,
    items2 = this.items,
    newItems = {};
```
"Declaration techs list is empty" или "List of declaration techs is empty", и точка после "Skipping operation".
Yeah, it wasn't the best place to put it, but I was trying to avoid breaking anything else that might work with startDefaultServer.
This whole method scares me, and not in a good way. We need to find a new API for this to work, because monkey patching all this state and then cleaning up is a mess, and very very fragile.
These links are wrong, the methods are on `QuerySet`, not `Model`.
@apollo13  I just updated my branch to clean the extra hashes. I raised ``ValueError`` because it's more consistent with the rest of ``ModelForm`` (raises ``ValueError`` when no ``model`` is specified and when ``save`` is called when the form doesn't validate.
Just for the compatibility of Linux and Windows LDTP, I have named the filename / class name as same :-) Not a big deal, if required, we can change it. Thanks
Is "Probability-functions" a common sense?  Other modules are named without hyphen.
`brew --config` is the extreme example, and it's mainly due to the all the spotlight, pkgutil and mdfind usage that we do now to determine Xcode versions and paths. The actual time it takes to interpret the code is pretty negligible.
Feels great with this syntax.
I'm ok with supplanting both methods.  We want to very carefully control this section of code anyway because it ges to the core of our model.  The heroku cli method that invokes commands also looks very similar to run! - convergent evolution because the problem is the same.  Using run! as a starting point for customization is acceptable.
Use the results formatter only when printing out the results.  No functions should be called in here that may cause an error.
The '{DAV:}write' property must still aggregate everything. Aggregate privileges may be non-abstract.

Don't change `getDefaultSupportedPrivilegeSet` though. That method is only used if `getSupportedPrivilegeSet` returns `null`.

Literally the only thing you should/could do is copy the body from `getDefaultSupportedPrivilegeSet`, and remove set the abstract properties to `false`, where you need them.
Pardon maître :)
API requests^
We can be more descriptive:

"Make the http request to the reddit server and handle the response."
Can you update `.travis.yml` and remove the jdk section as well?
Imho, cksum and sha256 are essentially doing the same thing: Both are creating a hash of the file. The advantage of cksum is that it is defined in posix. I also see the advantage of sha256 as a better hash algorithm, but for detecting "it changed", I think it's sufficient.
capital `i` please, sir
I think the new javadoc is more confusing than the old javadoc.  We can't enumerate _all_ the ways a test could time out, so why choose these few?
It's there to prevent a FOUC. The value from the stylesheet is being read in by the JavaScript and masked by the inline style, which has to be toggled during reading. (Ugly, but better than trying to parse the CSS I think.)

On Tuesday, July 17, 2012 at 4:06 PM, Dan Imal wrote:

> > @@ -1,6 +1,7 @@
> > .component.audio-player
> > - %a.button.playpause{ dojoAttachPoint : 'playpause', title : "Play" }
> > - :if useHtml5Player
> > - :if !androidAudioFallback
> > - %a.button.rev30{ dojoAttachPoint : 'rev30', title : "Rewind 30 seconds" }
> > - .spinner{ dojoAttachPoint : 'spinner', style : 'background-color: transparent;' }
> > + %div
> > + %a.button.playpause{ dojoAttachPoint : 'playpause', title : "Play" }
> > + :if useHtml5Player
> > + :if !androidAudioFallback
> > + %a.button.rev30{ dojoAttachPoint : 'rev30', title : "Rewind 30 seconds" }
> > + .spinner{ dojoAttachPoint : 'spinner', style : 'background-color: transparent;' }
> > 
> 
> 
> I hadn't noticed before that this had an inline style. Is there a reason for that? It should be in the stylesheet. 
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Toura/mulberry/pull/532/files#r1182465
> 
>
another formatting nitpick: this would be easier to read and understand if it were on multiple lines
Good question. relativeID is a little different than parent in that it's more flexible. relativeID + the "position" constant allow clients to specify locations both in terms of parents or by siblings (Insertion position constants). NJ did point out in another comment that I need to add more detail about this constants since they are not valid in all cases. The usage between AddMenu and AddMenuItem is slightly different, so I need to document it.

-----Original Message-----
From: Peter Thiess [mailto:reply@reply.github.com] 
Sent: Wednesday, May 23, 2012 1:51 PM
To: Ty Voliter
Subject: Re: [brackets] Menu, MenuItem, key bindings, and Command API (review only) (#927)

> +
> +
> +    function _getHTMLMenu(id) {
> +        return $("#main-toolbar .nav").find("#" + id).get(0);
> +    }
> +
> +    function _getHTMLMenuItem(id) {
> +        return $("#main-toolbar .nav").find("#" + id).get(0);
> +    }
> +
> +    /**
> +     * Adds a top-level menu to the application menu bar which may be native or HTML-based.
> +     *
> +     * @param {!string} name - display text for menu 
> +     * @param {!string} id
> +     * @param {?string} relativeID - id of Menu the new Menu will be positioned relative to

Question: Is the relativeID something like the parent? I know we're not yet implementing sub menus, but maybe the API should have something like a parentID, even if those are top-level parents for now?

---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/927/files#r871114
Hey, while you're in this file -- it looks like there's an indentation error on line 1084 (JSLint flagged it).  Mind fixing?
Triple `=`'s, missing in this pull req.
it is changed when showing hintscontent https://github.com/ajaxorg/cloud9/pull/2149/files#L0R178
without this if something (e.g `:` command in vim mode) changes console value and focuses it hint for old text was shown
Can I merge the `module` and `exports` branches (since `exports` is already an object, and `module.exports == exports` by default)—or is it important to prefer `module` to `exports` here?
Though functionally correct, this (and the setter code) are not really dealing with types. Would probably be clearer to say mangleJavaName(field.getName(), true).
Aren't there two of you working on this?
oh! oops
Hmm, I'm thinking, because the nav itself is a list of nav item, and a nav item potentially has a sub-menu, can we instead of having a 'li's with divs, can we improve our semantics by just using lists (either 'ul' or 'ol')?

So we'd have

    <ul>
      <li>
        <a href="#">Americas</a>
        <ul>
          <li>
            <a href="#">Canada West</a>
          </li>
          <!-- Rest of the sub-menu 'li's here -->
        </ul>
      </li>
      <!-- Rest of the root 'li's here -->
    </ul>

That way we can keep the semantics.  It seems we're switching to divs for layout-related purposes.  :)

We can talk more about this if you'd like!
Right, your solution is far better.
Changed to just auth.
Initialize this var `_cachedValues = {};`
I am pleased to discover the abundantly obvious "history" button on github:

https://github.com/saltstack/salt/commits/develop/debian

^ ^ this batch basically started Nov. 5
I don't like this change due to;

1. I think this might be too special case to add a new function to the interface.
2. The interface already allows you do this without the added function.

I do however think this change does point out something which is unnecessary cumbersome to do with the current interface. Do you have an example of real code where you need/use this feature.
Why did this change?

This is the type of thing that should have a UT that self documents that this inspector responds only to *aaa* events and does *xyz* for such and such reason.
Why duplicate accepts() here?  In the flow of an accepted event accepts() will be called 2x instead of only one time.  I don't understand what the reason is folks guard inspect again with accepts().. I've seen this in a few places.
Actually why isn't this called LdapUserManagerTest? Convention and IDE friendly.
so the understanding is these must be available in third party repo? is ok to me since these are ITs
but here it would probably be easiest to do this:

public static final int[] TIME_INTERVALS = { 0, 3, 7, 14, 21 };
Is this used anywhere? I'd prefer if we could put it somewhere other than util. Ideally wherever it is used, but if that's not doable let's create a new file called AnnotationConstants or something like that.
code style: we don't use the in operator AFAIK normally, we  just use jsonData.propertyNames to check for existence
Is the 'password' declaration still needed, since it is included as converge_ui below?  I don't see it used in any of the pull request 'diff' content.
If a server has a lot of records to be updated using execute with a single SQL statement may be more efficient.
Did you mean to keep this consistent and use the Extensions instead of the Resources version? Same comment for 2 lines below.
Sure thing. I'm pretty fuzzy on puppet lang code style...
I made this modification because RestKit deadlocks if only one mapping operation is allowed at a time in the scenario I just outlined.

Notice that in my scenario, a request for a wish list is getting mapped while the earlier request for a user is on-hold waiting to connect the wishLists relationship.  So there is in fact only one mapping operation going on, it's just that the one is on-hold and blocking the other from being able to run.

... It is kinda messed up. :-/
I would prefer to load english only as default for now. The chinese one is pretty heavy weight. We can bias the default engine at flash time just as we set the time (using the locale of the dev machine).
Inkscape is a tool, copyright is owned by the author. 

I don't think this is created by you right?
call.rc fails. fixed that
Caching is certainly a good option for performance tuning later.  I should just go look, but does astropy have any caching utils available?  Seems like LRU with a specified amount of memory allocated for caching would be good.  Most out-of-the-box caching solutions I've seen just fix the number of objects instead of the memory used.
Remove the end comma.
因为这次不准备实现。
```python
url = isinstance(value, basestring) and urlparse(value)
return url and (url.scheme or not url.netloc or not url.path)
```
We are at 1.50 for Windows AFAIK? Perhaps you can create another small patch for this.
Not sure this is easier to read...
PHP 5.4 complains about `Strict (2048): Only variables should be passed by reference [APP/Cake/Utility/ObjectCollection.php, line 322]`. Just FYI.
No need for this really.  Returning null could actually be useful for checking failure to load view file.
Yup the selection is a bit funky. The `true` is needed to support the historical behavior, otherwise specifying a url or throwing an exception were the sensible options.
You're write, that still is a bit ugly. :)

Some suggestions:

1. Replace call to group.has_admin_user?(current_user) with use of cancan, and put it (and the redirect and flash message on failure) in a before_filter. 

2. Create a UserInvitation object. This wouldn't be an active record object, just use ActiveModel to get validations. Make this action look like a normal CRUD action. You're attempting to create a UserInvitation with the specified attributes. You try and save it. It's save method will either send the invitation (on success) or generate errors on failure. Those errors will be validation errors on the UserInvitation model as normal. You could add the validation errors on base, and then if save returns false just write those errors to the flash.
Should this be `self.widget().GetValue()`?
weapon_type_id
AccountBalance
You should name the parameters
OK, that's corrected.
Thanks :)
ups. its syntactically correct and does nothing, thus test did not fail... removed.
Get the path to .git from the repository object. Something like this:

```
public void Parse(string authorsFilePath, IGitRepository repository)
{
    var savedAuthorFile = Path.Combine(repository.GitDir, "git-tfs_authors");
    // ...
```
This part of the PR is fine.
So to be clear on the logic here: if there is a cached response for this request, then we don't want to use the cache.

If this is the case, why not always set the "don't load" flag?
I don't think emitting signals in the constructor makes sense? How could there possibly be already a connection to this signal?
Can't all this be removed and we just use `::Mongoid.models` ?
IIRC we actually require the 10.6 SDK in our AOSP build, but I guess 10.7 (or 10.8) worked for you in Mountain Lion?
I was trying to do something like that - but at the time my poor brain couldn't think up the right way to do it! Cheers for the tip :)
Why not just `user = bundle.request.user` - you've already got a user object, why query for it again?
faster solution is to rename the instance variable for what it really is: a tuple.  it's private and so will never be a generator.
stdout and stderr are just strings (clipped to length). What would calling repr on it buy us?
This doesn't seem necessary. Is there a reason for this change?
May be useless, but it does not hurt to be sure.
`$object ` on a property level validation is value of the property. I believe it can be different from simply calling `$this->property` in the `$method`.
Good question :) Maybe I had the impression that it has a "bad influence" on resolveAttributePath?

I can come up with a better name for this function. But I do not know if it makes sense for test - it's quite esoteric problem here.  Anyway, I dangerously approach to the violation of Dave Jones' first law :)

"Davej's laws of kernel hacking #1: If the number of iterations a patch goes through to get it right is greater than the number of lines in the diff, it probably isn't worth it."
both are executed every time, no ?
Looks like this is the remaining thing to figure out. Added the flag for now with default `null` and tests.
Can the context be retrieved from the DIC directly?
I agree with @Seldaek that `if`s would be better.
I prefer to have explicit SQL statements.
いらなくないっすか?
You're right: chars remaining count was not accurate on edit without this line so we changed the definition of `characters_remaining` (see line 3), which solved the problem.
computing -> computed
Hier brauch ich ein paar Erläuterungen
Thanks. I did intend to do that, but clearly forgot! :smile:
Pourquoi un seul ?
return route || null
почему не использовать уже созданные gem'ы для авторизации?
I was not at all certain removing this was a good idea - was simply trying to eliminate what I saw as unnecessary code. I *think* we want to see all draft pages no matter their publisher state, but it's still debatable.
why removing this ?
why do you allow failures?
I HATE comments!!! :-)
Why read-only.? To be backwards compatible it needs to go the other way as well, for write-attribute and for add. Convert the complex object into a filter-spec and write the filter-spec attribute.

The filter and filter-spec attributes should be declared as alternatives so an add op can only set one or the other.
I'm not sure why there are so many changed lines in this file, perhaps it was using tabs? Anyway, this is one of the two things I changed in this file
@sanbor If the user doesn't have Cufon installed or has opted into using native text rendering then `_render_native` is called rather than the original `_render` method. The code is pretty straight forward - what don't you understand?
Oh nevermind, I read that too fast. You can use `JSON.stringify(dictionary, null, 2);` for pretty print:
https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/JSON/stringify#Syntax
I realized that was not entirely clear.  We do want to call hide() after a timeout.  Just we can set collapsed rather than worrying about animating the opacity.
I'm wondering if having `app.downloading` would not be sufficient here ? and maybe even better because it would be done when the app is updating, right ?
Can you specify why is 9999?
I 've moved the clearForm to the end for clearing the text only when message sent success or resend cancelled. it should trigger resend action while message sent failed.
What about textarea?
If possible lets try to remove this dep in month.
...or have you found any bug there?
nit the name of the `create` function is a big misleading. And this apply to the `pick` function too now that we have 2 activities.

maybe `postNewSuccess` and `postPickSuccess`, or something else if you have a better idea :)
two-space indentation please
I think there is, @alivedise do you remember?
has arrived
tomorrow i can ask them in Madrid, good night Vivien
nit: missing `alt` attribute
Need is probably a strong word. I like being explicit, and I'm under the impression that being a little more explicit like this is faster for the browser.
trans block on its own line maybe? for readability!
I'd like to see some specs for the changes; in my day-to-day TDD process working on VCR, I don't usually run the cukes and I'd like to have some specs fail if I break it here.  You can see the current specs for ideas of how to do this:

https://github.com/myronmarston/vcr/blob/master/spec/vcr/test_frameworks/cucumber_spec.rb

On top of that, I don't really think of this as a separate feature that warrats its own cucumber feature file.  This doesn't add any documentation value to what's published to relish.  Instead, I'd like to see changes to the [existing cuke](https://github.com/myronmarston/vcr/blob/master/features/test_frameworks/cucumber.feature) to exercise this functionality.
1."服务端上传文件" 改成 "服务端/PC 端上传文件"
2. "客户端直传文件" 改成 "移动端直传文件"
3. 去掉 "网页直传文件" 部分
It is essentially delayed now, there is no overhead anymore. rememberInitialPage had the caching piece that has been moved to rememberPage so it is cached. the window.replaceState is not really an overhead issue
You could use pluralize too.
Did you populate and reindex the database?
「」 to ""
misspelled 'rasterize' (raterize)
Prefer to return Pipe since RichPipe is only an enrichment class for Pipe.
added that its to the local node.
Switch all this to using ```field_hierarchy```
How about:

Selected messages have been deleted.
Selected topics have been deleted.
wtf is `AL` and `ar`?
Try something more semantic.
I believe this requires a global install of nodeunit.  You can make it work for anyone using `./node_modules/.bin/nodeunit`
Should there be a reverse operation for the unformat() method?
Should test `state->socket` to make sure its not `NULL`. It should `return -1` when `state->socket` isn't set to maintain backwards compatibility.
Style: need a blank line above here.
@djmitche: Thoughts on the name?
On Sun, Jul 22, 2012 at 10:13 PM, Hanno Schlichting
<reply@reply.github.com>
wrote:
>>
>> -import ConfigParser
>> +try:
>> +    from hashlib import md5
>> +except ImportError:
>> +    from md5 import md5
>
> Ah, ok. I just saw the "import new" variant in other places.

You probably did.  md5 is is for "backward compatibility" and I think
new is somehow preferred, but I hate import ... as, so I prefer using
md5 in this case.

>
> I thought I remembered something about this versus Python 2.4 compatibility, but my memory was wrong.

This try/except is all about Python 2.4 compatibility.

Jim

-- 
Jim Fulton
http://www.linkedin.com/in/jimfulton
Jerky is better than bacon! http://zo.pe/Kqm
probably a single check will be enough

```coffeescript
for proto in utils.getPrototypeChain this when _.has proto, 'events'
  @_delegateEvents proto.events
```
You're right, I should definitely do that
Yep, you're absolutely right. Silly mistake.
Should be `if (...) {`
public methods should come first
what did you include graphael-rails for?
Should specify this is `shortName` (or whatever it is).
Best to file a JIRA issue, in which case it suffices to just mention e.g. `<!-- JENKINS-12345 -->` in the source file.
%{timestamp} means the 'timestamp' field (in @fields of the event), I think this may be better written as "%{@timestamp}" though perhaps as long as this special case is documented as such it's fine.
>>>>> Mike Burns <notifications@github.com> writes:

> In github.cabal:
>> @@ -140,8 +140,8 @@ Library
>> -- Packages needed in order to build this package.
>> Build-depends: base >= 4.0 && < 5.0,
>> time,
>> -                 aeson == 0.6.0.0,
>> -                 attoparsec == 0.10.1.0,
>> +                 aeson >= 0.6.0.0,
>> +                 attoparsec >= 0.10.1.0,

> I've had problems with these changing wildly between releases. Did something
> in Hackage change such that I won't have to worry?

I'm not sure what exactly changed, but installing github was wanting to
reinstalls a huge number of packages due to the fixed dependencies.

John
use (
Yeah even the original context-menu used sync messages so this is unchanged really. We could make it async but that would probably break compatibility with the API as well as make it possible for the context menu to appear before we've heard from all the content scripts so lead to some flickering. I don't know if it's worth trying to solve that problem now or waiting until it is a real problem.