On Wed, Apr 29, 2015 at 1:14 PM, Alvaro Herrera <alvherre@2ndquadrant.com> wrote: > > Fabrízio de Royes Mello wrote: > > > > Looking at the patch again I realize the code is very ugly, so I'll rework > > the patch. > > Yes, I think get_object_address should figure out what to do with the > representation of CURRENT DATABASE directly, rather than having the > COMMENT code morph from that into a list containing the name of the > current database. Please see about changing SECURITY LABEL at the same > time, if only to make sure that the representation is sane. >
I have this idea:
1) Add an ObjectAddress field to CommentStmt struct an set it in gram.y
2) In the CommentObject check if CommentStmt->address is InvalidObjectAddress then call get_object_address else use it
