Re: pgsql: Adjust basebackup.c to suppress compiler warnings.

From: Alvaro Herrera <alvherre(at)commandprompt(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, pgsql-committers <pgsql-committers(at)postgresql(dot)org>
Subject: Re: pgsql: Adjust basebackup.c to suppress compiler warnings.
Date: 2011-01-11 20:00:05
Message-ID: 1294775954-sup-6748@alvh.no-ip.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

> Magnus Hagander <magnus(at)hagander(dot)net> writes:

> > I noticed you didn't create a function prototype at the top of the
> > file for this function. Was that intentional, or missed? If it was
> > intentional, can you give me a quick lesson in when it should be done
> > and whe nit shouldn't? :-)

I think normally you can omit the prototype when the function definition
comes before its first usage.

--
Álvaro Herrera <alvherre(at)commandprompt(dot)com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Magnus Hagander 2011-01-11 20:12:34 pgsql: Add missing function prototype, for consistency
Previous Message Tom Lane 2011-01-11 19:44:04 Re: pgsql: Adjust basebackup.c to suppress compiler warnings.