>  /// This macro can be used (by just passing it a type) to forward the 
> `VMState`
> @@ -572,9 +548,9 @@ const fn phantom__<T>(_: &T) -> 
> ::core::marker::PhantomData<T> {
>                  }
>                  Some(test_cb_builder__::<$struct_name, 
> _>(phantom__(&$test_fn)))
>              },

I want to keep the comment here as C version did, because there's an array flag 
:

// Though VMS_ARRAY is set, the num is 0: no data, only run test_fn callback

> +            flags: VMStateFlags(VMStateFlags::VMS_MUST_EXIST.0 | 
> VMStateFlags::VMS_ARRAY.0),
>              ..$crate::zeroable::Zeroable::ZERO
>          }
> -        .with_validate_flag()
>      };
>  }
> 
> Otherwise the series looks great, thanks!

Let me refresh the v3. :-)

Thanks,
Zhao


Reply via email to