It seems you didn't read all of my last email (it was a bit rambling).
Here's the conclusion:

So testing done for nil is correct, and that now works for me; I don't
understand why I had to change it! I must have changed some other
setting that was the real culprit; probably using (bbdb-complete-mail
nil t).

I suggest adding the following comment:

    ;; Consider cycling
    (when (and (not done) bbdb-complete-mail-allow-cycling)
      ;; The pattern has matched no records; it is most likely the
      ;; result of a previous match, and we want to cycle to the next
      ;; email address.
      ;;

Otherwise, sorry for the noise.

Patch, adding this comment, and fixing the kill non-existing window bug,
below.

--
-- Stephe
--- a/lisp/bbdb-com.el
+++ b/lisp/bbdb-com.el
@@ -1801,8 +1801,8 @@ as part of the MUA insinuation."
     ;; Clean up *Completions* buffer, if it exists
     (when bbdb-complete-mail-saved-window-config
       (let ((window (get-buffer-window "*Completions*")))
+       (set-window-configuration bbdb-complete-mail-saved-window-config)
         (when (window-live-p window)
-          (set-window-configuration bbdb-complete-mail-saved-window-config)
           (quit-window nil window)))
       (setq bbdb-complete-mail-saved-window-config nil))
 
@@ -1913,6 +1913,10 @@ as part of the MUA insinuation."
 
     ;; Consider cycling
     (when (and (not done) bbdb-complete-mail-allow-cycling)
+      ;; The pattern has matched no records; it is most likely the
+      ;; result of a previous match, and we want to cycle to the next
+      ;; email address.
+      ;;
       ;; find the record we are working on.
       (let* ((address (mail-extract-address-components orig))
              (record (and (listp address)


-- 
-- Stephe

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
bbdb-info@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bbdb-info
BBDB Home Page: http://bbdb.sourceforge.net/

Reply via email to