From: Eric V. Smith <ericvsmith@users.noreply.github.com>
Date: Wed, 16 May 2018 11:24:00 +0000 (-0400)
Subject: bpo-33534: Remove unneeded test. (GH-6897)
X-Git-Tag: v3.8.0a1~1838
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9285835a05ad0319acef111340266c0f85ed549c;p=python

bpo-33534: Remove unneeded test. (GH-6897)

This condition as already tested before this code is called.
---

diff --git a/Lib/dataclasses.py b/Lib/dataclasses.py
index 480c6f7416..bb77d3b405 100644
--- a/Lib/dataclasses.py
+++ b/Lib/dataclasses.py
@@ -544,12 +544,11 @@ def _hash_fn(fields):
 
 
 def _is_classvar(a_type, typing):
-    if typing:
-        # This test uses a typing internal class, but it's the best
-        # way to test if this is a ClassVar.
-        return (a_type is typing.ClassVar
-                or (type(a_type) is typing._GenericAlias
-                    and a_type.__origin__ is typing.ClassVar))
+    # This test uses a typing internal class, but it's the best way to
+    # test if this is a ClassVar.
+    return (a_type is typing.ClassVar
+            or (type(a_type) is typing._GenericAlias
+                and a_type.__origin__ is typing.ClassVar))
 
 
 def _is_initvar(a_type, dataclasses):