From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: make more use of RoleSpec struct |
Date: | 2016-12-28 14:53:58 |
Message-ID: | 20161228145358.zi3x26w2hwhsywnc@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Peter Eisentraut wrote:
> Here is a small cleanup patch to make more use of the RoleSpec
> struct/node throughout the parser to avoid casts and make the code more
> self-documenting.
This makes sense to me. I started to do this at some point when I was
writing RoleSpec; eventually got annoyed about fixing the fallout (I was
working to get somebody else's patch committed) and went back to how it
is. Glad to see it done.
The only functional issue might be the removal of the IsA() checks. If
we don't cast any Node before passing it to any of those functions,
there should be no problem because any misfeasance will be reported as a
compile-time warning. Perhaps it's worth adding IsA() checks in loops
such as the one in roleSpecsToIds().
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Pavel Stehule | 2016-12-28 14:56:56 | Re: proposal: session server side variables |
Previous Message | Fabien COELHO | 2016-12-28 14:38:39 | Re: proposal: session server side variables |