Re: Unnecessary global variable declared in xlog.c

2021-11-16 Thread Amul Sul
On Wed, Nov 17, 2021 at 7:36 AM Michael Paquier 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 a

Re: Unnecessary global variable declared in xlog.c

2021-11-16 Thread Michael Paquier
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 signature.asc Description: PGP signature

Re: Unnecessary global variable declared in xlog.c

2021-11-15 Thread Tom Lane
Michael Paquier 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 th

Re: Unnecessary global variable declared in xlog.c

2021-11-15 Thread Michael Paquier
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

Unnecessary global variable declared in xlog.c

2021-11-15 Thread Amul Sul
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 remove_global_declaration.patch Description: Binary data