[FFmpeg-devel] [PATCH] avformat/utils: add support for reading ID3 tags at start of wav

[Tomas Härdin]

ons 2018-12-12 klockan 19:26 +0100 skrev Paul B Mahol:
> Fixes #4140.
> 
> > Signed-off-by: Paul B Mahol <onemda at gmail.com>
> ---
>  libavformat/utils.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/utils.c b/libavformat/utils.c
> index 93e588ee1e..7afef545fe 100644
> --- a/libavformat/utils.c
> +++ b/libavformat/utils.c
> @@ -646,7 +646,7 @@ FF_ENABLE_DEPRECATION_WARNINGS
>  
>      if (id3v2_extra_meta) {
>          if (!strcmp(s->iformat->name, "mp3") || !strcmp(s->iformat->name, "aac") ||
> -            !strcmp(s->iformat->name, "tta")) {
> +            !strcmp(s->iformat->name, "tta") || !strcmp(s->iformat->name, "wav")) {
>              if ((ret = ff_id3v2_parse_apic(s, &id3v2_extra_meta)) < 0)
>                  goto fail;
>              if ((ret = ff_id3v2_parse_chapters(s, &id3v2_extra_meta)) < 0)


Could this be generalized? Checking for specific demuxers seems a bit
off..

/Tomas