[FFmpeg-devel] avformat/aiffdec: parse replaygain metadata

Submitted by Moritz Barsnick on March 13, 2019, 9:17 a.m.

Details

Message ID 20190313091740.32543-1-barsnick@gmx.net
State New
Headers show

Commit Message

Moritz Barsnick March 13, 2019, 9:17 a.m.
Signed-off-by: Moritz Barsnick <barsnick@gmx.net>
---
Tested against sample provided here:
https://ffmpeg.org/pipermail/ffmpeg-user/2019-March/043717.html

 libavformat/aiffdec.c | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Moritz Barsnick March 18, 2019, 8:28 a.m.
On Wed, Mar 13, 2019 at 10:17:40 +0100, Moritz Barsnick wrote:
> Signed-off-by: Moritz Barsnick <barsnick@gmx.net>
> ---
> Tested against sample provided here:
> https://ffmpeg.org/pipermail/ffmpeg-user/2019-March/043717.html
> 
>  libavformat/aiffdec.c | 5 +++++
>  1 file changed, 5 insertions(+)

Friendly ping.

Cheers,
Moritz
Paul B Mahol March 18, 2019, 10:01 a.m.
On 3/18/19, Moritz Barsnick <barsnick@gmx.net> wrote:
> On Wed, Mar 13, 2019 at 10:17:40 +0100, Moritz Barsnick wrote:
>> Signed-off-by: Moritz Barsnick <barsnick@gmx.net>
>> ---
>> Tested against sample provided here:
>> https://ffmpeg.org/pipermail/ffmpeg-user/2019-March/043717.html
>>
>>  libavformat/aiffdec.c | 5 +++++
>>  1 file changed, 5 insertions(+)
>
> Friendly ping.

LGTM

Patch hide | download patch | download mbox

diff --git a/libavformat/aiffdec.c b/libavformat/aiffdec.c
index 7c701e0c70..ea2955816a 100644
--- a/libavformat/aiffdec.c
+++ b/libavformat/aiffdec.c
@@ -29,6 +29,7 @@ 
 #include "isom.h"
 #include "id3v2.h"
 #include "mov_chan.h"
+#include "replaygain.h"
 
 #define AIFF                    0
 #define AIFF_C_VERSION1         0xA2805140
@@ -348,6 +349,10 @@  static int aiff_read_header(AVFormatContext *s)
         }
     }
 
+    ret = ff_replaygain_export(st, s->metadata);
+    if (ret < 0)
+        return ret;
+
 got_sound:
     if (!st->codecpar->block_align && st->codecpar->codec_id == AV_CODEC_ID_QCELP) {
         av_log(s, AV_LOG_WARNING, "qcelp without wave chunk, assuming full rate\n");