#2589 closed task (fixed)
Remove CParser
Reported by: | Josh | Owned by: | leper |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 18 |
Component: | Core engine | Keywords: | |
Cc: | Patch: |
Description
CParser is slow and of poor quality. It should be phased out in favor of individual dedicated parsers.
Ticket #2005 previously tried to replace CParser, but it resulted in something nearly as bad.
Attachments (1)
Change History (11)
by , 10 years ago
Attachment: | no-cparser-GUI.diff added |
---|
comment:1 by , 10 years ago
no-cparser-GUI.diff was committed in r15213:
"Start phasing out the use of CParser in the GUI. Makes GUI parsing generally faster and stricter while adding better tests and debug information."
Only remaining use of CParser in /source/gui is the text parser.
comment:4 by , 10 years ago
Milestone: | Alpha 17 → Alpha 18 |
---|
comment:5 by , 9 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
I have local changes for TerrainProperties.cpp
and a rewrite of the parsing logic in GuiText.cpp
which remove the use of CParser in there. The last remaining use of it is in ConfigDB which I'll tackle next.
A start at replacing cparser in the GUI code.