Re: Removal of duplicate variable declarations in fe-connect.c - Mailing list pgsql-hackers

From Haribabu Kommi
Subject Re: Removal of duplicate variable declarations in fe-connect.c
Date
Msg-id CAJrrPGeQdr-YfLdeGxMFb4erqNWU77E49mD2HoHPntHTBY26Ag@mail.gmail.com
Whole thread Raw
In response to Re: Removal of duplicate variable declarations in fe-connect.c  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
On Fri, Feb 22, 2019 at 3:22 PM Michael Paquier <michael@paquier.xyz> wrote:
On Fri, Feb 22, 2019 at 11:33:17AM +1100, Haribabu Kommi wrote:
> During the development of another feature, I found that same local
> variables are declared twice.
> IMO, there is no need of again declaring the local variables. Patch
> attached.

Indeed, fixed.  That's not a good practice, and each variable is
assigned in its own block before getting used, so there is no
overlap.

Thanks.

Regards,
Haribabu Kommi
Fujitsu Australia

pgsql-hackers by date:

Previous
From: Haribabu Kommi
Date:
Subject: Re: Libpq support to connect to standby server as priority
Next
From: Haribabu Kommi
Date:
Subject: Re: pg_basebackup ignores the existing data directory permissions