Fix failure with pgperlcritic from the TAP test of synchronous replication

Oversight in 7d81bdc, which introduced a new routine in perl lacking a
return clause.  Per buildfarm member crake.

Backpatch down to 9.6 like its parent.

Reported-by: Andrew Dunstan
Discussion: https://postgr.es/m/16da29fa-d504-1380-7095-40de586dc038@2ndQuadrant.com
Backpatch-through: 9.6

diff --git a/src/test/recovery/t/007_sync_rep.pl b/src/test/recovery/t/007_sync_rep.pl
index be8cd1d18a340b9d04143494081798f30d7aa8de..442b1180a0d0b8d04020f8ad457e3c190a43f81a 100644 (file)
--- a/src/test/recovery/t/007_sync_rep.pl
+++ b/src/test/recovery/t/007_sync_rep.pl
@@ -41,6 +41,7 @@ sub start_standby_and_wait
 
        print("### Waiting for standby \"$standby_name\" on \"$master_name\"\n");
        $master->poll_query_until('postgres', $query);
+       return;
 }
 
 # Initialize master node