Re: [emacs-wiki-discuss] planner-bookmark.el

[Dryice Liu]

Dryice Liu <address@hidden> wrote:

> Thanks Frederik for the great idea of integrating bookmark.el to
> planner. The good thing is I only need to take care of the URL
> handling, and bookmark.el will take care of the rest :)
>
> Attached is the code. Note this is my second lisp (see, I'm growing :))
> so welcome and comments :)

Hmmm, there's a good chance that there's space in the bookmark
name. So I should use planner-make-link instead make the link myself:

======================================================================
--- planner-bookmark.el.orig    Thu Dec 16 22:10:45 2004
+++ planner-bookmark.el Thu Dec 16 22:07:48 2004
@@ -84,11 +84,11 @@
                               bookmark-name))
     (if (and bookmark-annotation (string-equal bookmark-annotation ""))
        (setq bookmark-annotation nil))
-    (concat "[[bookmark://" bookmark-name "][" 
-           (if bookmark-annotation
-               bookmark-annotation
-             bookmark-name)
-           "]]")))
+    (planner-make-link
+     (concat "bookmark://" bookmark-name)
+     (if bookmark-annotation
+        bookmark-annotation
+       bookmark-name))))
 
 ;;;###autoload
 (defun planner-bookmark-browse-url (url)
======================================================================

-- 
Cheers,
Dryice

http://dryice.3322.org