labofsahil/github-event-dataset-2012 huggingface dataset:
Migrate build system to Gradle

## Major changes

 - Remove Ant-based spring-build and related resources
 - Replace with Gradle-based build
 - Remove (and .gitignore) all IDE metadata files
 - Remove hand-maintained Maven poms in favor of generation by Gradle
 - Move integration-tests subproject to root src/test dir
 - Move spring-framework-reference subproject to root src/reference dir
 - Rename org.springframework.* subprojects => spring-*

See individual messages for the commits included in this pull request
for details on each of these changes.

Issue: SPR-8116


## Plan for merging

@jhoeller, @costin, @rstoyanchev, @poutsma, @sbrannen, @markfisher -- I'm planning to merge these changes into master on Monday.  It would be great if in the meantime at least a few of you could take the new build for a spin.

Even if you don't get a chance to try it before Monday, merging into master will be low-impact, as everyone is focused on 3.1.x work right now anyway (3.1.x will remain permanently on spring-build).

Just follow the instructions at https://github.com/cbeams/spring-framework#building_from_source, but instead of cloning a new repository as the documentation suggests, you can simply add me as a remote to your existing spring-framework repository and check out the 'gradle' branch there:

    $ git remote add cbeams git@github.com:cbeams/spring-framework.git
    $ git fetch cbeams
    $ git checkout gradle
    $ ./gradlew build

## Documentation

I would also appreciate feedback on these new wiki docs:

* [Gradle build and release FAQ](https://github.com/cbeams/spring-framework/wiki/Gradle-build-and-release-FAQ)
* [SpringSource repository FAQ](https://github.com/cbeams/spring-framework/wiki/SpringSource-repository-FAQ)

Note that a lot of the information there is general and not specific only to Spring Framework, but I wanted to get all this down in one place for now.  We can think about hosting it somewhere more general later.  While these are written for a mixed audience of committers and end-users, I want to make sure that this is documentation that any project lead can look at to understand how things work, particularly with regard to release management.

Basically, if anything is confusing to you, it should probably be in one of those FAQs.  Please let me know what questions you have as you try this out, and we'll get it in there.
Fixed directory path to get proper plug-in shell access.

The App::path returned the incorrect path to the AppCore which the ShellDispatcher couldn't read.
Make it easier to reuse form generation code, tidy urls and pep8
Bug 705767: Validate credentials before completing classic setup + fixes

Added check for user existence with 1.0/user; use server URL if info/collections then returns 404.

HTTP request to server is set to be "kept alive indefinitely" and isn't consumed on consumeEntity - leads to next HTTP request connection getting shut down. Not sure what to do about that.
LPS-24850

@matethurzo
Update README.md

Added I-20 form
INTSAMPLES-82 Fix Exception

Prematurely removed local directory.
Fixes signing of the Windows Store SDK

Other versions of the Facebook C# SDK are signed, but for some reason the Windows Store one is not.  This prevents other signed applications/libraries from being able to use the Facebook C# SDK for Windows 8 apps.
Basic tests for SFDCAccounts and SmartStoreExplorer (going through login/fetch from server/logout - not testing smartstore side at all yet)
Merged unstable into unstable20.
Workbook and samples coinciding with the Mobile SDK 1.1 release
Fix django admin 'create user'

The django admin 'create user' page is currently broken on a clean django install using longerusernameandemail. These three commits fix it up:

 * in f2ce156
   * django.contrib.auth.forms.UserCreationForm doesn't define an 'email' field by default
   * there's no need to 'update_field_length' on the UserCreationForm or UserChangeForm email fields since these fields are generated by a ModelForm - so they'll use the max_length parameter from the model layer, which has been already updated by our monkey patching.
 * in 4baac16 added tests for the basic CRUD functionality of the django admin interface.
Updated edit range logic

Updated edit range logic. Only inserts currently work.
Make timezoneList translatable.

* Added english default translation
* Added german translation
* Usage:

		$('input.datetimepicker').datetimepicker({
		     showTimezone: true,
		     timezoneList: [
		       {"value": "-1100", "label": "minusEleven"},
		       {"value": "-1000", "label": "minusTen"},
		       {"value": "-0900", "label": "minusNine"},
		       {"value": "-0800", "label": "minusEight"},
		       {"value": "-0700", "label": "minusSeven"},
		       {"value": "-0600", "label": "minusSix"},
		       {"value": "-0500", "label": "minusFive"},
		       {"value": "-0400", "label": "minusFour"},
		       {"value": "-0300", "label": "minusThree"},
		       {"value": "-0200", "label": "minusTwo"},
		       {"value": "-0100", "label": "minusOne"},
		       {"value": "+0000", "label": "plusMinusZero"},
		       {"value": "+0100", "label": "plusOne"},
		       {"value": "+0200", "label": "plusTwo"},
		       {"value": "+0300", "label": "plusThree"},
		       {"value": "+0400", "label": "plusFour"},
		       {"value": "+0500", "label": "plusFive"},
		       {"value": "+0600", "label": "plusSix"},
		       {"value": "+0700", "label": "plusSeven"},
		       {"value": "+0800", "label": "plusEight"},
		       {"value": "+0900", "label": "plusNine"},
		       {"value": "+1000", "label": "plusTen"},
		       {"value": "+1100", "label": "plusEleven"},
		       {"value": "+1200", "label": "plusTwelve"},
		       {"value": "+1300", "label": "plusThirteen"}
		       ]
Complete german translation

nothing to add
Add more conditions to control display of switch box.

Switch box was displaying empty for non-resellers. Added additional
conditions at the opening and closing of the #switchbox so it displays
only when needed.
LPS-25678

Missing performance improvement in upgrades starting from 5.1.8 or 5.2.8.
added composer.json and set tests as suggested
LPS-24268 Web Content Granted Email: invalid Link to Article and article title

This commit fixes
1) Article title in the notification email
2) Correct article url in the notification email
Bugfix for extended mysql module

Vagrant user tries to get status with mysql-command instead of mysqladmin-command
Added Provider Images API

Added Provider Images API.

Does not communicate with factory on create and delete of Provider Image.  To be added during factory integration.
Update init.p3.rc

MTP Device permission.
Release notes for Arquillian Seam 2 extension
Fixed Xcodef 4.x warnings and iOS5 bug discussed in issue #81

fixed and ready to go
add repo

add collective.responsive1140 repository
Rails templates that more closely mirror a standard non-DM Rails app.

This addresses Issue #28. Currently, while the rails templates do specify Rails 3.1, they remove support for Assets/Sprockets, resulting in a broken app (stylesheets/javascript fail to function properly).

I've also updated this to act more like a default Rails 3.1 application. I think it's misleading to the user, especially if they're new to rails, to turn off support for some Rails components (ActionMailer, ActiveResource), Testing (encouraging RSpec over TestUnit), and JS (jquery-rails) by default; it's my belief that the only thing these templates should do is swap out DataMapper for ActiveRecord, and leave everything else identical to a vanilla Rails 3.1 application. And this exclusion is not consistent; for example, the `require 'action_mailer/railtie'` and `require 'active_resource/railtie'` lines are both commented out, but only the `actionmailer` gem is included in the `Gemfile`.

This might be a bigger philosophy discussion on what to include/not include by default... maybe having two sets of templates, one "opinionated" template with stripped down Rails components, RSpec instead of TestUnit, etc., and then one nearly identical to Rails' default settings? If everyone feels that would be better, I can update the pull request with two sets of templates.
Updated wlan bridge description.. again
Fix tests

this will make tests pass for now but more work needs to be done on refactoring
Updating README.md with latest changes made to build process …

-cmake removed
-webplatform clientFiles changed
Standard backend status reporting

Adds last_flush and last_exception output to the stats command. These match the stats reported by other backends which display the delta since the last event, or the statsd instance was started.

I find the delta reporting a bit confusing, so I also added error which shows 0 when things are going well, 1 if an error was encountered in the last flush. This is in a separate commit if you'd like to omit it.
```
$ echo stats | nc localhost 8126
uptime: 2
messages.last_msg_seen: 2
messages.bad_lines_seen: 0
instrumental.last_flush: 2
instrumental.last_exception: 2
instrumental.error: 0
```
Highcharts mangles labels like "Retail < 84 Days"

It seems that Highcharts will choke on this label, and you end up with "Retail " instead of the full "Retail < 84 Days".
just a fix to a deprecation message

Sorry, I am very new here at github ^^. I only have used your gem and felt very certain to be able to fix this realy small issue ;) 


This is just the minor fix for deprecation-warning in rails 3.2.1 on "set_table_name"
Change the framework to use web platform.js

This should add a point of stability that we can maintain across webworks
revisions.

Also includes framework cleanup to reduce and centralize all the
confusing request logic into a single module with an understandable
workflow.

Also includes unit tests for this work which were previously missing
from the framework.

Testing for this should involve full regression.
VoltageControl:fixed part3

Minimum voltage is fixed to 1.15v at a start-up
Add ifnull to users sql

wrapping the name colums in IFNULL so concat will work if one or more columns is null
Modals should be resizeable.

I think it doesn't do anything wrong, but could be helpful some times (like in my project)
Updated to bootstrap2
Fix rotation reboot

fixes green screen and reboot caused by rotation animations
refs #18 - test README build process
Dev

I made some changes. Please review.

Ivan
Updates

- Fix false negative for localStorage test
- Fix localStorage check in store method
- Clean up whitespace and quotations
Fixed config and SQL.

There were a few issues I needed to fix with some of the config/SQL stuff. I'm pulling now with these changes- I believe Jason needs them to continue his work. The app will import message formats from the database now and read the config file on start-up.
Fix #74

Changed several things.  I changed the name of the admin function that enqueues the admin js because it was getting called by the `add_action` in `Core.php`.  Also added a hook before a WPSQT page is loaded so that I can enqueue admin js only when viewing a WPSQT page.

This should fix issue #74.
Add syntax highlighting plugin for LSL.

This is based on Devin Weaver's bundle repository, as most single files floating
around in the web are just dead links.
Parsing the 'Genre' field for artists into a taxonomy, marcato_genres.

Really helps filtering artists by genres, in this case. There probably a couple more places where data could be stored in a taxonomy instead of meta fields, especially where relatively complex querying by those values is involved.
support for class, interface and etc with some fixes

it was added support for "class", "interface", "library", "source", "get", "set", "return", "case" and "catch".
and fixed a few to see commit please.
Added basic support for Twitter Cards (among other things)

* Plugin will now try to add `<html prefix="og: http://ogp.me/ns#">` automatically
* Updated the Open Graph logo to the current version
* Added `height` and `width` information, where appropriate
* Added `og:image:secure_url` to YouTube videos
* Added Locale information
* Will try to pass along audio embeds

In addition, I have started to add basic support for [Twitter Cards](https://dev.twitter.com/docs/cards)
* Added support for `twitter:creator` and `twitter:site`
* Added `twitter:player` for YouTube videos
* Started to add `twitter:player` support for embedded audio, but Twiter's HTTPS requirements will make that more difficult than I had anticipated.

I apologize that this is such a monster of a pull request.
I could easily break this down into a smaller pull request(s), if that would be easier for you to digest.

### ToDo
* Add Description
* Find a way to add `twitter:player` support for embedded audio
* Support more video embeds
* Add a setting for changing the default image for a site
document getProtection (dmd pull 856)
deprecate.dd fixes

Minor fixes for http://dlang.org/deprecate.
Jared King

:-)
Update the URL to the world's funniest thread.

The project got moved when it was abandoned.
Update 0001/0000.txt
webkit -> moz -> others

For a better readability, keeping the navigator list in the same order can be
Add ability to override from name/address. (using Mail_Postmark::from())

In use on my own site, but hasn't been unit tested.
I really like this kind of commit!
better hope your repo doesn't involve eg**gpl**ants...
Thx for the tips Memphiz and Elupus - as you say - I did it bass-ackwards. Was going to just play safe and do a PR as I'm only semi-versed in Git, but my lowly patch was going to send a notify to every dev, which seemed silly given it's scope and low-impact nature. Pretty sure I understand what I did wrong (got some gentle slaps from IRC along with tips). Sorry for the oops ;)
This is all good except isn't strictly necessary because of the `try{}catch(){}` around the qSA calls which then defers to qwery() which in turn goes to the selectors.

There's even tests for it working:

 * define strange custom pseudo here: https://github.com/ded/qwery/blob/65e225567052a8037d8cc1c6e2b6c211cadc9e02/tests/tests.js#L3
 * invoke it here: https://github.com/ded/qwery/blob/65e225567052a8037d8cc1c6e2b6c211cadc9e02/tests/tests.js#L478-481

I believe also there's a slight performance hit from using `.apply`, particularly now in the newer engines because it's more difficult to optimise. So perhaps a `(qwery.nonStandardEngine ? selectNonNative : supportsCSS3 ? selectCSS3 : doc[qSA] ? selectCSS2qSA : selectNonNative)(selector, root)` (or whatever the arguments need to be)

I'm also a bit sad to see `.noConflict()` go. You're going to need to update the Bonzo tests: https://github.com/ded/bonzo/blob/7efcfb1fe51762daed1be574ab67f72b697caa42/tests/tests.html#L40
Can it fail? (why the word "Attempts"?)
10번 라인에 코멘트
I recall giving some feedback on this commit. I can't find the original discussion, but I would really love (if not require:) to see some test cases for such a major bug given that after the forward port there will be two code bases to keep up to date.
ok, I see now. Removed.
Thanks I will build it from there.
You've used this check in a number of places, I would extract it to a helper method:

``` ruby
def can_create_post_for(open_mic)
  host?(open_mic) || admin?
end
```
Attend mais tu comptes utiliser UDP maintenant ? Tu pars sur quoi là ?
Yep, it can do. Look at the example given at the end of:

http://www.gnu.org/software/make/manual/make.html#Prerequisite-Types

```
$(OBJS): | $(OBJDIR)
     
$(OBJDIR):
    mkdir $(OBJDIR)
```

This is saying the the objects should be made after their containing directory is made. Both the objects, and the directory that conatains the objects are made by Make. The order only dependancies is used to hint Make in which order it should do these.

How is this applicable to your use case here?

To put it another way, what happens if you don't specify this dependency? Does anything break?

If it nothing breaks, why are you using it?
because both are wrong:
1 numeric index per field
Should be `#{bin}/wget`

Or better yet:

````
mktmp do
    system "#{bin}/wget some-file-from-homebrew && cat some-file"
end
````
The fourth message should be "Plugin cannot be null" isntead of "Priority cannot be null".
Did you create a PR ? you should!
ok, i quite like the naked @ - i read it as 'this' . in ruby i've never thought of how to pronounce it. it was just a token indicating the instance/self. oddly, i guess because i use it in isolation in CS i'm automatically reading it as 'this'. nevertheless, if it is idiomatic, i'll give using 'this' a go
This is weird and feels like it should just be handled by the Makefile itself; is this because there's something we're not building?
Do you manage to have proper drag and drop on apps like Vim with tmux? This is the only thing that prevented me using it...
Ok krave5.
I suspect that this may actually be a Metamorph bug so I'm not sure if this is the place to put it. Any failing test is a good start though.
you know what might be nice would be to darken or lighten the red/green/yellow (forgive me if this does this, I was going off code alone and it looks like it highlights the whole row one color)  i'm home and don't have my dev environment set up on this laptop right now

you could probably do this by adding a second class on hover and having that class have different colors but still keep similar coloring
/r好像是按照程序中的样子输出
/s好像是全部转成string输出
Changes from here
Still loving the comments. It makes the code much easier to follow
I'd like to see this reviewed and merged in to the H-uru fork if at all possible. Would you mind opening a pull request?
These could be set to return the int id, might as well do something useful with the return, since it is void anyway.
Should use string interpolation:

    if (Set::check($entry, "{$modelName}.{$uploadDirMethodField}")) {
I've done it several months ago. Check my fork. Also diablo miner is supported in my fork.
You pushed with merge conflicts somehow.  I've removed some of the problems, locally for me, but I don't know if I deleted the right ones.
Can we not push code that doesn't build into develop
What do folks think about the order of arguments for these two functions? I'm torn between this and (n, n0, A, A0).
no, this good. in world - only time (in ms) on execution some linear parts of world::update. mostly 0.
а не карты поломаты, а только некоторые места на некоторых картах. по большей части в аутленде.
yeah this should be happening only after `_step`. /me is puzzled if this is actually necessary.
Ok! Sounds good! ;)

> Hey Sanaz! Ryan and I (team 3) are working on fixing the overview metrics
> to account for deferred stories, and we're planning on pulling this from
> your repo. I hope you don't mind!
>
> Thanks!
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Sanaz/agilefant/commit/338631ce0538b8d393bce4fd9fdf6527c0c0780b#commitcomment-875855
>
Wegen der Vollständigkeit: b66e2c5f5eeed2f21696c5c6d4bb8e0d7b1e8ee9
Na dann schreibs doch davor ;) Du bist der Entwickler ;)
> From my point of view we both want it to be possible for clients to set the language. We both want this but still there is a debate about it.

Huh?

Anyway, good to hear you changed your mind.
yes using on gc
mtmaps work by this method:
- create X threads
- start work on X maps
- wait for one thread to finish map work, start new map in free thread
while has maps for work.
if one map (continent?) work very long time - his be work alone after end work all other maps.
ergo - one map has HUGE number of objects for update. supposed (see below).
Yes, but there are still problems with SD2
if your in linux 1) check TZ value 2) check (or install and setup) ntpd server. only after this restart system and try cleanup instance_rest.
[ 76%] Building CXX object src/game/CMakeFiles/game.dir/IdleMovementGenerator.cpp.o
In file included from /root/mangos/src/game/MovementGenerator.h:28,
                 from /root/mangos/src/game/IdleMovementGenerator.h:22,
                 from /root/mangos/src/game/IdleMovementGenerator.cpp:19:
/root/mangos/src/game/StateMgr.h:37: error: 'UnitActionPtr' has not been declared
/root/mangos/src/game/StateMgr.h:49: error: 'UnitActionPtr' does not name a type
/root/mangos/src/game/StateMgr.h:54: error: 'UnitActionPtr' does not name a type
/root/mangos/src/game/StateMgr.h: In constructor 'ActionInfo::ActionInfo(UnitActionId, int, UnitActionPriority, bool)':
/root/mangos/src/game/StateMgr.h:38: error: class 'ActionInfo' does not have any field named 'action'
/root/mangos/src/game/StateMgr.h: At global scope:
/root/mangos/src/game/StateMgr.h:77: error: 'UnitActionPtr' does not name a type
/root/mangos/src/game/StateMgr.h:87: error: 'UnitActionPtr' has not been declared
/root/mangos/src/game/StateMgr.h:88: error: 'UnitActionPtr' has not been declared
/root/mangos/src/game/StateMgr.h:92: error: 'UnitActionPtr' does not name a type
/root/mangos/src/game/StateMgr.h:95: error: 'UnitActionPtr' does not name a type
/root/mangos/src/game/StateMgr.h:107: error: invalid use of incomplete type 'struct MovementGenerator'
/root/mangos/src/game/MotionMaster.h:27: error: forward declaration of 'struct MovementGenerator'
make[2]: *** [src/game/CMakeFiles/game.dir/IdleMovementGenerator.cpp.o] Error 1
make[1]: *** [src/game/CMakeFiles/game.dir/all] Error 2
make: *** [all] Error 2
That's development/testing environment. We don't want to set somebody's computer for potential attack from remote hosts (even in local network) by default. Can you add it as a test.py command line parameter?
no test? :( There is already one for mobile that you can add to.
Need to write the proper javadoc comments for this.
doh! you stolen my commit!
Not sure here. But empty string have a pointer though it is empty and shouldn't be send or checked.
Bahh it wouldn't let me commit earlier. I think I fixed everything now, sorry about that.
Honestly? I don't know. It might have to do with difference in fixity of operators versus do-notation. I just noticed in my benchmarks that ResourceT was too slow, so I copied the implementation of >>= from ReaderT.
Seems to work.  Will fix the output a bit.
Out of curiosity, why was the hash seed changed from the file content to the file path + modification date? Relying on the date seems less reliable, and using the file path breaks cross-server compatibility.
You confused mysql_real_escape_string with intval()! mysql_real_escape_string only makes sense inside single quotes, to prevent someone breaking out of a string.
Hab die Zeile doch gar nicht angefasst :< ... code fixed, test fixed
should extend to 2012?
Yeah, I just don't want to have a monster commit out there. I had 90% of fluent search implemented, I need to pull the branch out. I think it took about 2 hours.

---
Jeremiah Peschka, Managing Director, Brent Ozar PLF, LLC
Microsoft SQL Server MVP

On Feb 28, 2012, at 6:01 PM, OJ Reeves<reply@reply.github.com> wrote:

> Leave this one with me mate, I'm kinda half way through it. Plus when I'm
> done you can critique. Not sure if it'll be done by the end of the week,
> but it'll be done soon. Fluent search is a must :)
> 
> OJ
> 
> On Wed, Feb 29, 2012 at 10:59 AM, Jeremiah Peschka <
> reply@reply.github.com
>> wrote:
> 
>> If you don't have time, I can pick this up. I think I see your direction
>> with this. If we've got some issues in github, it'll be easier to split
>> workload between us.
>> 
>> ---
>> Reply to this email directly or view it on GitHub:
>> 
>> https://github.com/OJ/CorrugatedIron/commit/d92c39ba3591aae832109ab2fb19c6bb7a43ee09#commitcomment-1023866
>> 
> 
> 
> 
> -- 
> 
> OJ Reeves
> http://buffered.io/
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/OJ/CorrugatedIron/commit/d92c39ba3591aae832109ab2fb19c6bb7a43ee09#commitcomment-1024036
Fixes the next stacktrace

ERROR 08/03/2012 07:44:39 [ClientRequestExecutor]/SelectorManagerWorker

java.lang.NullPointerException
at voldemort.store.socket.clientrequest.ClientRequestExecutor.read(ClientRequestExecutor.java:186)
at voldemort.utils.SelectorManagerWorker.run(SelectorManagerWorker.java:98)
at voldemort.utils.SelectorManager.run(SelectorManager.java:194)
at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
at java.lang.Thread.run(Thread.java:662)
View config is opaque, we do not understand its contents here. Context is the set of information that the hosting page knows
Right. The reason for that is that you committed an .orig file, which I guess is used in KDiff or something?

I'll fix it by reverting to my last commit. We made the same changes by deleting those three classes anyway.
Hi Claude,

Sorry about the slow reply. We're in crunch at work, I'll have another look at it when I have time.

Cheers,
Christian.
Do you mind if I merge in each commit as you do it (You won't need a PR, I'll use the fork queue)?
I see your comment but I still don't understand what `e("\0/")` is supposed to do. This is causing the following line to be generated in the static CSS files:

    input[type="image"],input[type="checkbox"],input[type="radio"]{width:auto;height:auto;padding:0;margin:3px 0;*margin-top:0;line-height:normal;border:0;cursor:pointer;border-radius:0 \0/;}

As far as I can tell, this is not valid CSS.
а почему не делать перед завершением работы плагина header()->saveState()? Надобность в цикле и прочих танцах сразу отпадет. См. https://github.com/negativ/wismut/blob/master/src/MainWindow_p.hpp#L64 , SSET -> https://github.com/negativ/wismut/blob/master/src/Settings.hpp#L59
DERNIER COMMIT - J'ai envoyé - Plus de modifs please !!
``isAutoUpdated`` would probably be a better name
The type attribute is completely unrestricted, so try to name it as accurately as possible. It's tattoos, so I'd call it Tattoos
bygg på =O)
ok .. makes sense.
It isn't clear what is checked. 

I suggest a more explicit name for this method:
 checkDirectory
 assertReadableDirectory

As a bonus, you could have it return the directory and white:

```java
this.basedir = checkDirectory(basedir, "basedir");
```

which is short and clear.
This was me.
While this is something that I was looking forward to having, when the Client API is released, I think that it should be removed so that a plugin could add it.
Unrelated to this commit, but I managed to get SC68 updated to the latest revision. It should be a good thing, since as stated here by the main dev: http://sc68.svn.sourceforge.net/viewvc/sc68?view=revision&revision=202, it solves some memory leaks and memory corruption issues with the library. The whole commit list is here if you consider using it:

https://github.com/lioncash/droidsound/commits/

also consider this commit too: https://github.com/lioncash/droidsound/commit/36c61f23301a807a4c64c562d33b2fbe0864e1d0, since it fixes issues according to the main devs: http://modplug-xmms.git.sourceforge.net/git/gitweb.cgi?p=modplug-xmms/modplug-xmms;a=commit;h=dd4b6fe3e6984cb441f03758c1ac9bb92bc7608a
Yes, agreed. We should also cater to newcomers and explain what cycle does.
Yes, I was considering strongly to just do explicit imports in songs for teaching purposes (both on bl and best practices)
Sounds good, I have tried, but there is a toplevel widget catching the events, let me see
I removed this on my branch; I don't think we'll need it once we merge the UI stuff.
ship it! *hurr durr*
What about moving this to Libmacgpg?
Did you mean to check DMBot in with the rest of this?
噗~
this comments on certain lines of code is quite cool. similar to what I said that other time at lunch with thinking of using Google Wave inside code IDEs.
great stuff...

I tried building a test build now and I don't get any OTA package created... Now reverting commits to see what it broke :(
Ich hab dir was auf arkania gepostet...
I see that you have fixed few problems with this gem. If you will ask me for a pull then I can update it https://rubygems.org/gems/flickr.rb
You don't use this variable.
Fair enough. This is just asking for proper cmd-line args parsing :)
tiptop :D
Good to know :)
I'll think about a better explanation and maybe an example.
Should I make this a paragraph in the narrative? I don't like having so to much detail in the docstring.
Changing from an Integer to String causes Postgres to blow up!  PG::Error: ERROR:  operator does not exist: character varying = integer
Hmm...this one is pretty baffling. I'm not sure what could be missing since bundler takes care of dependency issues and you do have your parse_resource.yml file. 

Are you sure you're pulling from 1.7.1? Just running git pull will pull the latest from the master branch, which is usually a few commits ahead of the latest gem-released version.
This expects the following DB updates:

delete from spell_script_target where entry=34874;
insert into spell_script_target values
(34874, 1, 17990);

insert into spell_scripts (id, delay, command, datalong, datalong2, comments) values
(34874, 0, 18, 0, 0, 'Despawn Underbog Mushroom');
```C++
void Analyzer::sendServerInfo(const QString &newName, quint16 newMaxNumber, const QString &newDescription) 
{
    QByteArray tosend;
    DataStream out(&tosend, QIODevice::WriteOnly);
    Flags network (!newName.empty() + (newMaxNumber != -1) << 1 + (!newDescription.empty() << 2));
    out << uchar(NetworkServ::ServerInfoChanged) << network;
    
    if(network[0]) {
        out << newName;
    }
    if(network[1]) {
        out << newMaxNumber;
    }
    if(network[2]) {
        out << newDescription;
    }
    
    emit sendCommand(tosend);
}
```
I just woke up! Don't kill me :}
should probably have a utility function to do that instead
To avoid the unnecessary double-select, how about extending `er/mark-defun` to tweak the white space quirks to match er/mark-outside-pairs?
sha512 and filesize missing here as well.
i'd recommend not to update this pkg as the older one has been well tested
Why this extra sleep? and wouldn't it be better to send 1 request (before the for loop), and wait until the reply arrived (just sleep)
> Did you test the triple === with 'confirm'? Why take the highest eq check?

I think in this case it doesn't matter, it is only irritating why === with confirm and == only with 
pending..
but that isn't a blizzlike addition.. is what we are getting at... 
they don't depend on a player type bot to balance the economy,
that's more for use on your own setup, 
when you don't have enough players to do the balancing.
Think the variable passed in hash should be `force_default_config` since `force_default` doesn't exist yet?
Sounds reasonable. Thinking a lit further, the list of possible annotations is closed anyway, since is implemented in clojure-maven, so no need to ensure the user can pass arbitrary classes.

Validation would help catch errors.
Generically speaking, you can have a custom spec file which packs a subset of Python files into a PYZ file, and then drop it on the disk somewhere. If you add the PYZ file to sys.path, PYZOwner will parse it and make the modules within it available at import time.

It's not the most used feature, but I don't see a reason to drop it.
Oh, man! :) Thank you, you're doing very well..
I think because it is a stack you only need to have a tag on the head node.
Shouldn't you be passing allocatedSize to resizeRow (and deleting this local var) ?
When did the privileges table get commented out? And why? @tilucsi @SgtRaccoon
makes release noop
It also works with things like `This commit closes/fixes/refs #1234`.
Shouldn't it at least return `$text`?
This probably merits us looking at the implementations in the welocally-places.php and unifying the implementation to be a smart way to do the admin-ajax with curl. More componentized and probably a separate class. https://github.com/welocally/welocally-places/blob/master/welocally-places.php#L18
This modifies passed dictionary which may be unexpected. My version does not.
Why wouldn't you turn this into a count call to the database?  I guess you aren't streaming data yet so it's not like you are counting client-side, but isn't there some overhead here?
Hmmm... I don't think datastore['VERBOSE'] returns a string, it should return a boolean value.  Are you sure?

Also, if datastore['VERBOSE'] is false, that means your verbose variable will not be initialized.  You might get an error like "undefined local variable or method `verbose'" when it reaches:  if (verbose == "true").
Looks fine to me.
I left remarks for some minor issues regarding the code quality and SCM best practices.
rofl^^
cool thanks!!
That's a really good point - maybe we should open it up to more media types? I think we definitely need something to give us clear idea of what the project is, but if that's audio or video then uploading that should be allowed.

oh, and news on the domain. I'm emailing back and forth with the guys who have the old domain... but I'm thinking that I'll get a signed bit if paper from Martin tomorrow and go to Valentin at CDA and get us a concordia url, that way we can have this thing up during or after class.
bra mellanklass
a) no creation date and no author tag; both parts are covered by the scm; 2)  I don't think this class is required at all (see above)
Would be great to see these initialisation actions grouped into functions like in initAll() in cgmp.admin.js:336. This would more clearly express the steps taken to get the plugin started.
these formats are declared in audio.h in system_core
you know those that are needed to be removed for the mic to work in old audio drivers..
new drivers are still no go
ascii, not asscii.
削除するの忘れてますね
Yes, I can submit a new pull request if needed.
This closes over outer.rootGuardianStrategy is this intentional?
oops, got it wrong
kitten died here
so if someone does:

    context become Actor.emptyBehaivor

they mean 

    context become receive

???
… exception
Use @tailrec and write it as a recursive lookup instead, then no need for return.
Very true, let's do that!
ah, no, we didn’t; forget it.
Is there a standard error for abstract classes? This is an abstract class that's subclassed by QballModel and OpdtModel.
"del search_params['filters']" necessary as SOLR cannot handle unknown parameters.
I would update the lastGenerated before sending the value in the wild by calling the callback...should not happen but you want the internal state updated before the sequence is 'published'
it's cached by platinum. I'd be much happier if this sort of thing was handled by the VideoDb. Is it a bad an idea to call GetStreamDetails during a directory fetch?
But only for connections that are lost. Not for connections that are requested by execute/callproc/chain/batch.

I'll do some research about keeping connections alive. Sockets can time out and if that's prevented there's no need for a complicated reconnection mechanism.
Space above, please.
That's a good question to answer. Probably near where you pass it the schemas to generate?
Thanks for adhering to the coding style! I appreciate it!
The number of `=` must be equals to the number of letters for the title.
Add a HandleScope.
You are missing a `the` in front of `Mcrypt and maybe we can use this notice to link to the PHP page:

> [!!] You have to enable the [Mcrypt extension](http://www.php.net/mcrypt) before using this class.
This is done for you in mappers.js - Abstract is the only one not done automatically.
can you clean this up ?
instead of asserting, I think a call to the callback with an error would be handy. Thoughts?
I prefer to check if `"mimeType" in opts`, and then to `delete options[method]` if `value == null` on set. (The double-equals catches both null and undefined.)
I released a 1.0.2 version of the maven plugin to Sonatype's OSS repo which fixes this, should be in central shortly.
You should add `$pdo`.
это у меня было так. при сборке. warning потому что не юзается
Probably we should make this work more generally? E.g. I might want to do `Post.joins(:comments).where(comments: { author: david })`. So I think there needs to be one level of recursion here.
doesn't this line shadow the `from` in the argument list as the very first thing, making the argument dead code?
I didn't get.

```ruby
self.attribute_aliases.merge!(new_name.to_s => old_name.to_s)
```

is equal to

```ruby
self.attribute_aliases = attribute_aliases.merge new_name.to_s => old_name.to_s
```
In that case authors is a symbol, in this case string. I think is worth to test if it works.
This is now fixed by replacing the `CheckCPSMethodsTraverser`
Sure you can, but you get the monomorphic Nothing => Nothing:
```
scala> def foo[T](t: T) = t
foo: [T](t: T)T

scala> foo _
res0: Nothing => Nothing = <function1>
```
I'm not certain about it, but as long as we have the polymorphic type for the free symbol, we might actually get away with capturing the monomorphic function and applying that. We'd have to see.

So let me get this straight -- Paul, you propose that we don't do eta expansion at all? Do you have a better alternative?
Ah, never mind, I see that f is func(x)
I generally like to see a space before the `>>>`, otherwise it looks too busy.
I think that this is my fault. I used the same style of `try except` blocks for the tests that are already merged.
`return is_real and is_number`
_Really_ minor nitpick: Change "presence" to "present."
quote marks are weird.
this looks like an editing/editor gaffe, but can you please revert the changes in numbering here, as these changes aren't ideal.
will this break other test cases?
replace with "return t.eachInOrer(..."
Although, looking at this, this is the mock class, so not only does it not really matter, but also I simply copy/pasted directly from the Symfony ContainerInterface: https://github.com/symfony/DependencyInjection/blob/master/ContainerInterface.php#L136

So IMO since it's directly from the other library, I shouldn't change it here. \*shrug\*
If this function can be called at any time cache is cleared, you have no idea what $phpbb_root_path is set to and you absolutely cannot do this.
Also `$result` doesn't exist at that point. It's just created for adding the data in `array_merge()` 2 lines below.
I have already removed the entire line?
These properties should gain docblocks.
Don't use count as a variable name. It might be a reserved word.
Is this an integer? Cast to integer. Otherwise you can also use sql_fetchrowset and get rid of the loop
fixture to file.
ăsta nu merge. nu te poți baza pe request.user când diferă portul. trebuie scos altcumva.

(încearcă local să ai UI-ul la localhost:8000, iar socketio la localhost:8888 - o să vezi că îți zice că nu ești autentificat, pentru că nu se autentifică pe toate porturile)
fără cod comentat, te rog
I'm wondering. Could we just let these bee handled in the catch-all else clause since `#inspect` will be called there?
``&amp;`` instead of &
Change from float to int. This could also be made be aware of subcells because infantry is dropped.
I'm not sure what to do about the login for this. I am looking to modify the current logic, but would be willing to look at something else if that works better
fix tabbing. also, should be "else if" , not two separate "else" and "if" statements.

```
else if (enteredDate > currentDate) {
// ...
}
```
this seems like a hack-y solution.Why do we need to add +1 to the month (but not to day or year)? will this work consistently?
Thanks, it's an ongoing battle to make Theano code easier to get into... I think things like this help.
In general should we be quoting paths, just in case there are imbedded blanks? There are a few places in this script that would fail for such paths.
use cls not self for classmethods
&mdash; would look better
Why not an empty list?
what was the observed improvement spawning directly rather than using the sup?
Use a stack-allocated array with CF_BUFSIZE to avoid the malloc overhead.
Why you need these columns as private members?
Constraints and indexes really aren't the same things, does it really make sense to have them in the same method?
Although semantically the same, I'd rather return `kCFBooleanTrue` and `kCFBooleanFalse` here, just to make more explicit the problem that's being solved.
Sure, I'll to validations
Thats done
Tests start clean so there shouldn't be a need for a reset here.
It's ok in a .cc file. Troublemaker was YIconLoader.h.
The using decl in the .cc files just became obsolete after I ran a bunch of sed commands to fix the sources.
Please remove the duplicate copy
@andy10gen ?
OK, no delete option. That answers my own question above.
Because it was blindly copy/pasted from base.html, which needs fixed too.

When base.html was made originally, there wasn't a better place to link to. There should have been a TODO comment.
This change doesn't make sense to me. Doesn't the `should_follow` method do filtering? So if a request is marked `dont_filter` why should it still also have to pass `should_follow`?

Also, it seems like this change will require adding `dont_filter` to every single request, which seems wrong.
Sorry, just being bitten by one of the side effects of js in the page, so it's a good opportunity to share the idea.

One of the main issues is that if you load your dependencies (jQuery and so on) from an external file, there is no guarantee that the functions defined in the external files will be available at the time when your inline script will be executed. I'm having problems right now in webistrano, because if I try to organize the files in a different way (for example putting jQuery in the default jammit package) jQuery is not available to many of these inline scripts, so they will all fail.
So the Ushahidi PHP code does:

```php
$message_from = preg_replace("#[^0-9]#", "", $message_from)
```

Which I guess means we should do something similar.
muessen mal schauen wo das überall verwendet wird. aber macht denke ich sinn.
That's legit, I think, if we let contributions cross children's lines.  Remember... children aren't children... extension graphs are teh more complete combinations of a common graph and another graph.   The only reason it seems weird is because you're thinking in OO inheritance terms because of the langage.  Graph A has a, b, c in set keyed with S.  But Graph A + B having a, b, c, and d keyed with S is entirely plausible and makes total sense... divorced from the parent/child metaphor.
Because scope() is used as a pass-through int he original code for bindings when they're linkRequested, which a singleton binding will be.
Crap.  This was WIP.
I don't see a reason to repeat the Javadoc of `TestWatcher` here. Can we remove it?
If we are using a lock every time we access fListeners, then why can't fListeners just be a `LinkedList`?
I think the canonical Java infinite loop is

while (true) {}

I think if we name the method infiniteLoop(), and use that style, we can drop the comment-within-a-javadoc below.
These seem very tall. Let's try scaling it down to maybe... 48px?
`return description || '';`
Don't we need to translate this? I'm just curious if that was intentionally removed.
Doh... I was not thinking correctly when adding those new lines. No wonder when I tried with two new lines and I didn't see that in my testing.
Nit: chaulk -> chalk
Sorry, I missed that. Keep it like it is.
Need to use 4 spaces instead of Tabs.
The negatives here hurt my brain, why not ```raise if should_raise_on_exception?(e) ```
It would be nice to know the git hash that is equivlent to svn 3229 so that the CAUTION can be updated.
Can probably remove this now the `Multiple` backend has it's own `each` method.
Yeah this is something that was added to bootstrap popover, which lets you
delegate a popover. Sort of like $(document).on('click', '#myElement',
...). It makes it so that when things change, the events that were set up
still work and weren't bound to a particular node that is now gone.

I tried it a while back and it didn't work (and had been broken for a
while) so I just commented it out.


2012/12/12 Biyeun <notifications@github.com>

> In corehq/apps/app_manager/templates/app_manager/releases.html:
>
> > -                            "Once you've fully tested an application, you can star it to mark it as released. " +
> > -                                    "When a CommCare application searches for updates, only starred builds will be considered."
> > -                            )
> > -                });
> > -                row.removeClass('build-released');
> > -                row.addClass('build-unreleased');
> > -            }
> > -            var LATEST = 'build-latest-release';
> > -            $('.' + LATEST).removeClass(LATEST);
> > -            $('.build-is-released [data-is-released="true"]').first().each(function () {
> > -                var row = $(this).parents('tr');
> > -                row.addClass(LATEST);
> > +        var el = $('#releases-table');
> > +        ko.applyBindings(new ReleasesMain(o), el.get(0));
> > +        el.show();
> > +        {# This currently doesn't work because of a bootstrap update, #}
>
> This seems like it shouldn't work? the intention of popover is to apply it
> to a specific element...not the entire document?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dimagi/core-hq/pull/373/files#r2391367>.
>
>
is this distinction needed ? these two bodies look exactly the same
2 spaces left
We use the hosted libs in the other test files.
Do you want 2-space indenting everywhere? There are a few other places where I did aligned indenting.
It sounds like if you have a boolean property that matches the name of that element (Form in this case), Caliburn.Micro will automatically bind it to the visibility. Haven't tried it, but that should clean things up, if so.
"Realizations of this interfaces are intended to manage streams, i.e. to retrieve data that can be retrieved from distant servers. This will be useful to manage some OGC standards like WMS or WFS."
I agree with @ebocher
Not good. `escape` (HTML escape) is already applied automatically in Django forms (unless you turn it off). You have to URL escape, not HTML escape. What if `next_url` contains `http://foobar.com/;delete_everything=yes`?
Now I will ask you to think a bit before ask stupid questions. So please think what code do. And what are you doing.
lessify please:
`@green`
I don't mind changing it, but why is a relative url preferable here?
Message Please!
you probably want to uriencode the title and other strings, so you don't get into XSS trouble.
This needs an explanatory comment.  From above it looks like shrink and shrink-alt are the same, so why this dance?
Can you add `cp('-R', 'cornfield', DIST_DIR);` as well?
There's already a "Additional resources" section further down, with the same link
'secore' should be 'secure'
改成if
简化成rc == NGX_AGAIN就可以了。
don't remove the comment split by the buffer ?
This should probably also be configurable and have a more sensible default
What are these changes for? Can they be part of a separate pull request?
I see!

This is a nice trick ... However it might be dangerous in real life code.
People might want to remove the extra parenthesis like me and it would
break the code :x
-- 
Christopher CHEDEAU - Vjeux
EPITA - LRDE - Ing3 2012
Curse.com Web Developer

http://blog.vjeux.com/


On Tue, Apr 10, 2012 at 4:31 PM, Irakli Gozalishvili <
reply@reply.github.com
> wrote:

> > @@ -23,11 +23,11 @@ A protocol is a named set of function signatures:
> >  ```js
> >  var protocol = require('protocol/core').protocol
> >
> > -var Sequential = protocol(('Logical list abstraction', {
>
> That's in fact intentional trick for adding documentation :D
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Gozala/protocol/pull/1/files#r668515
>
Have consistent names for the variables. DirPath or DirectoryPath- only one of them
Yes I fixed that mistake and gave it back.
According to rails docs, no: http://guides.rubyonrails.org/active_record_querying.html#passing-in-arguments

However, it should use self. and not User.
why would you do this to homer
Similar to Array above, `var notice_obj = {};`
discussed in the hallway and going to change the commands to   job_history and  job_tasks
Should this be updated to AuthorizationSupportMethods?
Can you put this is resource string or something.