[Mplayer-cvslog] CVS: main/libmpcodecs vd_ffmpeg.c,1.115,1.116

[D Richard Felker III]

On Thu, Jan 01, 2004 at 07:51:46PM +0100, Michael Niedermayer wrote:
> Hi
> 
> On Thursday 01 January 2004 19:18, D Richard Felker III wrote:
> > On Thu, Jan 01, 2004 at 04:56:03PM +0100, Michael Niedermayer CVS wrote:
> > > Update of /cvsroot/mplayer/main/libmpcodecs
> > > In directory mail:/tmp/cvs-serv24429
> > >
> > > Modified Files:
> > > 	vd_ffmpeg.c
> > > Log Message:
> > > disable slices & dr1 if motion vector vissualization is used, still
> > > doesnt work though, maybe i should reverse the last mv vis patch ...
> >
> > IMO yes. MV vis should be a separate filter using the exported mv's.
> yes it should be, but that would make it more difficult to use for everyone 
> who uses libavcodec in their applications ...

Well the same could be said about software-scaling (hey, we _need_ a
scaler to convert 480x480 svcd to square pixels!) or postprocessing or
denoising or inverse telecine or ...

My point is that yes, there are a lot of features that would be
beneficial for all players to have, but putting them all in lavc
doesn't make sense. It's just bloat....and when you step over the
bounds of just bloat and into the bounds of "buggy bloat" (like this
mv visualization stuff), it's bad IMHO...

Rich