Re: pgsql: Fix for early log messages during postmaster startup getting lost - Mailing list pgsql-committers

From Neil Conway
Subject Re: pgsql: Fix for early log messages during postmaster startup getting lost
Date
Msg-id 1171251947.5454.200.camel@localhost.localdomain
Whole thread Raw
In response to pgsql: Fix for early log messages during postmaster startup getting lost  (mha@postgresql.org (Magnus Hagander))
Responses Re: pgsql: Fix for early log messages during postmaster startup getting lost
List pgsql-committers
On Sun, 2007-02-11 at 07:59 -0400, Magnus Hagander wrote:
> Fix for early log messages during postmaster startup getting lost when
> running as a service on Win32.

FYI, it is considered good practise to commit a patch at approximately
(or exactly) the same time on all branches, so tools like cvs2cl will be
more likely to collect the changes together.

>     pgsql/src/backend/postmaster:
>         postmaster.c (r1.519 -> r1.520)
>         (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/postmaster/postmaster.c.diff?r1=1.519&r2=1.520)

Comments like that are fragile (elog.c could change, for example), and
basically content-free anyway, IMHO. If you need to make SysLoggerPID
part of postmaster.c's external API, why not just do that, remove the
comment, and add the extern declaration to postmaster.h?

-Neil



pgsql-committers by date:

Previous
From: mha@postgresql.org (User Mha)
Date:
Subject: pgsql: Fix pg_standby to build on msvc.
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Fix for early log messages during postmaster startup getting lost