[Python-checkins] gh-97799: use inspect.get_annotations in dataclass (#97800)

larryhastings webhook-mailer at python.org
Mon Oct 3 18:46:17 EDT 2022


https://github.com/python/cpython/commit/00b5a08c807ebebf4180c06aac0c9b5c7d6c547f
commit: 00b5a08c807ebebf4180c06aac0c9b5c7d6c547f
branch: main
author: larryhastings <larry at hastings.org>
committer: larryhastings <larry at hastings.org>
date: 2022-10-03T15:46:09-07:00
summary:

gh-97799: use inspect.get_annotations in dataclass (#97800)

dataclass used to get the annotations on a class object using
cls.__dict__.get('__annotations__').  Now that it always imports
inspect, it can use inspect.get_annotations, which is modern
best practice for coping with annotations.

files:
A Misc/NEWS.d/next/Library/2022-10-03-14-42-13.gh-issue-97799.Y1iJvf.rst
M Lib/dataclasses.py

diff --git a/Lib/dataclasses.py b/Lib/dataclasses.py
index a567a33d646f..05d62b625760 100644
--- a/Lib/dataclasses.py
+++ b/Lib/dataclasses.py
@@ -920,10 +920,7 @@ def _process_class(cls, init, repr, eq, order, unsafe_hash, frozen,
             if getattr(b, _PARAMS).frozen:
                 any_frozen_base = True
 
-    # Annotations that are defined in this class (not in base
-    # classes).  If __annotations__ isn't present, then this class
-    # adds no new annotations.  We use this to compute fields that are
-    # added by this class.
+    # Annotations defined specifically in this class (not in base classes).
     #
     # Fields are found from cls_annotations, which is guaranteed to be
     # ordered.  Default values are from class attributes, if a field
@@ -932,7 +929,7 @@ def _process_class(cls, init, repr, eq, order, unsafe_hash, frozen,
     # actual default value.  Pseudo-fields ClassVars and InitVars are
     # included, despite the fact that they're not real fields.  That's
     # dealt with later.
-    cls_annotations = cls.__dict__.get('__annotations__', {})
+    cls_annotations = inspect.get_annotations(cls)
 
     # Now find fields in our class.  While doing so, validate some
     # things, and set the default values (as class attributes) where
diff --git a/Misc/NEWS.d/next/Library/2022-10-03-14-42-13.gh-issue-97799.Y1iJvf.rst b/Misc/NEWS.d/next/Library/2022-10-03-14-42-13.gh-issue-97799.Y1iJvf.rst
new file mode 100644
index 000000000000..71097d29d344
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2022-10-03-14-42-13.gh-issue-97799.Y1iJvf.rst
@@ -0,0 +1,2 @@
+:mod:`dataclass` now uses :func:`inspect.get_annotations` to examine the
+annotations on class objects.



More information about the Python-checkins mailing list