This is a totally trivial patch for something that was a very minor nit that

annoyed me the other day while I was documenting my current project. It
makes pg_dump use the same layout for types as for tables, by putting "\n\t"
before the first field and "\n" before the final ");"

Can't really justify this too much except to say I had an itch and I
scratched it ;-)

Andrew Dunstan

diff --git a/src/bin/pg_dump/pg_dump.c b/src/bin/pg_dump/pg_dump.c
index 66b10c2aa77c60df051d2d23fa363a36938736c3..0c38d4a8ef3ac95e6252c2a139c2b36cb69cacc5 100644 (file)
--- a/src/bin/pg_dump/pg_dump.c
+++ b/src/bin/pg_dump/pg_dump.c
@@ -12,7 +12,7 @@
  * by PostgreSQL
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_dump.c,v 1.338 2003/07/25 21:02:52 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_dump.c,v 1.339 2003/07/27 03:47:22 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -3432,12 +3432,12 @@ dumpOneCompositeType(Archive *fout, TypeInfo *tinfo)
        attname = PQgetvalue(res, i, i_attname);
        atttypdefn = PQgetvalue(res, i, i_atttypdefn);
 
-       if (i > 0)
-           appendPQExpBuffer(q, ",\n\t");
-       appendPQExpBuffer(q, "%s %s", fmtId(attname), atttypdefn);
+       appendPQExpBuffer(q, "\n\t%s %s", fmtId(attname), atttypdefn);
+       if (i < ntups - 1)
+           appendPQExpBuffer(q, ",");
    }
-   appendPQExpBuffer(q, ");\n");
-
+   appendPQExpBuffer(q, "\n);\n");
+   
    /*
     * DROP must be fully qualified in case same name appears in
     * pg_catalog