[MPlayer-dev-eng] Re: OpenDML Read/Write support

Ivan Kalvachev ivan at cacad.com
Mon Mar 22 01:10:20 CET 2004

I also notice this bug, and even
wrote a workaround.
But I don't dare to commit it because my
test was unsuccessful. I encoded movie
with lavc that
WMP refuse to play, VDub claim it have 0x0 size,
stuff like this.
I send my patch agains current CVS, take a
look and fix the bugs.
I also don't have time to merge it with
current solution.

Best Regards
   Ivan Kalvachev

Nico said:
> Jindrich Makovicka wrote:
>> D Richard Felker III wrote:
>>> On Sun, Mar 21, 2004 at 06:16:34PM +0100, Jindrich Makovicka wrote:
>>>> Tobias Diedrich wrote:
>>>>> This updated patch now includes Video Property Header support (Among
>>>>> other things stores aspect ratio).
>>>>> However neither Window Media Player nor Media Player Classic seem to
>>>>> support it...
>>>> Nice, now the aspect encoding goes totally bogus :)
>>>> When I encode a mpeg2 dvb capture (4:3) and crop to 16:9, the
>>>> resulting file is has still 4:3 in the header, and stuff like
>>>> -noaspect or specifying aspect in -lavcopts doesn't make any change.
>>> Arrgg, 1000000000l!!!!! You have to get the aspect from the END of the
>>> video filter chain, not from movie_aspect!
>>> Someone remove or fix this brain damage asap!
>> I vote for the removal, as lavc can encode/decode its own aspect
>> headers, and this patch effectively disables already available method
>> of specifying the aspect ratio. I already have a working patch with
>> the bogus parts commented out with #ifdefs, should I commit?
> Please, don't remove it: if fixed it's useful. There are players that
> understand the AR header in avi
> but not  in the video stream.
> Maybe enabling it with a suboption is the right way to go.
>        Nico
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
-------------- next part --------------
A non-text attachment was scrubbed...
Name: odml_aspect.patch
Type: application/octet-stream
Size: 6652 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040322/9d1a85fc/attachment.obj>

More information about the MPlayer-dev-eng mailing list