Re: zheap: a new storage format for PostgreSQL

On Sun, Nov 11, 2018 at 11:55 PM Kuntal Ghosh
<kuntalghosh(dot)2007(at)gmail(dot)com> wrote:
>
> On Sat, Nov 10, 2018 at 8:51 PM Daniel Westermann
> <daniel(dot)westermann(at)dbi-services(dot)com> wrote:
> >
> > >>Thanks. Initializing the variable seems like the right fix here.
> >
> > ... just had a warning when recompiling from the latest sources on CentOS 7:
> >
> > labels -Wmissing-format-attribute -Wformat-security -fno-strict-aliasing -fwrapv -fexcess-precision=standard -O2 -I../../../../src/include -D_GNU_SOURCE -I/usr/include/libxml2 -c -o tpd.o tpd.c
> > tpd.c: In function ‘TPDFreePage’:
> > tpd.c:1003:15: warning: variable ‘curblkno’ set but not used [-Wunused-but-set-variable]
> > BlockNumber curblkno = InvalidBlockNumber;
> > ^

This variable is used only for Asserts, so we need to use
PG_USED_FOR_ASSERTS_ONLY while declaring it.

> Thanks Daniel for testing zheap and reporting the issue. We'll push a
> fix for the same.
>

Pushed the fix now.

--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com