Page MenuHomePhabricator
Differential D7323

Fix a lexer issue with SimpleOptions and backslashes in unterminated quotes
ClosedPublic
Actions

Authored by epriestley on Oct 15 2013, 1:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 18, 9:01 PM
Unknown Object (File)
Wed, Jan 15, 8:44 AM
Unknown Object (File)
Sun, Jan 12, 5:49 PM
Unknown Object (File)
Mon, Dec 30, 11:23 PM
Unknown Object (File)
Mon, Dec 30, 3:45 PM
Unknown Object (File)
Dec 13 2024, 10:53 AM
Unknown Object (File)
Dec 12 2024, 3:05 PM
Unknown Object (File)
Dec 8 2024, 11:21 PM
View All Files
Subscribers
aran
SEJeff

Details

Reviewers
btrahan
SEJeff
Commits
rPHUaa45dfcc2c33: Fix a lexer issue with SimpleOptions and backslashes in unterminated quotes
Summary

The SimpleOptions lexer has an error when parsing a line like a="\, where the lexer can't accept the backslash inside the unterminated quotes.

Test Plan

Added failing unit tests; they now pass.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

SEJeff accepted this revision.Oct 15 2013, 4:00 PM

Pulled in and confirmed this fixes the bug

epriestley closed this revision.Oct 15 2013, 4:03 PM

Closed by commit rPHUaa45dfcc2c33 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
lexer/
1 line
2 lines
parser/
__tests__/
19 lines

Diff 16493

View Options

src/lexer/PhutilLexer.php

Loading...
View Options

src/lexer/PhutilSimpleOptionsLexer.php

Loading...
View Options

src/parser/__tests__/PhutilSimpleOptionsTestCase.php

Loading...
Phabricator · Built by Phacility