Re: Adding a --quiet option to initdb - Mailing list pgsql-patches

From Andrew Dunstan
Subject Re: Adding a --quiet option to initdb
Date
Msg-id 1138199284.3268.11.camel@swithin
Whole thread Raw
In response to Adding a --quiet option to initdb  (Devrim GUNDUZ <devrim@commandprompt.com>)
Responses Re: [HACKERS] Adding a --quiet option to initdb
List pgsql-patches
On Wed, 2006-01-25 at 16:08 +0200, Devrim GUNDUZ wrote:
> Hi,
>
> Attached is a patch which adds --quiet and --q option to initdb. I
> personally needed this option while writing a document and taking
> screenshot :) It only shows the error and warning messages, as well as
> the last lines.
>
> I've updated the docs. Regression tests pass.
>
> This is my first patch to PostgreSQL source, so please guide me if I
> have done something wrong.
>

Devrim,

What's wrong with just sending stdout to /dev/null? If that eats error
messages too then we should probably fix initdb to send those to stderr.

But if we are going to do this, then I also noticed a couple of things:

. you should explicitly initialize the quiet variable, in keeping with
the style of the others nearby.

. the idiom

   if (! quiet)
   {
      fputs(_("some message"),stdout);
      fflush(stdout);
   }

should not be endlessly repeated. Make it a macro or a function.

I wonder if we can just set rid of all those fflush() calls by
unbuffering stdout with a single call to setbuf() or setvbuf()?



cheers

andrew




pgsql-patches by date:

Previous
From: Devrim GUNDUZ
Date:
Subject: Adding a --quiet option to initdb
Next
From: Devrim GUNDUZ
Date:
Subject: Re: [HACKERS] Adding a --quiet option to initdb