named references test failures

[zaach]

named references test failures

[cdibbs]

Yes, these were all passing. The mods to ebnf-parser require the changes in jison. I just pulled down two fresh copies to be sure. Here is the other pull request: zaach/jison#181. Note that I (believe I) fixed the speed issues in the token stack enhancement, too, but we can get rid of that if it hinders the named references enhancement.

[zaach]

@cdibbs Yes, could we decouple the PRs? I intend to make the token stack optional either way, like other parser features.

[cdibbs]

@zaach Sorry about that. I'm still pretty new to git/github. I can work on making the token stack optional as I find time, later.

[cdibbs]

@zaach Thought I'd check in on this - were you ever able to get this working? zaach/jison#185 decouples the PRs. I looked over what I was doing, and the biggest gotcha was that I temporarily sym-linked the updated Jison module into the ebnf-parser node_modules directory before rebuilding ebnf-parser (not sure whether there's a better way to do this). Without the Jison changes at the BNF level, the EBNF stuff cannot work.

[zaach]

Oh, I didn't notice the new PR. I'll test it out locally – thanks!