]> granicus.if.org Git - php/commit
Syntax errors caused by unclosed {, [, ( mention specific location
authorAlex Dowad <alexinbeijing@gmail.com>
Wed, 8 Apr 2020 11:19:39 +0000 (13:19 +0200)
committerNikita Popov <nikita.ppv@gmail.com>
Tue, 14 Apr 2020 09:22:23 +0000 (11:22 +0200)
commit80598f12507c5cbde04163289e4d2575f05d2a0c
treef5d3c26400dcb577e6c508469d1de086f5fda695
parentd4471c6aaecab2f1cc467ea8e92040c73685bd21
Syntax errors caused by unclosed {, [, ( mention specific location

Aside from a few very specific syntax errors for which detailed exceptions are
thrown, generally PHP just emits the default error messages generated by bison on syntax
error. These messages are very uninformative; they just say "Unexpected ... at line ...".

This is most problematic with constructs which can span an arbitrary number of lines, such
as blocks of code delimited by { }, 'if' conditions delimited by ( ), and so on. If a closing
delimiter is missed, the block will run for the entire remainder of the source file (which
could be thousands of lines), and then at the end, a parse error will be thrown with the
dreaded words: "Unexpected end of file".

Therefore, track the positions of opening and closing delimiters and ensure that they match
up correctly. If any mismatch or missing delimiter is detected, immediately throw a parse
error which points the user to the offending line. This is best done in the *lexer* and not
in the parser.

Thanks to Nikita Popov and George Peter Banyard for suggesting improvements.

Fixes bug #79368.
Closes GH-5364.
NEWS
Zend/tests/bug60099.phpt
Zend/tests/eval_parse_error_with_doc_comment.phpt
Zend/tests/require_parse_exception.phpt
Zend/zend_globals.h
Zend/zend_language_scanner.h
Zend/zend_language_scanner.l
ext/tokenizer/tests/token_get_all_variation17.phpt
ext/tokenizer/tokenizer.c
tests/lang/syntax_errors.phpt [new file with mode: 0644]