[MPlayer-dev-eng] patch for minor typos

Mike Castle dalgoda+mplayer at gmail.com
Fri Jun 10 17:29:14 CEST 2011


On Fri, Jun 10, 2011 at 1:31 AM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> Applied without the first hunk.

What was wrong with the first hunk?   Not apply correct or disagreement?

-    // timestamp: HH:MM:SS.mmm, filepos: 0nnnnnnnnn
+    // timestamp: HH:MM:SS:mmm, filepos: 0nnnnnnnnn

That is a correct change, as far as I can tell.  The IDX files follow
that format:

timestamp: 00:00:01:401, filepos: 000000000
timestamp: 00:00:03:370, filepos: 000000800


and that's what the code below actually parses:

 724     if (*p != ':')
 725         return -1;
 726     line = ++p;
 727     while (isdigit(*p))
 728         ++p;
 729     if (p - line != 3)
 730         return -1;
 731     ms = atoi(line);


mrc


More information about the MPlayer-dev-eng mailing list