labofsahil/github-event-dataset-2012 huggingface dataset:
Mifosx-54: modifications to staff relations in creation and edit UI. Ass...

...ociate Loan Officers during loan creation


Keith,

There seemed to be some merge issue in for the commit labeled "mifosx-32: support ability to modify loan application" . I have copied over the actual changes and deleted duplicates for messages.properties.
Front page copy

This is a rework of front page copy to be more direct. It introduces links to the planet download page, license page, and using openstreetmap page in context.

![front page copy](http://dl.dropbox.com/u/68059/Screenshots/mfncrmf70gp5.png)
Fixes a bug that causes an unhelpful NullReferenceException when running under Mono

...o

If running this library under a Mono environment instead of .NET framework
and a TargetInvocationException is caught within
LazyInitializer.Intercept(IInvocation) then you will see a stack trace something
like:

System.NullReferenceException: Object reference not set to an instance of an object
  at NHibernate.ByteCode.Castle.LazyInitializer.Intercept(IInvocation invocation)

This is because it makes use of reflection to invoke the private/internal method
'InternalPreserveStackTrace' of the Exception type.  This is undocumented
functionality and not a part of the API.  This method does not exist within the
Mono framework.

* This commit simply null-checks that method and - if it does not exist - skips
  attempting to call it.  Whilst we may lose the stack trace as a result, we
  still preserve some of the original exception and avoid throwing a completely
  unhelpful exception.
Change rating text and styling
Fix bug #2897

Corrige un pb d'alerte d'erreur de la part de pie.htc, lors d'un retour de non-impression avec ie9.
first draft of Twitter bootstrap
Fix documentation inside lib file
preparing clj implem: added multimethod
Fix bug in viewing unicode indexes

- Unicode comparison wasn't implemented in the fileview layer
  which meant  esp couldn't filter an index based on unicode.
- Unicode ranges also weren't passing the correct length

Signed-off-by: Gavin Halliday <gavin.halliday@lexisnexis.com>
Submission Copy of CUDA rasterizer

Sorry for the large commit.
Prepared release 1.0.0.

This pull request is the release branch for 1.0.0, it supersedes #1.

Pending issues are:
* automatic project creation
* project matching irregularities

Both might stem from either a client or an API hickup or a combination thereof and require respective API payload debugging.
Updated 2 Links

One for MDn for Web developers
Android Docs for Android Developers
Gridfield formaction renaming

@sminnee @hafriedlander @stojg I've done a bit of class renaming in GridField to make the relationships clearer, are you OK with that? Pull request is for the big-o integration branch, just a way to isolate the commits, really - no need for you to pull.
Add Leaflet

I'm the lead maintainer of the project so I'm obviously biased, but aslo pretty sure the docs are beautiful - check it out. :)
SOS-791 Added init.jsp

Hi Jon,

I added the init.jsp in the commit for SOS-794 and not in this.

Sorry about this,

Eudaldo.
Add explicit wait for copy to complete.

Problem with the teardown was actually that the test was not waiting for the package to copy. The test needs to be changed too (separate pull).
IE9: js error when closing the popup.

Steps to reproduce:
1. Open demo page in ie9: http://demos.badsyntax.co/tinymce-jqueryui-popups/ 
2. Open 'Edit HTML Source' popup
3. Press cancel button
4. JS error happens

The root of the problem is jquery's dialog destroy method. It removes iframe from dialog element and places it into document.body. This causes iframe to reload.
Separating blackberry.invoked from blackberry.invoke

This change separates the "invoked" event from the invoking functions.

Viewable docs:
blackberry.invoked:  http://blackberry-webworks.github.com/WebWorks-API-Docs/WebWorks-API-Docs-next-BB10-invoked-split/view/blackberry.invoked.html
blackberry.invoke:  http://blackberry-webworks.github.com/WebWorks-API-Docs/WebWorks-API-Docs-next-BB10-invoked-split/view/blackberry.invoke.html
Add length display of the title and text fields during submission

Add length display of the title and text fields during submission.
Title length display turns red to indicate when the title is too long
(>300 characters).

Applies to issue #149

Alternative to #156
Fix link -- caring.com missing scheme.
Throw exceptions properly

Adds individual exceptions and does a redirect to prevent the user from seeing `?WLS-Response` (so when refreshing the page the response isn't reused). Fixes #4.
Deps

Use DoctrineFixturesBundle tag "2.0"
ObjectMasks

Adding support for string object masks
Rewrite accept header when format specified

Rewrite the accept header to the specified format. Otherwise, it appears that the format is set to something that is not accepted.
Removed deprecation warning (slave_ok)

Removed deprecation warning (slave_ok)
Remove extra UIWindow methods and use applicaitonFrame.

Using applicationFrame is recommended in the Apple docs.

No need for extra UIWindow methods.
Minor tweaks and new utility methods in acceptance framework test-case

This commit includes the following:

* Minor cleanup for readability on failure error messages
* Documentation for with_master_running_on method
* New defaults for with_master_running_on:
** will automatically add "--daemonize" if you haven't specified either that or --no-daemonize
** will automatically add "--logdest=<puppetvardir>/log/puppetmaster.log" if you don't specify a logdest; sending acceptance framework log messages to syslog didn't seem very useful
** will automatically add "--dns_alt_names" if you don't specify it
** will automatically add "--autosign" if you don't specify it
** no longer deletes log dir between executions of master
* New "get_puppet_tmpdir_path()" method, to hopefully start quarantining acceptance test temp files in a common location
* Documentation for "create_remote_file" method
* New options for create_remote_file method to ensure files are visible to "puppet" user et al:
** "mkdirs": will attempt to create parent directory structure before creating file
** "owner": a username to chown the new file to
** "group": a groupname to chown the new file to
** "mode": file permissions to chmod the new file to
* new "mkdirs_on" method
* new "rmdirs_on" method
* new "file_exists_on?" method
documentation fix

closes issue #11
fixed fonts

fixes fonts for hdpi
Fix _config.yml. rdiscount needed a colon after it.
Fixed issue #17

NameError: name 'test' is not defined
changing ofEvents.update to ofEvents().update 

Thanks Kyle
More Wordpress 3.3 bugfixes

This update fixes a bug that occured when sorting tinymce enabled textareas by drag and drop. It also fixes a bug that caused the media library to insert images into the wrong editor when using more than one editor on the same page/post. While at it, we also made some minor improvements to the CSS. Enjoy! :)
Privoxy

I've added two unit files for Privoxy.

A simple service unit and a socket unit, for socket activation.
fixed abks:collect-files-rec: ignore non-existing directory

"anything-compute-matches: error when processing source: PDF Books" や
"Candidates must either be a function,  a variable or a list: abks:collect-files"
などのエラーを抑えることが出来ます。
Paste previous and Paste next commands

Added Cmd+Shift+V/Ctrl+Shift+V to paste previous history item and Cmd+Shift+Alt+V/Ctrl+Shift+Alt+V to paste next history item (à la TextMate).
BUGFIX Deprecated function Name() replaced with getName()
Made a "significance" test pass. You might want to check that my regexp isn't too generous

...rences consisting only of single-alphabetic character prefixes to numbers (for road numbers differing only by presence or absence of the letter). We may wish to break strings for comparison into arrays on semicolons for non-ordered comparison.
Enables a user to supply TinyMCE options to a text_editor form column via the "tinymce" option

Enables a user to supply TinyMCE options to a text_editor form column via the "tinymce" option. For instance, within an active_scaffold block, a user can supply:

```ruby
class FooController < ApplicationController
  active_scaffold :foo do |conf|
    conf.columns[:bar].form_ui = :text_editor
    conf.columns[:bar].options = {
      :tinymce => { :theme => 'advanced' }
    }
  end
end
```

to use TinyMCE's advanced theme rather than the default simple theme for the "bar" column.

Thanks for all the work with AS. If you'd like anything modified, I'm happy to do so.
LPS-25464 Should use ExceptionInInitializerError instead of RuntimeExcep...

...tion in static initialization.
LPS-18892
Use relative path for looking up the module

Use relative path in the dependencies.yml file in the sample app
remove

Enable Call Volume Hack on GSM (1/3)
Development

Trying to merge old branch
Fixing the main content of the theme.

-Removed sidebar_first
-Removed style.css


“You had me at meat tornado.” -Ron Swanson.
LPS-25943 Missing build.xml file in the antisamy hook
Added the table name in the model
LPS-24751 - initial service.xml

Not the last commit for this ticket...
pull on this
An issue with re-rendering and returning this from #view


Found an issue re-rending views.

Made #view return the view that was just initialized (good point @destos)
API stats

Using CacheDBThrottle as throttle for the API in order to get stats about API usage
Feature/multisites

IP Home integration (final : Twitt box, Newsletter box & 
Todo : implement views for these elements and replace fake texts by true dynamic content and it's over.
Upgraded plugin to Boostrap version 2.2.0
Made many changes!!

- More translations into Spanish.
- Setting the language files.
- Side panel with up to 3 diferenes realms.
- Information for each realm in the status page.
- Set of Languages ​​footer.
- Adjust to change page avatar.
fixed test (test_atom)

I'm working on a project which have a different language than english.
In `ActivityTestCase.test_atom` the laguage code `en-us` is hard-coded in the xml string during assertation, so this test always fails for me.

I changed it, to use the `LANGUAGE_CODE` from `django.conf.settings`.

By the way, nice work guys, keep going!
json->xml is blocked by & character

When converting json to xml, ampersands confuse the simple xml parser and need to be escaped.
Somehow missed that one during previous merge
Small fixes in the competition.js.coffee file

Added a check for actual track selection before allowing submit. Previously ended up with vague heroku error screen.

Also added a preventative measure for users on slow connections, attempting to vote on a track. Occasionally, the vote appears to be cast, but if the data hasn't fully loaded, the user may leave the page without ever properly casting vote. Just added a confirm function to the voting which is set to null after success.
Added Javascript and index.html to make the App working
Publish concept dictionary - META-251

https://tickets.openmrs.org/browse/META-251
Update select_time_widget.py

Fixed bug mentioned in http://djangosnippets.org/snippets/1202/
N panday version.1.0.0

New version of the .net bridge
[pt_BR][getting-started][why.md]First translation
Add the remaining news items from the main Aeolus site.

Note that this builds on top of the previous pull request, so needs that approved first.
Updated gem needed because explicit memcached gem was yanked

Needs updated gem.

http://rubygems.org/gems/memcached/versions
1.4.0 February 26, 2012 yanked
Block-based interface to the iOS SDK's REST API.

I wrote a new block-based interface for issuing and receiving responses to REST requests sent through the iOS SDK. It's contained within a new `Blocks` category on `SFRestAPI`.

The block methods exposed here should follow exactly the methods defined in `SFRestAPI` with one exception: my version of `requestForRetrieveWithObjectType:objectId:fieldList:` takes an `NSArray` instead of a comma-separated `NSString` for the `fieldList`. My version performs the comma-separation automatically and also eliminates duplicate fields specified in the array.

Using a block method to describe global:

```[[SFRestAPI sharedInstance] performDescribeGlobalWithFailBlock:nil 
    completeBlock:^(NSDictionary *results) {
        NSLog(@"results %@", results);
    }];
```

Or to run a query: 

```[[SFRestAPI sharedInstance] performSOQLQuery:@"select id from user limit 10"
failBlock:nil 
completeBlock:^(NSDictionary *results) {
        NSLog(@"results %@", results);
    }];
```
Made changes to addMessageForm

with the exception of adding company messages to database
changes to UI for bootstrap
Location Normalization

Please check if I got all lines where normalization has to be done.

Extension runs fine when testing. Locations are stored with a normalized ID in the WebSQL database and retrieving data from there seems to work fine, too (please confirm first before merging).

Issue #95
Compatibility with redcarpet 2.0.1

The redcarpet api changed in the 2.x, this updates the redcarpet.rb script to be consistent with the 2.0.x api.
environment detection

added environment detection, so it loads the right database config
Providing an example in the readme.

Improving the readme.
Fix for unchecked __class__ access in signals pane

It's possible for the im_self attribute of a receiver to not have a \__class__ attribute, if im_self is a metaclass instead.  When that happens, you get an error like "class MpttMeta has no attribute \__class__".

This patch fixes that by checking for the presence of the \__class__ attribute before attempting to use it.
Application Theme

Implemented Issue #4.
Early march

Added GSN Digital to the list of area companies looking to hire.
The "A Book Apart" link wasn't working in the credits section.
Githubbing the right way :)

These commits will add a University of Illinois open source license, three screenshots of the VPython renderers, and a README to tie it all together. Enjoy!
Gnome3.4

Hi there

I made small changes on the extension, now it hides on overview mode.

Thanks

[ArkaN]
I'm curious to know why you guys didn't decide to keep it `normal` but without the `!important`.
well I need some more time. I am running into some trouble while enabling all languages in the LINGUAS file. If i am blocking you just go ahead and update it yourself. I think that is the fastest way.
Maybe GetMoveAction or GetNavAction ?
According to the Python API for Py_BuildValue (see http://docs.python.org/c-api/arg.html#Py_BuildValue) the "b" in Py_BuildValue actually does not result in a boolean (True/False) but in an integer (from a character). Doesn't look like Py_BuildValue() has a way of returning the python objects True or False. That would have to be done using something like

if (bResult)
  Py_RETURN_TRUE;
else
  Py_RETURN_FALSE;
no reason, I was boring
We've gotten this player's real name at this point if possible,  so doing this again is wasted cycles!
No. module i18n.js doesn't accept keys like this. eg:

    toast.set('Destroy ...').show(-1);

should be 

    toast.set(_('destroy_dots')).show(-1);
The problem here is that this might break code that uses both ```contains()``` and namespaces, because the regex namespace is only being declared when no namespaces are passed in. The original global declaration isn't ideal either - I think the best way to do this would be to register the regex namespace in the namespaces dict if it is not declared yet, and to use an uncommon prefix to avoid naming conflicts (not perfect, though...).
@janmarek My jsme ve firmě taky Nette používali jako submodul, ale po té, co nám force push několikrát rozbil aplikaci, tak jsme přestali.
Hey, cool to read you @pixxelboy, Thanks for your proposal about design :) As David say, we talked about a special flow with fork & review.
Sorry, false alarm. It turns out that waf wasn't rebuilding `out/Debug/node`. After removing said file by hand and running `make test-debug` again, the test passes.
Reference parameters can't have defaults.
Fork libev just for a debugging tool ?!? Come on !
Oh, yeah, good point, will do it, thanks!
boost-build was not updated in this boost release
Ух ты ж нихрина себе! Совсем не ожидал! =)
It's GREAT! RESPECT!
I really think there needs to be comment here, as this is tricky code. You are using the code in the delay slot, and then jumping to the next instruction, so it is executing linear code. And then to make it more confusing, for larger values of 'i' the below 'Addu()' macro-instr expands to 3 instructions, one of which is in the delay slot. Be nice to explain this a bit more.

Here is a weird trivia, I just learned, the bal() instruction expands to a more generic "bgezal zero, offset", which is conditional, and always true, and the branch is taken. But you can code this as bltzal, zero, offset, which is conditional, and always false, and the branch will never be taken, but the RA register will still be updated. A very weak argument could be made that that is better self-docuementing, as code just continues execution after the branch delay slot.

OK, never mind, I am just insane, none of these weird edge-cases are self-documenting, and however this is done, it should be spelled out in comments.
This is just to update the selection when you change different folders on the tree
so should not have any impact
Can we inline these three folders instead of reusing `d`?
ur'welcome! Thanks for a idea. :)
Not sure, but I think it's a self-referring joke ;-)
I call the function with that : mouseEventCB(e, translateMouseCoords(e.clientX, e.clientY)); is that not ok ?
why, just conventions in docblocks?
Haha, sorry, that wasn't a request, it was just me yelling from excitement. Open Sesame! :D
In cppcheck code, both ways are common practise. I usually prefer the other way (removing name) and I don't know why I didn't use it there (probably just forgot about it in that moment), but in the end, it has the same effect. Simply writing name; does probably not work, although I haven't tested it.
Makes sense now, thanks.

2> timer:seconds(true).
** exception error: bad argument in an arithmetic expression
     in function  timer:seconds/1
ok...dann hau ich die erstmal wieder raus...
ans testen hab ich grad garne gedacht gehabt, sry.
It's a magic number.
it renders only the contents of `{% block content %}`
Chainsaws have ammo too, even if they're not guns! I can't see anything bad happening from removing this test. All items have an ammo type, even if not all items are guns. This might fix the segfault mentioned in the comments.
On Sun, Jan 15, 2012 at 11:32 PM, theMIROn
<reply@reply.github.com>
wrote:
> is regex really needed?
> simple strtoul will do the work, even if you need error checking.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/reubenhwk/radvd/commit/41c6192041879cbad8591d318e3b0e758edecf16#commitcomment-869907

The regex isn't really needed, but it is a somewhat stricter
validation tool compared to strtoul.  Other than it not being needed,
is there a good reason to use strtoul over regex?
能否将三个自动集成在一个设置界面里呢？
现行需要刷出票后，手动提交自动订票呢。
I mean lines 1134-1138 - they're copy and pasted again at 1142-1146
Where is the `name` variable coming from?
One thing not done here is to distinguish Oasis-db from `odb`.  Oasis-db is an on-line database of packages, mostly using oasis for package metadata.  `odb` is a simple ocaml program similar to `barbra` that originally used only Oasis-db as a source for package metadata and tarballs.  With the help of a `packages` file that provides package metadata (like dependencies, where to download, type of package), `odb` is able to install from local directories, local tarballs, as well as git, svn, cvs, hg and darcs repositories.  `odb` is < 500 lines of ocaml in a single file, designed to be run without being compiled, and depending only on ocaml and findlib, so I'd count it as pretty light-weight.
Those codes just process the raw results to standard format, the real analysis is done in tools/analysis.py
good point. Cherry-picked that one.
this newline isn't consistent with the one on 444. but that's just being nitpicky.
While creating a collection if the description is emtpy, no entry for the description is created in the translations table. If a description is provided, an entry is created in the translations table.

So later setting the description to blank should ideally delete the old translations entry for the description or set the translation text to an empty string. R193 to R196 makes sure that when description is left blank at the time of editing a collection, the translation text is set to a blank string.

So if I do as you have suggested, it neither sets the translation text to an empty string nor deletes the entry because description becomes a `NoneType` object when left blank.

R193 to R196 makes sure that whenever description is left blank, an entry in the translations table is created for an empty string.
sorry. It seems I misspoke. The response from cleanup is not confusing at all. The fact that the engine would be broken while not producing an error is what would be confusing. I am not sure there is an easy fix though.
until said comment is written, I'm not sure how to judge whether the implementation is correct.

Some tests would of course also help !!!!

How about you add some tests too.
for some reason, the first set of guards lets you pass while stealthed, but the others all catch you.
how about root_path? :)
I thought we could remove this now?
deprecated the function, I couldn't find any references in the code anyway for the cases handled by the function
As near as I can tell there isn't a configuration script available any more (note the lack of one in the [source tree](https://github.com/kr/beanstalkd/tree/v1.5)). For what it's worth, I've installed and run Beanstalk successfully using this change set.
Actually, it's not even that!  Unless something changed, we should be accepting ALL three possibilities: userId, screenName or emailAddress.
I already nuked one other part of that equation; when you (re)start the
server, the addon would preload any chunks with a chest in.

It should not do that any more (I hope); I should check that I did in fact
do that.

As such, unloading chunks with chests is probably good too.

I don't mind "officially forking" this addon, though I don't really know
what I should do in that regard.

Since I've added new stuff to it, it isn't quite the same addon any more.
For instance, one of the latest changes I did to it was to add "dropzone"
functionality. Basically, for a chest you can choose whether to use the
radius system, centred on the chest itself, or a dropzone system, which is
a rectangle block-shaped system consisting of 2 3D coordinates in the world.

This would allow you to have chests pick up items from a very precise area,
not centred on the chest.

For instance, I have created a storage room consisting of a cube-shaped
room with chests lining the walls from floor to ceiling (except the topmost
row), and they all have suck filters for their particular item, for a
dropzone in the middle of the room. All you have to do is drop items from
your inventory in the middle of the room, and it'll end up in the right
chest(s).

I also added a data structure that prefilters the chests it analyzes for
each item, depending on the item type. For instance, if you drop a stack of
dirt, it will first check all chests that has a suck filter that contains
dirt, and if none of those can pick it up (out of range, full, etc.) it
will check all chests with no filter at all. This means that it will never
analyze a chest with a stone filter (for example) if you drop dirt. With
diligent use of suck filters, you should have little impact on server
performance even with a huge number of diverse chests (unless you add
thousands of chests with no filter.)

-- 
Lasse Vågsæther Karlsen


On Wed, Feb 15, 2012 at 17:25, sorklin <
reply@reply.github.com
> wrote:

> I already did :).  I was just wondering if you knew why he did it?  I
> can't think of a good reason to keep them loaded.  I didn't want to break
> internal functionality by just disabling it.
>
> Have you thought about officially forking this?  They don't seem to be
> actually doing anything with this plugin, except for changing the POM for
> new RBs.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/lassevk/DropChest/commit/6f0618de51a6f0e7ff34376e5207a2c52b4a6215#commitcomment-969979
>
oof that looks like a good catch
The whole point of the cache was because move events didn't guarentee catching the move from x=3.99999... to x=4.00...1. This was due to a improvement in bukkit's code that would only throw it every ___ degrees of movment. Has that changed? Sorry for necroposting, I just noticed this on the network graph.
Is there a possibility here of making it so that we can use something other than the default ActorSystem?
Cypress helped me with this.
It's not much of a no-no as it only effects source code until compiletime.
Sorry I hope this is ok its just your licence. Said we could edit and use as long as we don't claim as our own ...

This commit is not really a problem with vnp just some dimes the negative nodes in pex stop us from talking and interacting 

The other commit is fixing a bug that when another plugin changes the tabname to a coloured one when someone vanished and reappeared there was a colored name and a white name
`msb_inverse`
I don't think we want to change the histogram to point at the heatmap.
Not really necessary. I did put this in at one point, but there's never any utf-8 in the document, and Unicode strings sent over the websocket have no implied encoding either.
Oh man, the CSS gurus are upon us. Never thought to do this with CSS.
Now I know why you need the MC files at assets, you can take the stuff like font or textures.
We should move the logic of deciding what's <strong> based on the current date, instead of hardcode it
I can give you the SQL for YTDB
Can you break down this long line?
I'd keep the original one if you don't mind. proposal 1 makes it harder to read, proposal 2 makes an additional concatenation which is less efficient ;)
with the power of grayskull, and to a lesser extent  [this] (https://developer.apple.com/library/mac/documentation/Cocoa/Reference/ApplicationKit/Classes/nsapplication_Class/Reference/Reference.html#//apple_ref/occ/clm/NSApplication/sharedApplication) doc page
Works for me :D
也可以
here you need the creation/refresh timestamp and the timeout value and then calculate the actual remaining time
This is very important
not necessary!
I'll merge this if you add back the try/catch. It's there so I can build the docs and push this live from my mapnik-free laptop.
I reverted 5312987e0b49e555d7df5551d8cb5410ab8cd129 and committed ce1a5c2b16cc96fb979b941ef0b14b334f91085c
嗯，好像是没转过来，回头我再看看
在 2012-02-28 19:36:21，"Ubun Tenkei" <reply@reply.github.com> 写道：
>我就用的直接替换传的，竟然还没改过来？
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/Moligaloo/QSanguosha/commit/ca4a2b4ca5e22297d01264b47fbfcac6342de50f#commitcomment-1020328
这个是说编译后的文件以后就用fmodex.dll来代替？那编译后的程序里要不要删除那个MP3什么的？
Да по первости я наломал дров ))))
I pushed an fix for this into the master
В этом коммите все минимальные требования к заданию выполнены. В админке есть спонсоры, можно загружать их лого (будет выводиться как есть, т.е. размер). Никаких особых махинаций со спонсорами, организаторами и т.д. тут нету.
hebelek
That was NOT the issue, this is: var DEPLOY = __dirname + "/pkg/";
Patching now.
This is exactly what i've been looking for. I can't get the code to compile though. I keep getting read errors from files in the maven repository. Is there any way I could get your compiled plugin with this feature in it? the worldguard flag mob box limit.
I feel, let the document generator be psr-1 compatible :-) . We are making standard . So why do we need to bother about it ? .
@daeltar please feel free to open an issue and/or send a patch (ideally making this configurable) - nothing will change from a commit comment!
The line above this has a comment explaining what it does: "Precompile additional assets (application.js, application.css, and all non-JS/CSS are already added)." And, since the example I removed is so similar to the new line, I don't think there's much value leaving it in. I assumed it was standard to remove the example in cases like this, but I can leave it in if it's not.
looks correct!
I changed Dialect to implement ConversionContext (see below), so the dialect object is passed instead of EXTRACTER.
Fixed in 7b6a44958f8358d92897d4a256b572e79e854611
Question: How do you catch all these compatibility bugs?
Assuming the class is a unique number... I couldn't figure out what the $key value was, but you get the idea.
Please try to find also other possible remaining calls to addScalarResult in SqlWalker. =)
Whats this for ?? And what dose it fix?
Let this be the default too ?

So 
    case 'httponly' :
    default :
If Point has an equals method, you could do
assertThat(callback.getLocation(node), equalTo(new Point(...,..)))
inline instead of having this method.
All headings cannot be h1.
Why did you remove `Refinery::News::Engine.load_seed` and now this from migration?
This should be based on the max-bson-size from the currently connected server and not a constant.
honestly im not sure these spells i added in comments shows they still exist but where in game im not sure...
I know now thw situation, I was a bit worry about it.
I haven't had a chance to test this yet, though it looks good on the surface. But we should pull this into this repo, since we're (regretably) developing on another repo temporarily while we figure out how to handle multiple repos.
Леша, просьба. Если нет явной необходимости подключать новые библиотеки - не делать этого. У нас есть человек с windows, где установка любой дополнительной вещи - мучение, отнимающее время нашей команды.
Om du kör en var_dump($dsn) efter varje gång du hämtat eller ändrat den, så borde du se vad som händer.
What a concept.. symbolic names
I første omgang blev filen åbenbart ikke slettet ordentligt i GIT.(Altså registreret)
Just look at the StrUtils class,  you can certainly add it to this class, or move this checkString to that class.
Hello, Yes It's simply because imageTag directly search in /images folder... I can use another method directly at objet level.
[CPUfreq_test]testing this... +revert 'Clean power_up_devices' commit first
Heroku refuses to compile a Ruby app unless ```Gemfile.lock``` is present in the app being pushed, so maybe this line should be removed?
needed to add 'yui3-dd-draggable' to requires because IE is stupid
That's where I had them originally, but I am keeping these in Commands so that someone else can make a different application base class if they want.
How were these builds made?  Where's the source code?
do you have any problems with AXI oc??
looks good! Perfectly true, this bug was known but not fixed for quite some time now. Thanks!
Sounds good. Thanks ahead!
I think you need to split your tests in two… corresponding to the two bugs :)

You can test almost everything without HTTP at all — after all, you just built a ClientsRepository, right? So instantiate one, push records in and out, make sure it does what you expect.

Then you can test the next layer by bringing up your own test HTTP server (take a look at the existing tests), which will allow you to see both ends of the exchange, and will remove our dependency on production Sync infrastructure!

Finally, if you really, really want to, you can rename part of these tests "TestRealClientsEngine", and have them talk to phx.

So:

* Unit test as best you can;
* Layer integration tests on top of that;
* Finally, layer on real-world smoke tests.

Test the smallest thing you can. You're not doing that in these tests.
Should it still create a userdesc if password is empty?
-            case RIL_REQUEST_SETUP_DATA_CALL: ret =  responseSetupDataCall(p); break;
+//            case RIL_REQUEST_SETUP_DATA_CALL: ret =  responseSetupDataCall(p); break;

This one should stay and call base version.
If you want you can do the pull request.
The wording may need some tweaks.  I did it in the GH editor which made me somehow nervous...
bleah...
Well done. Most would forget that up until that point, we're only concerned w/ capturing numbers.
Hoofdletter aan begin van de zin ;)
Thanks, and sorry for the duplication, I found your fix after pushing this one.
What does getCategorySmilies() do?
I couldn't find such a method (I just know getSmilies() and getCategories()) :)
james Bottomley

USB surprise removal of sr is triggering an oops in
scsi_dispatch_command().  What seems to be happening is that USB is
hanging on to a queue reference until the last close of the upper
device, so the crash is caused by surprise remove of a mounted CD
followed by attempted unmount.

The problem is that USB doesn't issue its final commands as part of
the SCSI teardown path, but on last close when the block queue is long
gone.  The long term fix is probably to make sr do the teardown in the
same way as sd (so remove all the lower bits on ejection, but keep the
upper disk alive until last close of user space).  However, the
current oops can be simply fixed by not allowing any commands to be
sent to a dead queue.

Cc: stable@kernel.org
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Forgot I also added the netcfg commands too..
Any chance of publishing these to GitHub pages?
Awesome! Thanks, that fixed it. Great work on the gem—it fills a real need.
I'm trying to find a fix for the camcorder we are no't modding that app so it should be a prop file issue or config somewhere ... this doesn't appear fix the camcorder should fix the vzw apps issues though. 

but I would like nocoast to test so I know it's not just my camcorder that won't fix
Also, I look forward to entirely phasing out SVN...it'll take a lot less time to build that way.
you are missing this part of the patch also:
https://github.com/IceColdSandwich/android_packages_apps_Phone/commit/478b73032377f443f40d67f380532d71c0884bfe
In Android, heavyweight windows represent Activities, which have more meaning than simply being a UI element. For this reason, they need to be handled slightly differently. If you were to attempt to reuse the windows over and over again, you will see a slight black flash when you transition between them. Creating the windows on demand makes for a smoother transition.
Почему `UTIL.format()`, а не `JSON.stringify()`?
Про это не нужно писать, или нужно переформулировать.
Например, так: «узлы выполняются безусловно»
I don't believe we normally require modules to explicitly require "intl". I thought it was implied and taken care of based on the "lang" meta-data.

Maybe we should have the test follow that pattern, in case people have modules based on it.
So should we delete it while there's nothing valuable in there?
Should probably use FQCNs here for Ack and Failure
Yep, great work!
What is the unit of the average? Percent?
this is not used anywhere, right? if it were, the “my-dispatcher” would not be defined …
This diff is a little wonky; _function_wrapper is unchanged lower down in the file.
the fuel directory is in lower case, so it should be named like that here too.
ce code doit être à un seul endroit qui ne dépend pas d'une application, arbitrairement j'avais mis sur le planète par facilité, mais oui on peut le mettre sur www.mozfr.org, par contre pas dans un dossier dépendant d'un cms, c'est indépendant des outils utilisés et il faut un dossier mozfrtab au cas où on ait besoin de rajouter du js par exemple. D'ailleurs on pourrait même mettre un sous-domaine à part ;)
Well, all tests pass. I assume, there's no tests for this specific error message. Probably, should add, though.
why do we need to check for the shape attribute?  if bvals can only be arrays... simply call np.asarray on them
I dunnow. @MrBago - does this also fall under "clean up after the release"?
spelling: "associated"
I'm using the first child and last child classes for the exact thing they were intended for here
Ideally these would be hooked up so you didn't need the extra query per item (the items will all be the same, so at worse it's a single query for all of them, right?)
What about #318?
Sure. I could do `_working_dir_mgr` rather than `_wd_mgr` too.
This will not work in `3.3`. You need to use `DB::expr()`.
Probably makes sense to reference a more recent version of  the MySQL docs. 5.0 is pretty old by now.
Ah, since the pinch and pan gestures are communicated as a single `QGestureEvent` and we still want Qt to handle the panning. In that case, since we also want Qt to handle the panning if it happens at the same time as changing scale by pinching, I don't think we should be returning `true` at all, right?
I'd make this an `elseif` to safe a condition check. Same applies to addFunction and addFilter.
This can be written as `self.selectionColor ?: self.backgroundColor`.
Shouldn't this be `selectedTextBackgroundColor`? That's what was used originally.
Although they aren't necessary, I do always add them for clarity. It tells someone reading the code there's going to be an array.
My preferred style is to only advance one indent on subsequent lines. I don’t align with an operator on the previous.
finished test instead
Agreed.   This should not be exposed.
I am fine adding new tests (am sprinting at pycon), but I am not sure what kind of tests you had in mind ?

Regarding true quad precision, I don't think we have any platform supporting those yet, or I misunderstand the test you have in mind.
As you already know that return_value is an array you probably should use Z_ARRVAL_P instead of HASH_OF.

Or even easier just use the `add_next_index_zval` API function:

    add_next_index_zval(return_value, *zvalue);
I agree the ```qiime.check_id_map.check_mapping_file``` step would be a good addition, though this would be done after the release.

Since we are not performing authentication, the published spreadsheet would need to be editable by anyone in order to get the highlighting to work (which I don't think we'd want to do). Perhaps if we add authentication as some point in the future we could add this feature as I agree that it would be really awesome to have.
Я поправлю, но я не понимаю, чем плох указатель. Мы повсеместно и в огромных количествах передает в конструкторы объектов указатели на их родителей, например
не очень понял ответ. если я вызову у этого класса setProperty(), оно заменит существующее значение заданного свойства новым. чем этот метод отличается?
whitespaces are welcome :)
Could you move this title to `class AbsenceValidator`?
I think that would be nice.  :)
Please move this to the top of file
@carlosantoniodasilva Good catch on tr vs gsub.  As far as the one liner goes, I chose not to do it because it looked rather gross.  I try to adhere to readable code over, one liners, but if that is the preference I will go ahead and do it.
Created a separate issue for `emptyValDef`: https://issues.scala-lang.org/browse/SI-6762. Current situation with incompatible changes in scala-reflect.jar is unclear, so I won't include this refactoring in this pull request.
This should be ``array([0, 0])``.
Spelling (separate).
I think this import is no longer needed.
Ok, I already made the change on my computer, but I will submit it after my other pull request for matrices goes in, because at the moment Matrix does not admit general iterators as input. Is this good enough?
We should add `erfc(-z) --> 2 - erfc(z)`.
Can you supply a reference for this as an inline comment to `__call__`? It is difficult for me to understand.
I do not understand. This looks at all three-element subsets of S.
> At the beginning I was using a tuple, but Aaron suggested the idea of a dict (#1940) to let the possibility to extend and to not fix a priori the number of different dimensions.

I see. Is there a situation in which creating new dimensions is appropriate?
Don't put spaces around `=` when it's used to denote keyword arguments.
Oh my god I am stupid (not to see this). Thank you very much. Fix is pushed.
better to change to `now resolves view file only once`
Indentation convention is 4 spaces.
We need L-GPL (in an upgradable form so we can upgrade it to v3).
We should decide on a convention for this.
Engaging ANGRY MODE!
This is the pattern that some of our other swizzling categories follow. It's a bit nicer in that it doesn't require a heap allocation (like a block that closes over state does), but it's going to be about the same amount of code regardless.
Ditto re println.
Needs a colon. 

I know it's sort of formulaic, but "consider the following prototype operation:"  has the benefit of telling the reader to do something, rather than just declaring the identity (i.e. "this is it!")
The part where the function takes format as an argument and then you return a timestamp in a different format.

Looking at current develop, this function is defined but only used in the test suite?
This is getting passed into the environment parameter instead of the options parameter.
Please also follow out guidelines here. Break queries onto multiple lines and use `"` instead of `\'`.
Here is the package builder file: https://github.com/phpbb/phpbb3/blob/develop/build/package.php
I'm pretty sure that's what he means.
removing if (!sizeof($id_ary) && !$search_id) does precisely that, now if no results are returned trigger_error is not called, hence the search query remains intact in the search box
dito?
Hieght -> Height

you also might not want to capitalise it
If it should be a function then a static method on the manager would probably be the best approach here.
After discussion IRC it was decided to change all instances of `MD_` to `META_`.
was this a note to self?
We can't call out to the FS though, both of these things need to be burned into the agent.
Posibil să fie și pe aici ceva problemă cu tab?
OK, I took out the other quick exit...I guess the rationale could be that most of the time you won't be sending an Identity permutation so the quickness of that exit doesn't matter much.
I'll change this to a forEach call, just for consistency, but I really don't see a problem with "psuedo" scoped vars. As long as the programmer is aware that we only have function (not block) scope then I don't see the problem. I guess in this case you could move the declaration of the `property` var up to the top of the fn..
``!"".equals(displayVal)`` is enough
"DynDnsProperty" is probably a better name
Do you remember why you changed this? I don't understand why this is needed.
moar indent
Why you are registering two handlers for the same event?
This behavior doesn't exist in the current implementation of `HttpResponse`.
This is wrong, `SESSION_COOKIE_NAME` is the name used in the cookie, not an attribute name on the `request` object.

Besides it isn't equal to `'session'` by default.
Please remove this as it is not tied to the project but to your own setup. You should ignore it on your machine only (and it can even be done globally to avoid redoing it for each project): https://help.github.com/articles/ignoring-files

Btw, github maintains a repo with common gitignore files, and one of them is the [Eclipse one](https://github.com/github/gitignore/blob/master/Global/Eclipse.gitignore)
Thank you, this is the kind of feedback I'd like to have.
On Dec 11, 2012 8:48 PM, "bootstraponline" <notifications@github.com> wrote:

> In lib/gollum/frontend/public/gollum/javascript/gollum.js:
>
> > +}
> > +
> > +// Generic HTML escape function
> > +function htmlEscape(str) {
> > +  // The (slower) alternative is: return $('<div/>').text(str).html();
> > +  // http://stackoverflow.com/questions/1219860/javascript-jquery-html-encoding/7124052#7124052
> > +  return String(str)
> > +    .replace(/&/g, '&amp;')
> > +    .replace(/"/g, '&quot;')
> > +    .replace(/'/g, '&#39;')
> > +    .replace(/</g, '&lt;')
> > +    .replace(/>/g, '&gt;');
> > +}
> > +
> > +// Given a page name and a current path, returns a fully qualified path.
> > +function rel2abs_page_name(path, name){
>
> rel2abs_page_name( path, name ) {
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/github/gollum/pull/598/files#r2388027>.
>
>
good call

On Wed, Jul 11, 2012 at 3:28 PM, Sean Hess <
reply@reply.github.com
> wrote:

> >
> >        // servers to deploy to
> >        servers: {
> >          dev: "deploy@dev.mycompany.com",
> >          staging: ["deploy@staging.mycompany.com", "
> deploy@staging2.mycompany.com"]
> > +        prod: {
> > +          hosts: ["deploy@mycompany.com", "deploy@backup.mycompany.com
> "],
> > +          cron: [
> > +            {name: "someTask", time: "0 3 * * *", command: "node
> sometask.js"},
> > +            {name: "anotherTask", time: "0 3 * * *", command: "node
> secondTask.js"}
>
> What if you use a hash instead of a name property? It fits better
> stylistically with the layer config:
>
> cron:
>   someTask: {time: "...", command: "..."}
>   anotherTask: {time: "...", command: "..."}
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/idottv/gogogo/pull/7/files#r1144789
>
As you can see in the jsperf, it's a lot slower. Didn't think it was worth saving a couple of bytes over performance?
Wait, @gnarf37, are you simply saying to set `start = 100` and still do the loop, or are you saying something else? Because if you're saying the former, then it would be easy enough to assuage @gibson042's fears by writing a test that sees what happens when `font-size: 100em`.
How about `unmatched`?
What version of gevent are you running? I think this support only landed in 1.0.

The alternative approach which will require more work is to move our Gobject mainloop to a thread, and do something along the lines of what I suggest in #166 + SocketIOServer(...).start() instead of serve_forever() in  the main thread to keep gevent happy. This _might_ work even with an old gevent, but I can't be sure.
Blank line between `require` and `class`.
I don't think this is still used, right ?
I do another commit delete all development_dependency like you ask
Crap.  This failed to fail because the "symbol not found" is followed by the expected "No injectable members" error.  It's caused by "symbol not found" without reverting this code, which isn't really the intention of the test, but it fits the test verification criteria.   

Hrm... I am not 100% sure how to not overspecify this.  I wonder if we have a straight-up compilation error if we need some of these errors.  This is a validation error, where a dependency is missing... but the code didn't even compile... should we run validation in this case?  

If we want that, then I wonder how to specify that the error we got was the only one we were testing for without articulating a lot of "not contains" critieria and having to specify all the typical compilation errors like symbol not found and syntax errors.  That seems brittle.
`"grunt-contrib-coffee": "~0.2.0"`