[MPlayer-dev-eng] [PATCH] Fix offsetting in Theora video files

Giorgio mywing81 at gmail.com
Tue Mar 16 19:51:49 CET 2010


2010/3/16 Reimar Döffinger <Reimar.Doeffinger at gmx.de>:
> On Tue, Mar 16, 2010 at 01:33:59AM +0100, Aurelien Jacobs wrote:
>> On Mon, Mar 15, 2010 at 01:34:56PM +0100, Giorgio wrote:
>> > Hello :)
>> >
>> > This patch fixes offsetting in Theora video files.
>>
>> You may be interested to fix this issue also with -demuxer lavf  (and
>> thus with fftheora). This is especially interesting as now fftheora is
>> faster than libtheora.

I'd like to see this fixed in fftheora too, but I've never looked at
the code :) Plus, when I try to use -demuxer lavf I keep getting
'[VD_FFMPEG] DRI failure.' no matter what vo I use (I tried x11, xv,
gl, gl2, sdl) and this makes decoding slower.

> I don't really like doing this at the decoder level unless the cropping is
> at most 16 pixels. Anything beyond that is IMO a task for video filters.

Yes, I think using a filter would be a better idea. But, I don't
understand the '16 pixels' part... the cropping is at most 2 pixels
(in case of odd offsets and 4:2:0 files, but it is generally
recommended that the offsets and sizes should be multiples of 2, so
"good" files will not be cropped at all) if you consider that outside
the picture area there no sensible data as stated in the format
specification: "The portions of the frame which lie outside the
picture region may contain arbitrary image data, so the frame must be
cropped to the picture region before display."

I've updated the patch to handle all the cases (now both offsets and
picture sizes can be odd or even) and fixed some mistakes, so now I
was able to use several vo to export images (pnm,png) and apply
several filters (flip,mirror,rotate...).

Giorgio Vazzana
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayer-theora-offset-01.diff
Type: text/x-diff
Size: 4858 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100316/f71498fd/attachment.diff>


More information about the MPlayer-dev-eng mailing list