[FFmpeg-devel] [PATCH] avformat/avidec: recognize H265 fourcc as hevc

[Michael Niedermayer]

On Sat, Mar 07, 2020 at 08:23:34AM +0100, Paul B Mahol wrote:
> On 3/6/20, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Fri, Mar 06, 2020 at 09:18:50PM +0100, Paul B Mahol wrote:
> >> Fixes decoding of came2_2020-01-13__20-38-58_21-00-00__Chn2.avi
> >>
> >> Signed-off-by: Paul B Mahol <onemda at gmail.com>
> >> ---
> >>  libavformat/avidec.c | 2 ++
> >>  libavformat/riff.c   | 1 +
> >>  2 files changed, 3 insertions(+)
> >>
> >> diff --git a/libavformat/avidec.c b/libavformat/avidec.c
> >> index 412e4a8479..7fdc0fbf8b 100644
> >> --- a/libavformat/avidec.c
> >> +++ b/libavformat/avidec.c
> >> @@ -832,6 +832,8 @@ FF_ENABLE_DEPRECATION_WARNINGS
> >>                          st->need_parsing = AVSTREAM_PARSE_FULL;
> >>                      if (st->codecpar->codec_id == AV_CODEC_ID_RV40)
> >>                          st->need_parsing = AVSTREAM_PARSE_NONE;
> >> +                    if (st->codecpar->codec_id == AV_CODEC_ID_HEVC)
> >> +                        st->need_parsing = AVSTREAM_PARSE_FULL;
> >
> > why does this not check for the codec_tag ?
> > the addition of a new tag in the table below makes this patch look like it
> > possibly affects only that tag
> 
> Are you sure about this?
> Do you have other file(s) and can prove its not needed?

i think you misunderstood me
i dont know at all, i was just asking if you know because the
patch looked like its added for the new tag

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200307/a031a43b/attachment.sig>