[FFmpeg-devel] [PATCH] libavcodec:add dxva2 decoder for h264, mpeg2, vc1.wmv3

[Wei Gao]

Hi,Michael

Thank you for your reply.

iam not sure i understand the question, but yes
chroma_format_idc for h264  is always implicitly or explicitly stored
in SPS
for mpeg2 chroma_format is similarly stored in the ext seq header

I want to make sure that whether the check format code I wrote is enough to
check the h.264 and mpeg2 format for DXVA2 decode?

Thanks

Best regards.

2013/2/8 Michael Niedermayer <michaelni at gmx.at>

> On Wed, Jan 23, 2013 at 04:15:44PM +0800, Wei Gao wrote:
> > Hi,Hendrik
> >
> > - The handling of the parser in the h264 part of check_format looks
> > quite broken, i have no clue what its supposed to do? You have no data
> > to parse there.
> >
> > This parser, I reference vda process for h264 to check the format, I
> think
> > it just get the H264Context, using av_parser_parse2, and then check the
> > format.
> >
> > - get_mpeg2_video_format assumes there always is extradata, which may
> > not be the case, and should be checked.
> >
>
> > I just check whether the extra data is exist. Also I want to ask a
> question
> > that is the chroma_format always stored in the ext sequence header?
>
> iam not sure i understand the question, but yes
> chroma_format_idc for h264  is always implicitly or explicitly stored
> in SPS
> for mpeg2 chroma_format is similarly stored in the ext seq header
>
> [...]
>
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> Opposition brings concord. Out of discord comes the fairest harmony.
> -- Heraclitus
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
>