labofsahil/github-event-dataset-2012 huggingface dataset:
Thanks, but I'm unable to modify the new theme. Whenever I try to save the edit I get a 404. Perhaps I don't have the sufficient rights as I did not create the file but it has my name?

Anyway, please alter the span.cm-property {color: black;} -> #92A75C
To further clarify, here is where the previous behaviour is problematic:
```
<input type="radio" name="radiotest" value="true" />
<input type="radio" name="radiotest" value="false" />
```
When serialized, if no radio is selected, we get {"radiotest": false}, if true is selected, we get {"radiotest": true}, and if false is selected, we get {"radiotest": "false"}.
просто к сведению. если передать в формтастик collection то as => select можно писать (формтастик догадается сам)
Because of this commit XBMC freezing after boot. I revert this commit and everything back to normal. SABnzbd-Suite 2.0.1 - OpenELEC-devel
`adduser` returns success. Why bother?
If you think our user names are not safe, we should add a prefix/suffix to RUNAS
I would *like* to drop the location from the ship table, but that makes the trigger and rule definitions problematic to define.
podrían poner un comentario en los fix para cachar que se cambio porfao o que se arreglo? saludos
Dude, this is not fixed! http://jquerymobile.com/demos/1.0rc1/docs/pages/page-dialogs.html#&ui-state=dialog
It's better, but you are already starting to "drift."  If you notice, brady, tony, rod and my master openemr branches *NEVER* have anything that isn't already committed to the official master. 

Your CRLF changes shouldn't need to have been committed, and commits like: 
Merge branch 'master' of github.com:zhhealthcare/openemr; branch 'mas… …
d52ce42f63 Browse code
jacobzh authored 5 days ago
also are added clutter.
I can easily cherry-pick just the interesting changes in this case and be confident I'm not missing anything when it comes time to get the code into official repository, but it's better if you can keep your branches "clean".
class variable that defines if the movie year is printed when its name is printed
Thibault-

It appears that we all get copies of comments on commits.  I thought the comment would just be received by magiclko, who authored the change.  Regardless, I already pushed this bugfix to dev.

/Tom
wa'alaikumsalam.
@scrooloose To debug, I created a renamed tidy.exe to something else (tidy2.exe) and created tidy.cmd with this:

	@echo %0 %* > d:\test.txt

This lets me see what is being passed by vim/syntastic to tidy.

Before my modifications, the output looks like this:

	tidy -utf8 --new-blocklevel-tags 'section, article, aside, hgroup, header, footer, nav, figure, figcaption' --new-inline-tags 'video, audio, embed, mark, progress, meter, time, ruby, rt, rp, canvas, command, details, datalist' --new-empty-tags 'wbr, keygen' -e "index.html" 

This is basically what you'd expect.  The problem is that cmd.exe does not recognize single quotes (').  To wrap arguments with spaces in them on win32 or win64, it requires double quotes (").  Unfortunately it doesn't seem to be something that can be fixed with shellescape() because cmd.exe doesn't recognize escaped single (')... As a result, tidy.exe doesn't parse the parameters passed to it properly and therefor syntastic fails to display errors/warnings.

After my modification, this is what I see:
	tidy -utf8 --new-blocklevel-tags "section, article, aside, hgroup, header, footer, nav, figure, figcaption" --new-inline-tags "video, audio, embed, mark, progress, meter, time, ruby, rt, rp, canvas, command, details, datalist" --new-empty-tags "wbr, keygen" -e "index.html" 

And after putting tidy.exe back and removing tidy.cmd, syntastic works properly.

Note: Another thing I tried - and it works:

    let makeprg="tidy ".encopt." --new-blocklevel-tags ".shellescape('section, article, aside, hgroup, header, footer, nav, figure, figcaption')." --new-inline-tags ".shellescape('video, audio, embed, mark, progress, meter, time, ruby, rt, rp, canvas, command, details, datalist')." --new-empty-tags ".shellescape('wbr, keygen')." -e ".shellescape(expand('%'))." 2>&1"

Maybe you'd like this last solution better?  (Fortunately shellescape() knows to replace the single quote (') to double quote (").)
I think you would have to have it like so:

Navigate -> Get to Next Page -> Purge

so _navService.PurgeAllBackstackPages();
or _navService.PurgeBackstackPage(PageName);
অনুবাদ
and tried to fix High Overlord Saurfang's vehicle entering. fixed, by problem in something else when casting spell Control Vehicle, need to research (maybe faction problem or some passive flag)
So, GO 176360 Still going to be spawned by db?
Sorry: 0.019 ~ 1 Week
seems to be a little silly to add a dep like this for 3 lines of code

      s = TCPSocket.new(Deployinator.graphite_host,Deployinator.graphite_port)
      s.write("deploylong.#{stack}.#{type} #{duration} #{Time.now.to_i}")
      s.close
default shouldn't be QT_TESTING_EVENT_PLAYBACK_DELAY ?
That was only with BigNum with to_hex and to_word. Then _hex is not an attribute of Byte either, and I should think of it as a conversion too, but really use the "cached" version?
No more luck. Note that the bytecode file I've got does contain "_src_": `ebin/cb_who_src_mail_view_confirmation_txt.beam`
Don't use Tom G's source - one of the commits (not sure which) has caused the problem of rebooting the phone. Fork my kernel source instead.
Please use JavaDoc for comments like this one.
Hi,

Actually, reading the manual once again, I realize that step 3 for Windows is more confusing.  What is meant by 'still as _postgres_ user' there? What if I reorganize it to more align with step 2 in Windows section?
If you just toggle, you get wrong highlighted lines if all except one line are checked and you then check the "check all" box. Then only the previously unchecked line is highlighted and all others are in non-highlighted visual state, but still checked.

Also the attr function does not return on every browser the element it was called on, especially older ones.

While the attr function can be a show stopper, the highlighting of the lines is just a little incorrect, it looks slightly better this way ;)
The new enum is missing though. I forgot to add it to core (see WPP for it)
```sql
-SET @AT_AREA_52_SOUTHSET := 4472;
+SET @AT_AREA_52_SOUTH := 4472;
-SET @AT_AREA_52_NORTHSET := 4466;
+SET @AT_AREA_52_NORTH := 4466;
-SET @AT_AREA_52_WESTSET := 4471;
+SET @AT_AREA_52_WEST := 4471;
-SET @AT_AREA_52_EASTSET := 4422;
+SET @AT_AREA_52_EAST:= 4422;
Figured out who the credits go to, it's @Fredi. :)
https://gist.github.com/1497219
Yes, commented out
Shouldn't this test be included in the superclass' spec, Cuore.Service.Spec.js, since the emit method is inherited? (I mean, the superclass will define this behaviour, right?)
No worries.  I'd honestly be surprised if a lint-like tool is able to pick up these conventions.

There is a reason why method names can be prefixed with _ (underscore) and __ (double underscore) in Python.  The former means that the function's docstring won't be evaluated by default (sort of a protected member, however it's still exposed) while the latter means the function is not exposed (like a private member).

As for uppercase global variable names, this is compsci 101.
Yeah, will do.  Mind if I update the git ignore to add those files, the same thing happened last time, I commited.  I think it may be easier for everyone.  Nice catch on the abort, I was messing around with the test runner.  Kinda weird how it didn't affect how the application.  Ill get to it a bit latter today.  Nice moderation.
This is a usefull but slow in-Array searcher. Use it only if Array.buscar or Array.donde are not good enough
It looks like the edu.mit.blocks.workspace.WorkspaceEnvironment file is missing
I commented on this already, but isn't it dangerous to modify auto-generated code like this?
We should investigate using the type for this instead of regular expression heuristics.
Yes, but we must add the new request to queued requests to be processed in order of increasing RID. A request with a future RID may arrive first.
I would create a \@PostConstruct method and move the assignment of jetty to server.get() there.  It is safer than having this in the constructor as something that performs i/o in constructors causes really hard to troubleshoot issues.  the method annotated \@PostConstruct will be automatically called after all injections on the whole graph are performed.
Part | Points
------------- | -------------
base | 1
*Total* | 1

The distance function didn't hook into the unittests but it works fine.
blablabla 12点半了
This is cool.  Would you like to contribute this to tcollector so that everyone can benefit from this MongoDB collector?
Should be if `$app` exists.  A non-existent ID here would not produce an error to the user.
Well in case Opera crashed midway and we want to continue not from a first page.
Thanks. The file names definitely threw me off. The confusing bit was that the MappingRegistry still existed in master even though it wasn't used anywhere. Looks safe to remove, though.
Great -- this makes much more sense separated into its own class.
What is the purpose of this validation?  Is an invalid value considered an error?  (Right now it's just ignored -- but should this be stronger, e.g. an assertion?)
This has nothing to do with tdata.  Was it a mistake?
If transition target only shows transit to "tabscene" view, but "tabscene" view has defaultView, the defaultView should transition to. 
So set subIds to next.defaultView.
Hey chase,

I'd like to keep working on it. A couple people have offered to help. What would you like to help with? What are your strengths and time coming up?

Sent from my iPhone

On 4 Jan 2012, at 19:10, Chase <reply@reply.github.com> wrote:

> Booo.  Did anyone pick this initiative up?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/fakedarren/mootools-website/commit/f26f948e8ea903829791758cea3387a14b257ca6#commitcomment-837758
We should do a default fail and explicitly list which OS's we support. Such as

case $::operatingsystem {
  'debian': { # set variables with debian values}
  'redhat', 'centos', 'oel': { # set variables with RH values }
  default: {
    fail("${module_name} supports debian, redhat, centos, and oel. Your operating system is ${::operatingsystem}")
  }
}
If it is null or empty, set the access control list to default value (GLUSTER)
Please don't forget indentation.
Does this override the setting in cmsettings ?
also with default disabled, i get laggy scrolling with short swipes in market or root explorer.
I suggest using the patch from cm7 which decided automatically to use or not to use the scrolling cache based on a few expressions and the scroll velocity.
@kali- You want this change :)
Should I make another (new) pull request with the new commit that will concern Your comments (removal of the unnecessary lines, the two unimplemented methods)? I'm not very familiar with the github so if You could please give me a hint on it I would appreciate it...
nice. I added this to the LESS, and SCSS, as well as the old-css file in the coming update. Thanks!
Hi, can you send a pull request with this commits, please?
This should be a Runnable, not a Thread.
The helpers does not always give us the kind of code we want.
look at my last commit :P
Yea... sounds weird, but check this out:
https://github.com/ferrislucas/BenningtonModel/tree/master/src/ExampleFeature.Cms

I'll have to add some docs, but long story short it's just a quick
easy way to get event sourced CrUD. The idea here is that you would
code your input model and editor template, scaffold the rest, and
you're done.

On Tue, Feb 21, 2012 at 8:12 AM, Darren Cauthon
<reply@reply.github.com>
wrote:
> An input model domain object?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/digitalevolutiongroup/Bennington/commit/3a8ddbcf9226f942097e7f8230601aefc0008891#commitcomment-991131
Não é só trocar <%= semantic_form_for @aluno do |f| %> por <%= simple_form_for @aluno do |f| %>
e trocat "f.input" por  "f.association" ?
You probably should use a thumbnail without title when BeautifulSoup is not available, instead of displaying an error
Hi, all

Thank you for the discussion. I learned much from it!

I chose red to indicate accepted because it is the culture of the online judge site (http://acm.hit.edu.cn) in our university. And for Chinese, red means congratulation. Seven days ago, I just sent my nephew some money in red pack to bless him in the Chinese new year.

In the latest commit, I applied the official css style, .notifysucess and .notifyproblem, to show the judge result. Therefore, the color is defined in the theme chosen by admins/users, and the most cooperative with other colors such as background. 

I think this solution is good for all since everybody has the chance to choose his/her own flavor.
Has too much of an intro. Planning on introducing an "auto skip to next song" feature soon so this won't do sorry.
Are you extracting files from a different device? The video stuff is in system/lib rather than system/vendor/lib on the charge.
Oh, never mind the latter, I guess we've had that in `setup.cfg` for a while, I just didn't realize it was there.
You only need one set of back-ticks around `initializer()`.
See my note above please.
Can we do that in this merge?
Nice. This explanation about dead letters was really needed.
remove semicolon and add newline
Needs ScalaDoc
Is this guaranteed to only be invoked once?
Can't be empty. It will at least have the node you are currently talking to.
perhaps also add something about "don't pay for what you don't need", I mean at-most-once is cheapest, and other things can be added on top when needed, but not earlier
maxOption is used in the yield, and this line "converts" it to the right type
Is there a disadvantage to write this inside the `for` compared to doing it in the `yield`?
does it matter which exception is thrown?
triple negation?
case Failure(e: AskTimeoutException) is a very generic type of message and I'd recommend wrapping it in something more domain specific if possible.
"<>" = "!="
This is Python 2.7-only syntax; something like that would be Python 2.6-compatible: 

```
self.vowels = set(['a', 'e', 'i', 'o', 'u'])
```
This line is too long. I know sometimes things get a bit crazy in the templates but this is just silly can it be reformatted to be readable say here on github
This looks odd...
Why 2 ?
Why is `top` set to `false` here? This prevents generation of paragraphs (text blocks are instead reported as `type: "text"`). A custom extension could be to use `markdown="block"` and `markdown="inline"` to discriminate between those two use cases.
Please give reasonable names to your parameters in header files, otherwise a user will have to dive deeply into your code to understand what to pass to this function.
this should be immutable.IndexedSeq

    scala> val s: IndexedSeq[Int] = new scala.collection.mutable.ArrayBuffer
    s: IndexedSeq[Int] = ArrayBuffer()
not too worried about speed here, as we don't get the mousewheel events on mobile devices where the speed is the biggest concern
That line should be fixed too, then :)
Alright, I've added a pair of methods to allow preparation for responder change, and I've moved them into `TUITextEditor` using these methods.
As above, no need to convert from NS to CG.
I would like `conn_` better.
We should handle the returned ChannelFuture to fire an ExceptionCaught event if the handshake failed
This comment has a grammar issue. Can you fix / rephrase?
All `coverage` should be `overlap`.
зачем ему передавать ElementImpl, если он его сам может получить по NodeElement'у?
:+1: on sticking to Rails defaults with underscores.

On Dec 17, 2012, at 4:32 PM, Rafael Mendonça França wrote:

> In railties/lib/rails/templates/rails/info/_routes_wrapper.html.erb:
> 
> > @@ -0,0 +1,57 @@
> > +<style type='text/css'>
> > +  #routeTable td {padding: 0 30px;}
> > +  #routeTable    {margin:  0 auto 0;}
> > +</style>
> > +
> > +<table id='routeTable' class='routeTable'>
> I know they use dashes, but all the Rails helpers generate with underscore, so is better to use what Rails use.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
No worries.  I've pulled in this commit and fixed up the assertion in a3ad0a7b6718eef9462b519189cd67e2ca29bc28
Thanks for fixing the whitespace problems in this file :)
Because swing is part of the release and we're still supporting JDK 6, thanks to the whims of industry.  

If I add a new setting that disables swing for JDK 7 is that ok?   Won't be today, probably tomorrow.

Also, It's not a broomstick, it's a tree branch and it's in your bike spokes because you're not driving on the well-beaten path.   

I agree supporting Java7 is important.  I use it for all my projects.   The Scala build, though, is a JDK6 build right now.   It also fails to work for JDK7, so only advanced or very advanced, or off-roading maniacs try to build Scala with it.    I'll get you some support as soon as I can.  Today, I'm making sure M4 is ready to go.
I imagined isClass on a StubSymbol would throw the MissingRequirementException
How about making the warning more specific, e.g. `Annotation is discarded - use the @getter meta-annotation to preserve it`? 

Also "scala meta annotation" seems excellently googlable. Maybe we could omit the reference to `scala.annotation.meta`?
Note:  it seems this method might be useful in a base partest class.   Mind if move this then?
```
else:
    raise TypeError('Unknown hint: %s' % hint)
```
```
bell(6, 2, symbols('x1:7'))

or

>>> bell(6, 2, symbols('x:6'))
6*x0*x4 + 15*x1*x3 + 10*x2**2
```
(See note below regarding `symbols[0]`.)
I don't see where this logic is now.  I think you left it out.
spaces go after commas and not before.
Let's add the following logic to `Tr.__new__`:

```python
if not isinstance(args[1],(list,tuple)):
    indices = Tuple(args[1])
```

This is enable users to do `Tr(d,0)` if they are tracing out a single index.
In fact, replace it for every panel that is not correctly defined.
Not sure I understand this name. Maybe I'm just dumb.
My eye stopped at this. It doesn't matter in this particular instance, but I tend to prefer `!==` to `!=`
Please call this method getStudyOutcomeMeasure() since OutcomeMeasure is an interface on the global level (implemented by Endpoint and AdverseEvent), which is already confusing enough!
`malloc(sizeof(*stream->select_cb))` - which you will have renamed to select_async or some such :-)
Do we need to cache navigation in the admin?
Strand 011 and not 2011
I think "many thousand" is better than making it sound like we don't know what happens.
I like pessimistic concurrency? should be more clear?
When you assign a variable to a query....
@seth Makes sense.  Although, part of me is wondering if this might make more sense as a mixin so that we can make most of Chef's core objects sortable. @danielsdeleo ?
That code was already deleted from the PR.
This seems too fragile. Do we already have similar code elsewhere?
@imkingdavid Oops - there is a DOT here, should be a comma.
Ended up with this cause sigkill didn't work. But I guess no harm in sending both.
err of course supports_phrase_search(), confused our coding style with another ;-)
Space after `(int)`
What happens if an attachment is deleted from the post after it is reported?
We don't use underscores in such properties elsewhere.
This appears to clear microdata if it is set?
ideally this tested more than just `asserts.ok`, like check that it is a valid directory path? `fs.stat` it?
It's not really just the constructor here ... maybe change method name.
https://github.com/uxebu/bonsai/commit/28176b18461112e7101731f1472b15aa2151e0be
Sorting by Opd number, is different than sorting by uuid?
or any eclipse specific files such as .prefs below
PEP8: Surplus spaces.
Should this be `ctypes.c_char_p`?
Could simplify this a bit with:
```python
args = sorted(arg_text(item) for item in self.dependencies.iteritems())
return '<{}({})>'.format(type(self).__name__, ', '.join(args))
```
Something I realized about third-party Ops is that they may have to work with different versions of Theano.
Since, on older versions of Theano, there is no proper way of indicating an input gradient is undefined, or that this input is disconnected, I think having a config file flag would be the right way to do it (the flag would be ignored by previous versions of Theano, the parameter to tensor.grad would not).
I don't really care if the default value for next release is "warn" (and changed to "raise" later) or "raise", I think "raise" is reasonable.
I don't like setting the presenter of the view in the presenter constructor. I think the constructor should be used only for assign dependencies. this line of code could be put in somewhere like the start method of the activity.
Yeah, you're right. I'll remove it.
The correction for wrapSubtract(a) is in this pull request.
We want to lint current JSHint source code with the latest stable version hence `"jshint"` and not `__dirname`.
The METEOR_PATH environment variable allows developers to easily switch between their stable Meteor installation and their Meteor development bundle. This environment is also introduced according to the KISS principle, as I doubt you can easily figure this out automatically in CMD.
I don't think we should return at all here. There is no documented return value.
Oops, this was an idea I had initially but wasn't able to get enough data to complete.

Do we care about different distributions of Linux? I don't think it is right to set platform to "Ubuntu", because it isn't that different from other Linuxes.  If we do want to track distribution, it should probably be it's own field, I guess. In my sample data, Ubuntu was the only distribution that put something in the user agent, so I don't think we will know distribution very often.

choices: nix tracking distributions at all, or accept that we can only track Ubuntu and do something like "distribution" or "sub platform".
@ohwillie v1.0.3 has been tagged on github, so you can use the tarball download link to get a stable url of a version.

https://github.com/rwos/gti/tarball/v1.0.3
HTML is an acronym, so it should be written using uppercase letters.
What's the reasoning on this move? The alphabetical-ness was correct originally.
取り込みます！
Because the value will be out of context I going leave the # FIXME note instead of a const
Fewer files forces simplicity! And the files should be small too!
Whoops, this should be

    "Create your own qualifier annotations, **or** just use `@Named`."
I still wonder if this line isn't horrible, and we need getModuleAdapter() to return a properly configured module.
You can probably remove that commented-out line if you are adding the next line (link to static library) anyway.
The exception checking sections above are waste in my opinion. If a customer number is null the NullPointerException will be thrown in line 36 and the caller can reasonably assume from that exception.
In case of subscription id, the missing of it is OK.
you can assert (rc != -1) and drop the seen counter
update this URL to developers.hubspot.com
Do we need to lock in `GetEntity`? The other ones make sense, but I'm not sure we need to lock here. Is it more TableStorage-like to not lock for `Get`?
unused ``suiteName``
``allFunctions`` was changed to a property bag now instead of an ``Array``.
Typo - "Intrnal"
Done. Also cleaned up how the promise is handled (so it's clear that if it got rejected above it won't resolve below).
It can work for commenting, but it wont be possible to remove them on uncommenting, since you cant know if those were added by the user or by the code.
Does this not strike you as cheating? I think the test suite pass with the code running and stickifying the way it will in production.
Log! Log for your files!
Why isnt there similar code for the showDetail, addPost and editPost methods? 
Also, why is the same code repeated in two places? Instead of, say, having a method that does this work.
Junk character, please remove.
Rename to A2
I found that rb_raise accept print_f style strings, so maybe we can use it instead of converting to std:string
It isn't clear to me what this is. I wonder if a better name might make it become clear?
move to the top
As above - maybe include this with JS?
Wywalić wszystkie metody is_frontend
Since this exception gets logged for administrators, I think it is important to retain the mention of the repo ID value. Both name and id wouldn't hurt though.

Suggest this should read something more like:

"Repository [id= + id + ",name=" + repository.getName() + "] + "' cannot be deleted due to dependencies: shadow repository[id=" + shadow.getId() + ",name=" +shadow.getName()+"]."
Should this be $ENDTIME $ENDTIME?
This should probably be titled ``test_post_video_not_authenticated``.
Then this should be vap->va_gid?
I'm wondering what if attribute was dirty and also changed in hook? I think better solution would be passing additional argument to hook and modify it instead of object itself. What do you think?
This used to have placeholders that were overwritten by the packages.

For example:

DATABASE_PASSWORD = 'THE_DATABASE_PASSWORD'
query-cache-ttl also needs to be lowered with locations.

In general, we need to be more explicit that when you have a timestamp or locations in your data.cdb, you need to change these values.
Have you dealloc them?
I think they need to be dalloced in dealloc method.
(I found that some other variables also need to be dalloced)
Well, the problem is that make also uses `$()`. However, what you can do is use `$(shell $(JULIAHOME)/contrib/filterArgs.sh $(LDFLAGS))` which is a make intrinsic.
discussed and this is temporary.  ACK
sure, will update
Can you describe this prioritization somewhere? in man page of --help output. Just in case when user enter both label *and* id and they may wonder what will take priority.
Not uppercase though...
I was copying the logic of the Ruby launcher (except where explicitly intended).

`Properties.load` has a more extensive syntax. I can use it instead if you prefer.
I didn't know there is a `propertyName` property!
What about building an object that contains all the apps directly instead of relying on a makefile target? This shim will be used only for old version of Firefox and/or online demo. Otherwise you can start hacking by doing |cd $GAIA_DIR && make && firefox -profile $GAIA_DIR/profile| and everything should be set up automagically (even the pre-installed applications)
MonoDevelop doesn't offer any conceptual distinction. Os is what's desired here, but it doesn't have that option. It's something I intend to add to MonoDevelop.
Maybe I'll follow Android lead and make Optimize and Size both O2 for the time being.
This should be the style for the "small" tag. It's not always a good idea to apply font sizes to the entire footer, especially if you want things to be different sizes.

More of a nitpick.
imports from scipy should occur at the function level rather than the module level, so that people can at least import the module even if scipy isn't installed (scipy is not a strict requirement of astropy - only numpy).
When I build this in sphinx it issues a warning ``WARNING: Duplicate explicit target name: "astropy".``.  I think this is because sphinx is confusing this URL with a link to the `astropy` package.  An easy workaround is to change `` `astropy <...>`_ mailing list.`` to `` `astropy mailing list <...>`_. `` (which maybe makes more sense anyway)
method和function有点搞，要不叫被插函数，和插件？
还有，我们的plugin到底是什么？是一个函数还是很多函数组成的类？
`$plugin->getPlugins();` 有点怪
else 前面多了个空格吧。
There is a lot of history. Went from gen_sever to process per stat, to no process at all (just call folsom) to all these spawns. With 3 different metrics providers in the back ground. All load tested for 10s of hours. None of it satisfactory.
Actually, in retrospect, it would be best to `io_lib:format/2` the filename, then use `filename:join` to construct the path. Same for the other spots in this file.
Good to know, thanks :).
Hm I'd prefer passing the name of the lock ("criticalblock") as a macro parameter to LOCK, TRY_LOCK and IS_LOCKED. Especially the bare IS_LOCKED (with hardcoded name) look too much like magic to my taste, and can't be used with LOCK2 in this way.
Don't remove this, it's required for static builds !
Can you please document/explain these two macros? I don't know what they do.
why are these methods here? My intent was that serialization is entirely opaque to the user of the pipe (i.e. the widget). The enaml widget shouldn't care where its client lives.
At a quick look, here and in magic_paste are the only code where `TryNext` is caught and captured. I think the ability to change args/kwargs is intended to act within `CommandChainDispatcher`, not to propagate to calling code, and this doesn't affect that. But no, I don't think it's actually used at all. It's hard to see a use case that maintains weak coupling. Maybe it should just be ditched...
oups... yes
the default for `language` should probably be undefined or null (something falsey), then a statement more like:

`language = languages[config.language or getLanguage(source)]`

This could cover both branches of logic (stdin/file).  

While we're on the topic of specifying languages, if this is an available option, the command line help should print out a list of supported languages that can be specified.  We don't need to send users digging through the source to figure out what name to specify for a given language.  

Manually prepending the '.' to specified language also seems a bit brittle.  Perhaps the language could be specified by the 'name' paramter given from the language object itself, e.g. 'coffee-script', 'tex', 'sass', 'javascript'.
probably there's no default identity or credential, you can change identityName to "user to ssh the end point in (needs root)" or something. similarly credentialName and endpointNamt
"whith"? Sorry, the rest looks pretty good, this is the best criticism I could come up with.
not sure is this meta test above right or not. I thought transformed seq are new and do not retain meta data. that's my interpretation for now so all returns nil.
I think the only thing we are missing here is the ability to execute blocks as well, and not just providing symbols that point to method names.
All Xcode 4 and later. Could skip the stuff about installing the command line tools on Xcode 3.

Eric Shepherd
Sent from my iPad

On Jul 1, 2012, at 1:29 PM, Marshall Culpepper <reply@reply.github.com> wrote:

>> +  else
>> +      echo "You will need to install \"Xcode 3.2.6 for Snow Leopard\" to build Boot2Gecko."
>> +      echo "Note: This is a 4.1GB download, and requires a free Apple account."
>> +      echo ""
>> +      prompt_question "Do you want to download XCode 3.2.6 for Snow Leopard in your browser? [Y/n] " Y
>> +      if [[ $answer = Y ]]; then
>> +          run_command open https://developer.apple.com/downloads/download.action\?path=Developer_Tools/xcode_3.2.6_and_ios_sdk_4.3__final/xcode_3.2.6_and_ios_sdk_4.3.dmg
>> +      fi
>> +  fi
>> +  echo ""
>> +  echo "After installing Xcode, follow these steps:"
>> +  echo "  1. Run Xcode to allow it to configure itself."
>> +  echo "  2. If it asks for permission to update the command line tools, let it."
>> +  echo "  3. If it doesn't, open the Preferences, go to the Downloads panel, and install them."
>> +  echo ""
>> +  echo "Then run this script again to continue configuring to build Boot2Gecko."
> 
> Hrm, were these steps necessary in Lion / Snow Leopard as well, or just Mountain Lion?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mozilla-b2g/B2G/pull/52/files#r1081899
Not sure what you're talking about.
So I'm not sure how my comment got attached to this line.  Refering to code like:

error = InstallError('Failed to install %s' % src)
raise error.__class__, error, trbk

error.__class__ is just InstallError
I guess it is used, but only for `API_VERSION`. In that case may as well make `API_VERSION` a module-level constant, and probably put it in `moztrap/view/api` - the `MTModelResource` class doesn't seem to be needed.
except that a derived class later uses the _do_get_raw_dumps inside a loop with varying values for the 'crash_store' parameter.
This is kind of confusing. Navigating to this URL with a clean cache/cookies shows a form labeled "Sign In". Although this sounds more like a Persona bug, is there any way to get it to be a Forgot Your Password form directly?
This domain should go in the settings file with the standard persona login domain.
You're making this call multiple times per pageview.  It's easy to memoize:

```ruby
def info
  @info ||= Sidekiq.info
end

def processed
  info[:processed]
end

def failed
  info[:failed]
end
```
Shouldn't this be an else INIT_TOOL='update-rc.d' ?
you should not remove the empty line
this is wrong as it relies on the fact an error handler is registered (and turns errors into exceptions), which should not be considered as a requirement
missing spaec after ``if``
Because later on, `threshold` is controlled by HTML5 `localStorage`.
IMO `resolveType` shouldn't be part of the interface and the logic should be included in `addType`.
Resolving a type but not adding it to the registry that you called it on makes no sense, does it?
With just `addType` and `getType`, it is more clear.
@fabpot: Done.
This needs to be in requirements.txt and there should be note about it in the pull-request
Its only as big as it needs to be while contentArea-bigpage sets a defined size
kill this if we don't need it for now.
Agree. Updated, but this didn't update for some reason.
哦，对，红艳说的对，这里也是笔误，写错了。
然后就是，注释想强调传入一个object 该方法要传入的key值一定是那四种，object当然内部数据的顺序是可以变的。
I guess my point was more that I saw no reason to support them in PlasmaShop 1.0, which is why I removed them for PlasmaShop 2.0.  They are, after all, plain text, and aside from possible syntax highlighting for the standard format ones (xml and inf), we don't do anything special with them.  However, I suppose it doesn't *really* hurt to have them openable anyway, in case people still have those files laying around their games.
I will add error printing later, that is why I did it.

if (...) {
    A.error (...);
    return false;
}
Good question, I'll see how many I can remove.  Stay tuned.
Hmm. Do you think this ```try..except``` should be the default behaviour of ```get_extent```? I think I would support that.
Enchantibility? :p
try `rand() % num_pubsubs`

conversion to float and back seems like a less obvious way to see the range the result would be in (and there's a tiny chance that rand() will return rand_max and thus choice will be out of bounds by one) (and if float isn't auto-promoted to double, there might be a bigger intermittent problem)
why is this change in this pull request?
@cached_property may be useful here.
This executor service should take a ThreadFactory so the threads can be properly named in a thread dump.

Also, there's nothing in stop() to shut down this ExecutorService, so it seems to be counting on the gc to do it via the finalize() method on the ExecutorService impl. That's fragile.
I don't believe there is any real file-size or clarity benefit from using a map. Besides, doing so would add the (negligible?) overhead of creating a new object. Since this code is run on every mouse move, we might as well error on the side of performance, right?
This is random. Each one of these can potentially be unset, not just the ones you address here.

This is an upstream bug (tobie/ua-parser#93) that I already fixed there, pending review.

For now, lower `error_reporting` to not include E_NOTICE, or filter out ua-parser (which is what I do).
Include `()`. Also, are we relying on default parameters being implied or does it really not take any arguments? Because in the latter case I'd rather hardcode the parameters we intend to pass here since it is third-party code.
Ok, we can talk about that tomorrow in the office too, no need to rush. We need to wait for approval anyway as it's not bb+.
err, you're dumping with innerHTML. Still add a name since this is a good practice.
@fcampo, you want to kill me, but the github comment editor played bad with me.

The correct format for the if statement is:

if () {

}

On my previous comment it really went wrong :(
@vingtetun Please remember to see the latest comment!
Isn't it much easier to hide the edit button or remove the edit button event binding if you don't want to test the edit only for this week?
Updated in order to explain better why is there. Thanks for the tip!
// XXX: respect date format of the locale
       this.daydate.textContent = d.toLocaleFormat('%A, %B %e');

This replaces L45 - L48
I have never understood with it is needed to add a class to the body tag?
`window.parent.FakeAlarmManager.snoozeHandler()` ?
Arrr... Now it's used in Utils for checking 'contacts' ... I think that we could remove it once the library will be in Gecko
Nope - [MongoDB uses BSON internally](http://www.mongodb.org/display/DOCS/BSON), which gives them some advantages.  Since PostgreSQL < 9.2 can't use either natively, and 9.2+ can use JSON but not BSON natively, it balances out.

Really, though, the way that a persistence driver chooses to encode data for storage shouldn't be visible outside the driver - if BSON leaks out into the rest of the code we should work to purge that, so that it is possible to implement, eg, a Riak store without also having to use BSON.

Riak, FWIW, can't look inside BSON (easily), but can look inside JSON, so it would be strongly preferable to use JSON there.
It seems like you can either store options in the member `@options`, or pass it as an argument, but that doing both is probably more confusing than helpful.  (I know this mirrors other brokers, but quality has to start somewhere. ;)
This is the latest update to the code and SUPER IMPORTANT
ObjectOutputStream has to use writeUnshared, or the ObjectInputStream WILL THROW an InvalidHeaderException if you use a format for the Double flags other than #.##
So if you use 100.0, or 100, instead of, say, 100.01 or 100.00, it'll throw an error. This edit fixes this issue, and prevents any similar issues from cropping up using different data types. Each object is serialized individually using this method.
Visually:
[check]  Audio Input
  Audio Mixer [Dropdown] [Setup Button]
I like translate, but I would call the variable a "translation" rather than a "translator".  The variable is not doing the translating itself; the variable is an alternate version (translation) of the target variable.

On Jul 13, 2012, at 2:18 PM, John Freeman wrote:

>> @@ -0,0 +1,61 @@
>> +(function () {
>> +
>> +  var proxies = hd.__private.proxies;
>> +  var factory = hd.__private.factory;
>> +
>> +  var Translation = function Translation(source) {
> 
> Maybe we could call `scratch` `translater`: whatever you speak (assign) to the translater will get translated for its subject (target). Just an idea.
> 
> I think "translater" works well for when the resulting proxy is supposed to be the untranslated source, i.e., when the focus is on its outgoing validators. Perhaps "translated" will work well for the opposite direction (but implemented with the same proxy of course).
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/HotDrink/hotdrink/pull/41/files#r1161355
actually, i show only the plugins enabled. I will do the change to show all plugins and if it's enabled or not
This is part of the pi module, so I think it's better to include it in pi.modue, as otherwise we break module encapsulation...
Sure, this code surely needs more love.
This should probably read "None or SKIPPED, optionally via Deferred", so it's clear that None and SKIPPED are also options for Deferreds.  Also, no tabs plz..
No - if you add a skip to `Tests`, you'll skip it for fakes, too - not what
you want!
And do we perhaps want to ensure that only a single master sends this message?
I mean `../foo` works fine for me in master, which one of these was failing? the extra noise from taking out the extension makes it hard to tell
Please don't waste time aligning docblocks it's really annoying when modifying code, and it really doesn't provide much benefit.
Yes, was not aware.  So what if you want to avoid using bundle exec?  How do you disable?
I know this method has been added in a previous patch, but it's broken: it's not sufficient to just use the average Z when going from 3d to 2d, you have to sample it just like you do with x and y.
If the system is east,north,height missing to sample the z will cause some issues in case there is a datum shift, since a different Z above one ellipsoid will cause a different x,y projection on the other one, if the system is cartesian x,y,z (like the one used by the GPS satellites) then the problem will be even more evident.
Can we convert to bean's Event object?
This function should be accompanied by a test for it.  Since it's purely functional, takes a string and returns another, it's easy to test.

Furthermore, it needs a slightly more complete docstring that follows the [numpy docstring standard](https://github.com/numpy/numpy/blob/master/doc/HOWTO_DOCUMENT.rst.txt) that we use for all of IPython.  You can find a bit more about these details in our [developer guide](http://ipython.org/ipython-doc/stable/development/doc_guide.html).
No Javadoc, `@since`?
Also recommend using `@Nonnull` on the return value here if that is the intent.
Prefer `—` (m-dash) rather than `-` (hyphen) for this purpose.
is_yaml doesn't need to be an instance variable
raise, not throw ;)

'throw' is similar but different in ruby.
So it will be like:

```js
if (isMatching(tab.url, this.include))
  this._onContent(tab.content);
```
s/decedents/descendants/ here and elsewhere
I don't think chrome module is a goog placeholder for these as it doesn't give any priviledges and has nothing to do with existing attributes exposed by chrome. There may not be any ideal solution for this, but I'd prefer:
 - exposing them in require('@packaging')
 - a new specific pseudo module require("@base64")
And for these two options, this hack would only be used by api-utils/utils/data module.
Or expose atob and btoa to all modules by adding it to !globals.js ?
@Gozala What's your thoughts on this topic ?
Would be nice to add `x-default` to the
https://developer.mozilla.org/en/XPCOM_Interface_Reference/nsICommandLineHandler
It may be worth to talk about heritage classes. You aren't only checking `interfaces` for the given instance,
but classes specified in `implements` and inherited interfaces from `extends` parent.
This comment may be added here or in hasInterface method.
Or just `prefs/events` that implements `event/core` like API.
It is a usefull feature if we are using it. But it seems that base, which is either: requirer.uri or baseURI are always normalized.
Mutating things in an outer scopes later does not sounds like a good idea, usually makes it hard to figure out what changed when. Can't we avoid this in some way ?
This dependency is also should not exist in this module. If you'll move this function elsewhere it may be ok, in such case runtime require still will be problematic with ES.next modules. If that will remain necessary please submit a followup bug and point to it with a comment.
This comment no longer seems accurate
I think so, maybe you can help phrasing it better.  What I'm trying to say is that promise can be either rejected or resolved and that can happen only once, meaning that you can't resolve once and then reject once it's either one or another. I think without end it becomes less clear.
the alternative implementation tactic would be to always go through the getPublicKey call.  This would be problematic, however, because that code checks for authentication and provisioning urls.  this was the approach that affected the least number of lines, but we should consider how to simplify and improve in the future.
These changes are API that third parties might have used, so we need to document it, and ideally provide a deprecation warning and not blow up when the API is used.  For one major version.
This should probably massage the backtrace in the exception, so that it doesn't include this function - since, after all, it is the parent function frame that the problem occurs in, uniformly.
Ok, this is now fixed and I've update this pull request
I see, well after doing some Google'ing I think the portable answer would be to not use the shell option in Popen, but I'm not sure we can get away with that.  I will try some things out on Windows when I get a chance, but this isn't very high priority.  So, maybe we merge as is and leave a todo and a new issue to track the problem.
unused value
Maybe you could use `pathinfo()` and get all the info at the same time, filename, dirname, etc

http://php.net/manual/en/function.pathinfo.php
Put this case in the invalidPortProvider
use a temporal variable instead of do the same operation 4 times
Is weird see a `max` method when one of the arguments is called `max` too
please undo this change
Why is it that I spot the really crippling bugs like these after I lose motivation to continue on it
Ah, that does seem better.
missing space after echo, intentional ? x2
please done create a crappy api returning an object or false. Only PHP core legacy functions should be allowed to do it (and only for legacy). the value meaning "nothing" in PHP is ``null``, not ``false`` (and it was the return value previously as there was no explicit return value)
If you pass the same value for the type and the name, simply use ``create`` and pass it only once :)
Please fix the license header
Hm, but do we really need this? Bit 0 will be true, but $tag->{security}->{groups} won't contain a group 0, so we wouldn't be double-counting anyway.
We can talk more about this tomorrow.
`field.value.authors` instead?
Single # (pep8)
I'll have to run the demos in the sanity check command, there's no other (clean) way.

Running it in test(), so before make_install(), means that I need to get the `PYTHONPATH` correct myself, which is tricky.

Just running the demos by constructing a large sanity check command seems like the better solution.
Why should log_all be False?
One `(` too much ;-)
Sounds good.

On 1 Apr 2012, at 15:33, sgilda wrote:

>> -* Supports - This will run the method within a transaction if a transaction exists, 
>> -alternatively, if there is no transaction running the method will not be executed 
>> -within the scope of a transaction 
>> -* Never - If the client has a transaction running and does not suspend it but calls 
>> -a method annotated with Never then an EJB exception will be raised.
>> -
>> -## System requirements
>> +* Required - As demonstrated in the example. If a transaction does not already exist, this will initiate a transaction and	complete it for you, otherwise the business logic will be integrated into the existing transaction.
>> +* RequiresNew - If there is already a transaction running, it will be suspended, the work performed within a new transaction which is completed at exit of the method and then the original transaction resumed. 
>> +* Mandatory - If there is no transaction running, calling a business method with is annotated	with this will result in an error.
>> +* NotSupported - If there is a transaction running, it will be suspended and no transaction will be initiated for this business method.
>> +* Supports - This will run the method within a transaction if a transaction exists, alternatively, if there is no transaction running the method will not be executed within the scope of a transaction. 
>> +* Never - If the client has a transaction running and does not suspend it but calls a method annotated with Never then an EJB exception will be raised.
>> +
>> +
>> +JBoss Enterprise Application Platform 6 and JBoss AS 7 ship with an H2 datasource by default. This example shows how to transactionally insert key value pairs into the H2 database and demonstrates the requirements on the developer with respect to the JPA Entity Manager.
> 
> Sounds good. I believe we need a dash for 'in-memory' though. If you google it, almost every site uses the dash, including Wikipedia, IBM, and Oracle.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jbossas/quickstart/pull/167/files#r630747
This should probably be nex.getLocalizedMessage()