On Sun, Sep 07, 2014 at 07:23:14AM -0700, Thomas Adam wrote: > The mvwm repository's new-parser branch has been updated.
I've just pushed some experimental code to the new branch dv/new-parser. Thomas, you should probably take a look at what I'm doing right now. As a first step, I have started to replace the parsing code in __execute_command_line with hook functions defined elsewhere and stored in a structure. The first milestone is having a parser that provides the current syntax. Next, I want to carefully clean up __execute_command_line so that it needs fewer hooks with saner logic. For now I want to keep compatible, but eventually the hook structure should be replaced with another one provided by the new parser. One side effect of this first commit is that the static variable func_depth is eliminated. Instead, I've defined a cmdparser_context_t that is generated in __execute_command_line and contains all the state that the cmdparser needs to record. Eventually, nobody else should look at the contents of the structure, but at the moment it just contains an (still unused) copy of the command line and the current nesting depth. This is all work in progress and subject to a lot of refactoring. Any suggestions are welcome. Note: The code is completely untested as I haven't set um mvwm yet. :-) Ciao Dominik ^_^ ^_^ -- Dominik Vogt
