Skip to content

bpo-35798: Fix duplicate SyntaxWarning: "is" with a literal.#11639

Merged
serhiy-storchaka merged 1 commit intopython:masterfrom
serhiy-storchaka:compare-is-literal-double-warning
Feb 16, 2019
Merged

bpo-35798: Fix duplicate SyntaxWarning: "is" with a literal.#11639
serhiy-storchaka merged 1 commit intopython:masterfrom
serhiy-storchaka:compare-is-literal-double-warning

Conversation

@serhiy-storchaka
Copy link
Copy Markdown
Member

@serhiy-storchaka serhiy-storchaka commented Jan 21, 2019

Copy link
Copy Markdown

@jwilk jwilk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't quite understand the code, but I confirm that this change fixes the symptoms for me.

Copy link
Copy Markdown
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be possible to add an unit test?

@serhiy-storchaka serhiy-storchaka merged commit 4583525 into python:master Feb 16, 2019
@serhiy-storchaka serhiy-storchaka deleted the compare-is-literal-double-warning branch February 16, 2019 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants