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

Nico nsabbi at tiscali.it
Sun Jan 25 16:05:53 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, test this patch rather than the previous one in the thread: it 
handles correctly the special A/R cases (4/3 and 16/9);
xvid doesn't seem to know anything of "21:17" (never seen myself) and  
"2.21:1".

Thanks,
       Nico
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ve_xvid4_2.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040125/379b8772/attachment.asc>


More information about the MPlayer-dev-eng mailing list