public inbox for [email protected]
help / color / mirror / Atom feedFrom: Kyotaro Horiguchi <[email protected]>
To: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Cc: [email protected]
Subject: Re: fix stats_fetch_consistency value in postgresql.conf.sample
Date: Fri, 17 Jun 2022 09:43:58 +0900 (JST)
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]> <[email protected]>
References: <[email protected]>
<[email protected]>
<[email protected]>
At Thu, 16 Jun 2022 08:23:07 -0500, Justin Pryzby <[email protected]> wrote in
> On Thu, Jun 16, 2022 at 05:19:46PM +0900, Kyotaro Horiguchi wrote:
> > At Sat, 11 Jun 2022 09:41:37 -0500, Justin Pryzby <[email protected]> wrote in
> > > Note that this gives:
> > >
> > > guc.c:7573:9: warning: ‘dst’ may be used uninitialized in this function [-Wmaybe-uninitialized]
> >
> > Mmm. I don't have an idea where the 'dst' came from...
>
> Well, in your latest patch, you've renamed it.
>
> guc.c:7586:19: warning: ‘result’ may be used uninitialized in this function [-Wmaybe-uninitialized]
> 7586 | PG_RETURN_TEXT_P(cstring_to_text(result));
Ooo. I find that the patch on my hand was different from that on this
list by some reason uncertain to me. I now understand what's
happening.
At Sat, 11 Jun 2022 09:41:37 -0500, Justin Pryzby <[email protected]> wrote in
> with gcc version 9.2.1 20191008 (Ubuntu 9.2.1-9ubuntu2)
My compiler (gcc 8.5.0) (with -Wswitch) is satisfied by finding that
the switch() covers all enum values. I don't know why the new
compiler complains with this, but compilers in such environment should
shut up by the following change.
- char *result;
+ char *result = "";
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
view thread (42+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
Subject: Re: fix stats_fetch_consistency value in postgresql.conf.sample
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox