labofsahil/github-event-dataset-2012 huggingface dataset:
Slight maintenance modifications

Some slight modifications for the 1.1.0 release
Fixed typo in 4th paragraph.

Changed has to had in 1st sentence of 4th paragraph.
NH-3063: Turkish-I problem in ValidateColumn

ValidateColumn() fails in Turkish environement because of Turkish-I problem.

JIRA: https://nhibernate.jira.com/browse/NH-3063
Fixed migration again and some packaging problems.
Add some more NO vars to speed up build
Final_version_rasterizer!
Cleanup

Hi,
I made a little project cleanup (not code-related at all). Two things were made:
 - Removed generated files (bin and gen folders) from git, added them to a .gitignore file,  so they'll never be commited at all (source of useless conflicts)
 - Removed unused resources like icons or layout. My initial issue was with the strings: I'm working on an application with both default (english) and french locale. Lint was forbidding me to export my application because the string "hello" from TPSVG wasn't translated... Pretty annoying :-)
Initial commit of xapi 95 client examples
Adding French accented characters in onlyLetter* regular expressions
Added GPS Options

timeout = 20secs
max age = 30secs
high accuracy
Show banners that are activated today
please pull this

...Full Application screen.
Address Robin Wood's bug report in the framework mailing list
Cvfix

Fixed opencv video and gstreamer data type
Use MySQL instead of MySQLi

If you want to pull this, it should works on every server with mysql support.

That means every server running xampp for linux (not only xampp, apache, etc.) can now run it an it has the same results as MySQLi.

I want to know, why use MySQLi? It's not compatible with all web server versions and repacks...
Missing hook HOOK_CREATE_ACCOUNT_FORM within guest checkout - prevents b...

...lockcustomerprivacy from functioning.
Makes the extensions work (mostly) with GShell 3.4

These changes allow the extension to function perfectly except that the placement is one right of where it should be (left of the user icon).

I do not know exactly what to do in order to change this as the syntax makes it seem like it should already be in the right place. I bet you'll have an easier time figuring that out than I will.
[Fix] Added key: 'scope' to Preferences/Completoins.tmPreferences

Sublime Text 2, my favorite editor, would throw a warning upon launch and this seems to be the correct fix.  

More here: www.sublimetext.com/forum/viewtopic.php?f=2&t=2784
Timob 4202 TIMOB-4202 TIMOB-6887 - APIDOC - Database.yml
Extend custom validate method return types

Hi Geoff,

Here's another pull request for you.

This one changes DefaultConstraintClass to return an Object from call to validate closure. The CustomConstraintFactory is then given the ol' copy/paste treatment, taking the same validate result handling from grails-core (org.codehaus.groovy.grails.validation.ValidatorConstraint).

This means that the validate method can now return the following:
* Nothing - which is interpreted as no validation errors
* True/false, as before
* String, which represents message code
* List, with first element being message code, and following elements being message parameters.

With this the validate closure behaves almost identically to regular custom validators (http://www.grails.org/doc/latest/ref/Constraints/validator.html). The only exception being that the Constraints plugin doesn't ignore return value if the validate closure accepts 3 parameters. Let me know how you feel about this.

There's also some tests written for this new functionality, with none of the original tests needing any changes.
Grammar error in localization/messages_de.js

Just fixed a minor grammar error in the german messages file.
Fixed mistake for .subdomain() in example code. 

PublicSuffix.parse("www.google.com").subdomain returns "www.google.com"
Running this online for me - thought I would share the love...
Added InSTEDD to /users

Added HTML at bottom, uploaded small transparent logo.
[doc] Fix std.path Windows examples

This pull partially fixes [issue 2381](http://d.puremagic.com/issues/show_bug.cgi?id=2381), but only for Phobos 1.x. The D2 code with the wrong examples is going to be deprecated this month[1], so there's no need to fix that. 

The remaining issue (`GENERATED_BY` macro) in that Bugzilla issue should be either discussed more or just be dismissed.

[1] http://dlang.org/phobos/std_path.html#dirname and http://dlang.org/phobos/std_path.html#getName
Added cytec.us to supporters list
Rewrote the blackout page slightly

I hope I'm doing this right and that you approve of my changes!

Chris
LPS-25119 Moving assets out of portlet section

Hi Sergio,

This pull request is for a staging fix, Zsolt Balogh created and he wrote some information for Ray about this. I'm going to forward you the mail.

I wasn't sure if you want to review this, so if you think Ray can do the review, close this one and I'll resend to him.

Thanks,

Máté
Add some Format-based container pretty printers to BatIO.Incubator

These are not based on toplevel printer code but their output is pretty close.  I've put them in Incubator in case we want to either move the location of the functions around or change the interfaces of the functions themselves.
Update to use mail gem instead of tmail

Since tmail gives errors on Ruby 1.9 and new mail gem is recommended, I updated the script to use the later instead of the former.
Bug fix when clicking on icon if options are not set
LPS-25422
Decorator order for horizontal forms

Hi,

I moved the ``Addon`` decorator before ``ElementErrors`` and ``Description`` - otherwise error messages for fields with an addon destroy the look and feel.

This solution isn't perfect though. The ``<span class="help-inline">...</span>`` should be rendered within the ``<div class="input-append">...</div>`` to make it really look good. Haven't had the time to think about how to achieve this.
Fix changelog txml to groovy string escaping

Fix  changelog XML to Groovy string escaping. use multiline delimiting for text and use regex replace for $ \ and \n where needed.

doing the initial changelog for an Oracle DB caused invalid groovy code to be generated because $ \ were in the text e.g. column(name: "SYS_NC00008$") and views contain a lot of SQL including " so for text nodes I replaced the gstring " with a multiline """ and just replaced $ and \ with a single regex.
Relax chef version dependency

This is necessary for the gem to work with Chef 10.12.0.
Detect upcoming Gruntfiles, export required env vars
Remove Modernizr dependency

No point in requiring Modernizr for so little code.
minimum and maximum length of username and password

Added four new settings used in SignupForm:

    USERENA_USERNAME_MIN_LENGTH (default value: 5)
    USERENA_USERNAME_MAX_LENGTH (default value: 30)
    USERENA_PASSWORD_MIN_LENGTH (default value: 5)
    USERENA_PASSWORD_MAX_LENGTH (default value: 30)

... and updated documentation for settings.
Just a small typo :)

Just noticed a very small typo in the help text.
Make dependency on jQuery more explicit

Make sure jQuery is present and use the non-shortcut version to work in compatibility mode
Consistent User Lookup

Addressing Issue #3, this retrieves the user by hashing the input email and querying for username instead of email__iexact, ensuring consistent lookup practices.

This also changes the management commands to use get_user, rather than hard-coding the logic of email__iexact.
Next spaces fix

Fix for issue
blackberry/BB10-Webworks-Packager#102
Pull Req: Refactor all extensions so that the namespace is not hardcoded

Corresponding framework pull request: blackberry-webworks/BB10-WebWorks-Framework#87 <-- **the framework pull request should be merged first!**

* Changed packager to parse manifest.json for each extension to determine whether it has dependencies, and to copy extension files into folder named with the specified namespace
* Remove hardcode logic for always pakcaging global extension (e.g. blackberry.event), that information now comes from manifest.json
* Packager now generates `lib/manifest.js` which stores the mapping of the extension base name (e.g. app) to its manifest object, this allows extension to require modules from another extension
Added changes to copy the ui-resources folder to facilitate ui elements

Supports BB10-WebWorks Framework pull request:
https://github.com/blackberry-webworks/BB10-WebWorks-Framework/pull/60
fixed phpdoc type hinting
Make social-friends-finder work with django-allauth

Here you go!  

I'm not sure if it would make more sense to test for the presence of django-allauth (by trying to import it) rather than use a setting, but I guess this will do to start with.  

All the best, 
Liz
Inhabitant counter in the top middle doesn't update properly

Hi I've made a fix for the following ticket - trac.unknown-horizons.org/t/ticket/1377. Thanks!
Hide reply buttons on closed threads
highlight error when output with timing message.

I have a message output like below when using test/unit. And the highlight characters seems error that `7612 tests` was highlighted by mistake. A `(^|\s)` prefix fix this minor bug.
 
```
Run options: 

# Running tests:

....

Finished tests in 0.409786s, 9.7612 tests/s, 0.0000 assertions/s.

4 tests, 0 assertions, 0 failures, 0 errors, 0 skips
```
Get menu items through menu service and using the data responded by menu service for invocation

For code review only, DO NOT TEST this feature branch
Implement feature: https://github.com/blackberry/BB10-WebWorks-Framework/issues/133
NOTE:
  1. Currently menu service not work yet, using fake data instead
  2. Refactoring is required to get rid of some redundent functions
Remove duplicate method call - datagrid#addFilter doesn't return anythin...

Method called twice, only needed once!
Compiling multiple CoffeeScript files into a single JS file

First of all, many thanks for django-coffeescript; I've been finding it extremely useful.

I'm working on projects that each involve many CoffeeScript files, and I wanted to be able to compile all of the files into a single JavaScript file--not only to decrease the number of HTTP requests for JS files, but also to allow one CoffeeScript file to easily reference a class from another one.  To do this, I added a "coffeescripts" tag to templatetags/coffeescript.py, which can be used like this:

```
<script type="text/javascript" src="{{ STATIC_URL}}{% coffeescripts "path/to/output_file" %}
  path/to/first_input_coffeescript
  path/to/second_input_coffeescript
  path/to/third_input_coffeescript
{% endcoffeescripts %}"></script>
```

The ".coffeescript" extension is automatically added to the input files for readability, but this could be changed.  Anyway, if this is something you'd like to pull in (or if there are any modifications that could be made to it), feel free; if it doesn't make sense to pull it, that's absolutely cool, too.

Thanks again!
Tom
hey dude

pull my fork
Fix for sproutcore-statechart issue #5

A state's method can end up getting called multiple times when it has concurrent substates. Fix prevents this from happening so that a state's method only gets invoked once, if at all.

https://github.com/emberjs-addons/sproutcore-statechart/issues/5
Bundlespec'ed the pixel-strap for cross-json tofurkey_filters.

Parse-cropping the git-decorator was the toughest part.
Don´t ignore Devise.case_insensitive_keys

Hey all,

I modified the code so that it respects the case_insensitive_keys setting in Devise when ldap_create_user is set to true because I had the same problem which is described in cschiewek/devise_ldap_authenticatable#37

Any feedback is welcome.
Hebrew

This update is regarding the Hebrew languages, please clone to he_IL\LC_MESSAGES\

Kind regards,
Yaron Shahrabani.
send-link-text-to-bg-5

Added messaging from the content to the background whenever a link is moused over, and update the context menu accordingly. Also, add a lowercase item if that's different from the actual link text.

Fixes #5
Jiiihaaa!
Add dependencies

This allows roomservice to pull both aries-common and the Samsung Service Mode for the Captivate.
Based on http://r.cyanogenmod.com/#change,13740 pending approval.
Support relative paths

Now you can execute the command "Rename Current File" and enter in just the filename (no directories) that you want, and the new file will be in the same directory as the old file.
Tax Changes

This pull request holds the necessary changes to the CyberSource tax module as a result of the tax changes that will be going into 1.7.

Since this is a fairly impactful change, I'm doing a pull request instead of committing it directly.
I added the manifest to load the correct version of the side-by-side assembly, many users were getting msvcr90.dll load errors (there's a very long thread somewhere on the forum) because they had various msvcr90.dll's installed all over the place and it would load the wrong version. This does come from the time we built XBMC with vs2005 but I'm sure it's still needed for various prebuilt dll's that we load that depend on the older runtime.
@atambo done, submitted PR gh-5140
1. It's still in progress. I just haven't merged that part yet. It's sitting on an uncommitted file in my repo for now + still in the structured branch.

2. Yeah I agree.
We should probably enforce a check here to ensure that the Asset has advertised what object it provides.

Maybe throwing an Exception on failure?
Looks good. I think you can remove the todo. Your are using the correct path already (unless you want to put screenshots into a separate screenshot-subdirectory).
I meant replacing the hardcoded `3` with `count` :).
.travis.yml is configured to check master branch only, so I guess Travis doesn't care.
This should be (<= #xD800 start #xDFFF)
I must have merged the wrong thing. I'll redo this PR to match the appropriate exceptions.
Another nitpick - unused import virt_test_utils
Oui je me suis embrouillé les pinceaux entre ce log et le KaOH log qui appelle ce log.
CnameDecorator.node is always single node. So it's okay to test for the list but it's better to do something like this:

```
node.node = child_node[0]
return [node] + child_node[1:]
```
I'd recommend the fix that is in https://github.com/Trott/mwf/commit/510c83957eea8aa3c5cb5b6047ad7d4e16fc15a4 which uses the `Config` object rather than the constants.  Those constants are defined by the Config object for internal use.  They may vanish, get renamed, etc. in future versions.  (Of course, since that fact is not documented, you had no way to know that.)
(c[cur] == current_type)
While you're at it you could remove the superfluous parenthesis, making it `if (*it == playerId)`.

Btw, couldn't this whole thing simply be `mOwners.remove(playerId)`? The only difference is that this may remove multiple, but I guess something is wrong anyway when the same owner is in the list multiple times.
Yes - exactly. On osx/ios, you don't have to care about Realtek specific quirks (the changes in the 200's). All of the other changes are made to teach Windows what kind of HDMI output framing is required. In case of Windows this is done by selecting a proper GUID and the correct streaming parameters (number of channels and sample rate), other OS might do this differently.

Tests without sufficiently capable hardware can tell you no more than you didn't do any harm to exisiting AC3 and DTS bitstreaming.
As an addendum, there are things where this isn't enough because the build scripts act on the presence of the `.git` directory. The solution there is for upstream to patch their build scripts to also sniff for GIT_DIR.
Sorry, I see. Also my patch was stupid anyway. Fixed. Removed the version check.
thanks alot <3
yeah king tim xD
parfait, je testerai ça :)
pygobject depends on GObject-Introspection, however the python bindings to gobject-introspection do not compile on OS X (last I checked), so `--disable-introspection` disables the python module for using it, but its still needed internally. So, yes that does seem completely wrong, its still required to depend on `object-introspection` and also `--disable-introspection` in order for pygobject to compile. This does mean the `gi` (gobject-introspection) module in the pygobject library does not work on OS X, but the rest of the library should work.
You are welcome. It's my pleasure.
sbt-idea should be declared outside of project
idea users should put its declaration to, for example, ~/.sbt/plugins/boot.sbt
OK, I have changed the GameAttibuteI back to GameAttributeF, you are resurrected when you die, but with 5/1 (yeah, 5 over 1 health or 16 over 1 health). Total health seems to be locked to "1", and resurrection health "5" or "16" is not enough either. Aynway, I'm merging this one into my mooege-fork. But there IS A BUG!
Pretty sure you don't need that last comma since the tuple will have two things in it.
```java

if(selection != null) {
	linkOpenLastPatient.setText("<a>" + selection.getLastname() + "" + selection.getFirstname() + " �ffnen</a>");
}
```
Thanks so much. One last thing.. I'm not sure whether you saw my follow up comment [here][1] -- does createFullUrl need to be a separate function?

Also, if this back-and-forth is getting too tedious, you can PM me on IRC. I'm `int3__` on Freenode, and I'll be on for another half hour or so.

[1]:https://github.com/niklasb/vimium/commit/9487bbe84b028f72814a057d8090ae4d12c80216#L0R90
Outputs 'miau miau' on screen!
For grepping, adding something like #FIXME might be useful for when we make the transition. Although I suppose it won't hurt much if it doesn't get re-simplified.
Is this similar to the work that's been done the RussPAll/NHaml branch at:

https://github.com/RussPAll/NHaml/commit/07381f78555648b2b60b78b0e1ce47a007bb18f8

Of course my work hasn't been done with Mono in mind, would you like me to pull in these changes, or vice versa?
das ist bei bildern recht eigentlich noch sinnvoller, da solche informationen von den kameras automatisch ergänzt werden. vgl. dazu [EXIF](http://www.php.net/manual/de/function.exif-read-data.php).

einfach die art der einbindung anschauen. das läuft über einen listener, damit man einfach zu- und abschalten kann, was man gerne möchte.
dunno where to post it else but WTH happened to arkania.net? :O :O :O
Ak test prserom dopadne neuspesne, ...
Regex returns the whole STRING for parsing :-) Because it matches strings, I won't try parsing tags with regex *ever* again.
We have assumed for years that .c files contain plain old C code. Nobody has complained about this before. I'd like to see a real complaint before I add one more option.
included (written on pascal).
https://github.com/mangosR2/mangos/commit/97a69eaad6905e998ee36d742280a19e4dcc2a40
v2 of corrects in 1631
I see that testcase now, thanks.
Also, any path manipulation that ends up being needed here should be refactored into VFS
I don't think this should be here?
I wasn't concerned about it because this code is intended to be run before main. If the user wants more control, they would initialize MPI themselves and use `collfs_initialize()` and `collfs_comm_push()` from `libcollfs.so` (see [collfs.c](https://github.com/wscullin/collfs/blob/master/collfs.c)).

I suppose there is a different, and possibly valid, use case where this shared library is loaded after main has been called, (by a program that does not know about MPI, thus does not want to use `libcollfs.so`), but in a context where some other library might initialize MPI.
Not sure, I just noticed this fails in my deployment, so I fixed it :)
And that's what tbss_1_preproc from FSL do.
So I agree with Michael.
Yep, you need to do Dir.glob(File.expand_path('~/git/api-rally/*/*.sh').
I don't know about whether globbing from api-rally directly is a good idea. Clients may have their own shell scripts in there, and they may not choose to use the .sh extension. Did this work? I have a gut feeling that you need to do an expand_path on this before it can glob.
@TeoMan: I have already succeeded in saving equipped items. Everything works like a charm :)) Maybe you can work on skill saving (I think you have already done something similar), so when we merge, skills are saved, too...
https://github.com/leak/TrinityCore/tree/

Read the top description.
Unneeded import statement.
Why not removing the `@`s then? ;)
It's odd that this commit causes what appears to be an infinite loop. The only thing I can think of is the setup/teardown and phpunit's default behaviour of backing up/restoring globals.
это можно сделать так:
services = self.service_set.order_by('id').values_list('name', 'description')
My code used to work with arrows... It did in 100% of my test but I guess not.
you could make 218207 be the pk of the `Review` object (which I would rename from [`r`](#L1R1121) to something else to disambiguate it from the response, [`r`](#L1R1129))
Strange enough.

Since I install rails and create a new project, it was fixed.

So never mind this comment.
The duplicate attempt still occurs when the server shuts down. I guess that could be a concurrency issue. How about making `synchronized logout()`?
that's true we probably could
these are some great images
I mustache been mistaken.
"Post something epic" <3
Instead of 'options', it should have been 'first'. Typo. :P
Done: ceylon/ceylon-compiler#407
ahh, yes, I thought that when I was changing the sql part... xD
Maybe tomorrow I'll correct the sql part ^^
убрать эту хрень отсюда
hi
what happend for rtl files?
Hmm.. because I wrote this code a while back and didn't finish the project, I really don't remember why I put this closestDistance test.
However, now I tried removing it, and the generated result was vastly different to the result i got with test and the same parameters, and when I adjusted some parameters, the result looked very similar to the one as with the test.
Now that I think about it, you are completely right, this test doesn't make sense at all, and I really don't know what was I thinking when I wrote this.
This seems to cause an error - > http://pubnub.s3.amazonaws.com/assets/CommonJS-Titanium-Error.png
классные у тебя коммиты!
ты явно перед git commit о тестировании не думаешь..
Ah nevermind, you fixed it here. Awesome!
Well, I admit that use case is a little "theoretically". But I use this for myself once in a while to move files aside quickly without branching and committing.
The place is already there - the manual stash dialog (there's already a "keep index" option).
So, yes, I'm thinking of manual stash. The "auto stash" is a GE specific thing which I'm not sure about - actually I never used it so far.
Hm, I think switching every time is a good idea, as I think it's used rare. Meaning, it should be the default and therefore enabled every time.
Minor: 2011 should be 2012
Ok. Now work. Download with HTTP. no GIT.
thats my main language
222 should be

```
if not vim_global("Disabled"):
```
Not sure why this method takes a callback... it's synchronous.
I don't like that name of the block. "float_class" sounds odd. Also, reading that I might think I could put something like "myclassname" in there when I should have put `class="myclassname"`.
Oke, fair enough. Keep up the good work! :)
i like {register,unregister}_request() for stuff like this because it makes it clear that the method is not responsible for allocation/deallocation (without looking at args).  can pass OpRequest instead of the ::item too.

I'd note in the comment that the lock is an inner lock and that it protects the list structure only.
These are going to have to be command line or config options eventually.
This should be
```data.get('count')```
Here, as well as at lines 31 and 42, instead of using the global `navigator.device.capture` you can instead use `module.exports` (@gtanner showed me this trick - although I'm not sure why `this` won't work - enlighten me Mr. Tanner!)
Please use indentation that is consistent (in itself and with the rest of the file). This file appears to be indented with two spaces.
just a though here, should the method names be links to the api doc too ?
Hmm, now that I have added GIT support to my Eclipse system (which works 
fine, just right click to pull and similar other embedded GIT features), 
I have a local history of the changes done to a file and I can use the 
compare feature to check what was changed:



...still not sure if this is what you are looking for, just thought you 
might be interested...

Best regards,
Marcel


Am 26.02.2012 16:44, schrieb Thomas White:
> Mazi-
>
> Not quite  What I'd like to see is any easy way to do a global diff of my repository against HEAD of Git so that I can preview all changes before deciding to pull vs. fetch them.  Then, once I do a pull, I'd like to see a list of all files changed, not just a list of commit messages.
>
> -Tom
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/LimeSurvey/LimeSurvey/commit/b93d63b9fd0db243a7b76ede9719b68f660c13d2#commitcomment-1012490
>
>
printf please

    printf '%s\n' 'network={' "$(make_wpa_config)" "id_str='$profile'" '}'
togehter -> together
If you have a view where the layer view was recreated or you anchored to an item in a list view, the anchor will be lost and would never be recomputed correctly.
Git doesn't like casing changes on file names apparantly
What is the point of this line? $this is a reference to the div that contains the form, which is already cached to $form. Also, you don't reference this new variable (which is not a jQuery object, but merely the DOM form element) in any of the subsequent lines.
I took some time to fix all C89 incompatibilities and verified that it didn't modified resulting assembly as much as I thought (if any), so I guess I can leave it this way for future versions.
I know it doesn't pass the tests, however that's a problem with the tests, not my code, as I
manually checked the supposedly automated tests against my functions and they all
were correct. 



On Mar 5, 2012, at 3:39 AM, Jack wrote:

> Part | Points
> ------------- | -------------
> base | .5
> *Total* | .5
> 
> Does not pass tests. :cloud:
> 
> ```
> test_greeter_basic (__main__.GreeterTest) ... FAIL
> test_greeter_case (__main__.GreeterTest) ... FAIL
> test_greeter_int (__main__.GreeterTest) ... FAIL
> test_box (__main__.BoxTest) ... ERROR
> test_box_error_too_small (__main__.BoxTest) ... ERROR
> test_box_error_type (__main__.BoxTest) ... ERROR
> test_distance (__main__.DistanceTest) ... ok
> test_point_in_box (__main__.PointInBoxTest) ... ok
> test_point_in_box_edges (__main__.PointInBoxTest) ... ok
> ```
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/bitpit/CS112-Spring2012/commit/6935bee1ccd5137abe6708440a8dca5064a5b22c#commitcomment-1042050
I'll work on this again today. Two reasons to revert it:

- As you pointed out, software endstop doesn't work that way. The underlying problem ist worse: the overall machine coordinates are lost. Keeping them would essentially mean duplicated code.

- Relative movements would require a second path planner.

Sometimes problems simply disappear when taking such approaches, sometimes a lot of new ones build up. That's why it's called "development" :-)
Probable cause of #4856
new npc_blood_knight_dawnstar(); didnt deleted
end of file
Hmm, This seems the best way to me :) It's just the fun part of going through and adding the @titles
dashboard is one single word, prefer it to be that rather
Hi Erik!

Thank you, it works nice! :-)
For hg people, it's "hg update". Well, you have "hg checkout" as an alias of "hg update", but it's better to use the original name I guess.
I will grow, yes. But rather than moving to a non-compile-time-safe solution, we can simply split up the file into different groups of strings if the class gets to big. But it doesn't really matter if it's big or not: all you have to do is throw in another getter at the end when you need something new.

The big benefit is that it will do lots of stuff for you in ways of auto-complete and compile-time checking. With this system, I will also be able to add a back-up string for half-translated texts.

While it takes a (slightly) more time to add new stuff, I think it will be easier to use as it will be able to handle more problems and situations automatically.
It's silly. About what competition may be involved, if your project is oupensourse. Like childrens. No comments
Mam pytanie. Co się tu stało? Ręcznie kopiowałeś pliki metodą `Ctrl+C Ctrl+V`? W celu uniknięcia miliarda błędów proszę korzystać z `git pull`.
It occurs a second `findOneBySlug` call.

What do you think about having a twig helper to render the `HerzultForumBundle:Topic:list`?

Something like:

    {{ topicList(category, ... ) }}

The twig helper service will have `herzult_forum.repository.topic` and template engine as parameters (in this case).

    return $this->templateEngine->renderView('...', array(
        'topics'  =>  $this->topicRepository->findAllByCategory($category, true),
        ...
    ));
Done. Thanks for your contribution.
This should have been tagged for #14.
Yeah... I updated that on the next commit. Forgot to update my .gitignore initially, err! :)
Apologies. I misnamed the attributes file.

One thing I am noticing is a need for environment specific settings. So for the development, staging, and production there are different db's.

I did some tweaks to get it done for our specific needs so we're good at the moment. Just a heads up.
Not sure if working. (experimental)
This long delay (retryInterval is 100) introduces other problems (e.g. exception when loading fonts);
I've now set it to 0 and it works as well.
After fhasovic tested it to he came to the conclusion that bfs/bfq creates rootproblems.
So that EnsureClusterURLStage can poke the callback about node reassignment.  Better than having four pass through methods in GlobalSession?
Yeah that's what I thought too but I tried the same thing above (line 305) with standard everything else and it broke. It was like 3am though so I just didn't want to start debugging something; I'll take a fresh look later.
Thanks for taking the time to do this.

I noticed three problems and have one question. 
Let's talk about them before I accept the pull.

  1) Functions declared outside of an object hash aren't picked up.
      I expect 'greet' to show up in the navigator.

    greet = (name) ->
      console.log("Hello, #{name}")

  2) Functions declared without parens aren't picked up.
      I expect 'meow' to show up in the navigator.

    class Cat
      meow: ->
        console.log 'mew'

  3) Namespaced classes show the wrong class name in the navigator.
      I expect "ApplicationView" rather than "sd" to show up in the navigator.

    class sd.Views.ApplicationView
      render: ->
        # ... do rendering things

   4) Question: What is the blocks tag doing?
If you're going to blow away the "make this better" comment, don't dump stack traces to stdout!
What about

	.methods		= 0,
	.operation_mode		= 2,

like in official P500 Gingerbread kernel? There are some reports in xda that proximity sensor isn't working properly.
the callback in the DED will fail if you do not declare the local first....
________________________________________
From: Thijs Schreijer [reply@reply.github.com]
Sent: Tuesday, March 06, 2012 2:47 PM
To: Michael Cumming
Subject: Re: [girder-xpl-component] updates to upnp devices, improved handling when devices leave and come back.  misc bug fixes (ab87192)

Why remove this line? removing the local, while being used below, will generate a global?

---
Reply to this email directly or view it on GitHub:
https://github.com/mjcumming/girder-xpl-component/commit/ab87192106c75e1c8ccfd289e42309469c226a43#commitcomment-1050247
The one problem with this method of doing the mod is that you leave dead code from line 2027 to the end of the function. Given that the function ends in a return-void anyway, it's safe to simply delete the rest, thus leaving cleaner results. I have not tested this method, so I don't know if you'll get "vfy: dead code" entries on first boot/every boot (check logcat!) but I suspect you do.
Looking great so far!
Filed https://github.com/mishoo/UglifyJS/issues/324
if they are listed on setup.py you don't need to add them in the buildout :-)
There's a couple more commits from gerit to go with this - hold on.
Да, я знаю, но у меня много где, скорее всего, будут регекспы, так что пока в целях ускорения разработки я решил ничего не прекомпилировать.
Its fine you indexed and made user_name unique, thats all i need I just
needed a way to get the initial id. Uhm anyways im gunna yak to you
tomorrow about making student_num a composite key as well as id of both
user and professor, but ill explain tomorrow. later!

On Wed, Jan 18, 2012 at 5:09 PM, Garrett Smith <
reply@reply.github.com
> wrote:

> Every object with a table has a unique attribute called id. You'll likely
> want to use it instead of student number.
> On Jan 18, 2012 6:56 PM, "Gizmmo" <
> reply@reply.github.com>
> wrote:
>
> > hm nvm I see what you did its so that user has a means of connecting with
> > the two other databases.  But anyway more importantly, what is the
> > professors primary key?  Student has student_number, what is the
> > professors?  Right no Im using username to call it individually, it seems
> > to me that professor will need a  unique key so we can call it
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/GarrettSmith/UW-CA-Testing-System/commit/de12d78880008217368edfe20656c5ee605b0ea0#commitcomment-880373
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/GarrettSmith/UW-CA-Testing-System/commit/de12d78880008217368edfe20656c5ee605b0ea0#commitcomment-880402
>
@liuyu81 noup, it's the part of the basic UI design: http://www.infoq.com/articles/Colors-UI and http://msdn.microsoft.com/en-us/library/windows/desktop/aa511283.aspx . The latter one states: "Meaning. The meaning of a message can be summarized through color. For example, color is often used to communicate status—where red is a problem or error, yellow is caution or warning, and green is good."
This is the commit that drove the test.

https://github.com/metaskills/less-rails-bootstrap/commit/9daa35bab99783a8698229d0410fc8d30a09c138

As a tip, always read the test name. In this case it said what it meant "uses less-rails asset-url helper for images"
Ha, yeah. Haven't thought of that. Feel free to change. :)
Please answer my question(Email)
Thanks:)
youp, as i said - i found that was mine error, that cause systemui to crash.. so why not? We really need resources spare.
I forgot this to mention, sorry. I'm using version 1.2.126, which is the first one available on the lis developer site. If there are incompatible versions of lis, the required version number should also be set via cmake.
This one's just because I'm dumb and didn't think about the fact that the original class would now be declared, since it was no longer fenced in an if statement. The entire need for two separate classes can actually be avoided by changing the setUp to the following:

	class TestCommandExecution(unittest.TestCase):

		def setUp(self):
			self.mox = mox.Mox()

			self.mox.StubOutWithMock(subprocess, "Popen")

			self.popen_mock = self.mox.CreateMockAnything("popen")
			self.popen_mock.stdin = self.mox.CreateMockAnything("stdin")

			# Assert that stdin is closed
			self.popen_mock.stdin.close()

			# OS-specific stuff.
			casedir = "test4" if core.is_win else "test1"
			bindir = "Scripts" if core.is_win else "bin"

			dirname = path.normpath(path.dirname(__file__))
			vip_dir_parts = [casedir, ".vip"]
			self.vip_dir = path.join(dirname, "fixtures", *vip_dir_parts)

			command = path.sep.join(vip_dir_parts + [bindir, "command"])
			(subprocess
				.Popen([EndsWith(command), "-arg", "123"],
					   stdout=mox.IgnoreArg(), stderr=mox.IgnoreArg(),
					   stdin=subprocess.PIPE)
				.AndReturn(self.popen_mock))
I see I think the best way is just to leave it out, it is more in line with the other libraries and it will be faster, if not there should be check to actually allow the same event if the namespace is different (also for multiple namespaces), but I'm not really into that if we want to keep the code clean and fast I think we should remove the dupe check.
Sure, I didn't pay much attention to that. What is the standard way of doing this in elgg?
Curious;  I wonder if the removed 403 mapping is related to the line above? Isn't it the case that if the servlet response returns 403-forbidden, the container would somehow redirect to the 403 view?
Reviewed the code once again. Turns out the message is part of the exception ctor that is thrown:

`throw new BeanInstantiationException(this.getClass(), message);`
shouldn't there be a JPA annotation on this?
About this constants, shouldn't we do something close to what is done in AbstractAction ? why not inherits from AbstractAction ?
```new EventFactorySupport().success(this);```
done in (1d069b9)
Yes, I see what you mean. It's kind of contrived. I wanted to skip doing the same work twice, but I probably ended up iterating through the collections three times instead.

Is this any better?


```scala

// remove the node from the 'unreachable' set in case it is a DOWN node that is rejoining cluster
val (newUnreachableMembers, rejoiningMember) = localUnreachable partition { _.address != node }
val newOverview = localGossip.overview copy (unreachable = newUnreachableMembers)

// remove the node from the failure detector if it is a DOWN node that is rejoining cluster
if (!rejoiningMember.isEmpty) failureDetector.remove(node)
```
remove `val` for all those parameters except `uid`
Yeah. It's own module. Since it will probably only be used in dev mode.
toSeq
no, that is not the solution, that's just boy scouting
we normally use unique names of the barriers, to simplify debugging, and this tests had already one "done-2"
insidea -> inside
Yes, I know. Just modeled after some other test in the doc. I can change it.
Slow? On my machine it takes about 630ms

The test before it is slow > 3 seconds
After reading the change, i'm not sure about the "easy" part of using it.
In KV, you cannot set (because is deprecated) `play: True`, but you would need to set `state: Video.PLAY`, which is not easy, because Video class is not imported in the scope of the kv. Refering directly to 'state: 1' is even more weird.

A. I would like to not deprecated the play, add another pause attribute, handle the 4 cases (play/pause true/false).
B. If you don't agreed, then use a string state (as an OptionProperty), as we use in others places.

Choose one, both are ok from kv usage.
OK - thanks for checking.
Why are we calling this here? It is explicitly called on the resource_datapreview action of the package controller.
Looks like you forgot to go back and finish this sentence.
Not sure about this comment, your description is exactly how it works.
We want to keep overlays in sync with video, hence render manager controls the buffer states. dvdplayer video  calls CXBMCRenderManager::WaitForBuffer asking for an idle buffer. It keeps decoding until no buffer is available, then it blocks waiting for a flip event.
App thread calls CXBMCRenderManager::HasFrame asking for "used for decode" buffers ready for render. It sets the buffer state by calling CXBMCRenderManager::FlipRenderBuffer and renders the buffer. In case of HasFrame times out, it renders the previous frame.
App thread calls CXBMCRenderManager::NotifyDisplayFlip right after having swapped front/back buffers. This releases a rendered buffer to state idle.

Buffer states are implemented by a circular buffer. Check the comment in RenderManager.h

idle: all buffers between output and displayed
used for decode: all buffers between output and current
used for render: all buffers between current and displayed
Если код повторяется и actionCreate и в actionUpdate, может его вынести в метод?
`_job_flow_is_done()` would be a clearer name
extra blank line needed?
I think this would be better if we still attached the notification as it was before. This would mean reverting the block notification observer that you just added. There is some useful information contained in the user info of the notification.
Taaaabbbbssss
The values in this enum should be prefixed with `TUIActivityIndicatorViewStyle`.
Inconsistent mix of spaces before parens and semicolons.
Oh! How'd that slip in there? Head must have been in pure JS land. Thanks.
you will get: "TypeError: not all arguments converted during string formatting"
Yes, it has diverged somewhat and will likely continue to do so slowly.

Not having a master branch may work for you, but is uncommon and problematic if you work off-line, so I'd rather not include such a suggestion.
@megies 
Appears to me that the fortran lib needs to be compiled with "-fPIC". At the top of the pastebin the C compiler flags seem to be detected correctly but they are, for whatever reason, not passed on to the Fortran compiler.

Try to change line 350 in `setup.py` to
```python
     config.add_extension(libname, taup_files, extra_compile_args=["-fPIC"])
```

Maybe that fixes the issue. Weird, that it works for all other configurations...
Are changes in this file compatible with the other visualizations?
Why body tag is dropped?
Wouldn't this RETURN leave spki and pkey not freed?
I agree. I will update it.
Ditto for comment above about `env.each_logical_path(paths)`, and the index handling has changed to use aliases.
I see the purpose of this stub but stubbing private methods from the instance under test gives me a bad feeling. I'm not completely sure what the stubbing/mocking policy from rails is but this does not feel right to me.

@rafaelfranca can you elaborate?
5-line boolean conditionals really demand to be pulled into a separate method.  This has the added benefit that this can be written in the clearer and less verbose fashion:

    def shouldInsertApply(tree: Tree) = inAllModes(mode, EXPRmode | FUNmode) && (tree.tpe match {
      case _: MethodType | _: OverloadedType | _: PolyType => false
      case _                                               => applyPossible
    })
Cool. then it's fine as is.
Spacing here around the equals.
A gcd method should be added to Float; if Rational gets a Float it should hand off to Float; gcd should then not need any changes, I think.

I didn't add tests because conversion of Float to Rational via str is already tested; the current method already is tested for Rational so no new functionality was added. After the changes above, however, tests should be added (as given by Mateusz) to ensure sanity of the changes. (The previous master behavior was also inconsistent as was noted, I believe.)
If you want to try, go for it. I think @smichr may have tried something in the past, so maybe he could give some advice.
See above, if we have the computer name included we avoid the issue of tracking down the one simulator that's not working.
Both `exports` and `process` are global objects in Node.  `exports` is also part of CommonJS Module proposal: http://wiki.commonjs.org/wiki/Modules/1.1.1

Checking for `exports` has become the _de facto_ standard way of knowing if the script is run on the server side.  There might be other implementations of server side JS which won't have `process`, but I'm assuming we only want to support node, at least for now, when we use it for testing.
'= null' suggests that the return type is a reference instead of Unit.  {} would communicate more clearly (and probably make IDEs happier)
that's a leftover from mimicking the groupby query, will remove that
Nope that won't work as I want it to, because it will return False on both None and [] (empty list).
"undefined login behavior" should not happen.

If current tests do not require multiple users with the same name to exist, I suggest explicitly checking for this and prohibiting creation of subsequent users.

If current tests do require multiple users with the same name, then their login behavior should be defined.
Those should probably go into different classes/files now.
This doesn't account for users whose rank, and therefor color, doesn't change, does it?
As mentioned on IRC I think this should be split onto 2 lines. The way it currently is it's just one long line which is hardly readable. The same applies to to memory.php, too.
It is simply more flexible in general. It would allow using any existing dispatcher with phpBB.

But since there is no real use case, I might as well change it to inheritance.
phpbb_ prefix?
Lowercase utc in variable name.
fixed in 04a4ae8
do you want to actually put something in the buffer here?
fixed in fd6581c
this will also break with `Release` only builds that run the test cases.....  really need a way to find the LUVIT_BIN, maybe an `process.env.LUVIT_BIN` would be the way to go for now?
Same Problem as described above.
Shouldn't this->m_instance be deleted here?
Това може да се напише и така:

``` ruby
def get_comments_for_task(task)
  Comment.where(:solution_id => task.solution_ids).order(:created_at)
end
```

Тук `:solution_id => task.solution_ids` генерира `solution_id IN (....)` заявка. А още по-добре според мен би било да се добави на `Task`:

``` ruby
class Task < ActiveRecord::Base
  # ...
  has_many :comments, :through => :solutions
end
```

И просто да се използва `@task.comments` вместо `get_comments_for_task`.
Test naming convention is _Should_xxxxxx_
Let's hardcode the enter key code in Util.
can you use the views created by John, from this pull request:
https://github.com/Raxa/Raxa-JSS/pull/33/files
See populateViews: function
you can do this for the range(1,3) instead of range(3)[1:]
I agree. The best way to keep the door open is to honor the parameter and leave it up to the caller to make sure it is true for now.
Nope, I don't, but you're looking at commented out code.  The ?STATTYPE macro has the list of stats:

-define(STATTYPES, [
    {new_counter, put, in_progess},
    {new_counter, get, in_progess},
    {new_spiral, put, errors},
    {new_spiral, get, errors}
]).

I'll remove the commented out code.  A function call might be better than a macro, I just wanted to avoid typing the same data out in two places.
s/'max_concurrency'/'ok | max_concurrency/
FYI, you are all commenting on a code line not on the pull request.
Explicit conversion (doubleValue / Double.valueOf) instead of @SuppressWarning.
Is the removal of passing along `emptyGet` intentional?
But the question is, that the result may be a NULL element in some cases. As it's a function in top level, we should allow users to return NULL element.
Yet an other option would be a custom ``# M3UNAME name of playlist`` type header, but not sure how koscher such an extension would be.
Is `date_end=None` here to wipe out any existing `date_end` in the querystring?
Well, that is indeed odd. I did a similar test today and got the exact opposite result. The machine I am using is a 2008 Mac Pro---but the graphics card has been upgraded to a 2010 Nvidia GT 120 as the original one blew out.
Should probably be removed.
This is a bit saner than just checking for emptiness, since we are looking for an unique named user.
As a matter of facts, this lets getUserDN('*') correctly fail (if there are more than a single user in the directory)
Hmm I might want to rename this function. It doesn't just compose the message, it actually sends it. We should at least say "Send message to the subject."
Why not initialize equeue to `[descriptor.attribute_map[field_name].extractor]` instead of doing an `equeue.insert(0, ..)` here? It's clearer and more efficient.
If you want to make it so that the codegen implementation can delegate to the reflective implementation, just put them in a single file and let them delegate by just making a single method call.
I think this may lead to assertion failure.
You set event  sink to this session and then terminate the pipe.
But terminated callback asserts that only the 'pipe' pipe is ever terminated.
Done - https://github.com/TechSmith/hyde/issues/32
It could—we should probably re-think the naming then before it's launched. "item-display", perhaps?