[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#25032: 25.1; `bookmark-set-internal', `bookmark-set-no-overwrite'
From: |
Stefan Kangas |
Subject: |
bug#25032: 25.1; `bookmark-set-internal', `bookmark-set-no-overwrite' |
Date: |
Tue, 2 Jul 2019 19:07:57 +0200 |
Eli Zaretskii <eliz@gnu.org> writes:
> > +If OVERWRITE-OR-PUSH is nil, then raise an error if there is
>
> We use "signal an error".
Fixed.
> > +already a bookmark named NAME; if `overwrite', then replace any
> > +existing bookmark if there is one; if `push' then push the new
> > +bookmark onto the bookmark alist.
>
> It is generally better to precede a description of several optional
> behaviors with a short summary:
>
> OVERWRITE-OR-PUSH controls what happens if there is already a
> bookmark by that NAME: nil means signal an error, `overwrite' means
> replace the existing bookmark, `push' means ...
Fixed. Although I chose the wording "with the same name" rather than
"by that NAME", since the name could also come from the prompt.
Thanks for the review; attached an updated version.
Best regards,
Stefan Kangas
0001-lisp-bookmark.el-bookmark-set-internal-Doc-fix.-Bug-2.patch
Description: Binary data