[3.12] gh-116040: [Enum] fix by-value calls when second value is falsey (GH-116072) (GH-116476)

e.g. Cardinal(1, 0)

(cherry picked from commit 13ffd4bd9f)
This commit is contained in:
Ethan Furman 2024-03-07 15:26:13 -08:00 committed by GitHub
parent 4efd20e23a
commit d771729679
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 46 additions and 4 deletions

View file

@ -166,6 +166,13 @@ def _dedent(text):
lines[j] = l[i:]
return '\n'.join(lines)
class _not_given:
def __repr__(self):
return('<not given>')
def __bool__(self):
return False
_not_given = _not_given()
class _auto_null:
def __repr__(self):
return '_auto_null'
@ -718,7 +725,7 @@ class EnumType(type):
"""
return True
def __call__(cls, value, names=None, *values, module=None, qualname=None, type=None, start=1, boundary=None):
def __call__(cls, value, names=_not_given, *values, module=None, qualname=None, type=None, start=1, boundary=None):
"""
Either returns an existing member, or creates a new enum class.
@ -747,18 +754,18 @@ class EnumType(type):
"""
if cls._member_map_:
# simple value lookup if members exist
if names:
if names is not _not_given:
value = (value, names) + values
return cls.__new__(cls, value)
# otherwise, functional API: we're creating a new Enum type
if names is None and type is None:
if names is _not_given and type is None:
# no body? no data-type? possibly wrong usage
raise TypeError(
f"{cls} has no members; specify `names=()` if you meant to create a new, empty, enum"
)
return cls._create_(
class_name=value,
names=names,
names=names or None,
module=module,
qualname=qualname,
type=type,