diff options
| author | Marton Balint <cus@passwd.hu> | 2024-11-10 14:10:03 +0100 |
|---|---|---|
| committer | Marton Balint <cus@passwd.hu> | 2024-11-16 20:58:49 +0100 |
| commit | d884606ce0f46161ada47568b3d7336d697c1061 (patch) | |
| tree | a123fe7043ad63c57b17a0484321ef9b748b5cb7 | |
| parent | 25efe34e6f1353b50c147098355250e2f401ca33 (diff) | |
| download | ffmpeg-d884606ce0f46161ada47568b3d7336d697c1061.tar.gz | |
avutil/rational: never return greater num/den than the maximum allowed in av_d2q
This reverts 887d74c47efa70d7d1513e9492d41cf7f88dee0b, because it ignores the
maximum allowed numerator/denominator. Even if the result was rounded to zero
or infinity, it should not be cause to ignore the maximum allowed number, this
"feature" is unintuitive and undocumented.
Signed-off-by: Marton Balint <cus@passwd.hu>
| -rw-r--r-- | libavutil/rational.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/libavutil/rational.c b/libavutil/rational.c index 329fbf3302..1b7c6350e9 100644 --- a/libavutil/rational.c +++ b/libavutil/rational.c @@ -118,8 +118,6 @@ AVRational av_d2q(double d, int max) // (int64_t)rint() and llrint() do not work with gcc on ia64 and sparc64, // see Ticket2713 for affected gcc/glibc versions av_reduce(&a.num, &a.den, floor(d * den + 0.5), den, max); - if ((!a.num || !a.den) && d && max>0 && max<INT_MAX) - av_reduce(&a.num, &a.den, floor(d * den + 0.5), den, INT_MAX); return a; } |