Skip to content

bpo-30217: add the operators ~ and | to the index #1502

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 9, 2017
Merged

bpo-30217: add the operators ~ and | to the index #1502

merged 1 commit into from
Jun 9, 2017

Conversation

marco-buttu
Copy link
Contributor

Fix bpo-30217: Missing entry for the tilde (~) operator in the Index. Also the | operator was not indexed.

@mention-bot
Copy link

@marco-buttu, thanks for your PR! By analyzing the history of the files in this pull request, we identified @birkenfeld, @ncoghlan and @benjaminp to be potential reviewers.

@@ -394,10 +394,13 @@ Bitwise Operations on Integer Types
pair: bitwise; operations
pair: shifting; operations
pair: masking; operations
pair: inversion; operations
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Inversion or inverting?

Copy link
Member

Choose a reason for hiding this comment

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

I think this index entry is not needed. ~ is a bitwise operation. There are no separate index entries for bitwise conjunction and disjunction.

@marco-buttu
Copy link
Contributor Author

Thanks @serhiy-storchaka, I remove the last commit (that introduced the pair: inversion; operations entry).

@matrixise
Copy link
Member

@serhiy-storchaka maybe we could merge it.

@serhiy-storchaka serhiy-storchaka added docs Documentation in the Doc dir needs backport to 2.7 labels Jun 9, 2017
@serhiy-storchaka serhiy-storchaka merged commit 5eb7075 into python:master Jun 9, 2017
@serhiy-storchaka
Copy link
Member

Yes, thank you for the reminder. Do you mind to backport the change to other branches?

@marco-buttu
Copy link
Contributor Author

Ok @serhiy-storchaka

@Mariatta
Copy link
Member

Thanks for the PRs @marco-buttu 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants