From: | "Andrew Dunstan" <andrew(at)dunslane(dot)net> |
---|---|
To: | <pgsql-patches(at)postgresql(dot)org> |
Subject: | nitpick consistency patch for pg_dump.c |
Date: | 2003-07-18 15:15:23 |
Message-ID: | 54578.199.90.235.43.1058555723.squirrel@www.dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
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
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2003-07-18 17:13:18 | Re: nitpick consistency patch for pg_dump.c |
Previous Message | Bruce Momjian | 2003-07-18 04:44:00 | Re: %x, %g warnings |