labofsahil/github-event-dataset-2012 huggingface dataset:
Js Minifier Update

Hello,

I renamed the Js Minifier repository to comply with the Sublime Text Github organization.
Assigning by reference allows you to use the original CodeIgniter object...

... rather than creating a copy of it.
Issue #37 fix : Yaml Highlighting lang files

Tested successfully against the worst sample yaml files I could found.

Some examples : 

```key\a: va\lue
key_b: va:lue
key_c: {key_c1: value, key\c2: va\lue, key\c3: va:lue}
[a, b, c, d]
[a:b, c\d]
main:
    pattern: ^/
    pattern: (.*)
    a\b\c\d: a::b.c:d
        - { p: ^/a/$, r: R_N }
--- !<tag:cs .com,2002:invoice>
---
Time: 2001-11-23 15:01:42 -5
User: ed```
Update languages/woocommerce-delivery-notes-fr_FR.po
Test Fixes

portal/ldap
portal/dbupgrade/sampledatalatest/blogs
portal/dbupgrade/sampledatalatest/wiki
portal/permissions/documentsandmedia
portal/tags/blogs
properties/mailintegration
LPS-24991
added Collections project and modified README
Additional touch tweaks. Very responsive screen.

Higher noise threshold while charging.
Do not open internal links in a new window

Because it is just so logical!

http://phpbbsocialnetwork.com/viewtopic.php?f=6&t=4596
Added possibility to set the resizer adapter.mode

``` yml
# app/condig/config.yml
# ...
sonata_media:
# ...
    resizer:
        simple:
            mode: outbout   # default inset
```
Фикс для айфрейма

Перенёс код для изменения размера в единый с кнопками `EventListener`. Плюс айфрейм не показывался, если была включена проверка обновлений при каждом открытии страницы и было доступно обновление.
cast the port of the device url
readme fix

Fix the issue #2
Update app/views/admin/pages/tabs/_copywriting.html.erb

Changed

```ruby
<% copywriting_phrases = @page.copywriting_phrases %>
```

to

```ruby
<% copywriting_phrases = @page.copywriting_phrases.order([:scope, :name]) %>
```

So it orders the phrases correctly after the scope and then the name!

This is an awesome plugin! :-)

Cheers!
Tested brancing, mergin etc.

including squashing!
Fix in app/assets/javascripts/application.js 

Removed line "//= require_tree ." in  app/assets/javascripts/application.js 
See http://stackoverflow.com/questions/7281907/rails-3-1-issue-with-javascript-include-tag-in-application-html-erb for more details
Implicit wait

OK this is my stab at the implicit_wait issue.  Here is my thinking about it.

1. implicit_wait will make selenium failures take longer so by default it should be turned off:  implicit_wait=0.0
2. When s2l is initially created an implicit_wait value can be set that will then be used by each new browser that is created.  This is largely a convenience function.
3.  When creating a new browser i.e. calling open_browser there is now an optional argument `implicit_wait` if this is specified then new browser will be created with whatever implicit wait value is passed.  If not the browser will use the s2l default implicit_wait time set during library creation.
4.  set_selenium_implicit_wait will now update the default implicit_wait value and it will overwrite the implicit wait value for all currently open browsers.
5. get_selenium_implicit_wait will return the default implicit_wait value.  There is a potential issue here in that if you call open_browser pass in an implicit_wait time and then call get_selenium_implicit_wait the 2 values might not match.  The only work around I can see to this is to extend WebDriver it self to store / return the currently set implicit wait time but I don't see a real need for it at this point so I'm leaving it alone.
Isotope

Testing out isotope. The code may be sloppy. I need to test some stuff out though, it will be scrubbed later today.
Скрытие по названию видео

Добавляет выражение `#video` для скрытия постов по названию видео. Учитывая то, что названия загружаются асинхронно, пришлось делать отдельные функции:

+   `filterTextTube` - вызывается после каждого загруженного названия и сверяет, подходит ли название под регулярные выражения. Если да, то помечает, что пост надо скрыть (вызывает `hidePost`), помечает, что пост скрыт с помощью выражения `#video` и если в течении 500мс не будет загружен ни один заголовок, скрывает все помеченные посты.
+   `unHideTextTube` - отображает помеченные посты. Вызывается при удалении/отключении выражений.
+   `hideTextTube` - скрывает посты по названию видео (предполагается, что все названия уже загружены). Вызывается при добавлении/включении выражений.
Hi there, maybe you can incorporate this into the extension. It's noob code but works ;)

adjusted layout and enhanced style to work with radiant

 * add member now narrows the selection down to not-included members
 * add/remove buttons are now +/- radiant images with action styling
 * the table is displayed in a better fashion
 * the owernship drop down now does not interfere with the assets layout
 * added I18n
Still bugs me, but I figure it's not worth the headache trying to figure out why.
We're capitalizing DOI elsewhere in the code; for consistency, maybe make this DOIHelper.
Just realized that it's not a "model" but a QObject, so maybe qSlicerExtensionsModelManager would be better or just qSlicerExtensionsManager.
(Sorry for saying the same thing three times regarding #80; I didn't grasp where a comment made on an individual commit shows up.)
We also need a version that accepts a single value rather than an array.
plain vanilla handler:

app.get('/test', function(req,res) {
  res.end();
});

1. every "curl" (using localhost) will result in an exception (actually showing a bigger problem than i reported): 
curl -v  http://localhost:8888/test


Uncaught TypeError: Cannot call method 'emit' of null
    at Socket.onServerResponseClose (http.js:780:21)
    at Socket.emit (events.js:88:20)
    at Array.<anonymous> (net.js:320:10)
    at EventEmitter._tickCallback (node.js:192:40)


2. to show the problem i reported - simply issue that "curl" with multiple urls (default in curl is keep-alive). example:
curl -v http://localhost:8888/test http://localhost:8888/test http://localhost:8888/test http://localhost:8888/test http://localhost:8888/test http://localhost:8888/test http://localhost:8888/test http://localhost:8888/test http://localhost:8888/test http://localhost:8888/test http://localhost:8888/test

server warning/exception:
(node) warning: possible EventEmitter memory leak detected. 11 listeners added. Use emitter.setMaxListeners() to increase limit.
Trace: 
    at Socket.<anonymous> (events.js:139:15)
    at ServerResponse.assignSocket (http.js:786:10)
    at HTTPParser.onIncoming (http.js:1454:11)
    at HTTPParser.onHeadersComplete (http.js:102:31)
    at Socket.ondata (http.js:1387:22)
    at TCP.onread (net.js:354:27)
@shadowhand https://github.com/kohana/kohana/blob/3.3/develop/install.php#L60
Yay! Nice work!
I'd like to use one character due to shrink horizontal line length. How about using `D` (inDices, or Dimensions) instead?
var name = "\\";
is this done? or does it need more work?
Very nice :) This is great that we can also pass down URL param.
Inspect is usefull for debugging because it gives you a better insight into the data structure and its contents.
I don't see why the `name_space()` method contains an underscore?
looks complicated :)
so logger.setLevel sets teh level of *this* logging.Logger instance? or globally?
Oh, also setting the index to ".index = AE_CH_FL-1;" is not correct, you need to set its index based on the input stream's index as the order could be both FL,FR, or FR,FL.
Je n'y ai pas forcement pensé. Mais effectivement, ça serait mieux.
Yes
I have moved it to /examples.
I merged it, admittedly without a test compile. Shame on me. But shame on other people for submitting a not-ready pull request! ;)
Does MySQL support database names with a forward slash in them?
(08:29:08 PM) tomeofgreen: made a simple change : https://gist.github.com/1936760
(08:30:25 PM) tomeofgreen: Than got rid of the error, but not it just spits back the node without my pushes fromt he dolist. 
(08:30:59 PM) zokosiman: You're not defining newNode as anything, I Think.
(08:31:12 PM) zokosiman: you just have it available int he let( ((newNode))
(08:31:19 PM) zokosiman: shouldn't you end up doing like
(08:31:49 PM) zokosiman: (setq newNode (list (gensym "garden") (format nil…..)) (push newNode nodes))))
(08:32:43 PM) tomeofgreen: I think you on the do with that
(08:32:54 PM) ***tomeofgreen has not seen setq only setf
(08:32:58 PM) zokosiman: oh
(08:32:59 PM) zokosiman: setf
(08:32:59 PM) zokosiman: sure
(08:33:02 PM) tomeofgreen: lol
(08:33:06 PM) tomeofgreen: cool
(08:33:30 PM) tomeofgreen: I'll try that out and see what I can find, thanks for the input dude.
(08:33:36 PM) zokosiman: npbro
(08:33:47 PM) ***tomeofgreen is going on a redbull and sushi run
Haha! Best commit ever ;)
Ya not going to adjust the legacy URLs, but I did entirely drop support for linking to them (and errors without projects), which means data migrations is now entirely required
yep, you're right. :coffee: hasnt kicked in yet
типа такого? http://ru.wowhead.com/spell=12296
I assume this means our tarball distributions will require an install of libgfortran?
You can actually build QtWebkit (since it is a part of Webkit) completely separately from Qt. Here's how:

http://trac.webkit.org/wiki/BuildingQtOnLinux

Once you have it built, you can copy the libQtWebkit.so* into your QTDIR/lib, or you can do

    chrpath -d bin/phantomjs
    LD_LIBRARY_PATH=/path/to/qtwebkit/Build/WebKitBuild/Debug/lib/:$LD_LIBRARY_PATH bin/phantomjs
That's much better indeed
This looks good!

It won't work against the train-2012.02.02 because in scripts/production_locales ``CONFIG_FILES`` is hardcoded:

    // Pick up production languages                                               
    process.env['CONFIG_FILES'] = path.join(__dirname, '..', 'config', 'local.json');
Or maybe it's possible to just don't count any binary files in the language statistics? That might also help in the future.
please don't use SGVector's of non-numerical types but rather dynarray/dynamicarray
And committed again. If it's any consolation, I've been awake for about 20 hours :p

Thanks a bunch
This example badly needs some update :) I understand you took it from the upstream kernel install control code, but I need to update that too :)
The commit message should be "Revised the creation of rotation matrix for ligand::mutate"
Already done at the end of disconnect :)
Wheeww, I thought you were saying that `modifyMVar` was faster than `modifyIORef` =).  I don't know how `MVar`s are internally implemented, but there must be a reason.  And I don't see anything wrong with the test.  If you wanted something even more complete, you could add a plain loop in IO summing something outside a reference using an accumulator, and a test using `TVar`s.

Note that if your test had multiple threads accessing *different* `IORef`s with `atomicModifyIORef`, I guess performance would suffer even more.  (Accessing the same `IORef` would not be meaningful since on this case the primitives would be giving different guarantees.)

So there you go, another performance improvement.  When will things stop getting better? =O =)
ah pas cool les problèmes d'encodage :/
how's this look if you add `class="listing-footer"`?
don't delete these. andy wrote these for keeps. these are good to stay. but move them into `submit/tests/` if you'd like.
Matt,

I think it is better to use the branch Diff3 rather than Diff2...
- diff_path does not generate a patch compliant with unified diff (my change almost did, but context was missing). 
- Diff3 makes use of an unified diff :)
Yeah, I thought you were asking me the question about the white space. And, no, I haven't. I is noob :/
Ok...that was good..you can commit it to the codebase..its useful..
For me, as a non native with the English language, I associate the action to "pin" something as marking it important somehow. In a way similar to "liking" something but without the emotional aspect of it - leveraging the objective importance more than my personal preference. (And I hope all the years of learning English at school have not been a complete waste. ^_^)
If that is the intention, then I can say I like ("pin") the idea of having something of our own for the (probably) most used action you can apply to a post in a social network.
Geoffrey fixed this in the 5.3 .x branch:  https://issues.jboss.org/browse/GUVNOR-1785
Do you mean:
   Template '%s' does not reference any resources


ie. The set is empty?
@solnic one thing you'll see in DM1 is that I provide "state" objects that know how to commit/rollback. They were an extraction from the DM UoW work, but I never got the rest of the UoW working.

I think that might not be such a bad thing to consider; a state machine is created for each object added to the session, the state machine is responsible for handling state transitions of the object when the session is committed or rolled back.

For example there could be a state for representing a "dirty" object. It would have a commit and rollback method. When the session is asked to commit, the "dirty state machine" would know how to persist the object and return a new state machine for representing the "clean" state. Likewise, if the session was asked to rollback, the "dirty sm" would reset the object state, and return a new clean sm.
What is all this about « Simon's game » ?
Can you be for patch-like specific please? :-) Thank you.
I've tested the code and seems to be working for me too (for those that haven't read https://github.com/callmeal/Flickr4Java/issues/2#issuecomment-3505254, which pretty much says the same).

I have two remarks regarding your commit:
1- it seems to me that getRequestTokenEndpoint(String callbackUrl) is never called, since the implementation of OAuth 1.0a in Scribe uses OAuth10aServiceImpl which in turn calls the getRequestTokenEndpoint(). Feel free to correct me if I'm wrong.

2- when doing the 2nd step in the OAuth dance (by calling the getAuthorizationUrl function), Flickr seems to be a bit picky regarding the perms attribute. That is, by not passing it, I get a "Oops! Flickr doesn't recognise the permission set." error after accepting the request on Flickr's site. I've unblocked this in a quick and dirty fashion by adding "&perms=read" to the string returned by the getAuthorizationUrl function, but this is definitely not a good solution. Have you (or anyone else) encountered this problem, and if so, do you have an elegant solution for it?
Existence of std::bitset<5> - Section begin:
hmm, forget it. We will likely receive a bunch of 404 responses for deleted branches and each one would try to trigger the auth
Thanks for doing this. I basically hacked together all the support modules for clojure-py over Christmas vacation. So I admit, I was more concerned with results than clean code. Thanks for taking the time to clean this up.
New in 0.2.9 is that we're reading the `data-responsejs` attribute inside a ready function, because if it's not done in a ready function and the script is loaded in normal external tags in the `<head>` then the `<body>` wouldn't be there yet and the data value would come back `undefined`.
I love the 3pm border around each image - looks very nice :)
you're defining index_a and index_b in the middle of the code
"optionally" ?
Should line up with the quote character, not the first character inside the quotes.
Thanks for the quick fix.
Assuming that this actually works it looks good to me.  Except: you should remove the old ./manage.py require and update the Readme.
Lots of extra whitespace
+hello world!
Yeah, that's what I'm thinking. I kind of hate the repetition but I don't live in JSON-land and I suspect it's the idiomatic solution...
Yeah too bad their server is way slower than google docs ;)

Ian


On Mon Mar  5 17:28:31 2012, Andy Rossmeissl wrote:
> woohoo!
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/brighterplanet/earth/commit/8b51b04ee3b828841ae8ee15a3d885db85a8408a#commitcomment-1043712
is -> if

is "intercepted" really the word you want on 291?
How about s/handle/Logger instance/.
This does not fix the issue in ticket 75. It does at least minify the coffee files, but does not process require calls within those files.  Try to test it against the test case project I included in the ticket. I've added some comments on the ticket as well.
Can you please explain why we can do without the call to connect()?
Brady,
While there isn't a need to bind static strings, I personally think it's more consistent to do so and probably good practice.
Also suppose that it were a static string with special characters that needed to be escaped instead of an integer.  Wouldn't it make sense, even though static to set it through a bind?
Plus suppose in the future someone wanted to reuse this code and change it to a variable?
D'uh! I should have seen that :-) OK, at least that fixes my view on the shell :-)
@harikt: thanks for that. It looks like ApiGen uses the PHPDoc/DocBlox/JavaDoc notations.  So it's another *generator* but it's not another *convention*.  Any others?
Although the code is not written professionally but WG begins to take shape
Does there need to be a top.restoreSession() call somewhere?
Not mine.

-----Original Message-----
From: Thomas White [mailto:reply@reply.github.com] 
Sent: February-18-12 12:30 PM
To: Tony Partner
Subject: Re: [LimeSurvey] Fixed issue #05752: statistics_user.php - always shown in baselang (f495f51)

can you also push this to the dev branch, please?

--- 
Reply to this email directly or view it on GitHub:
https://github.com/LimeSurvey/LimeSurvey/commit/f495f51389775abb690e3c90c58f968e08ee537d#commitcomment-981324




=======
Email scanned by PC Tools - No viruses or spyware found.
(Email Guard: 7.0.0.21, Virus/Spyware Database: 6.18900)
http://www.pctools.com/
=======
Ok, I'll correct this today ;)
Have you tested this change with eclipse ?
@hramos, how's testing coming along? Let me know if you need any help.
I have just figured out that. I have had "Optimize import on the fly" switched on.
Just keeping a watchful eye out for ya!
Damned! Are you sure?
What is your error? It doesnt find the relation key?

--
Envoyé de mon iPhone

Le 5 mars 2012 à 12:20, Toni Uebernickel<reply@reply.github.com> a écrit :

> This breaks the `AbstractDataLoader`.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/propelorm/PropelBundle/commit/3397d7422dff3b9067ae69f7899c30b46dd17508#commitcomment-1042615
There were 2 issues here. The first is a timing issue -  the map is not being created until the jquery `document.ready` function - `$()` and the map bounds_changed event can't be tied to the widget until after the map is initialized. Also the map variable was being set as a local in main.js, and the widget bounds changed listener was expecting it to be attached to window - e.g. `window.map`.

So I moved it to the main.js file after the map is initialized and updated the map reference which works, but not sure if that's where you think it should live in your code. So feel free to move it but keep in mind the timing as well as using a valid map reference for the widget event.
嘿~ 试试看 janus vim distuibution? 
https://github.com/carlhuda/janus
自定义配置不需要fork, 只要自己建立一个repo即可, 比如我的:
https://github.com/crhan/.janus
I see.  Ignoring a separate model instance with the same id would be inconsistent to say the least.  Thanks for the clarification!
@rejuxst implemented this but it references teams not players in order to work for all team sizes. It is technically possible for winner to be none when it can't be determined.

```python
print "Winner {0}".format(rep.winner.players[0] if rep.winner else "Unknown")
```
You might want to consult a dictionary on the spelling of "amount". Also, in the interest of i18n,  do not mix languages, please.
Hi Steven. Any word back on whether you can create a pull request for this?
The philosophy behind this not being in the program is that if you need to save in anything but UTF-8 then the software that consumes your subtitles needs to be replaced.
@jacobcore, I've never seen you contribute shit so shut up and get back to making money from our work you idiot.
Loss of home star penalties? 

http://www.hrwiki.org/w/images/thumb/a/a4/NewestHomestar.png/180px-NewestHomestar.png
We use trace for that and the following syntax

    log.tracef("Retrieve database %1$s", dbName);

It's a bit faster as the string is not concatenated.
Makes sense I suppose. I expect Apple's code is looking for `.` and `..`. Only when that occurs does the full cleanup happen.
AFAIK, jQuery doesn't provide a "safe" shortcut to use. The same thing can be achieved with a map as well, though it is somewhat more verbose:

    var revisions = $.map($('.revisions'), function(el) {
        if ($(el).attr('revname') === title) {
            return el;
        }
    });
codestyle x.x
perhaps code would be clearer if we just removed this? We can always look at git history if we decide we need the code back...
Fixed in the newest push. Please, review.
> in contrast to simply using the `:branch' property, which will cause el-get-to try to merge your local changes

Ah! Now I see why `:branch` exists!
I agree with your your compile-time checking point, but why using the class for simple strings too? Also, my 50 line localization class actually supports fallback language for missing translations.

Since at the moment we don't have much strings to display I won't insist further, but the few projects with l10n I have seen (including Minecraft) all used something close to my approach of managing strings. Maybe a third opinion would be welcome!
Was man aber auf jeden Fall machen sollte, prepared Statements verwenden:

```php
<?php
$gm->setQuery('select * from '.rex::getTablePrefix().'module where output LIKE ?', array('%'.$searchtext.'%'));
```
Will do a more thoro' review tomorrow  but this does seem safer. 

G.
This fix not works for me.

If learn higher rank (than 1) of profession, example enchant. In rank 1 is all right, in profession menu has disenchant and enchant button, but when i learned rank 2 - disenchant button isn't here and here is enchanting rank 2 and rank 1 button and dont show any others profession (dont learn second primary profession, becouse enchant rank is as 1st prof and rank 2 as 2nd prof)

I think, than is not db bug and sql-fix dont fix it.
I think than is problem in somewhere in loading spell_rank or comparing of ranks, but game shows all learned ranks, we need show only highest rank of prof
------------------------------------------

#228 - Here 2010phenix posted fix in spellMgr.cpp and this fix works for me.
i've found out that this error doesn't happen when rebooting from android (gtacomb); cap buttons work ok in that case.. even treznorx ics .39 kernel seems to behave the same way.. i can't test if recovery reboot acts the same, cause i have kernel installed in recovery partition
Me parece bien. Actualizo
about the deepCopy() thing:

You still have to deepCopy() a TiledTextureRegion, if you want to use the same TextureArea on different TiledSprites.
Redis-launcher is a Java library for controlling a redis server programmatically.

Installing from Maven Central
```xml
<dependency>
	<groupId>uk.co.datumedge</groupId>
	<artifactId>redis-launcher</artifactId>
	<version>0.1</version>
</dependency>
```


Installing from source
mvn -Dredislauncher.command=/path/to/redis-server install
alright. thanks for the update!
No puede ser, si no hay DoScriptText aqui jajaja
== kernel was able to boot at this state ==
This if-statement would allow a username that has 0 characters.
Thanks for the message. You are right, there's alignment issue I forgot to
consider. I'm also thinking to improve the interface of fast_slob a bit, so
I will be pushing out the fix along with other changes soon.

I've been testing on goldfish emulator, which might be why the alignment
issue didn't show up in the first place. But I did found out that the
application crash was due to some other reason, will push out that change
very soon as well.

Thanks again for pointing that out.

Regards,

Rong




On Mon, Feb 20, 2012 at 4:14 PM, syndtr <
reply@reply.github.com
> wrote:

> The crasses can be caused by the buffer that allocated by fast slob is not
> well aligned. I implementing your fast slob in the old binder and get
> BUS_ADRALN, which mean Invalid address alignment.
> Fix it with this:
> #define MAX_ALLOC_SIZE          98304
> #define ALLOC_SIZE_SHIFT        3
> #define NUM_BUCKETS             4
>
> The NUM_BUCKETS must not be odd number. Hope it helps.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/rong1129/android-binder-ipc/commit/bdc050fcf0b7f33a310a01557c6cfea417a954df#commitcomment-983624
>
111 ".jelly" length is 6, but ".groovy" length is 7.
How about ?
updated the gem with fixes..... a bit late though, didn't get enough time over weekend.....

now, I'll add a way to also get node-value with tags and not just inner-node value, "when required"

I've two things aimed for it after that:
* experimentation for performance updates
* only-attribute based node search

what should be on priority
I think it's better to only look for the first language code. Will remove it.
Ойлоло. Знаем мы, ога.
Енто у вас в отладчике. А вставьте
	alert(post.Text.indexOf('\n'));
	alert(post.Msg.textContent.indexOf('\n'));

в

function togglePostVisib(post) {
	alert(post.Text.indexOf('\n'));
	alert(post.Msg.textContent.indexOf('\n'));
	post.Vis = post.Vis == 1 ? 0 : 1;
	setPostVisib(post, post.Vis);
	savePostsVisib();
}

И понажимайте на кнопку скрытия, дабы увидеть реальный результат
What do you mean by "plain drawable icon"? @drawable/sync_icon? This is not a rename, it is using a different (existing) resource, so that Android can select the correct resolution icon automatically.
Here is a note on a line
Actually there was no bug here, I just cleaned up the presentation in the source. If the customer wasn't logged-in, class was: " template.." so began with a space. I was already 'in there' changing the code with a previous commit.
Cool, thanks. Would you mind adding some tests by creating a sub-app in tests/testproject and covering your use case in tests/testproject/testapp/tests.py?
At home, I have the latest version of Debian, Boost 1.47.0, and I don't have libboost_thread-mt.so, just libboost_thread.so

You can just leave it out of setup.py, so why bother?
Was a bit sceptical if val() works for all fields symmetrically, particularly stuff like checkboxes and multi selects,
but seems to be the case (it passes arrays around for non-scalar values): http://api.jquery.com/val/
\subtitle gabs kannte meine LaTeX-Distri nicht aber ich bin auch zufieden, wie es aussieht.
rm temp file :((
yeah i2c completely

On 10 February 2012 01:24, treznorx <
reply@reply.github.com
> wrote:

> Did this break something?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/fards/android-tegra-nv-2.6.39-rel-14r7/commit/d72a038a0a3e131956ac5ed19d8587c6927a88e1#commitcomment-951899
>
What is this dependant apon? Why does it work now?
sorry for reformating the pom :-(
the important config is : 
<dependency>
     <groupId>emma</groupId>
     <artifactId>emma</artifactId>
     <version>2.1.5320</version>
</dependency>
Great Job bro glad to see ya chiming in
geemus:

So I looked into adding some error checking for this... but it doesn't appear that the OpenStack compute service even defines the old :openstack_idententy_service_name option which means we can be sure nobody is using it externally.

If they were it would just throw an  Unrecognized argument exception. Given it is just used internally I think this part of the change should be totally safe.
I'll remove that patch.
How does this sound? 915251f80d72789e84ee1e3f42d4909df5b2e66d
ok, will do/
Not 100% sure but use can use  (string)$myVar; for casting
There is an implicit conversion involved for `max` here, right?
Use max in Duration instead, and define a `1.nanos` constant.
Good point, it is Java API, will fix
This is not used by the new remoting, it is there to make the old  
NettyRemoteTransport happy (it inherits this stuff). I have a completely  
separated serialization service.
Well, I guess they're not Essential from a bootstrap standpoint.
However, I'm not seeing how a user will have package descriptions for git or svn, and their dependencies, unless we provide those  at bootstrap time, or if they use another selfupdate method initially and then switch--the latter course seems like it wouldn't be desirable for folks behind firewalls.
I totally agree. How can one define these extension points?
Same as above, doesn't cover the partial case.
IMO, remove the bold here. Maybe it's just the word choice, but calling out so strongly to a phrase that is so vague ("plenty") feels awkward. I'd leave the phrasing and just remove the bold.
oh I thought it was, I'll check

On 8 October 2012 22:25, jmarshallnz <notifications@github.com> wrote:

> In xbmc/filesystem/UPnPDirectory.cpp:
>
> > @@ -449,6 +457,9 @@ bool CUPnPDirectory::GetResource(const CURL& path, CFileItem &item)
> >                  }
> >              }
> >
> > +            if (pItem->HasVideoInfoTag() && pItem->GetVideoInfoTag()->m_resumePoint.IsSet())
> > +                pItem->m_lStartOffset = STARTOFFSET_RESUME; // resume point set in the resume item, so force resume
> > +
>
> This shouldn't be required I think - i.e. the user should be prompted if
> the resume point is set?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/xbmc/xbmc/pull/1569/files#r1789224>.
>
>
btw, you should simply use ``if`` as the previous ``if`` leaves the method because of the exception
What happened to my comment?
Rename for ServiceConfigurationHandlerregistry + javadoc
move posn-x to previous line?  (same for posn-y and make-posn below)
Yeah, it was simpler to keep it in bytes. Ideally you'd be able to set a string like `"1M"`, `"10K"`, which we'd convert.
Well noted.  Done
It could be helpful to have an optional `Function::inverse(double y)` method:

```
virtual int inverse(double y) {
    throw CycNotImplementedException("blablabla");
}
```
These changes need to be reverted.
You're quite right – sorry I missed that. They should both be `YES` (I don't know why the other one wasn't). Similar to the dot syntax thing, it doesn't change any behavior, but just matches Objective-C convention.
That function takes an integer input- not a `CFIndex` so a cast is necessary.
These durations continually change - there's 5 or more of them.
Eugh. It was bitching at me about not supporting weak and for some reason I went away from strong too. Fixed.
This should be `NSUInteger`.
Oh alright :]
I take that back, it needs to be allocated on the heap anyway so you might as well use calloc().
```sh
sudo chmod -R a+rwx application/cache
```
See: [man chmod](http://en.wikipedia.org/wiki/Chmod)
Not adhering to Java naming convention.  Please do not prefix a variable with a underscore.
Notes section goes above Examples. Most of what's in this section belongs under Examples though.
yes, including php.h in Zend Engine files is definitely a no go.
This should be

```c
if (zend_parse_parameters_none() == FAILURE) {
    return;
}
```

Otherwise the function will continue to run even if parameter parsing fails.
пока плагин с драконом не закончен, наверное его лучше закомментировать
Противоречит стайлгайд 3.4
ну ладно, всем все равно :)
ок
/cc @drogus @josevalim any idea?
Tests throw "method is already defined" warnings.
Nice, I feel like I have to learn Japanese :)
We need `flatten` for cases such `order(:id, [:date, :name])`
But i agree, overall we do not need it
Actually, this is automatically adding an index, there is no way to rollback. I would like so suggest a different approach that may also simplify things a bit:

1) Always generate `create_join_table` with a block;

2) Always generate `t.index` for both attributes, the difference is: if the user passed `users:index`, `t.index` will appear uncommented, if not, it will be commented.

The reason is the same others commented here, we cannot make a smart guess about the application indexes and since indexes are not free, we should not add them at will.

Thanks a lot for your work.
Because timeLeft allocates a new object.
i personally don't think longer names are a problem, so i'd go for `isImplementationArtifact` everywhere.
I think this can achieve the intended end more easily as
```
lc(line.tail stripPrefix cmd trim)
```
It's strange I'd have written it as it is, assuming it was me; I wonder if there's some case where I had to preserve trailing whitespace.
The first argument for the label tag should be the attribute name on the model. The second should be the translated version. Therefore, this should be:

     <%= f.label :name, t(:name) %>
I put a print statement to show what `exp(a)**b` expressions were no longer
being simplified to `exp(a*b)` and got the following when running the test
(bin/test -k mellin):

```
no change exp(-x) -I
no change exp(pi*(-_a - 1/2)*exp_polar(I*pi/2)) 1/2
no change exp(pi*(-a - 1/2)*exp_polar(I*pi/2)) 1/2
no change exp(I*pi*(-a - 1/2)) 1/2
no change exp(pi*(-_a + 1/2)*exp_polar(I*pi/2)) 1/2
no change exp(pi*(-a + 1/2)*exp_polar(I*pi/2)) 1/2
no change exp(I*pi*(-a + 1/2)) 1/2
no change exp(pi*(-_a - _b)*exp_polar(I*pi/2)) 1/2
no change exp(pi*(-_b - a)*exp_polar(I*pi/2)) 1/2
no change exp(pi*(-a - b)*exp_polar(I*pi/2)) 1/2
no change exp(I*pi*(-a - b)) 1/2
no change exp(-I*pi*a - I*pi*b) 1/2
```

The rationale for combining powers is in the previous commit.
evalf has lots of other args to consider, e.g. e.evalf(n=3) -- see the evalf docstring and parameter list. Those options should be available to all `_eval_evalf` routines.
bu type casting'i sadece this.element = ... yerinde yaparsan her yere yazmana gerek kalmaz.
Yup, I think so. If that's not the desired behavior, it's trivial to write a similar handler using `matchAll`. Good call on the names.
I've read on PHP.net that using mkdir mode can cause troubles with fpm. If I'm understanding your question right.

Alexander Makarov <notifications@github.com> wrote:

>> +
>> +	/**
>> +	 * Creates directory for $dst path with $mode and $recursive creation is allowed.
>> +	 * For concurrent compatibility chmod is used instead of mkdir's $mode.
>> +	 *
>> +	 * @static
>> +	 * @param string $dst path to be created
>> +	 * @param integer $mode access bitmask
>> +	 * @param boolean $recursive
>> +	 * @return boolean result of mkdir
>> +	 * @see mkdir
>> +	 */
>> +	private static function mkdir($dst, $mode, $recursive)
>> +	{
>> +		$res = mkdir($dst, $mode, $recursive);
>> +		chmod($dst, $mode);
>
>Why doing `chmod` here?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/yiisoft/yii/pull/1146/files#r1810581
What do you mean by "it's a double here"?
You can just do:

return this->get_id() == other->get_id() && 
            this->get_hist_name() == other->get_hist_name();

instead of the return true/false
Yeah, since one always use the functions instead of the tuples, I think it's okay to leave it.
This seems a bit fragile to me. Let me know if you want me to implement it in a less fragile way.
Yeah, the reason I think it's fragile is if someone does customization to the output the order of nodes could break. It's also easy for this to accidentally break in the future if someone changes QUnit internally.
I've now added view support. This also includes supporting them when doing a normal inventory upload, which is something the inventory support itself was previously missing.
It lists all extensions that exist and for them to be registered they need to have an ext.php

If there is an extension and its missing that file it would show this error which makes it incorrect.
Curly bracket on new line?
2005?
Should be user, not account
Because createFromFormat needs format and value to be the same system?
This is not an issue because we require 5.3. http://php.net/manual/en/function.is-a.php
https://github.com/phpbb/phpbb3/pull/608#r1361502
There something wrong here, I'll fix this.
could be null. need check for that here.
Does something like "999999L" works in C as well?
Drop the "is_", it should simply be named: preboot_exists
Since we're there, wouldn't it be better if I passed the result of the function?
Findbugs / PMD usually complains about this kind of thing as the handle will be disposed of naturally by garbage collection.  Years ago there was some theory that nulling things out early guaranteed that the memory was cleaned up but that theory has long since gone away.  I think that with java 4 and before it probably was necessary but as of java 5 -> it shouldn’t be an issue any longer.  Both these tools simply state to stop doing this and to remove the code.

 

Thanks for merging!

 

I do have a couple of issues I really was trying to resolve in all of this.  I have issues with 3 tests that have never ran on my machine.  I’ll keep digging on those.  Maybe I can get lucky and get the issues resolved.  For the new “git” ant task you added, what software specifically do I need to add for that to work?  When I build, it fails so I have to comment that out to get the build working at the moment.  I’m guessing I simply need a different version of git than I’m using.  I’m using git hub for windows so likely need another client.

 

Thanks,

 

Jeremy

 

From: Daniel Doubrovkine (dB.) [mailto:notifications@github.com] 
Sent: Sunday, August 19, 2012 12:07 PM
To: dblock/waffle
Cc: Jeremy Landis
Subject: Re: [waffle] Findbugs / PMD / Junit Updates (#21)

 

In Source/JNA/waffle-core/src/waffle/windows/auth/impl/WindowsCredentialsHandleImpl.java:

> @@ -92,7 +92,6 @@ public void dispose() {
>                      if (WinError.SEC_E_OK != rc) {
>                              throw new Win32Exception(rc);
>                      }
> -                    _handle = null;

This kind of stuff is useful for debugging. This way we know the handle has been disposed and the object is in sync. Is there a good reason to remove it?

—
Reply to this email directly or view it on GitHub <https://github.com/dblock/waffle/pull/21/files#r1410118> . 

  <https://github.com/notifications/beacon/cfbemxk8F8smNvKeAYgxsfGkJ_8-7DJ1XLdoVKfSKExIACJewjVQPil_mPry_E1h.gif> 

  _____  

No virus found in this message.
Checked by AVG - www.avg.com
Version: 2012.0.2197 / Virus Database: 2437/5209 - Release Date: 08/19/12
I beieve this would be better handled in the mixins-date-based reference so taking it out.
Having them is a style nit of mine :) There's no consistent usage in Django so far, but I can drop them if needs be.
It's a bug fix.

    $ cabal help sandbox-init
    Usage: sandbox-init cabal [FLAGS]
Is it possible to avoid "SuppressWarnings"?
You mean GP4.2 is now SQL standard compliant here?
Nit: I don't think this closing brace is indented correctly.  (Also there should be spaces before the {s.)
Can use:

    s.change_make_var! 'CCS', ENV['MPICC']
I think that we should set the default of low_pass to None too. We shouldn't be doing things behind people's back.
We should use -v with --verbose instead of -d - it is far more common among command line programs.  We should also purge references to --trace (tie it into --verbose).  The text should be more appropriate - "Display more output from your commands".  If we truly need a debug level above verbose then we can introduce it later.
These two conditions can be grouped probably:
https://github.com/owncloud/core/blob/d62f138153a01437709675f44a46027c28aa0281/apps/user_ldap/user_ldap.php#L139
https://github.com/owncloud/core/blob/d62f138153a01437709675f44a46027c28aa0281/apps/user_ldap/user_ldap.php#L145
Ok, i change it to lowercase.
try: except: added because this broke on boolean operations in com.take_multi_2d - so will have to revist at somepoint
I'll take 'never happen' with a pinch of salt - we don't know how the code might change in the future.

Yes, I'd also thought that a base class with two subclasses would be more 'correct', but probably overkill. ;-)
so the license is present if there is at least one single line comment at the beginning at the file, regardless of what it contains (possibly nothing) ... i'm afraid this is totally incorrect
This is external API usage. I suggest getting familiar with mongodb.
da man als entwickler wahrscheinlich davon ausgeht, dass die extension aufgerufen wird wenn man sie registriert, ist es denke ich im sinne des entwicklers ihm das mitzuteilen...?

Vllt könnte man es auch als WARNING unterbringen..
Aborting raises an exception, so it's unnecessary to return.
``` ruby
Ariane.dynamic_breadcrumb.should be_true

```
This is pretty dense.  It could use examples, or link to a particularly helpful set of testcases.
I'm fine with the change.  Can you match indentation to the rest of the file?  Thanks.
no need for the return here
Yup! I haven't had a chance to run the CloudServices unit tests against this yet, got stuck sitting through VS2012 installers... going to try it now. Do you want to wait on the merge, in case we find something unpleasant?
I like it! I wanted to move the list out of the way so the user can see symbol or line highlighting. This looks much better
This embedded style sheet should be moved to an external style sheet.
Also, just to clarify, BEFORE and AFTER will make the new item a sibling of the relative item, whereas FIRST and LAST will make it a child, right? (In other words, BEFORE and AFTER make sense for both menus and menu items, whereas FIRST and LAST only make sense for menus and submenus.)
Change to ``require("i18n!nls/strings")`` after fixing the require ``config()``. See prior comment.
I ended up moving this function so it's an inner function of normalizeKeyDescriptorString(). I also renamed it since it doesn't actually answer if a string is a modifier, it's more of a specialized string compare for normalizeKeyDescriptorString().

-----Original Message-----
From: Randy Edmunds [mailto:reply@reply.github.com] 
Sent: Wednesday, July 11, 2012 7:02 PM
To: Ty Voliter
Subject: Re: [brackets] HTML Tag Code Hinting (#1236)

> +
> +        if (hasCtrl) {
> +            // Windows display Ctrl first, Mac displays Command symbol last
> +            if (brackets.platform === "win") {
> +                keyDescriptor.unshift("Ctrl");
> +            } else {
> +                keyDescriptor.push("Ctrl");
> +            }
> +        }
> +
> +        keyDescriptor.push(key);
> +        
> +        return keyDescriptor.join("-");
> +    }
> +    
> +    function _isModifier(left, right, previouslyFound, origDescriptor) {

This function needs a header comment

---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/1236/files#r1146441
Our coding style is to use camelCase, not under_scores.
This works to vertically scroll rename box into view, but it can also be out of view horizontally.
It's used for unit tests. When loading a test window, we don't want that window to automatically check for updates since that could disrupt tests by showing a modal dialog.
We always want to populate the file tree. The changes below are caused by changed indentation.
These are out and in master already.
It shouldn't matter one way or the other really. It's just a counter and reconnecting the the master will nearly never fail. It perhaps should though; I'll move it.
Can you put this on the same line as the method please?
Why not `request.env['HTTP_SCRIPT_NAME'] || request.env['SCRIPT_NAME']`?
It's good to know the classes actually exist, but it'd be nicer to go one step further and assert they're actually Exceptions.
See how your friend is doing this in #77.
Ahhh. Sorry. Yes. In this case it's better leave it is in your original pull request.
Why are you caching this value if it is only used once?
Jks
font-family: "Open Sans"
If this is supposed to be inside <code>.butter-timeline</code> then why not and take advantage of LESS?
Above there I have a check if it is a number, which 0 hits, and we just return it if so. It is already a number, and in seconds.
move this check into the for loop
Should return the promise?
String.format() probably a little better here?

    throw new IOException( String.format("Cannot rename file '%s' to '%s': %s",
        hiddenTarget.getAbsolutePath(), target.getAbsolutePath(), e.getMessage()), e );
52100 is too aggressive for weak server. What about 20000?
Yeah I think we should make the interface return `Future[Unit]` :\ I'll file a ticket
how is this Answer related to the one in SPARQL.scala?
I think that you ignore here too much cases like ```\ ```
Perhaps we can drop the _jl_ in front of library handles, now that it is in a module?
There is already a condition for skipping the steps, would it be possible to incorporate the check there to have only one if there?
Actually this entire method s not used (there is no route to it).  I think it was done this way because that name_search actually was calling to pulp which returned all results.  In other places, we switched to using elastic search which did not require this trickery.  I'm gonna remove this method alltogether as system templates now uses  Packages_controller#auto_complete_name
same here. there is leading tab instead of spaces
I don't think this is actually needed is it?   @user.update_attributes should suffice.
Interesting method name considering it takes no params.. I realize you moved it from SystemTasks where it used to accept tasks.. Do you wanna try to change this to something like task_message or something :) .. I know a bit nit picky but will be nice )
Is there a reason that we chose to add the resource here vs add the 'promote' action to the existing resource?  The existing resource is scoped under organization; however, from CLI, we do have the organization information that can be passed in.
you should use dependency injection for the Pusher instance instead of creating it internally (which will avoid having to modify the source code of the library to go in debug mode btw)
Add space here.
Oops, I think the "shade" is dead code, my bad.
`'北京' as default?` I would recommend remote the prototype object altogether (since you are setting them in the constructor function.
If you delete all the code, you could remove the method! :)
Oh wait, you are saying nodes are not stopped until this process and its children have died already. I get it.
I'm thinking for minutes now, but have no clue, where you would like to place max(hSocket + 1) or do you mean max(hSocketMax, hSocket + 1)?
There is already a line that says when it disconnects, so with this patch, you'll see the Misbehaving: (80 -> 100), followed by a line that says disconnecting. It can be derived that the disconnection was for misbehaviour as the two lines would be together, and the Misbehaving number would clearly have reached 100 (or higher).
I refered to your comment here and just wanted to know if it includes a clean shutdown of threads started via ``CreateThread()``, if the client crashes and if the crash is unrelated to the function here itself.
I would add a check on filters.length == 1
When true, the logical filter is useless
```php
return strrev(reset($parts));
```
Zelfde opmerking als eerder (ivm optionele parameters). Deze is al geminified dus daar kan ik wel inkomen.
I like this class. Salat is a fine serialiser and half an ORM but anything persistence implementation related is best well hidden behind an interface that only has high level calls. I think you are doing a good job of that here.

I am sometimes wary of letting the serialised object return through a method call result. e.g. `getDeploys` hands back the `DeployRecord` case classes. The reasoning is that serialisation case classes can attract nasty compromises to do with the serialisation library that have no business being in a domain value object that I want to use in my nice design.

A pet peeve. Taken to extremes I'm copying values between almost identical simple case classes, of course.
Can this file import it from IPython.utils.encoding as well?
Good catch!  Fixed here and in `interactiveshell.py`, and added a comment explaining this so it doesn't happen again.
Isn't this case already covered?  I see it does a join and a split on ','?
Regarding 'osx' as an option:  I have no idea...

The list of options was formed by copy & paste from here.