On 2019-08-17 14:57, Michael Paquier wrote: > On Sat, Aug 17, 2019 at 10:11:27AM +0200, Peter Eisentraut wrote: >> I was a bit confused by some of the comments around the SCRAM function >> read_any_attr(), used to skip over extensions. >> >> The comment "Returns NULL if there is attribute.", besides being >> strangely worded, appears to be wrong anyway, because the function never >> returns NULL. > > This may have come from an incorrect merge with a past version when > this code has been developed. +1 for me for your suggestions and your > patch.
committed -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
