[FFmpeg-devel] [PATCH] Fix return value for incomplete H264 frame packets

Michael Niedermayer michaelni
Tue Aug 24 17:15:11 CEST 2010


On Tue, Aug 24, 2010 at 11:12:33AM -0400, Mike Scheutzow wrote:
> Michael Niedermayer wrote:
>> On Tue, Aug 24, 2010 at 12:53:58PM +0200, P?sztor Szil?rd wrote:
>>> Michael Niedermayer:
>>>> also until you read the relevant parts of h264 & h222 you are not qualified
>>>> to work on this. Its not something you will get working reliable by trial
>>>> and error.
>>> I'm not planning to work on this.
>>
>> i know, noone wants to fix it people just want to fill the code with hacks
>> that work for their samples.
>
> michaelni - please explain what method you have in mind for adding  
> timing information to the decode path for an h264 stream.

read h264 read h222 implement it in AVParser and do it completely not just
the subset needed for the current failing sample file


[...]
> I was expecting a different approach, where AVPacket.pts was set prior
> to the h264 decoder. If it was implemented this way, would that mean  
> that the place to add code is the h264 parser?

yes and if needed the code surrounding parsers and if needed to factor
code out of h264*

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

The real ebay dictionary, page 3
"Rare item" - "Common item with rare defect or maybe just a lie"
"Professional" - "'Toy' made in china, not functional except as doorstop"
"Experts will know" - "The seller hopes you are not an expert"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100824/14fd12a4/attachment.pgp>



More information about the ffmpeg-devel mailing list