User "Brion VIBBER" changed the status of MediaWiki.r97889. Old Status: new New Status: ok
User "Brion VIBBER" also posted a comment on MediaWiki.r97889. Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/97889#c23085 Commit summary: handle naming checkpoint file with first/last pageID when the file is empty Comment: I think this'd be the same as just doing a sprintf("%09d", $this->firstPageWritten) but it doesn't hurt to be explicit in the check against false, rather than implicitly coercing the false to 0. _______________________________________________ MediaWiki-CodeReview mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview
