qemu-trivial
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] libdecnumber/dpd/decimal64: Fix compiler warning from Clang


From: Philippe Mathieu-Daudé
Subject: Re: [PATCH] libdecnumber/dpd/decimal64: Fix compiler warning from Clang 15
Date: Thu, 10 Nov 2022 21:00:30 +0100
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.4.2

On 10/11/22 14:11, Thomas Huth wrote:
Clang 15 from Fedora 37 complains:

  ../libdecnumber/dpd/decimal64.c:620:8: error: variable 'n' set but
  not used [-Werror,-Wunused-but-set-variable]
    Int  n;                     /* output bunch counter */
         ^
  1 error generated.

Remove the unused variable to silence the compiler warning.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
  libdecnumber/dpd/decimal64.c | 3 +--
  1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/libdecnumber/dpd/decimal64.c b/libdecnumber/dpd/decimal64.c
index 4816176410..290dbe8177 100644
--- a/libdecnumber/dpd/decimal64.c
+++ b/libdecnumber/dpd/decimal64.c
@@ -617,7 +617,6 @@ static const uInt multies[]={131073, 26215, 5243, 1049, 
210};
  #endif
  void decDigitsToDPD(const decNumber *dn, uInt *targ, Int shift) {
    Int  cut;                 /* work */
-  Int  n;                    /* output bunch counter */
    Int  digits=dn->digits;     /* digit countdown */
    uInt dpd;                 /* densely packed decimal value */
    uInt bin;                 /* binary value 0-999 */
@@ -676,7 +675,7 @@ void decDigitsToDPD(const decNumber *dn, uInt *targ, Int 
shift) {
      bin=0;                       /* [keep compiler quiet] */
    #endif
- for(n=0; digits>0; n++) { /* each output bunch */
+  while (digits > 0) {             /* each output bunch */
      #if DECDPUN==3               /* fast path, 3-at-a-time */
        bin=*inu;                          /* 3 digits ready for convert */
        digits-=3;                 /* [may go negative] */

Or we could we backport the upstream fix :)

https://gcc.gnu.org/git/?p=gcc.git;a=blobdiff;f=libdecnumber/dpd/decimal64.c;h=f72c5730ac3b473e2c334f604e7e52da9711fa72;hp=269eaecade5115a62e979897742cd5decec0681e;hb=8a79685989bff33f479d0ac2df0e18d55d3ba78b;hpb=e1d1842b5432472330384d1523bb3c3132c4fea0



reply via email to

[Prev in Thread] Current Thread [Next in Thread]