Michael Paquier <mich...@paquier.xyz> writes: > On Fri, Feb 19, 2021 at 02:21:21AM -0500, Tom Lane wrote: >> +1, but I think the first period in this comment is redundant: >> + AC_DEFINE([USE_OPENSSL], 1, [Define to 1 to build with OpenSSL support. >> (--with-ssl=openssl).])
> I guess that you mean the second period here to be more consistent > with the others? That would mean the following diff: > + AC_DEFINE([USE_OPENSSL], 1, [Define to 1 to build with OpenSSL support. > (--with-ssl=openssl)]) Hm. It should be consistent with the rest, for sure. Personally I'd put the only period at the end, but I suppose we should stick with the prevailing style if there is one. regards, tom lane