Page MenuHomePhabricator

Fix bugs in python lexer
ClosedPublic

Authored by sophiebits on Jun 6 2014, 12:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 21, 12:45 AM
Unknown Object (File)
Sat, Nov 16, 3:34 PM
Unknown Object (File)
Tue, Nov 12, 2:39 AM
Unknown Object (File)
Thu, Nov 7, 9:38 PM
Unknown Object (File)
Oct 14 2024, 1:09 PM
Unknown Object (File)
Oct 9 2024, 4:12 PM
Unknown Object (File)
Sep 12 2024, 9:32 AM
Unknown Object (File)
Sep 12 2024, 9:32 AM
Subscribers

Details

Summary

Two things:

  • The appearance of \\ or \' or \-followed-by-a-newline in a raw string would improperly end the string
  • A disallowed newline in a string would cause an error instead of trying to continue anyway
Test Plan

Ran it on a file that was failing for me that was something like:

x = r'\\"'
# hello there

Diff Detail

Repository
rPHU libphutil
Branch
pylex2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 892
Build 892: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

sophiebits retitled this revision from to Fix bugs in python lexer.
sophiebits updated this object.
sophiebits edited the test plan for this revision. (Show Details)
sophiebits added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 6 2014, 12:25 AM
epriestley updated this revision to Diff 22445.

Closed by commit rPHU7e75bf271c66 (authored by @spicyj, committed by @epriestley).