[MPlayer-dev-eng] Re: [PATCH]time stamps in spudec.c

[Tobias Diedrich]

Arne Driescher wrote:

> Explenation:
> A command like start_display/stop_display/force_display for a subtitle gets 
> an argument that specifies the delay when the command should be
> executed. (E.g. for start_display this delay is most often 0 so that
> subtitle start time is equal to pts. However stop_display is normally
> the same pts but the delay is greater than 0.)
> The time offset is based on the 90KHz clock divided by 1024.
> In the current version the delay is calculated as ticks in the 90KHz
> clock and not in pts100 units as it should be.

This looks wrong to me.
The unit of pts100 is clock ticks in the 90KHz clock.
If the time offset is based on 90KHz/1024 then
date = get_be16(this->packet + start_off) * 1024
should be perfectly fine.

-- 
/* Tobias */     int main(int a,char **b)    /* PGP: http://9ac7e0bc.2ya.com */
{char i,j,t,*r=*++b;srand(getpid());for(;*r;){for(;*r&&!isalpha(*r);r++);for(i=
1;isalpha(r[i+1]);i++){j=rand()%i;t=r[i];r[i]=r[++j];r[j]=t;}r+=i+1;}puts(*b);}