1. Line 1898:
,
|(let ((bbdb-inside-electric-display t)
`
in line 1890 it's defined as constant:
,
| (defconst bbdb-inside-electric-display nil)
`
byte-compiling error:
,
| In electric-bbdb-display-records:
| bbdb.el:1898:11:Warning: attempt to let-bind constant
| `bb
Line 91:
,
| (replace-string "\r\n" "\n")
`
byte-compiling error:
,
| In bbdb-whois-sentinel:
| bbdb-whois.el:94:45:Warning: `replace-string' used from Lisp code
| That command is designed for interactive use only
`
should it be changed to replace-match?
I use latest cvs source, and found following errors:
1. Line 2748:
(modify-syntax-entry ?* "w" mail-mode-header-syntax-table)
byte-compiling error:
In bbdb-define-all-aliases:
bbdb-com.el:2748:35:Warning: reference to free variable
`mail-mode-header-syntax-table'
in emacs/lisp/ChangeL
> "Ronan" == Ronan Waide <[EMAIL PROTECTED]> writes:
Ronan> On September 22, [EMAIL PROTECTED] said:
>>
>> I have a lot of VM folders for my email, and VM provides the
>> capability to set up an alist for filing. So I've added a
>> mail-folder field to my BBDB database, an
bbdb.el, line 1542:
,
| (insert (replace-in-string notes "[\r\n\t ]+" " "))
`
should this be:
,
| (insert (bbdb-replace-in-string notes "[\r\n\t ]+" " "))
`
?
replace-in-string is xemacs-specific function.
--
