Clarify rvalue reference to unique_ptr parameter in initialize_generic

Author: YannickJadoul

include/pybind11/pybind11.h

@@ -256,6 +256,9 @@ class cpp_function : public function {
     /// Register a function call with Python (generic non-templated code goes here)
     void initialize_generic(unique_function_record &&unique_rec, const char *text,
                             const std::type_info *const *types, size_t args) {
+        // Do NOT receive `unique_rec` by value. If this function fails to move the unique_ptr, then
+        // we do not want this to destuct the pointer. `initialize` (the caller) still relies on the
+        // pointee being alive after this call. Only move if a `capsule` is going to keep it alive.
         auto rec = unique_rec.get();
 
         // Keep track of strdup'ed strings, and clean them up as long as the function's capsule