Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Wikipedia: Wikitech
Re: [MediaWiki-CVS] SVN: [37787] trunk/phase3/includes
 

Index | Next | Previous | View Flat


Simetrical+wikilist at gmail

Jul 17, 2008, 10:20 AM


Views: 392
Permalink
Re: [MediaWiki-CVS] SVN: [37787] trunk/phase3/includes

On Thu, Jul 17, 2008 at 8:19 AM, <werdna[at]svn.wikimedia.org> wrote:
> + $h = popen( $cmd, 'r' );
> +
> + $diff = '';
> +
> + do {
> + $data = fread( $h, 8192 );
> + if ( strlen( $data ) == 0 ) {
> + break;
> + }
> + $diff .= $data;
> + } while ( true );
> +
> + // Clean up
> + pclose( $h );
> . . .
> + $diff_lines = explode( "\n", $diff );

Couldn't all this be replaced with the following?

$diff_lines = array();
exec( $cmd, $diff_lines );

_______________________________________________
Wikitech-l mailing list
Wikitech-l[at]lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Subject User Time
Re: [MediaWiki-CVS] SVN: [37787] trunk/phase3/includes Simetrical+wikilist at gmail Jul 17, 2008, 10:20 AM
    Re: [MediaWiki-CVS] SVN: [37787] trunk/phase3/includes tstarling at wikimedia Jul 18, 2008, 12:57 AM

  Index | Next | Previous | View Flat
 
 


Interested in having your list archived? Contact lists@gossamer-threads.com
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.