[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