Careful here too ... there are hundreds of UNUSED reports. I see no benefit to actually changing code to conform with a mindless program (i.e. coverity) that disagrees with a deliberate programming practice.
I have been marking issues like this intentional/insignificant/ignore and assigning to "jbj". 73 de Jeff On Aug 16, 2013, at 4:13 AM, Pinto Elia wrote: > RPM Package Manager, CVS Repository > http://rpm5.org/cvs/ > ____________________________________________________________________________ > > Server: rpm5.org Name: Pinto Elia > Root: /v/rpm/cvs Email: devzero2...@rpm5.org > Module: popt Date: 16-Aug-2013 10:13:29 > Branch: HEAD Handle: 2013081608132900 > > Modified files: > popt popt.c > > Log: > fix coverity CID 1057440: Unused pointer value (UNUSED_VALUE) > > Summary: > Revision Changes Path > 1.161 +1 -1 popt/popt.c > ____________________________________________________________________________ > > patch -p0 <<'@@ .' > Index: popt/popt.c > ============================================================================ > $ cvs diff -u -r1.160 -r1.161 popt.c > --- popt/popt.c 5 Sep 2011 11:46:28 -0000 1.160 > +++ popt/popt.c 16 Aug 2013 08:13:29 -0000 1.161 > @@ -1692,7 +1692,7 @@ > if (opt->longName) { > if (!F_ISSET(opt, ONEDASH)) > *s++ = '-'; > - s = stpcpy(s, opt->longName); > + (void)stpcpy(s, opt->longName); > } else { > *s++ = opt->shortName; > *s = '\0'; > @@ . > ______________________________________________________________________ > RPM Package Manager http://rpm5.org > CVS Sources Repository rpm-...@rpm5.org ______________________________________________________________________ RPM Package Manager http://rpm5.org Developer Communication List rpm-devel@rpm5.org
