labofsahil/github-event-dataset-2012 huggingface dataset:
Updated README to fix bullet lists
Fix IE linebreak vanishing problem

It should really closes #7 . Tested with IE7, Firefox 15, Chrome 21, Safari 5.

It can cause conflicts with other PRs, merge it as last.
Update installation instructions

@ppawel - I updated installation instructions as I went.

- General review and cleanup
- Add more command by command instructions
- using postgres 9.x as an assumption, pointing to other resources for 8.x
- Instruct to use osmosis fork
Style update

Aligns equal signs
kyrgyz translation

updated
Update docs/languages/en/modules/zend.db.adapter.rst

Removed duplicate text 'not generally required'
Added Multicannon Powerup

Adds Multicannon Powerup and capabilities. Multicannon adds to cannonballs to the ships projectiles when the space bar is hit.
Fixed broken links in sidebar when using __label__ and/or __plural__ attributes on Model (sub-) classes.
Bug fixes

in RESPONSE_HEADERS:/Set-Cookie2?/ but did match in REQUEST_COOKIES.

Also there was a bug related to comparisons on collection keys that do
not exist. When a request contains a cookie that has not been saved to
the SESSION collection before, the intention is for tx.anomaly_score
to be incremented by 5 (critical) and the rest of the checks skipped,
but this does not happen. Any test on a collection key that does not
exist always returns false. This means the test on SESSION:VALID "!@eq
1" returns false, when the intention is for it to return true if the
session cookie has not been seen before.  And the following rules in
the block are run which triggers 981059,981060,981061 to return true
since it is a new session collection withou ip_hash or ua_hash keys.
Build 1.1

Fixed more problems.
Move initialize and run into the same line
Update Java/DocuSignSample/readme.txt

Ant will not treat properly as absolute path.  
Ant (even on Windows) expects forward slashes as path separator.
Htmlhelpers glyphs (work in progress)

Created html helpers package; please review and check if this is what you had in mind.

I've created the nuspec file and added it to the build script.
I also added this html helpers  package to the sample package (so that I could easily test with it).

I'd like to do some more testing but at this stage feedback is already appreciated.
Luupit
Step 2: Title the box

This is stuff in the box
Support localization within a TextTemplateRenderer output - Report 443

Support localization within a TextTemplateRenderer output - Report 443
Change fenced code block to non-greedy regexp

Fenced code block was greedy by default, so if you have multiple fenced code block, everything in between will be rendered as part of it. Changing the regexp to a non-greedy one fixed the problem.
Fixed #3: "Could not find init script for postgresql-9.1"

The postgresql service name on Ubuntu doesn't depend on the psql's version now.

P.S. I used Postgresql from the `pitti/postgresql` ppa source.
Updated form set default options to 2.1
Make any blank entries nil during an update

This allows values to be clear by passing an empty string ('').
Hhh 7009 4.0

Create dialect for Postgres 8.2 and later which supports "if exists"
in drop statements. Update resolver to return the correct dialect
version.
Merging unstable into master
tweak img css

I'm guessing this is a typo and it should center the images.
Exposed sprockets_environment from Jasmine::Headless

* Allows for manipulation of the Sprockets::Environment for things like adding/removing pre/postprocessors from within spec_helper.rb (more flexibility than just registering engines).
* Also, fixed specs that were failing on files_list.rb whenrun standalone due to missing require in file_checker.rb
Ensure uniqueness ("slug", "slug-1", "slug-2" etc.) when setting slug manually.

RE: [@al's comment on #43](https://github.com/hakanensari/mongoid-slug/pull/43#issuecomment-4194931)
Updates to Concordion.NET and Fitnesse examples

The Concordion.NET changes should get the example working for you.  You'll need to get the build to copy the concordion config file to the right location.  I just did it manually, and figured you'd know how to do it in MSBuild.  I also updated the README for concordion.net.

For fitnesse, I found the location was case sensitive, so have made a small change to the install.sh.
Add an error for unsupported language features, and print it for indexing upon a binary

We'll need to add support for this feature soon (it's used in Part 50), but the error message can be reused.
SNS message handling

* Introduced `AWS::SNS::Message` to wrap [raw JSON SNS messages](http://docs.amazonwebservices.com/sns/latest/gsg/json-formats.html).
** To-do: support the different `canonical_string` for SubscriptionConfirmation and Unsubscribe SNS types
* `Message` also deals with crypographically authenticating SNS messages via [the method documented by AWS-SNS](http://docs.amazonwebservices.com/sns/latest/gsg/SendMessageToHttp.verify.signature.html)
* `Message` also builds in (crude) support for extending itself based on the originator of the message.  The idea is that there's one mixin per originator.
** `FromAutoScaling` is one such originator, and partially wraps the SNS data that auto-scaling writes into the body of its notifications

To-do:
* Cucumber-based integration tests.  Need a steer there, since the only thing I've come up with is to for-real create a `Topic`, and then `subscribe` a newly-created `SQS` to notifications and assert off them.  I imagine there must be either better options for a test-harness, or some way of providing credentials so that the developer doing the work can put them somewhere to be read and all the tests then run off of those...?
* The JSON that the specs use link against real resources (I think; they were created weeks ago now and to be honest I've forgotten whether those still exist or not), and probably either shouldn't, or should link against AWS-owned ones so that they're less likely to ephemerally disappear in a puff of cloud :-)
Add link to github repository in README
fixed a typo

Added a semicolon to the end of a font-family declaration.
compatible with Vagrant 0.9

Made vagrantboxes compatible with 0.9. Also modified the command to be "boxes" instead of "vagrantboxes", which seemed to me to be redundant to type "vagrant" 2 times in a row. So now the command is:

    vagrant boxes search

Looking at your last commit time, i'm wondering if the project is still active? If it's not, do you mind if i'd start maintaining it myself?
Twitter URLs should point to HTTPS now that it is the default
Added MFFSDamageSource class and some sources
Optimized the repo to support kpm

- Added package.json-s
- Added changelogs
- Renamed readmes
- Separated snippets from plugins

**Important**: There's still a README missing for the `color` plugin! Would be nice, if this was added. :)
MIFOSX-60 added support for direct loan charge create and remove methods...

Support for add and remove loan charges directly.
Need to extract a Samsung's Audio HAL file (audio_policy.exynos4.so) needed for the microphone to work
Styling improvements for printing.

* Improved printing of popups
* Improved printing of single location static page
* Improved printing of comparison static page
* Improved printing of directions
Issue 105 image node

@phtaylor: Converted the Texture node to Image node, Image to Image2D.

 options.wantRGBA +  options.wantHDR have been moved to options.format, similar as in the Image3D node. Furthermore I have moved the Image2D pixels dgnode to be sliced, and added a uniforms dg node for the metadata. All operators have been adapted, and tested all demos using images, including the Animation review tool (since it modifies images) and video.
VirtualBox Package

I had this in my own [repository](https://github.com/mdellanoce/chocolateypackages), but I'm doing very little Windows development these days and I'm worried I won't be able to keep up with updates. I'm wondering if your repository might make a better home?
bierta
add offline build script

We have the assisted digital in another repository - and soon more content will end up in its own repository, away from the code (which is good).

However this means to test a change to the Assisted Digital paper, you have to.

1) Change the stuff in the AD repository
2) Push this live
3) Run `./local-build.sh`

Sometimes you want to be able to edit the AD content you have locally, without having to push.

So this adds `./offline-build.sh` which builds purely from what you have at that time within `source/`, and doesn't pull anything in from the internet. This is also useful if you're on a train and working, or something.
Add git-tfs integration and a "commit and test" build button to the pend...

...ing changes window
Upgraded for new bundle format.

Here ya go!
Add setting for option depth

Control how deep the select box gets rendered.

$('#nav>ul').mobileMenu({
	switchWidth: 480,
	depth : 2 // Currently defaults to 1
});
Add missing 'typeof' to fix html5 mime check

I think this makes sense, please let me know if I am crazy :)
TIMOB-7694 Null check on table view's native listView before accessing.

https://jira.appcelerator.org/browse/TIMOB-7694

test case is fail case.
[TIMOB-7182] Create new session ID when resuming app.

See JIRA ticket TIMOB-7017 for the testing instructions. QE vetted the fix before the pull request was submitted (check w/Natalie) so there may not need to be a functional test from engineering.
32bit x86 support post-ARC via GC + static lib

This fork had two primary goals: re-instate fragile ABI compatibility without losing ARC, and create static library targets for macos/ios. I also got a bit OCD along the way with warnings, you'll see what I mean in the commits (implicit conversions, implicitly atomic properties, etc). There's also a commit that fixes global static variable shadowing that I don't think was intentional and may cause some debugging confusion in the future.

The xcode configuration is in xcconfig files pretty much entirely for readability without having to fire up xcode. This is how it's set up to use ARC everywhere but 32bit x86:

```
CLANG_ENABLE_OBJC_ARC                               = YES
CLANG_ENABLE_OBJC_ARC[sdk=iphoneos*][arch=*]        = YES
CLANG_ENABLE_OBJC_ARC[sdk=iphonesimulator*][arch=*] = YES
CLANG_ENABLE_OBJC_ARC[sdk=macosx*][arch=x86_64]     = YES
CLANG_ENABLE_OBJC_ARC[sdk=macosx*][arch=i386]       = NO
GCC_ENABLE_OBJC_GC                                  = unsupported
GCC_ENABLE_OBJC_GC[sdk=iphoneos*][arch=*]           = unsupported
GCC_ENABLE_OBJC_GC[sdk=iphonesimulator*][arch=*]    = unsupported
GCC_ENABLE_OBJC_GC[sdk=macosx*][arch=x86_64]        = unsupported
GCC_ENABLE_OBJC_GC[sdk=macosx*][arch=i386]          = required
```

There was honestly only one case where I found something that needed to be changed for compatibility with GC, and it was a very small change.

My 32bit iMac will thank you. ;)
Mailchimp Webhooks

Improve our existing MailChimp integration #49

### TODO

- ~~Add tests for `MailChimp::WebHooks`~~
- ~~Create cron job to check for new unsubscribed users which were not disabled via the web hooks~~
- ~~Fix account disabled integration test~~
- ~~Explore [exception notification for rake tasks](https://gist.github.com/551136)~~
Allow to define pluralization without spaces

Hey!

Today, I spent a little time to understand why pluralized translations were not correctly rendreded. When I define my translations, I don't use any spaces. For example, my translations looks like:

``` yaml
key: '{0}Aucun commentaire|{1}1 commentaire|]1, Inf]%nb% commentaires'
```

This translation is correctly rendered by symfony but not by the bundle.

This PR solves this issue.

Thanks for reading.
Updated Luxembourgish translation
Marcel arthur

Bonsoir,

Je voulais vous demander de jeter un oeil à mon code pour savoir si ça allait gloablement.
Mais j'ai un petit pb, j'ai remodeler une partie de mon arborescence, j'ai maintenant des doublons anciens dossiers / nouveaux dossiers. J'ai pourtant fait un git add *, puis commit puis push, ce qui d'habitude fait l'affaire, mais là rien. Même les git rm sur les anciens fichiers ne passent pas...
add sidebar block support

Related to GH-2 issue I've opened, here is my proposal to add support for asciidoc sidebar block support.

You also need to add a bit of CSS in  bootstrap/docs/assets/css/docs.css, to improve render of sidebar block title

    div.title {
        color: #5A5A5A;
        font-weight: bold;
        margin-bottom: 0.5em;
    }
 
Here is the online version of examples/base-css-book.asciidoc file modified to demonstrate the new feature :  
http://laurent-laville.org/asciidoc/bootsrap-backend-issues/base-css-book-sidebarblock.html#sidebar_block

Enjoy !
- Laurent
Encoding

Adicionado fromEncoding='latin1' no BeautifulSoup (correios/scraping.py) para correção de palavras acentuadas.
Implementation of async task execution and some bug fixes.
Merci

Merci pour cette traduction en espérant que cela fonctionne
修改中英文间距, 还有一些 CSS 调整

修改涉及首页和全部章节, 还有 README:

* 中英文之间需要一些间距, 知乎上学来的习惯, 我觉得有必要
* 页面使用更多的代码标记, 修正反引号的标记... 不知道这里用的是哪一门标记语言?
* 修改一些小写的 `haskell` 为 `Haskell`

没有修改但在考虑的:

* 是否要全局使用英文标点,
  * 因为涉及到代码用英文标点更整齐
  * 而且编辑器对英文标点容易识别, 修改时有一些方便
* 大陆地区一些首页引用的脚本加载不太正常, 需不需要下载到 repo 里?
Alec, is this a problem in this case?   Further down in the code I use the publishedMonographDao to specifically retrieve the monograph associated with that format, but restrict it with the pressId.  An export only occurs if the monograph is in the current press.
There are tests for the api in the follow up patch now. Not all that extensive, but there's a good coverage page at least for the network view.
NOTE2: Also, the reason for all the other lines getting modified is that my emacs is set to nix extra whitespace before line endings.
Why was this changed? It's been beautiful to have clean folders in B3 even if folder in filemode was not in library.
There's now an example in the documentation, lets see how it goes. https://github.com/kraih/mojo/commit/5dc2d5fa17913b783455b42d4840797ff3898bc3
This is my urls:

```python
urlpatterns = patterns('',
        url(r'^admin/', include(admin.site.urls)),
        url(r'^api/', include("api.urls")),
        url(r'^print/', include("api.print_urls")),
        url(r'^lib/', include("lib.urls")),
```

mongrel2.conf part:

```python
django_handler = Handler(send_spec='tcp://127.0.0.1:9997',
                       send_ident='34f9ceee-cd52-4b7f-b197-88bf2f0ec378',
                       recv_spec='tcp://127.0.0.1:9996', recv_ident='')

routes = {
    '/api': django_handler,
    '/mf': django_handler,
    '/print': django_handler,    
    '/lib': django_handler,
    '/admin': django_handler,
```

When i request "/api/user/info.json", django can not resolve url, because PATH_INFO == /user/info.json in this time, but django need full path in PATH_INFO
https://github.com/radar/forem/pull/162
These change does not seem to be reflected when installing sip.js using npm.
Somebody please reward this man for finding a vulnerability...
Fair enough. Thanks for the explanation. Interesting plan. :)
QtTesting_DIR is used in Slicer/CMakeLists.txt, maybe it's not useful. 
Again, it should be decided if it makes sense to separate the "build" location from the "install" location. If it doesn't, then QtTesting_DIR is not useful.

Yes the topic is just here to be cherry-picked.
only 'raise QemuIOError' should be here.
Why have this instead of just one description as the __doc__ attribute?
I think I failed...
Will investigate that. AFAICS, regex works like a charm!
Hahaha! This just happened to me. Super job on the dedication from a new MaNGOS user!
Reviewed can now be merged
注意代码的格式化
Note to self: Fix the bug before pushing.
Oh no, you've picked up the bad habit of using a "..." commit line.
material.onUpdate(world, x, y, z - 1);
typo in commit message: 
added dev::internal::fast_delegate<> - variadic template version of FastDelegate library.
i did this.
This reminds me how much I hate this definition:

```
error(e::Exception) = throw(e)
```

Why does there have to be confusion over whether exceptions are thrown with `throw` or `error`? If you rethrow an exception using `error` without knowing the type of the argument, it might or might not wrap the argument in an `ErrorException` before rethrowing it. This behavior could be called `errorize()`, since it makes something into an exception if it isn't already. Seems strange to me. I think I'd be willing to horse-trade you 2-space indents in exchange for this ;)
No, the OF controller is now part of hva, and can be enabled with the option 'config.edge_networking = "openflow"'.
Next time we should import from somewhere (we have the user/pass already in remora's config).  Putting passwords on the CLI isn't good because they stick in .bash_history and such
good call. this inline function is not as performant in some js engines and is harder to understand when reading.
"The cpuflags-test program should be active after migration and it's not"
I recommend doing a bitbake pseudo-native (actually, the target doesn't matter, as when it's skipping the task executions, the bitbake wrapper in oe-core will error out after the first attempt to bitbake pseudo-native), then using the browser at https://gist.github.com/1861549 (modified version of the interactive part of pstats.py from the python source) to browse the profile data. Since this commit emits separate profile data files for the server, ui, each parse process, and each task worker process, you can pass all of these, or any subset of them, to the stats browser on the command line to get a view of whatever portion of the bitbake execution you're investigating.
are those all spells that I pushed to ICC branch? can I delete that branch?
frogs rules!
(shrug) - just do not have to check these points for all creatures, need only for players ...
This doesn't seem to do at all what this comment says.
teste de comentario sobre o commit
lamento no haber revisado esto antes T_T de aqui en adelante esta mal todos los says, de aqui como de reisa.

Para hacer esto, tienes que usar el #DEFINE asignandole a un numero (casi lo mismo que en la db), y aca abajo tomando el numero por un SAY_*****

Creo que esto es mejor revertirlo :(, ademas de ley toca meter lo que es loot y asignacion de guid.
readibility and editability > formatting guidelines
http://iot.insigma.com.cn/docs/django-1.3-docs-html/ref/signals.html?highlight=post_save#post-save
created
A boolean; True if a new record was created.
Oh that's fine, don't worry about it :)
This breaks for modules in `node_modules` :(

Eg:

var minimatch = require('minimatch');

I believe this is because of module's `require.paths` Array -- the directory of the script isn't there so node won't know where to look for the non-native modules.
Do these latest commits address errors that are generated when parsing imports (previously not thrown)? Or just fixing something related to the 1.2.0 release? Thanks.
Ik stel voor dat we even een nieuwe Repo maken, is wss wat overzichtelijker dan de wirwar die we nu hebben.
All of the world.hpp code for my way:

<pre>class interiorness_equivalence_class {
public:
  interiorness_equivalence_class(tile_contents c):data(c);
  bool operator==(interiorness_equivalence_class other)const { return data == other.data; }
  bool operator!=(interiorness_equivalence_class other)const { return !(*this == other); }
private:
  tile_contents data;
};</pre>

All of the world.hpp code for your way:

<pre>inline tile_contents interiorness_equivalence_class_representative(tile_contents t) {
   return t;
 }
inline bool interiorness_equivalent(tile_contents t1, tile_contents t2) {
  return interiorness_equivalence_class_representative(t1) == interiorness_equivalence_class_representative(t2);
}</pre>

Neither of those seems so much more not-worth-it to me.

Actually, we need to change this name more deeply. "Interiorness"? Interiorness of what? Right now, there's no term for the 'what', and this is really about the 'what'. Right now we're saying "Are these contents equivalent in terms of, uh, whatever concept it is that we care about for the purposes of interiorness?" That's why it's called "interiorness_checking_type" - because THE CONCEPT is the one for which we CHECK when we're considering interiorness. It's NOT the interiorness, it's the THING ON WHICH WE DO INTERIORNESS CHECKING, hence the name. You can't just remove "checking". Your "tile_contents_equivalent_for_interiorness()" is a better name, admittedly, but it's still...

Of course, the underlying concept isn't a real concept, since it's dictated by how we think about interiorness. So I think it's a better idea to rephrase it to NOT refer to the underlying concept... and there are two different ways we use this, one of which requires transitivity and the other of which doesn't... so, I guess...
<pre>bool it_is_okay_for_tiles_with_these_contents_to_be_marked_interior_when_they_are_neighbors(tile_contents, tile_contents);
bool a_tile_can_switch_freely_between_these_contents_without_changing_any_interiorness(tile_contents, tile_contents);</pre>

I'm pretty sure those are the right names, subject to conciseness. And for the moment, they can both just be implemented as "a == b"
Reading through http://zsh.sourceforge.net/Doc/Release/Expansion.html#Filename-Expansion it looks like `OMZ=~+` is the most succinct and correct.
Am I right in thinking that the only possible reason why koExports might *not* be defined is if you're running knockout-raw.js (rather than one of the built files)?
I get  ^M when replacing from visual mode in Vim also.

But when in command mode, the cursor moves down to the first non-blank of the newly created line and reindents it

On Jan 5, 2012, at 12:47 AM, Sublime HQ Pty Ltd<reply@reply.github.com> wrote:

> For me, vim replaces each character in the selection with a ^M. This is with vim 7.3 on Ubuntu with no .vimrc (same thing for me on OS X with vim 7.3, but there are a few entries in my .vimrc there)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/sublimehq/Vintage/commit/f9767c521ec1c25aedec12ce9e6d3948c4b9acb8#commitcomment-839480
A fairly stupid way actually...I wanted to have a "+" sign just like that on Chrome...but didn't have time to look into various widgets.
Works fine.
Thanks..
I guess we should apply for a FlexGet api key?
I would drop all the mouse* events and add dblclick instead
When will Regular Expression support be released to Friendly Id? Any approximate date decided for this? Or is there any way I can do some tweak to my code to support this untill it is released?
I don`t like this !
Creo que no es necesario seguir todo al pie de la letra, si creemos
que algo le sirve y no cuesta nada hagámoslo. Ya tendremos tiempo de
cambiarlo si le molesta.
No, flame rings spawn ten seconds after Halion is engaged, and DOOR_BOUNDARY is only instantly.
I think that [nokogiri's .travis.yml](https://github.com/tenderlove/nokogiri/blob/master/.travis.yml) is a good sample.
Link to my other commit with more info https://github.com/CyanogenMod/android/commit/4949e71916244e8cf82961ffbaceb0bf5b0c6dc3
What are "area" and "fragment"? Is it mentioned somewhere?
I would prefer to have a g_object_unref on priv->recent_log in the dispose function.
Otherwise that means that the connect signals can still fire events on a PengeEverythingPane that has been freed.
I'll have to look at the example I wrote tomorrow (it's in work), and test to see if I can resolve this properly. It's most likely just a case of not using the fnEnd signature properly on my end. (I assume el would play the role of initialTarget?)
nice one
thx.
Perhaps this should be "Python3.framework", actually.
Just filed issue #233, which describes a critical error that may be related to this commit. While I obviously can't say for certain whether the error was introduced in this commit or in a subsequent commit, I thought I'd mention it here just in case Andrea wants to take a look.
Ну вроде как да, давно употребляется в веб дизайне. Можно еще навигационная цепочка, но хлебные крошки помойму лучше. Если применять термин "Навигационная цепочка" то при программировани следовало-бы использовать в качестве обозначения что-то вроде "NavigationChain" :)
Hmm, good points about the ship's ID. I am going to have to play around with this and see if we can find a reliable way of tossing the ID around between tables or storing the initial location somewhere in a  column like.. initial_location or which planet the ship originated from (this may limit future development though if ships can one day be created at somewhere that isn't a planet).
Kroc, please tell me if this is wrong and horrible.
主动选择给什么牌的话，用一个技能卡不就解决了，参考缤纷系列的孔融，让梨的牌就是对方给的
there is a serious warning when compiling
There are still 7 pending test cases. I'll write notify you when passing all test cases.
The green area, that's the area I finally was able to get to work.  Boy now that was some work.  Now, I going to try a function with a if statements.
it does what it says on the doc. It tries to stop what it can during the shutdown. And in the case of OOME you are at the mercy of gods anyway.
Any reason why this needs to be changed?
If this was added to allow chrome to display the content, you can probably stop using it for chrome once we specify the style sheet.
There's another NPE when trying to search for extensions, I didn't get to the bottom of it yet.
Should make it useful and good looking at some point.
Общий. И его больше нет.
Ja das hab ich bemerkt von wegen 4.3.2 core
One big question: WHY?!
![Design preview](http://i.imgur.com/rOgen.png)
Thanks for catching this! Much appreciated.
But it's less code then before :p
J'ai ajouté une constante correspondant à l'adresse commune, parce que je trouvais NB_MAX_CLT + 2, pas très transparent.
Very big commit... If it possible to shrink it to several small commits that will contain a single functional units?
getting the name out of an XMember is expensive. we must do it lazily, only in case an exception is actually thrown and we need to provide the information as a debug help.
This is not a public interface anyway.. it's replacing the role of the original class which was exposing XMember as well.
`@_word`

Actually, the variable name should match the method name, so you can use an `attr_reader`. `@to_word`? It's a little bit confusing, but it's not used anywhere else in the implementation.
Since the window.session object is left untouched and only the session data object is given back to the window.session.start function that wouldn't be neccessary, though I just noticed that window.session.start would be the only way to get to the data, which initially wasn't intended.

Looks like I screwed up there.

I'd go with caching the window.session.start function, clearing windows.session and then putting the results into it.
Yea, I know I should have started the repo earlier. I probably still have work to do on this. For instance, unless you plug it into your computer when you first start it up as well and start the serial monitor, the only way to know what IP it has is to look at the connections on your router. I need to figure out some way to fix that.
I see yes, though it's strange gcc is giving me errors with ine but NOT with _inline_ when "-ansi std=c89 -pedantic-errors" is used...
And from what you say, MSVC actually DOES support inlining but through a different keyword ? What a mess...
Wouldn't it be easier in your MSVC Makefile to define _inline_=_inline if it is what it understands ?

I know about the code coming from MAME (YMxxxx/68K/Z80) defining INLINE macro but I am not going to revert all of them to "static INLINE" like you did when cleaner/faster solutions exist. Also, does that mean MSVC is also so crappy it does not support such things as define INLINE="status _inline" ?

As that point, another solution would be to just force INLINE definition in the code (types.h for example) with #ifdef MSVC or something like that (doesn't Microsoft Compiler defines some kind of pragma to specify code what compiler is running ?)
These are both in there already Beta prep 2/22/2012
Just wanted to confirm - this change means that the SharedMemoryModelBase never actually tries to pulls objects from the cache anymore, and always creates a new instance?  I thought the point of the library was to prevent creation of duplicate instances representing the same underlying model?  Sorry if I've misunderstood, but I can't see how the caching functionality is still in place now.
Just wondering about the purpose of excluded the aliases here? I'm trying to use FactoryGirl to create ActiveResource mocks (where there aren't real associations) so this isn't working:

    ignore do
      merchant { Factory(:merchant) }
    end

    merchant_name { merchant.name }
    merchant_id   { merchant.id   }

Where the idea is:

    Factory(:resource, :merchant => Factory(:merchant, ...))
It is normally best to use a StringAttr in a class/struct rather than a StringBuffer.  It is smaller, and can represent NULL as well as an empty string.

If this was changed you could then get rid of the special casing of "/" and set the value to "" instead.  There would need to be several other associated changes - since it can now be NULL.
Thanks for adapting it! I made some comments on your comments in boundaryfunctions/Articles@d5f05511. To make it short: I think it's a pretty understandable translation, now.
Yop,
Je coince sur un bug :-(
Quand on fait un resize d'un block précédent un autre pour une même série, le block suivant se décale vers le haut...
Si tu as le temps de jeter un oeil, je suis preneur car je ne vois pas de solution.
Line note
Don’t need that comment. Also comments suck, you know.
sory, not being fluent I may not use the adequate tone
urls, refresh, failed
connect, email, no-connection
-> /b2m/
COOL!! I love it :)
能够看得懂就成了，又不是写作文。。。。
Please remove noisy comments
This is the version number that appears at the top of the ref. documentation here: http://nhforge.org/doc/nh/en/index.html
ahhh I meant clearer. Silly me!!!
Daumen hoch von meiner Seite.
I wrote a response this thread: grosser/gettext_i18n_rails#49
hmmm not sure I can do that ... maybe you've sent the email to the wrong
person?

On Tue, Feb 28, 2012 at 3:55 AM, Ingo Schommer <
reply@reply.github.com
> wrote:

> Can you please add unit tests to GridFieldTest for that? Or to keep the
> class/test symmetry intact, to a new CMSGridFieldPopupFormsTest
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/silverstripe-big-o/sapphire/commit/4fb9dc5cb9c0c7885da3f3874a0aaf193483938a#commitcomment-1015250
>



-- 
Felipe Skroski | Creative Director
SilverStripe
http://silverstripe.com

Phone: +64 4 978 7330
Mobile: +64 21 0245 6465
Skype: felipeskroski

*♻ Please consider the environment before printing this email*
+ width: 34%;
Why don't just delete it ?
I don't see how this test is more valid.  Care to elaborate?

For instance your test would fail with "/url?=http://", I chose to do it my way, since it seemed the safest.
Passing HOST_ID in place of cluster id?
I wrote a couple weeks ago about the the infinite scroll: http://danofames.github.com/2012/02/15/details.html.

the implementation I ended up on made simple modifications to jquery.renewal.js itself, adding a config function to prepare the "move style", and moving when the eventAfterMove event was triggered. this seemed to be in line with making the core of jquery.renewal.js simple, but an API to extend.

that change is now pushed: https://github.com/danofames/jquery-renewal/commit/02c3ff9b95413c4fc7d57158dbf7857404b7afa8.

but, the final approach could be improved. the method of extending is kind not pretty.

an infinite scroll, using those changes: https://gist.github.com/1918996.

I hope to re-address the approach to extending jquery.renewal.js in a more straight-forward manner, so that `if (config.infinite)` is not needed in the jquery.renewal.js file itself.
We generally take the view that it's better to have 90% of your items apply, and track the failures, than abort.

We don't have failure tracking yet (Bug 709371), so for now this just logs and continues.
Cool, I figured that looked kind of like ugly Ruby :)
This modification throw an exception now in my project:

Variable "sonata" does not exist in SonataUserBundle:Form:form_admin_fields.html.twig at line 4
That is sooo much nicer!
2 часа сидел переводил, тупая ошибка:) отправил пул с исправлением
I see what you mean. I'm doing the same for "normal" projects but I'm feeling a bit uncertain about whether this should be done in a spec API.
If we want to go that route I think we should use more "neutral" addresses (e.g. `...@beanvalidation.org`), so users don't get confused about what's API and what's implementation. In any case we should have a consistent approach in all listings/API types.
How do you call this script?
A statistics ncdf file is not created when an experiment is (re-)loaded, but created just before analytics are calculated.
You make @ConradIrwin sad :-)
The docs say it can be a string or an object, not anything at all. Either way you're only checking one type, though, so you're not ensuring safety unless you check for both (not that I'd recommend this). If you prefer to check for an object rather than a string, that's fine, but I still recommend doing a `typeof` comparison instead of wasting function all overhead on Y.Lang. Incidentally, `Y.Lang.isObject()` also returns true for functions and arrays.
It's better to put this change and the setProperty change into a separate commit: redefine explicit property as non-default property
Agreed that discussion replies should be upgraded to entities as well.

And we want to start storing "target_guid" in the river when possible, but doesn't have to happen for this, since $object->getContainerEntity() usually gets you the same result.
Why not: `this.putAll(...)` ?
Should be `MemcachedBaseTests` and probably abstract.
Boolean values need to be set to their string counterpart to be properly indexed via our existing search pipeline. It's probably not something we would need if we were indexing directly from the application layer.

On Dec 20, 2012, at 4:30 PM, dymsza <notifications@github.com> wrote:

> In extensions/wikia/Search/WikiaSearchIndexer.class.php:
> 
> > +	 * @todo these need to be updated any time one of these values change for a wiki. could get dicey. will def need atomic update.
> > +	 * @param int $wid the wiki id
> > +	 * @return array containing result data
> > +	 */
> > +	public function getWikiPromoData() {
> > +		wfProfileIn(__METHOD__);
> > +		$homepageHelper = new WikiaHomePageHelper();
> > +		$detail = $homepageHelper->getWikiInfoForVisualization( $this->wg->CityId, $this->wg->ContLang->getCode() );
> > +		wfProfileOut(__METHOD__);
> > +		return array(
> > +				'wiki_description_txt' => $detail['description'],
> > +				'wiki_new_b' => empty( $detail['new'] ) ? 'false' : 'true',
> > +				'wiki_hot_b' => empty( $detail['hot'] ) ? 'false' : 'true',
> > +				'wiki_official_b' => empty( $detail['official'] ) ? 'false' : 'true',
> > +				'wiki_promoted_b' => empty( $detail['promoted'] ) ? 'false' : 'true',
> > +		);
> Why you convert bool to string ?
> 
> —
> Reply to this email directly or view it on GitHub.
>
Don't pass as a reference
what is this? looks like wrong constructor parameters to Address, use one of the Address apply
systemClassLoaderChain.contains(cl)
I think it should go a little something like this:

case class MixMetricsSelector(...)
To be symmetric with the Scala API, do we want to return "RandomAccess"? http://docs.oracle.com/javase/1.4.2/docs/api/java/util/RandomAccess.html
The docs does not say anything about the mutability of the list returned.
I didn't get at first that you only need this if they don'g match with inflection.  :email has to be there for the Wagn::Views declarations (format :email)
the new Renderer class for Json
Ah, of course.  So, I agree that using a simplex method is best.
Even with major versions we shouldn't break compatibility as such. If before_search_view is necessary let's keep the two extension calls (before_view and before_search_view) for the time being and mark before_view with a comment or a log message as deprecated so people have time to adapt to it.
If it was still needed, wouldn't we want to add it only if legacy_templates was on, to save on page load time otherwise? (We did this for the activity streams on user profile pages I think)
My thinking was that it's simplest for us to enforce that the entire list of addresses gets overwritten by an update, and that this would be the most explicit way of making that obvious.

Do you think overwriting the whole list is a reasonable approach? I doubt anyone will have more than a couple of addresses per notification. Would this behavior of allow_multiple and having an update overwrite all existing values be intuitive in that respect? I will happily to change it if you think that will be the better user experience.
Convention-wise in these docs we've been tagging file names as preformatted text with double `.
(confirmed)
UPPERCASE :)
This return value for invalid counters (empty dict, step number) seemed just as appropriate as `(None, None)`.
Missing License header
No, we need constantized user_model_name (not just a string), look [here](https://github.com/tomgi/merit/blob/customizable_action_user_class_name/lib/merit.rb#L21)
make spacing consistent.
New queue as opposed to piggy-backing on the existing timers queue, is what I mean.
should be integer constants?
I think it makes sense to contribute OpenLayers.Format.Picasa and OpenLayers.Format.YouTube to upstream OpenLayers. Especially since the method overridden here is not part of the OpenLayers API.
сначала private slots, потом private
чем простой int не угодил?
в список инициализации
нужен комментарий к классу
```ruby
# If Date bits were provided but blank, then return nil
```
I think this change is fine. I hate these whitespaces
Got it. So this send is only to call the method dynamically not to call a private method.
@carlosantoniodasilva `+=` would work, but having duplicates would be meaningless and doing a union is cheap insurance.

One thought: should we normalize the keys to symbols as well?
Maybe you read beforeTyper that way because you have a pre-existing idea of what "atPhase" means.  I propose that from first principles, there's no way to know what being "at" a phase means with respect to that phase.

So I made "beforePhase" (which is "before phase runs") and afterPhase (which is "after phase runs".) The ones with phase names are just the obvious substitution: "beforeTyper" is "beforePhase(typerPhase)" which is not "atPhase(namerPhase)" because "atPhase(namerPhase)" means before the namer phase.

It just works out that "atPhase" means "beforePhase" because that's how atPhase was being used.  If it were up to me I'd get rid of "atPhase" entirely.
To be honest, I'm on a fence w.r.t this one. Indeed, it's duplication, but on the other hand it's a very minor one, and it's hard to get it wrong. E.g. if you mis-copypaste and write just `LiteralAnnotArg(value)`, compiler will complain. Also LiteralAnnotArg and Constant are very stable, so it's unlikely that they will be changed and those changes will be incorrectly propagated to their copypasted usages.

Nevertheless I'll think about how to reformulate this code so that there's no need in duplication at all.
Btw, you can use mapValues as long as you also end with toMap.
Ok, I see that this is convenient for the IDE. If it's just for `apply`, then it should be easy to have a workaround in the refactoring library. It just doesn't feel like a good solution, abusing TransparentPositions like this :)

So, what about `unapply` calls? Shouldn't they get the same treatment?
Making this env var name more descriptive would be very helpful, eg. API_PORT
Several "doctests" demonstrate multiple possible outputs of expand.
Imho these abbreviations (e.g. writing `x in morphism` instead of `x in morphism.components`) are more confusing than helpful (i.e. writing `x in morphism.components` is fairly self-explanatory, whereas `x in morphism` is rather arcane - what is being iterated over?)
This was necessary when I added it to .args because it needs to be the third argument (when rebuilding the expression from .args).
I think you should formulate docstrings as full sentences.  For example, "Returns strong generators for dummy indices.", "Assumes the metric is symmetric."  Also, the first sentence of the docstring, which grasps the sense of the function, should be separated from the rest of the comment by a blank line.
`True if (eq == 1 or eq is True) else (False if (eq == 0 or eq is False) else None)` ? I don't think there is a method that does this, though.
this is wrong. You have there unmerged changes
I think removing `&& $attribute[0]!==chr(0)` is enough. No need full reversion because `is_callable()` will cause problem.
Could you help to do this? Thanks.
We should explicitly separate static variable cache for the default MIME types source and for the custom MIME types source. Otherwise old behavior would lead to the errors.
Could you sort this while you're here?
inline comment 2
to avoid downloads during the tests it might be interesting using some artifacts here already pulled in by some side dependencies of karaf; wdyt?
'ref' don't load anything, it only defines AutoLoads :)
use any of ref features then you will have 'threads' loaded.
My mistake, added commit to make that var local
Add a `self.assertIn(db_num, self.router._down_connections)` in between here to show that the connection was marked as down.  Otherwise, I could replace the implementation of `mark_connection_down` and `mark_connection_up` with `pass` and this test would pass.
`commit->getProject()` can return null ?
assuming that the API addition means a version bump
This critical section is not really critical but... you'd normally aggregate the FD_ISSET checks outside of the critical section and then set them once with `stream->select_events |= events`.
@piscisaureus I think that applies to `uv_rwlock_init()` and friends as well.
Very nice use of stubs.
missed this one on Friday - done.
I don't believe that this will change. This is how mailhide / recaptcha defines the popup.
If we're in whyrun mode, and a resource's action is blocked, we're not going to send the action, and we would still want to log that in the Event Dispatcher. Thus, the trailing unless on this line shouldn't be there, but shifting this code around doesn't appear to make it significantly better.
var should not go in the for. This is not C or java, and thus variable hoisting can lead to unintended behaviour. IMO `$.each` should be used instead.
But it's PHP, we don't need standardised function names!!!!1

On 14 Nov 2012, at 11:00, Erik Frèrejean <notifications@github.com> wrote:

> In phpBB/includes/api/api.php:
> 
> > +		$result = call_user_func_array(array($this, $func_name), $this->args);
> > +		$this->handleOutputData($result, $format);
> > +
> > +		return true;
> > +	}
> > +
> > +	/**
> > +	 * Handle output data - turn array into string of something like JSON or
> > +	 * XML and then output it.
> > +	 *
> > +	 * @todo: Hook for adding other output formats.
> > +	 *
> > +	 * @param array $data Array of data to handle.
> > +	 * @param string $format Format to turn the data into.
> > +	 */
> > +	protected function handleOutputData($data, $format)
> Camelcase, should be handle_output_data
> 
> —
> Reply to this email directly or view it on GitHub.
>
Alright, i'll change that
With dialects the base dialect would be, hopefully, ansi sql.
Take out the space.
The "active/latest/whatever" revision is the current post. It does not have a post revision ID because it is not stored on the revision table. So, I give it index 0 so I always know where in the array the revision is.

As for sorting, I suppose I don't necessarily need to because I am ordering them by revision ID in the query, and the 0 index will be before the others even without a sort method.
Already been said and I agreed, I just hadn't fixed it yet?
@p are you still wanting me to rework this? imo it's fine as is, but if you think it is too confusing, I can rework the logic.
The isset($_SERVER['PHPBB_FUNCTIONAL_PATH']) is redundant.
How is it that sphinx backend has no get_word_length?
Nope, I'm fine with adding support for it though. If we decide to do that this also means we can't already strip bad names in the API server.
ah. fool me once...
:thumbsup:

I don’t know why I had completely blanked and not thought of 1. I also disliked it very much for number two so that is  a big one. Should we handle someone having white space between the number and unit as well?

`'7 deg'`?
A short comment would by nice, something like "global model cache when on the client".
and should be ```var l```, if you do use it, to prevent global scope
This test has been skipped as this was too brittle. We'll have xit on it for now and remember that this needs to be tested better later.
Would be better to re-use existing login and use admin views to limit user credentials by role (as applies to rest of the modules).
Also shouldn't be public.
Fewer methods would be nice, but I think that requiring the user who wants an sql.Date to notice the util.Date we return really IS an sql.Date and then cast it ("trust us, it's safe") is throwing them under the bus. 
Rather than that, I think I would rather just return an sql.Date declared as itself and let the user looking for a util.Date make do with the substitution.

But I actually feel better about the full set of methods as coded.
not required when you change definiton of function (sn required)
DON"T USE PUT event handlers in html attributes!!!
Мне кажется, потребуется поделить зависимости у модуля технологии на `getBuildDependencies()` и `getCreateDependencies()`
Yes :(, fixed.
My intention was more or less as you say here (initial trap + additional ones for exceptional cases).  So we should probably have the initial one happen at this early point still, the cleanup of the others looks nice though.
we cannot use empty.jsp.. as it is still being used to clear frame on the tasks tab.
Hence created another file blank.jsp for view purpose and empty.jsp fro redirection purpose to avoid confusion
Please review the [jQuery Core Style Guide](http://docs.jquery.com/JQuery_Core_Style_Guidelines)
This is a ridiculously long implementation for such simple functionality. MADLIB-579 will hopefully make this trivial.
Make the comment for verbosity consistent.
Why is it not necessary now?
Bare excepts are uber bad. You need to either:

1. specify which exceptions you're handling
2. specify Exception

When you specify a bare except, it catches SystemExit, KeyboardInterrupt and a variety of other things it shouldn't catch. Plus you're ignoring the exception, so it silently fails, so you'd never know.

Please fix this.
Usually we would just say

```rb
Plugins have been installed to:
  #{HOMEBREW_PREFIX}/sbin
```

and not expose the Cellar path unless it is really necessary.
I see. Unfortunately we have a query that has the file_target in the WHERE directly and that is executed often. I catched that with an sql logger. I will remove this index in the pull request until we have a better solution. Thanks
print break mod_wsgi
Are there any editors people are using that don't understand UTF-8? I prefer being able to read the characters. Would you mind changing it back?
Changing it to


Return generator of flair mappings.

Each flair mapping is a dict with three keys. 'user', 'flair_text' and 'flair_css_class'.
This line and the next two are custom lines added by Syte, which I believe you must manually add when upgrading bootstrap-modal.