Skip to content
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.

Sign up for GitHub

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

Jump to bottom

gh-105497: [Enum] Fix flag mask inversion when unnamed flags exist #106468

Merged
merged 2 commits into from
Jul 11, 2023
Merged

gh-105497: [Enum] Fix flag mask inversion when unnamed flags exist #106468

merged 2 commits into from
Jul 11, 2023

Conversation

ethanfurman
Copy link
Member

@ethanfurman ethanfurman commented Jul 5, 2023 •
edited by bedevere-bot
Loading

For example:

class Flag(enum.Flag):
    A = 0x01
    B = 0x02
    MASK = 0xff

~Flag.MASK is Flag(0)

This restores 3.10 behavior:

~Flag.A = <Flag.B: 2>
(~Flag.A).value = 2
~Flag.A is Flag.B = True
~Flag.A == Flag.B = True

~Flag.Mask = <Flag.0: 0>
(~Flag.Mask).value = 0
~Flag.Mask is Flag(0) = True
~Flag.Mask == Flag(0) = True

@ethanfurman
Fix flag mask inversion when unnamed flags exist.
a6beb03 
For example:

    class Flag(enum.Flag):
        A = 0x01
        B = 0x02
        MASK = 0xff

    ~Flag.MASK is Flag(0)
@ethanfurman ethanfurman added type-bug An unexpected behavior, bug, or error stdlib Python modules in the Lib dir 3.11 only security fixes 3.12 bugs and security fixes needs backport to 3.11 only security fixes 3.13 bugs and security fixes needs backport to 3.12 bug and security fixes labels Jul 5, 2023
@ethanfurman ethanfurman self-assigned this Jul 5, 2023
@bedevere-bot bedevere-bot mentioned this pull request Jul 5, 2023
Closed
@ethanfurman ethanfurman requested review from ambv and hugovk July 5, 2023 21:45
@ethanfurman
Copy link
Member Author

That last commit should say IntFlag is KEEP -- Enum s don't have boundaries.

@ambv ambv merged commit 95b7426 into python:main Jul 11, 2023
@miss-islington
Copy link
Contributor

Thanks @ethanfurman for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 11, 2023
@ethanfurman @miss-islington
pythongh-105497: [Enum] Fix flag mask inversion when unnamed flags ex…
99ca7d8 
…ist (pythonGH-106468)

For example:

    class Flag(enum.Flag):
        A = 0x01
        B = 0x02
        MASK = 0xff

    ~Flag.MASK is Flag(0)
(cherry picked from commit 95b7426)

Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
@bedevere-bot
Copy link

GH-106620 is a backport of this pull request to the 3.12 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 11, 2023
@ethanfurman @miss-islington
pythongh-105497: [Enum] Fix flag mask inversion when unnamed flags ex…
784be81 
…ist (pythonGH-106468)

For example:

    class Flag(enum.Flag):
        A = 0x01
        B = 0x02
        MASK = 0xff

    ~Flag.MASK is Flag(0)
(cherry picked from commit 95b7426)

Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
@bedevere-bot bedevere-bot removed the needs backport to 3.12 bug and security fixes label Jul 11, 2023
@bedevere-bot
Copy link

GH-106621 is a backport of this pull request to the 3.11 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Jul 11, 2023
Yhg1s pushed a commit that referenced this pull request Jul 11, 2023
@miss-islington @ethanfurman
[3.12] gh-105497: [Enum] Fix flag mask inversion when unnamed flags e…
6968f9e 
…xist (GH-106468) (#106620)

gh-105497: [Enum] Fix flag mask inversion when unnamed flags exist (GH-106468)

For example:

    class Flag(enum.Flag):
        A = 0x01
        B = 0x02
        MASK = 0xff

    ~Flag.MASK is Flag(0)
(cherry picked from commit 95b7426)

Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
Yhg1s pushed a commit that referenced this pull request Jul 11, 2023
@miss-islington @ethanfurman
[3.11] gh-105497: [Enum] Fix flag mask inversion when unnamed flags e…
318f6ae 
…xist (GH-106468) (#106621)

gh-105497: [Enum] Fix flag mask inversion when unnamed flags exist (GH-106468)

For example:

    class Flag(enum.Flag):
        A = 0x01
        B = 0x02
        MASK = 0xff

    ~Flag.MASK is Flag(0)
(cherry picked from commit 95b7426)

Co-authored-by: Ethan Furman <ethan@stoneleaf.us>
@ethanfurman ethanfurman deleted the enum-invert branch February 1, 2024 01:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ambv ambv Awaiting requested review from ambv

@hugovk hugovk Awaiting requested review from hugovk

Assignees

@ethanfurman ethanfurman

Labels
3.11 only security fixes 3.12 bugs and security fixes 3.13 bugs and security fixes stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ethanfurman @miss-islington @bedevere-bot @ambv