diff options
author | Eric V. Smith <ericvsmith@users.noreply.github.com> | 2021-05-03 03:24:53 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-05-03 03:24:53 -0400 |
commit | 99ad742ea913e421d012c1a623029eac31bdfe85 (patch) | |
tree | c4645c0b2590b0c8c082986e6fd390ab7304e80c /Lib/dataclasses.py | |
parent | 72720a2639368436da0981983549d000170623cc (diff) | |
download | cpython-git-99ad742ea913e421d012c1a623029eac31bdfe85.tar.gz |
bpo-44015: dataclasses should allow KW_ONLY to be specified only once per class (GH-25841)
bpo-44015: Raise a TypeError if KW_ONLY is specified more than once.
Diffstat (limited to 'Lib/dataclasses.py')
-rw-r--r-- | Lib/dataclasses.py | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/Lib/dataclasses.py b/Lib/dataclasses.py index 363d0b66d2..cbba320e01 100644 --- a/Lib/dataclasses.py +++ b/Lib/dataclasses.py @@ -930,6 +930,7 @@ def _process_class(cls, init, repr, eq, order, unsafe_hash, frozen, # we can. cls_fields = [] # Get a reference to this module for the _is_kw_only() test. + KW_ONLY_seen = False dataclasses = sys.modules[__name__] for name, type in cls_annotations.items(): # See if this is a marker to change the value of kw_only. @@ -939,6 +940,10 @@ def _process_class(cls, init, repr, eq, order, unsafe_hash, frozen, _is_kw_only))): # Switch the default to kw_only=True, and ignore this # annotation: it's not a real field. + if KW_ONLY_seen: + raise TypeError(f'{name!r} is KW_ONLY, but KW_ONLY ' + 'has already been specified') + KW_ONLY_seen = True kw_only = True else: # Otherwise it's a field of some type. |