labofsahil/github-event-dataset-2012 huggingface dataset:
Include http_build_query() for php4

phpBB 3.0 requires PHP 4.3.3 and SN too however http_build_query() is only in php5. therefore we need to include it to the code

https://www.phpbb.com/community/viewtopic.php?f=70&t=2008845&p=13200908#p13200647

Automatically closed issue #158
A couple of typos fixed

Just found and fixed a couple of typos while having my Macbook read it out to me on a flight yesterday. 

I'm only half way through but have learnt heaps already. Thanks for putting this together!
INTDSLGROOVY-11 + minor imp

- INTDSLGROOVY-11
- @CompileStatic for Closure handlers
- nano fixes
Adding rim:userAgent to the widget config element

Fixes PR  240955.
Simple fix for issue #6.

On error plugin throws exception as described in issue #6.  This commit checks to see if the viewer is valid before updating.
Hungarian translation

Hope you wanted it this way. :)
Rendered button is no more primary by default

just a little update about default primary button style : we can't override button style if setted
Pull Req: Fix when adding event listener forth time disabled callback invokation.

This pull request solves blackberry-webworks/BB10-Webworks-API#15 issue.
Includes unit tests.
Build files changes and a few minor fixes
Fix a NPE in synapse JavaClient

Previously, PaginatedResults was being instantiated incorrectly.
expose StorageUsageController getItemizedStorageUsageForNode in the java Synapse client
Fix schema path/id and make schemas translatable

This changes org.mate.apps.caja-open-terminal to org.mate.caja-open-terminal and uses /org/mate/caja-open-terminal as the schema path. I also made the gschema translatable and removed a few old mateconf cflags.
Jellybean

Should bring infuse4g up to build against current cm10 mainline.
Append filename with .gz when it gzips, this way if you run sync_media_s...

...3 multiple times, once for regular upload and once again for a gzipped copy, there will be distinct file names. Currently if you run 1) ./manage.py sync_media_s3 mybucket and then 2) ./manage.py sync_media_s3 mybucket --gzip, all of the filenames in 1) are the same in 2) so all of the regular ones get replaced by gzipped copies. It would be better if we could keep the names different. Thanks!
Merge Change-Id: I1f9b3faa77d49e26ef43982ff57cbbf0c3be8b40 to Gingerbread

https://github.com/CyanogenMod/android_frameworks_base/commit/14fcf900ce65e5c8c952c95ee12041f8de16fbb7
Fix invalid JNI call signature.
Bug: 5029536
Change-Id: I1f9b3faa77d49e26ef43982ff57cbbf0c3be8b40
MailProcessor example

Hello,

Could you put those commits to make an example of MailProcessor as asked in the message https://github.com/activewarehouse/activewarehouse-etl/commit/1981c537e4b8a718769f6d3394104061c8cc15b6#commitcomment-424707

I think I am done with all the notifications you send me.

Thanks
Notabs

This changes the tabs metaphor into simple blocks with less styling. Tested in LTR, RTL, and small:

![rtl](http://dl.dropbox.com/u/68059/Screenshots/d.png)

![ltr](http://dl.dropbox.com/u/68059/Screenshots/e.png)

![small](http://dl.dropbox.com/u/68059/Screenshots/f.png)
Adding lat/long variables.

Adding the ability to use latitude and longitude variables in templates.
fixed clear and save log
fixed enum in mainform.cs, added repeat name error checking
request pull trus merge

cuma ngganti print jadi echo,
sama ngetest fitur github  maklum baru nyoba
some updates

- allowed slashes to be entered into the podcast category setting
- added help in an error message to fix the error
Issue 3011 - alias should have assignment syntax

http://d.puremagic.com/issues/show_bug.cgi?id=3011

Documentation fix for the syntax: `alias ident = Type;` and `alias this = ident;`
Custom 404 Error Page

I added a custom 404 error page.
First version

Bon dia,

T'envio la meva primera versió, crec que no hi ha cap error, si en trobes algun, envia'm un mail.

Bones festes,

Ferran
Pneumatics Stuff
Support for terminal-notifier

Hey,

I've added a basic [terminal-notifier](https://github.com/alloy/terminal-notifier) support. 

Additionaly, there are some small changes to the error messages as they could be confusing for newcomers.
Win32 curl download link

@WalterBright This pull request relies on [curl-7.24.0-dmd-win32.zip](http://gnuk.net/d/curl-7.24.0-dmd-win32.zip) being uploaded to `ftp://ftp.digitalmars.com/` before it should be merged.
Actually it was already pushed. I cleaned my fork so now. but thanks anyway :)
Thanks again, fixed.
No worries sounds good, I did start with that change but I pushed it down into zfs_umount_snap because the change above fixes the temp snapshot destroy in the diff, but think deleting a snapshot using zfs destroy and getting dataset not found comes from a different caller that also   not setup to deal with ENOENT from zfs_unmount_snap, so we might need to identify that caller as well. 

Do you know the path zfs destroy would take? I put a print_backtrace in the stap script I was using in the zfs_unmount_snap but it didn't seem to make sense and then I ran out of time and went on hol's...
On Feb 8, 2012 9:09 AM, "Brian Behlendorf" <
reply@reply.github.com>
wrote:
>
> Thanks for running down the root cause on this.  I think a slightly
cleaner fix would be just to check for ENOENT as a safe return code from
 `zfs_unmount_snap()` in `dsl_dataset_user_release_one()`.  This was the
ENOENT behavior is preserved for all other callers which might need to know
why an unmount failed.  I'll roll this change in to an updated snapshot
patch.
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/b333z/zfs/commit/2d69db27f9f459d4205050899589893dee4564bc#commitcomment-942984

No worries sounds good, I did start with that change but I pushed it down
into zfs_umount_snap because the change above fixes the temp snapshot
destroy in the diff, but think deleting a snapshot using zfs destroy and
getting dataset not found comes from a different caller that also not setup
to deal with ENOENT from zfs_unmount_snap, so we might need to identify
that caller as well.

Do you know the path zfs destroy would take? I put a print_backtrace in the
stap script I was using in the zfs_unmount_snap but it didn't seem to make
sense and then I ran out of time and went on hol's...
This commit breaks multi-channel PCM output over HDMI using DVDPlayer (PAPlayer always had a broken layout). See this bug report: http://trac.xbmc.org/ticket/11805
TODO (DS/AL): defensively check this
You are right. There was a problem with unit test, we have to use instanceof \Traversable instead of instanceof Traversable otherwise the check fails.
This area is marked for refactoring, so I won't submit a pull request (unless you want one), but instead of doing an unshift with the object, couldn't you just pass it into the initial value for reduce (the second parameter)? And the same suggestion would go for the setNestedProperty below.
Thank you for this modifications. Just one stupid question: is this new stuff already well tested?
I was thinking in implementing the gauge fixing functionality...
:awesome:
或許可以把更新tray的行為，如destroy_tray(), disp_tray_icon()...的TRAY_ENABLED block去掉
TRAY_ENABLED的判斷全交由eve.c中定義destroy_tray(), disp_tray_icon()時處理，
換句話說，當use_tray=N時，所有關於tray的操作只會 call 一個空的函數。
這樣做可能比較不累贅。
Ahh forget it ... the its an axis and everything gets strange i guess ... i'll do it with the holdtime.
True! But I try not to use it much because it's Rails, not Ruby.
make variables private explicitly
make them final also, if it makes sense
Throw and error/log something if firstByteTime is not a long. That condition represents a bug in the code, so it'd be a good idea to make it clear to the developer when it happens.
I'm curious what the performance difference is, datetimestruct vs. creating a Python datetime. The other trouble is that the latter will hit the Python memory allocator
@chatur Could I ask to regenerate `unix` for freebsd 32 and 64 with this bindgen patch? `sig_t` didn't get generated properly.
twilio-ruby will be a runtime dependancy
Porqué usarlo en vez de ruby-debug?
Should this be > 4.2?
Oops, good catch! I wonder why this didn't cause any problems?
Ah forgot to do that. Thank you!
Yes, it has been tested and has been running in production for about a week.
I agree with you. I can parse the path in connectionpool.py. I originally wanted to use get_host because it was available.
Maybe `file->open` should throw the first exception. The rethrow suggests we expect it to do so.
Did so in 48229ac
??-

On Feb 15, 2012, at 5:34 PM, Bryan Veloso<reply@reply.github.com> wrote:

> Seriously!? :trollface:
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/eldarion/pycon/commit/25489f2423045f433dde6d45b408976494350847#commitcomment-971651
This commit actually breaks playback of all live streams (TVHeadend as backend). Recordings still work.
No worries.

When you install cocoapods and run pod setup, it will install a git pre-commit hook which will verify the changed specs.

On 27 feb. 2012, at 00:42, Zachary Waldowski<reply@reply.github.com> wrote:

> That's quite odd. Checking it right now, so sorry!
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/CocoaPods/Specs/commit/728512924adc0a953165f3a18c9ead427ec982bf#commitcomment-1013229
2012-02-25 02:58:24 ERROR:World::Statistic AHBot 0 AuctionMgr 0 BattleGroundMgr 0 Begin 0 CLI 0 LFGMgr 0 MapMgr 92 MapPersistentStateMgr 0 MassMailMgr 0 Quests+RandomBG 0 RemoveAllObjectsInRemoveList 0 Sessions 14 TerrainMgr 0 UpdateResulttQueue 0 Uptime 0 WorldPvPMgr 0

2012-02-25 02:58:24 ERROR:Map::Statistic map (id 571, inst 0, diff 0): AVG exec time 69.487968 ms, objects/type: 0/0 0/1 0/2 4173/3 442/4 1586/5 1/6 52/7, auras 14904, holders 10310
2012-02-25 02:58:24 ERROR:Map::Statistic map (id 0, inst 0, diff 0): AVG exec time 35.318531 ms, objects/type: 0/0 0/1 0/2 2862/3 399/4 1967/5 0/6 92/7, auras 11901, holders 8209
2012-02-25 02:58:24 ERROR:Map::Statistic map (id 1, inst 0, diff 0): AVG exec time 35.151711 ms, objects/type: 0/0 0/1 0/2 2472/3 351/4 881/5 4/6 155/7, auras 8224, holders 5642
2012-02-25 02:58:24 ERROR:Map::Statistic map (id 530, inst 0, diff 0): AVG exec time 21.167517 ms, objects/type: 0/0 0/1 0/2 2825/3 276/4 1103/5 0/6 64/7, auras 4599, holders 3064

Is world statistics right? Not enought data yet, this comment only for know why soo much zeros.
Mostly issues fixes. Problem in duplicated or wrong data.
Whats the benefit of allocating a variable for this enum?
Using file_exists would be a little quicker here.
why can't you use `:last-child`?
@ahocevar see "Copyright 2008, OpenLayers" @  https://github.com/openlayers/docs/blob/master/conf.py#L42
Хоршо, а то сам натыкался на проблему с этим связанную но забыл создать тикет
Hi!
I'm using client/X11/xfreerdp -x a7 -u "<user name>" <host>
Target is Win7 with network authentication enabled.

This should:
   turn off Desktop wallpaper
   turn off full window drag
   turn off menu animations
   turn off mouse cursor shadows
   turn on font smoothing

I guess the most obvious thing is the desktop background. With 
cbPassword + 2 I always get the default performance settings. Reverting 
it restores the previous, correct, behaviour.

Note that I really don't have a opinion about whether just reverting it 
is the right thing to do, as I don't know what adding + 2 is supposed to 
fix.

Best regards,
Christer Palm

On 02/06/2012 12:10 AM, adambprotiviti wrote:
> How did you verify?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/FreeRDP/FreeRDP/commit/927bc717809030a793a0626b37f7c82a8a937150#commitcomment-934885
>
needless spaces on line 41
@sunaku don't now if commenting here will be signaled to you

but if I've not mistaken this parameter was missing 
( I love Celsius :P )
closed #9 closed #10
I don't want to add another command line option for this, it clutters the command line API.

A better solution would be to make the `Junit` class subclassable so you can override this behaviour in a subclass.
I think this should be the other way around - the global constant should be used to set the config value.
should there be a ResolutoinRect here?
oh sorry it was not loading on my end,
i attempted to install the application using the install folder and got a blank page at the etape5.php
did some digging and this seem to fix it
So simple :)
conf in save_settings() is not being freed after use. Also, it might be a good idea to check if config_file_new returns NULL.
Please fix up the description to be accurate and descriptive on all file header comments.
This is cool. I started on something like this for buster-mode some time too. I added jump-to-...-other-window and put them on the keys instead, as I prefer to have test and implementation side-by-side.

Also, what do you think og `ji` in place of `jl` - "jump to implementation"?
These params are not relevant now
I just changed it to do all lookups by library name, instead of the loosey-goosey switching back and forth I was doing before. I think this should make everything happy. 

If it does end up that library names are a pain, we can switch to just requiring movies to be exported as well and use symbols everywhere. I just didn't want to add more work for the artists if I didn't have to.
This is where I added some changes.
The 'homepage' route points to webcrawler/main/index. It allows us to redirect there without seeing the controller and directory in the url.

The 'login' and 'logout' routes perform a similar function of hiding the controller from the url.

Overwriting the Route::url function in this case just lets us create prettier code.
Weird, I don't show "input, error, output" in my source file. Maybe this was against an earlier commit? Anyway, I see your point about the assignment and I'll fix that. I'm going to simplify the Windows/Java checks, too.
no problem; there's also a version 2 on sourceforge via svn, but it does not work with combos correctly in SM5 (and I don't know why).
(that's not meant as a job for you in this commit, davor, just a general statement)
hey why did you clean span 13-24?

we talked about that in https://github.com/twitter/bootstrap/pull/2143

and have some respect please, don't call this shit
Can we add a comment explaining this

~~~ js
// don't shiv elements that can't have child nodes or namespaced custom elements to avoid bugs with overwriting `createElement()`
~~~
Need to capitalize the M in marketplace.
Why 10 ?
I believe the calls at line 578 and 596, "vierws = get_couchdb_views(datastore_name)", won't work.  For the directory, data store and resource registry services, the datastore name is prefixed with the sys_name, so there won't be a match.  If you want to do this call, I'd recommend changing get_couchdb_views to do a comparison of the end of the datastore name string.
nice work! hope I didn't mess you up too much before I left for my interview. If you have any questions or want to pair up one day let me know. Are you a RoR programmer?
Good point.  My intention was to make sure that we actually had hosts in the hostgroup the check was going to apply to, but clearly I didn't get that right.  Thanks for catching that.
there's something I fundamentally don't understand here...this feeds the "which device types does this addon have" functionality
yep - on it
why?
https://github.com/ChaosEMU/ChaosCore/commit/d618983450d2dca4697d8df3d0595cb8ca3abf65#L6L668
Grazas por avisar, a ver se esta tarde actualizo a tradución. Un saúdo!
where does `$schema` come from?

maybe (the line above): `$model` -> `$schema = $this->schema()`
`Cmon, when do I ever find bugs in your code?
ok,will do that later on this week,and submit to you,cheers!
Sorry for fail commit message. I added new lines to the judge output.
Second query call shouldn't hit the cache., because of that adding. If the tests fails, query cache is not invalidated, ie. second query call hits the cache and entity inserted before this call is not included in the result.
названия методов по-прежнему хуже некуда
This is just removing files because I organized them into their own folder.
use xlt() fucntion
Is putting this information into a statistic the right thing to do? My intent is to expose the triggered build information so that it can be used later, and this seemed like a nice place to store it. This seemed preferable to adding a new member variable.
This is almost too trivial to mention, but it's strange to have both https://www.github.com and https://github.com mentioned (I suggest we stick with the latter for brevity). Also, hyperlinks in reStructuredText can be broken on the space between the display text and the URL, and should be, to enhance reading flow.
You can achieve the same modulus operation by django built in feature {% if forloop.counter|divisibleby="4" %}
I'm sorry ,I did not find any thing useful in your link .What I need is a help to install koha 3.0.0 on opensuse 11.1?
Looks good. Thanks Jim!
All that styling should be coming out.
does it work for you this way with protocol="tcp,udp"?

iptables I think doesn't support comma separated protocol specifications, and I don't think f2b anyhow pretreats those
Thomas, I'm not sure this is right. Isn't it better to handle this problem in xwiki-platform-wysiwyg-war ? I don't like the fact that gwt-user dependency is duplicated. GWT modules are like any other Java module, the difference is at package time, when Java code is compiled to JavaScript. From the point of view of Java, gwt-user is like any other dependency. The fact that we don't want it to go into the war is due to the way the GWT code is packaged in the end (i.e. compiled to JavaScript) so I prefer to fix the packaging module, i.e. xwiki-platform-wysiwyg-war instead. WDYT?
Thanks! I got the patch from my friend and didn't check it thoroughly. I applied cretindesalpes's solution because it seems cleaner. Fixed in 28f061e .
Funny,

I was just discussing converting the navigation project into a collection of extension methods for things like this. I must look at your code to see why it isn't working for the initial view. It could be because at the stage of calling that, you have not yet navigated.
Thanks, I've updated autoloader locally and will push it soon.
Opss!!!! sorry for commit title

"remove broken IOCTL_MFC_BUF_CACHE and reduce MFC memsize to 32mb"

Thanks to @jt1134
n here too
It works fine now with your last commits

thanks!

HTC Desire HD
Just so's y'all know, we now have limited support for multiple games.  I did some things I'm not proud of tonight, including using a global for GAME_ID in the browser, so feel free to comment on my various mistakes and/or yell at me.  But it works!
What?!  Why?
Please use parens in `def`s. Your code will definitely work without them, but it looks a bit odd. I know your way is more consistent with how we call methods, and I don't know why people prefer having the parens, but it seems the right thing to do.
The point of the new gem layout was to make it faster to create new releases of neo4j.
Another reason is that it will make it more obvious which license are being used.
I like your idea, mainly because it is simpler - let's do it like that.
Yes, it's enough to lock the gem dependency in neo4j gem to the neo4j-community gem.
I understand not adding the model twice but why throw an error here?  Couldn't the duplicate model just be skipped?  Has there been discussion about this before elsewhere?
Date is not optional. Otherwise, when would the notification fire?
Either no punctuation or an exclamation marks. Exceptions get exclamation marks.
awesome -- nice to wrap this up in a model method :+1:
It shouldn't break anything, rather it should ensure that opening folder contents can work. When using plone.app.toolbar its likely people will have a custom diazo theme that won't take account of the complex plone UI (theming should be simple after all.)

Perhaps some work will be required to make overlays look a bit nicer when opened in a new tab. One idea from CMSUI (not implmented yet) was to have some javascript that detected an overlay intended view was open alone and redirected to a link that opened it as an overlay. Perhaps it would be better to rewrite these links in javascript to use fragment identifiers, something like:

    http://site/context/folder_contents -> http://site/context#!folder_contents

I'm not sure how far we got implementing these fragment links, but being able to have a link that automatically opens an overlay will be useful.
Hi, @programmin1. This is great.

Killing code is one of my favourite ways to change the code. :)

Just as a hint for the future, instead of just commenting out code, simply remove it, putting an extended explanation on the commit message. This makes it easier to:

1. Read the code latter (fewer, non-tricky lines are usually easier to read than more lines).
2. More importantly, if the removal is properly documented in the history, we can simply revert a commit with `git revert`.
In this method, shouldn't we also check:
 if (!config.Configurer.HasComponent<IDocumentStore>())

So we will not override it?
It looks like email is used if you're creating a customer on stripe:
https://stripe.com/docs/api

But the description seems to be used for charges.
if (type != POINT_MOTION_TYPE && type != EFFECT_MOTION_TYPE || point != POINT_FROSTWYRM_LAND)
one question @kandera this solve the issue of the go dissapearing when you move to a different grid?
I think it will remove ...
Surely dev requirements?
upstream has to be fixed yet
對，我剛剛 make html 在本地可以。
readthedocs 就不行 T___T
Ok seriously, you couldn't have done this in a loop? :P
I know, but I haven't found any info about any of those ap_bonus you give as I said before (4.0.6a addons, wowhead, wowwiki, wowpedia, patch notes). Would you please can link me at least one of those ap_bonus information?

Sorry the insistence, but I'd really like to know from at least one of those numbers came from.
This will only replace the spell ingame,still casting Moonfire,i guess we gotta apply the 333 aura.
https://github.com/loliponi/Long-Spring-Text/wiki/Todo-and-Goals
Fully but not really fully. Visual effect of the Quake (blue light lines) is still missing and I dont really know how to do this. Quake spell targets AoE with visual effect, so maybe there should be spawned some dummy npc's for targeting of this spell, they should form the lines...
You have to keep this so some of the tests don't break.
email_1_5.txt - what is email client name?
What's the function? Restrict certain people to certain SN's, or none at all? Clever. Very clever. Some might even say... Machiavellian.
true, but for d6 this would be a regression was my initial point
That's hardly styling - changing the number of default rows. Should be a separate commit.
s/shit/shift :)
Not all browsers have a console object. Production sites do not need any logs.
Good job, this fix is working.
![i teach my kids to be accepting of other browsers as long as they are firefox](http://i.qkme.me/3647i5.jpg)
Oeps verkeerde haken
Then your wrongly updated QQ i got
wow-update-base-15211.MPQ
Thanks a lot for providing Licence Info.
I like what I see
I think it's better to be a bit more lenient here, and let users to the specific version pointing in their app composer.json.

But it would be good if @Seldaek would throw together a few best practices.
Hi,

Awesome to see you've been playing with fluent-dashboard's code. I'm curious what apps you've used these icons for.
If you have something usable, I'd love to merge it back into the original repository.

Greetings,
Diederik
I can say as I do that? I have the xampp
Son of a!
revert this commit to fix reboot on charge :)
Then merge that bastard!

On 23 January 2012 10:21, Warren Seymour <
reply@reply.github.com
> wrote:

> It's the same as the 16px png, just converted to .ico for maximum
> compatibility.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/warrenseymour/unified-diff-website/commit/276fab2213d35b5631b61d90db2bedd1b2b7868e#commitcomment-891662
>
Смени `disqus_shortname` на свой.
И в файле `_includes/disqus` тоже
this group id doe snot seem to exist in maven central.
were you refering to :
<dependency>
    <groupId>emma</groupId>
    <artifactId>emma</artifactId>
    <version>2.1.5320</version>
</dependency> 
instead ?
Still broken, now with undefined methods
just want to try xD
Wait, what?  There is a destroy hook that gets called on page switch?  That is fscking grand - had no idea this existed - and yes, this looks about right - I'll play with it a bit tonight and make sure that there are no leftover orphans - I think the only thing I can imagine might be left over is pickerContent, but that should get nuked with the page (or the dialog) - um.  Does this need a hook for "leave page in cache"?  Or is jQM doing that?

Thanks!
~j
Since this commit, I get this error:
The service "sonata.admin_doctrine_orm.block.audit" has a dependency on a non-existent service "simplethings_entityaudit.reader".

Should I use the audit reader if I use Sonata Admin ? Is there a way to deactivate it ?
So are these :).
Please no bare exception handling.
As the "text" string is computed for output (see the "return" below), I think it's easier / better to log it directly.
sorry, this was not an actor
no, because in that case we would abstract over the things we create ;-)
By inlining these throws, you just reverted an earlier review comment :) I'm fine with your changes though.
it is not obvious how this could ever be false; maybe explain the mechanics of this test driver in its class comment
default supervision strategy restart, so they both get restarted, and lose state, so that's not good. I'll add a Resume strategy for all NonFatal errors.
does the nullness matter?
yes, how did i miss that!! updated.
Yeah, wasn't sure about that one, compose by itself doesn't sound right tough. I haven't actually used dataflow yet, so I'm not sure how to improve that.. may be "composition"?

But the other changes are good.
Well, don't. I don't think people are using it, and it might be outdated :/ So remove that change, and you can push.
This will cause a 500 Error on apache - what are you trying to achieve?

I don't like the look of the exception substitution - maybe it's the best approach but not sure
Hmm.. I'd rather see that you add this to the place where RenderSoftware is used and without the ifdef.
On this note.. My thoughts here has always been to separate m_decode and m_render. dvdplayer decodes, calls flippage, it signals app thread new frame, but doesn't make the new frame available for render yet. FlipPage blocks until a new decode buffer i available. 

This mean that app thread can keep rendering previous frame until it request a new frame. When it request a new frame it check if a frame is available after decode. If there is, it swaps this in and make the previous render frame available for decode (this wakes up decode thread that was waiting for a buffer).

In a sense it's a buffer system where each buffer can have 3 states. Idle, Used for render or used for decode.
This is still wrong. Seems you didn't fix this up.
Double quotes only, please. :)
I do like this way better, and I think we can make the other usage cases more like this ...
btw, what's the difference between a Scheme and a SchemeIdentifier?
what's the point of this? if it is to measure freshness in order to display file list from freshest to least fresh, wouldn't it be better to have an ordered array, and just bring a file to the front of the array when it's been touched? that way, the list is always from the most fresh to least fresh.
Perhaps defining magic numbers isn't the way to go.  A const static CGFloat would make more sense here.
It's a large and expensive process to convert to a CTLineRef, truncate it, and convert it back to a string to draw. This seemed a lot simpler, as much as it doesn't have as many features and reusability.
in createentrypoint.cmd.php, the name is totally use less. see the comment on it. so, just return a path
it seems this array element is not used at all. why to create it ?
Outdated now that we bundle openssl?
Pourquoi?
This should probably check for `.html` instead of just `html`. (ie, include the `.`)
Proč to je pod latte, když to s latte nesouvisí? Mění to hodnotu Nette\Utils\Html::$xhtml.
Because it needed to be done by hand
Plus remove their declarations (which will be automatically handled), and add a comment saying NUMPY_API above their definitions (see other API functions for examples).
лучше убрать инклуд
Cool, I'll make the change.
@carlosantoniodasilva Maybe not here because this is fixing an issue and it will be easier to backport.
I meant that it went into a loop when running tests right after the commit that introduces this deprecation. I haven't tried to call it in different ways. It happens in `/rails/activerecord/test/cases/fixtures_test.rb:603:in 'test_cache'` which calls `rails/activerecord/test/cases/helper.rb:85:in 'create_fixtures'`, where `ActiveSupport::TestCase.fixture_path` is called (i've deprecated `fixture_path` in favor of `fixture_directory_path`).
This chained and one-thing-per-enumaration code looks much better for me. Since there are usually no more than few order clauses, the performance difference should be negligible.
Just using the same syntax as the similar code below it.
It's not super specific to this code because it's all over the place, but: how can it be right to have code which handles AnyRef and Object specifically, but then nothing else? If I define "type Bippy = Object", are the Bippies going to be treated as Objects? If so, then why isn't that good enough for AnyRef? If not, then why does AnyRef get special dispensation?

So, depending on the answer to the above, maybe that should be "tpImplied =:= ObjectClass.tpe" rather than "(impliedSym == AnyRefClass || impliedSym == ObjectClass)".
I do understand why it is just easier to throw an exception, but I also believe that throwing and catching exceptions is as hard to understand (or actually harder) than the checks that you would introduce. There has to be a better way.
Maybe better than using the where clause: ``num[z == 0] = 0``, ``den[z == 0] = 1``?
marker for Cadair.
On 11.06.2012 15:40, Christopher Smith wrote:
>> +fields. But it is perfectly legitimate (although in this case uninteresting)
>> +to factorize over polynomial rings such as `k[Y]`, where `k` is some fixed
>> +field.
>> +
>> +Thus the polynomial manipulation algorithms (both
>> +complicated ones like factoring, and simpler ones like addition or
>> +multiplication) have to rely on other code to manipulate the coefficients.
>> +In the polynomial manipulation module, such code is encapsulated in so-called
>> +"domains". A domain is basically a factory object: it takes various
>> +representations of data, and converts them into objects with unified interface.
>> +Every object created by a domain has to implement the arithmetic operations
>> +`+`, `-` and `\times`. Other operations are accessed through the domain, e.g.
>> +as in ``ZZ.quo(ZZ(4), ZZ(2))``.
>> +
>> +Note that there is some amount of *circularity*: the polynomial ring domains
>> +use the level one classes, the level one classes use the level zero functions,
>
> should level one be "L1"?
>

I don't know ... should it? The notation is from 
http://mattpap.github.com/masters-thesis/html/src/internals.html which 
seems to use long and short forms interchangeably.
Yes, cf. #1667.
`S(1)` ? [or does `inf` do this automatically?]
> 
> Try doing the following
> 
> ```
> import random
> random.shuffle(terms)
> ```
> 
> and run the tests a few times.  This will hopefully tell you a little more about what kinds of errors can happen, based on functions that expect certain arg ordering.
> 

All tests passed:
https://github.com/goodok/sympy/pull/2/
I just read the whole thread again and I'm contradicting myself. Sorry, some clarity:

1. LaTeX in doc strings (plain text) is hard to read, I think it is a good idea to minimize it or eliminate it.
2. For a grander vision of the package which would include many dynamics methods and support many workflows, it would be nice to have the general functionality to not be so Kane specific. For example, Angadh is going to add the Lagrange method this summer and there is much functionality that can be used with the Lagrange class and the Kane class. The less the general functionality is tied to a specific method, the more users may adopt, use, and extend the software.

The second point is not what I want to get into in this pull request, or maybe ever for that matter. So just ignore that for this pull request discussion as it is derailing. But I do think point 1 is relevant.
This should be put back because MatrixExpr should not have the `__getitem__ -> _entry -> __getitem__` loop
Gilbert, Angadh, Colin and I discussed the name issue.  We came to the conclusion that "point mass" was more generic and could represent a particle, the mass center of a rigid body, or some combination thereof.  So, in the interest of being more generic, we felt that inertia_of_point_mass was more appropriate.  Additionally, this way the user only has to have 1) a sympifiable expression for the mass, 2) a Vector, and 3) a ReferenceFrame.  Items 2) and 3) go hand in hand (you need a ReferenceFrame to make a Vector), so we felt this approach was the most minimal and impose the least restriction on the user, and therefore the most desirable.  Requiring the user to use the Point or Particle classes might be more overhead than is needed for from some common use cases of this function, so we felt it was best to stick to the more minimal approach.
yes, thats how I implement it. If subclass doesn't specify its lense with ```lens "somelense.lns"``` then it raise exception. So it is attempt to catch programmers errors.
where does the string `"animation"` come from? why is it in `x[1]`?

for the id of the div, you can replace all non-alphanumeric characters with `_` in `options$label` which should be unique enough
The maven plugin needs to be updated then :D If the new version isn't out,
that not a big deal, I was just wondering

On Wed, Sep 5, 2012 at 10:49 PM, Brandon Donnelson <notifications@github.com
> wrote:

> In pom.xml:
>
> > @@ -193,7 +193,7 @@
> >
> >          <aopalliance.version>1.0</aopalliance.version>
> >          <cglib.version>2.2</cglib.version>
> > -        <gae.version>1.6.1</gae.version>
> > +        <gae.version>1.7.0</gae.version>
>
> I tried with 1.7.1 again and I get unpack errors.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ArcBees/GWTP/pull/7/files#r1540716>.
>
>



-- 
Christian Goudreau | CEO - Président
M: 1.877.635.1585 | S: christian.goudreau
ArcBees <http://www.arcbees.com> | Blog <http://blog.arcbees.com> |
Facebook<https://www.facebook.com/QueenOfTheBees>|
LinkedIn <http://www.linkedin.com/company/arcbees>
Was the IOR already squared and just the var name was wrong, or did you change the behavior to fix it?
Why add a try_files here?
There is good chance this array is allocated on the stack with escape 
analysis... nevertheless it is possible to optimize by allocating it 
during a format change or by simply allocating one that is large enough 
to hold all planes (I think the maximum amount of planes is something 
like 4 with exotic formats).  Instead of planes.length one must use 
getPlaneCount() then in the loop.

I don't think it is worth it though, but perhaps it can be rewritten 
slightly.

On 8/09/2012 14:20, Mark Lee wrote:
>
> In 
> src/main/java/uk/co/caprica/vlcj/player/direct/DefaultDirectMediaPlayer.java:
>
> >                   Logger.trace("lock");
> >                   // Acquire the single permit from the semaphore to ensure that the
> >                   // memory buffer is not trashed while display() is invoked
> >                   Logger.trace("acquire");
> >                   semaphore.acquireUninterruptibly();
> >                   Logger.trace("acquired");
> >  -                plane.setValue(nativeBuffer);
> >  +
> >  +                Pointer[] planes = new Pointer[bufferFormat.getPlaneCount()];
>
> This method is invoked every frame, is it possible to optimise this to 
> not keep re-allocating this pointer array? Would it still work if this 
> was created in the setup callback? I think it's pretty cheap to 
> allocate that array these days, but every scrap of performance counts 
> for video playback?
>
> Don't submit a new patch, just interested if you think it is worth 
> optimising that or not.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/caprica/vlcj/pull/93/files#r1560560>.
>
Not in the scope of this refactor
required for the LOPackage to work
A valid point :) I've pushed all updates, thanks for the valuable feedback!
check_permissions can probably just happen once before the path split to consolidate a bit.
There's no need to run this replacement again if this is restructured a little bit. The replacement can be run once, before the truncation, and access to the result kept around for the expansion.

Something like

```coffeescript
      excerpts
        .select('blockquote').each (d) ->
          chars = 180
          quote = d.message.annotation.quote.replace(/\u00a0/g, ' ')  # replace &nbsp;
          if quote.length >= chars
            d3.select(this)
              .html(quote.substring(chars) + "<a class='ellipsis'>...</a>")
              .on 'click', ->
                if d3.select(d3.event.target).classed('ellipsis')
                  d3.select(this).html(quote)
          else
            d3.select(this).html(quote)
````
You mean TargetConditionals.h always defines TARGET_OS_IPHONE? Looking at my copy of TargetConditionals.h that does indeed appear to be what's happening. In that case, okay, but can you fix up the style issues?
Shouldn't that be `static` instead of `UV_EXTERN`? Those functions aren't used outside of process.c.
standardising on plural form?
I think this class could do well with a little bit of documentation as to what it does.
all distance values
documents with a coordinate pair that is within...


also things are within the radius *of* something, not *from* something.
the fact that mongodb creates a collection implicitly upon use is not noteworthy.
A matter of preference, I guess. I'm thinking of the stubs here as setting up the context of the test, that is creating the truth of the alternate reality in mock-land for the tests. The code can call `Dir.pwd` as much as it likes, as long as it gives the right answer at the end.

Alternatively, since using mocks ties your tests more tightly to the implementation, one could argue that they should be as strict as possible, so that the tests will fail (and then get updated) if the implementation changes.
No space before ().

I would like to see the anonymous function brought out and named, and if you want to pass arguments into it use curry or the jquery equivalent of it.
For the record I favor doing the extra query to determine post count in pruning - correctness first.
Зачем этот файл? Если делать поддержку apache, то делать полностью, а не какой-то непонятный кусок
I would recommend adding `else` section putting into the logcat that the test case is disabled!
remove empty else block
I've just moved this, not removed it.
The exception is not thrown if mime type is not found, the exception is thrown when none of those functions exist, so it's possible that there is no exception thrown and the return is empty.

I meant for it to serve a different purpose, a possibly misconfigured server.
This file should be into the tests subfolder, named test_sharedvalue.py, and have a function (for instance named test_shared_borrow) to run the test.

Also, I don't think we can use a fixed shape size for this kind of test, if we want it to pass on all computers.
I would suggest something like this (it's a bit more complex to implement though):
- start with an array using N bytes in memory
- try shared with borrow=False, catch the exception which is raised when a MemoryError occurs (as Fred said, we shouldn't use sys.exit)
- if the exception doesn't occur, free memory, multiply N by 1.5, and try again
- if the exception occurs, try shared with borrow=True: this time it should work. Then end the test.
If I understand correctly, this function produces the information that would be contained the `.clients` attribute of the variables, if these variables had been imported into a FunctionGraph.  Is that true?  The .clients attribute is formatted slightly differently: `var.clients = [(node, idx), (node2, idx2), ...]`

If so, then this function does a useful graph analysis and it not particular to grad() at all, so how about moving it to graph.py? Rename it to be clearer (`clients()`?),  and add to the docstring that it's similar but not the same as the .clients attribute FunctionGraph puts in. There is room for refactoring with other methods in graph.py, but that can easily be another PR.
\w is not quite the same as what HSQL allows for SQL symbols. HSQL allows $, but doesn't allow numbers at the beginning of the symbol (without quotes?).
Drop obsolete comment.
The opposite version will be more natural: is_mac_address_known :-)
reserved_names constants
FYI I realize it's out-of-scope for this PR but in all other places we tend to use the newer `-spec` rather than the `%% @spec` form. The origins of this file are old enough to have used the comment form, so let's consider that a janitorial task for later.
I understand the use of the capital here as it becomes part of the framework, still question it's usage as part for arguments and internal variables
Yeah that's a good point. I think I was leaning towards that with IndexSet. This could be just a specialized  header instance.
Also, complete and always callbacks do NOT have the same signature so you won't call with the arguments in the correct order.
Should we drop Chrome and just mention Safari 5.1 now?
Sorry, I tend to deal with a mixture of 2,3,4 spaces and/or tabs for indenting between JS and Perl.  Drives me crazy...
Does accounts-ui still work now that we don't guarantee that password is last? As far as I can tell, you removed this comment and related functionality without replacing it with anything equivalent.

Alternatively, we could refactor accounts-ui to not require password to be last in the list but just to render them completely separately.

BTW, a helpful way to test changes to accounts-ui is by using the accounts-ui-viewer app in examples/unfinished/accounts-ui-viewer.
>That seems pretty straightforward and the differences between services doesn't seem to justify a special API (which would end up very complicated.)

Agreed.

> 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.

Adding multiple accounts for the same service is actually another use case that I have. Suppose you have two twitter accounts and want data from them both?

@possibilities @avital Thanks for your time!
So, for the things I mentioned (well, mostly for the "if not tld.tld or tld.subdomain"), I think we want to write a test case, to make sure it's doing what you think it should.  (Or write a full test that exercises that case.)
@jacknagel Thanks a lot for you advice, I split the string into strings to make it slightly more readable in the following commit. But this still has to be in one invocation. Otherwise, the values of variables uid and gid will be lost if I put them into different invocations.
Check the indentation here.
@n1k0 removed because not part of *.coffee and not used because init to default value

```
upTo = ~~casper.cli.get(0) || 10  # max 10 links
```
I guess some code should be factorized here =)
Is this expected to ever happen?  If not, raise a specific exception.
Not afairc. I'll correct, test and push.
We should probably put the version check at the top of the function. Doing the check every row will probably be fairly wasteful for large result sets.
Das schätze ich anders ein, ich denke, dass man meistens "de" haben möchte. Bisher hat man ja oft, da es kein extra Feld für die Sprache gab, direkt das Kürzel als Name genommen, damit zum Beispiel in den URLs das Kürzel erscheint und nicht der volle Name. So heißen zum Beispiel die Sprachen auf redaxo.org im Backend "de" und "en".
Mir ist noch keine Seite untergekommen, wo "de_DE" eingetragen war. Es wird sicherlich vorkommen, dass man das braucht, aber nicht die Regel sein.
You're looking at the wrong code block.  I just meant something like:
```
my $time_violation = ($h_time[-1] >= $bs_arr->[0]->{time});
```

Any downstream code that uses `$time_violation` should use it only in the boolean context (i.e. don't compare to 0 or 1, just test true or false).
Shall do.  Mostly playing with travis config at this point - will 
probably split that out.
This was unnecessary! we do the same thing a few lines up.
Allocating here will result in a memory leak if `destination` turns out to be null.
For pgm and epgm transports, the above condition is always false.
Is this what you mean?
I think it would return a null which would pass a null into the copy constructor. NOT WHAT WE WANT. 
Good catch!
I'll fix this before the merge
added in new commits
That's ok. So what do you prefer? IOException or an IllegalArgumentException?
Please don't commit commented code!
There is a lot of duplicated code in there, couldn't this be handled by a helper function that works for double, float and int and the if/else just calling that function with the right parameters?
hmm, it should only really have caused an issue if someone was connecting over and over and even then, it would probably be ok.

Were you just connecting with one client?
On this line. Overall your formatting doesn't really fit conventions, but I guess it's fine for now since this is a pretty important feature.
should be 'storedTables'
Yes. Otherwise, when hitting CMD-S, Chrome will attempt to save the current website. This prevents that.
missing jslint ``$`` global
state var is unused
Anyway, I'm good with pushing this change to the next sprint.
These CSS rules should be scoped with a descendant selector like the ones above -- as is they could easily break stuff elsewhere in Brackets.
https://github.com/ajaxorg/cloud9/blob/master/plugins-client/ext.language/refactor.js#L89 yells "WHAT ABOUT ME!?"
It error'd without. We use observers a lot more than the guy that wrote the gem. His was just broken.
Thanks for the solution, I think this is better than mine. I didn't thought about the validation before.
Isn't this wrong?
命名方式還是盡量用小寫+底線
Nitpick, but why the auxiliar value. We can return this directly.
This is not good. If  there is no `id` in `fb` there should be some error, not just silently try to store `None` (the default) into the database. And same for others.
We should probably not have this in our user-facing templates.
This comment is no longer in sync with the code it precedes, please update to reflect what's really happening.
We should set this to "allow-scripts" so allow JS to run in the iframe.
My report just reads `Popcorn=@VERSION, Butter=@VERSION@`. Is that wrong?
Python's 'platform' module provides a lot of details about OS.
http://docs.python.org/library/platform.html
platform.system() returns 'Linux' or 'Windows'
-1
shouldn't this protect against null ? Also, please use the goodies helper class instead and stop using String.format
It's a typo.  It was supposed to be 'not uncommon', and I'd prefer not to change it to 'common' since I don't have any hard data on what fraction of drives behave this way.  Let's go with the suggested '...because drives are sometimes known to misreport...'.
doing a full rectify-zone would be expensive; an incremental version would be awesome...
You can actually remove this  self.index_settings.  These are automatic within the AR models, and this method isn't actually used.
In our Pulp V2 migration branch, we were able to do the following to test the pure Katello model without having the glue code present.  Which might be helpful, or worth discussion.

    AppConfig.use_cp = false
    AppConfig.use_pulp = false
    AppConfig.use_elasticsearch = false

    Object.send(:remove_const, 'Repository')
    Object.send(:remove_const, 'Package')
    load 'app/models/repository.rb'
    load 'app/models/package.rb'