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!