[Bug 627459] theoraenc should provide option for TH_ENCCTL_SET_DUP_COUNT

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=627459
  GStreamer | gst-plugins-base | 0.10.x

--- Comment #26 from Alexey Fisher <bug-track at fisher-privat.net> 2011-05-12 09:27:01 UTC ---
(In reply to comment #25)
> * Please don't mix multiple changes in a single patch. Attach one patch for the
> dup-property/SET_DUP_COUNT and another one for removing the discont function.

ok. i removed it because it will not work with discont function

> * libtheora will handle the bitrate/keyframe constraints internally already and
> will not output more dup frames than allowed

Not correct, if we apply more dups than keyframe freq, th_encode_ctl will
return error. And we will get only one frame back. So why should we fail, if we
can set a limit for dups?

> * You should keep a queue with buffer timestamps and durations for the dup
> frames and use these values for the output instead of interpolating

I thought about this, but why? The dup setting make sence only with ogg,
also only with constant frame rate, only after videorate. If we get not
constant rate it will fail, so or so.

> * I think your GAP flag handling is wrong. You should always keep the last
> non-gap buffer, count the number of gap buffers and on the next non-gap buffer
> you encode the previous non-gap buffer, get the dups and then keep the new
> non-gap buffer and do everything from the beginning again

Ok. This will cure the timemachine effect.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.