Re: [21.2] LDAP add and delete functions + interrupt fixes

Ben> Again, please please follow standard conventions. Ben> The lower-level functions should be called ldap-add, ldap-delete, etc. (Or even Ben> better, use ldap-put, ldap-remove.) No -internal, -basic-, -1, whatever. These are Ben> parallel to the database routines. Ben> I'm still not completely convinced we need convenience wrappers. The db interface Ben> doesn't have them, for example. IIRC someone said on that list nobody actually uses the db interface. That might explain why we don't need those wrappers. LDAP on the other hand is already used by three packages. Ben> If you want them, though, call them something like create-ldap-entry, or Ben> similar names -- i.e. longer, to convey that they do more than you might Ben> expect. Though I'm not convinced this renaming is good, I'll send the corresponding patch, you probably know better than I what's good for XEmacs. I still think someone used to LDAP will find its way more easily through the current high-level API which reminds the traditional LDAP command-line tools rather than through the naming you propose. Someone not used to LDAP won't care that much. Ben> Also, the message displaying should be optional, and not the default. I agree. Oscar