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 ;-) cheers andrew *** pg_dump.c~ 2003-06-25 00:08:19.000000000 -0400 --- pg_dump.c 2003-07-17 15:34:52.000000000 -0400 *************** *** 3412,3418 **** i_attname = PQfnumber(res, "attname"); i_atttypdefn = PQfnumber(res, "atttypdefn"); ! appendPQExpBuffer(q, "CREATE TYPE %s AS (", fmtId(tinfo->typname)); for (i = 0; i < ntups; i++) --- 3412,3418 ---- i_attname = PQfnumber(res, "attname"); i_atttypdefn = PQfnumber(res, "atttypdefn"); ! appendPQExpBuffer(q, "CREATE TYPE %s AS (\n", fmtId(tinfo->typname)); for (i = 0; i < ntups; i++) *************** *** 3423,3433 **** 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"); /* * DROP must be fully qualified in case same name appears in --- 3423,3431 ---- attname = PQgetvalue(res, i, i_attname); atttypdefn = PQgetvalue(res, i, i_atttypdefn); ! appendPQExpBuffer(q, "\n\t%s %s", fmtId(attname), atttypdefn); } ! appendPQExpBuffer(q, "\n);\n"); /* * DROP must be fully qualified in case same name appears in ---------------------------(end of broadcast)--------------------------- TIP 2: you can get off all lists at once with the unregister command (send "unregister YourEmailAddressHere" to [EMAIL PROTECTED])