Address review suggestions

Author: sergey-miryanov

Modules/_functoolsmodule.c

@@ -1076,7 +1076,7 @@ _functools_reduce_impl(PyObject *module, PyObject *func, PyObject *seq,
     for (;;) {
         PyObject *op2;
 
-        if (Py_REFCNT(args) > 1) {
+        if (_PyObject_IsUniquelyReferenced(args)) {
             Py_DECREF(args);
             if ((args = PyTuple_New(2)) == NULL)
                 goto Fail;

Modules/_testcapi/object.c

@@ -258,7 +258,7 @@ negative_refcount(PyObject *self, PyObject *Py_UNUSED(args))
     if (obj == NULL) {
         return NULL;
     }
-    assert(PyUnstable_Object_IsUniquelyReferenced(obj));
+    assert(Py_REFCNT(obj) == 1);
 
     Py_SET_REFCNT(obj,  0);
     /* Py_DECREF() must call _Py_NegativeRefcount() and abort Python */
@@ -275,7 +275,7 @@ decref_freed_object(PyObject *self, PyObject *Py_UNUSED(args))
     if (obj == NULL) {
         return NULL;
     }
-    assert(PyUnstable_Object_IsUniquelyReferenced(obj));
+    assert(Py_REFCNT(obj) == 1);
 
     // Deallocate the memory
     Py_DECREF(obj);

Objects/dictobject.c

@@ -5667,16 +5667,10 @@ dictiter_iternext_threadsafe(PyDictObject *d, PyObject *self,
 
 #endif
 
-static bool
-has_unique_reference(PyObject *op)
-{
-    return _PyObject_IsUniquelyReferenced(op) == 1;
-}
-
 static bool
 acquire_iter_result(PyObject *result)
 {
-    if (has_unique_reference(result)) {
+    if (_PyObject_IsUniquelyReferenced(result)) {
         Py_INCREF(result);
         return true;
     }

Objects/listobject.c

@@ -79,8 +79,7 @@ ensure_shared_on_resize(PyListObject *self)
     // We can't use _Py_CRITICAL_SECTION_ASSERT_OBJECT_LOCKED here because
     // the `CALL_LIST_APPEND` bytecode handler may lock the list without
     // a critical section.
-    assert(_PyObject_IsUniquelyReferenced((PyObject *)self) ||
-           PyMutex_IsLocked(&_PyObject_CAST(self)->ob_mutex));
+    assert(Py_REFCNT(self) == 1 || PyMutex_IsLocked(&_PyObject_CAST(self)->ob_mutex));
 
     // Ensure that the list array is freed using QSBR if we are not the
     // owning thread.

Objects/longobject.c

@@ -6953,7 +6953,7 @@ PyLongWriter_Discard(PyLongWriter *writer)
     }
 
     PyLongObject *obj = (PyLongObject *)writer;
-    assert(_PyObject_IsUniquelyReferenced((PyObject *)obj));
+    assert(Py_REFCNT(obj) == 1);
     Py_DECREF(obj);
 }
 
@@ -6962,7 +6962,7 @@ PyObject*
 PyLongWriter_Finish(PyLongWriter *writer)
 {
     PyLongObject *obj = (PyLongObject *)writer;
-    assert(_PyObject_IsUniquelyReferenced((PyObject *)obj));
+    assert(Py_REFCNT(obj) == 1);
 
     // Normalize and get singleton if possible
     obj = maybe_small_long(long_normalize(obj));

Objects/unicodeobject.c

@@ -1717,7 +1717,7 @@ unicode_dealloc(PyObject *unicode)
             // Successfully popped.
             assert(popped == unicode);
             // Only our `popped` reference should be left; remove it too.
-            assert(_PyObject_IsUniquelyReferenced(unicode));
+            assert(Py_REFCNT(unicode) == 1);
             Py_SET_REFCNT(unicode, 0);
 #ifdef Py_REF_DEBUG
             /* let's be pedantic with the ref total */