Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 3.7.2

Request #14237 Line length should be measured by mb_strwidth
Submitted: 2008-06-25 16:18 UTC
From: miau Assigned: squiz
Status: Closed Package: PHP_CodeSniffer (version 1.1.0RC2)
PHP Version: 5.2.5 OS:
Roadmaps: (Not assigned)    
Subscription  


 [2008-06-25 16:18 UTC] miau (Koji Ando)
Description: ------------ While most multi-byte characters are reprenseted by 3 octet in UTF-8, they take up 2 character widths. Test script: --------------- >cat php.ini | grep mbstring.internal_encoding mbstring.internal_encoding = UTF-8 ; mbstring.internal_encoding setting. Input chars are ; given by corresponding encode setting. When empty mbstring.internal_encoding >php -i | grep mbstring.internal_encoding mbstring.internal_encoding => UTF-8 => UTF-8 >phpcs --standard=Zend multi_byte_sample.php Expected result: ---------------- 2 | WARNING | Line exceeds 80 characters; contains 81 characters Actual result: -------------- 2 | WARNING | Line exceeds 80 characters; contains 86 characters

Comments

 [2010-09-10 04:56 UTC] squiz (Greg Sherwood)
-Status: Assigned +Status: Closed
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.