Re: Commands/ directory reorganisation

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: John Gray <jgray(at)azuli(dot)co(dot)uk>
Cc: pgsql-patches(at)postgresql(dot)org, Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Subject: Re: Commands/ directory reorganisation
Date: 2002-04-15 00:33:31
Message-ID: 25254.1018830811@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

John Gray <jgray(at)azuli(dot)co(dot)uk> writes:
> Patch attached in line with my previous statement on -hackers about
> reorganisation of files in the commands directory. (Note that define.c
> has been kept, holding some of the common support code for define
> routines.)

This patch looks reasonable to me, with one significant gripe and a
couple of minor ones.

The significant gripe is that I don't think domaincmds.c should exist;
domains are not really different from types and so I think their
commands should be in typecmds.c. Splitting into two files doesn't do
much except eliminate the possibility of sharing code via "static"
subroutines.

I have one or two trivial coding-style issues too, like not wanting to
see "extern" on routine definitions.

If no one has any other objections, I'll fix the things that are
bothering me and check it in.

regards, tom lane

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2002-04-15 00:34:50 Re: Commands/ directory reorganisation
Previous Message Bruce Momjian 2002-04-15 00:06:05 Re: BETWEEN Help