all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Strange behaviour after Show/Hide Speedbar in Emacs 22.1
@ 2007-06-25  9:51 Angelo Graziosi
  2007-06-25 10:29 ` Juanma Barranquero
  0 siblings, 1 reply; 15+ messages in thread
From: Angelo Graziosi @ 2007-06-25  9:51 UTC (permalink / raw)
  To: emacs-devel; +Cc: Eli Zaretskii



I have observed this behaviour in Emacs 22.1:

Click on Options | Show/Hide | Speedbar : the speedbar is shown.

Then click on Options | Show/Hide | Speedbar : the speedbar is hidden.

Now the strangeness: trying to exiting Emacs by clicking on the 'X'
(close) button of the window (each window has three buttons: minimize,
maximize, close), Emacs does not exit and in the minibuffer:

   Attempt to delete the sole visible or iconified frame

In 'Messages':

   speedbar-handle-delete-frame: Attempt to delete the sole visible or
                                 iconified frame


It look as if the speedbar were still 'live'

Now one can exit from Emacs only using File | Exit Emacs  or C-x C-c.


This behaviour happens on Cygwin and on GNU/Linux Kubuntu Dapper 6.06.


   Angelo.

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25  9:51 Strange behaviour after Show/Hide Speedbar in Emacs 22.1 Angelo Graziosi
@ 2007-06-25 10:29 ` Juanma Barranquero
  2007-06-25 10:34   ` Leo
  2007-06-25 13:17   ` Eric M. Ludlam
  0 siblings, 2 replies; 15+ messages in thread
From: Juanma Barranquero @ 2007-06-25 10:29 UTC (permalink / raw)
  To: Angelo Graziosi; +Cc: Eric M. Ludlam, emacs-devel

On 6/25/07, Angelo Graziosi <Angelo.Graziosi@roma1.infn.it> wrote:

> It look as if the speedbar were still 'live'

It is live, if somewhat hidden :)

Try the following patch.

             Juanma


P.S.: Eric, is this patch OK?



Index: lisp/speedbar.el
===================================================================
RCS file: /cvsroot/emacs/emacs/lisp/speedbar.el,v
retrieving revision 1.87
diff -u -2 -r1.87 speedbar.el
--- lisp/speedbar.el	19 Feb 2007 13:37:23 -0000	1.87
+++ lisp/speedbar.el	25 Jun 2007 10:24:07 -0000
@@ -1012,6 +1012,8 @@
     )
   ;; Frame modifications
-  (set (make-local-variable 'dframe-delete-frame-function)
-       'speedbar-handle-delete-frame)
+  (if speedbar-frame
+      (set (make-local-variable 'dframe-delete-frame-function)
+	   'speedbar-handle-delete-frame)
+    (kill-local-variable 'dframe-delete-frame-function))
   ;; hscroll
   (set (make-local-variable 'automatic-hscrolling) nil) ; Emacs 21

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25 10:29 ` Juanma Barranquero
@ 2007-06-25 10:34   ` Leo
  2007-06-25 10:49     ` Juanma Barranquero
                       ` (2 more replies)
  2007-06-25 13:17   ` Eric M. Ludlam
  1 sibling, 3 replies; 15+ messages in thread
From: Leo @ 2007-06-25 10:34 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: Eric M. Ludlam, Angelo Graziosi, emacs-devel

----- Juanma Barranquero (2007-06-25) wrote:-----

> On 6/25/07, Angelo Graziosi <Angelo.Graziosi@roma1.infn.it> wrote:
>
>> It look as if the speedbar were still 'live'
>
> It is live, if somewhat hidden :)
>
> Try the following patch.
>
>              Juanma
>
>
> P.S.: Eric, is this patch OK?

Can the latest speedbar from cedet be installed in trunk?

-- 
Leo <sdl.web AT gmail.com>                         (GPG Key: 9283AA3F)

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25 10:34   ` Leo
@ 2007-06-25 10:49     ` Juanma Barranquero
  2007-06-25 10:49     ` Nick Roberts
  2007-06-25 13:10     ` Re[2]: " Eric M. Ludlam
  2 siblings, 0 replies; 15+ messages in thread
From: Juanma Barranquero @ 2007-06-25 10:49 UTC (permalink / raw)
  To: Leo; +Cc: Nick Roberts, emacs-devel

On 6/25/07, Leo <sdl.web@gmail.com> wrote:

> Can the latest speedbar from cedet be installed in trunk?

I don't know how updates from cedet have been handled before.
According to the change logs, Nick Roberts seems to be the most active
developer wrt to speedbar.

             Juanma

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25 10:34   ` Leo
  2007-06-25 10:49     ` Juanma Barranquero
@ 2007-06-25 10:49     ` Nick Roberts
  2007-06-25 10:58       ` Leo
  2007-06-25 13:22       ` Vinicius Jose Latorre
  2007-06-25 13:10     ` Re[2]: " Eric M. Ludlam
  2 siblings, 2 replies; 15+ messages in thread
From: Nick Roberts @ 2007-06-25 10:49 UTC (permalink / raw)
  To: Leo; +Cc: Juanma Barranquero, Eric M. Ludlam, emacs-devel, Angelo Graziosi

 > Can the latest speedbar from cedet be installed in trunk?

It will surely break changes that I have made for using with GDB.

What benefits does it bring?

-- 
Nick                                           http://www.inet.net.nz/~nickrob

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25 10:49     ` Nick Roberts
@ 2007-06-25 10:58       ` Leo
  2007-06-25 11:30         ` Nick Roberts
  2007-06-25 13:22       ` Vinicius Jose Latorre
  1 sibling, 1 reply; 15+ messages in thread
From: Leo @ 2007-06-25 10:58 UTC (permalink / raw)
  To: emacs-devel

----- Nick Roberts (2007-06-25) wrote:-----

>  > Can the latest speedbar from cedet be installed in trunk?
>
> It will surely break changes that I have made for using with GDB.

That means when users install cedet they break GDB unconsciously.

> What benefits does it bring?

Some bug fixes I guess.

-- 
Leo <sdl.web AT gmail.com>                         (GPG Key: 9283AA3F)

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25 10:58       ` Leo
@ 2007-06-25 11:30         ` Nick Roberts
  0 siblings, 0 replies; 15+ messages in thread
From: Nick Roberts @ 2007-06-25 11:30 UTC (permalink / raw)
  To: Leo; +Cc: emacs-devel

 > >  > Can the latest speedbar from cedet be installed in trunk?
 > >
 > > It will surely break changes that I have made for using with GDB.
 > 
 > That means when users install cedet they break GDB unconsciously.

That might be true, I don't know how Cedet works.  With the best will in the
world it's not possible to test the consequences for all the external packages.
I find it hard enough trying to make sure I haven't broken things _within_
Emacs!

 > > What benefits does it bring?
 > 
 > Some bug fixes I guess.

Doesn't sound very compelling.

Trying to be more positive, I think it would be a good idea to move the watch
expressions for GDB out of the speedbar and use tree-widget anyway (I'm hoping
some-one else might do that - dreaming, I know).

Also if ECB/Cedet gets integrated into Emacs then, of course, we'll have to
ensure it all works together.

-- 
Nick                                           http://www.inet.net.nz/~nickrob

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re[2]: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25 10:34   ` Leo
  2007-06-25 10:49     ` Juanma Barranquero
  2007-06-25 10:49     ` Nick Roberts
@ 2007-06-25 13:10     ` Eric M. Ludlam
  2 siblings, 0 replies; 15+ messages in thread
From: Eric M. Ludlam @ 2007-06-25 13:10 UTC (permalink / raw)
  To: Leo; +Cc: lekktu, Angelo.Graziosi, emacs-devel

As far as I know, the CEDET/CVS version of speedbar is different from the
Emacs/CVS version by revision number only, and perhaps by a few small
changes that are in Emacs only.

Eric

>>> Leo <sdl.web@gmail.com> seems to think that:
>----- Juanma Barranquero (2007-06-25) wrote:-----
>
>> On 6/25/07, Angelo Graziosi <Angelo.Graziosi@roma1.infn.it> wrote:
>>
>>> It look as if the speedbar were still 'live'
>>
>> It is live, if somewhat hidden :)
>>
>> Try the following patch.
>>
>>              Juanma
>>
>>
>> P.S.: Eric, is this patch OK?
>
>Can the latest speedbar from cedet be installed in trunk?
>

-- 
          Eric Ludlam:                 zappo@gnu.org, eric@siege-engine.com
   Home: http://www.ludlam.net            Siege: www.siege-engine.com
Emacs: http://cedet.sourceforge.net               GNU: www.gnu.org

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re[2]: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25 10:29 ` Juanma Barranquero
  2007-06-25 10:34   ` Leo
@ 2007-06-25 13:17   ` Eric M. Ludlam
  2007-06-25 13:45     ` Angelo Graziosi
  2007-06-25 22:46     ` Re[2]: " Juanma Barranquero
  1 sibling, 2 replies; 15+ messages in thread
From: Eric M. Ludlam @ 2007-06-25 13:17 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: Angelo.Graziosi, emacs-devel

Hi,

  The speedbar frame stays "live" but hidden so that the next time
speedbar comes up, it will be faster.  You can use 'q' to hide
speedbar, and 'Q' to really kill speedbar.

  I tried it out, and it seems the speed difference is currently very
small now, so the optimization is not as important as it once was.

  I suspect your patch might leave a bunch of invisible frames about,
but it's been a while since I've been in that code so I'm not sure.

Eric

>>> "Juanma Barranquero" <lekktu@gmail.com> seems to think that:
>On 6/25/07, Angelo Graziosi <Angelo.Graziosi@roma1.infn.it> wrote:
>
>> It look as if the speedbar were still 'live'
>
>It is live, if somewhat hidden :)
>
>Try the following patch.
>
>             Juanma
>
>
>P.S.: Eric, is this patch OK?
>
>
>
>Index: lisp/speedbar.el
>===================================================================
>RCS file: /cvsroot/emacs/emacs/lisp/speedbar.el,v
>retrieving revision 1.87
>diff -u -2 -r1.87 speedbar.el
>--- lisp/speedbar.el	19 Feb 2007 13:37:23 -0000	1.87
>+++ lisp/speedbar.el	25 Jun 2007 10:24:07 -0000
>@@ -1012,6 +1012,8 @@
>     )
>   ;; Frame modifications
>-  (set (make-local-variable 'dframe-delete-frame-function)
>-       'speedbar-handle-delete-frame)
>+  (if speedbar-frame
>+      (set (make-local-variable 'dframe-delete-frame-function)
>+	   'speedbar-handle-delete-frame)
>+    (kill-local-variable 'dframe-delete-frame-function))
>   ;; hscroll
>   (set (make-local-variable 'automatic-hscrolling) nil) ; Emacs 21
>

-- 
          Eric Ludlam:                 zappo@gnu.org, eric@siege-engine.com
   Home: http://www.ludlam.net            Siege: www.siege-engine.com
Emacs: http://cedet.sourceforge.net               GNU: www.gnu.org

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25 10:49     ` Nick Roberts
  2007-06-25 10:58       ` Leo
@ 2007-06-25 13:22       ` Vinicius Jose Latorre
  1 sibling, 0 replies; 15+ messages in thread
From: Vinicius Jose Latorre @ 2007-06-25 13:22 UTC (permalink / raw)
  To: Nick Roberts
  Cc: Juanma Barranquero, Angelo Graziosi, Eric M. Ludlam, Leo,
	emacs-devel

Nick Roberts wrote:
>  > Can the latest speedbar from cedet be installed in trunk?
>
> It will surely break changes that I have made for using with GDB.
>   

It depends on which speedbar is loaded first.

If Emacs speedbar is loaded first, GDB is ok, but CEDET may have problems.

If CEDET speedbar is loaded first, CEDET is ok, but GDB will have problems.

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: Re[2]: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25 13:17   ` Eric M. Ludlam
@ 2007-06-25 13:45     ` Angelo Graziosi
  2007-06-25 15:57       ` Re[4]: " Eric M. Ludlam
  2007-06-25 22:46     ` Re[2]: " Juanma Barranquero
  1 sibling, 1 reply; 15+ messages in thread
From: Angelo Graziosi @ 2007-06-25 13:45 UTC (permalink / raw)
  To: Eric M. Ludlam; +Cc: Juanma Barranquero, emacs-devel



On Mon, 25 Jun 2007, Eric M. Ludlam wrote:

> Hi,
> 
>   The speedbar frame stays "live" but hidden so that the next time
> speedbar comes up, it will be faster.  You can use 'q' to hide
> speedbar, and 'Q' to really kill speedbar.

This works. Using Q, Emacs can be closed with the X button of the window.

Thanks,

   Angelo.

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re[4]: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25 13:45     ` Angelo Graziosi
@ 2007-06-25 15:57       ` Eric M. Ludlam
  2007-06-25 18:17         ` Juanma Barranquero
  2007-06-25 20:43         ` Angelo Graziosi
  0 siblings, 2 replies; 15+ messages in thread
From: Eric M. Ludlam @ 2007-06-25 15:57 UTC (permalink / raw)
  To: Angelo Graziosi; +Cc: lekktu, emacs-devel

>>> Angelo Graziosi <Angelo.Graziosi@roma1.infn.it> seems to think that:
>
>
>On Mon, 25 Jun 2007, Eric M. Ludlam wrote:
>
>> Hi,
>> 
>>   The speedbar frame stays "live" but hidden so that the next time
>> speedbar comes up, it will be faster.  You can use 'q' to hide
>> speedbar, and 'Q' to really kill speedbar.
>
>This works. Using Q, Emacs can be closed with the X button of the window.
  [ ... ]

Is there some new Emacs feature that won't close unless all frames are
closed?  If so, is there a way to mark a frame (like Speedbar's) such
that it does not participate?  Speedbar won't contain anything of
value that needs saving, and thus should not be counted in a list of
such frames.

Thanks
Eric

-- 
          Eric Ludlam:                 zappo@gnu.org, eric@siege-engine.com
   Home: http://www.ludlam.net            Siege: www.siege-engine.com
Emacs: http://cedet.sourceforge.net               GNU: www.gnu.org

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: Re[4]: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25 15:57       ` Re[4]: " Eric M. Ludlam
@ 2007-06-25 18:17         ` Juanma Barranquero
  2007-06-25 20:43         ` Angelo Graziosi
  1 sibling, 0 replies; 15+ messages in thread
From: Juanma Barranquero @ 2007-06-25 18:17 UTC (permalink / raw)
  To: Eric M. Ludlam; +Cc: Angelo Graziosi, emacs-devel

On 6/25/07, Eric M. Ludlam <eric@siege-engine.com> wrote:

> Is there some new Emacs feature that won't close unless all frames are
> closed?

No. In the OP's bugreport, Emacs doesn't close because there's an
error executing speedbar-handle-delete-frame, where it tries to

  (delete-frame speedbar-frame)

while speedbar-frame is nil.

             Juanma

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: Re[4]: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25 15:57       ` Re[4]: " Eric M. Ludlam
  2007-06-25 18:17         ` Juanma Barranquero
@ 2007-06-25 20:43         ` Angelo Graziosi
  1 sibling, 0 replies; 15+ messages in thread
From: Angelo Graziosi @ 2007-06-25 20:43 UTC (permalink / raw)
  To: Eric M. Ludlam; +Cc: lekktu, emacs-devel



On Mon, 25 Jun 2007, Eric M. Ludlam wrote:

> >>> Angelo Graziosi <Angelo.Graziosi@roma1.infn.it> seems to think that:
> >
> >
> >On Mon, 25 Jun 2007, Eric M. Ludlam wrote:
> >
> >> Hi,
> >> 
> >>   The speedbar frame stays "live" but hidden so that the next time
> >> speedbar comes up, it will be faster.  You can use 'q' to hide
> >> speedbar, and 'Q' to really kill speedbar.
> >
> >This works. Using Q, Emacs can be closed with the X button of the window.
>   [ ... ]
> 
> Is there some new Emacs feature that won't close unless all frames are
> closed?  


So far I have observed that only with 'speedbar'.


   Angelo.



If so, is there a way to mark a frame (like Speedbar's) such
> that it does not participate?  Speedbar won't contain anything of
> value that needs saving, and thus should not be counted in a list of
> such frames.
> 
> Thanks
> Eric
> 
> -- 
>           Eric Ludlam:                 zappo@gnu.org, eric@siege-engine.com
>    Home: http://www.ludlam.net            Siege: www.siege-engine.com
> Emacs: http://cedet.sourceforge.net               GNU: www.gnu.org
> 

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: Re[2]: Strange behaviour after Show/Hide Speedbar in Emacs 22.1
  2007-06-25 13:17   ` Eric M. Ludlam
  2007-06-25 13:45     ` Angelo Graziosi
@ 2007-06-25 22:46     ` Juanma Barranquero
  1 sibling, 0 replies; 15+ messages in thread
From: Juanma Barranquero @ 2007-06-25 22:46 UTC (permalink / raw)
  To: Eric M. Ludlam; +Cc: Angelo.Graziosi, emacs-devel

On 6/25/07, Eric M. Ludlam <eric@siege-engine.com> wrote:

>   I suspect your patch might leave a bunch of invisible frames about,
> but it's been a while since I've been in that code so I'm not sure.

At the very least, this patch should be committed. There's no point in
trying to delete-frame nil.

OTOH, I confess I don't entirely understand the way speedbar handles
frames. For example:

 emacs -q
 M-x speedbar-frame-mode
 "q" on the speedbar frame
 M-: (frame-list)    => the speedbar frame still exists

but

 emacs -q
 C-x 5 2
 M-x speedbar-frame-mode
 "q" on the speedbar frame
 M-: (frame-list)   => the speedbar frame does not exist

??

             Juanma


Index: lisp/speedbar.el
===================================================================
RCS file: /cvsroot/emacs/emacs/lisp/speedbar.el,v
retrieving revision 1.87
diff -u -2 -r1.87 speedbar.el
--- lisp/speedbar.el	19 Feb 2007 13:37:23 -0000	1.87
+++ lisp/speedbar.el	25 Jun 2007 22:39:43 -0000
@@ -1054,7 +1054,8 @@
 If the deleted frame is the frame SPEEDBAR is attached to,
 we need to delete speedbar also."
-  (let ((frame-to-be-deleted (car (car (cdr e)))))
-    (if (eq frame-to-be-deleted dframe-attached-frame)
-	(delete-frame speedbar-frame)))
+  (when speedbar-frame
+    (let ((frame-to-be-deleted (car (car (cdr e)))))
+      (if (eq frame-to-be-deleted dframe-attached-frame)
+	  (delete-frame speedbar-frame))))
   )

^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2007-06-25 22:46 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-06-25  9:51 Strange behaviour after Show/Hide Speedbar in Emacs 22.1 Angelo Graziosi
2007-06-25 10:29 ` Juanma Barranquero
2007-06-25 10:34   ` Leo
2007-06-25 10:49     ` Juanma Barranquero
2007-06-25 10:49     ` Nick Roberts
2007-06-25 10:58       ` Leo
2007-06-25 11:30         ` Nick Roberts
2007-06-25 13:22       ` Vinicius Jose Latorre
2007-06-25 13:10     ` Re[2]: " Eric M. Ludlam
2007-06-25 13:17   ` Eric M. Ludlam
2007-06-25 13:45     ` Angelo Graziosi
2007-06-25 15:57       ` Re[4]: " Eric M. Ludlam
2007-06-25 18:17         ` Juanma Barranquero
2007-06-25 20:43         ` Angelo Graziosi
2007-06-25 22:46     ` Re[2]: " Juanma Barranquero

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.