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

[Michael Niedermayer]

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130208/9abcfbe2/attachment.asc>