labofsahil/github-event-dataset-2012 huggingface dataset:
Fix bugs in mutt colors

Dear Ethan,

this patch replaces ANSI color names by colorx names in mutt.  It fixes two issues:
- Fix mutt-colors-solarized-\*-16 in a (solarized) 256-color terminal: I use a 256-colors terminal (rxvt-unicode) configured to use solarized (through a .Xresources file). Therefore, I want to use mutt-colors-solarized-\*-16 to get the most accurate solarized colors.  But when using a 256-color terminal, mutt-colors-solarized-\*-16 (compiled with ncurses) fails because mutt uses "enhanced" colors; for instance, brightgreen will *not* be color10 anymore, but some fancy colorxxx.
- Enables bright colors without bold: moreover, as @elliottcable explained in issue #90, it is better to emit ANSI codes directly rather than the bright* colors because, at least in mutt, they imply a __bold__ typeface.  With my patch, you have access to bright colors without having your half of your interface turning bold.

As a side-effect, you can also use bright colors as background colors (and I suspect your "weird issues" with ncurses are related to this too).  My second patch removes the warnings about this.

Best regards,
Gabriel
tweak for better behaviour as a live switcher

live-switching ability doesn't actually work since it keeps appending to head instead of replacing the existing theme (if one is present).  custom styling of the theme can also be applied on top by ensuring that a theme for the same version is in the head before any custom styles.

also switched to google's https cdn so that sites behind a https domain can use this.
Next next webplatform

This will still not have webInspector working until @rcruz has his change pushed into master.
minimize execSync calls in identity extension

Similar to optimization put in last week for app/system.

This saves ~80ms on load.
Fix storage list

It's currently not possible to unmount the external SD.
Updated with details of minor enhancements released in 0.9.12
Patch 1

Makes filters in list view accept get parameters too.

from this dicussion : https://github.com/cedriclombardot/AdmingeneratorGeneratorBundle/issues/130.

Tested locally, seem to work great !
Create query class with initialize and search
typo の修正

1章の typo および表現の改善です。
Initial Commit

Here we go
Updated plugins_dir_path to plugin_dir_path in load_file()

Update plugins_dir_path to plugin_dir_path:

http://codex.wordpress.org/Function_Reference/plugin_dir_path
fix for ./lib/Mojolicious/Plugin/QooxdooJsonrpc.pm
All OAE scripts converted to Rspec.
Added documentation for scoped imports.

Resolves: http://d.puremagic.com/issues/show_bug.cgi?id=7329
Seems to break building. http://pastebin.com/2ta2GN3H
Should this still be commented?
Neslo by to vyresilt pres ->order()?
Ok I misunderstood the feature... I started with ``Post.first.roles`` using the ``has_many`` association giving all roles of a Post and then I tried to apply this on the class level.
I'm gonna change the method name ``with_role`` into ``find_roles`` and investigate how to implement this feature.
Here it is not needed, I just wanted to print the output of command so people can see what is happening. Another potential problem is that now run just executes and does not wait for command to complete which might lead to some potential races. This output prevents this, but might not be the best solution.
Just a question, where did you find the standards/guideline for adding examples to akka?
nevermind on the last question.  I can see which ones are based on platform dirs being populated.  Having a Access Denied issue on Windows 7 on x86_64 that I am trying to figure out.
Hmm. I have `delete-trailing-whitespace` in my before-save-hook. It's probably easiest for me to do this:

    (defadvice delete-trailing-whitespace (around disable-when-read-only activate)
      "Do nothing if buffer is read only"
      (unless buffer-read-only
        ad-do-it))

However, in general having buffer-modifying stuff in `before-save-hook` could be a problem with this mode. The following should disable `before-save-hook` in rebase-mode, if that's a better solution:

    (defadvice server-edit (around no-before-save-hook-in-rebase activate)
      "Disable `before-save-hook' in rebase-mode."
      (let ((before-save-hook 
             (when (not (eq major-mode
                            'rebase-mode))
               before-save-hook)))
        ad-do-it))

Maybe we could call every element of `before-save-hook` wrapped in `ignore-errors` with `buffer-read-only` set to t, so that all the "good" ones will succeed but hooks that modify the buffer would be stopped. How does that sound?
Good point.  Something like this should do the trick?

```
if( m_serviceThread.IsAlive )
   m_serviceThread.Join();
```
You need <wchar.h> here as well on *nix.
I like the idea. sometimes i have multiple api classes behind the same base path :-)
But the solution could be better.
Not much difference between calling a function multiple times or creating an array and passing it.
Add comment to upstream.
http://www.bugfree.org.cn/blog/?p=87#comment-2371
Not sure why the new_admin layout was specified in there and not higher up, but that was mostly copy paste generated and will be removed when we update the admin interface soon.
****
记事本SB了
i believe that the Deferred return from `parse` is going to resolve to the wrong thing (`list`) unless you return `result` in here.
I didn't remember what it was on Norwegian
http://devcenter.heroku.com/articles/cedar

> Ruby webserver (Webrick vs Thin)
>
> Because Cedar doesn’t inject any gem dependencies into your app, a vanilla Rails app will run on Webrick (the built-in webserver in the Ruby standard library). We highly recommend Thin for all apps. To use Thin, add it to your Gemfile:
바로 이 부분이 키 포인트이빈다. toggle() 함수입니다. slideToggle() animation 효과를 추가해 줍니다.
I know you've mentioned before that you had more plans for this branch, but I still think it's worth merging as-is for the sake of code cleanup and better organization.
yes, sure, thank you - copy and paste from go instead unit :(
is zfs available on all SunOS Boxes? If not the Resolution.exec returns nil and then we'll get an exception when scan is called on nil. This would cause error messages on all system that do not have the zfs binaries. A simple test whether zfs_v is nil should fix that
Another one that might be an option for people that want an informative statusline but don't really want to go through hoops might be https://github.com/Lokaltog/vim-powerline. Very easy to just place in the .janus folder and off you go! =)
And additionally, (how) might it affect our PID tuning?
Was it working otherwise? I accidentally left on some debugging tracebacks.
This would've thrown a NullPointerException.  drive wasn't constructed yet.
@MBAACC ？？？为什么要english name
you probably could've merged these two or removed the one above
jashkenas' Docco doesn't yet, but I'll still do a pull request.
Changed is_valid_isbn10 to explicitly check for "X" rather than ASCII 88 to prevent X-ending ISBNs from failing that validation.
Was meinst du mit DRY-Prinzip?
> Just check the code below with --debug:

I get the same results with 1.52 and git head.

> Where's the problem, then? Which code would confuse cppcheck? No one answered that with facts.

Good point. It would be nice to talk about some specific code.
:+1: However, can you move the recommended "live" configuration into the comment so that people know what to do with "ignore"?
But why you change the value to 50, if the map/unit refresh period is 100-200ms?
How i can disable Everything in Combat Log ?
@rsa желающих ищите портировать или тестировать?xD
How dare you leave debugging code in here.
very sorry - but i haven't that problems with arenas...
Did we ever issue a pull request for this? If not, might be worth a try so we don't need to maintain a fork.
This logic should certainly be shared.
IIRC, the behavior should be like this, assuming for every case that i=1:

b=i++ //b is 1, i is 2
b=++i //b is 2, i is 2
b=i-- //b is 1, i is 0
b=--i //b is 0, i is 0

And of course by the time the visitor is called, the compiler already made sure that "i" is mutable (++ and -- should fail when called on immutable values). Table 6.9 of the Ceylon spec details it in terms of predecessor/successor.
burası da kritik.  dizin üzerinde `run` görevi çağrıldığında dizini `touch` ederek zamanını güncelliyoruz ki menüde en tepede görünsün ;-)
Whoops, it should be 708362
Where is the commit with the changes? Did you pushed from to shazarlx/hypervm to hypervm/dev branch?
if the user fixes this error, do we need to `.removeClass('error')` or `.html('')`?
Ah cool, that's all landed so I can flip that?
I think it disappeared when deepak committed. DEEPAK! FETCH BEFORE YOU PUSH!
In the how to moderate doc (in 912f9916a904ded0f3371eb75578e98d8036869e), there were some leftover HEAD, >>>>>>>> , and ===== objects from a patch or something.

Fixed in : 190021c00b0bb70d5e006608538df323b8902aca
Do you have any use case in mind in which `create()` should not use `new`?
Also, right now there's no way to pass another function to `SingletonInstanceLoader` (since it can't be passed to the `instance()` method which calls it) and `create()` is not virtual, so it can't be overridden.  So we end up always using `new` anyway.
2172行目のbreak文の後、2165のwhileループからも抜けだす必要があると思いますが、どうでしょうか。

現状だと以下のような状況のときにGroupAに新しいSpriteを追加してもnextSiblingがSpriteBではなくSpriteCになってしまいます。

Scene___Group1___GroupA___SpriteA
　|　　　　　|______SpriteB
　|
　|______Group2___SpriteC
The error message here doesn't match the condition it's testing, does it?  It's a bit strange to report "filename is too long" when the path is empty...
it also add the same message when i go to view all users 
https://www.jaysonberger.com/image%202.JPG
can't do that unfortunately - it's a limitation of css transitions
いまあるコントローラーはerbを探しにいってるみたいでhamlが実行されない。
What about for Rupees?
e => ex
ErrorException is a way of wrapping mysqli errors into an exception object, so we can practice exception chaining.  For things like PDO, we'll just throw the InvalidQueryException with the PDOException as the $previous exception.  More work on this after beta3.
Finally this was fixed.
Lol, this was KILLING my server.
Here you go:

https://github.com/nulogy/spreadsheet/tree/for_zeno

The patch only contained one commit by the way (explicitely for excluding the gemset changes).  :)

Cheers,
Clemens
Maybe this should be a word instead of a phrase?
This commit along with 8551692ca0 is only equivalent to renaming src/src.pro to src/subtivals.pro which breaks the project layout, and prevents QtCreator to load it.
I had tried the xpath you mentioned already. Here is what is happening. Below is the XML that represents the ldap server and the mysql server. When the above xpath is executed, the elements that are returned (apart from all the other elements that contain the same '@computer' attribute) are the Instance sub element of the LDAPServerProcess and the MySQLProcess. The MySQLProcess can be checked directly but there is no way to get the Parent of the Instance element using any functions in the IPropertyTree class, other than traversing the tree from the root. Therefore I cannot tell the type of the Instance by just looking at it. Hence I had to treat LDAPServerProcess separately..

  <LDAPServerProcess build="_"
                     buildSet="ldapServer"
                     cacheTimeout="5"
                     description="LDAP server process"
                     filesBasedn="ou=files,ou=ecl"
                     groupsBasedn=""
                     ldapPort="389"
                     ldapSecurePort="636"
                     modulesBasedn=""
                     name="ldapserver"
                     sudoersBasedn="ou=SUDOers"
                     systemBasedn="cn=Users"
                     systemCommonName=""
                     systemPassword=""
                     systemUser=""
                     usersBasedn=""
                     workunitsBasedn="ou=workunits,ou=ecl">
   <Instance computer="myldap000021" name="s1" netAddress="10.0.0.21"/>
  </LDAPServerProcess>
  <MySQLProcess build="_"
                buildSet="mysqlserver"
                computer="mysql000020"
                description="MySQL database"
                name="mysqlserver"
                port=""/>
Two single quotes at the end.
this may be related to the percona choice.
shouldn't it be enough to switch the both lines?

`_originShadow` is removed from superlayer, and if superlayer was the last one holding onto `_originShadow` is is released.

And I would expect, that `insertSublayer:atIndex:` would implicitly remove the inserted layer from it's current superlayer, but found no docs that support that. In that case the `removeFromSuperlayer` message could be removed completely given `self.layer` is guaranteed to be non nil.
Haha of course, threads must be treated with care. 
In this case, this thread calls the function _threaded_fill(), and can be started only once a time.

All the functions that the thread knows, had there corresponding gtk.gdk locks and unlocks when a gtk object is used (the real important here to protect is the model), so only one gtk.gdk thread can access each gtk object. In another words if this new thread is working with the model, no other thread will be allowed due the locks. However could be something missing errors are human.
Don't you need a break here? I guess you don't want to Use the doors after each encounter :)

Please make a pull request to SD2 for this after you fix it.
And it's not booting ...
Kind of makes me wonder why no kernel abovve 2.6.35 boots on the HTC Leo ...
Bootloader restrictions?
Thanks for the input, Nacin! To be explicit:
    $screen = get_current_screen();
    if ( !method_exists( $screen, 'add_help_tab' ) ) return;
Procure seguir o padrão que estou usando. E não abrevie nomes, .button é SEMPRE melhor que .btn
with this the old algorithm is a bit faster; in a benchmark I get that the new algorithm starts to get faster around n = 190
```
         if len(a) < 190:
            n = len(a)
            for i in xrange(n - 1):
                b = a[i]
                for c in a[i + 1:]:
                    if b > c:
                        inversions += 1
```
can this fix be ported to all python addons? as i am still having the issus with xbmc-ps3d stealing xbmc webserver port??
I know how to use git.
You sounded like you didnt like 15 1 line commits so I squashed it all into 1 fresh commit.
BTW, you know how to use these attributs for making OTA package?
https://github.com/IceColdSandwich/android_build/blob/ics/tools/releasetools/ota_from_target_files

from line 15

I'm interested in --changeid and -e

what ever I try, they just don't get applied

I think that my calling of otapackage (the same way you call it) lord/aospX brakes this
Tests pass, but the code appears to be growing! A lot of repetition and near-repetition. I'm pushing this as-is in the hope that you'll be able to help me bang it into shape. I'll be out of town until Sunday afternoon, at which point I'll try to clear my plate before PuzzleNode Week begins.
ytuyt
Prior to this change, we were penalizing for fewer matches than expected, which is very likely a "normal" outcome rather than the result of a duplication event.  We're still filtering to ensure matches are on the same chromosome and within physical range of one another.
Very bad use of continue. Use inversed if comparation.

For example:

<pre>
if ($client === $c->nname) { 
    //Make things here
}
</pre>
Dont forget mono compile
-     <Compile Include="Core\GS\Generators\WorldGenerator.cs" />
+    <Compile Include="Core\GS\Map\WorldGenerator.cs" />
I'd just do the standard 404 page.
oO wrong information from sharq... he is copying everything ^^
sry :-)

Credits are going to kraven5!
Should we have a test case for the correct ordering?
@isyoon 뭔데? 담주에 나갈수 있으려나 고민중인데 아직 프로젝트 셋팅도 못해서 분위기 살피는 중 ㅋㅋㅋㅋ
할수 있는건 해야지.. ㅋㅋㅋ
you can set that in the designer
Two spaces between code and comment.
`[split..-1]`
Oh, haha. My fault. I actually forgot to set the winner_known value when I did the rewrite. Make that two things I need to fix regarding results.

Expect the fixes to be pushed shortly.
We're going to put some code in libsnes.c so we can dump the sound back to file, then we'll compare it against the dumped sound we get from a PC and go from there.
Can this please be merged and a new gem released?  I'm waiting on this to pin my app to the gem since vendor/plugins is being depreciated soon.
what about just passing "?diazo.off=1" in the url? it should be unthemed then
방금 수정해서 커밋 완료.
Yes, and already fixed. Will be pushed to master repo later.
*** note *** that after you do this pull this as recommended by Rei:
"then in your app do a bundle update, make sure the stale migration is deleted, then do rake spree_warehouse:install:migrations, drop the db and recreate/migrate it"

*** note *** on my mac, i have to use bundle exec before any rake commands it seems ( radar suggested i should be doing this wherever a Gemfile is involved )
Machiavelli fix you commit troble eye of acherus

commit 2293d2d7dfab3470c6d1fbdb84d7df691ac93240
Typo

DELETE FROM `script_texts` WHERE `entry` BETWEEN -1000273 AND -1000264;
It should but the way I just wrote it makes it easier as the spell summons a creature at a single position, yet there are four positions it should be summoned at.
A script should not get a warning for every event it does not implement. If you want to warn about misnamed timers, this function needs an additional bool parameter that is not set for events. If you want to warn about misnamed events, then we could check for functions named event*() but not in the set of defined events on load (could be a job for lint, but not sure if that is the right tool for the job anymore).
achá-lo
LGTM, just has this issue and was looking for the solution.
This should be settings with reasonable default values instead of this plain checks
yes. as you can see from my sole commit, i am a fan of what you've done in getting me started with my customization. i can't wait for your presentation from clojure conj to be released, so i can use it to get my musician friend into clojure. there aren't many clojure developers (i.e. i know of zero) around these parts.
We'll look into GUI stuff later in the semester, you'll probably like that, just because its not web programming. I don't think we're going to get to checkout [Bottle](http://bottlepy.org/docs/dev/)
It's not a good idea to comment on commits, please use "Issues" next time. And it would be more visible too. Distribution was uploaded to PAUSE several minutes ago. After several hours it should be available on almost all CPAN mirrors.
As far as I see, if shares are public, security setting from share to user changes nothing. And if shares are not public, security=share just does not work.
http://wowinstant.com/wild/mushrooms
http://www.wowhead.com/spell=88747
This bug bit us as well.  Have you submitted a pull request to basho?
@jmikola take care to change this constraint before tagging a release
Übernommen
im sure its not bad ..
Confirm. I got this error too.
+1 Please
You did this in the master branch :-( Please do it in a sub-branch first then when it's fully tested merge with master.
Thanks @toastdriven !

Haven't had a chance to play with the code yet, but in principle as I already said I think it looks good and I do like the approach.

Re `apply_limits` - I agree, just that it might create some false expectations for less-experienced django/tastypie users (I'm just thinking they might expect that the same magic that limits the result-set for GET/DELETE will somehow happen for POST/PUT if they use it inside `apply_limits`). I guess that's still the user's responsibility after all, and nothing really stops them from creating their own `apply_write_limits` or any other function to achieve the same thing (e.g. via `hydrate_x`).

Don't know what I was thinking about the `bundle.obj`. I see that it does indeed get set correctly.
no need for the oauth flag anymore, and xfbml and appid is already in the scripts hash

I'd rather have them only in init though.
Hey

I would suggest like @tomchristie mentioned on #145 to yield renderers here instead of instantiating all of them.
Кстати, там, вроде, далее по коду вы аще навесили eventRefLink'ов
Они тогда лишние, случайно?
These should all be pulled from runtime config. We have an external site-config.groovy file that can contain sensitive values and override default values on a per-installation basis.
I'm glad that it works now.

Frank

On Mon, Jan 16, 2012 at 6:37 AM, Mark Jackson
<reply@reply.github.com>
wrote:
> Frank
>
> Ok ... did a full clean / make and all now seems fine.
>
> Must have been some weird hang over from the different commits.
>
> Thanks for checking.
>
> Regards
> Mark
>
> On 15/01/12 21:02, Frank Hunleth wrote:
>> I verified that there are no source code differences between the two
>> Linux kernels, so the problem must be somewhere else. I did switch to
>> using mdev, but I don't think that's the problem. Anyway, Ethernet
>> works fine for me. Here's a copy of the build that I'm using right now
>> if you'd like to try it:
>>
>> http://dl.dropbox.com/u/37770195/beaglebone/uImage
>> http://dl.dropbox.com/u/37770195/beaglebone/rootfs.ext2
>>
>> I don't have Ethernet start up automatically, so run "udhcpc" to bring
>> up the interface and get an IP address.
>>
>> On Fri, Jan 13, 2012 at 10:32 PM, Frank Hunleth
>> <fhunleth@troodon-software.com> wrote:
>>> Just wanted to give you an update. I rebuilt clean with the current
>>> tree to make sure that I didn't have any stale files. Ethernet still
>>> works for me. I haven't checked the patches again yet.
>>>
>>> On Fri, Jan 13, 2012 at 11:36 AM, Frank Hunleth
>>> <fhunleth@troodon-software.com> wrote:
>>>> Ugh. There must be something subtly different with the patches that I
>>>> didn't see. I'll double check this weekend. Thanks for helping debug
>>>> this.
>>>>
>>>> On Fri, Jan 13, 2012 at 11:10 AM, Mark Jackson
>>>> <reply@reply.github.com>
>>>> wrote:
>>>>> No ... it's still not working.
>>>>>
>>>>> I can't ping with commit f9614a157d23ed5d10e5b148e723c70ba611a84d.
>>>>> But ping works with commit 10ef7e71118146fd91cbb5fa01f6a3b704b75897
>>>>>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/fhunleth/buildroot-beaglebone/commit/f9614a157d23ed5d10e5b148e723c70ba611a84d#commitcomment-870438



-- 
Frank Hunleth
Troodon Software LLC
http://troodon-software.com/
Damn! I have applied the patch to the wrong file!
to get it worked with Symfony 2.0.10 I needed change it to:
locale_key:   locale
Hello, but the development of Opera-Translate was interrupted? It was a very useful extension.
... revert !?
```ruby
periodo.should have(1).turma
# ou 
periodo.turmas.should include turma
```
remove this commit. I have already applied it to master.
Ok,
sorry, will remove the line.
No. It has nothing to do with the hibernate flushing issue. The problem was that indentifier and identifierType were validated separately that is if you changed identifier then it was validated against old identifierType instead of updated identifierType. Now I always validate updated identifier and identifierType.
But why would you want to see that your phone is fully charged if it's turned off, and not want that if it's turned on? I don't really understand that, IMO one wants consistent behaviour.

I converted charge_only_mode to using liblights because we got exactly that kind of complaints ('I have disabled the charging LED, why is it on if the phone is off? That annoys me when I have the phone on my desk at night !!!one).
Don't include `@return` tag when there is no return.
```state```の部分ですよね。 ```absence``` や ```attendance``` 等の値を入れるようにすることに賛成です。ただ、 ```:user_event``` には ```state``` は値なしでいいかなと思ってます。 ```FactoryGirl.create( :user_event , state: :absent )``` のようにテストの度に設定すればいいかなと思ってます。
Code style: `is_dir(...)` instead of `is_dir (...)`.
Nitpick: Double quotes in this project. Otherwise, this is looking good so far!
OK, this is fine then. Usually I use return false in event handlers to say "no, I really mean it, don't run anything else."
Should simply just be `log`.
I think this should just be `throw e`, since the parent handles the call anyway. Potentially then, we could mark `handleLdapError` private in the parent class.
Wont this break current scripts that use PercentColorMMEx?
why is this case class, and other exceptions are not?
Thinking a bit more, I just added big warnings to the docs. It only serves its purpose if it prints the whole tree, nothing else.
faith in humanity - 1 :-)
"actor could after the restart go into a buffering mode"

Might be misleading if someone interprets this line as this is a built-in functionality (not something the user must implement himself).
If we make the change in an isolated commit then it's easy to revert it if
there's any problems.

Cheers,
√
Ok, I renamed it to ByteString1C now. :-)
possible to use receiveTimeout instead?
Sounds great, let me know if any of this isn't clear to you. Also I'd like to know if this direction finder works for your kernel models and if so, how fast or slow it is.
Why do we need a logic action for the proxy?
should be redirect(rsc['url']
)
"initially" is misspelled.
Yeah, I also didn't write this part. Again, I had no idea what it was for or who uses it, so I decided not to change it.
this should not be necessary if the 'change' event handler from above is used.
Is there an specific reason for using *setattr* instead of simply doing *self._id = _id* ?
```py
keys = keys & instance._changed
```
As above, these can all be properties.
lolxcode. I wonder if there's a better way to be creating this frame. It's too massive for my personal taste.
It is not wrong, it is incomplete.
Should we perhaps derive these error classes from the most similar Python exceptions, such as ValueError?

That way, the user can simply catch ValueError, instead of having to import another error class (ideally, we could just have used ValueError from the start, but I see that the specialized exception is basically used to template the error message).
Maybe just inserting "when the" before "new base".
That was also my first thought, but I don't understand why and won't be
able to get back to it until the middle of next week.
On Jul 13, 2012 6:09 PM, "njsmith" <
reply@reply.github.com>
wrote:
>
> >
> >  """
> >  import string
> > -import sys
> > -
> > -if sys.version_info[0] >= 3:
> > -    rel_import = "from . import"
> > -else:
> > -    rel_import = "import"
>
> I bet it's the lack of this code that's causing the import problem on Py3.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/numpy/numpy/pull/334/files#r1159923
You might as well commit it then. I checked the URL and it looks good to me. I assume PyPi can figure out the versions from there?
Might as well put spaces around == while you are at it.
Not that it makes a difference, but zpp failures normally use just `return;` instead of of any particular `RETURN_*` value.
паблик-поля is baaad, mkay?
комментарий немного сбивает с толку. мы же вставляет элемент с ID in "внутрь" линка с edge. наверное, надо как-то перефразировать
почему весь файл поменялся?
Use `references` here or `belongs_to` in all the example.
Yes I have the same issue as you. Trying to find out.
I think it is better to maintain the indentation.
Because the RFC states that browsers have to allow at least 4096 bytes per cookie. The browser _may_ provide more than that, but there's no guarantee that more than 4096 bytes is allowed. Going with the RFC is easier and probably more reliable than sniffing user-agent strings and trying to accommodate each individual browser's actual max. cookie size.
There's no reason it can't be a val.  Sometimes when working on these super-sensitive one-wrong-move-and-here-come-the-cycle-induced-errors I turn things into defs while I'm working on them as a sort of general purpose palliative.  When it's called immediately after the def, it isn't serving any purpose. (I don't think a type's symbol can ever change - can it? But normalization does uncork laziness, that would be what it is defending against.)
There is one in the jvm group of the tests.
Already starting to undo all my good works, huh? No relative imports!
I think so. I am asking for the invoice template which doesn't exist, so it is falling back and rendering the default template for the action, show.
Adding @hazelnusse to this discussion too.
Why do you prefer an internal import to a bottom import?
Changed to `commutative_diagrams`, which is definitely better than just `commutative`.
Stuff in SymPy should be rebuildable and hashable. This almost (completely?) implies immutable. I suppose more discussion is necessary here. I will mention it on the next blog post I suppose.
The README has been updated and a new feature for query matching has been added.
Actually I do not see much gain of moving the two objects `.header.framed` and `.header.hi.tex` to two files respectively; I mean they are simply string constants and it is probably not worth it to read and write through extra files. At the end of the day, they will be concatenated in `insert_header()`, and that will be the time to write the header into a separate file if `opts_knit$get('self.contained') == TRUE`.
Is there a way to actually run these?
Need example here.
Please aim for minimal diffs - if you don't need to touch a file, don't.
Ok, that's a good idea to use ```^``` instead of ```\A```. We can ignore the corner-cases for now.
Lines 13 and 14 are indented one level too far
Yeah, that is probably ok, at least to start.
Superfluous casts.
Should not have changed these in this file.  Logger.new makes an instance of the logger; we only want to make changes when the logger "log this" methods are being called, e.g. logger.debug, logger.info. logger.error, logger.fatal, logger.warn.
I think this should be an `Option[String]` to avoid nulls
Reworked all of this yesterday.  Will push changes out today.
It would seem a bit inconsistent to not support views -  I can see a scenario arising when somebody starts an inventory and expects it to just work with views, by which time we will have forgotten that it was left out of this patch. If it is not too much effort, I think views should go into this patch - if it would blow the scope then this should be an immediate follow up item.
missing one spance
You probably want this outside the `django_statsd` middleware.
Can we move this whole thing into a class method on `ApiClient`, e.g., `ApiClient.reregister(name)` which returns an instance of `ApiClient` or raises a 404?
No, there isn't. It really isn't necessary to have changes to sequences on different tables all grouped together in the same transaction.
Coding Guidelines state to use double quotes were needed. We never escaped single quotes manually.
Python code?
aici ar fi bine cred să redirecteze la categoria in care a fost adaugata intrebarea.
BTW: I think Nokogiri has real canonicalization support now...
`to_sym` is not really intention-revealing. what about changing this to something like `name` or ... i don't know.
constructor checks are notoriously poor ways to determine type. If we really want an Object we may want to use Y.lang.isObject here.
Reformat comments of this type into jsdoc standard.
Do you want to remove this conditional, and just perform this assignment before the loop?  It's technically superfluous if there are already events waiting, but it's cleaner, and probably a negligible cost.
notice that this file is unchanged (except for added comma). please remove from pull request
This kind of logic (setHtml('<div align="center" style="color:#FF0000"><b>BMI Status: Underweight</div>')) really shouldn't be handled in the controller. 

Also, you could abstract this better (and make your code more maintainable) by creating a helper method, something like:

updateBmiDisplay(color, bmiStatusText)
{
  updatedHtml = '<div align="center" style="color:#' + color + '"><b>BMI Status: ' + bmiStatusText + '</div>';
  Ext.getCmp('BmiStatusId').setHtml(updatedHtml)   
}
I think all newly added .js should be linked in one js file and link to that js file should go here!
1. Neither of these context managers deletes the file on the way out.
2. Re-opening NamedTemporaryFile isn't cross-platform.
3. Re-using `f` for two different types of values (and it isn't a great variable name anyway!).

It'd be best to replace the first context manager with a call to `IrisTest.temp_filename()`. For example:
```python
with self.temp_filename() as temp_path:
    with open(temp_path, 'wb') as temp_file:
```
On Wed, Jul 11, 2012 at 12:13 PM, lamblin
<reply@reply.github.com>
wrote:
> Those boundaries will have to be redefined at some point, but we're not there.
> My feeling is that as long as a "canonicalizing" optimization makes the graph simple/better/faster, and has no reason to be undone during the "specialize" phase, we might as well register it in specialize as well. That's what we do for a lot of simplifications, like neg(neg(x)).
> Anyway, since I can't think of an optimization introducing new CAReduce nodes that reduce over broadcastable dimensions, I don't think it makes a difference for now.

Good point. I won't change it now, but point taken.
I know I suggested you use this, but looking at the doc (http://docs.python.org/library/platform.html?highlight=platform#platform.architecture), it looks like it may not be 100% reliable on MacOS.
Would sys.maxsize always work?
Also, what happens if you don't add the assert_fast = True? It still goes fast right? Is that worth testing?
Should that be Apache Software License V2?
'protected' would be better here imo.
this should probably be a skipped test instead of a monkey patch
In light of all of this helpful discussion I intend to make the following changes to the ant report task

* Add a new optional options element
* Add a new sourcedirectives child of the options element
* Add a requirecomment attribute to the source directives element 

These will have the following defaults:

* source directives are ignored unless the sourcedirectives option is specified
* when enabled, source directives require a comment unless requirecomment="false" is set

Example:

```
<jacoco:report>

    <options>
      <sourcedirectives requirecomment="false" />
    </options>                        

    <executiondata>
        <file file="jacoco.exec"/>
    </executiondata>
                        
    <structure name="Example Project">
        <classfiles>
            <fileset dir="classes"/>
        </classfiles>
        <sourcefiles encoding="UTF-8">
            <fileset dir="src"/>
        </sourcefiles>
    </structure>
                        
    <html destdir="report"/>
                        
</jacoco:report>
```

The same principle should be followed when exposing source directive parsing from maven or EclEmma.
Please don't start names with an underscore. See, e.g., (here)[http://stackoverflow.com/questions/228783/what-are-the-rules-about-using-an-underscore-in-a-c-identifier] for reasons why. I still don't see the need to change the existing name, which was `studentT_CDF`. You said you used a script for generating code, but the script could be changed, couldn't it?
Olek once reported this: "In a SELECT statement which does not pass the results directly back to the user ORDER BY clause is not enforced to run on the master."
First, think about why the value of task.model is different from algo.incrModel: the former one is the model resulted from last iteration, the latter is the model in the middle of this iteration, which is subject to change in this transition function within the same iteration but different tuples. The resulting loss for the two cases: loss = f(m_k_0, t1) + f(m_k_0, t2) + ... + f(m_k_0, t_N) = \sum_{i=1}^{N} f(m_k_0, t_i); loss = f(m_k_1, t_1) + f(m_k_2, t_2) + ... + f(m_k_N, t_N). m_k_i is the resulting model at the i-th tuple of the k-th iteration, t_i is the i-th tuple, f(m, t) is the objective function we are minimizing. N is the total number of tuples. I decided the former one is more desirable...
This doesn't allow emote id zero as you noted in the fixup commit message.
Sorry, I shouldn't have committed that line. I reimplemented the accelerometer per microsoft standards that's why it got commented. As soon as I remove that comment, it is ready for merge.
There should be a space between the `#` and the `TODO`.
The src and dst of this symlink are identical.

In the ocaml formula, this is done like `ln_s HOMEBREW_PREFIX+"lib/ocaml/site-lib", lib+"ocaml/site-lib"`, in order to allow other packages to install ocaml bindings and have them picked up by the main ocaml formula. Is that relevant here?
Since this is in the RPM extensions, can we remove the distributor ID from this message? The user doesn't need to know the dirty details, just that they tried to bind to a repository and it failed.
fliegt der testcode hier noch raus? ggf. kannst du ja unit tests stattdesssen schreiben...?
ahh sorry, I do use NCrunch and thought it was automatically added so I removed it. Will fix it up.
The horrible error was not cut and pasted here, but you still need to remove these arguments to "pop"
@ibeauregard Personally, I Kevin's proposal conveys the intent much more clearly: it makes it explicit that you first compare the names, and only if they're different do you compare the full signature.
This fixes a crash caused by the editor not being set.
That name is not making sense for me. Can you change it to _doOpenFilesWithOptionalPath ?
Gabriel,

Thanks for submitting contributions to Brackets, but this code doesn't belong in the openAndSelectDocument() function. This will change the entire meaning of the function.

In general, the code should work like this: when a file is clicked in either the working set list or the project tree, and it is *not* already selected, the openAndSelectDocument() function should be called. If the file *is* already selected, then code should be called to rename it.

Also note that your code currently only works on second click in the working set, but I think it should also work in the project tree.

Not sure how familiar you are with github, but after you have updated your code, you don't need to submit a new pull request -- just push you're changes to this same branch and they will show up here. Also, add a comment to the pull request to say you are "ready for another review" so I will get notified with an e-mail.

Thanks,
Randy
if you have a different solution, that's fine, as long as the border is on the button bar.
I think getKeymap() is unneeded now that Menus doesn't use it.  We should probably remove it, since it's temping to get the map and add/remove entries directly instead of going through addBinding()/removeBinding().
Ditto -- remove return keyword
No, it is correct in the context of the cursor position immediately after the equal sign. We still show attribute value hints only when the user force with Ctrl+Space and the cursor is within an existing value. I know we're not consistent, but I don't think we should try to block it for all cases of unquoted values.
What is have_seal used for?
Hm, maybe it doesn't. I had made some changes earlier that made it necessary, but I can reverse them.

Any suggestions where it should happen?
I really like when git does not properly recognize new files :-)
These are magic numbers. I know that's unfortunate... I may use something smarter than a gridlayout, but not now.
I think we are quite flexible right now (i.e. any place that takes an OID also takes an object, and so on). But besides fixing this specific case here, if you find any other cases where we're being to stringent with types, I'd love to see that PR!
For language you have that.
Spaces aren't really needed here.
this function is not used.
Self reminder, change the 0px to 0.
I believe we did this to combat a race condition that was being hit to ensure the buttons load properly.
Ah, good eyes. Yes, it is. I'll fix that.
Put path here no *
Actually I made a mistake.

This should be

    except Exception as e:
        log.err(None, <msg>)
Yup, importing the software you're trying to install is a bad idea
A good idea. But since we are not in the context of the factory girl adapter it would be necessary to create one for just this purpose, which seems for me somehow wrong.
But maybe the method "factory_girl_class" isn't at the right place? By putting it in a more globally context, it would be accessible to everybody. On the other hand this would violate the idea of defining adapters. 
Frankly, I didn't dive deep enough into the matter to make here a proper decision. I just solved the problem I encountered with factory-girl 3.5.
Have also a look at spec_helper:58
```ruby
Object.const_set("FactoryGirl", factory_class)
```
I don't think this works. At the moment you use FG > 2, you redefine the constant FactoryGirl, and even worse, you undefine it afterwards. This may cause problems the moment you link FG > 2 in your gem.
ngx_str_t 的用 %V ?
You can ignore this one, but if you want to avoid having `java.util.List` everywhere you can do `import java.util.{List => JList}` then just use `JList` everywhere
add some stats here so we can see how much debug traffic we get
I think this isn't available in stable-1.0, or?
We are not the checking the order of commit paths. containsAll would only make sure that all of them exists and not the order.
stderr as well. And in all exceptions below.
It was, I need to nuke it.  Sorry!

It was moved to another area.
it will never get to this callback, err == null, res.statusCode == 200 ....

do you have the same issue?
Possibly: It may be reasonable to check `stream.readable` and do a bit of special-case logic if it turns out to be `false`.

I'll look into it, but I don't want to try to wedge that into this PR. In particular, if I do anything I'll want to do it uniformly to the other classes as well.
all functions in this file need the same treatment ... i'll skip leaving the line comment to every one of them.
We could do that but I feel like webapi.js is already huge and like you mentioned this code is only useful in mainstream FF.
I personally prefer to keep the fallback data in a separate file. The getAll mechanism for app loading is already async so waiting for the XHR request has no real downside.
PR #460 is not only for HDF5 but to also parse any other thing that is a ndarray-like object.
If AstroPy allows `pypi/futures` as optional dependency, I can replace `Future` class with that, no problem.
One thing I like in Ruby that I wish Python had are predicate functions for functions/methods that only return true or false.  Then you can write things like `build_with_cython?` and it's clear that it's a question, not an action.
Sure--  `self.set_undefined_options('foo', ('a', 'b'))` just means "if `self.b is None`, ensure that the 'foo' command is finalized and set `self.b = foo.a`".
In this case we're talking about an option that's called 'dist_dir' on both commands--so it's just "set my dist_dir equal to bdist's dist_dir if my dist_dir is undefined".
But do you see that line 4 does absolutely nothing under any circumstances?
yes that would be more logical order
Whatever this is supposed to do, I don't think it works. Clicking the "Verify Message" tab gets me to another set of tabs (Sign and Verify) with Sign selected...
You should also pass the 'test' connection as the 'ds' key in the array as a default
yes, because the assertions then would include a normal / somewhere along the line.
Links fixed too...
Very useful addition. Thanks!

I'd probably wrap this in an `if` rather than render an empty string. Basically just to save some work on edges that don't have labels.
Because i wanted to show you exactly my case, but yes it's the same. So what i can i do to solve this problem ?
@stof interface does not contain this method.
Ah, ok. I overlooked that somehow
thanks! Corrected. I had a test cas for this but it desappeared and so, this part of the code was not tested :(
This is not an accurate test of whether virtualenvwrapper is installed.  The WORKON_HOME env need not be defined (and in fact is not under default circumstances), and its default value is $HOME/.virtualenvs, not $HOME/Env as your tests would suggest.
Oh, [I know](https://github.com/laravel/laravel/pull/664). I was doing this because it is done this way throughout most of the core. Oh well, I'll probably change it...
I don't think you need that
`<YOUR_PASSWORD` -> `<YOUR_PASSWORD>`
I stopped passing in tags everywhere when they weren't any tests to check for them.
Out of curiosity, why would we want to not do a secure connection?
I didn't--that's a mistake!
Instead of `runtime` I would call it `duration`. Also there shouldn't be the need to call `test.get()` because the property will always be available.
oops, this was a mistake.  I'll just leave it on one line like it was before.
This `log` function appears to be redundant with the one at the top of the file.
The `c = ` isn't needed.
internal api requirements - four functions that are applied by a dispatcher, they must then all share a function signature/api.
Why the differences with the previous lines?
I'm assuming that `psycopg` automatically convers `True` and `False` to `true` and `false`.

Last time I looked at that was many years ago and then I had to manually convert it to the words `true` and `false`.
OK port is there and works now
``while True:``
Ah, good point. I'm not sure if Minitest lets you order some tests, but I suppose you could test `false` first, then `true`. At that point I don't know how much real value the test would give then.
Could you please remove the spaces around the "!"? (see coding standards)
`WithDate` isn't necessary anymore then
Yes, you are right, I'll make the default to just  return the new CacheItem object. Thanks
@eriksencosta thus, I'm not sure we can use expectedException for PHPUnit internal exceptions
Are you referring to when it is hidden due to the media query?
This looks wierd wouldnt the correct way be `DumperCollection[]|DumperRoute[]`
When I used to run `sympy-bot` some time ago, it used to tell me that my password was to be stored in a Python variable for the period the bot runs.  If I am not mistaken then, perhaps, what happens here is not so much different from the previous state of affairs.
forgot about changing that, will do
actually this auto-generated call to nil out the outlet is useless. removing alltogether.
Any exceptions are currently caught it Network_server.read_config, but it may indeed be better to translate them into more sensible exceptions in the Network_config module itself. I'll change that.
Style: indent two spaces left on lines 18-21, 24-27, 33, and 37-38.
you could mention that this class is based on a V-representation of the polytpe.
By the way, it could be nice to also return a minimal H-representation if needed (2x| edge |) matrix + | edge | vector....
Het "Product" object zou ik graag zien als aparte class binnen SuperPC/models.
Works for me! However, how does a developer find a package version that matches their SDK version?
> Why did this not happen before?

We used to have injection-running and router-creation interspersed, like so:

```javascript
var properties = ... get keys ...; // for injections

instantiate router;

for all properties, run injections;
```

Now that it's factored into methods, we instantiate the router first, and then we collect the keys and run the injections. Hence the collected keys will contain the newly created router as well.
@Sunight  可以用三元运算，或者后面情况多了可以用`switch`，python写多了很容易忘记用`switch`的
"It" should not be capitalized.
This is a little awkward, but I think that's my fault. Given how other options work, I think this is a pretty fair compromise.
added instructions for import. @hisakonog can you check these instructions and fix any typos when you make your import screen cast?
this fixed the bug of hover text going white
The jetty cookbook adds this file and my intuit_lc_solr cookbook updates it. If I remove this the the solr changes will be overwritten. That is probably OK because the solr cookbook will put them back. I think I am talking myself into this change because from the jetty perspective it shouldn't care what downstream cookbooks do.

Removing it.
IIRC should not be necessary; when you use the load->library action a property is automatically made:

`$this->CI->BaseWsRequest`