diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2025-09-17 12:37:59 +0200 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2025-10-14 11:03:59 +0200 |
| commit | 8999ca00a4b69b0b7332bf3ae2fc17d473923793 (patch) | |
| tree | bbfd3841f08495ea37b512fc24d7d90ab7f59500 /scripts/lib/kdoc/kdoc_parser.py | |
| parent | 5b4fa9780728ba526f0529199ef90d6132a3faf2 (diff) | |
| download | qemu-8999ca00a4b69b0b7332bf3ae2fc17d473923793.zip qemu-8999ca00a4b69b0b7332bf3ae2fc17d473923793.tar.gz qemu-8999ca00a4b69b0b7332bf3ae2fc17d473923793.tar.bz2 | |
rust: migration: extract vmstate_fields_ref
This is useful when building a VMState for generic structs, because you have
to avoid nested statics. Using vmstate_fields! will fail in the likely case
where the _FIELDS static uses Self from an outer item, because that is
forbidden.
The separate macros are needed because you cannot just do
.fields(vmstate_fields_ref! {
vmstate_of!(PL011State, clock),
})
The value returned by vmstate_fields_ref! is not promoted to static, which is
unfortunate but intentional (https://github.com/rust-lang/rust/issues/60502):
error[E0716]: temporary value dropped while borrowed
--> rust/hw/char/pl011/libpl011.rlib.p/structured/device.rs:743:17
|
738 | / VMStateDescriptionBuilder::<PL011State>::new()
739 | | .name(c"pl011/clock")
740 | | .version_id(1)
741 | | .minimum_version_id(1)
742 | | .needed(&PL011State::clock_needed)
743 | | .fields(vmstate_fields_ref! {
| | _________________^
744 | || vmstate_of!(PL011State, clock),
745 | || })
| ||_________^- argument requires that borrow lasts for `'static`
| |_________|
| creates a temporary value which is freed while still in use
746 | .build();
| - temporary value is freed at the end of this statement
Thus it is necessary to use the "static", whether explicitly or hidden by
vmstate_fields.
Reviewed-by: Zhao Liu <zhao1.liu@intel.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'scripts/lib/kdoc/kdoc_parser.py')
0 files changed, 0 insertions, 0 deletions
