r72290 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r72289‎ | r72290 | r72291 >
Date:16:00, 3 September 2010
Author:reedy
Status:ok
Tags:
Comment:
Followup r72238, remove &
Modified paths:
  • /trunk/phase3/includes/DjVuImage.php (modified) (history)
  • /trunk/phase3/includes/media/Bitmap.php (modified) (history)
  • /trunk/phase3/includes/media/DjVu.php (modified) (history)
  • /trunk/phase3/includes/media/SVG.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/media/Bitmap.php
@@ -363,7 +363,7 @@
364364 $cmd = wfEscapeShellArg( $wgImageMagickConvertCommand ) . ' -version';
365365 wfDebug( __METHOD__.": Running convert -version\n" );
366366 $retval = '';
367 - $return = wfShellExec( $cmd, &$retval );
 367+ $return = wfShellExec( $cmd, $retval );
368368 $x = preg_match('/Version: ImageMagick ([0-9]*\.[0-9]*\.[0-9]*)/', $return, $matches);
369369 if( $x != 1 ) {
370370 wfDebug( __METHOD__.": ImageMagick version check failed\n" );
Index: trunk/phase3/includes/media/SVG.php
@@ -95,7 +95,7 @@
9696 ) . " 2>&1";
9797 wfProfileIn( 'rsvg' );
9898 wfDebug( __METHOD__.": $cmd\n" );
99 - $err = wfShellExec( $cmd, &$retval );
 99+ $err = wfShellExec( $cmd, $retval );
100100 wfProfileOut( 'rsvg' );
101101 }
102102 $removed = $this->removeBadFile( $dstPath, $retval );
Index: trunk/phase3/includes/media/DjVu.php
@@ -109,7 +109,7 @@
110110 wfProfileIn( 'ddjvu' );
111111 wfDebug( __METHOD__.": $cmd\n" );
112112 $retval = '';
113 - $err = wfShellExec( $cmd, &$retval );
 113+ $err = wfShellExec( $cmd, $retval );
114114 wfProfileOut( 'ddjvu' );
115115
116116 $removed = $this->removeBadFile( $dstPath, $retval );
Index: trunk/phase3/includes/DjVuImage.php
@@ -249,7 +249,7 @@
250250 $cmd = wfEscapeShellArg( $wgDjvuTxt ) . ' --detail=page ' . wfEscapeShellArg( $this->mFilename ) ;
251251 wfDebug( __METHOD__.": $cmd\n" );
252252 $retval = '';
253 - $txt = wfShellExec( $cmd, &$retval );
 253+ $txt = wfShellExec( $cmd, $retval );
254254 wfProfileOut( 'djvutxt' );
255255 if( $retval == 0) {
256256 # Get rid of invalid UTF-8, strip control characters

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r72238Add some $retval = '' before some wfShellExecreedy22:15, 2 September 2010

Status & tagging log