Thread: [CEDET-devel] Re: semantic-flex
Brought to you by:
zappo
From: David P. <dav...@wa...> - 2004-02-17 14:09:18
|
[...] > `semantic-lex' function calls the variable `semantic-lex-analyzer', > so that would be recursive. > > Perhaps it should be `semantic-simple-lexer' instead. Very good point! As it looks that all the language setups explicitly set `semantic-lex-analyzer', perhaps the default value could be nil? For now, I will commit the fixes to document.el. Thanks! David |
From: Eric M. L. <er...@si...> - 2004-02-17 14:31:11
|
>>> David PONCE <dav...@wa...> seems to think that: >[...] >> `semantic-lex' function calls the variable `semantic-lex-analyzer', >> so that would be recursive. >> >> Perhaps it should be `semantic-simple-lexer' instead. > >Very good point! As it looks that all the language setups explicitly >set `semantic-lex-analyzer', perhaps the default value could be nil? > >For now, I will commit the fixes to document.el. [ ... ] Originally there was no need to set the lexer to any value which is why there is that default. It might make sense to have `define-lex' accept an optional major-mode and auto-install it as an override method. Thanks Eric -- Eric Ludlam: za...@gn..., er...@si... Home: http://www.ludlam.net Siege: www.siege-engine.com Emacs: http://cedet.sourceforge.net GNU: www.gnu.org |