[FFmpeg-devel] [PATCH]Only set tff in x264 encoder for interlaced frames

[Michael Niedermayer]

On Fri, Mar 15, 2013 at 12:26:32AM +0100, Carl Eugen Hoyos wrote:
> On Friday 15 March 2013 12:21:10 am Carl Eugen Hoyos wrote:
> > Hi!
> >
> > FFmpeg currently (re-)sets x264's tff parameter for every frame, this
> > breaks lossless encoding.
> > Attached patch fixes lossless encoding for many progressive samples, like
> > ticket #2165.
> 
> New patch attached that fixes the issue if interlaced encoding was not 
> selected, afaict the b_tff parameter only makes sense for interlaced 
> encoding.
> 
> Please comment, Carl Eugen

>  libx264.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> b35a128443e69026740595e1e225127f3c50f855  patchx264tff2.diff

LGTM

[...]


-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle
-------------- 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/20130316/d5492c35/attachment.asc>