On Tue, Jan 31, 2017 at 10:04 AM, Tom Lane <[email protected]> wrote:
> Haribabu Kommi <[email protected]> writes: > > On Mon, Jan 30, 2017 at 5:18 PM, Michael Paquier < > [email protected]> > > wrote: > >> #define USER_AUTH_LAST uaPeer > >> StaticAssertStmt(lengthof(UserAuthName) == USER_AUTH_LAST + 1, > >> "UserAuthName must include all user authentication names"); > > > Thanks for the review. Added the static assert statement. > > This isn't exactly bulletproof, since somebody could add another enum > value and forget to update the macro. Still, it's better than nothing. > I tried to make it a shade more idiot-proof by putting the #define > physically inside the enum list --- you'd have to really have blinders > on to not notice it there. (Not that people haven't made equally silly > mistakes :-(.) > > Pushed with that adjustment. Thanks for working on this! > Thanks for your support. Regards, Hari Babu Fujitsu Australia
