>>>>> "Ben" == Ben Wing <ben(a)666.com&gt; writes: Ben> Again, please please follow standard conventions. The lower-level Ben> functions should be called ldap-add, ldap-delete, etc. (Or even Ben> better, use ldap-put, ldap-remove.) No -internal, -basic-, -1, Ben> whatever. These are parallel to the database routines. Ben> I'm still not completely convinced we need convenience wrappers. Ben> The db interface 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 Ben> create-ldap-entry, or similar names -- i.e. longer, to convey that Ben> they do more than you might 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.