[FFmpeg-devel] [PATCH 04/17] Use INFINITY and NAN macros instead of 1/0 and 0/0

[Ronald S. Bultje]

Hi,

On Tue, Jan 18, 2011 at 2:32 PM, Mans Rullgard <mans at mansr.com> wrote:
> ---
> ?cmdutils.c ? ? ? ? ? ? ? ? ? ? | ? ?2 +-
> ?libavutil/intfloat_readwrite.c | ? ?8 ++++----
> ?2 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/cmdutils.c b/cmdutils.c
> index 1e7211a..a77e6e6 100644
> --- a/cmdutils.c
> +++ b/cmdutils.c
> @@ -198,7 +198,7 @@ unknown_opt:
> ? ? ? ? ? ? } else if (po->flags & OPT_INT64) {
> ? ? ? ? ? ? ? ? *po->u.int64_arg = parse_number_or_die(opt, arg, OPT_INT64, INT64_MIN, INT64_MAX);
> ? ? ? ? ? ? } else if (po->flags & OPT_FLOAT) {
> - ? ? ? ? ? ? ? ?*po->u.float_arg = parse_number_or_die(opt, arg, OPT_FLOAT, -1.0/0.0, 1.0/0.0);
> + ? ? ? ? ? ? ? ?*po->u.float_arg = parse_number_or_die(opt, arg, OPT_FLOAT, -INFINITY, INFINITY);
> ? ? ? ? ? ? } else if (po->flags & OPT_FUNC2) {
> ? ? ? ? ? ? ? ? if (po->u.func2_arg(opt, arg) < 0) {
> ? ? ? ? ? ? ? ? ? ? fprintf(stderr, "%s: failed to set value '%s' for option '%s'\n", argv[0], arg, opt);
> diff --git a/libavutil/intfloat_readwrite.c b/libavutil/intfloat_readwrite.c
> index 79fe186..d9b3752 100644
> --- a/libavutil/intfloat_readwrite.c
> +++ b/libavutil/intfloat_readwrite.c
> @@ -31,13 +31,13 @@
>
> ?double av_int2dbl(int64_t v){
> ? ? if(v+v > 0xFFEULL<<52)
> - ? ? ? ?return 0.0/0.0;
> + ? ? ? ?return NAN;
> ? ? return ldexp(((v&((1LL<<52)-1)) + (1LL<<52)) * (v>>63|1), (v>>52&0x7FF)-1075);
> ?}
>
> ?float av_int2flt(int32_t v){
> ? ? if(v+v > 0xFF000000U)
> - ? ? ? ?return 0.0/0.0;
> + ? ? ? ?return NAN;
> ? ? return ldexp(((v&0x7FFFFF) + (1<<23)) * (v>>31|1), (v>>23&0xFF)-150);
> ?}
>
> @@ -49,7 +49,7 @@ double av_ext2dbl(const AVExtFloat ext){
> ? ? ? ? m = (m<<8) + ext.mantissa[i];
> ? ? e = (((int)ext.exponent[0]&0x7f)<<8) | ext.exponent[1];
> ? ? if (e == 0x7fff && m)
> - ? ? ? ?return 0.0/0.0;
> + ? ? ? ?return NAN;
> ? ? e -= 16383 + 63; ? ? ? ?/* In IEEE 80 bits, the whole (i.e. 1.xxxx)
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?* mantissa bit is written as opposed to the
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?* single and double precision formats. */
> @@ -88,7 +88,7 @@ AVExtFloat av_dbl2ext(double d){
> ? ? ? ? ? ? ext.mantissa[i] = m>>(56-(i<<3));
> ? ? } else if (f != 0.0) {
> ? ? ? ? ext.exponent[0] = 0x7f; ext.exponent[1] = 0xff;
> - ? ? ? ?if (f != 1/0.0)
> + ? ? ? ?if (f != INFINITY)
> ? ? ? ? ? ? ext.mantissa[0] = ~0;
> ? ? }
> ? ? if (d < 0)
> --
> 1.7.3.5

OK.

Ronald