On Thursday, July 24, 2003 at 16:25:44, Ronan Waide wrote: > On July 24, [EMAIL PROTECTED] said: > > Thus I propose the following change ... however this will > > break default behavior, i.e. being asked to create a new > > record for people which are already in the BBDB ... > > > > So have a look at this an vote for it or against ... > > What does your patch do that, say, using a function for > bbdb/mail-auto-create-p doesn't do?
1) The point is: IF THERE IS ALREADY AN RECORD FOR THIS PERSON! In this case bbdb/mail-auto-create-p is not honored, i.e. regardless of its value you get asked if you want to create a new record. Answer NO the the "Add address ..." (bbdb-always-add-addresses = nil) then you get asked "Create a new record for ...", but IMHO this should not be done if bbdb/mail-auto-create-p is nil. Furthermore, it should also honor PROMPT-FOR-CREATE (which my patch does not do so far). After digging in the code of bbdb-message-sender I recalled that this also annoys me from time to time ... 2) The change of bbdb-always-add-addresses's semantic allows NOT-SO-LISP-EXPERIENCED users to utilize the same hooks as for ignore-(some|all)-messages, i.e. all they need is customize. Otherwise they need to write some small wrapper defun ... or we add another wrapper, but I dare adding more code. > I don't quite see the benefit in adding yet another layer > on top of this The patch is NOT "yet another layer", but slightly corrected/enhanced/eased semantics. Bye Robert ------------------------------------------------------- This SF.Net email sponsored by: Free pre-built ASP.NET sites including Data Reports, E-commerce, Portals, and Forums are available now. Download today and enter to win an XBOX or Visual Studio .NET. http://aspnet.click-url.com/go/psa00100003ave/direct;at.aspnet_072303_01/01 _______________________________________________ [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/bbdb-info BBDB Home Page: http://bbdb.sourceforge.net/