GlobalFunctions: Deprecate swap()

This unprefixed function, added in r7198 (2b3f4c749d3d), has been
unused since r12411 (f2a59db33f).

It is short and simple enough that on the rare occasions it is needed,
it can be inlined or copied into the calling class as a private method.

Alternatively, the idiom `list( $a, $b ) = array( $b, $a )` can be used.

Change-Id: Ieb4602597a54eb21a5de177fee6dafa7ac71ce1d
This commit is contained in:
Kevin Israel 2014-06-17 21:40:03 -04:00 committed by Krinkle
parent c7772da144
commit 26e1e083e8
2 changed files with 4 additions and 0 deletions

View file

@ -2592,10 +2592,12 @@ function wfIsHHVM() {
/**
* Swap two variables
*
* @deprecated since 1.24
* @param mixed $x
* @param mixed $y
*/
function swap( &$x, &$y ) {
wfDeprecated( __FUNCTION__, '1.24' );
$z = $x;
$x = $y;
$y = $z;

View file

@ -369,6 +369,8 @@ class GlobalTest extends MediaWikiTestCase {
* @covers ::swap
*/
public function testSwapVarsTest() {
$this->hideDeprecated( 'swap' );
$var1 = 1;
$var2 = 2;