Speed up __eq__

changelog.d/1310.change.md

@@ -0,0 +1 @@
+Speed up the generated `__eq__` methods significantly.

src/attr/_make.py

@@ -1864,20 +1864,20 @@ def _make_eq(cls, attrs):
     globs = {}
     if attrs:
         lines.append("    return  (")
-        others = ["    ) == ("]
         for a in attrs:
             if a.eq_key:
                 cmp_name = f"_{a.name}_key"
                 # Add the key function to the global namespace
                 # of the evaluated function.
                 globs[cmp_name] = a.eq_key
-                lines.append(f"        {cmp_name}(self.{a.name}),")
-                others.append(f"        {cmp_name}(other.{a.name}),")
+                lines.append(
+                    f"        {cmp_name}(self.{a.name}) == {cmp_name}(self.{a.name})"
+                )
             else:
-                lines.append(f"        self.{a.name},")
-                others.append(f"        other.{a.name},")
-
-        lines += [*others, "    )"]
+                lines.append(f"        self.{a.name} == other.{a.name}")
+            if a is not attrs[-1]:
+                lines[-1] = f"{lines[-1]} and"
+        lines.append("    )")
     else:
         lines.append("    return True")
 

tests/strategies.py

@@ -75,7 +75,7 @@ def _create_hyp_nested_strategy(draw, simple_class_strategy):
 bare_attrs = st.builds(attr.ib, default=st.none())
 int_attrs = st.integers().map(lambda i: attr.ib(default=i))
 str_attrs = st.text().map(lambda s: attr.ib(default=s))
-float_attrs = st.floats().map(lambda f: attr.ib(default=f))
+float_attrs = st.floats(allow_nan=False).map(lambda f: attr.ib(default=f))
 dict_attrs = st.dictionaries(keys=st.text(), values=st.integers()).map(
     lambda d: attr.ib(default=d)
 )