Apply suggestions from code review

eendebakpt · efimov-mikhail · web-flow · commit 37fc61d77ad0 · 2025-10-26T21:35:14.000+01:00
Co-authored-by: Mikhail Efimov &lt;efimov.mikhail@gmail.com&gt;
diff --git a/Lib/test/test_set.py b/Lib/test/test_set.py
@@ -2163,7 +2163,7 @@ def test_set(self):
         self.assertRaises(TypeError, _testcapi.pyset_add, s, [])
 
     def test_frozenset(self):
-        # Test the PySet_Add c-api for fronzetset objects
+        # Test the PySet_Add c-api for frozenset objects
         assert _testcapi.pyset_add(frozenset(), 1) == frozenset([1])
         frozen_set = frozenset()
         self.assertRaises(SystemError, _testcapi.pyset_add, frozen_set, 1)
diff --git a/Objects/setobject.c b/Objects/setobject.c
@@ -1177,7 +1177,7 @@ make_new_set_basetype(PyTypeObject *type, PyObject *iterable)
 void
 _PyFrozenSet_MaybeUntrack(PyObject *op)
 {
-    if ((op == NULL) || !(PyFrozenSet_CheckExact(op))) {
+    if (op == NULL || !PyFrozenSet_CheckExact(op)) {
         return;
     }
     // if no elements of a frozenset are tracked, we untrack the object

Original file line number	Diff line number	Diff line change
`@@ -1177,7 +1177,7 @@ make_new_set_basetype(PyTypeObject type, PyObject iterable)`
`1177`	`1177`	`void`
`1178`	`1178`	`_PyFrozenSet_MaybeUntrack(PyObject *op)`
`1179`	`1179`	`{`
`1180`		`- if ((op == NULL) \|\| !(PyFrozenSet_CheckExact(op))) {`
	`1180`	`+ if (op == NULL \|\| !PyFrozenSet_CheckExact(op)) {`
`1181`	`1181`	`return;`
`1182`	`1182`	`}`
`1183`	`1183`	`// if no elements of a frozenset are tracked, we untrack the object`