GNU bug report logs - #76789
31.0.50; [PATCH] speedbar: New speedbar-window-mode

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#76789: closed (31.0.50; [PATCH] speedbar: New speedbar-window-mode)
Date: Sat, 15 Mar 2025 08:55:02 +0000

[Message part 1 (text/plain, inline)]

Your message dated Sat, 15 Mar 2025 10:54:39 +0200
with message-id <86v7say8q8.fsf <at> gnu.org>
and subject line Re: bug#76789: 31.0.50; [PATCH] speedbar: New speedbar-window-mode
has caused the debbugs.gnu.org bug report #76789,
regarding 31.0.50; [PATCH] speedbar: New speedbar-window-mode
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
76789: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=76789
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Vincenzo Pupillo <v.pupillo <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 31.0.50; [PATCH] speedbar: New speedbar-window-mode
Date: Thu, 06 Mar 2025 21:42:12 +0100

[Message part 3 (text/plain, inline)]

Ciao,
this patch adds speedbar-window-mode as an alternative to the default 
speedbar-frame-mode. Calling speedbar-window-mode will close speedbar-frame-
mode if it is open, and vice versa.
speedbar-window-mode, thanks to display-buffer-in-side-window, can be placed 
on the left side, right side but also on the top or bottom. 

Thank you.

Vincenzo

[0001-Add-a-new-command-speedbar-window.patch (text/x-patch, attachment)]

[Message part 5 (message/rfc822, inline)]

From: Eli Zaretskii <eliz <at> gnu.org>
To: Vincenzo Pupillo <v.pupillo <at> gmail.com>
Cc: 76789-done <at> debbugs.gnu.org
Subject: Re: bug#76789: 31.0.50; [PATCH] speedbar: New speedbar-window-mode
Date: Sat, 15 Mar 2025 10:54:39 +0200

> From: Vincenzo Pupillo <v.pupillo <at> gmail.com>
> Cc: 76789 <at> debbugs.gnu.org
> Date: Fri, 14 Mar 2025 20:52:33 +0100
> 
> Thank you Eli for the review,
> 
> In data giovedì 13 marzo 2025 10:57:41 Ora standard dell’Europa centrale, Eli 
> Zaretskii ha scritto:
> > > From: Vincenzo Pupillo <v.pupillo <at> gmail.com>
> > > Cc: 76789 <at> debbugs.gnu.org
> > > Date: Tue, 11 Mar 2025 16:41:22 +0100
> > > 
> > > Thank you Eli.
> > > In this updated version of the patch I corrected what you indicated and
> > > added something in the manual, rather short :-(.
> > 
> > Thanks.  I meant to install it, but building Emacs after applying the
> > changes yields warnings:
> > 
> >   In speedbar-frame-or-window:
> >   speedbar.el:1009:2: Warning: docstring has wrong usage of unescaped single
> > quotes (use \=' or different quoting such as `...')
> > 
> >   In speedbar-window-mode:
> >   speedbar.el:1045:13: Warning: assignment to free variable
> > `speedbar-select-frame-method'
> > 
> >   In speedbar-window--buffer-live-p:
> >   speedbar.el:1079:11: Warning: defsubst `speedbar-window--buffer-live-p'
> > was used before it was defined
> > 
> >   In speedbar-window--width:
> >   speedbar.el:1113:11: Warning: defsubst `speedbar-window--width' was used
> > before it was defined
> > 
> >   In speedbar-current-frame:
> >   speedbar.el:1135:11: Warning: defsubst `speedbar-current-frame' was used
> > before it was defined
> > 
> > Please fix those.
> 
> Done
> 
> > 
> > Also, here:
> > > +@cindex @code{speedbar-window}
> > > +Optionally, the speedbar can be displayed as a window, splitting the
> > > +windows of the selected frame, in both terminal and graphics modes.
> > > +Only one speedbar window can be open at a time.
> > > +
> > > +It is possible to switch from displaying the speedbar in a separate
> > > +frame to displaying it in a window and vice versa simply by using the
> > > +@kbd{M-x speedbar-window @key{RET}} or
> > > +@kbd{M-x speedbar-frame <at> key{RET}} command.
> > 
> > please mention and index the user option you added.
> > 
> > > +** Speedbar
> > > +
> > > +---
> > > +*** The new command 'speedbar-window-mode' opens Speedbar in a window
> > > instead +of a frame.
> > > +
> > > +*** New command 'speedbar-window' is an alias for 'speedbar-window-mode'.
> > > +'
> > > +*** The new user option 'speedbar-prefer-window', tells 'speedbar' to
> > > open
> > > +a side window instead of a frame.
> > > +
> > > +*** 'speedbar-easymenu-definition-trailer' is now a function.
> > > +
> > 
> > These NEWS entries should be marked with "+++", since this is now
> > documented in the relevant manuals.
> 
> Done. 
> 
> The new version of the patch is attached.

Thanks, installed on the master branch, and closing the bug.

I needed to make a few minor followup changes; please study them and
try to avoid the same pitfalls in the future.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.