diff options
author | Serhiy Storchaka <storchaka@gmail.com> | 2022-06-26 11:24:04 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-06-26 11:24:04 (GMT) |
commit | 1b27ec5ac62606b6b5da34dfb794d5975949f864 (patch) | |
tree | 2c4af8fbc22c13479b3788d6d6eaec4c9f30a422 | |
parent | 5ce819f3c50a30074c843d57c3488a911e4fa578 (diff) | |
download | cpython-1b27ec5ac62606b6b5da34dfb794d5975949f864.zip cpython-1b27ec5ac62606b6b5da34dfb794d5975949f864.tar.gz cpython-1b27ec5ac62606b6b5da34dfb794d5975949f864.tar.bz2 |
[3.11] gh-93820: Pickle enum.Flag by name (GH-93891). (GH-94288)
(cherry picked from commit 536985814a7116f14c9bc90aa1b3e3d36d5b2367)
Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
-rw-r--r-- | Lib/enum.py | 16 | ||||
-rw-r--r-- | Lib/test/test_enum.py | 44 | ||||
-rw-r--r-- | Misc/NEWS.d/next/Library/2022-06-16-11-16-53.gh-issue-93820.00X0Y5.rst | 1 |
3 files changed, 58 insertions, 3 deletions
diff --git a/Lib/enum.py b/Lib/enum.py index 5bd4da9..e11093c 100644 --- a/Lib/enum.py +++ b/Lib/enum.py @@ -1286,7 +1286,21 @@ class Flag(Enum, boundary=STRICT): """ def __reduce_ex__(self, proto): - return self.__class__, (self._value_, ) + cls = self.__class__ + unknown = self._value_ & ~cls._flag_mask_ + member_value = self._value_ & cls._flag_mask_ + if unknown and member_value: + return _or_, (cls(member_value), unknown) + for val in _iter_bits_lsb(member_value): + rest = member_value & ~val + if rest: + return _or_, (cls(rest), cls._value2member_map_.get(val)) + else: + break + if self._name_ is None: + return cls, (self._value_,) + else: + return getattr, (cls, self._name_) _numeric_repr_ = repr diff --git a/Lib/test/test_enum.py b/Lib/test/test_enum.py index c510f1b..ac5c52b 100644 --- a/Lib/test/test_enum.py +++ b/Lib/test/test_enum.py @@ -66,10 +66,27 @@ try: class FlagStooges(Flag): LARRY = 1 CURLY = 2 - MOE = 3 + MOE = 4 except Exception as exc: FlagStooges = exc +class FlagStoogesWithZero(Flag): + NOFLAG = 0 + LARRY = 1 + CURLY = 2 + MOE = 4 + +class IntFlagStooges(IntFlag): + LARRY = 1 + CURLY = 2 + MOE = 4 + +class IntFlagStoogesWithZero(IntFlag): + NOFLAG = 0 + LARRY = 1 + CURLY = 2 + MOE = 4 + # for pickle test and subclass tests class Name(StrEnum): BDFL = 'Guido van Rossum' @@ -2964,9 +2981,32 @@ class OldTestFlag(unittest.TestCase): def test_pickle(self): if isinstance(FlagStooges, Exception): raise FlagStooges - test_pickle_dump_load(self.assertIs, FlagStooges.CURLY|FlagStooges.MOE) + test_pickle_dump_load(self.assertIs, FlagStooges.CURLY) + test_pickle_dump_load(self.assertEqual, + FlagStooges.CURLY|FlagStooges.MOE) + test_pickle_dump_load(self.assertEqual, + FlagStooges.CURLY&~FlagStooges.CURLY) test_pickle_dump_load(self.assertIs, FlagStooges) + test_pickle_dump_load(self.assertIs, FlagStoogesWithZero.CURLY) + test_pickle_dump_load(self.assertEqual, + FlagStoogesWithZero.CURLY|FlagStoogesWithZero.MOE) + test_pickle_dump_load(self.assertIs, FlagStoogesWithZero.NOFLAG) + + test_pickle_dump_load(self.assertIs, IntFlagStooges.CURLY) + test_pickle_dump_load(self.assertEqual, + IntFlagStooges.CURLY|IntFlagStooges.MOE) + test_pickle_dump_load(self.assertEqual, + IntFlagStooges.CURLY|IntFlagStooges.MOE|0x30) + test_pickle_dump_load(self.assertEqual, IntFlagStooges(0)) + test_pickle_dump_load(self.assertEqual, IntFlagStooges(0x30)) + test_pickle_dump_load(self.assertIs, IntFlagStooges) + + test_pickle_dump_load(self.assertIs, IntFlagStoogesWithZero.CURLY) + test_pickle_dump_load(self.assertEqual, + IntFlagStoogesWithZero.CURLY|IntFlagStoogesWithZero.MOE) + test_pickle_dump_load(self.assertIs, IntFlagStoogesWithZero.NOFLAG) + @unittest.skipIf( python_version >= (3, 12), '__contains__ now returns True/False for all inputs', diff --git a/Misc/NEWS.d/next/Library/2022-06-16-11-16-53.gh-issue-93820.00X0Y5.rst b/Misc/NEWS.d/next/Library/2022-06-16-11-16-53.gh-issue-93820.00X0Y5.rst new file mode 100644 index 0000000..70bb1e6 --- /dev/null +++ b/Misc/NEWS.d/next/Library/2022-06-16-11-16-53.gh-issue-93820.00X0Y5.rst @@ -0,0 +1 @@ +Pickle :class:`enum.Flag` by name. |