[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