[FFmpeg-devel] [PATCH] Implement PAFF in H.264

Martin Zlomek martin.zlomek
Thu Sep 20 23:42:27 CEST 2007


The attached patch fixes obvious typo (array length) in reference picture
list initialization.

PS: I am very sorry for wrong mime type of my previous mail attachment.


On Wed, 19 Sep 2007 21:28:18 +0200, Martin Zlomek <martin.zlomek at email.cz>  
wrote:

> Hello,
>
> better but not perfect results are given when the attached patch
> is applied over Jeff's patch.
> This patch fixes pic_id of reference frames/fields
> in "Initialisation process for reference picture lists".
>
>
> On Tue, 18 Sep 2007 20:30:39 +0200, Jeff Downs <heydowns at borg.com> wrote:
>
>> Attached is a patch that implements PAFF in H.264.
>>
>> I don't expect this to be accepted right away, due to size and
>> complexity,
>> but I want to get it out there for review and for people to try out and
>> comment on.
>>
>> It is working stable for me on a number of samples, including ones that
>> mix PAFF with progressive and/or MBAFF.
>>
>> Basic overview of how it works:
>> 1. Fields are decoded and deinterlaced.
>> 2. If non-paired fields are encountered, they are not output, but are
>> used
>> for reference if so marked.
>> 3. Reference lists are kept as they were before, but now may also  
>> contain
>> unmatched field pairs.  Reference list construction splits field pairs
>> out
>> to fields as needed.
>>
>> There is a major shortcoming/known issue with this patch;  it disables
>> the
>> error concealing calls for field pictures.  This is because the error
>> concealer needs slices to be contiguous regions of the picture buffers.
>> I'd appreciate input on how best to augment the error concealer to  
>> handle
>> interlaced slices (or if I am just missing something....)
>>
>> Also interested in hearing how this should be split into smaller patches
>> (I know that at least the addition/changes to header documentation  
>> should
>> come out).
>>
>>  	-Jeff
>
>
>



-- 
Martin Zlomek
martin.zlomek at email.cz
-------------- next part --------------
A non-text attachment was scrubbed...
Name: paff.2.patch
Type: text/x-patch
Size: 432 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070920/dca55443/attachment.bin>



More information about the ffmpeg-devel mailing list