From 13d176d840f85a58dd1411621c0af93a3877bfa7 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Michael=20Gro=C3=9Fe?= Date: Mon, 11 Mar 2019 19:10:54 +0100 Subject: [PATCH] Use splat operator in signature, not func_get_args Most of the original scope of this patch was implemented in 8665536045c What remains is to use the splat operator in the method signature as well which is more readable than using func_get_args and doesn't annoy phan. This is now possible, because we are no longer testing on HHVM and these mocking in HHVM tests was the only place where this was broken. Change-Id: I52912d1810b955b05cd17fab424f80c201883411 --- includes/htmlform/HTMLForm.php | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/includes/htmlform/HTMLForm.php b/includes/htmlform/HTMLForm.php index 58c16021fc..b16801a2a8 100644 --- a/includes/htmlform/HTMLForm.php +++ b/includes/htmlform/HTMLForm.php @@ -297,14 +297,10 @@ class HTMLForm extends ContextSource { * Construct a HTMLForm object for given display type. May return a HTMLForm subclass. * * @param string $displayFormat - * @param mixed $arguments,... Additional arguments to pass to the constructor. - * @suppress PhanCommentParamWithoutRealParam HHVM bug T228695#5450847 + * @param mixed ...$arguments Additional arguments to pass to the constructor. * @return HTMLForm */ - public static function factory( $displayFormat/*, $arguments...*/ ) { - $arguments = func_get_args(); - array_shift( $arguments ); - + public static function factory( $displayFormat, ...$arguments ) { switch ( $displayFormat ) { case 'vform': return new VFormHTMLForm( ...$arguments ); -- 2.20.1