From 251516bce69f42007d403eebf526194b7e2a803b Mon Sep 17 00:00:00 2001
From: Aaron Gokaslan <aaronGokaslan@gmail.com>
Date: Fri, 21 Oct 2022 12:51:26 -0400
Subject: [PATCH] Cleanup casters to release none() to avoid ref counting
 (#4269)

---
 include/pybind11/cast.h       | 6 +++---
 include/pybind11/functional.h | 2 +-
 include/pybind11/stl.h        | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/include/pybind11/cast.h b/include/pybind11/cast.h
index b89e4946a..5699d4fb0 100644
--- a/include/pybind11/cast.h
+++ b/include/pybind11/cast.h
@@ -248,7 +248,7 @@ public:
         return false;
     }
     static handle cast(T, return_value_policy /* policy */, handle /* parent */) {
-        return none().inc_ref();
+        return none().release();
     }
     PYBIND11_TYPE_CASTER(T, const_name("None"));
 };
@@ -291,7 +291,7 @@ public:
         if (ptr) {
             return capsule(ptr).release();
         }
-        return none().inc_ref();
+        return none().release();
     }
 
     template <typename T>
@@ -537,7 +537,7 @@ public:
 
     static handle cast(const CharT *src, return_value_policy policy, handle parent) {
         if (src == nullptr) {
-            return pybind11::none().inc_ref();
+            return pybind11::none().release();
         }
         return StringCaster::cast(StringType(src), policy, parent);
     }
diff --git a/include/pybind11/functional.h b/include/pybind11/functional.h
index 4034990d8..102d1a938 100644
--- a/include/pybind11/functional.h
+++ b/include/pybind11/functional.h
@@ -110,7 +110,7 @@ public:
     template <typename Func>
     static handle cast(Func &&f_, return_value_policy policy, handle /* parent */) {
         if (!f_) {
-            return none().inc_ref();
+            return none().release();
         }
 
         auto result = f_.template target<function_type>();
diff --git a/include/pybind11/stl.h b/include/pybind11/stl.h
index 8f243502e..2d144b598 100644
--- a/include/pybind11/stl.h
+++ b/include/pybind11/stl.h
@@ -311,7 +311,7 @@ struct optional_caster {
     template <typename T>
     static handle cast(T &&src, return_value_policy policy, handle parent) {
         if (!src) {
-            return none().inc_ref();
+            return none().release();
         }
         if (!std::is_lvalue_reference<T>::value) {
             policy = return_value_policy_override<Value>::policy(policy);