LGTM. Thanks.

@maxauthority Is this a PR that can be merged in without someone else looking over?

@gkatsev: Patch is written in a good way, it even has a NEWS entry (which
is great!) and certainly makes sense and actually a bug fix.

So yes, sure, just pull it.

@zoltan: Thanks for the quick patch! I think there is a also an associated
bug report. If so, please also close it on GitHub.

Martin

On Thu, Jul 31, 2014 at 6:09 AM, Gary Katsevman notifications@github.com
wrote:

LGTM. Thanks.

@maxauthority https://github.com/maxauthority Is this a PR that can be
merged in without someone else looking over?

—
Reply to this email directly or view it on GitHub
#18 (comment)
.

Btw, if the description or commit message contains something like closes #5 or fixes #5 it will close the associated issue when the PR gets merged in.

Remove me please.
On Jul 31, 2014 9:38 PM, "Gary Katsevman" notifications@github.com wrote:

Btw, if the description or commit message contains something like closes
#5 or fixes #5 it will close the associated issue when the PR gets merged
in.

—
Reply to this email directly or view it on GitHub
#18 (comment)
.

@zoltan: sorry for including you, didn't know github sends mail for @ people, and I just wanted to reply to the "other" zoltan (klinger). Anyway, this bug is closed so after this message, there shouldn't be any new messages, and I don't see how I could remove you from this issue now. Please just select "unwatch" from the github's issue and you should be fine in future.

Sorry for the inconvenience.