[PEPr] Comment on PHP::PHP_CodeSniffer
| From: | Christian Weiske | Date: | Thu, 20 Jul 2006 08:18:15 +0000 |
| Subject: | [PEPr] Comment on PHP::PHP_CodeSniffer | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-43544@lists.php.net to get a copy of this message | ||
Christian Weiske (http://pear.php.net/user/cweiske) has commented on the proposal for
PHP::PHP_CodeSniffer.
Comment:
- I would reverse the order of parameters. The files should be at the end
of the param list, multiple files should be allowed.
- There should be an option to list all available coding standards
- Abilitity to set a default CS is needed
- Show the line in which the errors occured (if enabled by parameter)
cweiske:/data/php-gtk/two/Dev_Inspector>
/data/cvs/php/php5install/bin/phpcs --help
PHP Warning: substr_compare(): The start position cannot exceed initial
string length in /data/cvs/php/php5cli/share/pear/PHP/CodeSniffer.php on
line 799
PHP Warning: substr_compare(): The start position cannot exceed initial
string length in /data/cvs/php/php5cli/share/pear/PHP/CodeSniffer.php on
line 799
Usage: /data/cvs/php/php5install/bin/phpcs <file>|<dir>
--standard=<standard>
<file> : a specific file to check
cweiske:/data/php-gtk/two/Dev_Inspector>
/data/cvs/php/php5install/bin/phpcs . --standard=PEAR
PHP Warning: substr_compare(): The start position cannot exceed initial
string length in /data/cvs/php/php5cli/share/pear/PHP/CodeSniffer.php on
line 799
PHP Strict Standards: Assigning the return value of new by reference is
deprecated in /data/cvs/php/php5cli/share/pear/MIME/Type.php on line 102
PHP Strict Standards: Assigning the return value of new by reference is
deprecated in /data/cvs/php/php5cli/share/pear/MIME/Type.php on line 298
PHP Strict Standards: Assigning the return value of new by reference is
deprecated in /data/cvs/php/php5cli/share/pear/PEAR.php on line 563
PHP Strict Standards: Assigning the return value of new by reference is
deprecated in /data/cvs/php/php5cli/share/pear/PEAR.php on line 566
PHP Strict Standards: Non-static method PEAR::getStaticProperty() should
not be called statically, assuming $this from incompatible context in
/data/cvs/php/php5cli/share/pear/MIME/Type.php on line 23
Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=426
--
Sent by PEPr, the automatic proposal system at http://pear.php.net