Page MenuHomePhabricator

Further refine the Java syntax lexer for escaped strings
ClosedPublic

Authored by epriestley on Oct 25 2018, 4:32 PM.
Tags
None
Referenced Files
F13061826: D19763.diff
Fri, Apr 19, 8:03 PM
F13056396: D19763.id47215.diff
Fri, Apr 19, 1:29 PM
Unknown Object (File)
Sun, Apr 7, 9:12 PM
Unknown Object (File)
Sun, Mar 31, 4:52 AM
Unknown Object (File)
Fri, Mar 29, 1:51 AM
Unknown Object (File)
Mar 20 2024, 10:31 PM
Unknown Object (File)
Mar 20 2024, 10:31 PM
Unknown Object (File)
Mar 20 2024, 10:31 PM
Subscribers
None

Details

Summary

Ref T13210. See PHI886. Previously, D19731 improved some behavior here, but didn't go quite far enough and we still got strings with \" wrong.

Parse backslash followed by anything, or any character except double quote or backslash.

(If this is still somehow wrong I can just copy one of the other lexers which is a little more complicated but gets this right.)

Test Plan

Screen Shot 2018-10-25 at 9.30.40 AM.png (591×275 px, 26 KB)

Diff Detail

Repository
rPHU libphutil
Branch
java2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21043
Build 28595: Run Core Tests
Build 28594: arc lint + arc unit