* bug#4348: 23.1; bookmark-delete don't increment save counter when using batch arg.
@ 2009-09-05 6:59 ` Thierry Volpiatto
2009-10-09 6:50 ` bug#4348: marked as done (23.1; bookmark-delete don't increment save counter when using batch arg.) Emacs bug Tracking System
0 siblings, 1 reply; 2+ messages in thread
From: Thierry Volpiatto @ 2009-09-05 6:59 UTC (permalink / raw)
To: bug-gnu-emacs
Please write in English if possible, because the Emacs maintainers
usually do not have translators to read other languages for them.
Your bug report will be posted to the bug-gnu-emacs@gnu.org mailing list,
and to the gnu.emacs.bug news group.
Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:
Hi,
as the title say, bookmark-delete don't increment
`bookmark-alist-modification-count'
when using batch arg.
,----
| (if batch
| nil
| (bookmark-bmenu-surreptitiously-rebuild-list)
| (setq bookmark-alist-modification-count
| (1+ bookmark-alist-modification-count))
| (if (bookmark-time-to-save-p)
| (bookmark-save))))
`----
As you can see, saving code is called only in else section.
Here the redefinition i made in bookmark+.el that fix the problem:
,----
| (unless batch
| (bookmark-bmenu-surreptitiously-rebuild-list))
| (setq bookmark-alist-modification-count
| (1+ bookmark-alist-modification-count))
| (when (bookmark-time-to-save-p)
| (bookmark-save)))
`----
This bug don't appear when you delete from bmenu-list because in this
case bookmark-delete is called whithout batch arg.
But that is annoying for external programs that call bookmark-delete
with batch arg (e.g anything, icicles).
NOTE:
If you make this change, you will probably want to remove unused saving
code in `bookmark-bmenu-execute-deletions'.
If Emacs crashed, and you have the Emacs process in the gdb debugger,
please include the output from the following gdb commands:
`bt full' and `xbacktrace'.
If you would like to further debug the crash, please read the file
/usr/share/emacs/23.1/etc/DEBUG for instructions.
In GNU Emacs 23.1.1 (i686-pc-linux-gnu, GTK+ Version 2.16.5)
of 2009-07-31 on tux
Windowing system distributor `The X.Org Foundation', version 11.0.10503000
configured using `configure '--prefix=/usr' '--build=i686-pc-linux-gnu' '--host=i686-pc-linux-gnu' '--mandir=/usr/share/man' '--infodir=/usr/share/info' '--datadir=/usr/share' '--sysconfdir=/etc' '--localstatedir=/var/lib' '--program-suffix=-emacs-23' '--infodir=/usr/share/info/emacs-23' '--with-sound' '--with-x' '--without-toolkit-scroll-bars' '--with-gif' '--with-jpeg' '--with-png' '--with-rsvg' '--with-tiff' '--with-xpm' '--with-xft' '--without-libotf' '--without-m17n-flt' '--with-x-toolkit=gtk' '--without-hesiod' '--without-kerberos' '--without-kerberos5' '--with-gpm' '--with-dbus' 'build_alias=i686-pc-linux-gnu' 'host_alias=i686-pc-linux-gnu' 'CFLAGS=-march=i686 -pipe -O2' 'LDFLAGS=-Wl,-O1''
Important settings:
value of $LC_ALL: fr_FR.UTF-8
value of $LC_COLLATE: nil
value of $LC_CTYPE: nil
value of $LC_MESSAGES: nil
value of $LC_MONETARY: nil
value of $LC_NUMERIC: nil
value of $LC_TIME: nil
value of $LANG: fr_FR.UTF-8
value of $XMODIFIERS: nil
locale-coding-system: utf-8-unix
default-enable-multibyte-characters: t
Major mode: Emacs-Lisp
Minor modes in effect:
eldoc-mode: t
icomplete-mode: t
icicle-mode: t
delete-selection-mode: t
minibuffer-depth-indicate-mode: t
auto-image-file-mode: t
shell-dirtrack-mode: t
partial-completion-mode: t
show-paren-mode: t
display-battery-mode: t
display-time-mode: t
diff-auto-refine-mode: t
outline-minor-mode: t
recentf-mode: t
savehist-mode: t
desktop-save-mode: t
tooltip-mode: t
mouse-wheel-mode: t
file-name-shadow-mode: t
global-font-lock-mode: t
font-lock-mode: t
global-auto-composition-mode: t
auto-composition-mode: t
auto-encryption-mode: t
auto-compression-mode: t
line-number-mode: t
transient-mark-mode: t
Recent input:
<up> <up> <up> <up> <up> <up> <up> <up> <up> <up> <up>
<up> <up> <up> <up> <up> <up> <up> <up> <up> <up> <up>
<up> <up> <up> <up> C-c SPC <f11> a <down> <down> <down>
<down> <down> <down> <return> C-c C-g d e l e t e <down>
<down> <down> <down> <down> <down> <down> <down> <down>
<down> <down> <down> <down> <down> <down> <down> <down>
<down> <down> <down> <down> <down> <down> <down> <up>
<up> <up> <up> <return> <down> <down> <down> <down>
<down> <down> <down> <down> <down> <down> <down> <down>
<down> <down> <down> <down> <down> <down> <down> <down>
<down> <down> <down> <down> <down> <down> <down> <down>
<up> <up> <up> <up> C-SPC C-n C-n C-n C-n C-n C-n M-w
<up> <up> <up> <up> <right> <right> <right> <right>
<right> <right> <right> <right> C-u C-c C-g C-g M-x
s u b m i t <tab> <backspace> <backspace> <backspace>
<backspace> <backspace> <backspace> C-g <f5> a p b
u g g <backspace> <backspace> <backspace> <backspace>
<backspace> e m m a a c c s s SPC b b u u g g <down>
<return> <f11> e t d e f a <backspace> <backspace>
<backspace> <backspace> a l i a s SPC s e n d - b u
g - e m a c s <backspace> <backspace> <backspace> <backspace>
<backspace> <backspace> <backspace> <backspace> <backspace>
e m a c s - b u g SPC <S-up> C-x b <return> <S-down>
r e p o r t - e m a c s - b u g <return> <f11> e t
C-! s e n d - e <tab> <return> <down> <up> <f11> e
t <C-up> <backspace> <backspace> <backspace> <backspace>
<backspace> <backspace> <backspace> <backspace> <backspace>
<backspace> <backspace> <backspace> <backspace> <backspace>
<backspace> <backspace> <backspace> <return> <f11>
e t M-x r e p o r t - e m a <tab> <down> <return>
Recent messages:
Loading em-prompt...done
Loading em-script...done
Loading em-unix...done
Saving file /home/thierry/.emacs.d/.eshell/alias...
Wrote /home/thierry/.emacs.d/.eshell/alias
usage: report-emacs-bug: (TOPIC &optional RECENT-KEYS)
History item: 128
Saving file /home/thierry/.emacs.d/.eshell/alias...
Wrote /home/thierry/.emacs.d/.eshell/alias
Computing completion candidates...
--
A + Thierry Volpiatto
Location: Saint-Cyr-Sur-Mer - France
^ permalink raw reply [flat|nested] 2+ messages in thread
* bug#4348: marked as done (23.1; bookmark-delete don't increment save counter when using batch arg.)
2009-09-05 6:59 ` bug#4348: 23.1; bookmark-delete don't increment save counter when using batch arg Thierry Volpiatto
@ 2009-10-09 6:50 ` Emacs bug Tracking System
0 siblings, 0 replies; 2+ messages in thread
From: Emacs bug Tracking System @ 2009-10-09 6:50 UTC (permalink / raw)
To: Karl Fogel
[-- Attachment #1: Type: text/plain, Size: 925 bytes --]
Your message dated Fri, 09 Oct 2009 02:45:35 -0400
with message-id <8763apawvk.fsf@red-bean.com>
and subject line Fix "bookmark-delete don't increment save counter when using batch arg."
has caused the Emacs bug report #4348,
regarding 23.1; bookmark-delete don't increment save counter when using batch arg.
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@emacsbugs.donarmstrong.com
immediately.)
--
4348: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=4348
Emacs Bug Tracking System
Contact owner@emacsbugs.donarmstrong.com with problems
[-- Attachment #2: Type: message/rfc822, Size: 9301 bytes --]
From: Thierry Volpiatto <thierry.volpiatto@gmail.com>
To: bug-gnu-emacs@gnu.org
Subject: 23.1; bookmark-delete don't increment save counter when using batch arg.
Date: Sat, 05 Sep 2009 08:59:44 +0200
Message-ID: <871vml27xb.fsf@tux.homenetwork>
Please write in English if possible, because the Emacs maintainers
usually do not have translators to read other languages for them.
Your bug report will be posted to the bug-gnu-emacs@gnu.org mailing list,
and to the gnu.emacs.bug news group.
Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:
Hi,
as the title say, bookmark-delete don't increment
`bookmark-alist-modification-count'
when using batch arg.
,----
| (if batch
| nil
| (bookmark-bmenu-surreptitiously-rebuild-list)
| (setq bookmark-alist-modification-count
| (1+ bookmark-alist-modification-count))
| (if (bookmark-time-to-save-p)
| (bookmark-save))))
`----
As you can see, saving code is called only in else section.
Here the redefinition i made in bookmark+.el that fix the problem:
,----
| (unless batch
| (bookmark-bmenu-surreptitiously-rebuild-list))
| (setq bookmark-alist-modification-count
| (1+ bookmark-alist-modification-count))
| (when (bookmark-time-to-save-p)
| (bookmark-save)))
`----
This bug don't appear when you delete from bmenu-list because in this
case bookmark-delete is called whithout batch arg.
But that is annoying for external programs that call bookmark-delete
with batch arg (e.g anything, icicles).
NOTE:
If you make this change, you will probably want to remove unused saving
code in `bookmark-bmenu-execute-deletions'.
If Emacs crashed, and you have the Emacs process in the gdb debugger,
please include the output from the following gdb commands:
`bt full' and `xbacktrace'.
If you would like to further debug the crash, please read the file
/usr/share/emacs/23.1/etc/DEBUG for instructions.
In GNU Emacs 23.1.1 (i686-pc-linux-gnu, GTK+ Version 2.16.5)
of 2009-07-31 on tux
Windowing system distributor `The X.Org Foundation', version 11.0.10503000
configured using `configure '--prefix=/usr' '--build=i686-pc-linux-gnu' '--host=i686-pc-linux-gnu' '--mandir=/usr/share/man' '--infodir=/usr/share/info' '--datadir=/usr/share' '--sysconfdir=/etc' '--localstatedir=/var/lib' '--program-suffix=-emacs-23' '--infodir=/usr/share/info/emacs-23' '--with-sound' '--with-x' '--without-toolkit-scroll-bars' '--with-gif' '--with-jpeg' '--with-png' '--with-rsvg' '--with-tiff' '--with-xpm' '--with-xft' '--without-libotf' '--without-m17n-flt' '--with-x-toolkit=gtk' '--without-hesiod' '--without-kerberos' '--without-kerberos5' '--with-gpm' '--with-dbus' 'build_alias=i686-pc-linux-gnu' 'host_alias=i686-pc-linux-gnu' 'CFLAGS=-march=i686 -pipe -O2' 'LDFLAGS=-Wl,-O1''
Important settings:
value of $LC_ALL: fr_FR.UTF-8
value of $LC_COLLATE: nil
value of $LC_CTYPE: nil
value of $LC_MESSAGES: nil
value of $LC_MONETARY: nil
value of $LC_NUMERIC: nil
value of $LC_TIME: nil
value of $LANG: fr_FR.UTF-8
value of $XMODIFIERS: nil
locale-coding-system: utf-8-unix
default-enable-multibyte-characters: t
Major mode: Emacs-Lisp
Minor modes in effect:
eldoc-mode: t
icomplete-mode: t
icicle-mode: t
delete-selection-mode: t
minibuffer-depth-indicate-mode: t
auto-image-file-mode: t
shell-dirtrack-mode: t
partial-completion-mode: t
show-paren-mode: t
display-battery-mode: t
display-time-mode: t
diff-auto-refine-mode: t
outline-minor-mode: t
recentf-mode: t
savehist-mode: t
desktop-save-mode: t
tooltip-mode: t
mouse-wheel-mode: t
file-name-shadow-mode: t
global-font-lock-mode: t
font-lock-mode: t
global-auto-composition-mode: t
auto-composition-mode: t
auto-encryption-mode: t
auto-compression-mode: t
line-number-mode: t
transient-mark-mode: t
Recent input:
<up> <up> <up> <up> <up> <up> <up> <up> <up> <up> <up>
<up> <up> <up> <up> <up> <up> <up> <up> <up> <up> <up>
<up> <up> <up> <up> C-c SPC <f11> a <down> <down> <down>
<down> <down> <down> <return> C-c C-g d e l e t e <down>
<down> <down> <down> <down> <down> <down> <down> <down>
<down> <down> <down> <down> <down> <down> <down> <down>
<down> <down> <down> <down> <down> <down> <down> <up>
<up> <up> <up> <return> <down> <down> <down> <down>
<down> <down> <down> <down> <down> <down> <down> <down>
<down> <down> <down> <down> <down> <down> <down> <down>
<down> <down> <down> <down> <down> <down> <down> <down>
<up> <up> <up> <up> C-SPC C-n C-n C-n C-n C-n C-n M-w
<up> <up> <up> <up> <right> <right> <right> <right>
<right> <right> <right> <right> C-u C-c C-g C-g M-x
s u b m i t <tab> <backspace> <backspace> <backspace>
<backspace> <backspace> <backspace> C-g <f5> a p b
u g g <backspace> <backspace> <backspace> <backspace>
<backspace> e m m a a c c s s SPC b b u u g g <down>
<return> <f11> e t d e f a <backspace> <backspace>
<backspace> <backspace> a l i a s SPC s e n d - b u
g - e m a c s <backspace> <backspace> <backspace> <backspace>
<backspace> <backspace> <backspace> <backspace> <backspace>
e m a c s - b u g SPC <S-up> C-x b <return> <S-down>
r e p o r t - e m a c s - b u g <return> <f11> e t
C-! s e n d - e <tab> <return> <down> <up> <f11> e
t <C-up> <backspace> <backspace> <backspace> <backspace>
<backspace> <backspace> <backspace> <backspace> <backspace>
<backspace> <backspace> <backspace> <backspace> <backspace>
<backspace> <backspace> <backspace> <return> <f11>
e t M-x r e p o r t - e m a <tab> <down> <return>
Recent messages:
Loading em-prompt...done
Loading em-script...done
Loading em-unix...done
Saving file /home/thierry/.emacs.d/.eshell/alias...
Wrote /home/thierry/.emacs.d/.eshell/alias
usage: report-emacs-bug: (TOPIC &optional RECENT-KEYS)
History item: 128
Saving file /home/thierry/.emacs.d/.eshell/alias...
Wrote /home/thierry/.emacs.d/.eshell/alias
Computing completion candidates...
--
A + Thierry Volpiatto
Location: Saint-Cyr-Sur-Mer - France
[-- Attachment #3: Type: message/rfc822, Size: 1684 bytes --]
From: Karl Fogel <kfogel@red-bean.com>
To: 4348-close@emacsbugs.donarmstrong.com
Subject: Fix "bookmark-delete don't increment save counter when using batch arg."
Date: Fri, 09 Oct 2009 02:45:35 -0400
Message-ID: <8763apawvk.fsf@red-bean.com>
fixed
--
Thanks for the report. Fix committed:
$ cvs ci -F msg lisp/bookmark.el lisp/ChangeLog
/sources/emacs/emacs/lisp/bookmark.el,v <-- lisp/bookmark.el
new revision: 1.133; previous revision: 1.132
/sources/emacs/emacs/lisp/ChangeLog,v <-- lisp/ChangeLog
new revision: 1.16394; previous revision: 1.16393
Mailing notification to emacs-diffs@gnu.org... sent.
@/home/kfogel/src/emacs/emacs-cvs>
$
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-10-09 6:50 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <8763apawvk.fsf@red-bean.com>
2009-09-05 6:59 ` bug#4348: 23.1; bookmark-delete don't increment save counter when using batch arg Thierry Volpiatto
2009-10-09 6:50 ` bug#4348: marked as done (23.1; bookmark-delete don't increment save counter when using batch arg.) Emacs bug Tracking System
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).