labofsahil/github-event-dataset-2012 huggingface dataset:
The ABC classes are unconditionally imported even though they are conditionally defined.
OK. That seems fine :-)
I haven't published it to maven central yet since I wanted to update the documentation first. I'll try to do that this evening and then publish to Maven Central. I really wanted to get this release out the door because there were indeed some blocking issues in it.
Thanks for the tip regarding the Basic setup wiki page ;)
I think I saw this code in your confoo slides? :)
I missed it, thanks.
good call, @robhudson
I am ok with that. The real solution is dropping the use of ets tables. That solves not only this problem but a bunch of others. That though is a larger project.
Thanks, I added a note and a link to the MapQuest Terms of Use
one note: the starting delay would be really long when user won't have panel's tray enabled. Yes, I think it won't be so often but it's possible...
I named it this way because HTML5 is less restrictive. It's HTML4 that is causing us problems.
I focused more to fix the errors in JSLint. Nothing really concerning, except creating function within loops, a lost 'new' before a function when it wasn't necessary, and that's it I guess. Need to work on this again sometime later, to see if everything is being created at the right time (globals, functions/variables created during events, and so it goes). Time to sleep :-) 
Cheers
From worldlistener:
public void onChunkUnload(ChunkUnloadEvent event){
		for(DropChestItem item:plugin.getChests()){
			if(item.getBlock().getChunk().equals(event.getChunk())){
				event.setCancelled(true);
				return;
			}
		}
	}
Nm, I'll take what I've got now.  I'll just merge your pull request and re-fix it :)

Thanks for your work!  I built it and the fix seemed to solve the problem I described in issue #5.
I actually did use `\\` at first but that actually resulted in a literal `\\` in the string. I don't really know lol.

As for the double quotes for the `ENV[]`, Ruby doesn't care as long as its in an interpolation thing `#{}` :)
rename this to `all_device_types` or something
else render ... где? нужно не ошибки показать если че
So as it turns out this just resolves a temporary problem.  The fact is that $rstart is causing the error.  So by removing the $rstart from the item purchase history pagination it will work.  Still doesn't solve the raid attendance pagination problem though.
loginToEndpointAsUsernameInDomainWithPasswordAndReturnApiKeyPair  throws NoSuchElementException on keypair not found
I just don't think that this is a good feature and will only be a documentation and support burden. A lot more useful is mouseenter and mouseleave which browsers don't natively support and must be faked, just like jquery does it: http://api.jquery.com/mouseenter/
Just for clarification, using empty() instead of isset() not only assures that the variable is assigned, it also checks if it have data in the same process. Avoids double checking and usually is faster. :)
Hi Dan,

Thanks for your time to review my pages. Definitely , I will perform the changes whatever you had suggested and i will push the same to github.
Again, move these to db/fixtures
It's not gnutls, but iksemel that is broken using an old method to get compile parameters for gnutls. The next version will use pkg-config to get gnutls libs compile flags, but it hasn't been released for a long time.  This just creates a script to act like what libgnutls-config does by passing it through to pkg-config.  Patching the configure script to use pkg-config seemed like much more work.
Based on my other comment in Docs controller changes, adding the code below will be a good decision IMHO.

	if ($_SERVER['LARAVEL_ENV'] == 'local')
	{
		$content = str_replace('/docs', URL::to().'docs', $content);
	}

With the above code, it will ensure that any link into the content section will also be translated.
I don't agree to this addition, it makes the Service a Singleton.
If you want to access to the current user, you should access the service using the current view model.
@jcsullins Yes, I bypassed gerrit to check it in.
However, it does require the addition of wpa_supplicant as a dependancy. Perhaps it can be an option selected with a flag at runtime?
plus ALOT of WoTLK blizz kept server side ( we all can answer why )
E qual seria esse motivo?
Да, я сейчас его вообще перестал писать, пока не устаканюсь и не пойму точно, чего хочу.
https://github.com/kriswallsmith/Buzz/blob/master/lib/Buzz/Message/Request.php#L7-11 :-/
Shouldn't this be ...`&& substr($filename, 0, 1) != '.'` (or `!==`)?
See line 150 of removed code.
`$filename` is not an array.
fix not by me, but i want to test it :P.
https://github.com/TrinityCore/TrinityCore/issues/4524
It breaks some tests. Looks like an extra ? is appended

```
Test FunctionalTest.redirectHomePage failed: expected:<'/computers[]'> but was:<'/computers[?]'>
```
This is the commit we should use on Mike to start with. Everything from here on isn't tested on Mike.
Но ведь оно работает только наполовину (не убирается список элементов из css, который слева).
Я всегда пользуюсь крестиком справа сверху чтобы закрыть хэлпер.
Rules like "#kunena li" should really be "#kunena ul > li" and "#kunena ol > li", because of some templates have really bad habit of using "#main ul > li", which basically overrides every other rule.
Would you like to provide an explanation, or do you honestly expect me to magically understand the rationale behind this patch? I tested my patch, and I clearly thought it worked. Pointing out a mistake is an exercise in futility if you don't explain what went wrong
what about making this whole stuff work even when your site is not at the root ?

```jinja
<img src="{{ app.request.basePath }}/img/logo-composer-transparent.png" />
```
You may be interested in https://github.com/antono/valum

It uses Soup internally but i going to add Server Interface and make it working via CGI, FastCGI or Soup.
The stuff here about generating passwords doesn't seem to address the concerns raised in ggozad's review.
Ya, still figuring out this git commit thing.  This commit should be included with https://github.com/joomla/joomla-platform/pull/735

Administrator tools don't use $live_site at all ($sef or not).  This makes the form action url relative.  Relative form urls will post to the host name site name unless otherwise specified.  So if the host is behind a proxy, the form request goes to the wrong address.
Birde sormak istiyorum bizim türklerde birden fazla banka hesabı olayı mevcut ya woocommerce de ise tek bir alan var çoğaltılabilirmi? yanı sıra bir eksiklikte mevcut adress formunun değiştirilememesi..
Absolutely nothing, it never did, but we can't just remove it.
misses //------------
I'm not sure what you mean.

The rake script looks out for files or directories called something.symlink. under the .dotfiles directory. If it finds the .symlink extension will create a symlink called .something in the home directory to the something.symlink entries under .dotfiles (be it a file or dir).

So in the .ssh example - I have ~/.dotfiles/ssh/ssh.symlink/config. This means that rake creates a symlink from ~/.ssh to ~/.dotfiles/ssh/ssh.symlink and the config file will show up as if it is in ~/.ssh/config. Any other subdirs of ~/.dotfiles/ssh/ssh.symlink/ will also appear under ~/.ssh/.
Just because cascading DCMs can work in some circumstances, it is still generally a bad idea and should only be done if there's absolutely no other reasonable choice.  Scroll down to "Cascading DCMs" in the link below):
http://www.xilinx.com/support/troubleshoot/clocking_debug.htm
What about changing the tag initialisation instead of this:

    $('.tags, #fields-duplicator .tags').symphonyTags();

I didn't test it, but the event is delegated so it should work.
PEX finally pushed out a release (1.19.1) that should fix this. If not, probably wouldn't hurt to contact them on IRC.
Mai bine folosești `including_validators`, să fie la fel ca restul.
@3kids in cata it doesnt happen either
I've tested on local 1-way and on a 10 node/20 slave multislave setup, seem ok
Boa, vou subir as paradas online agora =]
pues como no saquen algo para evitar eso, cualquier servidor privado estaría actuando ilegalmente :S
got some problems with my skype so im not in contact with znickq since the last 3 days, however, i asked him if i could refactor a large part due to some bugs which were in spoutMaterials which were hard to fix. however, im still working on this every day. as soon as its completed ill contact him again so he has to overviewer the whole source and give it a final ok or not. will still take some time however.
about the .zip.. hm.. yeah could do that too of course, was hoping to avoid people thinking they have to unpack them when they are called .zip
I tried looking up the upstream documentation/source to be sure about this, but couldn't find the relevant bits yet.

I'm on Fedora Core 16. Not sure if other Gnome 3 systems behave the same.
variable $branch is not initialized in the line: if (!empty($branch)) {
it would be better to initialize it to null explicitly before the line: foreach ($name as $branchName => $step) {
I bet @agramfort's brains are very noisy :)
Indentation is not proper again.
I like the idea, but I think we can do better: what if we made BROWSERID_CREATE_USER either:

#. A callable, in which case we return the result of that callable when passed the user's email. We can check if it's callable using the `callable` function.
#. True (or truthy, however you call it), in which case we create a user using the default algorithm.
#. Falsy (including not defined), in which case we return None.

Thoughts?
I'm with @main-- on this one since they're all doing the same thing.
On Jan 23, 2012 11:55 AM, "main()" <
reply@reply.github.com>
wrote:

> Why would we write three catch-statements when one would be enough?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Multiverse/Multiverse-Core/commit/c7d35f13f42a4441a54981637e57cf8b9caa6fd2#commitcomment-892844
>
Typo here, 'recipies' => 'recipes'
if no configuration for view, throw an error than return a promise
Please use multiples of $grid-x for most measures - applies to all measures in this commit.
great choice :)
W dniu 24.01.2012 15:06, umpirsky pisze:
> Why not this way:
>
> `public function sort(QueryBuilder $sortable)`
>
> and avoid checking instance and throwing exception.
>
> It is more readable IMO and this kind of error should be caught in tests.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Sylius/SyliusAssortmentBundle/commit/1aece260248bd4fd1971e6b9425a2f3890ed7a35#commitcomment-896184
Because the sorter interface is more general, it can be implemented for 
MongoDB driver or any other driver.
Only in ORM driver the sortable must be *QueryBuilder* instance.
The whole grid/sorting/filtering will be moved to separate bundle, 
*SyliusGridBundle* or smth like that.
Because i'm experimenting with the colors, but finally RGBA will be used everywhere, so that the color declarations are in one uniform style. 

Also, the styles are not final yet, so some color settings will surely change and that's why you'll still find some RGB, RGBA and HEX …

Some colors are done by setting a start color to be half transparent, so sometimes changing the color (e.g. for hover effects) will only be done by changing the transparency. Also i found out that some browsers have problems when switching from RGB to RGBA, so i try to use RGBA every time to avoid that.
Indenting…
Indentation doesn't look right and put spaces after and before parentheses for parameter list of subroutine.
Please see https://confluence.startsiden.no/display/THAI/Coding+guidelines
What are these? Are these keys you should keep private?
Hmm, I suppose it should have a test. What does 3 argument ln -s do anyway?
This commit resolves issue #12
Helpers são melhores e facilitam mais as coisas q só o Capybara!
Working too quickly again.  Thanks for the heads-up.
Why should Jenkins have 2 formats for referencing variables inside Jenkins? You know this has nothing to do with extracting environment variables from the OS? That is, ${var} also works in Jenkins running on Windows.
This doesn't work for doc comment like `@covers myFunction()` because of the trailing brackets.

Worth adding something like `$functionName = preg_replace('/\(\)$', '', $className);` and using that for test and reflection?
Use ../01-intro/index.html as the link
Hat python Exceptions oder so? Kannst du bitte irgendwo/irgendwann eine `throw new InsaneInTheMembraneException ("Insane in the brain)" nutzen?
I thought the standard process is to copy the vendor prebuilt files first, and then overwrite from the device prebuilt if it exists, otherwise use vendor prebuilt. At least that's the way it seems to be called in the device/motorola/triumph/triumph.mk file.
I would make a class out of CRS. Then this function is just a constructor.

Now it looks like `convertRowMajorSparsityToSRS()` is modifying objects outside its scope. While this is not wrong, it has a lot of potential to be incorrect or inconsistent.
Should we also describe savePath here?
The ring buffer works, but it requires iterating over every line in the file.  Tail does some fancy seek work to find the end of a long file much faster.  That could be done in python just as easily, but I'm still on the fence about whether its worth it or not.
why move this up here? or did Rex write all your code ;-)
Changed to:
"This option gives you the ability to change routes that have already been added to Camel.
  Consumer actors have a hook into the route definition process which can be used to change the route."
The wicked awesome about viktor’s proposal is that the tell operation would use the wrong sender reference, hence I think it should be removed on those grounds alone.
its applied to the channel, so any new clients pick it up automatically...

the loop below (if you look at the implementation `client_v2.Pause()`) just pushes a state update message
Do you know any of the underlying stuff about frozen? and all that?
Surprisingly, this badness is not introduced on this branch, though the comment is here.  Me thinks we should look at what this should be on develop.
Can you explain more what is the problem in this point? Sorry...
we should probably remove the month - the url seems to display the latest when that is not there

incidentally by this definition Safari and android browser should be included as well as more than one chrome version

Also the latest FF is 17 which has < 1% share so maybe this could be made clearer.

As an aside mobile.ckan should be considered at some point
Just to verify I'm understanding what I'm reading here. The DELETE is returning a task response code saying it's going to try to cancel it and creating a new task to do the cancel? I guess it's the accepted response that's confusing me. The reason this was occurring (I think) is that the client assumes an accepted response will carry the task to track the operation.

Here, it looks like we've finished the cancel and are returning the call repo for the cancelled task. In other places, we don't return the resource that was deleted from the DELETE call, so it feels like we're just returning it to appease the client-side parsing that expects an accepted to contain info on how to task that was created by the call.

It's possible I'm not fully groking the rationale here. Ping me and we can talk in voice, it'll probably be more clear if you explain it to me.
Mostly just for informational purposes, this might be easier to read and would very likely perform better (not that performance is an issue here) as a comprehension:

expected_tags.extend(TAG_DOCUMENT for x in range(12))
A bit surprised you didn't rename this to reboot_report_are_you_finally_happy_jdob
Good point. I wanted to raise for while that we are using uppercase constants while the idiomatic scala style is class name style (e.g MyConstant instead of MY_CONSTANT). In fact this field is not really a constant as each instance have their own copy... I suggest to discuss it offline.
Yes. This is now called just 'Callback' (one of the above commits removes all of the extraneous 'Addon' labels to all of the classes). All of this infrastructure is mainly to support python's inability to deal with calls from arbitrary threads - so we need to shuffle calls over to a thread currently running a script, and wait for that thread to call back into a method we have control over, before we can actually execute the callback.

To do this there's two main abstractions. The first is a 'Callback' which is a root class for a set of templates (CallbackFunction) for binding a call and it's parameters to an object (like you said). Here's an example of how it can be used:

Callback<MyClass, int>* callback = new Callback( myObj, &MyClass::doSomething, 5 );

This assumes that there's a class called MyClass with a method called 'MyClass::doSomething(int).' When you call 'execute' on the callback at some later point then 'myObj->doSomething(5)' will be called.
On my screen, they line up.
Needs to be set :null => false - will need to set a default of 0 on this line, then remove the default with a modify_column statement straight afterwards
Our new hook, can we put a test here for 'card with account'?   , :if => :card_with_account maybe?
Grammar: fortfahren => Fortfahren
what is this shouldn't `script` tag start js mode?
regular expressions are probably overkill (the string "find" method would have been fine), but this works.
The following line uses it I think:
case `remoteDaemon`
has to be > 0?
And then someone uses `|split(none)` and no exception is thrown. `is_empty($delimiter) && '0' !== $delimiter`?
This should be implied by `editable`, and vice-versa. There's no need for a separate property.
Long line. Please wrap at 80 columns.
s/would be an instanceof/is an/
I've seen the tests, but his method seems dangerous since you can call it even without unloading the DLL first.  That's a departure from the rest of this API, which is basically impossible to use incorrectly.  I see two ways to resolve this:

* If this is only for testing, then I'd rather go back to the way it was before; I don't want public methods that are only for testing.

* If this is useful beyond tests, then I'd rather have a Reload method that reloads the DLL and reinitializes libpd.  The idea is to encapsulate the reloading functionality so users can't put libpd in an inconsistent state, and to hide the implementation.
I am not sure this is needed. Perhaps we can just have a hard coded state in constants.
```js
// Tip (tooltip)
Mapper.register( function(){
  var key = 'Tip';
  Lu.map( _.filter( Mapper.$scope, function( item ){
    return ( _.indexOf( item.getAttribute( 'data-lu' ).split( ' ' ), key ) > -1 );
  } ), 'Tip', function(){
    this.settings.placement = 'above';
    this.key = key;
    this.hasDependencies = true;
  } );
} );

_.each( ['Above', 'Below', 'Left', 'Right'], function( placement ) {

  Mapper.register( function(){
    var key = 'Tip:' + placement;
    Lu.map( _.filter( Mapper.$scope, function( item ){
      return ( item.getAttribute( 'data-lu' ).indexOf( key ) > -1 );
    } ), 'Tip', function(){
      this.settings.placement = placement;
      this.key = key;
      this.hasDependencies = true;
    } );
  } );
} );
```
Something like this *should* work.

The split is included because the attribute can be space delimited.
Might be better to just change this to "<method> at <file>:<line>" to make the log messages shorter
<a href="http://tidy.sourceforge.net/">Tidy</a>-itis
Co takhle jen 

    $ex = new FatalErrorException($message, 0, $severity, $file, $line, $context);
    echo $ex;
I already corrent the mistake that you said above.
I think we should put `# :nodoc:` here since this class is not to be using in the applications
There's a double space before `&&` :)
Could you inherit from `ActiveRecord::Base`?
Perhaps we could call it something like `split_paragraphs`, or `strip_line_breaks`, wdyt?
(sorry for the delay)
Link is broken, it's in one of the outdated diffs.
So i will use `each_with_object ` then
This line probably breaks Linux compatibility. Need an If statement.
Ooops. I wanted to remove this, leaving it as a comment was not my intention.
For a test like this, stay far away from sketchy features like DelayedInit.  It saves you one line and cannot possibly reduce the number of confounding variables.
> Do you know the difference between ``bispev`` and  ``bispeu`` by the way? Docstring and code are pretty much the same.

`bispeu` evaluates a list of points [(x_1, y_1), ..., (x_n, y_n)], while
`bispev` evaluates on a regular grid defined by (x_1, ..., x_m) and
(y_1, ..., y_m).

I will change the docstrings of the `__call__` methods to be more precise.
perhaps use the list method like Cycle?
Yeah, there is. But I wanted only the that part of it which converts the function to simplest format (in string).
No, it just means that you have to conform to the Basic interface, which more or less just means that you need to have a .args with all Basic members that you can use to reconstruct the class.
Yes. The only note that it is true only  for nonzero vallues, but as we adhere to the cancellation of `(x-1)/(x-1)` in silence, then this is the same case:

```
>>> cos(x)/sin(x) * sin(x)/cos(x)
1
```
fixed this. I didn't realise this was in the momenta functions too.
This refers to Doctrine1 - this code also needs to be removed.
why are we appending extra whitepsace?
Hello Olaf, could you please try to avoid the extra multiplication?
Remeber we should always try to avoid extra operations cos most of our code code is time critical
We rather use extra memory in favor of extra cpu
As soon as you fix it let me know and i'll pull your commit
I think I'm going to change the behavior to the one described above as it makes more sense.
The _ notation is not very Scala-esque.
Why did this border have to go?
It strikes me that if a task i already running, it's possible that a TaskWrapper is already in the map and said TaskWrapper might have a callback of its very own.  If that is the case, dropping in a new TaskWrapper here is going to deadlock whatever else is waiting on the task to complete.
Why do we need to serialize the node and send it across a pipe? Seems like it would be just as well to collect the exit status of the child process using something like `exit_status = Process.wait2(pid)` and then see if it exited successfully with `exit_status.success?`
The constructor just calls set_driver and set_sql_driver, which both have the appropriate type hinting on them.
So put preview before submit? I can do that later today.
Isn't there a regex for matching valid variable names already? That one should probably be reused here?
Needs doc block for return value and parameter: `@param` and `@return`
It occurs to me that async.parallel will die if one error is passed back in the callback (ie, if one of the gets fails, the rest will be canceled).  We probably don't want this behaviour.  Instead, we should suppress errors from the request maker and do something sane with them.
This too. :-/
hrm, we shouldn't merge this bit.
Right, so just say params.details.target == nil
Changed - e10e57046f86f0c608a7365e93bb53f2c806fa49.
Yeah, I'd prefer to be explicit about what we want to keep as it was before.
In theory, in production, we will use the YCS combo rather than the app combo. But I do agree we should make this more performance, maybe creating a hash in memory for all YUI modules or just not dealing with YUI modules at all. Let me think about this.
Let's just get rid of the version numbers in this file altogether.
How would you suggest we proceed?

Would it be preferable to ensure that only contiguous NumPy data is passed to ```zlib.crc32()```. It makes sense to me and would tie the fix closer to the problem.
Are white spaces already stripped from the slot?
Changed to short form meaning
Minor: I would put this line inside the `with` block. Yes, `cm` leaks outside the block, but that isn't clean :)
The Python docs just refer to runtime C library documentation (http://docs.python.org/library/os.html#os.open), so no idea there.
Serializing a complete response (including content) is only possible with a non-streaming response. Only the common part (serializing headers) is left in the base class.
no introduction to regex capturing as there was on the old version here: around https://github.com/django/django/pull/248/files#L0L105
s/minor/less/
why do we need this file
cant we use empty.jsp
jshint wanted the named function at the top - It probably wouldn't of complained right beneath the `var` statement, but the order of those two things is pretty inconsequential
Really? I just tried on Chromium and found several globals leaking in ajax tests; mostly from jsonp. I assumed it was already on your radar, but now I'm wondering if it's environment-specific.
It didn't. :). I had checked that already.
It is not necessarily the same,  but it looks any directory name works fine.
Don't use madlib schema directly in test SQL.
Hmm. Is is really necessary to hide the test in a pgsql function, and then call that function?
Interesting, found this explanation: http://stackoverflow.com/a/10857806/1352190
ahoy, you are absolutely right.
We're calling key_with_namespace twice, once here and once right after this method returns.  Can you remove that duplication?
There is not an ncursesw formula in Homebrew.
Can just do

```rb
cd 'fon-flash' do
  system "make fon-flash"
  bin.install 'fon-flash'
end
```
they didn't provide it. 
The source code is part of a bigger repo and the fon-flash folder is 3 years old.
The tool itsel isn't versioned.
Needs to be a list on the off chance prefix has spaces in it.

```rb
system "make", "install", "prefix=#{prefix}"
```

Also, you should be able to stick `BINDIR=#{bin}` on the end there and avoid the patch.
this is a constant - make it that way (also drop simple_regex)

    IP_AND_PORT
publishing an item shouldn't remove its retract_on date?
What do you want to test here?
i agree with rmocanu
One more thing I want to ask you @marcphilipp. I created #511, and on Monday i plan to create a similar pull request extended by osgi copied from my public repo.
Would you agree with me if we deploy these versions into https://oss.sonatype.org/content/repositories/snapshots
4.11-SNAPSHOT for #332
4.11.1-SNAPSHOT for #511
4.11.2-SNAPSHOT for OSGi + Maven pull request I will create on Monday morning

This way we may ask the community to test the release one by one, and improve. 
After that we may ask the clients JetBrains IDEA, Eclipse, etc, to be prepared in advance for a new official release 4.11 in Maven central as non-snapshot.

One more thing, the hamcrest-core:1.3 is not OSGi, and it should be. Today I want to create a github pull request in JavaHamcrest for that.
lol, I am bicodar
Externalize string, "Line {0}, Column {1}"
What goes into the attribute structure? Can you pass just the attributes you want to change or do you need to pass everything? For example, if I want to enable a menu item, do I need to get all attributes, set the enabled property and then set all attributes or can I just pass `{ enabled: true }`? 

We may want to have common attributes like `enabled` and `checked` be available as separate getter/setters since they will be accessed frequently.
Seems like we should handle this listener consistently with others (reassign the bound version to this).
Hey klingt in Deutsch etwas salopp, stattdessen würde ich eher sagen Obacht, Achtung, oder

Hallo! Eine neue Version von  {APP_NAME} ist verfügbar.
Per discussion, these TODOs were removed and we will instead rely on the default language fallback for now.
You can skip _$this->closed != true_ check. The append() method will not be called at all if the appender is closed.
Seems to be the clean solution to do a redirect. User (programmer) wouldn't expect the program to go on working after setting a redirect-header (source: G.Schmidl, me)
Please ask konigsberg about doing this. How does this differ from g.updateOptions({dateWindow: null})?
resque used json coder, and this symbols converting to strings, consumer when extract params works with strings. When i coding with marshal, symbols is symbols, and consumer can't work, because its wait of strings.
supprimer ce template et tout mettre dans views/_google.html.twig
This was here for a good reason: to keep people with no report-viewing permissions from seeing the sidebar.
you actually made the style of this line wrong ( it should actually be multilined if you're trying to make it correct )
These can just be functions, you don't need to explicitly use var

```
function unorderedReadyCallback( loadItem ) {
...
```
Let's remove this whitespace.
Don't need this extra space.
Can we try to do this on one line rather than two?
34 and 35 have this issue.
indentically challenged
Not sure I like that it throws away packets. But maybe the extradata isn't valid for data before that point.
Zamin zmergujemy, trzeba by uogólnić jednak parę elementów, w tym `prefix` opisujący składową frontendu. Proponuję dodać w settings.py zmienną FONTEND_PREFIX, która ustala, co traktujemy jako element frontendu a co nie.
Yeah, it's really frustrating because the Queue class does its own locking.

On the plus side, uncontended lock acquisition and release seems very fast:

    timeit.timeit("lock.acquire(); lock.release()", setup="from threading import Lock; lock = Lock()")
    0.3425159454345703

so it's just a drain on cleanliness and simplicity rather than performance :-|
callable function in contrast to a "class" holding functions
dl_m -> dm
@sam can you document this link_task business for the translators? thx! 

Just put a comment above all the places that you use it or at the top.
Ideally we want all style in the stylesheets
Also, it looks like it'd just be easier to do: if (!iscsi_available) exit(); and avoid a whole level of indentation. It seems that temp is always overwritten, which makes the malloc() above just a leak.
I don't fully understand this one. Can you elaborate a bit?
On 26.9.2012 16:43, Lukáš Zapletal wrote:
> No, because katello is installed in disconnected mode

s/is/can be/

But it can be even configured and I still may want to use this feature. 
And I maybe changed this value.

Mirek
why you removed that +1 (and that semicolon) and {1}?
Actually, you suggested this previously and it's a valid point, but I tried it before and the trigger doesn't fire when attached to that element.  So the body element seemed to be the next logical choice.
Nice find, @parthaa
I would rather prefer if we put in KATELLO_UPGRADE_HISTORY all executed scripts. Even those marked as 'always'. You will skip them in "done?" anyway, but you will have trackable record that it was really executed and not skipped. Which may be good for support to find that upgrade was run interactively and some script was skipped.
Hmm, you may even put inside that file timestamp when the upgrade was executed. So once-scripts should have there only one line, and always-scripts should have there multple lines and you can then easily "replay" what user really executed during upgrade.
Why is this file changed?
As I wrote above - maybe a better solution would be to create an overridden dependency resolver for ASP.NET hosting?
Remove the base.GetAssemblies() and just use the BuildManager
Better rename to badHostnameMsg
Use an #id instead of firstChild/lastChild to make sure changes in the layout does not broke the feature.
label here give away its origin :P
if it's deprecated, it should not be here
I think this should be ``long`` for consistency with other places. However, I have a slight preference for changing both of these to ``lon`` since it makes ``lon``/``lat`` pairs better aligned code wise (but super minor)
Can you use `dict.clear()` and `dict.update()` methods in setup and teardown to make the code tighter?
Here's where the problem is -- the variable `key` here exists even after the loop is finished and it masks the `key` variable which is an argument to this function.  Rename this variable and I suspect all will be well.
Yes, before this version I named it TimeBasedService, but actually it's day based, every time length should be a amount of  whole days.
But I agree to change the name, when there is no real time based (I mean the real time based should be account in seconds).
This is new behavior -- I don't think you want to exit here with an error if there is no proxy
I am very sure (but I don't have time to check with the C standard right now) that ="" is the same as ={0}; and does exactly the same as memset: initialize the array to zeros. No security theater please.
Or alternatively use `$args['fieldName']`.  You could make local variables for the most commonly used indexes as well.
Should be `if (`and `) {` as per the [coding conventions](http://book.cakephp.org/2.0/en/contributing/cakephp-coding-conventions.html)
@markstory Well, yeah, I could, but this test is more about that the values are set the correct way. Doesn't actually matter if it's `-2000` or `E_USER_NOTICE` or `'cat'`
The "required" word is very confusing, since you are making this big change maybe we want to do something like rails? They use validatePresence, which accurately describes that behavior.
Since the win32 build script replaces Admin.c with W32Admin.c, this will actually never be built for win32.
The ifdef which you removed was just a broken check left over from before I implemented W32Admin.c
this file also needs indentation review.
When you're typing in the new message box and it wraps to a new line, this allows you to see the line you're typing and not the first line.

However, it's not supposed to be a part of this pull request as it has nothing to do with the https youtube issue. I think I might have incorrectly created this pull request because it keeps adding my new commits instead of being just up to cc563c8
Should probably go all the way up to 4096. And just pick ALL CAPS
```php
$start = (!isset($_GET['start']) || $_GET['start'] < 0) ? 0 : (int) $_GET['start'];
```
installed_type_id
OK, otazka je ci s tym nechceme nieco spravit ;-) Ak ma jeden z tych formatov oproti tomu druhemu nejake vyhody, nechceme ho pouzivat na oboch miestach? Ak nie, cim je sposobene, ze maju tie situacie ine poziadavky?
I think my intent with the `Choose` method was to make it so that all the cleanups would run, and then Choose would choose the first non-success status code. So:

```
return Choose(_cleanupWorkspaces.Run(), _cleanupWorkspacesLocal.Run());
```

That has the disadvantage of running them all, even if the first one fails. So maybe something like this would be better:

```
public int Run()
{
    return RunAll(_cleanupWorkspaces.Run, _cleanWorkspacesLocal.Run);
}

private int RunAll(params Func<int> cleaners)
{
    var result = GitTfsExitCodes.OK;
    foreach(var cleaner in cleaners)
    {
        result = cleaner();
        if(result != GitTfsExitCodes.OK)
            break;
    }
    return result;
}
```
The `window.close()` will trigger the prompt to save if necessary, save can still be done if the kernel is dead.
We will need a pretty convoluted logic to check before killing and preventing the already in place verification to pop up again...
I guess it a small drawback to have a 'kernel as died, restart it ?' if the user does really want to quit...

But I can look at how to do it.
It looks like @fperez wants this part in hooks.py as a different PR.  So I guess your comment will not be wasted.

BTW, stdout and stderr is printed by default because I don't catch it in subprocess.Popen.  Do you think it is better not to show them by default?  I am just following what os.system does but it looks like omitting stdout/err (at least) in non-blocking case because it can printed in unexpected way.
make sure we clear cached proxies if user wants to do so
I'm still figuring out the best practices in the jclouds codebase but in general, I'm not a big fan of such suppressions. At the very least, you could use a generic type RestAnnotationProcessor<?> ?
This naming seems kind of weird to me. A `json_dict` isn't really a thing. Also, it's not just reading, its parsing and returning. Best alternative I can think of is `dict_from_json_at(path)`
I just removed `preventDefault` altogether as you suggest.
add `aria-required="true"` here to support browsers/AT that doesn't support just required :)
So this should be two times the iteration duration means 2x500ms right?
oh, but moved this to base.py so its more reusable.
Yes, it shows the tooltip instead now. But it's true that we lose the definition of UUID - what I can do is add it to the tooltip: "The UUID (Universally unique identifier) of your..."
submodule这个有相关文档告诉使用者如何使用吗？
Did this turn out to be a bug in Alfredo in the end? (https://riptano.hipchat.com/history/room/32188/2012/09/05) 
I'm just curious.
This has to be described in the usage help and in the README
wrong indentation :)
Woops! Fixed and squashed.
You fixed it in this commit: https://github.com/bschussek/symfony/commit/d60626efd5a17fcca018e8d6edeb693de90f53ef
OK, fair point for the `files` section... I'll revert that part tonight.
一番外側の describeに1つあればいいと思います。
赞萌提示！
Besides the odd formatting here (looks like somehow spaces found their way in) lets refactor this into a new  common init method, and add the following:

    self.layerContentsRedrawPolicy = NSViewLayerContentsRedrawOnSetNeedsDisplay;
Let's set a more descriptive event type. Maybe servicecheck.state_change.up and servicecheck.state_change.down, respectively
Thanks,
I didn't want to use multi threads because of the GIL, but I guess you are right, it wouldn't have that much effect in our cases.
Isn't gradle supposed to generate these?  Not sure why it didn't, but i can add them.
i would phrase this as ". In the latter case, a warning message will be printed in the console."
spaces missing here
Yeah I wasn't sure of the best approach.

Should it use Array.prototype.slice/reverse instead of calling it on the object directly?
no spaces needed
This commented out code should probably be removed. I can remove it before merging, but since some other changes are necessary to this PR, before merging, I think it would be easier if you did it.
This reload stuff can't be done in Stage.MODEL. See https://github.com/bstansberry/jboss-as/commit/2787b20d78c82e835f0a658becd54937ef0a9900
This needs i18n. IIRC I added a commit to the one on master to do that.
Could be, but seems silly.
"Rack is tries to abstract"

"is" doesn't belong
Small l10n fix
I've moved above
I think you could use hidden property to make this item invisible(and leave some comment that we need to revert this selector in v2).
As the above for function naming.
There is 2 spaces before 'id'
What about using .children instead of childnodes? I think you don't care about text nodes right?
good idea!,  as we avoid many condition checking over the selection arrays
OK, makes sense. Unfortunately it’s inherited by children elements…
If you `setCapture()` on mousedown, you would need to `releaseCapture()` on mouseup right?
tiny nit: space after:
I made some major changes to GaiaApps it does all the same stuff and a bit more.
This section needs to start out hidden, too, because when the app is launched as an activity from other apps, we don't always start with the thumbnails.  I don't what to start displaying one section and then switch, so I start out with everything hidden.
please either remove all `console.*` calls or use a `DEBUG` variable so we don’t get too many messages in the `adb logcat`.
done, see #3958
Nice! The bug is already referenced!
Ok -- just saw this. 

Definitely thing it makes sense to lowercase, trim, and compress whitespace. 
http://stackoverflow.com/questions/2898192/how-to-remove-extra-white-spaces-using-javascript-or-jquery
Same as https://github.com/redu/help-center/pull/6/files#r2120639. Are there going to be links inside these legends? o_o
In my opinion lines 27-28 are not necessary because line 33 implies the correctness of these lines.
Same deal...
Hello Michael,

you are referencing something from your repository even without version here?

Regards
Mirko
Why does this return `None` instead of querying the EC2 pricing history?
$this->flagged = array();
Disable your 4chan X.
Should add some sort of dellimiter here to indicate the "class" has stopped. I usually just drop in something like:

////////////////// Related Utility Functions /////////////////////

See most other classes for an example.
it'd be nice if we had a test for invalid dirs too
It has only recently been added to the main astropy package, and the documentation doesn't reflect it yet, but you can replace this with a directive ``.. automodapi:: photutils.photometry`` (from the `astropy.sphinx.ext.automodapi` extension, which is already included via the global astropy docs) to get a somewhat fancier (and standardized) documentation system.
Hm, how about merging the TODO'd test only to 0.8.6, with the fix to master?
The same question as mentioned above.
@pborreli Can you be more concrete and less abstract? How can I apply your comment/opinion to the code in this PR?

- The purpose of this PR is to fix weird issue on Windows. If you are experiencing this issue, then this change could help you. If everything works for you, this PR doesn't change anything for you.

- Why an additional try before throwing an exception is "unrealiable" to you if the code fixes the issue?

- I sent this PR in order to fix the issue which hopefully make the library better for everyone. Honestly I didnt know that contributing can "look bad" to anyone (even if the code may not be as-good-as-you-would-write-it). 

I am open for suggestions and code improvements - just let me know the right direction or better solution.
As Daithi said, I did spend some time on this, but you are stuck with webdriver.
I think this should use `new WP_Thumb()` you can check for `errored()` then and not do it.
Why put this in a separate file?
Should probably extend `EventTarget`
k got it now
No it's not a race, unless by race you mean competition between resolvers to fulfill a promise. Yes it matters, promise should be resolved only once and to a value that was first passed to the `resolve` all subsequent calls should be ignored. In other words it should be about call order and not about timing (in your case it's about timing instead of order).
sure don't think so.  Only if refreshEmailInfo can change the record?
This newly created whitelist of fields does not include requiredEmail - this must be added to the whitelist or else we will regress in requiredEmail support.

Input checking was initially done in dialog/resources/state.js->"start" handler, the requiredEmail field was the first input we added.  When the code was added for TOS/PP, the implementer put the TOS/PP checks here but the original requiredEmail check was never moved.  To keep all these checks in one place, the requiredEmail check should be moved here as well.
This is awesome. Can you also remove it from resources/test.ejs?
Nit: `===` over `==`. Here it matters less but it's nice to be consistent.
Yes - I uncommented it
curly is off
Doesn't this method need this too?  Also, we should probably write a spec test that covers these.  I can take those tasks on if you guys like.
The FIXME above seems pretty severe to me. While I could not find any infos an the amount of memory required and the performance hit incurred by the record/replay facility, I suspect both to be significant.
True, I'll try to cover it with some spec.
Why aren't you using `this.options.tooltipPosition`? You should also add a default value to the defaults array.
this should be at the top and returned early…

so for example:

```js
if (!e.which === keyMap.left && !e.which === keyMap.right) return $items.eq(index).focus()
```
Just write ```array|Traversable```, as Traversable was already imported.
From [packagist.org/about](http://packagist.org/about):
> Note that most extensions don't expose version information, so unless you know for sure it does, it's safer to use "ext-apc": "*" to allow any version of it.

However, @Maks3w is right that the value under `suggest` is meant to be human-readable, and not contain version information like `require` does.

According to the [composer.json schema  about 'suggest'](https://github.com/composer/composer/blob/master/res/composer-schema.json#L104):
> This is a hash of package name (keys) and descriptions (values) that this package suggests work well with it (this will be suggested to the user during installation).

Versus what it says [about 'require'](https://github.com/composer/composer/blob/master/res/composer-schema.json#L79):
> This is a hash of package name (keys) and version constraints (values) that are required to run this package.

I was originally misled by [an example](https://github.com/kriswallsmith/Buzz/pull/31/files) I came across and the above quote from packagist.org. I'll update to remove the asterisk.
This line is not applicable
Fixed. Also in the rest of the class.
Use PHP-style comments, not shell/perl-style. (i.e., use "//" or "/* */")

That said, I'd move this to a "\@todo" notation in the docblock. I'm unsure if it really needs tochange, if the functionality works as-is.
Yep, of course, 'call' *shame*

2012/12/16 Thomas Cantonnet <notifications@github.com>

> In library/Zend/Stdlib/Hydrator/Strategy/ClosureStrategy.php:
>
> > +    protected $extractFunc = null;
> > +
> > +    /**
> > +     * Function, used in hydrate method, default:
> > +     * function($value) {
> > +     *     return $value;
> > +     * };
> > +     * @var callable
> > +     */
> > +    protected $hydrateFunc = null;
> > +
> > +    public function __construct($extractFunc = null, $hydrateFunc = null)
> > +    {
> > +        if (isset($extractFunc)) {
> > +            if (!is_callable($extractFunc)) {
> > +                throw new \Exception('$extractFunc must be collable');
>
> callable, you wrote collable everywhere ;)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/zendframework/zf2/pull/3230/files#r2429643>.
>
>
also not sure why you add the whole path again since they are in the same folder? aren't they?
Thanks. I hate that there was an exception being swallowed. Not sure I remember why it was being swallowed. It must have been a darn good reason because I don't like them at all.
This looks like it would send the update if the player did *not* change worlds.
This changes the format of the packages.json, isn't it ?
yeah, it's strange.  The original methods were named "hilite" after http://hilite.me/. I went with "code markup" as being a bit more sane name. This class needs to move out of SimpleTree at some point.
what's wrong with ``if 'fallbacks' not in language``? Also wouldn't it be smarter/faster to sanitize the setting in ``post_patch`` so we could ignore all the checks here?
yeah .. ``0`` probably also doesn't make sense .. so i am fine if you just remove the ``!isset()`` part since its redundant now
Yes, we should fix this when we also allow you to add doc strings to protocol functions.
Is integer here a variable or a type? If a typle, I assume we are hard coding some types?
Also, we can't depend on `Code.loaded_files`. For example, `Mix` uses `Code.load_file` to load some files and invoking it here would cause mix files to also be reloaded. If we want this functionality, we probably should store the modules given to `r/1` in the current process dictionary.
This comment is incorrect, remove.
I guess you have mixed tabs and spaces here. Can you please make sure to use only spaces?
No, xbmc will inject the proper id
why not equals?
It doesn't have to be prefixed with the slash.

    composer.lock
    libs/
    !libs/.*
    !libs/web.config

is fine :)
I like the idea of this function, but it needs some tweaks.  I'll give you extensive style advice here, but we should talk about the overall strategy of the rest of the code before you make too many other changes.  (My style guide is at https://github.com/hadley/devtools/wiki/Style)

I'd recommend calling this `temp_package` and please make sure you surround every `=` with spaces.
It cannot because it is a one to one relationship and not a real property
ok i'll merge it to prod
what about using isTransient() in the checks of it inside save() and delete()? That way I would think since you are accessing the already there public property, you wouldnt be violating anything?
上から順番に評価されるので、rootみたいなのは一番下がいいと思いました
that line is missing from the merge i guess?