On Sat, Jun 17, 2017 at 1:53 AM, Michael Paquier <michael.paqu...@gmail.com> wrote:
> Hi all, > > I have noticed the following thing: > --- a/src/test/ssl/t/001_ssltests.pl > +++ b/src/test/ssl/t/001_ssltests.pl > @@ -34,8 +34,6 @@ sub run_test_psql > # The first argument is a (part of a) connection string, and it's also > printed > # out as the test case name. It is appended to $common_connstr global > variable, > # which also contains a libpq connection string. > -# > -# The second argument is a hostname to connect to. > sub test_connect_ok > { > my $connstr = $_[0] > > But test_connect_ok and test_connect_fails do not have a second > argument as the hostname is appended directly into $common_connstr. > One could argue that the first sentences should just read "the argument" once the second one isn't referred, but I can't get too excited about that. Thus, patch applied as-is - thanks! -- Magnus Hagander Me: https://www.hagander.net/ <http://www.hagander.net/> Work: https://www.redpill-linpro.com/ <http://www.redpill-linpro.com/>