[FFmpeg-devel] [PATCH] Implement PAFF in H.264
Michael Niedermayer
michaelni
Mon Oct 8 16:57:33 CEST 2007
On Thu, Oct 04, 2007 at 03:07:09PM -0400, Jeff Downs wrote:
> On Thu, 4 Oct 2007, Jeff Downs wrote:
>
> > I'll continue to split as I have time, given how productive this round
> > was.
> >
>
> OK, here is the next round of split patches. I took these diffs against a
> tree with the defreflist and reordering patches (posted short time ago)
> applied, but they will likely also apply clean to current svn.
>
> Again, in suggested application order with commit message.
>
>
[...]
> 8. paff-picmgmt.patch
> Manage Picture buffers for fields as well as frames. Pair complementary
> fields into one MPV Picture. Part of PAFF implementation.
>
>
>
> After these, there are two more -- one to indent after #8 and one to
> actually enable PAFF handling by properly defining FIELD_PICTURE.
> Since the indent one is quite large, I'll wait until these can be looked
> at to post the last two.
>
> -Jeff
[...]
> @@ -3991,8 +3998,50 @@
> }
>
> if(h0->current_slice == 0){
> - if(frame_start(h) < 0)
> + /* See if we have a decoded first field looking for a pair... */
> + if (s0->first_field) {
> + assert(s0->current_picture_ptr);
> + assert(s0->current_picture_ptr->data[0]);
> + assert(s0->current_picture_ptr->reference != DELAYED_PIC_REF);
> +
> + /* figure out if we have a complementary field pair */
> + if (!FIELD_PICTURE || s->picture_structure == last_pic_structure) {
> + /*
> + * Previous field is unmatched. Don't display it, but let it
> + * remain for reference if marked as such.
> + */
> + s0->current_picture_ptr = NULL;
> + s0->first_field = FIELD_PICTURE;
first field is set to 0, 1, FIELD_PICTURE and tested against 0/not 0
this is inconsistent
also it should be here !=0 already
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The educated differ from the uneducated as much as the living from the
dead. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071008/872098d3/attachment.pgp>
More information about the ffmpeg-devel
mailing list