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.

How about changing the name of tla-inventory-delete to
tla-inventory-delete-files?

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.

Regards,

Milan Zamazal

-- 
SomeProgrammersLikeWritingLikeThis.However,IDontThinkThisFormOfCommunicationIs\
AGoodIdea.IApologizeToAllWhoCantReadMyTextsWrittenInATraditionalStyle.

Reply via email to