On Wednesday, 4 March 2026 07:16:59 CET Akihiko Odaki wrote: > Reallocate buffers when the active device change as the required buffer > size may differ. > > Signed-off-by: Akihiko Odaki <[email protected]> > Reviewed-by: Phil Dennis-Jordan <[email protected]> > Acked-by: Christian Schoenebeck <[email protected]> > --- > audio/coreaudio.m | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/audio/coreaudio.m b/audio/coreaudio.m > index 23c3d1f80ac5..e4ec1df971c8 100644 > --- a/audio/coreaudio.m > +++ b/audio/coreaudio.m > @@ -471,6 +471,7 @@ static OSStatus init_out_device(CoreaudioVoiceOut *core) > core->device_id = device_id; > core->device_frame_size = device_frame_size; > core->hw.samples = core->buffer_count * core->device_frame_size; > + audio_generic_initialize_buffer_out(&core->hw); > core->ioprocid = ioprocid; > > return 0;
There is no specific reason to insert that call between the struct initializers here, or is there? I mean the device is suspended at this point, so it should not matter. Just looks a bit weird. But in general, LGTM: Reviewed-by: Christian Schoenebeck <[email protected]>
