labofsahil/github-event-dataset-2012 huggingface dataset:
Adding basic wifi connectivity (Using networkmanager 0.7)

The following changes add wifi wpa connectivity using a settings hash generator. 
I hope this proves useful to you in your work on this gem. Unfortunately my company is using machines that only have NetworkManager 0.7 currently, so I had to alter some of your object path namespaces for the older API.

Special note, figuring out the DBus byte array for SSID's was a particularly fun exercise. See the notes in networkmanager/settings_hash for some info about this.

I hope you find at least some of this helpful with your progress on this gem. Your work has helped me immensely. Thanks for all your hard work on this.

Any questions please contact me at steven.steffen@ownpointofsale.com

Over the next couple days I will be working on specs for this.
Added Armor and Weapon.h's
Decision tree graphviz export fix

I ended up with a perfectly valid DT that would fail when I tried to call the export_graphviz function.

There was originally an assertion statement "assert node_id != -1" which was changed to a more readable "if node_id == 1: raise Error" statement. It seems that the minus sign was left out in the update. Oops!
added link to official fork

just sending newcomers to the slightly more active fork.
incorrect class name

Good morning!

A quick fix for #1052 [1]

Cheers!

[1] - http://bugs.web2project.net/view.php?id=1052
made updates for polymorphic photo sharing
Update components/com_content/models/articles.php

contact.id is referring to com contacts that has been deleted
Modified verson of pikachoose 4.4

Pikachoose version 4.4 has been modified so it no longer contains jcarousel.
Jcarousel has been moved to its own project (collective.js.jcarousel) and is set as a dependency.
The "original" js is still included in the product, renamed to jquery.pikachoose.original.js, although I can not see any reason for using this instead of the new js file and jcarousel "combined".
Headers new visuals

### Features
This PR implements the all new visual guidelines from UX in the component "header".
It also have some new themes and a refactorized way for managing them.

### Requires
This PR has dependency with:

* New folder structure: https://github.com/mozilla-b2g/Gaia-UI-Building-Blocks/pull/25
* Menus and Dialogs: https://github.com/mozilla-b2g/Gaia-UI-Building-Blocks/pull/26

Please @RudyLu @timdream and @jmcanterafonseca could you review it?

Thx a lot!
Sheng Huang's work done

seas_id: shehuang
Issue 7321 - returning void considered unsafe by safety inference

http://d.puremagic.com/issues/show_bug.cgi?id=7321

This is a shortfall of fixing bug 6902.
More fixes on ScriptExecuter

This commit fixes:
- Buffered output is no longer produced by background method. Instead, we print each char received from the forked process. This way we can run processes with small bits of feedback (such as wget)
- Background method ends with an actual (Integer) return code, this way the end callback is called so we can print the proper return value
Added a PHP class to ban Varnish ripped from https://github.com/timwhitlock/php-varnish
sumo sponsorship

should be ok, but haven't been able to preview locally yet
Rakep build enhancements

Here are a couple of small enhancements to `rakep build`. I added a `--clean` option to run a clean build (equivalent to `rakep clean && rakep build`) (closes #34) and made `rakep build` print out the files it's creating (closes #32).
Sample project for SPR-9610

Webapp fails to start up.   The workaround is
commented out.  Fix should be in FrameworkServlet
somewhere probably.
Fix requirement

Requiring explicitly branches is not such a great idea for reusable libs like this because it means users are forced to use that, and if there was a 2.1 stable they couldn't even use it. For now this will still install the master branch since it has a 2.1.x-dev alias.
Build fix for OSX 10.6 and Ubuntu 11.10

Added a find_package for OpenGL and the necessary target_link_libraries
to get this framework to build on OSX 10.6. Not sure if the "required"
find_package will negatively impact linux/windows.

Added support for Ubuntu 11.10 (GNU G++ 4.4 compiler). The default
gcc-4.5 compiler is not supported by CUDA, so users specify the 4.4
alternative when first running cmake: "CC=gcc-4.4 CXX=g++-4.4 cmake".
The change of compiler is applied by the set(CUDA_NVCC_FLAGS ...).
Modify workflows and permissions to be consistent with the ones used in Plone

Permissions and workflow modifications in order to have a similar way of dealing with polls as we deal with other Plone contents
Update a comment in lock_timeout.rb
Mostly documentation fixes and silencing obs-api cron job
Restuctured2
Fix for BBB-92

-Changed from processing by locale goups to processing ony by one
-Now based on userTimeZone instead of defaultTimeZone
Signed-off-by: jfederico <jesus@123it.ca>
Remove typo in the styles for rank 9 that created a white text on white ...

...There is a superfluous '17' error in the styles.css that ends up with a white text on white background display
Fragment class.
Upload resumption

Implemented automatic upload resumption.

Now `$ glacier-cmd upload <vault> <archive> --resume` will check whether there is such an upload in progress (kept in the bookkeeping db) by checking file name and file size to get the uploadid. After that it works the same as if `--uploadid <uploadid>` were given.

Changed bookkeeping db to use the archiveid (or uploadid) as key, instead of the file name.

Inventory bookkeeping code is wrong; needs to be rewritten. Disabled here.

Added `updatedb` command for people upgrading from previous versions. This updates the db to use the new keys, and purges any items it can not recognise (i.e. no archiveid/uploadid key present).
Java Buildpack Unit Tests

Unit tests for all scripts in the Java buildpack including tests for special M2 caching logic.
Headset Fix from ratje

This is the Headset fix from ratje
Develop

added example of ofFile/ofBuffer to load .csv file

added example of ofFileDialogResult for opening/saving files
Changes to AdamInternetusage.xml

Updated the xml from a representative AdamEzyChoice usage feed.
Replace printf by ofLog in /libs/OpenFrameworks

I replaced printf occurences by equivalent ofLog. The changes are quite simple, but please review before merging, I can't even test most of those changes.
Some occurences still remain, please advise how to proceed:

ofSystemUtils.cpp contains MacOS-only printfs (which I have no access to), and it's not clear to me which log levels would be appropriate for those.

ofTrueTypeFont and ofThread contain conditional printfs with bools `printVectorInfo` and `verbose`, respectively. This is obviously to manually enable logging, but if we replace the printfs with ofLog, it would be possible to not get logging output, even if those bools are true, depending on the log level.
So how should we treat that? 
`OF_LOG_FATAL_ERROR` would ensure output if the bools are true, but sounds really too drastic for most of that output.
OTOH, leaving the printfs would ensure appropriate output, but it couldn't be managed/redirected like all other logging output, so that's not very consistent.

Aside of the occurences in ofSystemUtils.cpp, ofTrueTypeFont and ofThread, there remain no relevant, non-commented, logging printfs in `libs/openFrameworks`.
Please discuss, and I'll amend this PR.
Relevant issues are #833 and #294.
Service writer bg
Korean translations : Profiles - frameworks part

good job
Zoo Visitor members categories bind to Mailchimp groups and merge vars

Hi Stephen,

I had to improve a bit my implementation of Zoo Visitor integration with your Mailchimp Subscribe addon.
Now it supports categories in the zoo visitor channel, and bind those categories with the mailchimp groups and merge fields.

I hope the code is not too messy and I will be glade if you could have a look at it.

Best regards,

Pv
Fix for ampersand bug (and some commented out work in progress towards fixing others)

...mpliance). But we haven't set any expectations for this, no tests are specific on the matter.
Added detection for shows with air date only in file name
LPS-25228
Readme fix and allow tinymce theme choice from javascript helper

* reformatted README
* added second param option to tinymce_managed_javascript_tag to allow using the simple theme

If this request is ok, I have also modified it for the rails3 branch
Teacher class reg tests

This branch is the addition of several tests for the Teacher Class Reg Module as well as a couple minor changes to the module itself.
Fixed string-comparison bug in service runtime.
JBRULES-3153 ChangeSet loading will take 5 minutes without an internet connection

I upgraded the KnowledgeAgent to use Options for configuration in addition to strings
Glpi0.83

Upgrade deploy to glpi 0.83
French translation and little change to JS to permit it usage

Hello, 

I propose the french translation with a litlle change in JS code witch integrate translated string to permit usage of simple quote character.

Thank's for this plugin.

BR
Plugin

Hi Boone :) 

So in general what I did:
- made a plugin based on your code
- table is create while activation or if it doesn't exist
- BuddyPress abstraction (so in case no BP on a site - 2 its functions used in a plugin won't break anything - I hope)
- admin area with a list of items from the table (pagination, sorting, bulk deletion)

I hope, it will be useful for you :)

Would be nice to publish it to WP repo (imho).
SN Theme Generator

Compiler allows more effective create any CSS file for phpBB Social Network.
Allow to use many things that are allowed using [lessphp](http://leafo.net/lessphp/docs/).

Otherwise we use now this compiler. But is included into phpBB folder.
Here is out of root folder of phpBB.
Coup de pinceau sur le blog 

Modifs que je t'ai montrées vendredi.
Assez gros changement tout de même au niveau de la structure, donc si application lookée chez nos clients il faudra vérifier son rendu.
Улучшенная предзагрузка изображений

Единственный минус - теперь не поддерживается распознавание раржипегов в лисах < 18, хотя это можно исправить. Впрочем, теперь любая фича, использующая воркеры не будет работать в Firefox'ах < 18.
Distribute Update to Use couchdb _replicator


-Update distribute to use couchdb _replicator database.  Now we will have a document in _replicator database for all the replications attempts.

-Update the distribute  tests to take account the use the couchdb _replicator database.

-Add one more test file lr/tests/functional/distribute/large_network_test.py to test distribute on a network of 15 nodes in three different communities and five different networks to test distribute on nodes with more than one connections across different networks and communities.
Added requisites and installation info... and one request

Hi John,

I install your plugin and view that I need contextmenu plugin to work. I put it in the README file.

I add a @todo request :)

How to enable multiple messages selection?

Thanks
Use LogMessageF to add some semantic information

I replaced the `LogMessage` call with a `LogMessageF` call to add some semantic information which aids me to find where I fired my log messages. I hope you can merge and tag it, so I can update the CocoaPods spec and use it.

![Screenshot](https://dl.dropbox.com/u/4986/nslogger.jpg)
New name addition
Fix japanese langfile

Change Japanese lang file(settings.php).
fix javascript syntax error for IE 7
prevent fatal errors when calling event_calendar lib functions

Without this I would get: PHP Fatal error:  Call to undefined function event_calendar_prepare_edit_form_vars() in /var/www/Elgg/mod/sched_conf/models/model.php on line 38
Important bmlutils fix, small recovery fixes

Sorry for the trouble - this should restore proper flashing for your tass, gio etc. builds. This will also fix some issues with recovery.
Merge johnnyg changes

They are better than mine
Configuration for new router initialization method

This change just moves router initialization parameters to instance configuration of the router itself within di config.

PR is strictly related with https://github.com/zendframework/zf2/pull/768
I wanted a simpler starting template
Fallback to local copies of JS if CDN is down.
Will use apples defaults of regenerate_interval is not set in config.yml
You don't want this line in here.
Why not just use __('locale.key.here') instead of TemplateManager::smartyTranslate(...)? If there's a reason for it, it should be documented here.
Dê preferência a `validates :nome, :presence => true, :uniqueness => true`.
Hi, thanks for looking into this, I'd love to support more `search_type`s.

The thing is, the proposed interface seems less then perfect to me. I think we should expose [all the URI parameters](http://www.elasticsearch.org/guide/reference/api/search/request-body.html), namely `search_type` and `timeout`.

So, something like this:

```ruby
s = Search::Search.new('index', search_type: 'scan', timeout: 1) do
  query { string 'title:foo' }
end
```

would make more sense to me. What do you think?

P.S.

Ultimately, I'd aim for something like:

```ruby
all = Tire.count 'articles'
```

and

```ruby
ruby = Tire.count('articles') { query { terms :tags, ['ruby'] } }
```

And of course a first-class DSL support for the scan/scroll searches.
1.9.2 causes segfaults in our build. Please see earlier builds on Travis for evidence for this. 1.9.3 is just as stable and so we would recommend using that.



On 04/01/2012, at 21:44, Tomasz Stachewicz<reply@reply.github.com> wrote:

> wait, why? 1.9.2 is considered stable ಠ_ಠ
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/spree/spree/commit/befede68594e6ca9358416262b0e800a4b1ce160#commitcomment-836445
Could this be an ordered dependency, then?

    help targets: | $(lastword $(MAKEFILE_LIST))

Note the `|` denoting an ordered dependency.
@WalkerCrouse That kinda of describes most of their actions lately...
Luckily they reverted the breaking Plugin change: https://github.com/Bukkit/Bukkit/commit/a628d2faff4da26c7605afb78e7dcf7e32a3900a
another line with trailing whitespace as well.
Slett kode.
s/subest/subset/ ?
Isn't it a waste to allot memory for a new list everytime? And wouldn't it be preferable to create your own list on the plugin side only if you needed it?
Holy cow, you ruined my one-liner! :)

I can deal with it, but why removing `nested.levels.offset=0`?
why do we need this var? can't we call this on line 345?
maybe allen can make sure that we enforce login on all internal /dev/ pages?
I think this name of the method should be more accurate or provided with parameter to let user know more.
why eps? this inserts "e$" into the trie... this is not the same as "$"... epsilon is not a character
Random thought: If you include -Ofast here (http://gcc.gnu.org/onlinedocs/gcc/Optimize-Options.html), the resulting program should be even speedier. The equivalent switch in Visual Studio has a big impact since the algorithm is not dependent on correct floating point calculations.
Looks good! Thanks Joe.
yes, a big one.

this one is for @bjorn.
It was a misunderstanding, I think just replace with `return false`  will be fine, so What can i do? close this issue and send pull request again?
This basically means toggle() gets called every time the user does something related to a setting that's saved. So for example, _every single time they click on a tree item_.
For me, too.
You might want to look at the chr_get_inventory function , too:
https://github.com/bjorn/manaserv/blob/b58a7890f5660588110d05ac79c3277f26614ebe/src/scripting/lua.cpp#L369
Chrome sometimes appends a newline to innerText but not to textContent; so textContent is better in Chrome
What version of Ruby are you using? And are you using any gems that monkey patch net/http? Because the request method should always return a HTTPResponse.

I do remember a problem I had myself with request not returning the response. I can't recall the details but I don't think I solved it with capturing the response in the block.
Thanks, will merge!
this probably bug some spells of ICC?
Краш-то и правда возможен (как только двое подтвердили - так и глянул). На онлайне более 2к я думаю и количестве нитей от 4 и выше. И в ТС его и правда исправили. Автогеном, нахрен и вместе со всеми выступающими частями. Оригинальная правка...
You think that data from dbc can be not blizzlike? =)
In any case, table in DB stay alive...
I agree, in Emacs if I press `M-TAB` it breaks up a line into lines with a max of 72 characters (and correctly indents stuff).

Not sure if that's easy to do in whatever program you're using, but I like breaking things up too. Don't worry about it now if it's too much trouble.
mallen kan ses live på http://spontus.se/ladok.html
let me commit myself in future, didnt finish testing yet
-- item 60211
In the case of a julia function, doesn't this exec return and need to be followed by `exit(0)`?
hm...ich bau mal nen check rein ob das property überhaupt existiert;)
and .cxx, .cc, .c++, .tpp, .txx, .C, etc.. Have a look at FileLister::acceptFile
Formatting issue:

```
if (History.enabled) {
```
Why only delayed spell? For example Gouge should cause damage, even when the target is immune to it's stun effect. Shouldn't it?
There is some bug regardling skills that not do damage on impact, this skills are nor getting the caster into combat, even the target don´t get in combat.

Is even worse because dots like unstable affliction or corruption don´t get you in combat because they dont do damage on impact like haunt. 

All fears, Kidney shot, Blind etc etc.


related to this patch?
I couldn't find any things that called the above code, and in testing couldn't see anything that broke by removing it, and send email to @childsb to see if he saw a problem removing it, and no response.

If I left the above code in during the AMD conversion, the conversion process would have been considerably more complex because there would have been a mixture of class-oriented code along with static-oriented code. Because the class-oriented code wasn't being used, then the result after AMD conversion was cleaner.
Why is it not better to sanitize the method name and always use class_eval? That will surely look cleaner, and besides, the method name with invalid characters which is created with define_method is available only through #send.
I added :foreign_key => "parent_id" to spec/app/models/game.rb in my working directory, and all specs still pass (I'm talking about this patch: https://gist.github.com/1594009) . What else should I test?
test coverage was missed but it would've helped, feel free to file a bug unless we're changing this template dramatically...your call in any case
I'm confused: Why are you talking about memory allocation and optimisation and pessimization? Do you really think the runtime cost of any of this is at all relevant?

To me, this new code is far easier to understand, and that makes it better, unless you see a serious problem with performance.
I wonder if it would be simpler to just look in <GamePath> and <GamePath>/data for the CD dirs, rather trying to come up with a heuristic to handle baldur.ini here?
```window.showModalDialog``` is not IE-specific, so not sure it makes sense to guard this as such. There's a few other problems with the spec though: 

* The HTML backing this spec is using ```document.all```, which doesn't work in Firefox. We should strive to write cross-browser compatible HTML for the specs (unless we're testing something very browser-specific).
* ```#hwnd``` is a Windows thing, and not something watir-webdriver will ever support. If you want this as part of the Watir gem's API, it should be in that implementation's own specs, not watirspec IMHO.

Is there a technical reason we don't just expose modal dialogs through the ```Browser#windows``` API? @jarmo added support for that to the Watir gem recently, and I think having just one API for all window handling would keep things a lot simpler from the user perspective.

I also don't quite get how ```browser.modal_dialog(:title, 'Modal Dialog')``` can work when the dialog in question has title "Forms with input elements".
float xOffset = (player->GetTransOffsetX() * cos(GetOrientation()) - player->GetTransOffsetY() * sin(GetOrientation()));
                float yOffset = (player->GetTransOffsetY() * cos(GetOrientation()) + player->GetTransOffsetX() * sin(GetOrientation()));
                player->SetPosition(GetPositionX() + xOffset, GetPositionY() + yOffset, GetPositionZ() + player->GetTransOffsetZ(), GetOrientation() + player->GetTransOffsetO());

is more precise
Awesome. Thanks for being willing to iterate on this.
done, I did a pull request, I just didn't know if changing the file extension would be too big of a change for SpecFlow, since there are books, blogs, and screencasts that already reference the .feature files...  I guess we'll see if the change is accpeted :)
That sounds pretty messed up. I wrote a test to simulate it and it looks like json.load() (or python's Unicode coercion?) already handles it so that's good.
Should this be more like the :current_year that is in attendee.rb, so it doesn't flip on January 1
When you use custom boolean fields 7 & 8 in phpBMS you cannot change the state, but when they are set as NOT NULL it works fine.
I think I did have a conditional here but removed at as I realised that combined_b or combined_l can never be nil as b1_available, b2_available are 0 by default.
This was the key feature that needed to added.  THANKS!
Porque estoy probando
El bug corregido consistia en que  se envia un caracter del hash hexadecimal para modificar el caracter, este oscilaba entrr 0 y f (char) que era corregido a su valor en ASCII mandando solo 48-57 y 97-102 (16 posibilidades) las cuales eran mas fácil romper.
should go below
Is there an upstream homepage location, or is the debian package page all there is?
why have an error label here that goes unused?
ahhh mira vos =) No entendía muy bien como funcionaba esa forma. Porque los ejemplos que encontré no me decían nada =/ Ahora lo arreglo =)
That switch should die.
I can dance to this.  +r
Hi

My idea was to clean up the code a little. One class pr database. Eg
OracleDatabaseHandler that had all the oracle specifics.
Now there is oracle, mysql stuff etc sprinkled across.

Haven't gotten the time to do anything more than what you have seen.

Regards
Per Ivar
On 2012 1 15 04:41, "Adam Schroder" <
reply@reply.github.com>
wrote:

> Saw these changes...can I ask the reasons why you did this?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/3komma14/PetaPoco/commit/dd022df68b483e4437748975a3de8f6b8b071cf7#commitcomment-868080
>
I like it. <3 conversations
No problem. For the most part, unless you're doing something really custom, the admin interface stuff doesn't need much in the way of design.
Note that this isn't quite the same. In some situations, sys.stdin.encoding can exist but be None. But this logic might be better inside the cast_unicode function itself.
Note that this is now broken, since functions are loaded after plugins, so auto-update cannot run update-omz.
This fixes #2

In order to activate jquery support a small snippet needs to be inserted in one of the loaded javascript files. This should be improved in the future to avoid the need of changing any of the tested files.
Hi! Sorry, didn't test that, but shouldn't this line be rather: "`...while (!(" + ... + "));"...`"? (That is, repeat-until ends when 'exp' evaluates to true.)
@eamelink Updated the ticket now that your CLA has been approved. http://tickets.opscode.com/browse/COOK-1043
ça fonctionne avec l'interface DE peter
Always remember to test with non-ascii characters. This is buggy:

Traceback (most recent call last):
  File "/home/osku/trelby/work/src/watermark.py", line 95, in OnOK
    if not util.writeToFile(fn, pdfdata, self):
  File "/home/osku/trelby/work/src/util.py", line 882, in writeToFile
    f = open(misc.toPath(filename), "wb")
  File "/home/osku/trelby/work/src/misc.py", line 73, in toPath
    return s.encode("UTF-8")
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe4 in position 12: ordinal not in range(128)
Looks like $areFieldsValid will contain the last value set and might not be *false* if set to *false* once in the middle.
Try
if (is_array(...)) {
I misspelled 'initial'. Of course.
this breaks reposync for me
I want to check all the components that satisfy the xpath if atleast one of them is mysql or ldap and none of the remaining are hpcc components. So, I guess, I can break in the else condition only if atleast one sql or ldap was already found.
nah, thats normal after save. its dirty because it needs to be refreshed as the backend might have updated something during the save (calculated property or default value for some mixin type).
why do you not want to lock a dirty or modified node?
i think locking a new, unsaved node will be a problem as jackrabbit does not know it yet. and it makes no sense whatsoever anyways, as other sessions can't see it anyways.
GitHub went south in my MacBook!
It's not needed for Live/XBMCbuntu anymore?
I KNOW THAT! 

Come on guys I didn't start coding last week. It still needs to be a new pull request.
they test the integration between the defer service and browser.defer
This one actually makes me feel like a hacker. My comments are 10x (well, 9.5x) longer than my code.
+msgstr "Nasze Warunki Korzystania z Usługi nie pokrywają twojego kraju (jeszcze). Sorki."
s/pokrywają/obejmują/

+msgstr "dodatek do firefoxa"
s/firefoxa/Firefoksa/

+msgstr "reklamy"
s/reklamy/zareklamuj się/

+msgstr "odwiedź [%(link)s](%(link)s) by dowiedzieć się więcej"
s/ by/, by/

+msgstr "lista redditów w których jesteś zaakceptowanym dodającym."
s/redditów/redditów,/

+msgstr "Nie mogę zrozumieć Twojego zapytania. Spróbuj ponownie."
s/Twojego/twojego/
bndtoolos? typo?
Try latest
Defaults are spread in two places now (AWSM and engineyard). Maybe we could modify the api endpoint to insert the defaults if they're not in the request? Does it output the defaults anywhere?

I like the pattern where a get to /models/new returns a "default" empty new object, so you get the servers defaults.
I'm all for simpler.
Y u no use camel-case
Another FYI and something I just noticed (but haven't verified) is that the order the pipes get closed seems to be wrong.

According to the docs (http://ruby-doc.org/core-1.8.7/IO.html#method-c-pipe):

> the two processes close the ends of the pipe that they are not using. This is not just a cosmetic nicety. 
> The read end of a pipe will not generate an end of file condition if there are any writers with 
> the pipe still open. In the case of the parent process, the rd.read will never return if it does not first issue a wr.close.
This is a *new* deprecation. New deprecations stay.
MUST MERGE THIS COMMIT.
> What about "CSS on, images off" scenario?

It's no better or worse than any other IR method that needs to work in IE7 or IE6. The only method I know of that works with CSS on and images off is [NIR](nicolasgallagher.com/css-image-replacement-with-pseudo-elements/), but that is IE8+
Anything that comes to your mind?
위키는 주말에 업데이트 할게... ㅎㅎ
I know, I know, just curious :-D
Hmm, when I try to install the latest version of `graphviz` using `npm install graphviz@latest` I still get 0.0.3, but the README indicates that 0.0.4 is done. Did you forget to publish it?
Agreed, lockProperties/unlockProperties should be both short-lived and be finished (lockcount==0) by time lockTransaction,
but lockProperties/unlockProperties will occur within the lockedTransaction. It's setting proplockcount=1, to avoid any nested lockProperties from attempting to changeMode again.

lockTransaction/unlockTransaction, is basically performing the same function as lockProperties/unlockProperites.
This doesn't work for albums or tracks, but maybe that's not important.
u get not a Ice Block ( 2 players ) to protect the another player for the Ice Bomb ( 1-3 phase )
Yes it is - it is preceding the rowsToSort.append() call in line 7383 which will add it at the end of the buffer without this flush call - either failing to reallocate again, or reallocating and using lots of memory.
Thanks for that Steve!  Worked a treat
```sql
UPDATE `creature_template` SET `AIName`='SmartAI', `ScriptName`='' WHERE `entry` IN (@NPC_ORSONN,@NPC_KODIAN); -- script npc_orsonn_and_kodian, the defines and enums in grizzly_hills.cpp have to be deleted
```

Did you read the comment on this line?
Grizzly_hills.cpp has to be adjusted, too.
"Note to self: HandleMailTakeItem not updated 4.x"

It's okay, I checked
https://gist.github.com/1612545
I think this can be done a lot easier:

Entry.objects.filter(status='approved', end_time__lt=date, project__id=97).values('activity__activity_bundle').annotate(Sum('hours')).order_by('activity__activity_bundle__order', 'activity__activity_bundle__name')
After this commit not work adding subpage in the django-cms, with 0.5.1 version all worked. Sorry for my English.
Why not just use `__import__(module, {}, {}, [''])` ?

```
>>> __import__('os.path', {}, {}, [''])
<module 'posixpath' from '/Library/Frameworks/Python.framework/Versions/2.6/lib/python2.6/posixpath.pyc'>
```
You don't have to add it...

What did you do to add this?
если этот боди из базы, то нефиг ложить туда больше 4096 символов. а если из юзер инпута, то вытирание всей мессаги вместо, скажем, обрезания будет непопулярно.
Nice work on reducing the complexity.
you could do battle.team(0).poke(...).nature() for example.
Ugh. I also changed the indent levels. Hence all the apparent changes.
actually.. close to what you had initially was better 
change the type of the jetty field to  Supplier<Server> as well, then just do jetty.get().whatever when you need it.  It is only calling server.get() in the constructor that is the problem. As long as the supplier only ever returns the same value, you are fine.
Can do..... but on the other libraries I checked it was defined as _DEBUG
I pulled each rule into its own class.
Shouldn't this be a class method?
I don't get it. A lot more code to do the same thing as before. Why? I don't even understand this new code.
you shouldn't use `ENV` to store configuration for this script since it's accessible everywhere and who knows what other code is going to do with it. this is particularly true when you use a name like `MAXIMUM_N_ENV_VAR` lol. if you MUST put it in `ENV` at least namespace it or something to avoid collisions (and you can leave the `ENV_VAR` part off ;)

i'd make the search upper-bound a named / hash parameter of the `find_n_for` method, tho the user should probably be able to set the default via a class `attr_accessor` or something
I'm not sure it'll work well with ahb & axi speeds over 400mhz, it might be overclocking the ram at those speeds, it's ddr2. One way to find out.
@Todo : The route pattern should follow the `fos_facebook.channel.url` parameter of the `config/facebook.xml` file.
No need to override a method on the prototype here. Register for the viewer's layerchange event to query the server for permissions.
Yeah, good point.  In fact, maybe the best way is to clamp a couple of these parameters to reasonable values.
Diese Beschreibung ist für den Benutzer sichtbar; wir sollten hier keine Klassennamen angeben (z.B. „Array index out of bounds“). Außerdem wäre die Angabe der Zeilennummer hilfreich.
I actually did read the manual, and the reason I brought it up is because you cannot specify to spill (you can only choose the sort algorithm).  I will remove the spill support from insert
I'm testing against PostgreSQL 9.1.  They are bytea columns.  I took the failing strings from the test suit and psql gave me errors as well.  When you say `'\xDEADBEEF'`, you're actually saying `byte DE + string ADBEEF`.
fluent ftw!
If you use init, then all the settings should go there in order to have it all in one place.
Thank you, please, consider looking again at the process of adding a query to .commands :

```js
self.commands.push({type:'insert', 'db_command':db_command, 'options':options, 'callback':callback});
```

Looking at the code the only place I saw theses ,commands being unstacked is here : https://github.com/christkv/node-mongodb-native/blob/master/lib/mongodb/db.js#L1330 (__retryCommandOnFailure)

But __retryCommandOnFailure is only called if there's no connection established : https://github.com/christkv/node-mongodb-native/blob/master/lib/mongodb/db.js#L1454

So in the program flow I do not see how pushed commands could be re-run (connecting -> connected -> never re run). Thank you for your patience.
Might want to use Record.putPayload().
Please don't add trailing whitespace.
Yep. Thanks a lot.

One of those things you don't see yourself anymore ;-)
Don't need this or the line before it
Missing docs that explain how this is supposed to be used.
Sorry if this breaks anything. I'll redo social integration once I get apache installed, probably this coming week/weekend
You do realize that supporting python 2.4 through 3.1 is going to cause us a lot of headaches?
We could either support 2.6 - future or 2.*, but supporting < 2.6 and >= 3.0 is madness.
(At least, that's the response I got in IRC when I asked how best to support 2.4 - 3.1)
i am excite
@jakul,

thanks for noticing. I'll prob. fix it tonight. Not sure how this happened though.
А почему так? Почему ты кавычки поменял?
I ran into this problem—thanks for the fix.
@ssaporta i made an example of using log4net across assemblies. the webconfig in the executing assembly (yours would be OOS.API) does the configuration. each assembly should have its own Log.cs convenience class. you can follow this lead to add logging to DataLayer.v2011.
don't know what happened... the update i've done only succeded when i done this one...
Yeah, JS templates and inline <script> are not feasible because of they way we load interfaces via ajax.
The only time we do that in core is for the TinyMCE config, as its too big and non-scalar to be contained in a HTML5 data attr.
For pretty much everything else, HTML5 is the way to go - now that we have Form->setAttribute() and FormField->setAttribute(), thats finally possible.
You should also disable the bundles in the kernel
move thsi to a function??? e.g. bee_find... whatever
I think so... I think I was expecting that it would get rid of all NAs,  but in fact we keep the ones that we NA when we started and only replace with 0 NAs (and infinite values etc) that were created during the log2() ?
It'd be good to figure out a way around this.
Please define a variable at the beginning of the function
Yes, that's the solution I had in mind. Do you just want to push that patch?
yes, works :-)
should not the seedRouter shut itself down after InitJoin instead?
Why channelVal?
yup, could keep them together in a Tuple2 also 
This has the down side that several competing threads would do the same hashing work, this is on the sender side so it's probably not usual to have many threads using the same actorRef

I think it would be perfect if first that detects could do the work and other continue with stale data.
Only case where I can think of where it could perhaps be bad is when creating the routees initially, but that is done from constructor anyway, isn't it?
For this we could use `ReentrantLock.tryLock()`
Is that good in the normal case, when nothing changes, i.e. quick op?
If there's a good place to look for examples here, that would be helpful to me.  I'm thinking you're looking for a signature along the lines of:

```scala

def onOpen[T](func: () => Unit)

// or

def onOpen[T](func: () => T)

```
On second thought: using `attachChild` means that the provider will treat this call as if possibly done out of the wrong context (`async=true` as for top-level actors) and use RepointableActorRef instead of LocalActorRef if one target address happens to match the local one. Seems like a slightly sub-optimal choice in a corner case (and hence not worth fixing), but I wanted to raise it nevertheless just in case someone falls over it when implementing ClusterActorRefs.
We would have to test, but you may be right.  I may have added it when something else was broken, or it may actually be necessary.
But why not simply let the DirectionFinder do its thing?
Yeah this functionality was added in master recently, has not been in a ckan release yet
I've added a check for a valid consumer to all of the REST API calls
It's coming out [W,L] now
I agree completely - just not sure what the adverse effects may be.  I suspect nothing, but not really sure.  I doubt skins are using IsEmpty(MusicPlayer.Foo) for determining whether to show VideoPlayer.bar for example.
That's what I have expected too in the first iteration but it broke behavior to what we had with SDL. SDL calls a depreciated function at this place: XKeycodeToKeysym. So I replaced this with XLookupKeysym.
The problem was key combinations like ctrl-s or ctrt-S. Without XLookupKeysym the character was always recognized as lower case (or upper case, don't remember exactly. So user defined keycodes in keyboard.xml were broken.
obj.sensors[0] is not necessary the 'ALWAYS' one (which trigger the default action)
something like https://github.com/laas/morse/blob/master/src/morse/builder/morsebuilder.py#L287 in `bge` might be better
sensors = [s for s in obj.sensors if s.type == 'ALWAYS']
Is there a reason that the other two examples of `Univ` shouldn't be `ManyUniv`?
shouldn't we require a name for all tasks?
qu'est ce que ça change ?
is it better to use `#!/bin/bash` or `#!/usr/bin/env bash`?

also, could you perform the pull request on the dev branch?  (I know! I should create a CONTRIBUTING.md file!)
that's legit. I'm on it.
Yep, I've done that, will merge it soon.
My point was that there’s no need for:

```js
var gaJsHost = (("https:" == document.location.protocol) ? "https://ssl." : "http://www.");
```

The scheme-relative URL in `g.src = '//www.google-analytics.com/ga.js';` wouldn‘t work correctly over HTTPS, either. Both URLs seem to serve the same file, but using [using `http://www.google-analytics.com/ga.js` causes an SSL security warning in IE6 on HTTPS pages](http://mathiasbynens.be/notes/async-analytics-snippet#comment-29).)
I completely disagree. Keep in mind that you need to have setting stored in `prototype` object for this check to be false.
I don't see the Error class defined anywhere, maybe you meant Exception?
We should not limit tip to thees node types. So remove lines 114 to 117
maybe put a "for more details on usage see X" link in the rose-brief-tour.html
ah yes, I agree this should use Arrays.fill.
Why is it cheaper? Maybe something to learn for me ;)

Sent from my iPhone. Excuse any typos....

Am 26.08.2012 um 13:20 schrieb Shay Banon <notifications@github.com>:

> In src/main/java/org/jboss/netty/channel/socket/nio/AbstractNioWorker.java:
> 
> > @@ -220,7 +253,32 @@ public void run() {
> >              }
> >  
> >              try {
> > -                SelectorUtil.select(selector);
> > +                long beforeSelect = System.currentTimeMillis();
> Why not use System.nanoTime? Its much cheaper, and also have have SelectorUtil.SELECT_WAIT_TIME_NANOS.
> 
> —
> Reply to this email directly or view it on GitHub.
>
The PyDataMem_* functions might be preferable here, might also be a bit of overkill...
prefix!
For consistency, this should be ``option_parser.error(...``, rather than raising the error directly.
верните этот файл, как был. для плагина с роботами есть отдельный qrealRobots.pro
["double check locking idiom" doesn't work](http://www.cs.umd.edu/~pugh/java/memoryModel/DoubleCheckedLocking.html), why are we using it?
Sadly not,  the assignment works, but then after the code has executed the assigned value is lost again :(
The behavior of this method closely follows the behavior of `ActiveRecord#touch` as shown here: https://github.com/rails/rails/blob/master/activerecord/lib/active_record/persistence.rb#L338-L357

The only caveat is that the `:udpated_at` attribute is something that is being derived form an instance method in `ActiveRecord` which of course is inaccessible in this case.
This comment could :fire:
Good point.

A runtime check if the arg is a task support is the only thing I see that could help here, and that's assuming that nobody stores sets of task supports.

Or rename the method altogether.
Previously we've had an invariant that if `MutableList` is empty, then `last0 == first0`. Now this is true when you create the list, but if you add and then remove from it, this invariant is no longer true.

I think it would be good to make `last0` point to a fresh `LinkedList`, `null` or some sentinel singleton right from the start.
Ah. I don't think it's possible to do that.
Maybe move up top?
Thanks! I forgot to edit those lines. I had left those in as notes for myself.
why not use just "point" instead of "otherpoint"? It is the other point with respect to what point?
The .args in a basic should be Basics as well (I'm not entirely sure, but I think strings are allowd ... @rlamy ?). My point is: the last argument should not be a list, but rather a (sympy) `Tuple`.
chached -> cached
pep8: spaces around `-`
Friendly. He argues because he believes strongly in his opinion.

Either case it is the core dev team that has the last word, naturally. Either you like what he did or reject it or pick it up and refactor it the way you want it to be.
This doesn't seem right. It is assuming a certain layout with ne=ne+1
You should be able to just call `grunt.task.clearQueue()` here, see [the relevant unit test](https://github.com/cowboy/grunt/blob/master/test/util/task_test.js#L296).
Dieses File brauchst du eigentlich nicht. Die ControllerAction "savegrouprequestAction" sollte nämlich nie auf dieses Template zugreifen sondern:
- Im Fehlerfall: requestgroup nochmals anzeigen mit den Fehlermeldungen
- Im Erfolgsfall: Weiterleiten auf die Gruppenübersicht (show)

Damit wird dieses html-Template überflüssig, da es gar nie erreicht wird.
A few of the `__init__` methods here have a `units` argument.  I mean those should be documented.  (If it's not clear, parameters to the initializer go in the class docstring rather than the `__init__` method itself.)
It describes the intent of the test better. I don't mind changing it back to test prefix, but imo this is superior.

For reference, Symfony only has one test that uses `@test` annotations, the `ProcessBuilderTest`.
Fair enough. We can change that some other time.
can you check if this is the way you're supposed to clear an object in JS (for GC purposes?)
Remove the if, leave only the change you made (don't need the old one anymore).
Will change to something more descripted and move.
Let's put this version in the main base pom to keep it consistent with the other modules.
should be separate blocks.
maybe end the previous paragraph with ... documents using th eMOngoDB JavaScript Shell, also referred to as the :program:mongo shell.

The :program:mongo shell is a full ...

Otherwise, it seems kind of heavy on the word JavaScript
so should we make a view test for $samples/The_following_is_a_new_section ? that's the main change here, right? the ability use live samples from content with the auto-generated section id's?
In fact, we can remove this warning, as, contrary to what I believed, under GRSecurity you actually can access /proc/sys/kernel just fine as non-root non-CONFIG_GRKERNSEC_PROC_GID user.
Lowercase phpbb.
Should be `return $parsed_array;`. The file might be missing, but the cache can still have data which should then be used.
This documentation is still missing. You can only document what you do, you don't need to explain why it's done that way.
The fix isn't too great, its possible for people to have sub-folders of plugins?
Why do we need extra line? :)
> ls -la /home/elvenfighter ;)

Those Gnome suckers just love to store /dev/urandom output in my ~/.*
But we should not pollute root. 

Anyway, you should look through this:
http://www.cmake.org/Wiki/CMake_FAQ#Out-of-source_build_trees

The main issue is that in-source build breaks out-source builds.
not quite a dup, but it seems like we could use a common routine for building a resource mock.
use `'include_dirs' in settings` instead of `has_key`
these HAUSNUMERO numbers will be wrong on every platform but Windows - read zmq.h carefully.  Perhaps the runtime-compiled version used by pyzmq-ctypes is preferable, as it is not vulnerable to this sort of error.
Great -- thanks for doing this. We should get rid of `str` now too...
e needs to be declared up with the other vars.
Forked issue as #129.
Coordinates are now never first class citizens within rule actions, so this is defunct.
你自己先加个TODO在这里吧。
Does jenkins ran this target (guessing from memory... I didn't start the VPN to log in and check).
Alex, would you like me to do anything else to this? I can cut my comments out & clean up the items you mentioned so it is a clean commit.  

Benjamin A. Petersen
ben@benjaminapetersen.me (mailto:ben@benjaminapetersen.me)
linkedin.com/in/benjaminapetersen (http://www.linkedin.com/in/benjaminapetersen)
facebook.com/benjamin.a.petersen (http://www.facebook.com/benjamin.a.petersen)
twitter.com/#!/bapetersen (https://twitter.com/#!/bapetersen)
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)


On Wednesday, June 27, 2012 at 9:50 AM, Ben Petersen wrote:

> Good call Alex. You are right.
> 
> Benjamin A. Petersen
> ben@benjaminapetersen.me (mailto:ben@benjaminapetersen.me)
> linkedin.com/in/benjaminapetersen (http://www.linkedin.com/in/benjaminapetersen)
> facebook.com/benjamin.a.petersen (http://www.facebook.com/benjamin.a.petersen)
> twitter.com/#!/bapetersen (https://twitter.com/#!/bapetersen)
> Sent with Sparrow (http://www.sparrowmailapp.com/?sig)
> 
> 
> On Wednesday, June 27, 2012 at 11:49 AM, Alex Ehlke wrote:
> 
> > > animate = animate || this.options.animate;
> > > 
> > > + // var keyword to avoid global!
> > > + // var tag = $(tag); 
> > > 
> > 
> > 
> > However this should never be undefined unless there is a programmer error, since `tag` is required, and so this will always bind to the `tag` parameter passed to removeTag. So no need for `var` here.
> > 
> > ---
> > Reply to this email directly or view it on GitHub:
> > https://github.com/aehlke/tag-it/pull/82/files#r1061040
> > 
> > 
> > 
> 
>
please don't use one-letter vars for anything than `i`, `j` counters. I know it's overcompilcated here to use long stuff, but in some time we'll switch to livescript or coffee-redux with `script-files |> map (.to-string!)`
Again whitespace.
Initialize *attr to NULL here to avoid undefined behavior through untested code paths?
This is worth considering - don't know the use case for this version of the constructor, but it doesn't distinguish itself much from the other constructor.  Again, I prefer a hot-fix or post-pull high priority issue.
button needs to be in the div so that it doesn't float to the right of the search text boxes like it does now
Comment no longer applies to code
currently no good way I can see to detect that IE9 and IE10 have fired interactive too early. Although, now that there's this repeatable test case, perhaps you can poke around too. For now, this seems to be the only way (although I agree it's dirty).
Hmm, this is just a single matrix multiplication if you use a matrix for coef. Please use Eigen for this multiplication instead of writing the loop yourself.
Nitpick, but is it necessary to do separate `make` and `make install` steps, or can we just do it in one?
...needed now because of the newly discovered `Formula.installed` :-P
What's the exact added value proceeding so? I mean starting string concatenation operations with an empty one… :)
Eh - I'm going to yank this at our deprecation window anyway.
For new code I think we should follow the convention of putting unit tests in `test/sqerl_pgsql_client_tests.erl` and using an ifdef export_all to allow testing of internal functions. Both approaches are reasonable, but I think the bloat and clutter of in-file tests will very often out weigh the benefit of their proximity.