[MPlayer-dev-eng] [PATCH/PROPOSAL] precise timestamps for encoded audio

Nico Sabbi nsabbi at tiscali.it
Mon Apr 25 20:59:33 CEST 2005


Alexander Strasser wrote:

>Am I blind or did you oversee the if-statement in between?
>The order can make a difference at runtime, see:
>
>/* the above lines may well change ret_size in some cases */
> 	if(r2 > 0)
> 		ret_size = r2;
>/* and here ret_size is output through mp_msg */
>	mp_msg(MSGT_MENCODER, MSGL_V, "\nSIZE: %d, max: %d, r2: %d\n", 
>ret_size, max_size, r2);
>
>Maybe I am just blind,
>  Alex (beastd)
>  
>

no, you are right.
Anyway Rich and I discussed this patch on irc, and we ended up 
concluding that
it's useless, because it doesn't fix the main reason why mencoder desyncs.
After all this patch makes works like other parts of mencoder.

    Nico




More information about the MPlayer-dev-eng mailing list