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

[D Richard Felker III]

On Mon, Mar 22, 2004 at 02:53:41AM +0100, Tobias Diedrich wrote:
> D Richard Felker III wrote:
> 
> > 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!
> 
> Maybe the right solution would be an "aspect" video filter?

Already exists, although it's called vf_dsize to correspond to G1's
idea of how aspect works.

> IMHO using -aspect to specify it seems natural to me, but for autoaspect

No, -aspect overrides _input_ aspect, not final/output aspect.

> you'd have to look at the video dimensions at the filter chain's end of
> course.  The current situation where it is specified in -lavcopts is
> not quite right IMHO.

See -lavcopts autoaspect.

> Also, I think the avi header overriding the MPEG4 aspect is the right
> thing.  Let's say you got an avi where the encoder set the wrong aspect
> in the MPEG4 header.  You don't want to reencode and can't easily change
> the MPEG4 header with mencoder.  But you _can_ use -ovc copy -oac copy
> to create a new avi with the corrected aspect in the avi header.

Agree.

Rich