Re: pgsql: Report progress of CREATE INDEX operations - Mailing list pgsql-committers

From Andres Freund
Subject Re: pgsql: Report progress of CREATE INDEX operations
Date
Msg-id 20190402190003.esiyxbmnm63jlydd@alap3.anarazel.de
Whole thread Raw
In response to Re: pgsql: Report progress of CREATE INDEX operations  (Alvaro Herrera <alvherre@2ndquadrant.com>)
Responses Re: pgsql: Report progress of CREATE INDEX operations  (Alvaro Herrera <alvherre@2ndquadrant.com>)
List pgsql-committers
Hi,

On 2019-04-02 15:41:46 -0300, Alvaro Herrera wrote:
> On 2019-Apr-02, Alvaro Herrera wrote:
> 
> > Report progress of CREATE INDEX operations
> 
> Argh, I pushed an older version of the commit.

Note that longfin complains about something different than the output
files:

https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=longfin&dt=2019-04-02%2018%3A49%3A03

lmgr.c:882:14: error: variable 'count' is used uninitialized whenever '?:' condition is false
[-Werror,-Wsometimes-uninitialized]
                                                                                   progress ? &count : NULL));
                                                                                   ^~~~~~~~
lmgr.c:883:12: note: uninitialized use occurs here
                total += count;
                         ^~~~~
lmgr.c:882:14: note: remove the '?:' if its condition is always true
                                                                                   progress ? &count : NULL));
                                                                                   ^~~~~~~~~~~      ~~~~~~~
lmgr.c:878:14: note: initialize the variable 'count' to silence this warning
                int                     count;
                                             ^
                                              = 0
1 error generated.
make[4]: *** [lmgr.o] Error 1
make[4]: *** Waiting for unfinished jobs....

Greetings,

Andres Freund



pgsql-committers by date:

Previous
From: Alvaro Herrera
Date:
Subject: pgsql: Update expected output for modified catalog definition
Next
From: Alvaro Herrera
Date:
Subject: pgsql: Prevent use of uninitialized variable