svn commit: r312119 - head/sys/kern
Ngie Cooper
ngie at FreeBSD.org
Sat Jan 14 05:06:15 UTC 2017
Author: ngie
Date: Sat Jan 14 05:06:14 2017
New Revision: 312119
URL: https://svnweb.freebsd.org/changeset/base/312119
Log:
encode_long, encode_timeval: mechanically replace `exp` with `exponent`
This helps fix a -Wshadow issue with exp(3) with tests/sys/acct/acct_test,
which include math.h, which in turn defines exp(3)
MFC after: 2 weeks
Tested with: clang, gcc 4.2.1, gcc 4.9
Sponsored by: Dell EMC Isilon
Modified:
head/sys/kern/kern_acct.c
Modified: head/sys/kern/kern_acct.c
==============================================================================
--- head/sys/kern/kern_acct.c Sat Jan 14 05:02:53 2017 (r312118)
+++ head/sys/kern/kern_acct.c Sat Jan 14 05:06:14 2017 (r312119)
@@ -469,8 +469,8 @@ static uint32_t
encode_timeval(struct timeval tv)
{
int log2_s;
- int val, exp; /* Unnormalized value and exponent */
- int norm_exp; /* Normalized exponent */
+ int val, exponent; /* Unnormalized value and exponent */
+ int norm_exponent; /* Normalized exponent */
int shift;
/*
@@ -481,7 +481,7 @@ encode_timeval(struct timeval tv)
if (tv.tv_sec == 0) {
if (tv.tv_usec == 0)
return (0);
- exp = 0;
+ exponent = 0;
val = tv.tv_usec;
} else {
/*
@@ -490,24 +490,24 @@ encode_timeval(struct timeval tv)
*/
log2_s = fls(tv.tv_sec) - 1;
if (log2_s + LOG2_1M < CALC_BITS) {
- exp = 0;
+ exponent = 0;
val = 1000000 * tv.tv_sec + tv.tv_usec;
} else {
- exp = log2_s + LOG2_1M - CALC_BITS;
+ exponent = log2_s + LOG2_1M - CALC_BITS;
val = (unsigned int)(((uint64_t)1000000 * tv.tv_sec +
- tv.tv_usec) >> exp);
+ tv.tv_usec) >> exponent);
}
}
/* Now normalize and pack the value into an IEEE-754 float. */
- norm_exp = fls(val) - 1;
- shift = FLT_MANT_DIG - norm_exp - 1;
+ norm_exponent = fls(val) - 1;
+ shift = FLT_MANT_DIG - norm_exponent - 1;
#ifdef ACCT_DEBUG
printf("val=%d exp=%d shift=%d log2(val)=%d\n",
- val, exp, shift, norm_exp);
- printf("exp=%x mant=%x\n", FLT_MAX_EXP - 1 + exp + norm_exp,
+ val, exponent, shift, norm_exponent);
+ printf("exp=%x mant=%x\n", FLT_MAX_EXP - 1 + exponent + norm_exponent,
((shift > 0 ? (val << shift) : (val >> -shift)) & MANT_MASK));
#endif
- return (((FLT_MAX_EXP - 1 + exp + norm_exp) << (FLT_MANT_DIG - 1)) |
+ return (((FLT_MAX_EXP - 1 + exponent + norm_exponent) << (FLT_MANT_DIG - 1)) |
((shift > 0 ? val << shift : val >> -shift) & MANT_MASK));
}
@@ -518,7 +518,7 @@ encode_timeval(struct timeval tv)
static uint32_t
encode_long(long val)
{
- int norm_exp; /* Normalized exponent */
+ int norm_exponent; /* Normalized exponent */
int shift;
if (val == 0)
@@ -529,15 +529,15 @@ encode_long(long val)
val);
val = LONG_MAX;
}
- norm_exp = fls(val) - 1;
- shift = FLT_MANT_DIG - norm_exp - 1;
+ norm_exponent = fls(val) - 1;
+ shift = FLT_MANT_DIG - norm_exponent - 1;
#ifdef ACCT_DEBUG
printf("val=%d shift=%d log2(val)=%d\n",
- val, shift, norm_exp);
- printf("exp=%x mant=%x\n", FLT_MAX_EXP - 1 + exp + norm_exp,
+ val, shift, norm_exponent);
+ printf("exp=%x mant=%x\n", FLT_MAX_EXP - 1 + exp + norm_exponent,
((shift > 0 ? (val << shift) : (val >> -shift)) & MANT_MASK));
#endif
- return (((FLT_MAX_EXP - 1 + norm_exp) << (FLT_MANT_DIG - 1)) |
+ return (((FLT_MAX_EXP - 1 + norm_exponent) << (FLT_MANT_DIG - 1)) |
((shift > 0 ? val << shift : val >> -shift) & MANT_MASK));
}
More information about the svn-src-all
mailing list