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
Unknown Object (File)
Tue, Feb 4, 2:19 PM
Unknown Object (File)
Fri, Jan 31, 12:49 AM
Unknown Object (File)
Tue, Jan 28, 4:18 AM
Unknown Object (File)
Tue, Jan 28, 4:18 AM
Unknown Object (File)
Tue, Jan 28, 4:18 AM
Unknown Object (File)
Sat, Jan 25, 3:46 PM
Unknown Object (File)
Thu, Jan 23, 2:04 AM
Unknown Object (File)
Mon, Jan 20, 2:34 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