On Sun, Jun 12, 2011 at 7:26 AM, <richhguard-monot...@yahoo.co.uk> wrote: > Hello, > I'm new to PostgreSQL and git, but having read through the wiki entries such > as http://wiki.postgresql.org/wiki/Submitting_a_Patch, I think I have a patch > worthy of submission. > > It's a readability improvement in src/backend/commands/comment.c > (CreateComments function), which changes the existing code from incrementing > a variable for use as the array index, to use explicit ``values'' instead. > > This has the following benefits > > 1) The structure of ``values'' is now clear at first glance. > 2) ``i'' is then only used for 1 reason; the for loop > > The patch is based on "master", and all existing tests pass.
Wow. That code is pretty ugly, all right. I think, though, that we probably ought to be using the Apg_description_<columnname> constants instead of writing 0-3. Care to update the patch? -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers