Aizaz Ahmed <[EMAIL PROTECTED]> writes:
> looks like there's some duplication between this array and the 
> static const char *const GucContext_names[] array in
> src/backend/utils/misc/help_config.c
> Is there some way we could have them both use the same array?

Good idea.  Please send a patch that exports the guc.c array for use in
the other file.  I'd lean towards the lower-cased spellings, though I'm
not strong about it.  (I'd also not use a dash in "super-user".)

> Also, as a side note, I don't think Tom is a big fan of using comments
> to indicate what needs to be kept in sync with what, if I can take the
> liberty to quote him the last time a situation like this arose:

I still stand by that opinion.  We've seen a couple failures now with
respect to these arrays, but I think that comes from having people
independently patching the same code with too long delay between patch
submission and application.  The comment wouldn't have prevented the
error, because patch(1) can't read comments.

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 8: explain analyze is your friend

Reply via email to