[PEPr] Comment on PHP::PHP_CodeSniffer
| From: | Laurent Laville | Date: | Tue, 25 Jul 2006 08:26:08 +0000 |
| Subject: | [PEPr] Comment on PHP::PHP_CodeSniffer | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-43562@lists.php.net to get a copy of this message | ||
Laurent Laville (http://pear.php.net/user/farell) has commented on the proposal for
PHP::PHP_CodeSniffer.
Comment:
Another comment related to Windows Platform.
I've run version 0.0.2 on command line with :
C:\wamp\php>php -d include_path=".;c:\wamp\php\pear" -f phpcs
C:\php\pear\HTML_P
rogress2
and i got lot of lines like on this file
FILE: C:\php\pear\HTML_Progress2\examples\circle\circle.php
--------------------------------------------------------------------------------
FOUND 4 ERROR(S) AND 0 WARNING(S) AFFECTING 4 LINE(S)
--------------------------------------------------------------------------------
[LINE 26] ERROR: Format error: expected "\n" found "
\n"
[LINE 29] ERROR: Format error: expected "\n" found "
\n"
[LINE 31] ERROR: Format error: expected "\n" found "
\n"
[LINE 35] ERROR: Format error: expected "\n" found "
\n"
I think it could be a windows CR not manage.
Perharps it should be into file
PHP\CodeSniffer\Standards\AbstractPatternSniff.php
method : processViolation
Tell me if i'm wrong (bad usage) or if i'm right (windows platform
problem).
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