[MPlayer-DOCS] CVS: main/DOCS/tech mpcf.txt,1.49,1.50

[Michael Niedermayer]

Hi

On Wednesday 21 April 2004 20:43, Richard Felker CVS wrote:
> CVS change done by Richard Felker CVS
>
> Update of /cvsroot/mplayer/main/DOCS/tech
> In directory mail:/var2/tmp/cvs-serv32395/DOCS/tech
>
> Modified Files:
> 	mpcf.txt
> Log Message:
> remove bad requirement
>
> Index: mpcf.txt
> ===================================================================
> RCS file: /cvsroot/mplayer/main/DOCS/tech/mpcf.txt,v
> retrieving revision 1.49
> retrieving revision 1.50
> diff -u -r1.49 -r1.50
> --- mpcf.txt	20 Apr 2004 23:00:28 -0000	1.49
> +++ mpcf.txt	21 Apr 2004 18:43:01 -0000	1.50
> @@ -281,7 +281,6 @@
>  	if the fixed_fps is 1
>  	time_base_denom MUST not be 0
>  	time_base_nom and time_base_denom MUST be relative prime
> -	time_base_nom MUST be < 2^16
i agree, but there should be some limit, or otherwise we will have to deal 
with 64bit time_base_nom which is _very_ difficult to work with as it 
requires 128bit intermediates for every operation unless we just convert it 
to float ...
how could 64bit time_base_nom be created? easy:
den= UINT32_MAX;
nom= (uint64_t)(float_fps/den + 0.5);

while this is inaccurate and ugly, its probably quite similar to how many 
developers would implement it IMHO

so IMHO we need a limit for all fractions or we can add some recommandition 
about which arbitrary precision lib should be used to demux nut ;)

[...]
-- 
Michael
level[i]= get_vlc(); i+=get_vlc();		(violates patent EP0266049)
median(mv[y-1][x], mv[y][x-1], mv[y+1][x+1]);	(violates patent #5,905,535)
buf[i]= qp - buf[i-1];				(violates patent #?)
for more examples, see http://mplayerhq.hu/~michael/patent.html
stop it, see http://petition.eurolinux.org & http://petition.ffii.org/eubsa/en