labofsahil/github-event-dataset-2012 huggingface dataset:
Make mock cordova.exec async

For compatibility with native cordova.exec, the mock implementation needs to be async.

Here's an example where the mock smartstore implementation not being async causes a problem:
http://jsfiddle.net/cwarden/kSQgV/
Переработан вывод в reactions.txt. Теперь можно мониторить количества со...

...вершённых реакций
Documentation Files Created

These are very unfinished documentation files. The eventual goal is to move them to the Github wiki once they are in decent condition.
fixed formatValue conflict

fixes http://d.puremagic.com/issues/show_bug.cgi?id=7628
Allows for clicking on nested DOM elements within each .text-suggestion (custom render + autoComplete plugin)

When using a custom `render()` method containing more than 1 level of
nesting, the `onClick` event is not triggered when not clicking
*directly* on the root `.text-suggestion` element.

 - http://goo.gl/MpY8f

Clicking anywhere in the green box will not trigger the `onClick`.
Clicking in the red box *(a small sliver at the bottom of the
`.text-suggestion`)* **does** trigger the `onClick`.

This changeset causes the `onMouseOver` and `onClick` events to look at
the `target`'s ancestor's to see if one of them is a `.text-suggestion`,
swapping the `target` to that DOM element if found.
SPReport fix
Sprint8 insight

RFE and bugs implemented during sprint8 ready for review
No reason to use soon to be deprecated jquery apis.

No reason to use soon to be deprecated jquery apis.
update icon path in variables

When using the icons, no icon is served and this error appears:

    ActionController::RoutingError (No route matches [GET] "/assets/twitter/twitter/bootstrap/glyphicons-halflings.png"):

Attached is a fix to the paths in /vendor/assets/stylesheets/twitter/bootstrap/_variables.scss
Bugfix

Fixed bug causing the form to submit to the default controller's default action.
added php geoip module support

### Purpose
enable the [php geoip module](http://php.net/manual/en/intro.geoip.php) using [the more complete implementation of the PECL geoip extension](https://github.com/Zakay/geoip)
Build Plone 4.1.5 and create toplone script 

I made these changes to the spanish documentation so I could import it to Plone.es website.

Basically it build a Plone 4.1.5 instance and the create toplone script, which is using funnelweb to send the docs to Plone. I had few problem to get the correct versions of packages so I pinned them in the version.cfg file, also I had to grab transmogrify.ploneremote from the github repo.

I've update also the documentation explaining how to upload it to a website, have a look to  [procesos_de_documentacion.rst](https://github.com/collective/collective.spanishdocumentation/blob/plone415/source/procesos_de_documentacion.rst) file "Subir la documentación a un sitio web Plone" section.

Thanks,
 Danilo
Fix segmentation fault in C code 

The condition `i < sizeof(food)` is wrong since it would return the size of the entire array – which seems to be filled with strings (i.e. char pointers). So if the array contains 5 strings `sizeof(food)` would probably return 5*8=40 on a 64-bit machine.

You could check against `sizeof(food)/sizeof(char*)`, but I guess that would just detract from the point you try to get across. :-)
Added listing for DojoModule

DojoModule aims to be a beginning for Dojo integration into zf2.
LPS-22546
german translation

Added German translation
Small tweaks to import barclamps [1/4]

Improve screen layout.  
Share code for parsing crowbar.yaml from inside the tar


 .../app/controllers/support_controller.rb          |   24 ++++++++++++-------
 .../app/views/support/import.html.haml             |    7 +++--
 .../app/views/support/restart.html.haml            |    5 +++-
 3 files changed, 23 insertions(+), 13 deletions(-)
LPS-24374 - fixing potential null pointer in event of missing category
LPS-25573
LPS-25528

Wrong permission check
Implemented visitPropertyUsingCustomHandler().

I've implemented missing visitPropertyUsingCustomHandler() method. It works, but in my case it was only used because visitUsingCustomHandler() was not called because of other error.

Now I'm not sure if there is any use case when visitPropertyUsingCustomHandler() will be called. Is there any special case when custom handler won't be called in visitUsingCustomHandler()? If not, then visitPropertyUsingCustomHandler() is redundant.
Crash on iOS4

Don't know if you planned to support iOS less than iOS5, but getHue:saturation:brightness:alpha: is only available on iOS5 or later.
Fixed apostrophes issues.
AudioLoopBackSampleToFile


This sample demonstrates how to record and playback audio simultaneously (Full Duplex) 
using the Audio Library APIs as part of the BlackBerry 10 Native SDK. This sample 
generates an output.wav file with capturing voice overlap of record and playback.

This sample is designed to easily plug into a VoIP application for encoding/decoding
the audio stream  

Feature Summary

Full duplex audio support
Interface to the microphone, receiver, loudspeaker and headset.
Hardware Audio Routing to select audio user interface (headset, speakerphone…)
Io-audio enables voice processing (AEC, NR, Gain Control) and routes audio to voice path
Volume control
genirq: Unmask oneshot irqs when thread was not woken

genirq: Unmask oneshot irqs when thread was not woken

When the primary handler of an interrupt which is marked IRQ_ONESHOT
returns IRQ_HANDLED or IRQ_NONE, then the interrupt thread is not
woken and the unmask logic of the interrupt line is never
invoked. This keeps the interrupt masked forever.

This was not noticed as most IRQ_ONESHOT users wake the thread
unconditionally (usually because they cannot access the underlying
device from hard interrupt context). Though this behaviour was nowhere
documented and not necessarily intentional. Some drivers can avoid the
thread wakeup in certain cases and run into the situation where the
interrupt line s kept masked.

Handle it gracefully.
Fix range violation in debug druntime builds with empty arguments

Programs linked to a non-release druntime build crashed when an empty argument was passed on the command line due a range violation (due to trying to take the address of the first character of an empty string).
INT-2356 - Update Jackson to 1.9.2

For reference see: https://jira.springsource.org/browse/INT-2356
Adding geophoto sample
Feature/facebooklogin

wip, still needs to have fixed all the glitches of integration
Made Requirements.txt install properly on my machine

Changed git+https:// to git+git://
Acl api

Added functions for getting and setting permissions on a folder, or inheriting the permissions from the parent. The docblock indicates the format to use when setting permissions, it would be good to know whether it is easy to understand or can be improved.

Added an example into the example client.

Paul added functions for adding, updating and deleting a user, adding and removing a user from a group and listing groups and roles.
Test Fixes

portal/dbupgrade/sampledatalatest/documentlibrary
portal/dbupgrade/transfersampledatalatest/documentlibrary

This should somewhat fix the document library lar tests. However, the issue where the wrench icon stops working will still cause failures.
fix createSnapshots command description
文字

as diff
Added support for shelve/unshelve

-Added support for Logging in/out by setting P4PASSWD for users who have it unset.

-Added initial support for SHELVE/UNSHELVE Currently only for entire CLs
Javascript username checking!
Add Lenovo A1

Change-Id: I0b8aa0a625e1f13e3d1e6b61d36c0066a81e5a6a
Next event api
Update composer to follow symfony@dev-master requirements concerning 

This is just a proposal, but it will allow me to do testing on doctrine 2.3 more easily.
Added code so that FUP users can see their details as well.

....1271.97 m

Added the functionality so that users with a FUP contract can see their details as well.
Affichage des réservations par type

Le code permettant de choisir le type d'affichage des réservations est implémenté, mais pas intégralement testé :)
Je n'ai pas pullé le main correspondant.
Added send_newsletter_continuous command and its implementation

permitting a better handling of mass mailing. See https://github.com/Fantomas42/emencia-django-newsletter/issues/36
Added Hebrew letters for T9 search

Hi Danesh,

Thanks a lot for the great work here,
I've been testing this change for the past weeks locally, its working great.
Regarding the key's images(drawable), can/should we change it according to user global language setting or by  future dialer's T9 specific language setting?

Best Regards
INTSAMPLES-55 - Update README.md for OddEven Sample

For reference: https://jira.springsource.org/browse/INTSAMPLES-55

* Update README.md to reflect correct classes
* Improve documentation
* Add Maven Exec Maven Plugin for easy execution from the command line
Share dist patches

Make the number of nodes configurable and introduce option to use DIST
Jwk code cleanup

Cleaned up TODO and FIXME statements in the JWK code
Fixed regexps

This way they work on 1.9 (actually tested on 1.9.3)
Test vom Tobi
Dooouuugg!!
Early march
Throw exception if class not exist instead of Fatal Error.

Allows to show user friendly stack trace
What do you suggest? A reset() method on the recurrence?
没看懂啥意思
I agree -- will create a new pull requests with:

Object frame = null;
try {
    frame = decode(ctx, channel, buffer);
} finally  {
    headerBlockDecompressor.end();
}
return frame;
@andyvb didn't see your comment while I was typing mine (we must have been commenting at the same time.. lol).. but the situation you describe is already handled by ActiveModel in Rails 3.1+.. that's what scoped attr_accessible whitelists are for.
Good point.  Will do.
Without that, migrations won't run; it's sort of a bootstrapping issue. Clearly I'll have to think of a better way to phrase to comment.
Oh snap. Yeah, that's definitely a typo. I blame the nightly hours I used to hack this together. Will fix ASAP!
s/accidently/accidentally/
Should you wish to reference a worker over a communication channel, using the worker's `uniqueID` is the easiest way to find the worker.
@tristen @samanpwbb Before committing in future it you should something like this on the file:

    sed -i 's/\ *inkscape:export-filename=".*"//g' maki-icons.svg
I think that one would read even better if you were to do the it like this:

    return '' if token == false || !protect_against_forgery?
    # ... rest ....
This fails if we already have uglify-js as a dependency in package.json :

```json
{
  "name": "test",
  "description": "will fail",
  "version": "0.0.1",
  "dependencies": {
    "uglify-js" : "1.2.x",
    "async": "0.1.x"
  },
  "engines": { "node": "0.6.8" }
}
```

npm install will fail.

.16 works ok
Forget to mention ckeditor and kcfinder -_-'
Yep, great idea. Looking forward to it.
Are these lines dead code? $linkParams is initialized again just below.
Why do you check for dracut on the host system? In build_initramfs() it's called for on the chroot system through spawn_chroot. Before the call it's emerged on the chroot system.
Исправлено несколько утечек памяти в истории сообщений (поиск, навигация, экспорт).
Кажется есть еще одна, но пока не уверен.
I do not see the point of this commit. As I wrote before the Pimpl idiom needs to go. Therefore you should merge your new sky manager class into SkyManager, which should be located in sky.hpp (btw. the new files don't follow our file naming convention, see the wiki for details).
In this one frame it causes glitches with the Lua rendered shield.
Esp. one rendering frame can be very long with 15-20FPS.
Thanks! It is moved.
Oh yeah totally. Just did it that way because referencing it without the isset() wrapper will usually pitch a reference error in PHP. You avoid the error by doing the isset() check — fuckin weird, though the test might account for that...

And yeah I'm trying to push the coverage way up on the core. There are things I'm not sure how to test running as CLI, but I'm not super worried as long as we catch core behavior and whatnot...
Applied! Thanks a lot for pointing this case out!
No, not that I'm aware of but I wanted to remain semi-compatible with the Apache module if that makes any sense :)
Yeah, let's leave it as is, we can always grep for 2.5 ;) Someday, maybe 10-20 years from now, everyone will be using 3.x and we can remove a lot of cruft, but I'm not holding my breath till then...
I've put "Liquid::classExists" for safety
You should just delete these comments as they will be confusing in the future if left in.
Ya recopile el CSS. Avisame si quieres que lo ponga mejor en mi servidor.



On Sun, Feb 26, 2012 at 7:04 PM, Mark Lybrand <mlybrand@gmail.com> wrote:

> He subido lo que me parece puede ser un layout de las secciones.  Manhana
> por la manhana comienzo a poner de nuevo las cosas interactivas.  Pero
> necesito saber si esto puede ser mas o menos lo que pensabas.
>
> Para verlo, requiere un servidor web... espera.... recopilare los archivos
> .less a archivos .css y cambio las referencias para que no sea necesario.
> toke.
>
> Mark
>
>
> On Sun, Feb 26, 2012 at 5:21 PM, Mark Lybrand <mlybrand@gmail.com> wrote:
>
>> hahahahahaha
>>
>> stop looking...
>>
>>
>> On Sun, Feb 26, 2012 at 5:05 PM, vchigne <
>> reply@reply.github.com
>> > wrote:
>>
>>> Very minimal I see
>>>
>>> ---
>>> Reply to this email directly or view it on GitHub:
>>>
>>> https://github.com/mayorio/mayorio/commit/0a9707e18197f17c7876283a99a673a4992d618a#commitcomment-1013334
>>>
>>
>>
>>
>> --
>> Mark :)
>>
>
>
>
> --
> Mark :)
>



-- 
Mark :)
I am too, in languages where variables have block scope. But in JS with function scope it's a chance for ambiguity and doesn't accurately reflect what the language actually does at runtime, etc. I'll stick with Crockford on this one :)
comme c'est pas bien de mentir !
Better to provide meaningful log messages with meaningful log levels rather than deleting them like ordinary printf's :-)
Hab da mal ein TODO comment reingemacht, damit wir das dann finden ;)
Habe es eingebaut. Vererbte Properties werden zusätzlich in den Funktionen mit beachtet.

7fed850ec53311d10e658f8a4241e579458cc751

Tests laufen durch. Eigene manuelle Tests haben funktioniert. Hoffe läuft ;-)
I never heard of that best practice, what are the advantages of doing this?
Yeah, it is much better.
Validate using

```java
import org.jclouds.JcloudsVersion;

public class Tester {
  public static void main(String[] args) {
    JcloudsVersion version = JcloudsVersion.get();
    System.out.println(version.majorVersion);
    System.out.println(version.minorVersion);
    System.out.println(version.patchVersion);
    System.out.println(version.snapshot);
  }
}
```

Run from the command line with jclouds-core and Guava on the classpath.
Can't wait. Srsly.
@Reamer WHY?
http://filebeam.com/3c1520257f1c96554cb29b4ea38c0898.jpg
http://filebeam.com/3cb3aacb9549b517fb614c0a829014ea.jpg
Wow how lazy ..maybe something unquie would be better
For the_ring_of_valor need too (for eyes)
if max = MapUpdate.Threads. perfect ^^ At least threads count increase by load works fine. Maybe add map.updatethreads to .server info command.
Sorry, I was referring to the line immediately above my comment.  I was wondering if we want to keep **"This text will be displayed if your browser does not support the AUDIO tag."** as the value for `children`, or if we should write something else. @JonFerraiolo
I suspect that the ```dojo.query("div" ``` will only find subnodes whose tagname is DIV. Should it be dojo.query "*" instead?
This is an ultra old pull request, it will be much more easier to recode this mess. Will close this request.
And I'm sure everyone on the internet is very proud of you, iffa, as well as thrilled to know that now that you vanquished 12 whole characters you have nothing better left to do with your time than brag about it and try to pass that off as an attempt at humor.
Yeah, device restart, so everything recached.

El 07/03/2012, a las 20:23, Ben Combee escribió:

> As long as you restart LunaSysMgr after the change to recache it and
> it works.  The "a" comes from the minimization process, but isn't in
> the original source file.
> 
> On Wed, Mar 7, 2012 at 1:18 PM, Rafa Bernad
> <reply@reply.github.com>
> wrote:
>> Tested on webOS. It works, understanding that webos uses the enyo-build.js file under root@localhost:/usr/palm/frameworks/enyo/1.0/framework/build. I've done testing by replacing that file, so if it's ok, you can pull it safely.
>> 
>> P.S. (Just FYI, original "startdrag", "drag", and "dragdrop" methods make use of that same naming convention, with "a" instead of "e" as a parameter).
>> 
>> El 07/03/2012, a las 18:14, Ben Combee escribió:
>> 
>>> have you tested this on webOS?  I don't want to pull anything that affects the scrolling experience there.
>>> 
>>> ---
>>> Reply to this email directly or view it on GitHub:
>>> https://github.com/Newness/enyo-1.0/commit/f352cdd7ad25e10fb9c2f2d978441e8f7259fb3c#commitcomment-1054472
>> 
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/Newness/enyo-1.0/commit/f352cdd7ad25e10fb9c2f2d978441e8f7259fb3c#commitcomment-1055102
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/Newness/enyo-1.0/commit/f352cdd7ad25e10fb9c2f2d978441e8f7259fb3c#commitcomment-1055127
Don't get hacked like github :)
on string values this will create a hard reference to them permanently since guidFor caches strings (because there is no way to add a property to strings).
TODO: Check whether require_once recognized files loaded via "auto_prepend_file"
I made number 2 be handled in PlayerJoinEvent so plugins can change it.
please add missing ones to config
This builds fine and all tests pass on my machine.
We need a unit test around this, I'm unsure. I think I made the change because the packets on the wire where not looking right.
Done. Bordered text was not nice => i used hd.png as background for it.
I can make it so it defaults to a hard link unless there is a configurable URL.  The Vendors needs to send the users to their site for customized users guides etc and it a real pain in the tush tu manual update 20-30 users code sets every update.
当然->当前
What about:

```javascript
// 1
new THREE.Matrix4().getInverse( parent.matrixWorld ).multiplyObject( child );
parent.remove( child )
scene.add( child );

// 2 
parent.matrixWorld.multiplyObject( child );
scene.remove( child );
parent.add( child );
```

The method would multiply the matrix of the object and extract the position, rotation and scale.

Less nice but more clear and powerful?
Нужно пустой строкой разделить, иначе в один абзац сольется
`$type = 'SQLite3'`would have been more obvious to read.
they actually fix an error and introduce a new one which I'm fixing.. lol.
`reflectClass('Function', new Function);` if `Fp` isn't used anywhere else?
Won't ```execute[archive-default-nagios-object-definitions]``` (from the nagios::server recipe) blow this away every time?
This resource will end up changing on every run.
The new connection clase we'll use.
I'm not crazy about this, but wfm
`PPC`? `Intel` we don't care about? man, this is dated.
you'll want to be sure to include the InnoDB `ENGINE` statement (which helps enforce the foreign-key relationships). [this](https://github.com/mozilla/zamboni/commit/f6954ae4#L22R21)

Django's `sqlall` management command doesn't do this for you by default. Try `mysqldump`:

    mysqldump zamboni <table> -u root -p
Uh, yeah. That's what I meant to do. Sigh.
nope, it takes 32 bytes of entropy to create a key. The 64-byte expanded form is internal. 16 bytes is too short.
Okey, I'll watch it :-)
Hi Elton, check out this fork https://github.com/jdp-global/egodatabase
nvm, i modified it for you already.:)
This is just confusion on my part.
Which includes besides #include <errno.h> ?
Found a serious problem, I think. You can't simply add a space at the front without adjusting line wrapping. Put on show formatting and input the text below:

                      JASON
                (angrily, staring at xxxx
                 random passersby aldsfkj)

You'll see how xxxx is at the very end of the available space, but then the next line goes one beyond it since that line is max length as well but the code is adding the one-more-indent at the front. So we're overflowing the space allocated to us, which is not good.

To fix that, wrapLine also needs changing.

wrapLine is critical, as are getScreen and generatePMLPage, so I'd also like some stats for how much these changes slow down the program.
The eye all the same falls...
ok, I see it now. why do we need to delegate digests from the outer app into the inner app? wouldn't the example (inner app) digest on its own when relevant events happen inside of its context?
Can you make pull request to my fork? I want that we have one spree-heroku instead a lot of forks.
you could show other basic stats in the message as well (to verify that basic expectations are met):

stats.isStatisticsEnabled() returns a boolean
stats.getEntityNames() returns a String[] 
stats.getSecondLevelCacheRegionNames() returns a String[]

Perhaps you could have a subroutine that returns the string value of the above for convenience (could also append stats.toString() which includes other values).
this may not actually be needed as these can mostly be moved to sai i beleive.
Oh, sorry. This is not ruby I guess. :)
That change happened in a6d1c98c89f3be550b
Possibly make into its own class

class="prohibitSelection"

.prohibitSelection {
  user-select: none; -webkit-user-select: none; -moz-user-select: none;
}
It just tells us the type is embedded. I should be able to get rid of it by just maintaining a Map of existing entities. I will make that change when adding support for embedded lists, which will use list brackets: `[Type]`. What about sets? The Haskell way is `Set Type`. I am tempted to use `{Type}`, which parses nicely like this.
bad midget. this broke coming ep page and who knows what else
As stated previously, this is already useful.
Style nit: Please remove the space in "find {"
s/.kava/.java/ ?
This change is only engaged when title is passed as a string. When it is passed as a tiddler, it is not. Is this a problem?
Reason for failing is:

 public class BootstrapDocument : HtmlDocument

inside Bottles.Deployment... and when doing this

factory.RegisterCommands(typeof(Recipe).Assembly);

public void RegisterCommands(Assembly assembly)
        {
            assembly
                .GetExportedTypes()
                .Where(x => x.Closes(typeof(FubuCommand<>)) && x.IsConcrete())
                .Each(t => { _commandTypes[CommandNameFor(t)] = t; });
        }

Boom!
Невниятное название было.
不理解什么叫鼠标穿越

2012/2/3 Tusooa Zhu <
reply@reply.github.com
>

> sawfish能不能像fvwm那样，鼠标穿越。
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Liutos/Configurations/commit/4242fffc2928973832ff156f81e673dc6fdaa19f#commitcomment-929933
>



-- 
Liutos Love Linux LaTeX Lisp Ling

我的博客，纪念我死去的GAE <http://blog.sina.com.cn/u/2031941547>
I don't think we need to modify, but looking at
http://packagist.org/packages/Aura/Autoload I see that the links are not
coming . I guess we don't need the '<>' symbols .

*Hari K T
*M: +91-9388758821 | W: http://harikt.com/blog



On Thu, Jan 5, 2012 at 8:06 AM, pmjones <
reply@reply.github.com
> wrote:

> Understood; we'll modify as needed and make a second release if we have to.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/auraphp/Aura.Autoload/commit/e4edea113d79208e39e003299f382dc47ec939ab#commitcomment-839059
>
Сейчас еще раз попробую без архива.
Indeed.  Thanks (addressed in 66e8fafdf295b5355be5e55590216a8122b7a213).
Sorry, I misread your first comment. I can change that, but first I'll need to know if this is applied to all yells, or just to specific yells.
That's because all applications are independent. You can change APPLICATION_NAME constant to 'blog' and blog application is fired up. I thought about something like ApplicationManager class to manage the whole apps together. Thanks for the idea.
At least some interface level javadocs would be nice explaining the idea of this interface
derive from pqPlayBackEventsDialog instead
256 ** digits. Follow the code style in the comments too! :P
More generic like what? :) (No really, I can't think of a more generic way right now, if you have one, I'll gladly listen.)

The difference between two `Time` objects is always in seconds. I chose minutes for the delays in `AlertingStep` and `EscalationStep` because that's the dimension of time you'll most likely think in when entering data. Rails' date & time helpers make it really easy to work with either (see below, line 13, the result of the calculation is always a `Time` again), so I don't think it's a problem—that is, if I stick to naming things `xyz_minutes`, `xyz_seconds` etc.
Thanks for the heads up, in fact, even that link doesn't work either - still takes me to the 'multicountry login'. I think http://www.wacom.eu/index2.asp?pid=294&lang=en&gm=2 works correctly, and have updated to that.
I think OPTION_RESOLVE_ALL should also be OK.
Incidentally, I have since come to the conclusion that this particular bit of code was a mistake and should be removed. Moving the status options onto the classes was good; adding shortcut methods was sloppy. It's unclear and violates the "one obvious way" principle.
You'll need to ask @richardkchapman
Working quite good but starting position is still wrong there and not lifts up. But anyway good work
Get on IRC more often please..
esto me parece bien, pero no seria mejor actualizar los voters ah q sea un objeto {userid:string, lastVoted:date}, es una idea pero lo podemos dejar así
ser ajudado -> "obter ajuda" looks better
requiring the ORM ? This is wrong. The bundle could be used for DBAL only too
Actually it IS volatile. And I think I meant before.
This means that the 'About' link on the nav bar points to the current page. I think this is confusing, so if anything it shouldn't be a link at all. Why can't we have an 'About MetaBrainz' page?
try putting
```
if __FILE__ == $0

 window = GameWindow.new
 window.show
end

```

this will only run this block when the file you are running is being executed directly, which should allow you to require this file for use in IRB
@dbu actually that's *the* problem. as long as we were asking for plain slashes in our url we were fine, now - using the FOSJsRoutingBundle - we asked for urlencoded node IDs and it doesn't work.
Diving into the router today.
re 4: "Lazy initialization is another name for race conditions" ... that really depends on how you design an object, if you share it etc. If you set an ivar `@connection` in a method `connect` that is called somewhere and then use the ivar next then that's not a better situation. Besides, I'd just point out that we have this code (which uses lazy initialization super heavily) running for months without any of these issues.

re 1: Single responsibility. The worker should deal with managing things that have a similar level of abstraction. Implementation details about closing connections, channels, queues for a particular messaging technology are not on the same level of abstraction as methods like :prepare, :work, :finish (i.e. manage other objects that belong to our domain layer). Same thing put differently: when I look at the Worker class I want to be able to easily read through what the Worker does in order to prepare, run and finish a Job. I don't want to look at details of how a certain messaging protocol needs to be managed.

re 3: I don't particularly care a lot about what famous people say if it's a bikeshed color. Our entire codebase follows this style, so let's stick to it. The reason why I favor it (which doesn't mean others need to) is that it makes it very easy to spot the break between public api and protected/private stuff. A single extra line with  the same indentation doesn't serve that purpose.
Hmmmm, my previous comment doesn't seem visible... Basically wanted to apologise Mike, my 96 column width is simply a personal thing, I think 100 makes sense to stick with and I'll be getting over my love of things that divide nicely by 2, 4, 8, & 16 accordingly...
Men, pls make updates. It will simply, instead of importing full's
Could this be further simplified to
  self.tags = data.get('filterTags', [])
Add a [:] if we need a copy instead of the original.
good，还是你研究的透彻
Fixed in https://github.com/saltybeagle/wdntemplates/commit/4899e5f4e37f4f50f43f079c691146c9be321b87
> Would be nice to include a non regression test :)

Done in 1dbf4c3

Spent a good 45mn on this, to have a robust test without adding 1s to the
test suite.
Strange return. What does it do inside an expression?
thanks to: SergeWinters
This looks awesome in action, nice work!
Maybe with a Pluma plugin. One, like the remove tailing spaces/tabs on save from Geany.
Typo 'environemnt'
Bonjour, au niveau de logs qu'elle est la réponse de siriproxy quand vous exécutez votre demande ?

Envoyé de mon iPhone

Le 13 févr. 2012 à 16:19, Doriginal <reply@reply.github.com> a écrit :

> Bonjour,
> J'ai installer votre version française de imdb plugin mais il ne reconnais rien lorsque je lui demande quelque chose, quel que sois la question, Dois-je voir, quand est sorti, qui a joué.....
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/Treenity/SiriProxy-SiriIMDB/commit/ac74cb3d0e3090c395fcf287e5ffdac7a8128fd9#commitcomment-960594
actually the easiest and cleanest way:
- (make sure you have a copy of your changes)
- delete your fork
- fork again
- apply the commits cleanly
- send another PR

does that help?
I think the [network graph](https://github.com/ps3mediaserver/ps3mediaserver/network) explains what happened.
If I read it correctly:

- chocolateboy's original commit was done on March 4th
- there were other commits to the master on March 5th
- chocolateboy merged these commits on March 6th and committed it.

Git cares about the order of the commits, which is why the the weird effects happen.

Rebasing would indeed roll back your own commits, fastforward to the latest revision and then replay your commits on top of it, avoiding the whole thing. I believe TortoiseGit comes with the "fetch + rebase" option to do this automatically.
Oh wow. This is sexy. +1
more media types might be a good idea. concordia url would be wonderful! Might not even need the other domain then. Does CDA allow us to rock MySQL, though? They might not =\
Few small things left:

- please upcase Url to URL in the info output
- Add a new line after the first line in the package description

Otherwise this is wicked, get some tests going and we can do a thorough review next week
get_ad_codes() should format the response based on some standard format....
None would maybe be better?
Heya

smal startup error, which ID had the npc ?

2012-03-06 17:09:24 ERROR: Script named 'npc_gauntlet_generator' does not have a script name assigned in database.

for the script ?
that's bullshit and you know it
It will only fail if commonache wasn't installed and that is almost always installed :)
be careful here; for the registration/template we should focus on DataTransferObjects (TDO); for the backward call we should rather use the default domain; the EKB will transform data there anyhow.
I suggest to move the comment above the line. Besides, we should not allow comments in the line, since we may need to have ';' in the line in the future, and many ini parsers do not regard `;` in the same line as comments. So this does not violate ini syntax. We could notice it now, and this should be fixed in the EditorConfig Core in the future.
No, doing this at build time makes no sense, because this is _running_ the app - possibly on a different machine.

I expect that long-term this file will be go away, and we'd end up with the single-stub (without the -bin), as Firefox currently does.  We can do the GST munging in C++ in that case.
Full stop at the end of sentences.

Clear linespace between short and long description in DocBlocks.
"false if the property is not to be shown in ..."
Why should you set this property?
Actually my first idea here was to create VERY simple class (later I've added other methods) that's compatible with current usage in core (db->split and db['write'] at the same time), nothing more. I'd gladly drop Array Access stuff and don't like it at all. Anyway we could think of dropping support for old multi DB standard since it wasn't working in out of the box version anyway. That would simplify whole thing.
It's just an idea, but can't the ```principalResolver``` be a property of the ```authenticationHandler``` as they are both distinguished by their ```supports``` method which should work the same when they are associated : the test line 213 is somehow redundant after the test line 169...
No, that's already in there. The petsc4py routines unify the 4 possible cases (Seq/MPI + non-blocked/blocked) in a single method by parameter introspection. As an aside, PETSc does not provide `MatCreateMPIBAIJWithSplitArrays`, no idea why.
Or of not super performance is needed:

value filter defined
is there a specific reason why you’re calling this twice?
what happens if activationTracker or registry dies or throws exception?
Why is this public and lazy?
Historical nomenclature has been "inbound" & "outbound"
You mean like a RemoteRouterConfig wrapping another RemoteRouterConfig?
That would be very wrong. I can add assert.
Good call :-)
Then clarify it to say "In the Scala Standard Library, a Future..."
because it was a duplicate (it’s already in fault-tolerance-sample.rst)
below.  you got rid of those, right?
More leftovers.  I'm going to remove some of this cruft and make sure new failures don't show up, but I'm pretty sure the whole sections is extra.
* `media` - default value
* DIRECTORY_SEPARATOR
On Mon, Apr 02, 2012 at 05:56:47PM -0700, Andreia Gaita wrote:
> > @@ -2937,13 +2949,8 @@ internal override IntPtr CreateWindow (CreateParams cp)
> >  					XSelectInput(DisplayHandle, hwnd.client_window, new IntPtr ((int)(SelectInputMask | EventMask.StructureNotifyMask | Keyboard.KeyEventMask)));
> >  			}
> >  
> > -			if (ExStyleSet (cp.ExStyle, WindowExStyles.WS_EX_TOPMOST)) {
> > -				atoms = new int[2];
> > -				atoms[0] = _NET_WM_WINDOW_TYPE_NORMAL.ToInt32();
> > -				XChangeProperty(DisplayHandle, hwnd.whole_window, _NET_WM_WINDOW_TYPE, (IntPtr)Atom.XA_ATOM, 32, PropertyMode.Replace, atoms, 1);
> > -
> > -				XSetTransientForHint (DisplayHandle, hwnd.whole_window, RootWindow);
> 
> Marking the window as transient usually signals the window manager that this window is a popup on behalf of another window, implying that the window manager should treat it differently (no decorations, no user interaction required to show it, etc). Dialog boxes and other windows of that type fit this scenario. If your window manager is not respecting that flag and/or not doing what is necessary for it, I guess that can be corrected in code, but the transient flag still needs to be there for window managers that respect that flag.

I understand the purpose of transient-for, but I don't see why a
window should be made transient just because WS_EX_TOPMOST is set --
a window can be both topmost and a top-level window in its own right.
Setting WS_POPUP sets transient-for as one might expect [1].

Wine is an additional data point here.  The only special handling
it does for WS_EX_TOPMOST windows, as far as I can tell, is to
set _NET_WM_STATE to _NET_WM_STATE_ABOVE [2], and it only sets
transient-for if the window has an owner [3].

Thanks,
-- 
Peter

[1] https://github.com/pcc/mono/blob/c4abf08318381f4abd968ff944347b6df7f1ecbb/mcs/class/Managed.Windows.Forms/System.Windows.Forms/XplatUIX11.cs#L1142
[2] http://source.winehq.org/source/dlls/winex11.drv/window.c#L1290
[3] http://source.winehq.org/source/dlls/winex11.drv/window.c#L1190
`self.qgrid_size // 2`
Minor pedant request: Can we make this a `.on('change')` instead? Or is this a change from another change set? Or is this view running an older jQuery?
$ diff ckan/config/solr/schema-1.4.xml ckan/config/solr/schema-1.5.xml 
    19c19
    < <schema name="ckan" version="1.4">
    ---
    > <schema name="ckan" version="1.5">
    118c118,119
    <     <field name="organizations" type="string" indexed="true" stored="true" multiValued="true"/>
    ---
    > 
    >     <field name="organizations" type="string" indexed="true" stored="true" multiValued="false"/>
    176c177
    < <copyField source="groups" dest="text"/>
    ---
    > <copyField source="organizations" dest="text"/>
Can we use logger.exception() here instead? That way we'll get the traceback as well (unless there's a reason we don't want that that I'm not thinking of).
So, I cleaned up the extra ruby initializers items a lot more too
I know you didn't change this, but why are we listing all the link_types here?
do we need this? don't we get proper returns from curl?
Nee, is dat zo belangrijk voor deze app? :P
This is a public method, so we can't just delete it.
I believe syslog's default behaviour is to print the first message and then suppress all duplicates until a different event comes in, at which point it'll print "last message duplicated N times", is this what setting this to 'true' does?
Ah, Thanks
make it a def
I was asked to remove them. I will add them back.
I opened a ticket for this.
That would be a huge array.
The only information that I was interested in, for amusement only, was which file
were people using as I walked in thefiletree, so that I could join and collaborate.
This really doesn't need to be a category in the first place. I think it was kind of ridiculous that it was broken out into its own implementation file, specifically for this reason.
This check is no good. Serialize the requests so you know when to expect the parse error or start two servers, one on common.PORT and the other on common.PORT + 1, so the requests can be made in parallel.
This is the only important line in the patch. The messing with the `colors` stuff is unnecessary IMO.
Hmm. Does this work without binding to dependencies-resolved here?
Yes, I updated the comment.
Check sentence.
thanks. I will update it soon.:)
oh, d'oh, you are right, i hadn't understand the question before.. i think i need more sleep :P it should be actual_value or self.userInput, my bad
+1 for
 
`(defined_app_const_base? ? defined_app_name : File.basename(destination_root)).tr('.', '_')`

its more readable.
Can be shortened to `raise ArgumentError, '...'`
@dmathieu :+1: thanks for your :scissors:
Can we just `yield` here?
Shouldn't that be `\s+`, meaning one or more spaces?
Ah right, I was looking for the changed message in 2.10.x.
it might actually have length 1, if it's a single guarded case
Isn't this unncecessary magic?
fixed in a16c58a
pep8: levels=False
On 11.06.2012 15:28, Christopher Smith wrote:
>> +.. autoclass:: ANP
>> +   :members:
>> +
>> +Level Zero
>> +==========
>> +
>> +Level zero contains the bulk code of the polynomial manipulation module.
>> +
>> +Manipulation of dense, multivariate polynomials
>> +***********************************************
>> +
>> +These functions can be used to manipulate polynomials in `K[X_0, \dots, X_u]`.
>> +Functions for manipulating multivariate polynomials in the dense representation
>> +have the prefix ``dmp_``. Functions which only apply to univariate polynomials
>> +(i.e. `u = 0`)
>> +have the prefix ``dup__``. The ground domain `K` has to be passed explicitely.
>
> explicitely->explicitly
>

done
Done.  I was having a similar inkling.
...but it had the following error:

```
Removed trailing whitespace from py3k-sympy\sympy\physics\quantum\represent.py
Traceback (most recent call last):
  File "bin\use2to3", line 132, in <module>
    text = f.read()
  File "C:\Python32\lib\encodings\cp1252.py", line 23, in decode
    return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x81 in position 5511: cha
racter maps to <undefined>
```
I think `shape[0]` and `shape[1]` should be instances of `Expr`, i.e. any `int` should have been sympified at instantiation. Then you don't need such extra check.
It will could cause ambiguous output if there is a multiple sums. This is why factor refuses to play with non-commutatives: it can factor them but the result may not be right. I could add a condition that the len of limits be > 1. Let me know your preference, please.
Part of me wants to hide/make private all of the parts that associate a scalar field to a particular coordinate system.
I guess this isn't related. Isn't it?
"Default value is empty string which means that no comment has been set for the column. Null value means that RDBMS does not support column comments at all (SQLite) or comment retrival for the active RDBMS is not yet supported by the framework."
I deliberated on this for a bit and landed on `callable` because it actually checks for the right "interface" (i.e., having a `__call__` method) whereas `isinstance` would only allow actual strings and not things-that-are-trying-to-look-like-strings.

Quack.
Put your own name? Just sayin. ;-)
After I moved those lines to lib/... this problem no longer produced, because require works only once anyway.
Done.
No english because it's already include in the jquery lib
but which have important difference?  are the next two sentences the difference?
near, in this context is a parameter to the geoNear command and is not an operator.
Run foo and bar on the...
Ahhh yes, my apologies, I misunderstood.
if we've moved this block to modal-control.js, I say we just remove this completely rather than just comment out.
breaks should be one tap to the left
We already have password-protected forums, and obviously there are permissions that can be used to create a "private" forum, as in "private to the staff members".
FYI this is the documentation for .attr(): http://api.jquery.com/attr/
Can be cut to one line like this:
`$readable_forum_ids = $readable_forum_ids ?: $this->get_readable_forums();`
So no "a" or "the"?
It seems like the logging should include the connection name.
nit: try to avoid code moves and functional adds in the same commit
cred că nu e ok whitespace, după cum l-a aliniat github. Vezi să nu fie taburi.
http://stackoverflow.com/questions/2110795/how-to-find-all-c-files-for-cmake-build-system
but and
I’d suggest to remove the `random` keyword from the function names, as this will be closer to the signatures in our other modules. Next, `random:int` should be `random:integer`, as in XQuery, there’s a difference between `xs:int` and `xs:integer`.. Rest looks fine!
SS ? sale nazi
Should we actually go for it? Can't we temp include username:pass like this?
Because very soon, once Login Screen is finalized, username & password will be taken from the Login Form.
I am just looping through a normal array viewStrings, not sure why I should use for..in?
nit: this is fine for the moment, but a recommendation in the future.

You comment says what the test does, but really what's written in your comment should be the name of the test (line 64). Why? Every time you run the test it displays the name. When people run the test and see that it passes/fails, they'll use the name to begin their debugging process (or simply to understand what failed in the product). Since the name of the test isn't actually what it does, that can cause confusion later.
Would using the markup be even better? e.g. something like ``* coords: list of either coordinate names or :class:`iris.coords.Coord` instances``
I'd say checking for transactional DDL support here makes complete sense - there's no point starting a transaction if they won't do anything, is there. I'll add it in and wrap the check round the transaction methods (I don't want to add too many overrideable sub-methods if I don't have to - it's an easy change to do later if we need it)
I'm still testing this, but in a more specific way. The inline diff makes it hard to see...
Below, I assert that `expected = '<2> [<1>bar</1>] </2>'` which shows double-filtering, but more specifically and concretely.
updated state names in previous commit, updated field name in last commit.
ok,good
BTW, < was incorrect, it should be <=, you can miss the very last packet. Indeed, if packet length is 178 bytes long, last data unit goes from 132 to 177.
Only when I or others will test it deeply.
Splitting these tests out made me think of another: we should test two _equal_ but not _identical_ objects. This can be done using two `NSMutableStrings`.
@jdevera When we set rtp, only files specified by rtp will be checked out, and `.gitmodule` is usually not included in rtp. But when a repo has submodules, the following command tries to update the submodules according to `.gitmodule` which is absent. Consequently, git will report an error that it cannot find that file.
Where is the default for `type` set? Looks like you deleted that line, but didn't do an assignment anywhere else
Yes, one would expect that there is a turning point where the cost of storing the distances is less than recomputing. Then again, the factor is at most 2, and we all know that updating many table rows in each iteration is a pain of its own. So I would assume that this implementation is a lot more performant for most typical inputs.
Should be documented with the proper Sphinx syntax, like the rest of the settings.
it sticks around; mktemp removes it.
Can't do this; zlib comes with the system and is not provided by Homebrew.
If rest_app.embedded can't be null this can be simplified.  If it can be null, it shouldn't be (should default to {}) if it can easily be changed in all referenced locations.
Rename as mentioned above.
Is this log item going to be useful? Some things I can think to include:

* item
* fetcher function
* traceback

Also, what about other exceptions?
LogCatcher recently got a `.messages` that is useful here (although it will not return errors).
c.user_is_admin is a global and may_view takes a user.  It is possible I could instead check specifically if the given user is an admin.  However, I will not be able to use the global for that.
This class could be abstracted with CDK's [GenerateMojo|https://github.com/richfaces/cdk/blob/develop/maven-cdk-plugin/src/main/java/org/richfaces/builder/mojo/GenerateMojo.java]
Oooh!  I like where you're going with this. But this change is agnostic about extension/child graphs.  Not sure I want to pre-sage that functionality.  It's part of the spec for child graphs.
Now that you have some experience with it, please tell me honestly: how do you feel about this API ?
The way it's setup now fits perfectly in the way we develop our corporate websites but you are right that we should provide flexibility. 

Maybe we could parse to the stdout but keep a flag for the theme? --theme to keep the current way working just as well.
@yihui @siiowl 

来灌个水

`-jn` 可以控制 `make` 同时执行的任务数量，可以想象，在多核环境下能够显著缩短编译时间。

至于 n 设成几，有人说等于核数，有人说是核数 + 1，还有人说核数 x 2 。。。
I'm missing how this test could have failed in the old implementation.  Can you help?
This check is done when inserting the node/removing the placeholder, but not when removing the node/inserting the placeholder. Is there a reason for that asymmetry?
I'm glad we're using the non-minified version now. :+1:
var options = options || { showLocationBar : true };
nice catch. fixed.
This means no items match when query is "", which is the opposite of the old behavior.  End result is that you no longer see an initial unfiltered list of all items before you start searching.  I think we'd like to keep that behavior.
Google Translate tells me that there are 2 r's in ``forrige``.
Is this the command ID? If so, it should be `commandID`.
My guess is this will resend all messages on every reconnect.
Oh, yes. You don't use it bellow. OK.
Please move to ready document event handler. Do not do this `#` links - if we store anything into URL anchor it will clear it!