[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH v3 5/7] audio/coreaudio: Remove a deprecation warning on
From: |
Christian Schoenebeck |
Subject: |
Re: [RFC PATCH v3 5/7] audio/coreaudio: Remove a deprecation warning on macOS 12 |
Date: |
Mon, 10 Jan 2022 14:43:09 +0100 |
On Montag, 10. Januar 2022 14:20:56 CET Philippe Mathieu-Daudé wrote:
> On 1/10/22 14:09, Philippe Mathieu-Daudé wrote:
> > When building on macOS 12 we get:
> > audio/coreaudio.c:50:5: error: 'kAudioObjectPropertyElementMaster' is
> > deprecated: first deprecated in macOS 12.0
> > [-Werror,-Wdeprecated-declarations]>
> > kAudioObjectPropertyElementMaster
> > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > kAudioObjectPropertyElementMain
> >
> > /Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frame
> > works/CoreAudio.framework/Headers/AudioHardwareBase.h:208:5: note:
> > 'kAudioObjectPropertyElementMaster' has been explicitly marked
> > deprecated here>
> > kAudioObjectPropertyElementMaster
> > API_DEPRECATED_WITH_REPLACEMENT("kAudioObjectPropertyElementMain",
> > macos(10.0, 12.0), ios(2.0, 15.0), watchos(1.0, 8.0), tvos(9.0,
> > 15.0)) = kAudioObjectPropertyElementMain ^
> >
> > Replace by kAudioObjectPropertyElementMain, redefining it to
> > kAudioObjectPropertyElementMaster if not available, using
> > Clang __is_identifier() feature (coreaudio is restricted to
> > macOS).
> >
> > Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> > ---
> >
> > Checkpatch:
> > WARNING: architecture specific defines should be avoided
> > #10: FILE: audio/coreaudio.c:47:
> > +#if !__is_identifier(kAudioObjectPropertyElementMain) /* macOS >= 12.0
> > */
> >
> > Should we define __is_identifier() to 0 for GCC on macOS?
>
> So apparently yes:
> https://lore.kernel.org/qemu-devel/2147921.xlN1UyrzLN@silver/
>
> Christian Schoenebeck suggests:
>
> #if !defined(MAC_OS_VERSION_12_0) ||
> (MAC_OS_X_VERSION_MAX_ALLOWED < MAC_OS_VERSION_12_0)
> #define kAudioObjectPropertyElementMain kAudioObjectPropertyElementMaster
> #endif
Note: line wrap. So don't forget a backslash at the end:
#if !defined(MAC_OS_VERSION_12_0) || \
(MAC_OS_X_VERSION_MAX_ALLOWED < MAC_OS_VERSION_12_0)
#define kAudioObjectPropertyElementMain kAudioObjectPropertyElementMaster
#endif
I guess you would have seen it in the editor anyway, but just to be sure.
>
> > ---
> >
> > audio/coreaudio.c | 16 ++++++++++------
> > 1 file changed, 10 insertions(+), 6 deletions(-)
> >
> > diff --git a/audio/coreaudio.c b/audio/coreaudio.c
> > index d8a21d3e507..73cbfd479ac 100644
> > --- a/audio/coreaudio.c
> > +++ b/audio/coreaudio.c
> > @@ -44,10 +44,14 @@ typedef struct coreaudioVoiceOut {
> >
> > bool enabled;
> >
> > } coreaudioVoiceOut;
> >
> > +#if !__is_identifier(kAudioObjectPropertyElementMain) /* macOS >= 12.0 */
> > +#define kAudioObjectPropertyElementMain kAudioObjectPropertyElementMaster
> > +#endif
> > +
> >
> > static const AudioObjectPropertyAddress voice_addr = {
> >
> > kAudioHardwarePropertyDefaultOutputDevice,
> > kAudioObjectPropertyScopeGlobal,
> >
> > - kAudioObjectPropertyElementMaster
> > + kAudioObjectPropertyElementMain
> >
> > };
- Re: [RFC PATCH v3 1/7] configure: Allow passing extra Objective C compiler flags, (continued)
[RFC PATCH v3 2/7] ui/cocoa: Remove allowedFileTypes restriction in SavePanel, Philippe Mathieu-Daudé, 2022/01/10
[RFC PATCH v3 3/7] hvf: Make hvf_get_segments() / hvf_put_segments() local, Philippe Mathieu-Daudé, 2022/01/10
[RFC PATCH v3 4/7] hvf: Remove deprecated hv_vcpu_flush() calls, Philippe Mathieu-Daudé, 2022/01/10
[RFC PATCH v3 5/7] audio/coreaudio: Remove a deprecation warning on macOS 12, Philippe Mathieu-Daudé, 2022/01/10
[RFC PATCH v3 6/7] block/file-posix: Remove a deprecation warning on macOS 12, Philippe Mathieu-Daudé, 2022/01/10
[RFC PATCH v3 7/7] gitlab-ci: Support macOS 12 via cirrus-run, Philippe Mathieu-Daudé, 2022/01/10
Re: [RFC PATCH v3 0/7] host: Support macOS 12, Roman Bolshakov, 2022/01/10