summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSerhiy Storchaka <storchaka@gmail.com>2022-06-26 11:24:04 (GMT)
committerGitHub <noreply@github.com>2022-06-26 11:24:04 (GMT)
commit1b27ec5ac62606b6b5da34dfb794d5975949f864 (patch)
tree2c4af8fbc22c13479b3788d6d6eaec4c9f30a422
parent5ce819f3c50a30074c843d57c3488a911e4fa578 (diff)
downloadcpython-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.py16
-rw-r--r--Lib/test/test_enum.py44
-rw-r--r--Misc/NEWS.d/next/Library/2022-06-16-11-16-53.gh-issue-93820.00X0Y5.rst1
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.