[FFmpeg-cvslog] avutil/rational: Document what is to be expected from av_d2q() of doubles representing rational numbers

Michael Niedermayer git at videolan.org
Sun Apr 14 23:59:48 EEST 2024


ffmpeg | branch: release/3.4 | Michael Niedermayer <michael at niedermayer.cc> | Wed Jan 31 02:37:57 2024 +0100| [c3cbefbdffd7a461f6ba36104fb51aff9f6c1dab] | committer: Michael Niedermayer

avutil/rational: Document what is to be expected from av_d2q() of doubles representing rational numbers

Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
(cherry picked from commit f465badb062c8023bc245f4878e7a6a082afc416)
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=c3cbefbdffd7a461f6ba36104fb51aff9f6c1dab
---

 libavutil/rational.h | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/libavutil/rational.h b/libavutil/rational.h
index 5c6b67b4e9..3ab7f59bd7 100644
--- a/libavutil/rational.h
+++ b/libavutil/rational.h
@@ -168,6 +168,10 @@ static av_always_inline AVRational av_inv_q(AVRational q)
  * In case of infinity, the returned value is expressed as `{1, 0}` or
  * `{-1, 0}` depending on the sign.
  *
+ * In general rational numbers with |num| <= 1<<26 && |den| <= 1<<26
+ * can be recovered exactly from their double representation.
+ * (no exceptions were found within 1B random ones)
+ *
  * @param d   `double` to convert
  * @param max Maximum allowed numerator and denominator
  * @return `d` in AVRational form



More information about the ffmpeg-cvslog mailing list