Minor cleanup/optimization in pg_dump.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 30 Dec 2021 19:29:32 +0000 (14:29 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 30 Dec 2021 19:29:32 +0000 (14:29 -0500)
commitc7cf73eb7b9e7911748ebe117a7219f21e504121
tree80d8d112379e1cc022480205469f8b0c187b88ef
parente68570e388f08c2e36ce7d2a9564941b89db6549
Minor cleanup/optimization in pg_dump.

In the wake of commits 05649b88c and 5209c0ba0, findComments() and
findSecLabels() no longer use their "Archive *fout" arguments,
so get rid of those.

While doing that, I noticed that there's no very good reason why
dumpCompositeTypeColComments() should be doing its own query to fetch
the column names of the composite type, when the calling function has
just fetched the same data.  Tweak it to use that query result.  This
probably doesn't save a lot for most people, because since 5209c0ba0
we won't get into this code at all unless the composite type has at
least one comment.  Nonetheless, it's a wasted query.
src/bin/pg_dump/pg_dump.c