Peter Eisentraut wrote: > On l?r, 2011-09-03 at 19:18 -0400, Bruce Momjian wrote: > > FYI, here are all the C files with over 6k lines: > > > > - 45133 ./interfaces/ecpg/preproc/preproc.c > > - 33651 ./backend/parser/gram.c > > - 17551 ./backend/parser/scan.c > > 14209 ./bin/pg_dump/pg_dump.c > > 10590 ./backend/access/transam/xlog.c > > 9764 ./backend/commands/tablecmds.c > > 8681 ./backend/utils/misc/guc.c > > - 7667 ./bin/psql/psqlscan.c > > 7213 ./backend/utils/adt/ruleutils.c > > 6814 ./backend/utils/adt/selfuncs.c > > 6176 ./backend/utils/adt/numeric.c > > 6030 ./pl/plpgsql/src/pl_exec.c > > > > I have dash-marked the files that are computer-generated. It seems > > pg_dump.c and xlog.c should be split into smaller C files. > > I was thinking about splitting up plpython.c, but it's not even on that > list. ;-)
For me, the test is when I feel, "Yuck, I am in that massive file again". -- Bruce Momjian <br...@momjian.us> http://momjian.us EnterpriseDB http://enterprisedb.com + It's impossible for everything to be true. + -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers