Bug fix in test. Minor comment enhancements.

Author: rwgk

include/pybind11/detail/common.h

@@ -661,7 +661,7 @@ template <class T>
 using remove_cvref_t = typename remove_cvref<T>::type;
 #endif
 
-/// Example usage: is_same_ignoring_cvref<T, PyObject *>
+/// Example usage: is_same_ignoring_cvref<T, PyObject *>::value
 template <typename T, typename U>
 using is_same_ignoring_cvref = std::is_same<detail::remove_cvref_t<T>, U>;
 

tests/test_type_caster_pyobject_ptr.cpp

@@ -90,8 +90,8 @@ TEST_SUBMODULE(type_caster_pyobject_ptr, m) {
         std::size_t i = 0;
         for (; i < vec_obj.size(); i++) {
             py::handle h(vec_obj[i]);
-            if (static_cast<std::size_t>(h.ref_count()) < i) {
-                break;
+            if (static_cast<std::size_t>(h.ref_count()) < 2) {
+                break; // Something is badly wrong.
             }
             h.dec_ref();
         }