[MPlayer-dev-eng] [PATCH] manpage patch for vstrict and MJPEG

[Oded Shimon]

On Monday 21 February 2005 07:49, Oded Shimon wrote:
> On Monday 21 February 2005 07:44, D Richard Felker III wrote:
> > On Mon, Feb 21, 2005 at 06:55:25AM +0200, Oded Shimon wrote:
> > > MJPEG encoding impossible without 'vstrict=-1', man page does not
> > > mention this.
> > > http://bugzilla.mplayerhq.hu/show_bug.cgi?id=236
> > >
> > > - ods15
> >
> > the text you wrote is wrong -- the 20% smaller thing only applies to
> > huffyuv, and i think it's outdated for that even, since the
> > nonstandard mode was moved to a different codec named ffhuff or
> > something. anyway there should be a proper explanation of why mjpeg
> > requires vstrict=-1, since the reasons are different than the old
> > huffyuv reason... and perhaps vstrict=-1 should be the default these
> > days since lavc is so anal..?
>
> I dunno about huffyuv, but I think it should be default for MJPEG since you
> can't encode to it anyway.
> Can't fix the manpage till someone is sure what it does and a decision is
> made about vstrict... I'll check if 'ffhuff' exists.

OK, just checked, it's called "ffvhuff", it's not mentioned in the man page, 
probably because it's "under development". It refuses to work without 
'vstrict=-1', and says this loudly. The codec "ffv1", which is mentioned in 
the manpage, does also this exact same thing. If the same applies to the 
MJPEG codec, I think it should do the same. Nothing in the error output makes 
you consider 'vstrict', especially since it worked fine in past versions. 
Also, when re-encoding from an mjpeg file to mjpeg (I don't see why anyone 
would want to do this, but still) - MEncoder doesn't fail, but skips EVERY 
frame until eventually it crashes with segfault at end of encode. If MJPEG 
really isn't supposed to work without vstrict, it should check for vstrict 
itself and fail immediately, like all the others.

I think the man page should say something like:

vcodec=
 ...
 mjpeg (see also vstrict)
 ffvhuiff (see also vstrict)
 ffv1 (see also vstrict)
 ...

vstrict=
 ...
 -1 allows some non-standard codecs to work. (mjpeg, ffvhuff, ffv1)

- ods15