Hi,
The user API looks ok, however the way you change add-binding removes safety checks: several uses must have the same default initialization and type, etc. This could cause future extensions to break.
I don't really see what you mean here (newbie warning repeated :)
I modified add-binding to accept an optional keyword param,
Add-binding was previously used only via make-binding-internal. Your patch bypasses all the checks that the make-* functions do, and the functionality they provide. Is that clearer?
Maybe a make-shared-binding is what's missing? Indeed something seems missing, e.g. "create only if not existing already, but I don't know the name of the possibly existing binding".
- attila (alias 101 on irc &no 'its not lisp code :) I was introduced to IRC in the late eighties but never felt comfortable with it: Too many concurrent discussions, despite threading GUI clients, and lots of bots & kicking. I very much prefer usenet news.
Regards, Jörg Höhle.