Daniel Gustafsson <dan...@yesql.se> writes:
> I think these are nice cleanups to simplify and streamline the code, just a
> few
> small comments from reading the patch:
> /* If no subcommands, don't collect */
> - if
> (list_length(currentEventTriggerState->currentCommand->d.alterTable.subcmds)
> != 0)
> + if (currentEventTriggerState->currentCommand->d.alterTable.subcmds)
> Here the current coding gives context about the data structure used for the
> subcmds member which is now lost. I don't mind the change but rewording the
> comment above to indicate that subcmds is a list would be good IMHO.
I think testing for equality to NIL is better where that's a concern.
> Might be personal taste, but I think the parenthesis should be kept here as a
> visual aid for the reader.
+1
regards, tom lane
