On 09/29/2017 07:15 PM, Alistair Francis wrote: > Apparently we don't use __MSC_VER as a compiler anymore and we always > require a C99 compiler (which means we always have __func__) so we don't > need a special AUDIO_FUNC macro. We can just repleace AUDIO_FUNC with > __func__ instead. > > Checkpatch failures where manually fixed. > > Signed-off-by: Alistair Francis <[email protected]> > Cc: Gerd Hoffmann <[email protected]> > ---
Module the typo fixes spotted by Thomas, Reviewed-by: Eric Blake <[email protected]> > +++ b/audio/audio.c > @@ -424,12 +424,12 @@ static void audio_process_options (const char *prefix, > const char qemu_prefix[] = "QEMU_"; > size_t preflen, optlen; > > - if (audio_bug (AUDIO_FUNC, !prefix)) { > + if (audio_bug(__func__, !prefix)) { I wonder if it would be nicer to have audio_bug() as a macro that supplies __func__ automatically; but it's not enough of a concern to prevent this patch from going in as-is. -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
