Re: Small xlog.c cleanup - Mailing list pgsql-patches

From The Hermit Hacker
Subject Re: Small xlog.c cleanup
Date
Msg-id Pine.BSF.4.33.0102271430240.354-100000@mobile.hub.org
Whole thread Raw
In response to Re: Small xlog.c cleanup  (Bruce Momjian <pgman@candle.pha.pa.us>)
Responses Re: Small xlog.c cleanup
List pgsql-patches
On Tue, 27 Feb 2001, Bruce Momjian wrote:

> > On Tue, 27 Feb 2001, Bruce Momjian wrote:
> >
> > > All source gets reformatted before release by src/tools/pgindent
> > > anyway.
> >
> > This is more than just a formatting change -- it turns
> > some static globals into static locals.
> >
> > Sorry, I should have been a little more verbose in my
> > description.
>
> Can you point me to the change.  I don't see it:
>
> ---------------------------------------------------------------------------
>

He's moved the "static globals" from around line 286 of
src/backend/access/transam/xlog.c:

> -static int     readFile = -1;
> -static uint32 readId = 0;
> -static uint32 readSeg = 0;
> -static uint32 readOff = 0;
>  static char readBuf[BLCKSZ];
>  static XLogRecord *nextRecord = NULL;
>

to be static locals inside of function ReadRecord around line 1216 in the
same file:

> @@ -1214,6 +1210,10 @@
>         bool            nextmode = (RecPtr == NULL);
>         int                     emode = (nextmode) ? LOG : STOP;
>         bool            noBlck = false;
> +       static int      readFile = -1;
> +       static uint32 readId = 0;
> +       static uint32 readSeg = 0;
> +       static uint32 readOff = 0;
>
> --
>   Bruce Momjian                        |  http://candle.pha.pa.us
>   pgman@candle.pha.pa.us               |  (610) 853-3000
>   +  If your life is a hard drive,     |  830 Blythe Avenue
>   +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026
>

Marc G. Fournier                   ICQ#7615664               IRC Nick: Scrappy
Systems Administrator @ hub.org
primary: scrappy@hub.org           secondary: scrappy@{freebsd|postgresql}.org


pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Small xlog.c cleanup
Next
From: Tom Lane
Date:
Subject: Re: Small xlog.c cleanup