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
Attachments:
remove_global_declaration.patchapplication/octet-stream; name=remove_global_declaration.patchDownload+1-2
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