Re: Large C files - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: Large C files
Date
Msg-id 201109061405.p86E5Wo16549@momjian.us
Whole thread Raw
In response to Re: Large C files  (Peter Eisentraut <peter_e@gmx.net>)
Responses Re: Large C files
List pgsql-hackers
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  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com
 + It's impossible for everything to be true. +


pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: B-tree parent pointer and checkpoints
Next
From: Bruce Momjian
Date:
Subject: Re: Alpha 1 for 9.2