Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH] avdevice/audiotoolbox: silence warning with new api
@ 2023-12-09 12:50 xufuji456 via ffmpeg-devel
  2023-12-11 15:28 ` 徐福隆 via ffmpeg-devel
  0 siblings, 1 reply; 3+ messages in thread
From: xufuji456 via ffmpeg-devel @ 2023-12-09 12:50 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: xufuji456

Building with macOS platform, the compiler has a warning: 'kAudioObjectPropertyElementMaster' is deprecated in macOS 12.0

Signed-off-by: xufuji456 <839789740@qq.com>
---
 libavdevice/audiotoolbox.m | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/libavdevice/audiotoolbox.m b/libavdevice/audiotoolbox.m
index aa49e2c992..ae91a2bc62 100644
--- a/libavdevice/audiotoolbox.m
+++ b/libavdevice/audiotoolbox.m
@@ -85,7 +85,11 @@ static av_cold int at_write_header(AVFormatContext *avctx)
     AudioObjectPropertyAddress prop;
     prop.mSelector = kAudioHardwarePropertyDevices;
     prop.mScope    = kAudioObjectPropertyScopeGlobal;
+#if !TARGET_OS_IPHONE && __MAC_OS_X_VERSION_MIN_REQUIRED >= 1200
+    prop.mElement  = kAudioObjectPropertyElementMain;
+#else
     prop.mElement  = kAudioObjectPropertyElementMaster;
+#endif
     err = AudioObjectGetPropertyDataSize(kAudioObjectSystemObject, &prop, 0, NULL, &data_size);
     if (check_status(avctx, &err, "AudioObjectGetPropertyDataSize devices"))
         return AVERROR(EINVAL);
-- 
2.32.0 (Apple Git-132)

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [FFmpeg-devel] [PATCH] avdevice/audiotoolbox: silence warning with new api
  2023-12-09 12:50 [FFmpeg-devel] [PATCH] avdevice/audiotoolbox: silence warning with new api xufuji456 via ffmpeg-devel
@ 2023-12-11 15:28 ` 徐福隆 via ffmpeg-devel
  2023-12-11 15:40   ` Thilo Borgmann via ffmpeg-devel
  0 siblings, 1 reply; 3+ messages in thread
From: 徐福隆 via ffmpeg-devel @ 2023-12-11 15:28 UTC (permalink / raw)
  To: =?gb18030?B?ZmZtcGVnLWRldmVs?=
  Cc: =?gb18030?B?0Oy4o8Kh?=, =?gb18030?B?dGhpbG8uYm9yZ21hbm4=?=

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="gb18030", Size: 2494 bytes --]

Hi, Thilo:
There is another patch that needs your review please.


Thanks




------------------&nbsp;Original&nbsp;------------------
From:                                                                                                                        "FFmpeg development discussions and patches"                                                                                    <ffmpeg-devel@ffmpeg.org&gt;;
Date:&nbsp;Sat, Dec 9, 2023 08:50 PM
To:&nbsp;"ffmpeg-devel"<ffmpeg-devel@ffmpeg.org&gt;;
Cc:&nbsp;"Ð츣¡"<839789740@qq.com&gt;;
Subject:&nbsp;[FFmpeg-devel] [PATCH] avdevice/audiotoolbox: silence warning with new api



Building with macOS platform, the compiler has a warning: 'kAudioObjectPropertyElementMaster' is deprecated in macOS 12.0

Signed-off-by: xufuji456 <839789740@qq.com&gt;
---
&nbsp;libavdevice/audiotoolbox.m | 4 ++++
&nbsp;1 file changed, 4 insertions(+)

diff --git a/libavdevice/audiotoolbox.m b/libavdevice/audiotoolbox.m
index aa49e2c992..ae91a2bc62 100644
--- a/libavdevice/audiotoolbox.m
+++ b/libavdevice/audiotoolbox.m
@@ -85,7 +85,11 @@ static av_cold int at_write_header(AVFormatContext *avctx)
&nbsp;&nbsp;&nbsp;&nbsp; AudioObjectPropertyAddress prop;
&nbsp;&nbsp;&nbsp;&nbsp; prop.mSelector = kAudioHardwarePropertyDevices;
&nbsp;&nbsp;&nbsp;&nbsp; prop.mScope&nbsp;&nbsp;&nbsp; = kAudioObjectPropertyScopeGlobal;
+#if !TARGET_OS_IPHONE &amp;&amp; __MAC_OS_X_VERSION_MIN_REQUIRED &gt;= 1200
+&nbsp;&nbsp;&nbsp; prop.mElement&nbsp; = kAudioObjectPropertyElementMain;
+#else
&nbsp;&nbsp;&nbsp;&nbsp; prop.mElement&nbsp; = kAudioObjectPropertyElementMaster;
+#endif
&nbsp;&nbsp;&nbsp;&nbsp; err = AudioObjectGetPropertyDataSize(kAudioObjectSystemObject, &amp;prop, 0, NULL, &amp;data_size);
&nbsp;&nbsp;&nbsp;&nbsp; if (check_status(avctx, &amp;err, "AudioObjectGetPropertyDataSize devices"))
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return AVERROR(EINVAL);
-- 
2.32.0 (Apple Git-132)

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [FFmpeg-devel] [PATCH] avdevice/audiotoolbox: silence warning with new api
  2023-12-11 15:28 ` 徐福隆 via ffmpeg-devel
@ 2023-12-11 15:40   ` Thilo Borgmann via ffmpeg-devel
  0 siblings, 0 replies; 3+ messages in thread
From: Thilo Borgmann via ffmpeg-devel @ 2023-12-11 15:40 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Thilo Borgmann

Am 11.12.23 um 16:28 schrieb 徐福隆 via ffmpeg-devel:
> Hi, Thilo:
> There is another patch that needs your review please.

Thanks for pointing me to it.

> ---
> &nbsp;libavdevice/audiotoolbox.m | 4 ++++
> &nbsp;1 file changed, 4 insertions(+)

LGTM & pushed.

Thanks,
Thilo

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-12-11 15:40 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-12-09 12:50 [FFmpeg-devel] [PATCH] avdevice/audiotoolbox: silence warning with new api xufuji456 via ffmpeg-devel
2023-12-11 15:28 ` 徐福隆 via ffmpeg-devel
2023-12-11 15:40   ` Thilo Borgmann via ffmpeg-devel

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git