labofsahil/github-event-dataset-2012 huggingface dataset:
Refactor lambda inference and fix related issues

For more robust lambda inference mechanism.

This also fixes following issues.
<a href="http://d.puremagic.com/issues/show_bug.cgi?id=7288">Issue 7288</a> - ICE(toir.c): with lambda return + auto
<a href="http://d.puremagic.com/issues/show_bug.cgi?id=7499">Issue 7499</a> - ICE(cast.c line 1495) with lambda array
<a href="http://d.puremagic.com/issues/show_bug.cgi?id=7649">Issue 7649</a> - Bad lambda inference in default function argument
<a href="http://d.puremagic.com/issues/show_bug.cgi?id=7650">Issue 7650</a> - Bad lambda inference in associative array literal
LPS-25792 Page Template name displays incorrectly

Site Templates has one some issue.
New commands to execute lines in the terminal.app should be valuable to users

cmd-return and cmd-option return added to execute line or selection in current tab or front window of terminal.app, activating terminal optionally
Simplify removeSuper

Forcing use of transaction to remove sub-files considerably
simplifies dfuutil method. Fixing behaviour when empty super
file would be left dangling because wildcard was used to delete
all sub-files.
NH-2722: Linq Count() does not respect previous calls to Select() or Distinct()

```
session.Query<Entity>()
    .Select(x => x.Property)
    .Distinct()
    .Count()
```
Results in a `count(*)` query being executed, rather than a `count(distinct Property)` query.

```
session.Query<Entity>()
    .Select(x => x.Property)
    .Count()
```
Results in a `count(*)` query, rather than a `count(Property)` query.

JIRA: https://nhibernate.jira.com/browse/NH-2722
Add CarrierWave uploader

Generic CarrierWave uploader feature
Upstreaming contribution from DamianZaremba

This pull upstreams some interesting fixes / enhancements from DamianZaremba
auto-psexec RC file

Hey guys,

 this is a quite simple resource file to check all discovered windows hashes from the database against all Windows fileservices via psexec. With the first found hash we could also use jtr to crack simple hashes.

With this we are able to check all hashes against all Windows filesharing services in a short timeperiod.

hopefully this is usefull for the framework. 

mIke
Fix background swap at boot-up

It was setting it to the clockwork logo at every load
Added minimum and maximum date selection & some UI updates

Hey there, 

Thanks for making this! It saved me a LOT of time!

I needed to make a few changes to the code for my own needs and I thought you might want them. I added options for a minimum and maximum date so you can stop people selecting dates in the past (or future) for instance. At the same time I also updated the calendar rendering to account for the min/max dates so it gives proper feedback about what dates are available. 

I also changed the calendar rendering to update on every date click so the calendars will now feedback instantly if dates become unavailable. Let me know if you spot any problems, I'll gladly fix them! 

Example usage: 

```
$('#rangedisplay').daterangepicker({
    format: 'dd-MM-yyyy', 
    startDate: '-90 days', 
    minDate: '-90 days', 
    maxDate: 'today' 
});
```
Markdown

* Fixed left angle brackets (`<`) being incorrectly displayed inside of code blocks.
* Expanded the whitelist for `<>` bracketed links to matched Reddit's whitelist.
* Added `prefixCursor` to functions usable by `EditControl`s.
(#12357) Fix root_home fact on Windows

Without this patch the root_home fact fails on windows.  This patch
fixes the problem by only calling methods on the object returned by the
`getent passwd root` command if the object evaluates to true.

Because there is no root account on Windows the code block simply
returns `nil` which makes the Facter fact undefined on Windows
platforms.

The root cause of the failure is that we always expected the command to
succeed and return something useful, and it may not on all supported
platforms.

Closes: GH-39
[TIMOB-7427] 1_8_X Wait for kill command to finish

[TIMOB-7427](http://jira.appcelerator.org/browse/TIMOB-7427)
Test is in JIRA

Duplicate of PR #1274 for 1_8_X branch
DB/quest: fix all mulgore quest

Every quest has been tested and the quest of that zone are all working now.
Home page grammar and readability fixes

Fixed a few grammar and readability issues on the home page while reading it. :)
ENHANCEMENT Better visual hierarchy in CMS for translated objects.

This should really be taken care of in CMS by default styling for CompositeFields, but this will do for now.
Add an option to fill portals with air on removal.

When useonmove is false, users have to set a portal on fire and then stand in the portal block for a few seconds, just like it's a nether portal.

If the portal is removed ("/mvp remove"), those portal blocks will persist unless the remover tears the frame down. If this removed portal is used by a player who doesn't know it's been removed, they'll expect to go one place and find themselves in the Nether instead.
Keep element style intact

Rather than removing the element style to show the hidden password input box, only clear the specific properties that were set by the script. I have element-specific properties set by my script on the real password input box that were being wiped out when the fake password input box was being clicked on.
This is really awesome.

You should add my stuff to your project.
Version update && .gemspec

Hi Andrew, 

I allowed myself to change the version number to 0.0.2 and add a .gemspec file (so it works with bundler)
please feel free to pull the changes if you like.

All the best, 

- Johannes
Just a quick typo fix

Just a quick typo fix (assuming it was a typo and you didn’t really mean “Jerkyll theme” ;) )
vorherige Korrekturen ebenfalls im Markdown korrigieren - LOL ohne Korrektur im Markdown waren die vorherigen Commits total sinnlos xD
Update members.md

Added Gabriel Barth-Maron to student roster.
Set framework

Added all files needed to work with (AOSP ICS framework with opengl and hwui CM9 folders) ;)
update builders for bpel and bpmn2 modeler >> master
add compatibility to both 3.2 and 3.4

the patch i submitted in issue #21 works for 3.4 but breaks in 3.2.

I look into documents about the argument passed to init(), it seems changed since 3.4, so I let codes to determine which to use.
add zh_CN i18n
Allow maqaf, geresh, gershayim in hashtags

In parallel with https://github.com/twitter/twitter-text-java/pull/29.  The branches I've submitted for twitter-text-(java|rb|js) all pass the extraction test added here.
dom ranges are great for selection of dom nodes

1) Added jQuery hook that (if jQuery is loaded) fires newSyntaxHighlighter event and passes the highlighter div id
2) Prefers ranges for quick-code selection with textarea method as fallback allowing for more seamless experience
Contextmenu

Can you integrate this method overriding ?
It allows having the context menus generated normally (especially after the long click patch)
Update show template label translation
Next config op

Changed the configure script in order to allow for builds without git submodules being invoked in the configure script.

The following changes were made:

1. Added the flag in the configure script for scm builds.
2. Changed the maven script to add argument to the invocation of the configure script.
3. Changed the maven script to pass on a parameter to the building of the Framework.
Patch 2

Correcta typo
gnome-3.6 compatibility changes

Changes to work with gnome-3.6.
Allow synchronous to be passed as an option to the agent initializer
Added BBM presence for other users

This change adds presence update information for other users on your BBM contact list.

This change is dependant on https://www.github.com/blackberry-webworks/BB10-WebWorks-Framework/pull/148
Currently not supported is getting the display picture of the user

Docs pull request: https://github.com/blackberry-webworks/WebWorks-API-Docs/pull/182
Viewable docs: http://blackberry-webworks.github.com/WebWorks-API-Docs/WebWorks-API-Docs-next-BB10-bbm-users/view/topics.html
Calabash 3.0
causes DEPRECATION WARNING on spec

DEPRECATION WARNING: Accessing paths using dot style as in `config.paths.app.controller` is deprecated.
Added support for new jQuery 1.7 APIs and callback object (tests not updated, but tested by hand)
black and white tracks are not same size - stroke-size issue

Hello,

I noticed that the white parts of the encoder-disk are not the same size as the black parts. (Nor the track-part-lengths nor the distance-between-tracks and tracks-width). Probably it is a bug, and I think, I can solve it very easily with setting the stroke size to zero.
I checked measure it in Inkscape by the straight-lines tool (shift F6) and now it seems nearly the same size.

To tell you the truth, I just started to use inkscape and python yesterday, and I did not understand your code deeply, so feel free not to agree with me.

Thank you for your great plugin, it helped me a lot.
Willam push
offer-atname-of-link-text-on-twitter-pages-11

Add @name to context menu options when over a link on a Twitter page.

Fixes #11
Comprehension for '%' character in non-pure unit types 

The label parser in registry.py was changed to support the '%' character in composite unit types such as '%/degC'.
Auto overwrite emptybuilder 2.0

port of #154 to 2.0

not tested on a project
The Initial Code

Issue: We need to start coding.

Take 2 - here is the start-up code needed for our HTML5 application. I have taken the time to create a basic UX with jQuery Mobile. Its not complete but this will start the ball rolling.
Fix a typo in Tx_FormhandlerSucription_PreProcessor_ValidateAuthCodeDB

This commit fixes a typo in call of the preprocessor ValidateAuthCodeDB

Tx_**FormhandlerSucription**_PreProcessor_ValidateAuthCodeDB.php
became:
Tx_**FormhandlerSubscription**_PreProcessor_ValidateAuthCodeDB.php

All the files referencing Tx_FormhandlerSucription_PreProcessor_ValidateAuthCodeDB were updated.
You certainly need to regenerate the md5 of the extension.
Performance Settings: Fix to remove preference if it's not supported

tested on passion,forget to enable compcache...
switch to openni_camera for kinect

Was using a deprecated ros package for the kinect.
Trad env_crawfordschoolnursesoffice_act3medsmission_english

Je l'ai mis au bon endroit cette fois ?
Make regex for /* */ comments less greedy.

I'm seeing the first `/* */` comment in my .scss file erroneously extend past the `*/` and all the way to the last `/* */` comment in the file.

Did some googling and found this page on the proper way to regex for `/* */` comments. http://ostermiller.org/findcomment.html

yayz for the internets!
Relecture du chapitre 8
Good call.  I've reverted that check back in [6a08675].  The condition at line 353 is now:  

```C
if(!PySequence_Check(data) || PyUnicode_Check(data))
```
Including the menu search to the makefile.am
I've just called `SDL_FreeRW(rw)` (see the commit I pushed into the main repository).
The resources that needs to be recorded for a transaction are the ones returned by the removeResource method call in approveTransfer.
What you commit in master, the terrorists win!
@dhh to reduce noise we can aways use `grep`... It would be nice to have all tasks documented with descriptions again.
@DouweM Aren't you glad though, at some level, that the issue was exploited in such a public way as to get everyone's attention? In terms of getting the message out there and inflicting a sense of urgency, I can't really imagine a better scenario than this. The cat was already out of the bag as the bug was reported days ago. But now people will have to take it seriously.
Well, looks like Heroku was just faster :) They've upped the support to 0.6.10 today, so I'll be gladly leaving the 0.4.x land behind. :) Still - I was running this for half a day in 0.4.7 and everything seemed to work just fine (except for the `package.json` declaration).
Do you mean that it's possible for some procedure to be calling fraction() it its results recursively?

Anyway, this looks good.
please use a pair of brackets after the first "if" to wrap the if/else pair
et je ne crois pas que tu ais besoin de faire ca ...
Re-adapt some of the entry points (the scheduler entry points were updated, for example), and go forward and test the system wide install... I need to get back to work on this.
Следует использовать кодировку UTF-8, чтобы не было таких вот проблем с русскими буквами. Также общей рекомендацией по документации будет совет попробовать Doxygen.
I've thought about merging this props to one in the past, that's a pretty good idea. Simpler == better.

So, after the change L19 will look like `(plugin::)? and plugin::pluginType`
Not arguing with the change, but why decompose the kv pair using a splat instead of in the block args?
I don't quite understand what aspect of the operation won't work and why? 

Also: 

- What changes in 2.2? 

- Is the fix to this problem to remove the `rs.remove`/`rs.add` procedure entirely?

- Would it be better to make sure that they specify the `_id` in the `rs.add` operation? 

Removing the operation makes the overall document less intimidating because users won't face a choice, but `rs.reconfig` is more intimidating/complex. Adding the `_id` field to `rs.add` and telling users to check for the `_id` manually is also more complex and potentially error-prone.
Hum, it strikes me as a kinda bad idea, if I want to know the email associated with a contact, I have to either do :

  Gandi::Contact.new('FOO1-GANDI').info['email']

which will call contact.info twice, the first time discarding the email, or I have to do :

  Gandi.call('contact.info', 'FOO1-GANDI')['email']

which gives the Gandi::Contact objet not really interesting :-)
Yeah, that's how the Django syndication framework works. get_object() is called to initialize the feed instance before get_object(). It's generally meant to look up the model object on which the feed items will be based (eg. parent blog post for a number of comments), but it's a decent place for side effects like this.
As long as you are adding tests, could you also add a test or two in which `x` is not given, perhaps one with the default `dx` and another with a `dx` given explicitly?
j'Ai apport une modif mineure qui corrige un bug
Things are really starting to go into Milestone quality here! :-)
jo, der setzt das föllig valsch ;-)
ich versteh auch nicht, was das bringt, ohne klappt doch alles?
He subido lo que me parece puede ser un layout de las secciones.  Manhana
por la manhana comienzo a poner de nuevo las cosas interactivas.  Pero
necesito saber si esto puede ser mas o menos lo que pensabas.

Para verlo, requiere un servidor web... espera.... recopilare los archivos
.less a archivos .css y cambio las referencias para que no sea necesario.
toke.

Mark

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

> hahahahahaha
>
> stop looking...
>
>
> On Sun, Feb 26, 2012 at 5:05 PM, vchigne <
> reply@reply.github.com
> > wrote:
>
>> Very minimal I see
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>>
>> https://github.com/mayorio/mayorio/commit/0a9707e18197f17c7876283a99a673a4992d618a#commitcomment-1013334
>>
>
>
>
> --
> Mark :)
>



-- 
Mark :)
this commit makes all priest buff's to don't apply istant but after some time.

please test your fixes before making a pull request :|
one problem with this is that the Express res.sendfile() will wont respond with the correct status code anymore, unless we tag the errors with `.status`. The `, true` seems like a bit of hack as well, what's the reasoning behind that again?
```
$ more progtest/test.c
typedef int using;

using foo(using y)
{
    using x;
    x = y * 2;
    return x;
}
$ ./cppcheck progtest/test.c --debug
Checking progtest/test.c...


##file progtest/test.c
1:
2:
3: int foo ( int y@1 )
4: {
5: int x@2 ;
6: x@2 = y@1 * 2 ;
7: return x@2 ;
8: }

```
x gets the varid..
Woohoo! Now I can throw away my custom clj-http fork :-)
Fixed in master.  Thanks!
I don't know how change the update_default without use of global variables. Maybe i could be global?
это вставка. а удаление залитого в предыдущий раз комплекта крич\го? ластинсертид тут никаким боком, там уже после могло много чего залиться.
Thanks, Michael! - this was exactly the right fix, but I didn't see your pull request until after I'd seen Joost's email and fixed the bug.
Unfortunatly it doesn't work... I've seen that post before, and I don't even know where did the writer came up with the "bind" namespace we see there...
the `''` should line up with the `'Addon` above
И как это проверить?
You're right, however that's outside the scope of changes I've made.

Feel free to supply a patch yourself and do a pull request. ;-)

On Tue, Feb 28, 2012 at 5:21 AM, Dominik Beerbohm <
reply@reply.github.com
> wrote:

> You should replace ereg with preg_match because it is deprecated.
> I just had problems with some scripts where I had overwriten the
> error_reporting (I don't know why) witch brokens on some ereg_replace in
> the Convert-Class :p.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/halkyon/sapphire/commit/8fdc531345d96127f66cc5064dbb209e01cbdd87#commitcomment-1015696
>
I want to be a little douchy and want more curly braces, easier on the eyes when reading
To avoid confusion for people hooking in
Heh You are right I will fix it
Hi Paul.. great to see you updating epublib :)

This borders on an issue I ran into for PageTurner: some books do have a cover-image inside the ePub, but don't actually declare it as such. It's often just the first image of the book. In my fork I had included some educated guessing which, if no declared cover could be found, would guess that the biggest image in the book was the cover. It worked fairly well, though it also got it wrong on a number of occasions.

I pulled this update into my fork, which breaks that logic and I figured before I invest time in making that work again I'd send you a note to see if you had a better idea of how to handle these books. I admit: they're crappy ePub files, but crappy ePub files are what we find out there :P

Alex
nice... I also added the following on line 62...

          location.hash = href;

...which updates the hash as a user changes tabs. Would love to see this tested and pulled in.
personally, I would inherit and extend `setUp` so that I could do this once:

    self.url = reverse('users.ajax')
    self.client.login(username='jbalogh@mozilla.com', password='foo')
By default django cascades on a delete, https://docs.djangoproject.com/en/dev/ref/models/fields/#django.db.models.ForeignKey.on_delete so I think we are ok.
Is it just renaming issue or are you expect some different functionality from what we have in `EmptyChainNode`
TODO: `if dev_only and settings.MARKETPLACE` to limit this to only mkt.
From http://www.cocos2d-x.org/issues/736

This is not an issue. Android devices running 2.1 will still be able to run the app.
That will only apply for devices using 2.2+
oops -- pushed the fix (didn't test though... must work! ;) )

d'oh -- just now read your commend on logError -- good idea!  -- commit (so I don't take the credit ;) ), I will merge it
"Chinese (traditional) translation"
@chuckremes that would be awesome. As I said on IRC, It's a bit of a hack for the logstash side to detect it. There's also the concern if that if a new string sockopt is added, we need to account for it here.

I did several tests using the Float trick and it seems to be pretty reliable. The issue on our side is that Logstash always sends down a string from the config hash type.
How about "bakåtstreck" or "omvänt snedstreck"?
you could fit this on one line: `plugin['guid'] = '{269eb771-59de-4702-9209-ca97ce522f6d}'`
I ain't gonna argue with that logic.
It seems like a hash must be passed to ```dulwich.repo.Repo.__get_item__``` [[1](http://www.samba.org/~jelmer/dulwich/apidocs/dulwich.repo.BaseRepo.html#__getitem__)] or it will raise the following exception. It occurs when a repository's name (which is passed instead) contains a dash (e.g. ``foo-bar.git``), or any other non-hexadecimal character, for that matter.

```pytb
       Traceback (most recent call last):
  File "../nano/nano.py", line 180, in __call__
    retval = callback(environ, **kwargs)
  File "../klaus.py", line 64, in wrapper
    **callback(env, **kwargs))
  File "../klaus.py", line 221, in __init__
    self.view()
  File "../klaus.py", line 240, in view
    refs = [repo[ref] for ref in repo.get_refs()]
  File "/var/www/klaus/env/local/lib/python2.7/site-packages/dulwich/repo.py", line 1099, in __getitem__
    return self.object_store[name]
  File "/var/www/klaus/env/local/lib/python2.7/site-packages/dulwich/object_store.py", line 112, in __getitem__
    type_num, uncomp = self.get_raw(sha)
  File "/var/www/klaus/env/local/lib/python2.7/site-packages/dulwich/object_store.py", line 300, in get_raw
    sha = hex_to_sha(name)
  File "/var/www/klaus/env/local/lib/python2.7/site-packages/dulwich/objects.py", line 91, in hex_to_sha
    return binascii.unhexlify(hex)
TypeError: Non-hexadecimal digit found
```

(Note that the line numbers given for ```klaus.py``` might be slightly off.)
I guess you can predict my comment on this one .. :)
a couple chances for exceptions here, so make local variables and then do assertEquals on them?
Admin users cannot delete themselves
Please consider removing these comments, they don't give information about the logic in your code.
I thought that but it doesn't seem semantic.
In my framework I have a class called Tools for this kind of method.

Anyway, that's why I think lambda functions are so important, sometimes a function just makes sense in a very specific code block.
I tried to sustain that thought and keeping all together.
thanks... :)
better to name this MarkLocals; Traverser is too generic.
g_path_get_basename ()

Returns : A newly allocated string containing the last component of the filename.

http://developer.gnome.org/glib/2.30/glib-Miscellaneous-Utility-Functions.html#g-path-get-basename

i.e. it looks like you're leaking this
Agreed.  The original reasoning was to release redis-rails for each specific version of Rails, after testing, to be sure each release would work across Rails versoins.  However, in the case of security vulnerabilities, this really isn't a good option.
add virtual keyword to remind that it's reimplemented from superclass
+
no more than 80chars per line
I don't think it matches cake's coding standards, though.
`if ($subquery !== false) {`
Hey thanks for helping out.
Adding a nested include key in cfoutput for text.html.cfm isn't a good idea since it causes slight performance problems and makes scoping a little bit redundant see the cffunction changes at: SublimeText/ColdFusion@985f23679

Also you might find hash tags outside of cfoutput tags in some cases so it's probably a good idea to just define them for the whole text.html.cfm scope. I added this here: SublimeText/ColdFusion@e30f848cae2bb4052d31c33ac7c4f0f589945107 and set the include to source.cfscript (although it may be better to refine to just function-calls and variables in the future)
Fun, i saw it too and i was fixing it.
I still got the too much lines filtered in output though.
I Taylorchu,
Reported in forum ? It's me :D, just fix this.
Return the results of the $mail call so that we can extend it via plugins.
```
:+?
```

I think you missed removing the "+?"  as well.
deprecated preprocessor abuse is deprecated
Лучше не привязываться к div? Он может потом стать и не div, придется переименовывать. Container например.
。。。别
The Reflection voodoo to sort out the objects from the classes and get them all loaded. Attempts to resolve the weirdness about objects mentioned in #154.
Could you publish this as a new version so it's easier to use via npm?
Thanks!
It must be a merging bug - javax.mail.internet.MimeUtility is fine.

On 2012.01.12, at 17:19, Gummi wrote:

> Is there any reason why this is not still "javax.mail.internet.MimeUtility"?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/idega/com.idega.core/commit/20ff88bc9fb9c86c54af0327900d6dab94eb57e8#commitcomment-861146
我就说在mac ports里找不到texlive-xelatex呢
Anything to look out for when upgrading from 0.13 to 0.14?
Why did you move the icon outside the link? It was like this on purpose, so that users can click on it directly.
We thought, the function should be clean. Means, if we ask for GS03, the result of GS03 (either present or absent) should be returned and the logic according to return value should be handled in the calling script. Anyway we dont have any issue. We will change it as suggested.
Quick question, what is the code_types.ct_id = 5 mean (ie what codes are stored there?)
UPDATE creature_template SET npcflag=16777216 WHERE entry=29488; -- down
UPDATE creature_template SET npcflag=16777216 WHERE entry=29501; -- up

this bug is already been addressed and will be fixed in next db release.
Жирный шрифт - некрасиво.
Understood. In that case, I'd recommend deprecating but not deleting the /dev branch (and making a new dev branch with a different name when the time comes)
A quick glance, but since ``USERENA_SIGNIN_REDIRECT_URL`` is always set, the ``success_url``, supplied with the view will always be ignored. Meaning that an user cannot change the `success_url` by changing the urlconf. Am I seeing this right?
Hold up, there's a bug here.  If next_time != 0, the next_time > this_time, but a cut_off condition is hit, the next step isn't triggered, and it should be.  Modifying to reflect.
Here I would really object, and here is the reasonning :

Whenever I clone a new repo, and I see it is maven based, the first thing I do is a `mvn test` to see if the developer made my life easy.
Using cassandra as the default repository implementation would stop me right away because 

. Run `mvn test` (no I would not read the whole documentation first !)
. Experience an error
. Be frustrated
. Find out that the error is cassandra related
. Therefore I will read your documentation.
. I would have to install cassandra.
. Run the cassandra script.
. Run your tests.

No way.

Use the filesystem as the default, and cassandra as an exercise to the reader.

Remember I am not cloning your application to learn cassandra !
That would be a no-go.
No worries, I'm the king of forgetting semicolons!
Can haz tests?
귀찮아서 한방에... ㅡ,.ㅡ

대략 내가 생각한건 다 적용한건데... 뭘 더 정리하면 좋을까?

현재 로컬 DB를 메모리 디비로 했는데... 이거 파일디비로 변경하고 백업데이터 일부를 넣어서 사용가능하게 바꿀까?
구지 그럴 필요가 있을까 싶기는 한데... 흠...

하이버네이트 기반을 JPA2 기반으로 변경하고 Spring Data JPA 로 가는것도 좋을것 같고...
테스트 오류 잡고나서야 가능할거 같고...

현재 웹 플로우 쓰는데가 있어?
그외에도 안쓰는건 좀 걷어내야할거 같은데... 벨로시티나 프리마커는 이메일 템플릿으로 쓰는건가...
I clearly asked for a jQuery plugin that could be passed an option to slide or fade.
It seems like these two tests could be refactored such that they only have to yield to a block to do the write_file section. Lemme know if that makes sense or not. Sometimes there is impedance between my brain's code and my brain's words.
We have a site running on GAE with the backend as rest service on an other server. So we keep no state on GAE, therefore all securesocial related state should be forwarded to our server.
It is applying three transformations to the type information - i) group ii) group sort iii) ungroup
I could write a function that handled it all in one.  I probably should add a getTypeShuffle() - but it would just call these 3 functions, but would be clearer.
Do we need to keep the reference to its container?
probably your manners...
Don't hardcode gossip.
en:
    flash:
I didn't do any patching at all.
At least I think so :)

Just created a repo an added You as a collaborator.
Thanks!
Can you please remove commented out code?
Please release 0.8. :)


*
Jos Valim
www.plataformatec.com.br
Founder and Lead Developer
*


On Mon, Mar 5, 2012 at 11:34 PM, Jos Valim <jose.valim@gmail.com> wrote:

> You got the power!
Excuse me, Mr.Conway. 
I just have 1 question about this book . How can we combine R to any mail server , such as : Daemon, Exchange server , to active the classification of emails. Thanks for your answering. Sincerely
lol - we'll see :)
这里也是一样的问题，同上一个review
Ok great. Your changes are a good direction. In particular the
"object" list which I did something similar to unify naming of
materials.

In case it helps prevent overlap I'm implementing in no particular order:

- server version of smgive
- dynamically colored blocks and recipes (batch processing)
- texture packing of local textures. Ie send 256x256 texture packed
with custom textures instead of lots of little ones. In particular
helps with explosion of combinations available from colorized blocks
- overriding of base materials by again using texture packing but
automatically assembling the texture pack zip to send to the client




On 18/01/2012, at 11:06 AM, iceReaper
<reply@reply.github.com>
wrote:

> however: i would not fork this version, cause im still refactoring some parts. i want to make materials and packs be loaded / unloaded at will. also make the materials accessable for other plugins, so they can rely on spoutMaterials for their custom materials. This should basicaly make SpoutMaterials an widely usable plugin. People just have to create an instance of the (under construction) SmpManager and then load the smp files for their plugin and voila, they can add their custom functions to the materials.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/iceReaper/SpoutMaterials/commit/f15ff40e64f0b4143352933189fcef0085780491#commitcomment-876448
On Friday 30 December 2011 08:37:06 you wrote:
> On MDN we have a 'Change email' page. [...]
> 
Well, in that case, I suggest you use your specific view to put a browser-id 
form object in the context for the specific template. Seems much cleaner than 
having the form present in the context when it isn't needed, even on your 
application, let alone others who don't have this functionality.
I'll would just bound that information to the current connection in memory, as it is only needed there (and the assistant is saved as it will be used persistant among connections).

But to your example, can you tell me how such a question answer flow is done on command level? Would need that info for the plugin system
Cache `$(this)`
确实是这样的。9的版本中文至少不会丢。
Almost like you know I'm looking :P
akira-cn is right... this changed makes this module works again without any addition addings like somebody wrote: http://bybunin.com/articles/8-kohana/24-pagination-kohana-32
Oh, and don't forget to send Lexikos a pull request.
I don't think you need to config this, "1.1" is already pre-defined in request_raw (please see client.rb).
How does the promise from this Deferred ever get resolved?
It was actually a pun on the Britney Spears  song :)
But with "you" it's twice as fun :)
Cool, pull request for this sent
As Deferred is a "class" I would keep the capital letter for the parameter (i.e. Deferred)
Corresponding CB: Bukkit/Bukkit-Bleeding@f7f2cadc3cc159c16d69781d4b336dc3e6a9d15b

The Multiverse branch that can be used to verify behavior: Multiverse/Multiverse-Core@4631fd0a612c6f9f2cdf50c4ac46233e5aec27a0
.length = 1440 /* 180MB for ICS */,
Dzięki tym testom zauważyłem, że rozwiązanie jest złe, tak więc nie martw się jak dostaniesz znów czerwoną kropkę. To akurat nie Twój problem będzie. ;P
Откуда у нас Доктрина?
ส่งงานแล้วค่ะ
You just reduced test coverage. This test starts with exactly the encrypted JSON that we'd fetch from the server. The two tests to which you refer create a record programmatically and give it a payload, then test that it round-trips through encryption.

Please restore this test.
Also, is there a technical reason why topic channels need to be served from a separate server? I thought the idea was to have (or at least allow) a shared namespace for user and topic channels?
Instead of pulling this one, I rewrote it slightly to combine line 46 and line 44.
Shaco how i find you? skype msn?
Is X-ControllerURL documented anywhere?
Is it possible to create a branch for people who are in SF 2.0.x whithout this merge please?

I can fork, modify and use the fork but, I think i'm not the only one with this problem
esta fallando pues se cambio el test, pero no la implementacion.
La implementacion sigue dando el error como slug, no como name.
Why `10.0` and not just `10`?
its only a start... you are free to put what you want in this script file... unlike 2nd-boot which could be used later...
The units of w have been changed to those of frequency, so that hbar*w is now in eV.
before_filterがいいと思いました
I usually call a function with spaces only between operators e.g. function(name = '3 axes')

As I understand it PEP8 calls for spaces around operators:
http://www.python.org/dev/peps/pep-0008/#other-recommendations

Please also use Numpy/Scipy coding style
https://github.com/numpy/numpy/blob/master/doc/HOWTO_DOCUMENT.rst.txt
@pfmoore hey, curious what your response to this thread about option groups is, specifically, whether we should worry about only using "add_option_group" (and not inserting directly into parser.option_groups).  I can see in the current optparse code that it doesn't matter,  but still concerns me.
ah, instead of an `ActorSystem`, the factory method should take an `ActorRefFactory` so that you can also create the supervisor below another actor
Changed the option name. Splitting up the tests so no need for the extra long interval.
Do I understand you correctly if I would pushing this down to a real toString method in `ConsistentActorRef`?
not keeping the string in a `val`
I think private[akka] should be good enough.
When does the data disappear?
The {} is not a function, it's a block of code that evaluates to a function.
Honestly I'm not a fan of it anymore. But nothing to do about it. 
Ideally I'd like to see:

    class A extends Actor { 
      become {
        ...
      }
    }

One thing to learn and no special treatment of the first behavior.
These should of course be documented
ok, but in the get scenario there is NO instance variable for the form to work with.  that can't be right can it?
This will break BC and is not wanted. The user doesn't know why this happens. Please remove it.
A .npy file?
But is it good to do this kind of magic by default?
Agreed, do you fancy making the change? I presume you've got the branch?
mock should be separate since it's a 3rd-party modules, and thus should appear below "import unittest" in its own group, according to PEP-8.
yes, but `fname`.
This feels dirty. Would you mind removing it for the time being? We can figure out what to do about this together.

The markdown conversion doesn't do good things for our collapsed one-liner styles, but I think we work something out that might make this unnecessary.
why the trailing slash?
I think we just want to generalize the idea of a card config.  This can just be a set based thing, and any of the sets need to be able to overload/override more general sets.  In other words, we want to be able to merge the effect of @card.config on on set the :respond_to? :config is true should merge it's settings with priority on settings to the most specific rule as usual.
What's the advantage? Having the if clause you can set a break point on either condition.
The problem is that we can never fully hope to support all artist credits types for music I think with the std::vector_string_, so it is better IMHO to give the user what they have set as the artist credits.

Check out the discussion on the thread, but I checked and we never use m_artist as an std::vector except to pass it to StringUtils::Join right now. Maybe there is more coming that I'm not aware of? 

The only place I see std::vector use in a vector fashion is to create the right entities and indexes in the DB, so why not leave all that code on the DB only?

Are you on IRC? We can discuss further, I want to make sure the changes are in the spirit of what you're trying to achieve.
The m_egl check isn't required, given the 4 previous uses of the pointer.
Вот. Точно, этого не хватало =) Только пробелами => отдели.
i would prefer a 2-liner, but that's me:

long_var_name = (
<tab> ceil(blah))
This is the warning message I just added
了解でヤンス
This should be just rolled into Account now.  authenticated_system too.
Why does this need to be allocated on the heap?
缩进啊大哥....
This is good for now - but as a heads up - we may reconsider this pattern of decoration. Would you like to lead?
It doesn't get the correct arguments, we need options
Why don't you just check the URL rather than the entire body?
`respond_to`
Yes, I was wrong. Closed the pull request. I wil fix it and make a new pull request.
FYI, The old method modified the options has as well:
```ruby
text.truncate(options.delete(:length), options) if text
```
Should we have a test like this for every method that takes an options hash? That seems maybe a little overkill? If not, then I'm not sure why we would do it here and not everywhere else. 

Saying that, if you do think that that a `.dup` should be added to all options hashes that delete keys, and an associated test written, I'm happy to do it.
It uncovered a bug in the optimizer, I opened a ticket this morning. https://issues.scala-lang.org/browse/SI-6188
I will address this.
That was wrong, see the latest commit for the working version...
Why a list instead of an ndarray?
postion -> position
Spelling (comparison)
Does the .exp() function work now for this matrix?
> Are you making it clear somewhere that the name of a morphisms has no relevance (other than for printing) for composite
morphisms?

The docstring in `Morphism` says:
> Two composed morphisms are equal if they have the same components, in the same order (which guarantees the equality of domains and codomains).  The names of such composed morphisms are not taken in consideration at comparison.
Thanks for the info. I've worked with ORMs for a long time, but I'm still learning the specifics of AR. I'll make this change.
Are we using sql? Why are we using the sql timestamp classes. Something that we couldn't get from java.util.date ?
name==link --> name == link

no need for parens
I would like someone with NetBSD to verify this.
"See read operations for more information"

doesn't quite read right?
And again with the same comment as below .. Shouldn't be special content here.
You are right that them being equal is consistent with incorrect behavior, and therefore, what you're trying to test. There is a way to manually set the mtime, atime. After writing to the file, I think you should (using the system call) File.touch(Time.now.to_i - 5) [or whatever it is], and set the expected values to the same, then run the :touch action, then check actual > expected.
It's unclear what this method does without understanding the rest of the codebase. *Why* is it doing what it's doing? Why would I want an additional logger with these default settings? IOW, please add some comments :)
Needs to be shortened. Remove personalized pronouns. Suggested rewording:
"A list of forums in which the user has the f_read permission but may or may not also have the f_read_other permission."
I don't quite think what you mean with this. The way it's currently handled the function is_enabled() strips the prepending "phpbb_avatar_driver_" from the class name. If an avatar driver wouldn't use that naming for its class name the "phpbb_avatar_driver_" will not get stripped form the config name and therefore it'll check for the config name that is named "allow_avatar_" + an appended full class name.
Something wrong with the formatting here?
The constructors should only take arguments the implementation depends on. A memory implementation of a data cache should for example not depend on `$cache_dir`. The `phpBB/config/services.yml` file and related mechanisms should make this possible.
I am getting the feeling that this code was not finished, as this test fails.
Which alternative do you want?
This is a bad idea imo. The controller helper instance is shared between controllers, it should not be allowed to change. Once controller should not be able to affect an other.

IMO url_base should be removed. Users can make their own convenience methods in their controllers.
Yes, this is exactly the same as above, but I don't think a new function should be created just for this case. A lot of code in the MCP is repeated in a number of areas--it's just a mess of procedural code in general. If some major work is done to these areas, yes, it should be rewritten, but I think since this is just a bug fix it should not introduce a different style to the code (especially since this is only two spots).
maybe add a 1line comment above (just to make haddock happy)
Why not just use WAI's request type?
I agree with the separation. Maybe the thing I have a "beef" with is the name. I think it's perfectly understandable that the function that executes the app adds the `X-Hails-Sensitive` since it needs to inspect the resultLabel. Maybe the simplest thing to do is rename `addXHailsSensitive` to `runHailsApp` which may be trivial & just calls the function `addXHailsSensitive` to add the header.
yes, it was int before, but the braces messed up the syntax highlighting/linting, and it didn't affect functionality.
This was a comment that was just moved around in the file.

Yeah, this function needs some love.  I think there's already an open bug on it :(

However, fixing this is outside the scope of the resource store redesign.
To provide the build:* context config. I tried adding to the environment:* context but I guess that's not permitted.

Some app configs are intended for use *only* with a hybrid build inside CRT, and potentially per device. The build:debug and build:release were chosen to mirror Anne-Lise's scripts for copying dev vs minified resources.
In fact, I think we should add an explicit 'skip_detect' config, which cross compilation would imply, but could be set manually in other contexts.
And it shouldn't be there, should it?
Please, remove these comments.
What's with the whitespace? :)
yea... JSbeautifier broked the HTML..
Now I am manually  doing it..
Instead of writing the full form definition in the controller, recommend you abstract this form into a "View", and just call it by xtype here.
move to Utils  or make ticket.
As currently defined `grid_north_pole` shouldn't be in the "Args" section. But taking into account the ellipsoid-via-containment change I'd rather see:
```
    def __init__(self, grid_north_pole_latitude, grid_north_pole_longitude,
                 north_pole_grid_longitude=0, ellipsoid=None):
```
fg depends on null_type. null_type depends on type. type depends on op. op depends on fg.
Originally, I made this public on the base class, which meant that it needed to be public down the hierarchy.
Since I later decided to define voltGetExpressionXML as the public interface that IndexAVL uses to get to Expression's voltGetXML (and some related processing), I should propagate the original visibility back down the hierarchy.
Don't need to test !m_unique.
If m_unique, n == p == m_root, so we test for p->parent == &NIL and we are done!
Actually, since you can't have AggregateExpressions except from an Aggregate, an Aggregate child is already implied when you find hasAggregate() == true.
The getChild(0) and class check can be removed, just in case someday some other node might get pushed down between the OrderBy and the Aggregate -- I could easily imagine someday a Projection node, there.
Is this loop somehow preferable to using [:width] to get a truncated sublist?
2001,1,2
Use a save priority.
You can use exists() instead of count()
Well done then! :P
@slfritchie Is it only certain apps? I could understand why unloading kernel or stdlib would do that.
On second thoughts and after searching the existing docs it should be `class-based`. The release notes certainly always use this form. I'll update all references not just those in this section.
I know that logging is currently not used, but it might/should in the future. Let us forget this now, this should be the subject of a separate ticket anyway. But then I'd advocate for unconditionally raising the TemplateSyntaxError.
Buffer overrun check.
I am just as used to point-free as I am to point-ful. For me it is just as clear, except the point-free one produces less hlint warnings :)
this # shouldn't be removed
P.S. I know this looks like a lot more code, but gzip will compress this nicely. And thanks for working through all the changes and suggestions. I'm still not sure if this pull request will get merged in, but it's important that we at least get to an optimal workaround so we know the true cost of fixing this bug.
Even though this is an internal property, the original article talked about this property and I've seen some people using it on stackOverflow. It should probably stay even though it's not documented.
I'm not saying let's do it completely automatic way.  We can have a short form where it infers the better way and users can also input their preference.
What is `plc` function?
style:  can you put spaces around operator '/'?
The platform check is independent of the version check. Currently we have:

    schopf$ src/bin/madpack -p greenplum -c :54322 version
    Password for user schopf:
    madpack.py : INFO : Could not parse version string reported by Greenplum DB. Will default to newest supported version of Greenplum DB (4.2).
    madpack.py : ERROR : Invalid database platform specified.

We might want to swap the version and the platform check.
This seems like an overly complex method of type verification, I do not understand why you can't simply rely on the databases type verification.   E.g. both "smallint" and "int" have built in casts to "bigint", no extra code written.

If the problem is that there is no automatic cast from "bigint" to "int" that is because it is a lossy conversion and you shouldn't force that to be automatic for the same reason that forcing the cast from float is a bad idea.
Thanks for describing the use-case @kennethkoontz.

The current way the Meteor auth system is set up is that once you auth against an OAuth provider your token is stored on the user document, as `user.services.google.accessToken`, for example.

My understanding is that also, each service has their own API (e.g. OpenGraph for Facebook) and that the actual API for access data is not part of OAuth. The only interface between OAuth and the service API is the access token.

So is the thing you're asking for just a helper function that allows you to make HTTP calls and automatically puts the access token into the query string? That seems pretty straightforward and the differences between services doesn't seem to justify a special API (which would end up very complicated.)

Separately, a use-case that wouldn't be covered by the current design is if someone wants to grant privileges to multiple facebook accounts. We currently only allow for one facebook account per user. That's something that we might want to think about in the future, but I don't think this it the use-case you're describing.
that's not not ugly :-)
My current thinking is to do this at link-time (in keg.rb) instead; the patch might look like this:

```diff
diff --git a/Library/Homebrew/keg.rb b/Library/Homebrew/keg.rb
index 6e0a857..1b8e413 100644
--- a/Library/Homebrew/keg.rb
+++ b/Library/Homebrew/keg.rb
@@ -80,6 +80,7 @@ class Keg < Pathname
       when INFOFILE_RX then ENV['HOMEBREW_KEEP_INFO'] ? :info : :skip_file
       when LOCALEDIR_RX then :mkpath
       when *share_mkpaths then :mkpath
+      when %r[^zsh/site-(functions|scripts)] then :mkpath
       else :link
       end
     end
```
What’s a better way to test it, then? `pdfsqueeze` without any arguments seems to exit with a non-zero status code.
qtconsole and notebook both always support utf-8 afaict,
Ok, now I see it! I forgot you'd already supplied a non-default configuration to fluid.set which ended up knocking off the original defaults you get for free. You need to put back "fluid.model.defaultCreatorStrategy" when you use fluid.set in order to reenable this behaviour. You can take a look at ModelTrasformations.js line 649 which sets up "setConfig" - you can ignore the "schema" bit for now which is a bit ambitious and just set up 

        var setConfig = {
            parser: parser,
            strategies: [fluid.model.defaultFetchStrategy, fluid.model.defaultCreatorStrategy]
        };
It's ok. I've changed it and I've also changed the next one. Pushed.