Re: [CEDET-devel] Please try Semantic 1.4.1
Brought to you by:
zappo
From: David P. <Dav...@wa...> - 2002-12-11 09:03:37
|
Hi Eric, [...] > That's great. I'd be tempted to have the function name more > explicitly say that it doesn't load the file, perhaps > `inversion-find-version-in-file' or `inversion-find-version-no-load' > or some such. Otherwise, I think it's great to check in. Thanks! I checked the changes in. What do you think about `inversion-find-file-version', or `inversion-locate-package-version' as the function uses `locate-file' to find the main package file=3F [...] > It might also make sense to pull out parts of `inversion-test' so > that the decoded version numbers can be passed in, eliminating the > `cedet-test' mock-package. I can think on this after you check in > your changes. Yes that would be great! > I'm tempted to also recommend that we put `cedet-add-to-load-path' > into inversion, or someplace useful. Mostly so that `cedet.el' > becomes very simple. Simple good, especially if we put lots of > configuration hints into it, such as > `semantic-turn-useful-things-on'. That's a great idea to have inversion providing all the necessary tools to install packages and manage dependencies! I completely agree with you! Thanks! David |