On Sat, Dec 30 2006, Patrick Campbell-Preston wrote:
> The primep function in bbdb.el has been broken for a very long while -
> try (primep 4) or (primep -1) for example. This is harmless for
> bbdb's own use of the function, as it happens, but not for the casual
> user [amusing story omitted].
>
> Here's a fixed version, perhaps someone with CVS access and copious
> free time could paste it in (before the next BBDB release):
>
> (defun primep (num)
Wouldn't it be better to rename it to `bbdb-primep'?
,----[ (info "(elisp)Coding Conventions") ]
| * If a package needs to define an alias or a new function for
| compatibility with some other version of Emacs, name it with the
| package prefix, not with the raw name with which it occurs in the
| other version. Here is an example from Gnus, which provides many
| examples of such compatibility issues.
|
| (defalias 'gnus-point-at-bol
| (if (fboundp 'point-at-bol)
| 'point-at-bol
| 'line-beginning-position))
`----
[ BTW, in Gnus we just had spurious problems because
xemacs-base/easy-mmode.el defines `replace-regexp-in-string'. :-/ ]
Bye, Reiner.
--
,,,
(o o)
---ooO-(_)-Ooo--- | PGP key available | http://rsteib.home.pages.de/
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
bbdb-info@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bbdb-info
BBDB Home Page: http://bbdb.sourceforge.net/
