Skip to content
This repository has been archived by the owner on Jun 4, 2019. It is now read-only.

sgrep - LEXER:unrecognised symbol, in st_start_heredoc rule:{ #22

Closed
joes opened this issue Feb 22, 2012 · 3 comments
Closed

sgrep - LEXER:unrecognised symbol, in st_start_heredoc rule:{ #22

joes opened this issue Feb 22, 2012 · 3 comments

Comments

@joes
Copy link

joes commented Feb 22, 2012

This is in a php-file without namespaces and that validates fine with 'php -l'

(ONCE) LEXER:unrecognised symbol, in st_start_heredoc rule:{
warning: parsing problem in ./include/rplAsync/rplAsyncJavascriptProxy.php

Does this mean that sgrep doesn't support heredoc syntax?

@aryx
Copy link
Contributor

aryx commented Mar 13, 2012

I just pushed a fix.
Thx for the bug report.

@aryx aryx closed this as completed Mar 13, 2012
@joes
Copy link
Author

joes commented Mar 14, 2012

I was looking at the file tests/php/parsing/heredoc3.php in 4692d1a and did not see the token T_DOLLAR_OPEN_CURLY_BRACES ('${') there. Would pfff still choke on that kind of token within heredocs or was this covered by your recent commit too?

@aryx
Copy link
Contributor

aryx commented Mar 16, 2012

I've added a new test. Yes it handles ${.
930721c
Thx.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants