From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Erik Rijkers <er(at)xs4all(dot)nl> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: prewarm compiler warnings |
Date: | 2018-11-23 16:08:11 |
Message-ID: | 20181123160811.h7qu6amnn3a4hrn6@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2018-Nov-23, Erik Rijkers wrote:
> gcc-6.3.0 complains about this:
>
> autoprewarm.c: In function ‘autoprewarm_main’:
> autoprewarm.c:210:9: warning: variable ‘rc’ set but not used
> [-Wunused-but-set-variable]
> int rc;
>
> The warning comes only in master.
Yeah, it's fairly new. I noticed a couple of other places that could
use the same adjustment and pushed them all together. (I used a cast to
void as all other places do that, which appeases Coverity about the
discrepancy.)
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Justin Pryzby | 2018-11-23 16:10:28 | Re: dsa_allocate() faliure |
Previous Message | REIX, Tony | 2018-11-23 15:53:57 | RE: Shared Memory: How to use SYSV rather than MMAP ? |