Re: [HACKERS] Incorrect comment in 001_ssltests.pl

Поиск
Список
Период
Сортировка
От Magnus Hagander
Тема Re: [HACKERS] Incorrect comment in 001_ssltests.pl
Дата
Msg-id CABUevEwXM0Zs0jwwg4s5tAv7cdCr0f3Ecx4nXCG28aZ=2SFxyA@mail.gmail.com
обсуждение исходный текст
Ответ на [HACKERS] Incorrect comment in 001_ssltests.pl  (Michael Paquier <michael.paquier@gmail.com>)
Ответы Re: [HACKERS] Incorrect comment in 001_ssltests.pl  (Michael Paquier <michael.paquier@gmail.com>)
Список pgsql-hackers


On Sat, Jun 17, 2017 at 1:53 AM, Michael Paquier <michael.paquier@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! 

--

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Magnus Hagander
Дата:
Сообщение: Re: [HACKERS] Typos in comments
Следующее
От: Marina Polyakova
Дата:
Сообщение: Re: [HACKERS] WIP Patch: Pgbench Serialization and deadlock errors