[MPlayer-dev-eng] Re: Endpos for Mplayer.

[Alban Bedel]

Hi Konstantinos Karydas,

on Tue, 21 Oct 2003 09:11:07 +0300 you wrote:

>  
> +
> +// --> kkar added this function from mencoder!
> +static int parse_end_at(m_option_t *conf, const char* param)
> +{
> +
These callback based options shouldn't be used anymore. You should
rather add a new option type in m_option.[ch] if needed. Also mplayer
shouldn't accept end_pos in size unit as it make no sense.

> +// --> kkar part added from mencoder
Who added/removed what is avaible from cvs log. No needed to start
cluttering the code with useless comments like that.

> +
> +  /*if (stop_to_sec) {
> +    int a,b; float d;
> +    
> +    if (sscanf(stop_to_sec, "%d:%d:%f", &a,&b,&d)==3)
> +	rel_stop_secs += 3600*a +60*b +d ;
> +    else if (sscanf(stop_to_sec, "%d:%f", &a, &d)==2)
> +	rel_stop_secs += 60*a +d;
> +    else if (sscanf(stop_to_sec, "%f", &d)==1)
> +	rel_stop_secs += d;
> +
> +     stop_to_sec = NULL;
> +  }	*/
> +
What is that for ? And where is rel_stop_secs declared ?
	Albeu
-- 

Everything is controlled by a small evil group
to which, unfortunately, no one we know belongs.