Milan Zamazal <[EMAIL PROTECTED]> writes: > The naming of the functions is a bit confusing. Note that `tla rm' > removes the file and its id, while `tla delete' removes only the > inventory id. In xtla, tla-inventory-remove removes the id and > optionally the file and tla-inventory-delete removes the file and > optionally its id. So the meaning is basically opposite to the tla > commands.
We should exchange the names of the two functions. > How about changing the name of tla-inventory-delete to > tla-inventory-delete-files? That's a good idea. We should look at all tla-inventory-* functions that take the marked files as input and rename them that way. > Another problem with tla-inventory-remove is that it is bound to `- f', > which conflicts with the view-mode keybindings (`-' is bound to numeric > prefix argument there). I apologize for raising up the keybinding > issues here again :-), but I think it would be much better to use > `SOME-PREFIX +' and `SOME-PREFIX -' for tla-inventory-add and > tla-inventory-remove respectively. What about ?a to add and ?r to remove - these are also available in PCL-CVS -- Stefan.
