adds comments at problematic spots in the code

Author: astrale-sharp

godot-core/src/meta/param_tuple/impls.rs

@@ -60,13 +60,15 @@ macro_rules! unsafe_impl_param_tuple {
                     $(
                         // SAFETY: `args_ptr` is an array with length `Self::LEN` and each element is a valid pointer, since they
                         // are all reborrowable as references.
+                        // no-commit: False with default parameters, should we change this?
                         unsafe { *args_ptr.offset($n) },
                     )*
                 );
 
                 let param_tuple = (
                     $(
                         // SAFETY: Each pointer in `args_ptr` is reborrowable as a `&Variant` for the duration of this call.
+                        // no-commit: False with default parameters, should we change this?
                         unsafe { varcall_arg::<$P>(args.$n, call_ctx, $n)? },
                     )*
                 );

godot-core/src/meta/signature.rs

@@ -78,6 +78,7 @@ impl<Params: InParamTuple, Ret: ToGodot> Signature<Params, Ret> {
         trace::push(true, false, call_ctx);
 
         // SAFETY: TODO.
+        // no-commit: problem with default args here!
         let args = unsafe { Params::from_varcall_args(args_ptr, call_ctx)? };
 
         let rust_result = unsafe { func(instance_ptr, args) };