Robert Haas <robertmhaas@gmail.com> writes:
> On Sun, Jun 12, 2011 at 7:26 AM, <richhguard-monotone@yahoo.co.uk> wrote:
>> It's a readability improvement in src/backend/commands/comment.c (CreateComments function), which changes the
existingcode from incrementing a variable for use as the array index, to use explicit ``values'' instead.
> 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?
Historically this i++ approach has been used in a lot of places that
fill in system catalog tuples. We've fixed some of them over time, but
I doubt this is the only one remaining. If we're going to try to remove
it here, maybe we ought to try to fix them all rather than just this
one. I agree that the main point of doing so would be to introduce the
greppable Apg_xxx constants, and so just using hard-coded integers is
not much of an improvement.
regards, tom lane