From e43a1490c868c3ac564c0ea13cc68a0118934870 Mon Sep 17 00:00:00 2001
From: Markus Glaser <mglaser@users.mediawiki.org>
Date: Tue, 19 Oct 2010 19:58:58 +0000
Subject: [PATCH] Follow up r74753, removed pass-by-references and @ operator

---
 maintenance/tests/selenium/SeleniumServerManager.php | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/maintenance/tests/selenium/SeleniumServerManager.php b/maintenance/tests/selenium/SeleniumServerManager.php
index 7abb34b1db..5ef4c6649d 100644
--- a/maintenance/tests/selenium/SeleniumServerManager.php
+++ b/maintenance/tests/selenium/SeleniumServerManager.php
@@ -179,12 +179,13 @@ class SeleniumServerManager {
 				// on its port. Note: this solution kinda
 				// stinks, since it uses a wait loop - dnessett
 
+				wfSuppressWarnings();
 				for ( $cnt = 1;
 				      $cnt <= $this->SeleniumServerStartTimeout;
 				      $cnt++ ) {
-					$fp = @fsockopen ( 'localhost',
+					$fp = fsockopen ( 'localhost',
 						$this->SeleniumServerPort,
-						&$errno, &$errstr, 0 );
+						$errno, $errstr, 0 );
 					if ( !$fp ) {
 						sleep( 1 );
 						continue;
@@ -194,6 +195,7 @@ class SeleniumServerManager {
 						return 'started';
 					}
 				}
+				wfRestoreWarnings();
 				echo ( "Starting Selenium server timed out.\n" );
 				return 'failed';
 			}
-- 
2.20.1