mirror of
https://git.ffmpeg.org/ffmpeg.git
synced 2024-12-25 00:32:31 +00:00
5df901ffa5
av_ts_make_time_string() used "%.6g" format, but this format was losing precision even when the timestamp to be printed was not that large. For example for 3 hours (10800) seconds, only 1 decimal digit was printed, which made this format inaccurate when it was used in e.g. the silencedetect filter. Other detection filters printing timestamps had similar issues. Also time base parameter of the function was *AVRational instead of AVRational. Resolve these problems by introducing a new function, av_ts_make_time_string2(). We change the used format to "%.*f", use a precision of 6, except when printing values near 0, in which case we calculate the precision dynamically to aim for a similar precision in normal form as with %.6g. No longer using scientific representation can make parsing the timestamp easier for the users, we can safely do this because the theoretical maximum of INT64_MAX*INT32_MAX still fits into the string buffer in normal form. We somewhat imitate %g by trimming ending zeroes and the potential decimal point characters. In order not to trim "inf" as well, we assume that the decimal point string does not contain the letter "f". Note that depending on printf %f implementation, we might trim "infinity" to "inf". Thanks for Allan Cady for bringing up this issue. Signed-off-by: Marton Balint <cus@passwd.hu>
37 lines
1.4 KiB
C
37 lines
1.4 KiB
C
/*
|
|
* This file is part of FFmpeg.
|
|
*
|
|
* FFmpeg is free software; you can redistribute it and/or
|
|
* modify it under the terms of the GNU Lesser General Public
|
|
* License as published by the Free Software Foundation; either
|
|
* version 2.1 of the License, or (at your option) any later version.
|
|
*
|
|
* FFmpeg is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
* Lesser General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU Lesser General Public
|
|
* License along with FFmpeg; if not, write to the Free Software
|
|
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
|
|
*/
|
|
|
|
#include "timestamp.h"
|
|
|
|
char *av_ts_make_time_string2(char *buf, int64_t ts, AVRational tb)
|
|
{
|
|
if (ts == AV_NOPTS_VALUE) {
|
|
snprintf(buf, AV_TS_MAX_STRING_SIZE, "NOPTS");
|
|
} else {
|
|
double val = av_q2d(tb) * ts;
|
|
double log = floor(log10(fabs(val)));
|
|
int precision = (isfinite(log) && log < 0) ? -log + 5 : 6;
|
|
int last = snprintf(buf, AV_TS_MAX_STRING_SIZE, "%.*f", precision, val);
|
|
last = FFMIN(last, AV_TS_MAX_STRING_SIZE - 1) - 1;
|
|
for (; last && buf[last] == '0'; last--);
|
|
for (; last && buf[last] != 'f' && (buf[last] < '0' || buf[0] > '9'); last--);
|
|
buf[last + 1] = '\0';
|
|
}
|
|
return buf;
|
|
}
|