Masatake YAMATO <[EMAIL PROTECTED]> writes: >> > I've solved 1 and 2. >> > You can force rebuild archive-tree by C-u tab. >> >> Good idea also. Perhaps more intuitive than my C-r binding. > > You bind C-r to tla--name-read-refresh-cache. > Is the behavior of tla--name-read-refresh-cache different > from passing nil as USE-CACHE to tla--archive-tree-build-revisions?
Not really. The difference here is that C-r doesn't actually do any completion, but just refresh the cache. Your interface is better actually. Anyway, refreshing the cache is something one will almost never want, and whatever binding/behaviour we define, it won't be really intuitive, and no one will read the doc to know how to do it. > BTW, tla--name-read-minibuf-map may be useful for the future extension. Yes, I was thinking of something like having a binding to chose from the bookmark list, another to chose from the partners list. This way, any function asking the user for a revision could benefit from the bookmarks and the partners features. > No problem. One more request is that the indicator; during rebuilding > archive-tree, it should be reported to the user by using `message'. Actually, I've seen that you used this feature, but I don't like it. This is the same for etags completion : You begin entering something, say "[EMAIL PROTECTED]", type TAB, and then, instead of displaying "[EMAIL PROTECTED]", it still displays "rebuilding cache for ...". I would like having this message _while_ the cache building is being made, but it should disapear just after. Perhaps I'm the one having something wrong in my configuration ? > Your code is good enough. I'd like you to move the code in xtla-read-moy.el > to > xtla-core.el; Go on ! > and remove xtla-read.el and xtla-read-moy.el. Well, there is probably some other good stuff in xtla-read.el (about scopes). I prefered restarting from scratch because I thought there was a simpler way to achieve the same result. By the way, this function could also be usefull for external code : One user may want to write a little function prompting for a revision name, his code should work by just loading xtla-core.el and xtla-defs.el. The function should have an ;;;###autoload and have a name with a single hyphen (tla-read-..., not tla--read-...) -- Matthieu
