Thread: Unnecessary global variable declared in xlog.c

Hi,

The attached patch moves the "LastRec" variable declaration inside
StartupXLOG() where it is supposed to be.

-- 
Regards,
Amul Sul
EDB: http://www.enterprisedb.com

On Tue, Nov 16, 2021 at 11:17:27AM +0530, Amul Sul wrote:
> The attached patch moves the "LastRec" variable declaration inside
> StartupXLOG() where it is supposed to be.

Yes, reading through the code, your suggestion makes sense.  I'll
double-check and apply if there are no objections.
--
Michael

Michael Paquier <michael@paquier.xyz> writes:
> On Tue, Nov 16, 2021 at 11:17:27AM +0530, Amul Sul wrote:
>> The attached patch moves the "LastRec" variable declaration inside
>> StartupXLOG() where it is supposed to be.

> Yes, reading through the code, your suggestion makes sense.  I'll
> double-check and apply if there are no objections.

I think LastRec was originally referenced by multiple functions
in xlog.c.  But it does look like it could be a local now.

            regards, tom lane

On Tue, Nov 16, 2021 at 02:08:54AM -0500, Tom Lane wrote:
> I think LastRec was originally referenced by multiple functions
> in xlog.c.  But it does look like it could be a local now.

Thanks for double-checking, applied this one as of f975fc3.
--
Michael

On Wed, Nov 17, 2021 at 7:36 AM Michael Paquier <michael@paquier.xyz> wrote:
>
> On Tue, Nov 16, 2021 at 02:08:54AM -0500, Tom Lane wrote:
> > I think LastRec was originally referenced by multiple functions
> > in xlog.c.  But it does look like it could be a local now.
>
> Thanks for double-checking, applied this one as of f975fc3.

Thank you, Michael.

Regards,
Amul