Improve documentation for non-macro-dropped structs

Author: abrown

crates/openvino/src/partial_shape.rs

@@ -13,7 +13,8 @@ pub struct PartialShape {
 }
 
 impl Drop for PartialShape {
-    /// Drops the `PartialShape` instance and frees the associated memory.
+    // We don't use the `drop...!` macro here since:
+    // - the `instance` field is not a pointer as with other types.
     fn drop(&mut self) {
         unsafe { ov_partial_shape_free(std::ptr::addr_of_mut!(self.instance)) }
     }

crates/openvino/src/shape.rs

@@ -8,8 +8,9 @@ pub struct Shape {
 }
 
 impl Drop for Shape {
-    /// Drops the Shape instance and frees the associated memory.
-    //Not using drop! macro since ov_shape_free returns an error code unlike other free methods.
+    // We don't use the `drop...!` macro here since:
+    // - `ov_shape_free` returns an error code unlike other free methods
+    // - the `instance` field is not a pointer as with other types.
     fn drop(&mut self) {
         let code = unsafe { ov_shape_free(std::ptr::addr_of_mut!(self.instance)) };
         assert_eq!(code, 0);