labofsahil/github-event-dataset-2012 huggingface dataset:
Position of done button in modal and ActivityIndicator added to toolbar

1.
The position of the done UIBarButtonItem in SVModalWebViewController can be changed using the doneBarButtonPosition property.

2.
Use the loadingBarButtonType property to decide if the refreshStopBarButtonItem should show a stop button or an activity indicator button. The two button are wrapped inside a loadingBarButtonItem whose getter method returns either a stopBarButtonItem or activityIndicatorBarButtonItem depending on the value of loadingBarButtonType.
Disable debug-toolbar middleware completely when DEBUG not True

Minor efficiency gain.
Added a nice question about ruby symbols and strings
H pce
Profile field edit using textarea does not parse HTML entities.

Profile fields which use textearea to be edited does not parse HTML entities. It is due to using `.text` instead of `.html`.

http://phpbbsocialnetwork.com/viewtopic.php?f=44&t=4451
Fix for non-iOS5 only ARC projects

No, this doesn't make the library ARC compatible.

What I have done, I changed a __weak to __unsafe_unretained.

__weak is supported in ARC in iOS5+ only (http://blog.mugunthkumar.com/articles/migrating-your-code-to-objective-c-arc/). The fallback is __unsafe_unretained

With this modification, people can still use this project in a ARC project, where all .m files use the --fno-objc-arc flag, without it, this library is completely useless as-is in any ARC project that support 4.0+
Päivitin README:tä

Siitä tuli toivottavasti tosi hyvä.
Folders now settable via config.php + Auto create folders + Sanitize filenames.

* Folder names can now be set via $['config'] as "snippetssync\_sn\_folder" and "snippetssync\_gv\_folder" respectively. _(the fallbacks of 'snippets/' and 'global\_variables/' remain if no folder is otherwise specified.)_
* Added automatic creation of snippets & global\_variables folders if they don't exist. _(If the designated folders don't exist yet, the script creates them if it can get write permissions to the folder.)_
* Any file extension can now be used (if any). _(As opposed to be confined to .html. Some might like to use .snip or .gvar or something…)_
* Filename is now being sanitized to allow only `a-zA-Z0-9-_` as per the requirements of global vars and snippets in EE. _(Previously it was possible (and write this to the DB) to create filenames that would result in illegally named global variables or snippets.)_

## Note!

__Please__ check my code, I did test everything but I wrote all this after only having just learned PHP + CI in the last 5 days. If stuff is not quite right I would appreciate you telling me what I have done wrong so I can do better next time.
if you define plotOptions: { line:{ dataLabels: {x:0} } } it would inter...

...pret x as -999, fixed by checking for 0.
Pull req - Removed configure.bat + updated maven build scripts

Removing windows configure.bat script + updated maven scripts. We will now only maintain the one configure script[sh]
Fixed FormView#remove() on serialize_only fields

Relates to #22.
Changed /

Changed /. My site is nice like rice with content.
Some reference for this conversion table would be nice (a URL?)
crap, okay... i'll see if i can get a new release out today.
I'd love to discuss what we can and can't do, but please take that discussion to the curl-library mailing list.
Shouldn't this have gone in `v0.6` branch?
to právě nevím ale mám ve zvyku po sobě uklízet
nevermind.. I just noticed it is in 3.2 and master
That certainly sounds like a better approach (this bit did feel a bit hacky... and it's the main bit).  
I'll test it to tomorrow and update.
What if "data" points to a malicious string from a specially crafted packet so that there is no \0 before "size" ends? Make sure you cut off displaying it at "size".
Shrug, up to you I guess. Win32 APIs accept both, so it seems reasonable to do so here. For Chromium, it's probably not a big deal as I can make gyp emit one or the other as long as I can tell what part of the input is actually paths.
Alternatively, just fix writing the size as described above
Could we use some include-guard logic to #error if this is #included after any other numpy header? I can imagine someone putting this after ndarrayobject.h, and getting confused why things are don't work the way they're supposed to, for example.
I'd prefer using the absolute pathname `/sbin/zonename` instead of relying on `/sbin` beeing included in PATH
it does fix master shapeshifter, moonkin form and posthaste
```user = User.find_by_username(params[:id]) or error 401```
looks more meaningful to me.
ugh...? Seems like bad idea to me, isn't better to check if script system has actually finished loading? And why are using this in here?
That last part should be in its own section, not in "Cleaning up history"
I think the sign of the continuity correction should depend on whether U is bigger or smaller than n1*n1/2.  See, for example,
    http://faculty.vassar.edu/lowry/ch11a.html
or
    http://www.unistat.com/guide/nonparametric-tests/unpaired-samples
No, it's better doing it with SpellClassMask, beacus the override aura have same spellclassmak of the affected spellfamilyflags :P
Ok the Endstops works, are NOT the Printing !!

My Prusa have an range  x and y 215mm
and Z 95mm

When i start the Print then drive Z UP ?? and crash at the Top of my Prusa
An the Homescreen i see -94.5mm ?? 
When i look in the LCD Menu ( Manuell drive x y or Z is see for X -214.5 an y teh same and Z -94.5mm
My Prusa stay at HOME ! Normaly must i see 0.0mm for all x y and Z
On the LCD i see - 215.5 ?!

Whis this Version can not Print !

Thanks !!!! for you Work !!

Hardwarekiller
fixed throughout
I know this test fails, but I wanted to see the testerfy output.
So there's no way to build from scratch then? Keeping both the julia and flisp parsers in sync seems unlikely to work well. I guess this is the classic bootstrapping problem. Maybe we need to keep a separate repository just to check a series of binaries into so that we can always recover without needing to keep the flisp parser in sync.
Damit kann man die Tests auch einzeln kompilieren. Sollte also jeweils - für jede ...Tests.cpp ein toller Name sein. :)
this might solve that too high cpu usage after like 20 hours uptime, or it has nothing to do with it ?
> who these days use .lookup spell anyway ;p

I do.
This is not good, controls now feel pretty broken :S

We lost "autonomous" behaviors of `TrackballControls` - before it was possible to have "momentum" type of response where user initiated behavior continued for a while with damping till motion didn't stop completely. This makes for nicer "feel" of controls.

That's why controls needs to be "powered" from the outside and not just respond to events.
Using the system libraries is intentional so we won't include this patch.
After I check this out, and try to run ./build.bash, I get:


## Step 00: initialize
At revision 13038.
sums-smack: FAILED
sha256sum: WARNING: 1 of 1 computed checksum did NOT match

and the script exits.
Здесь обязан быть заголовок с копирайтом. Скопируй из любого файла
Trevor, When you get a chance can you comment on this? I'd like to know whether to follow this procedure as I update files.
Ich würde zur Sicherheit noch folgende Zeile einfügen:

    Assert.False(false);

Related: http://www.youtube.com/watch?v=OXRi28W-ENY
Преренеси тест поближе к attribute
Is this supposed to be if the user is in group Admin OR Owner OR Client ... or if user is in group Admin AND Ownder and Client?
box.ssh() could serve both purposes (interactive = box.ssh(), non-interactive = box.ssh(command))
As I see it, interactive should call ssh via fork/exec and non-interactive should use ssh_execute()
How about only provide users the mechanism (scripts in tools directory, they can also be called as a lib)?
User can use tools directly or use the lib in scripts ( such as control file), it's clearer to understand.
can't we just say `amo.ADDON_SEARCH_TYPES` and forget the list comprehension?
おお、そうですねexpireもサポートしないと。。。
うっかりeleveldbを使うのも面白そう！
Would be happy to merge a pull request if you replace use of fromdos with tr -d (much more standard) and rebase it on the current master.
I like the grid-24.less idea.
This line causes
ErrorException
  Notice: Undefined property: Behat\BehatBundle\Runner::$container in vendor/bundles/Behat/BehatBundle/Runner.php line 72
space surrounds ==
This is not my translation.

But thanks anyway.
this reasoning is not the answer =)
This function does not do enough.
synchronization is necessary if the data is different
вот это, конечно, тоже не по стайлгайду
really cool addition!
The creation of new names (with Nu()) didn't work well. This commit corrects this bug.
Nice idea, also needs a way to put entries in particular positions
we should stuff this into utils, we've got this 3 places now
This causes problems when you throw child links in the mix tho. Every link that contains the current url would get the current class.
`Webapp` is defined in `webapps.models`. don't import it from here.
Added {% block title %} to New NN page
they shouldn't be added to the .gitignore the images are both an asset to our docs as well as part of our library.
@bbenezech 
Is there any reason for removing simplecov here?
I prefer using coverage analysis on writing tests...
there it is... mit license... everyone can contribute ;-)
Rather, I know it breaks the events. :P
I'm not sure what change did this: When you log in, one creature killed will be lootable, after that none are lootable. 
 
So, log in, kill one, loot, kill another and any after that are not lootable.
http://paste2.org/p/1885495 and last one these three crashes are repeating :-) this and first seems to be the same almost
export this so it will be configurable

```js
var suffix = exports.suffix = 'json';
```
//cc @sajjad-ser

You might want to check this one out.  I've move a log of the text into the lang files and there are English phrases in the Persian file at the moment.

Thanks
It's output to the user as 'Memory' currently, so not sure what you mean here.
Yes, it should be .rkt I hand-typed the extension and probably
committed a finger timing typo.

On Sun, Jan 8, 2012 at 17:30, calcnerd256
<reply@reply.github.com>
wrote:
> Why is it sorwimael.rtk? Shouldn't the file extension be .rkt?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jcbjhnsn/Sorwimael/commit/e8f464e42a1cdab978d49643950402bd3feceb37#commitcomment-848201
All that stuff is in my resque fork waiting for @defunkt to pull it in :)
He fixed it a different way. It's still fixed.
I think the change to the new property access syntax changed the semantics of invocation against nil, because this throws now if headers is nil. So I added the when.
In cases where the return value is used, it isn't a direct replacement. I would have to use this, which isn't very pretty:

```
(i -= 1) + 1
```

i -= 1 is the equivalent of --i.
well undergarun I tested both..and the only thing that ytdb, from my point, is better .. is the gossips scripts...
in rest? nothing...

I repeat only my point after hundreds and thousand of hours of tests... 
Also, icecrown is working better with udb than ytdb... that's a fact for me...
I can take a look into plone.app.linkintegrity, but this won't completely resolve the problem. I just found that PloneFormGen does similar get_module_info patch with (*args, **kwargs) arguments. So it'll cause the same issue.

For me it looks like patching code is doing it right using (*args, **kwargs)-like parameters, trying to stay valid for possible future updates to original get_module_info function. And the main problem is that python's default mutable arguments are not cleared between function calls.

I think that because plone.testing is trying to keep arguments clean, it should try to do it a bit better. Could it be possible to have it remember all default mutable values inside setUpApp and then re-store it in tearDownApp? Or would it be too complicated and excessive?
Actually, this is failing in a project that simply calls truncate() and unless core.rb requires it it doesn't work.. reverting..
thanks. someone just reported that he or she has problems with this plugin.
TBH, I think this should be considered a bug in libxml2.  They should probably unlink the node before freeing the list, or at least saving the pointers (like I'm doing in this patch).
Hmmmh, even if I authored this commit, I don't think we should merge it completely into the main branch...

On a stock Xcode4 installation, it prevents building. This could create headaches for potential contributors...

I prefer keeping the universal build (which requires keeping a Xcode3 installation), on a separate branch.
Должны понять, ведь наши конечные пользователи веб дизайнеры и разработчики сайтов.
Isn't `ng:autobind` superseded by `ng:app`?
I've only heard of [carton](https://metacpan.org/module/carton) which says:

> This software is under the heavy development and considered ALPHA quality till the version hits v1.0.0. Things might be broken, not all features have been implemented, and APIs will be likely to change. YOU HAVE BEEN WARNED.
We have discussed it with lein author and a lot of other people in #clojure, I have seen multiple projects solving it by simply avoiding implicit Clojure 1.2.1 jar dependencies with `:exclude`. Anyway, this is what travis-ci.org is for, revealing issues all around the toolchain :)
ill fix it today, thanks :)
значит будем делать xhtml. в config.xml можно задать mine тип только одной странице почему-то.
It kinda sucks having to have two combineFn methods( here and in popcorn.plugin ). Is there anyway we can make combineFn a utility method or something on popcorn, so that it only has to be written once?
This will do for the first pass, but for clusters running against corosync 2.0 this wont contain anything meaningful.
You'd have to do something like tools/ccm_epoche.c to get the true value.

Either that, or we punt on this and QE uses the above tool directly.
Thanks for your comments. Will keep that in mind.

Would you like me to patch the code?
Als ge copy_confi hier zet gebeurt da na update_feeds, wat niet de bedoeling kanziejn.
Ok is this CSS required on all pages ? if yes make a new css store it in static/exam/css and link it using the syntax above.
tak naprawdę to chyba i tak rozwiązanie tymczasowe...
A przynajmniej taką mam nadzieję :P
Because those binaries would otherwise be unavailable to the system and then are cleared at the end of the brew installation.  I recall needing `c-client` to properly install the PHP `imap` extension. The others are also useful for other applications, so it made sense to move them into the prefix.
Thanks for taking some interest in this plugin and giving it some love!
Really?  `siblingify` took 6 arguments, and it was pretty hard to describe what it actually did.  That edge case is unfortunate, but it would probably be avoidable if we made set{Next,Prev} "private", as in my original comment.
Its done through an overloaded GridField->getAttributes() method, so self-contained in the field
Can you remove the g_debug() statements please?
passiert ^^
Why is 'Created' not an i18n string, but the other attributes are?
switched to 5.0.**4**
I think in addition to

    binary => 1

it would make sense to allow forcing the actual argument to binmode,
by allowing any string:

    binary => ':utf8'
    binary => ':encoding(UTF-8)'

    binary => ':bytes'

and 

    binary => ':raw'

(which is identical to binary => 1)

Could you add that feature as well?
anything but the client.
isCacheProvided, jndiService, jtaPlatform don't seem to be used.
Ben, this is poor practice, you should never use the variable to do the equality check. You should always use the literal. I realise that it's safe due to the check for null, however if the logic changed and the null check vanished, then it would be wrong. The comment gives the correct solution, and I will implement that shortly while you're away on holiday :-)
Changed the URL since the repository has been moved from BugKick user to BugKick-Dev organization.
Why not use Session.Delete()?
Steve, please change `var clone = new ko.bindingContext(this.$data, this);` to `var clone = new ko.bindingContext(this['$data'], this);` I should have caught that earlier.
Im very excited with this!! Congrats trinitycore dev team you make this experience of learning even better
better made this with DISTANCE_EVENT and more better with phases
@igorspasic 
Merged into https://github.com/jhinkey/liferay-docs/tree/devguide-api for now.

I'll be pushing other changes to the API chapter into the devguide-api branch before I send a pull request to Rich. I'll include you in the pull request and close out this PR then. thanks.
could be replaced by addedinversion :D
Doesn't look like a name sequence.
Yep definitely an OS specific issue. The key names show properly on my windows 7 machine.
read what i posted.
Thanks for looking into it. I think maybe my index was screwed up because searching manually seems to be working well now. In fact, I think the results are even more accurate than before.

The following test is failing but I think its the same problem, just in my test environment. 

```1) Failure:
test: The Asset Toolbar should allow users to search for assets by keyword. (AmpleAssets::AdminTest) [/Users/tcmacdonald/Sites/ample_assets/test/integration/ample_assets/admin_test.rb:39]:
Failed assertion, no message given.
```
hcdi corectly gets up after patchram but i can't get any bluez tool to detect something (there's even some byte traffic but nothing)

did you used something kde, like bluedevil ? what peripherals worked?

Thanks a lot Mambasoft
Ahh, I was trying to avoid querying the git commits table as I imagined it would become quite large, doh.
Oh, so `nil` and `false` are valid values, if you don't want a tag. I get it now. Thanks for replying.
Right. But do something like this:
~~~ php
<?php

$content = "<html>
    <head>
        <title>Test</title>
    </head>
    <body>
        <h1>Title</h1>
    </body>
</html>";
?>

<div>
    <div>
        <?php echo Markdown($content) ?>
    </div>
</div>
~~~

Your codeblock (which appears inside those two divs) will now have two extra tabs in front of each newline (because I'm echoing that Markdown'ed content from two tabs in). To fix this, we replace every newline with a `<br >` tag and that keeps our codeblock on a single line in the HTML source, and no stray or extra spaces can appear.
Well, I don't like this too, because it will be better to have more descriptive descriptions :

It is described in TODO
Well, sometimes it's a string like `"    \n"` (which means "empty" to Braintree, I guess), so I'm testing that it's a real Hash and not Braintree's sad excuse for `{}`. Could definitely be wrapped in a method like `is_empty_hash_from_braintree?`.
Lou,

We decided to use the process instead of thread because we did not want to slow down the main pylons application.  Python has that Global Interpreter Lock issue, that is all threads are run within the parent process, so having the change monitoring on the same pylons process may case unwanted slowdown  to main applicaiton.  

What exactly is the pickle problem on windows, there maybe another solution.
Sorry. My first sentence seemed accusatory. I meant it to be a preface. Haha. :)

I agree there is a lot of merit in mirroring Django core. That's my inclination in general too. That said, I can't imagine a use case where having SendGridEmailMessage extend EmailMultiAlternatives would every cause any troubles, confusion, or lack of desired functionality for anyone. So it just seemed like the simple, clean way to go from my perspective.

If you're settled on it for now, that's fine. In the grand scheme of things, this is really minor.

Thanks for the discussion. :)
Why depending on rails? I think depending on railties is what you are looking for?
server nor response now, could not test. anyway, Anyway, I guess very little changes requires here.
+1 Hay que avisarle a Seba.
Hi @ralfstx, I've noticed that you have created the jsHint plugin and find it very useful. I'm working on a project (http://code.google.com/p/wro4j/) which provides a dozen of processors like jsHint (ex: jsLint, cssLint, packerJs, uglifyJs, cssMin, etc) and I think it might be useful to create an eclipse plugin which would host all those processors. Would you be interested?
Revert these changes in MPQ.cpp you fucked extractor
Agreed.  I noted that in the comments.  I only tried to slap up some basic
code quickly.  The view class will be passed the information from another
class.

--J


On 2/24/12 7:49 PM, "Federico Nusymowicz"
<reply+c-1009970-3d058a8af00d93c6fefaebfdecceb6e5b3bc1c74-1380837@reply.gith
ub.com> wrote:

> Doesn't seem like we should have object positions and colors in the view
> class. Those are the objects' properties... same goes for collision logic.
> 
> The view should just be about drawing stuff and handling events.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/fedenusy/Algorithm-Visualization/commit/deccb61e9d16420607c
> efd321655c6ab0b629212#commitcomment-1009970
@Sbte thanks for this. But could you put that credit file back in, i didn't make the icons and i think we should give the guy who made them credit (yes i have permission to use them from him)
Well, fixes it without Google Apps installed... still glitched out with Google Apps
Ahhh btw, I am not saying we should remove them in the Java code. There they are fine
On Tue, Feb 28, 2012 at 22:36, Kai Hendry
<reply@reply.github.com>
wrote:
> When using a kiosk with the "webconverger" default chrome with the URL
> bar, having Ctrl+L shortcut is very handy. Why disable it if there is
> a URL bar?
>
> I don't quite understand.

Because IMO user should have key bindings for dialogs and like;
specially if we call it and hide the back, the C-l combination can be
used to go out of the page.

-- 
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br
I'm so proud of you! Though you might want to have a more descriptive commit message, if you do that a dozen times it'll be hard to know when they all say "Update index.html"
Hi, I'm porting spyder to gingerbread now(in porting bootmenu progress), and will push my code soon, could U make a organisetion and may I use some of your code?
Lol! I Love the commit title :)
Yup, I will extract as much code to be generic and add possibility to make it "smarter" on what target gridfields to use.

What do you feel about in injecting entwine/javasript attributes via PHP? Is this something we moved away from in 3.0? 

    ```php
    // Pseudo code in filterform()
    Requirements::javascriptTemplate('gridfieldinjectedfilter.js', array('target'=>'#Form_EditForm_File');
    ```

    ```php
    // Or Requirements::customScript()
    Requirements::customScript('$("ss").entwine(".cms-filter-form").setTarget("#Form_EditForm_File");');
    ```
Which line
On Jan 22, 2012 9:20 PM, "turl" <
reply@reply.github.com>
wrote:

> Why was this enabled?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/photon-dev-team/android_device_motorola_olympus/commit/57193b525a5473c7eeabaec87ca5f55ff8e53a3e#commitcomment-890993
>
i though about this some more and i think my initial code is right. 
 the unwritten standard in the at the fog rackspace code is: 
 1. parameters on methods that must be specified have are specified as separate parameter.
 2. optional parameters are in and options hash parameter and specified as symbols.
unfortunately the hp_fog is written to a different standard.... -)
var selopt is unused
it would be good to add tests for missing key for the new value specs. I added some code that tries to infer expected_type from the specs so its good to exercise that.
on a debian/py2.7 system, using `sysconfig.get_path` in a virtualenv will give you `dist-packages` paths

   >>> import sysconfig
   >>> sysconfig.get_path('platlib')
   '<venv>/lib/python2.7/dist-packages'
   >>> sysconfig.get_path('purelib')
   '<venv>/lib/python2.7/dist-packages'

but virtualenvs use `site-packages`.  what does give a consistent path is this

   >> distutils.sysconfig.get_python_lib()
   <venv>/lib/python2.7/site-packages'

that's because it's patched in virtualenv to do so.
https://github.com/pypa/virtualenv/blob/develop/virtualenv_embedded/distutils-init.py#L88

I don't think you want to make your tests thinking the right location is `dist-packages`
We don't have to explicitly require `model-list`. I left it out of ModelSync.REST's requirements too.
They're both temporary for sure. Whatever you want to do.
Should be able to do without these. For instance, declare the map with the generic types in place?
All of wence accessor were written that way, I assumed that it had to do with property decorator being runtime whereas cython code is compile time ?
Have to able to instantiate without values
alright, added some boundary checks
Does this do what you think it does?
Ok! I will change it.
this looks like it might be useful when dealing with “discovering” libraries with native components, could this go into ReflectiveAccess?
A "normal" user will have no clue what the docs actually means, how does the half-life relate to the specified duration?
Please avoid trailing whitespace if possible.
@tito Yes yes, and actually that's why I proposed that change.
Consider now reify vs property (with property I'm referring to the current implementation):
1st call: They do almost the same thing (except that property does one more attribute lookup (`_dpi`).
2nd \*call\*: Actually this is not a call for reify version, since it has replaced itself with an instance attribute after the first call. So for reify this is sheer an attribute lookup. On the other hand, in the property version `dpi` gets called and it checks whether `_dpi` is None. It is clear that reify version is faster.
3rd, 4th &c. like above: it is an attribute lookup vs function call (actually two function calls `property.__get__` and `dpi()`) + attribute lookup.

I fail to see where reify can be slower and impact performance. Now the question is: how many calls to `dpi` an application will do? We cannot know, so that's why I think reify version is better. Finally, I'd like to point out that reify is not like memoize, since the latter always imply function calls.

Please bear in mind that this is not so aggressive as it may seem at a first glance. I'm just trying to find out what is the best possible solution to the problem. ;)
Python gives priority to subclasses in this order so in this case ModelArray is first and TensorFit is second. The getitem method I wrote for TensorFit breaks Tensor so I needed to make sure that Tensor inherits getitem from ModelArray and not TensorFit.
Ah, nice, didn't see that!
again translation issue use .format()
key is the id of the data (dataset_id (e.g. for dataset serach) or a hash if there is no key given (e.g. for SPARQL extension)).
It's much more readable to something like
' {key} {value}'.format(key=..., value=...)
than
'{0} {1}'.format(.., ...)
The docstring says "Notification that a consumer has been deleted.". Might want to rephrase to indicate if it is handling a notification that a consumer was deleted or firing off a notification. It's unclear from the method name if it's a handle method or a fire method.
Ah ah! I think there is a miss-understanding of what the column set width does for the fixed header case. The only width getting set here is for the width of the header column. Since this is no longer part of the row table the change is not reflected in those columns.
There MUST be a more elegant way to do this with string formatters..
What is this `mrjob.run` you speak of?
It's used internally, but theoretically by things not necessarily in org.apache.pig.data. I guess for now it's only used by BinInterSedes so it could be protected.
Does this path work when in a sub-directory of the project? Currently commands will walk up the directory hierarchy to find the project directory containing package.json.
Я похоже не очень понимаю как работают услуги, оператор, заказавший услугу - это action.assignedTo ?

Если так, то у нас есть случаи, когда одна услугу закрывается, удаляется из списка услуг кейса и вместно нее делается новая, в которой этого assignedTo уже нет? Правда в коде я удаления не вижу, но как-то оно ведь работает.
Typo: `confiugre` is `configure`.
Missing `user_id` scope?
getting rid of libxml is awseome! thanks
Controversial with @fjakobs I'm sure
so netty does not support talking to multiple UDP peers from the same socket, right?
By the way, this API will not work for Java, as objects cannot be companions to traits, leading to Java users having to import Transport$
that makes sense. would you be willing to add that commit, it's probably quicker than having a conversation on how exactly you want the sanitization be done.
I'll test it again. I think it's a PHPUnit issue from memory. Possibly how the annotation are read.
Since we're deallocating, this removal isn't really necessary.
Oops! :panda_face:
My brain was asleep... probably :\ Fixed.
on purpose - makes it easier to understand (each record in a  shape file has a record header and a type identifier).
`self.is_pending ||= true` est mieux. Tu prendras ça dans le merge.
Included due to rebase from BRAINSia/master.  Will remove since it is off topic to this pull request.
The out is timezone dependent, you could add a timezone section, eg:

````
--INI--
date.timezone=UTC
--FILE--
<?php
....
````
я придумал, как. вставьте вот это перед return:

	EdgeElement *edge = dynamic_cast<EdgeElement *>(getElemAt(newPos));
	if (edge) {
		edge->connectToPort();
	}
вернул как ыбло
If you make the above changes, then change this to `raise ParseError, e.message, e` (I think)
same as above: do in docrails and remove here.
Is this consistent with the rest of the codebase? I'm 50/50 on this kind of spacing. Just wondering.
I know it seems inappropriate, but it is reassembling the same behavior of others query methods for normal ActiveRecords Objects. I would be very weird for the same object to have different query methods behaviors... This should be independent of the persistence of the attribute itself (store or proper attribute-field)
Browsers show data from `value` attribute :)
Initial idea was to provide a way to overwrite the the connection for a request.

    Post.with_connection(@remote_db).all
You're right, it's foggy. What I was going after was that the different time intervals that `distance_of_time_in_words` handles (everything 2 minutes and after) -- there's an extra ~30 seconds around the interval which is actually included in the conversion. For instance, 90 minutes up to 24 hours is actually 89m30s up to 23h59m29s. The case statement doesn't include this information because `distance_in_minutes` is rounded at the beginning of the method. That's something we don't have to mention in the method itself because it's only valuable from the perspective of tests, but if you are reading these tests and you don't catch that the minutes are being rounded then you might wonder why a 30-second boundary is being tested here.

I can revise this comment to reflect this thinking. I also noticed the docs for distance_of_time_in_words needs to be revised as well, I'll do that shortly.
ModuleClassSymbol is not in the api for some reason... I've been looking at matching against it anyway, but it will probably be better to just check the flags
The test doesn't test anything unless "-deprecation" is given. (I tried compiling the test case without it and it produces the checkfile, and doesn't even say "there were deprecation warnings.")
Previously `typed(fun0)` wasn't called if `isErroneous` happened to be `true`. Isn't the current approach going to sometimes cause spurious errors?
On Tue, Dec 11, 2012 at 10:42 PM, Vlad Ureche <notifications@github.com>wrote:

> In test/files/presentation/doc.check:
>
> > @@ -0,0 +1,42 @@
> > +<html><body>
>
> Suppose Joe comes tomorrow with an idea on how to improve the Scaladoc
> layout. And makes a change to the common html generation code. Does it make
> sense that presentation compiler tests start failing?
>
I think I'm at the point where I agree the API should provide Comment
instance rather than html, and let IDE layout.
I'll refactor the code and the test.

>  —
> Reply to this email directly or view it on GitHub<https://github.com/scala/scala/pull/1722/files#r2382532>.
>
>
And throughout.

By the way, what text editor do you use (sorry if I've asked you this before.  I forget who uses what)?  If it's a decent one, if should be possible to spell check strings and comments automatically.
I just found `sympy.mpmath.arange` fucntion. I'll use it in next commit. Thanks for your  remarks.
I would remove the trailing e.g. before the colon since that is what the colon is implying.
Wow, we can even match on matrix size!
Wing IDE 101
Yes I think I mentioned it at some time, but I'm not actually using `edit-eclipse`. The default theme has a light gray background, and the background of `edit-eclipse` is white.
We're using phpdoc for comments, not one-liners.
Afaik the entries listed in the changelog are ordered by their issueid and not the order they were introduced. So this would be the wrong position in the changelog.

[Workflow](https://github.com/yiisoft/yii/wiki/Git-workflow-for-Yii-contributors) -> 7
Is this still relevant; what do we need to do to resolve this?
ok, kept options open

-------- Original Message --------
Subject: Re: [karaf] demos poms; karaf plugin (#6)
From: Andreas Pieber <notifications@github.com>
To: apache/karaf <karaf@noreply.github.com>
Cc: Andrei Pozolotin <Andrei.Pozolotin@gmail.com>
Date: Fri 17 Aug 2012 08:36:00 AM CDT
>
> In
> tooling/karaf-maven-plugin/src/main/java/org/apache/karaf/tooling/features/GenerateDescriptorMojo.java:
>
> > @@ -317,6 +332,9 @@ private void writeFeatures(PrintStream out) throws ArtifactResolutionException,
> >          if (resolver != null) {
> >              feature.setResolver(resolver);
> >          }
> > +        if (installMode != null) {
>
> considering KARAF-1735 the "old" solution was pretty fine tbh; but the
> new one is fine for me too; so, as u like
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/karaf/pull/6/files#r1402682>.
>
name==link ---> name == link
@rkyrychuk - thanks for the clarification and updated pull request.  It has been quite some time since I dealt with the binmode stuff so I don't remember the specifics, just that if you can call it you ought to as it sometimes causes issues otherwise.  I think you may be correct that this matters at read time instead of at size calculation time though.  Thanks for the help!
Can you please add a blank line above this?
drop or rephrase this line.
ooor you keep it as a list and only join it below when you're done adding pieces to it!
There are currently 44 comments with parents. We won't lose them, they just won't appear nested under their parents.
FWIW, we may need to revisit this code over time... I'm not sure what MindTouch slugification looks like, and we'd like to maintain some compatibility to not break links.

That said, I think this stuff might only get called for new pages, so this concern might not be all that troublesome.
Should be "Changed"
Otherwise you can't set the value to nil
My opinion is as they are named ("standard no other") they should not exist.

If you can assign a meaningful name to these roles ("private forum" transformed to a user role name?) then hopefully their name would be short enough to not require editing other lines. If you cannot come up with a succinct name for these roles I would say take them out.
It is not a new line. In an old file there was a tab on a blank line. My editor removed it after saving the file.
Why not use `max()` here? It'd be more readable (it took me a while to work out what it was doing).
This block looks like a copy paste of sqlite2 block immediately preceding. Instead the sqlite3 condition should be folded into the condition on sqlite2 block.
just put it into the previous global line.
there is no need for a new one
`$config` isn't set at this point.
It's an object as that is what jQuery.each uses as argument. We could call it `classesObject` but that doesn't really tell us what it's doing IMHO.
I will remove those defines.
> What's the reason for round-tripping this through the JSON parser? I think I recall seeing an issue about this but I don't recall the reasoning for it.

When you process a job through a real qless worker, the data gets round-tripped through JSON.  Thus, you might enqueue the job with args like `{ id: "foo" }`, (using a symbol as a key) but when the job gets processed by the worker, the data will be `{ "id" => "foo" }`.  Thus, it's important that your `#perform` method use string keys even if you enqueue the job with symbol keys.

Now consider the case where you are using `Job.build` to unit-test your job.  Prior to this, you could get a false positive, where you used symbol keys with `Job.build`, and ran the test, and got it to pass (where the test was using symbol keys, too)...but the real job will fail in production because the keys will be strings, not symbols.

Likewise, there's a similar problem with date/time objects...JSON doesn't natively support these.  Ruby provides the means to serialize them as strings, but they aren't automatically deserialized.  When qless round trips them through JSON, they will be strings.

So...round-tripping the data through JSON here helps you write tests that more accurately match how the job will run in production, and avoid false positive tests.

> Which JSON parser is being used? Since we are making two additional calls to the JSON parser for every job we should make sure we're using the fastest parser we can.

This is just using JSON from the ruby 1.9 standard library.  This is what all of Qless uses.  There are faster JSON parsers, but I'm not sure it's worth pulling in additional dependencies, given that the JSON parsing time will typically be dwarfed by the work of the actual job (such as network calls, or whatever).

It would be a nice feature of Qless to let you use a faster JSON parser, though...might be worth opening an issue for that.
It's probable I've massively misunderstood you, but I'm having difficulty interpreting your comment here.

My understanding here is that only including only one of these `should_receive`s in a context causes a failure because RSpec doesn't restore the original `Rouge::Compiler.compile`. 

If I uncomment the first `should_receive`, I see one call to `compile` hit the mock (builtins.rb:147 in `_compile_fn`), and then the second call to same (same line) yields a `NoMethodError` because RSpec has removed the original function.

If I uncomment the second, it fails immediately because the first call made doesn't have the expected arguments.

Is there a good way to test this without multiple assertions in the same run through?
While I prefer the one-line version, it does imply that `callback` returns something.

I'll adopt your change.
Reading the cli.json file should go into management/utils.js.  We'll want to add support for cli.json in the other commands in the future.
This is pretty risky because `getMojitTypeDetails` is a public method, and people will use it in many different ways, they might end up corrupting the cached data.
Just a detail. Such commented block of code could be removed.
It would be better if you were to make this an anonymous function, i.e. "Are you sure...", function() {
---the function goes inside here
});

I think it would be easier to read.
Can't have patientlist here.
I'd favor that! It's the only occurence of $settings, which means remove, as additional benefit.
return this.getFormatName(name, 'province');
Does this case depend not just on xdim and ydim having certain values but also on the axes having some specific value? Like [[xdim-1],[0]]?
If so, where do you check the value of axes?
Don't make a redirect on error, because the form gets cleaned and cannot be corrected.
Awesome, that is much better. I will fix that and rebase those commits and we should be looking good.
Can you provide a demonstration of why this change is necessary on [jsFiddle](http://jsfiddle.net) or similar?
`(!offsetParent)` => `( !offsetParent )`
For fixing a typo like this, how should it be handled? Should I make one more commit and push it here or?
The function seems to be used only once. But it might still be good to have when linear regression is used as a sub-state?
Two spaces in front of the?
Do you really need ``pathname2url``, or is all you really want a ``quote``?
This comment is a tiny bit confusing, because I read `Open` as a verb. I'd say use `(open, end]` but I doubt anyone but me and Will would get it.
`mecab_supoprot?` have to be a class method of the Hyperestraier class because it will be called in the class context later.
`depends_on 'cmake' => :build`, and by convention goes first in the list
rename this to keys or sshkeys
I'm completely fine with the whole spec being implemented. I'm just not that keen to try put menus into the vumi message format -- it's really the word "similar" that scares me.
Could this warning also move down into the __init__ like the session one did?
> Предлагаю такой вариант

Одобряю :) Но может всё же стоит упомянуть о "nicer usability". Например, так:
"мы настоятельно рекомендуем обновиться, чтобы получать такие подсказки и сделать свою работу комфортней / удобней"
I don't see this being used, is it still needed?
I've added stripping of the fields, by default with spaces, and if 'delimiter' is specified, it is used as the strip character. I'll document the meaning carefully (this is an extremely rare use case though).
I see we have a mixed use of urllib and urllib2. I see that urllib2 has the quote function that we need and that no where in the script we use urllib stuff. I suggest to remove it an include only urllib2.
the gram-schmidt we already have cannot be used here?
For what it's worth, the convention of using caps vs lowercase is to denote functions with constructors. `C` refers to a Component that can get new'd up. Since `R` can't be new'd up I think it would be `r`. Also, fwiw, I've never been crazy about the convention of using single character variable names in specs. Makes it difficult to grep around the file and see what the variable refers to.
Good idea. I'll add them

-----Original Message-----
From: Peter Flynn [mailto:reply@reply.github.com] 
Sent: Thursday, May 24, 2012 4:37 PM
To: Ty Voliter
Subject: Re: [brackets] Menu and Command and Shortcut Extensibility implementation (#936)

>          }
>  
> -        // create click handlers and populate cmdToIdMap
> -        var menuID;
> -        var commandStr;
> -        for (menuID in menuMap) {
> -            if (menuMap.hasOwnProperty(menuID)) {
> -                commandStr = menuMap[menuID];
> -                $("#" + menuID).click(createExecFunc(commandStr));
> -                cmdToIdMap[commandStr] = menuID;
> +        var name;
> +        if (typeof (command) === "string") {
> +            if (command === "---") {
> +                name = "---";
> +            } else {
> +                command = CommandManager.get(command);

General comment: these errors about ids being missing from the map or ids colliding should probably include the id in the exception string.  Makes reading error reports / stack traces easier.  (This also applies to some of the error cases in KeyBindingManager).

---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/936/files#r878979
Fixed and pushed

-----Original Message-----
From: Jason San Jose [mailto:reply@reply.github.com] 
Sent: Friday, May 11, 2012 12:42 PM
To: Ty Voliter
Subject: Re: [brackets] File Open Performance Tests (#841)

> @@ -64,6 +64,7 @@ define(function (require, exports, module) {
>      var EditorManager    = require("editor/EditorManager"),
>          Commands         = require("command/Commands"),
>          CommandManager   = require("command/CommandManager"),
> +        PerfUtils           = require("utils/PerfUtils"),

Indent of assignment

---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/841/files#r810596
This is to test that we show value hints based on the filter before the cursor and my test is setting cursor after the l in the invalid attribute value. I have an incorrect comment on line 518.

So if you have rtl, then it won't show you "ltr" if the cursor is somewhere inside the rtl, but will work if it is to the left of rtl.
Can you also remove this @param as it no longer exists?
Good idea. Will fix. ``SpecRunner`` has a redundant check which should be ok still.
I tried testing these shortcuts and they don't seem to work.
I put an encode function in this file for now. I sent a team e-mail to figure out which encode/decode code we want to use and where the shared code should live.
bad merge (likely my fault).  bfsize needs to be 16.
Shouldn't this be "mu4e"?  It seems there is no Emacs Lisp files in the top directory.
I wonder if we should have a specific email address for maintainers
Looks like a DriverException is being hidden behing a NullPointerException.
Unrelated to your fixes, but Shouldn't this be <pre>Mozilla Popcorn</pre>?
Don't need this `if` block.
Why not a dataurl like the other icons we use?
lint doesn't catch this comma? Odd.
So glad this is leaving.
Yes, it is OK
I did not know that. A trick I must remember.
Do we want to maybe just store the buyer on there, or all the buyer's attributes like confirmed and such too?
"identifier"
If you're going to move that code into a seperate function, I would prefer to do that switch outside of the function definition, that it is evaluated only during import and not on every call.

```python
if SSLContext:
    ssl_wrap_socket(...) # Python 3.2 implementation using SSLContext
else:
    ssl_wrap_socket(...) # Fallback implementation
```
Yeah, `~> 2.0` would cover 2.x.x
Opravit zdroj, jde o popis tlačítka.
This is also a hack I pulled together because I don't know Ruby/Rails well enough to do this in the right way. In Python/Django I would know how to manipulate the request/response object, and how to do list-expressions, tuples, and loop constructs that would allow me to fix the URI's in a few lines of code (and in a few minutes of work). In Ruby/Rails due to my lack of knowledge it would probably take me 1+ hours just because I would have to bounce back-n-forth with the tutorial/reference docs to figure out what I want.
It might be better to say 'common' rather than 'no uncommon'.
I see a lot of dependencies being added that I cannot really rhyme with the added functionality, why is Format.KML necessary etc?
If you wouldn't mind wrapping the function definition in parenthesis to indicate an immediate function definition - `(function(){}())`
This spacing looks weird.
Agreed, changed.

On Fri, Sep 21, 2012 at 5:27 PM, Paul A Martel <notifications@github.com>wrote:

> In src/ee/executors/unionexecutor.cpp:
>
> > +                                       m_output_table->name().c_str());
> > +                            return false;
> > +                        }
> > +                    }
> > +                }
> > +            // FIXME: node->tables[ctr]->
> onTableRead(undo);
> > +            }
> > +            return true;
> > +        }
> > +
> > +        boost::unordered_set<TableTuple, TableTupleHasher, TableTupleEqualityChecker> m_tuples;
> > +    };
> > +
> > +    boost::shared_ptr<SetOperator> SetOperator::getSetOperator(UnionPlanNode* node) {
> > +        UnionType unionType = node->getUnionType();
> > +        if (unionType == UNION_TYPE_UNION_ALL) {
>
> This would be clearer as a switch statement.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/alexeevm/voltdb/pull/3/files#r1665941>.
>
>
The chrome code should use an intent for that. I've filled bug 736632 to use an event until we have intents. This is still dirty but that won't force you to expose those APIs just to make the chrome code happy (there is still tests though)