[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] coreaudio: Notify error in coreaudio_init_out
From: |
Christian Schoenebeck |
Subject: |
Re: [PATCH] coreaudio: Notify error in coreaudio_init_out |
Date: |
Sat, 26 Feb 2022 12:30:32 +0100 |
On Samstag, 26. Februar 2022 11:18:44 CET Akihiko Odaki wrote:
> On 2022/02/26 19:16, Christian Schoenebeck wrote:
> > On Samstag, 26. Februar 2022 11:02:48 CET Akihiko Odaki wrote:
> >> Signed-off-by: Akihiko Odaki <akihiko.odaki@gmail.com>
> >> ---
> >>
> >> audio/coreaudio.c | 2 ++
> >> 1 file changed, 2 insertions(+)
> >>
> >> diff --git a/audio/coreaudio.c b/audio/coreaudio.c
> >> index d8a21d3e507..d7cfdcc4fc4 100644
> >> --- a/audio/coreaudio.c
> >> +++ b/audio/coreaudio.c
> >> @@ -604,6 +604,8 @@ static int coreaudio_init_out(HWVoiceOut *hw, struct
> >> audsettings *as, coreaudio_playback_logerr(status,
> >>
> >> "Could not remove voice property
> >>
> >> change listener\n"); }
> >> +
> >> + return -1;
> >>
> >> }
> >>
> >> return 0;
> >
> > Is this a pure theoretical fix, or does it actually fix an actually
> > encountered misbehaviour? I don't see any explanation in the commit log. I
> > mean the branch here is about removing a listener only.
> >
> > Best regards,
> > Christian Schoenebeck
>
> I saw branching to this block results in abort as core->hw.samples is
> not set.
Ok, then please add an appropriate description to the commit log of what your
observed. Except of that:
Reviewed-by: Christian Schoenebeck <qemu_oss@crudebyte.com>
Best regards,
Christian Schoenebeck