Remove debug print statements.

Author: EricCousineau-TRI

include/pybind11/cast.h

@@ -598,7 +598,6 @@ class type_caster_generic {
                                     if (!inst->simple_holder_constructed) {
                                         if (inst->owned)
                                             throw std::runtime_error("Internal error?");
-    //                                    std::cout << "Reclaiming shared_ptr\n";
                                         try_to_reclaim = true;
                                     }
                                 }
@@ -619,14 +618,11 @@ class type_caster_generic {
                         // TODO(eric.cousineau): This may be still be desirable if this is a raw pointer...
                         // Need to think of a desirable workflow - and if there is possible interop.
                         if (!existing_holder) {
-                            throw std::runtime_error("No existing holder: Are you passing back a raw pointer without return_value_policy::reference?");
+                            throw std::runtime_error("Internal error: Should have non-null holder.");
                         }
                         // TODO(eric.cousineau): This field may not be necessary if the lowest-level type is valid.
                         // See `move_only_holder_caster::load_value`.
                         if (!inst->reclaim_from_cpp) {
-                            if (!existing_holder) {
-                                std::cout << "No holder" << std::endl;
-                            }
                             throw std::runtime_error("Instance is registered but does not have a registered reclaim method. Internal error?");
                         }
                         return inst->reclaim_from_cpp(inst, existing_holder).release();
@@ -1559,9 +1555,8 @@ struct copyable_holder_caster : public type_caster_base<type> {
             if (lowest_type->release_info.can_derive_from_wrapper) {
                 do_release_to_cpp = true;
             } else {
-                std::cout << "WARNING! Python-derived C++ instance will soon lose Python portion. "
-                    << "Please consider having your base class extend from pybind11::wrapper<>."
-                    << std::endl;
+                std::cerr << "WARNING! Python-derived C++ instance will soon lose Python portion. "
+                    "Consider having your base class extend from pybind11::wrapper<>." << std::endl;
             }
         }
 

include/pybind11/pybind11.h

@@ -10,8 +10,6 @@
 
 #pragma once
 
-#include <iostream>
-
 #if defined(_MSC_VER)
 #  pragma warning(push)
 #  pragma warning(disable: 4100) // warning C4100: Unreferenced formal parameter
@@ -1096,11 +1094,9 @@ struct holder_check_impl<detail::HolderTypeId::SharedPtr> : public holder_check_
         // Check use_count(), assuming that we have an accurate count (no competing threads?)
         if (load_type == detail::LoadType::DerivedCppSinglePySingle) {
             if (h.use_count() > 1) {
-                std::cout << "SharedPtr holder has use_count() > 1 on destruction for a Python-derived class." << std::endl;
                 // Increase reference count
                 const auto& release_info = lowest_type->release_info;
                 if (release_info.can_derive_from_wrapper) {
-                    std::cout << "Attempting to interrupt" << std::endl;
                     // Increase reference count
                     object obj = reinterpret_borrow<object>(src);
                     // Release to C++.
@@ -1240,7 +1236,6 @@ class class_ : public detail::generic_type {
         int orig_count = self.ref_count();
         unused(orig_count);  // Suppress release build warnings.
         assert(orig_count == 3);
-        std::cout << "Using custom __del__" << std::endl;
 
         auto v_h = inst->get_value_and_holder(lowest_type);
         detail::holder_erased holder_raw(v_h.holder_ptr(), release_info.holder_type_id);
@@ -1627,8 +1622,6 @@ class class_ : public detail::generic_type {
 
         // Check tp_dealloc
         if (is_py_derived) {
-            std::cout << "Have non-pybind11 type" << std::endl;
-
             // TODO(eric.cousineau): Consider moving this outside of this class,
             // to potentially enable multiple inheritance.
             const std::string orig_field = "_pybind11_del_orig";
@@ -1651,9 +1644,6 @@ class class_ : public detail::generic_type {
                 object new_dtor_py = cpp_function(del_new, is_method(h_type));
                 setattr(h_type, "__del__", new_dtor_py);
                 setattr(h_type, orig_field.c_str(), del_orig);
-                std::cout << "Replacing dtor with pybind11 dtor" << std::endl;
-            } else {
-                std::cout << "Already has custom del" << std::endl;
             }
         }
     }

include/pybind11/pytypes.h

@@ -11,9 +11,9 @@
 
 #include "detail/common.h"
 #include "buffer_info.h"
+#include <iostream>
 #include <utility>
 #include <type_traits>
-#include <iostream>
 
 NAMESPACE_BEGIN(PYBIND11_NAMESPACE)