On Mon, Mar 17, 2025 at 06:11:35PM +0100, Paolo Bonzini wrote:
> Date: Mon, 17 Mar 2025 18:11:35 +0100
> From: Paolo Bonzini <pbonz...@redhat.com>
> Subject: Re: [PATCH 14/17] rust/vmstate: Add unit test for vmstate_of macro
>
> Thanks very much for the tests!
>
> On Mon, Mar 17, 2025 at 3:52 PM Zhao Liu <zhao1....@intel.com> wrote:
> > -pub use crate::bindings::{VMStateDescription, VMStateField};
> > -use crate::{
> > - bindings::VMStateFlags, callbacks::FnCall, prelude::*, qom::Owned,
> > zeroable::Zeroable,
> > -};
> > +pub use crate::bindings::{VMStateDescription, VMStateField, VMStateFlags};
>
> Does VMStateFlags have to be part of the public API?
I can do `use qemu_api::bindings::VMStateFlags` in vmstate_test.rs
directly, which is better since it's the raw value of VMStateField that
I need to check!
> > + assert_eq!(foo_fields[0].info, unsafe {
> > + ::core::ptr::addr_of!(vmstate_info_int8)
> > + });
>
> You can use & instead of addr_of here.
Thanks! Will fix.
Regards,
Zhao
