* Jeroen Roovers had this to say on [18 Sep 2008, 18:52:11 +0200]: [snip] > Sometimes when you do RC_OTHER, you get presented with "This IS window (n)." > That's rather stupid as you didn't even ask for window n in the first place, > but simply _some_ other window. -JeR > > --- process.c.orig 2008-07-23 08:36:05.000000000 +0200 > +++ process.c 2008-09-18 18:21:19.000000000 +0200 > @@ -1731,7 +1731,7 @@ > /* FALLTHROUGH */ > case RC_OTHER: > if (MoreWindows()) > - SwitchWindow(display && D_other ? D_other->w_number : NextWindow()); > + SwitchWindow(display && (D_other != fore) ? D_other->w_number : > NextWindow()); > break; > case RC_META: > if (user->u_Esc == -1)
The patch, as it is, looks good. Thanks! But, perhaps a more appropriate fix would be to make sure that display->d_other is updated appropriately? Do you have a screenrc file that can be used to easily reproduce the bug? Sadrul
