On Wed, 22 Oct 2025, [email protected] wrote:
From: Marc-André Lureau <[email protected]>
Use slightly better types for the job.
Signed-off-by: Marc-André Lureau <[email protected]>
---
audio/audio_int.h | 12 ++++++------
audio/audio_template.h | 6 +++---
include/qemu/audio.h | 8 ++++----
audio/audio.c | 30 +++++++++++++++---------------
4 files changed, 28 insertions(+), 28 deletions(-)
diff --git a/audio/audio_int.h b/audio/audio_int.h
index a97edd77dd..c804302e51 100644
--- a/audio/audio_int.h
+++ b/audio/audio_int.h
@@ -67,9 +67,9 @@ typedef struct STSampleBuffer {
typedef struct HWVoiceOut {
AudioBackend *s;
- int enabled;
+ bool enabled;
int poll_mode;
- int pending_disable;
+ bool pending_disable;
I don't know if this is part of a state for migration but if not maybe
it's better to also move pending_disable up before poll_mode to group the
bools that enable/disable something together (and the struct may need less
padding that way depending on what type by the compiler uses for bool).
Regards,
BALATON Zoltan
