[MPlayer-dev-eng] [PATCH] ve_xvid4.c catching up with xvid 1.0.0 beta3

Nico nsabbi at tiscali.it
Sun Jan 25 15:12:06 CET 2004


Ivan Kalvachev wrote:

>Sorry, but the patch have been rejected!
>    xvidenc_packed,
>    xvidenc_chromame,
>    xvidenc_vhq
>these variables are flags, and mencoder configure system only can set them.
>Setting them by default will make it impossible to turn them off!
>
>
>I can accept these if they are properly documented!
>    xvidenc_max_bframes - I think that this enables B frames by default
>    xvidenc_vbr_overflow_control_strength
>    xvidenc_vbr_max_overflow_improvement
>    xvidenc_vbr_max_overflow_degradetion -hmm
>    xvidenc_vbr_container_frame_overhead - mencoder does it's own bitrate
>       calculations, pleace make sure you won't fool them.
>
>New options (for -doc):
>
>chroma_opt - enable chroma optimization pre-filter.
>  Do you think that name chroma_filter, give better impression of the
>  function usage?
>
>turbo - use faster (and less accurate) ME functions
>dar_aspect - I don't like the implementation.
>  There are 2 parameters 'par' and 'aspect' that do one and same thing!
>  merge them. Use ve_ffmpeg.c as example how to handle aspect
>  caclulaction! I would like autoaspect function :)
>  Don't forget that the other aspect's can take not only float,
>  but also "16:9" "21:17" "2.21:1". Happy hacking.
>
>  I definitly don't like mixing float and integer calculation.
>  dar_x,dar_y are completly useless! and decrease accuraty, and I think that
>  the implementation is wrong! I just look at these lines:
>		dar_x = (int)(xvidenc_dar_aspect * 100.0f);
>-->		dar_y = 100;
>
>		frame->par = XVID_PAR_EXT;
>-->		frame->par_width = (100 * mod->mux->bih->biHeight) / dar_y;
>  yep, width=height!, and height changed to keep the aspect.
>  It may not be bug, but it is ugly!
>
>Best Regards
>  Ivan Kalvachev
> iive
>
>p.s.
>I may strip the patch and make "par" accept float values.
>But not anytime soon. Sorry
>
>
>_______________________________________________
>MPlayer-dev-eng mailing list
>MPlayer-dev-eng at mplayerhq.hu
>http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
>
>  
>

Please, review this patch.
I implemented the aspect and autoaspect (DAR) settings using the code in 
ve_lavc.c and libavcodec and reset
to 0 some flags previously initialized to 1 (hq_ac, chroma_opt, chroma_me).

It's a pity there is no way to set a variable to 0 (flag unset) using 
the CONF_TYPE_FLAG construct.


    Nico
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ve_xvid4.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040125/5813e4ee/attachment.txt>


More information about the MPlayer-dev-eng mailing list