Description: ------------ 1. Create a JS file "test.js" with just the contents "[]" (without the quotes, without any whitespace). 2. Check that file using any Standard, e.g. the Generic standard Test script: --------------- [] Expected result: ---------------- Only the time and memory get displayed. Actual result: -------------- klee@dmok:~$ phpcs --standard=Generic /tmp/test.js PHP Notice: Undefined offset: 2 in /usr/share/php/PHP/CodeSniffer/Tokenizers/JS.php on line 444 Time: 0 seconds, Memory: 2.25Mb This is the offending line: if (in_array(strtolower($buffer), $tokenTypes) === true && (preg_match('|[a-zA-z0-9_]|', $char) === 0 || preg_match('|[a-zA-z0-9_]|', $chars[($i + 1)]) === 0) The $i + 1 is the offset that is too big.

I can't replicate this issue. Can you please use the -vv command line argument to generate the tokenizer output. It should look like this: 94:trunk gsherwood$ php scripts/phpcs --standard=Generic temp.js -vv Registering sniffs in Generic standard... DONE (36 sniffs registered) Processing temp.js *** START TOKENIZING *** Process char 0 => [ (buffer: ) * char is token, looking ahead 8 chars * => Looking ahead 1 chars => [] => Looking ahead 2 chars => []\n * look ahead found nothing * => Added token T_OPEN_SQUARE_BRACKET ([) Process char 1 => ] (buffer: ) * buffer possibly contains token, looking ahead 8 chars * => Looking ahead 1 chars => ]\n * look ahead found nothing * => Added token T_CLOSE_SQUARE_BRACKET (]) Process char 2 => \n (buffer: ) *** END TOKENIZING *** *** START BRACKET MAP *** => Found square bracket opener at 1 => Found square bracket closer at 2 for 1 *** END BRACKET MAP *** *** START SCOPE MAP *** *** END SCOPE MAP *** *** START LEVEL MAP *** Process token 0 on line 1 [lvl:0;]: T_OPEN_TAG => Process token 1 on line 1 [lvl:0;]: T_OPEN_SQUARE_BRACKET => [ Process token 2 on line 1 [lvl:0;]: T_CLOSE_SQUARE_BRACKET => ] Process token 3 on line 1 [lvl:0;]: T_CLOSE_TAG => *** END LEVEL MAP *** *** START ADDITIONAL JS PROCESSING *** Process token 0: T_OPEN_TAG => Process token 1: T_OPEN_SQUARE_BRACKET => [ Process token 2: T_CLOSE_SQUARE_BRACKET => ] Process token 3: T_CLOSE_TAG => *** END ADDITIONAL JS PROCESSING *** [4 tokens in 1 lines]... DONE in < 1 second (0 errors, 0 warnings) Time: 0 seconds, Memory: 4.25Mb But I'll be looking for any differences of course.

klee@dmok:~$ phpcs --standard=Generic -vv Downloads/test.js Registering sniffs in Generic standard... DONE (35 sniffs registered) Processing test.js *** START TOKENIZING *** Process char 0 => [ (buffer: ) * char is token, looking ahead 8 chars * => Looking ahead 1 chars => [] * look ahead found nothing * => Added token T_OPEN_SQUARE_BRACKET ([) Process char 1 => ] (buffer: ) PHP Notice: Undefined offset: 2 in /usr/share/php/PHP/CodeSniffer/Tokenizers/JS.php on line 444 * buffer possibly contains token, looking ahead 8 chars * * look ahead found nothing * => Added token T_CLOSE_SQUARE_BRACKET (]) *** END TOKENIZING *** *** START BRACKET MAP *** => Found square bracket opener at 1 => Found square bracket closer at 2 for 1 *** END BRACKET MAP *** *** START SCOPE MAP *** *** END SCOPE MAP *** *** START LEVEL MAP *** Process token 0 on line 1 [lvl:0;]: T_OPEN_TAG => Process token 1 on line 1 [lvl:0;]: T_OPEN_SQUARE_BRACKET => [ Process token 2 on line 1 [lvl:0;]: T_CLOSE_SQUARE_BRACKET => ] Process token 3 on line 1 [lvl:0;]: T_CLOSE_TAG => *** END LEVEL MAP *** *** START ADDITIONAL JS PROCESSING *** Process token 0: T_OPEN_TAG => Process token 1: T_OPEN_SQUARE_BRACKET => [ Process token 2: T_CLOSE_SQUARE_BRACKET => ] Process token 3: T_CLOSE_TAG => *** END ADDITIONAL JS PROCESSING *** [4 tokens in 1 lines]... DONE in < 1 second (0 errors, 0 warnings) Time: 0 seconds, Memory: 2.25Mb

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.