[FFmpeg-devel] [PATCH] avutil/timestamp: avoid using INFINITY for log10 result in av_ts_make_time_string2

[Rémi Denis-Courmont]

Le 27 août 2024 10:00:08 GMT+03:00, Marton Balint <cus at passwd.hu> a écrit :

>IMHO the patch itself is harmless (simple, easy to follow), and if it helps somebody (no matter how unupported its use-case is), then why the hell not. But I can understand your point of view, that we should not bother with it, if it is unsupported in the first place, so I will just drop this, did not feel strongly about it anyway.

Can't we just fail configure or compilation hard if __FINITE_MATH_ONLY__ is true?

If MPV or whichever downstream wants to get this flag working, it's on them to do the hard work (and it's a lot harder than this patch).