Closed
Bug 1316981
Opened 8 years ago
Closed 7 years ago
FTLParser should setPosition on AST nodes
Categories
(L20n :: Python Library, defect)
L20n
Python Library
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: Pike, Unassigned)
References
Details
Attachments
(1 file, 1 obsolete file)
Right now, the primary use-case for the python parser is tooling, and for that, having source positions are crucial. We should setPosition() on AST nodes in the parser. Maybe doing that optionally. There's a with_source in the migration branch by stas, maybe that's the same switch
Reporter | ||
Comment 1•8 years ago
|
||
Gandalf did a first pass on this that we merged with https://github.com/l20n/python-l20n/pull/7. Sadly, that's not working out great, I'll follow up with a PR that adds tests, which fail on value parsing. There's a trailing whitespace problem, start of values with placables start as that start of the placable, not the value, trait keys have no space. The test isn't going very deep yet, it's mostly covering what I'd need to integrate compare-locales and elmo, which is very entity focuses, and ignores sections and comments.
Reporter | ||
Comment 2•8 years ago
|
||
Reporter | ||
Updated•8 years ago
|
Attachment #8812575 -
Attachment is obsolete: true
Reporter | ||
Comment 3•7 years ago
|
||
Resolving this as WORKSFORME, as I think we have all the positions (for the ASTs we have) now, but probably did that differently than this bug expected.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•