Re: Refactor xlog.c #1 - startup.c - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: Refactor xlog.c #1 - startup.c
Date
Msg-id CABUevEz7DadjXyfUnF23Tq5XG_RivkBv7S507mjJ81HUnmi4hA@mail.gmail.com
Whole thread Raw
In response to Re: Refactor xlog.c #1 - startup.c  (Simon Riggs <simon@2ndQuadrant.com>)
Responses Re: Refactor xlog.c #1 - startup.c
List pgsql-hackers
On Wed, Nov 2, 2011 at 14:33, Simon Riggs <simon@2ndquadrant.com> wrote:
> On Wed, Nov 2, 2011 at 1:14 PM, Robert Haas <robertmhaas@gmail.com> wrote:
>> On Wed, Nov 2, 2011 at 4:44 AM, Simon Riggs <simon@2ndquadrant.com> wrote:
>>> Patch removes stuff related to "startup process" and creates files
>>> under src/$DIR/postmaster for this code.
>>>
>>> This makes startup process look more like bgwriter, walwriter etc..
>>
>> In general, +1.
>
> Thanks for the review.
>
>> But I don't think we want duplicate function prototypes for the same
>> functions, so -1 for this hunk:
>>
>> + /* in xlog.c */
>> + extern void WakeupRecovery(void);
>> + extern void StartupXLOG(void);
>>
>> The existing prototypes in xlog.h seem like they should be just fine.
>
> That was to avoid having startup.c include xlog.h, which it doesn't
> really need to see.

Double prototypes seems to me like it's a cure worse than the disease to me...

> Not worried either way, so I'll do as you suggest.

+1.

--
 Magnus Hagander
 Me: http://www.hagander.net/
 Work: http://www.redpill-linpro.com/


pgsql-hackers by date:

Previous
From: Simon Riggs
Date:
Subject: Re: Refactor xlog.c #1 - startup.c
Next
From: Tom Lane
Date:
Subject: Re: Thoughts on "SELECT * EXCLUDING (...) FROM ..."?