OpenE2K/qemu-e2k
13
4
Fork 0
Code Issues 4 Pull Requests Projects 2 Releases Activity

libdecnumber: Fix warnings from smatch (missing static, boolean operations)

Browse Source 
Signed-off-by: Stefan Weil <sw@weilnetz.de>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
This commit is contained in:
Stefan Weil 2014-08-20 11:02:10 +02:00 committed by Michael Tokarev
parent 680dfde919
commit d072cdf3ba
1 changed files with 11 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
libdecnumber/decNumber.c
View File

@ -5275,8 +5275,8 @@ static decNumber * decMultiplyOp(decNumber *res, const decNumber *lhs,
/* 4. The working precisions for the static buffers are twice the */ /* 4. The working precisions for the static buffers are twice the */
/* obvious size to allow for calls from decNumberPower. */ /* obvious size to allow for calls from decNumberPower. */
/* ------------------------------------------------------------------ */ /* ------------------------------------------------------------------ */
decNumber * decExpOp(decNumber *res, const decNumber *rhs, static decNumber *decExpOp(decNumber *res, const decNumber *rhs,
decContext *set, uInt *status) { decContext *set, uInt *status) {
uInt ignore=0; /* working status */ uInt ignore=0; /* working status */
Int h; /* adjusted exponent for 0.xxxx */ Int h; /* adjusted exponent for 0.xxxx */
Int p; /* working precision */ Int p; /* working precision */
@ -5563,7 +5563,8 @@ decNumber * decExpOp(decNumber *res, const decNumber *rhs,
/* where x is truncated (NB) into the range 10 through 99, */ /* where x is truncated (NB) into the range 10 through 99, */
/* and then c = k>>2 and e = k&3. */ /* and then c = k>>2 and e = k&3. */
/* ------------------------------------------------------------------ */ /* ------------------------------------------------------------------ */
const uShort LNnn[90]={9016, 8652, 8316, 8008, 7724, 7456, 7208, static const uShort LNnn[90] = {
9016, 8652, 8316, 8008, 7724, 7456, 7208,
6972, 6748, 6540, 6340, 6148, 5968, 5792, 5628, 5464, 5312, 6972, 6748, 6540, 6340, 6148, 5968, 5792, 5628, 5464, 5312,
5164, 5020, 4884, 4748, 4620, 4496, 4376, 4256, 4144, 4032, 5164, 5020, 4884, 4748, 4620, 4496, 4376, 4256, 4144, 4032,
39233, 38181, 37157, 36157, 35181, 34229, 33297, 32389, 31501, 30629, 39233, 38181, 37157, 36157, 35181, 34229, 33297, 32389, 31501, 30629,
@ -5635,8 +5636,8 @@ const uShort LNnn[90]={9016, 8652, 8316, 8008, 7724, 7456, 7208,
/* 5. The static buffers are larger than might be expected to allow */ /* 5. The static buffers are larger than might be expected to allow */
/* for calls from decNumberPower. */ /* for calls from decNumberPower. */
/* ------------------------------------------------------------------ */ /* ------------------------------------------------------------------ */
decNumber * decLnOp(decNumber *res, const decNumber *rhs, static decNumber *decLnOp(decNumber *res, const decNumber *rhs,
decContext *set, uInt *status) { decContext *set, uInt *status) {
uInt ignore=0; /* working status accumulator */ uInt ignore=0; /* working status accumulator */
uInt needbytes; /* for space calculations */ uInt needbytes; /* for space calculations */
Int residue; /* rounding residue */ Int residue; /* rounding residue */
@ -6052,9 +6053,9 @@ static decNumber * decQuantizeOp(decNumber *res, const decNumber *lhs,
/* The emphasis here is on speed for common cases, and avoiding */ /* The emphasis here is on speed for common cases, and avoiding */
/* coefficient comparison if possible. */ /* coefficient comparison if possible. */
/* ------------------------------------------------------------------ */ /* ------------------------------------------------------------------ */
decNumber * decCompareOp(decNumber *res, const decNumber *lhs, static decNumber *decCompareOp(decNumber *res, const decNumber *lhs,
const decNumber *rhs, decContext *set, const decNumber *rhs, decContext *set,
Flag op, uInt *status) { Flag op, uInt *status) {
#if DECSUBSET #if DECSUBSET
decNumber *alloclhs=NULL; /* non-NULL if rounded lhs allocated */ decNumber *alloclhs=NULL; /* non-NULL if rounded lhs allocated */
decNumber *allocrhs=NULL; /* .., rhs */ decNumber *allocrhs=NULL; /* .., rhs */
@ -6086,11 +6087,11 @@ decNumber * decCompareOp(decNumber *res, const decNumber *lhs,
/* If total ordering then handle differing signs 'up front' */ /* If total ordering then handle differing signs 'up front' */
if (op==COMPTOTAL) { /* total ordering */ if (op==COMPTOTAL) { /* total ordering */
if (decNumberIsNegative(lhs) & !decNumberIsNegative(rhs)) { if (decNumberIsNegative(lhs) && !decNumberIsNegative(rhs)) {
result=-1; result=-1;
break; break;
} }
if (!decNumberIsNegative(lhs) & decNumberIsNegative(rhs)) { if (!decNumberIsNegative(lhs) && decNumberIsNegative(rhs)) {
result=+1; result=+1;
break; break;
} }