Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bug #61681: Malformed grammar #46

Merged
merged 1 commit into from
Apr 9, 2012

Conversation

nikic
Copy link
Member

@nikic nikic commented Apr 9, 2012

This fixes https://bugs.php.net/bug.php?id=61681.

After this writing "${func()}" is possible.

Generate T_STRING_VARNAME only if it actually is one. This is only the case
for "${varname}" and "${varname[offset]}" so we can just add a check for
} or [ after the LABEL.
@php-pulls php-pulls merged commit b233de0 into php:master Apr 9, 2012
php-pulls pushed a commit that referenced this pull request Apr 18, 2012
php-pulls pushed a commit that referenced this pull request Jan 17, 2014
php-pulls pushed a commit that referenced this pull request Jan 17, 2014
php-pulls pushed a commit that referenced this pull request Jan 17, 2014
This reverts commit 333264f.
php-pulls pushed a commit that referenced this pull request Jan 17, 2014
moriyoshi pushed a commit to moriyoshi/php-src that referenced this pull request Mar 15, 2014
git-svn-id: http://svn.php.net/repository/php/php-src/trunk@319770 c90b9560-bf6c-de11-be94-00142212c4b1
php-pulls pushed a commit that referenced this pull request Jul 12, 2016
…ain names

We don't actually fix this issue wrt. the empty $info array, because it is
not clear what this array should contain and we're concerned about the
potential BC break, but at least we fix the inconsistent handling of
resulting domains with 255 bytes (which raise an error), and longer domains
(which just return FALSE), what has to be considered a very minor BC break
if at all.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants