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

Compiler does not escape slashes in strings #1495

Closed
sjinks opened this issue Apr 12, 2017 · 1 comment
Assignees
Labels
Projects

Comments

@sjinks
Copy link
Contributor

@sjinks sjinks commented Apr 12, 2017

/workspace/zephir/ext/test/declaretest.zep.c: In function ‘zim_Test_DeclareTest_testStringDeclare1’:
/workspace/zephir/ext/test/declaretest.zep.c:40:76: warning: unknown escape sequence: '\w'
  ZVAL_STRING(&a, "/@(\w+)(?:\s*(?:\(\s*)?(.*?)(?:\s*\))?)??\s*(?:\n|\*\/)/");
                                                                            ^
/workspace/zephir/ext/test/declaretest.zep.c:40:76: warning: unknown escape sequence: '\s'
/workspace/zephir/ext/test/declaretest.zep.c:40:76: warning: unknown escape sequence: '\s'
/workspace/zephir/ext/test/declaretest.zep.c:40:76: warning: unknown escape sequence: '\s'
/workspace/zephir/ext/test/declaretest.zep.c:40:76: warning: unknown escape sequence: '\)'
/workspace/zephir/ext/test/declaretest.zep.c:40:76: warning: unknown escape sequence: '\s'
/workspace/zephir/ext/test/declaretest.zep.c:40:76: warning: unknown escape sequence: '\*'
/workspace/zephir/ext/test/declaretest.zep.c:40:76: warning: unknown escape sequence: '\/'
/workspace/zephir/ext/test/declaretest.zep.c: In function ‘zim_Test_DeclareTest_testStringDeclare2’:
/workspace/zephir/ext/test/declaretest.zep.c:55:74: warning: unknown escape sequence: '\w'
  ZVAL_STRING(&a, "/(\w+)\s*=\s*(\[[^\]]*\]|\"[^\"]*\"|[^,)]*)\s*(?:,|$)/");
                                                                          ^
/workspace/zephir/ext/test/declaretest.zep.c:55:74: warning: unknown escape sequence: '\s'
/workspace/zephir/ext/test/declaretest.zep.c:55:74: warning: unknown escape sequence: '\s'
/workspace/zephir/ext/test/declaretest.zep.c:55:74: warning: unknown escape sequence: '\]'
/workspace/zephir/ext/test/declaretest.zep.c:55:74: warning: unknown escape sequence: '\]'
/workspace/zephir/ext/test/declaretest.zep.c:55:74: warning: unknown escape sequence: '\s'
@sjinks sjinks added bug minor labels Apr 12, 2017
@danhunsaker danhunsaker added this to Needs triage in Zephir Bugs Dec 12, 2018
@sergeyklay sergeyklay moved this from Needs triage to Low priority in Zephir Bugs Feb 19, 2019
@AlexNDRmac AlexNDRmac self-assigned this Oct 12, 2019
@AlexNDRmac AlexNDRmac referenced this issue Oct 13, 2019
2 of 3 tasks complete
@sergeyklay

This comment has been minimized.

Copy link
Member

@sergeyklay sergeyklay commented Oct 13, 2019

Fixed in the development branch. Feel free to open a new issue if the problem appears again. Thank you for the bug report.

@sergeyklay sergeyklay closed this Oct 13, 2019
Zephir Bugs automation moved this from Low priority to Closed Oct 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Zephir Bugs
  
Closed
3 participants
You can’t perform that action at this time.