[MPlayer-cvslog] CVS: main/libmpcodecs vf_scale.c,1.62,1.63

[Michael Niedermayer]

Hi

On Fri, Feb 24, 2006 at 07:16:02PM +0200, Oded Shimon wrote:
> On Fri, Feb 24, 2006 at 04:51:27PM +0100, Diego Biurrun wrote:
> > On Fri, Feb 24, 2006 at 04:02:03PM +0200, Oded Shimon wrote:
> > > On Fri, Feb 24, 2006 at 02:56:21PM +0100, Diego Biurrun wrote:
> > > > On Wed, Feb 22, 2006 at 06:40:20PM +0200, Oded Shimon wrote:
> > > > > On Wed, Feb 22, 2006 at 12:44:34PM +0100, Diego Biurrun wrote:
> > > > > > 
> > > > > >   <noup>
> > > > > >     Disallow upscaling past the original dimensions.
> > > > > >       0: off (default)
> > > > > >       1: on
> > > > > 
> > > > > You should probably mention that it downscales dimentions seprately, and I 
> > > > > think, this can even break aspect ratio... For prescaled mpeg video etc.
> > > > 
> > > > Elaborate..
> > 
> > [...]
> 
> This no longer applies because Michael changed the patch... BTW Michael you 
> have a habit of adding/changing MPlayer behavior and keeping it hidden so 
> documentors can't keep up :) (-lavcopts vglobal  comes to mind)

indeed, these where unintended though, i should be more carefull with my
cvslog messages ...

OTOH, not mentioning everything in the docs might increase the number of
people who RTFS which again increases the average knowledge of mplayer and
improves their programming skills (not to mention their cryptoanalytical
skills for deciphering some of mplayers code...) that all will lead to
more patches and developers ...
predicting the effects of doing or not doing something aint easy ...

[...]

-- 
Michael