aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2023-10-10 19:52:33 +0200
committerMichael Niedermayer <michael@niedermayer.cc>2024-03-26 01:19:16 +0100
commite83e8d443b5b86aabf17d1cfb7fba9abf15e24fd (patch)
tree4376a6da6b6bfc942de708adec24713e95b3798e
parent5ebcca4e08bf6063b227a23d8e87e56bba52ff18 (diff)
downloadffmpeg-e83e8d443b5b86aabf17d1cfb7fba9abf15e24fd.tar.gz
avformat/jacosubdec: clarify code
add comments, rename variables and indent things differently Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r--libavformat/jacosubdec.c18
1 files changed, 9 insertions, 9 deletions
diff --git a/libavformat/jacosubdec.c b/libavformat/jacosubdec.c
index 8b0aa312fc..447397b6a7 100644
--- a/libavformat/jacosubdec.c
+++ b/libavformat/jacosubdec.c
@@ -128,28 +128,28 @@ shift_and_ret:
static int get_shift(unsigned timeres, const char *buf)
{
int sign = 1;
- int a = 0, b = 0, c = 0, d = 0;
+ int h = 0, m = 0, s = 0, d = 0;
int64_t ret;
#define SSEP "%*1[.:]"
- int n = sscanf(buf, "%d"SSEP"%d"SSEP"%d"SSEP"%d", &a, &b, &c, &d);
+ int n = sscanf(buf, "%d"SSEP"%d"SSEP"%d"SSEP"%d", &h, &m, &s, &d);
#undef SSEP
- if (a == INT_MIN)
+ if (h == INT_MIN)
return 0;
- if (*buf == '-' || a < 0) {
+ if (*buf == '-' || h < 0) {
sign = -1;
- a = FFABS(a);
+ h = FFABS(h);
}
ret = 0;
switch (n) {
- case 1: a = 0;
- case 2: c = b; b = a; a = 0;
- case 3: d = c; c = b; b = a; a = 0;
+ case 1: h = 0; //clear all in case of a single parameter
+ case 2: s = m; m = h; h = 0; //shift into second subsecondd
+ case 3: d = s; s = m; m = h; h = 0; //shift into minute second subsecond
}
- ret = (int64_t)a*3600 + (int64_t)b*60 + c;
+ ret = (int64_t)h*3600 + (int64_t)m*60 + s;
if (FFABS(ret) > (INT64_MAX - FFABS(d)) / timeres)
return 0;
ret = sign * (ret * timeres + d);