[Patch] Set default values of iswitchb-default-method to maybe-frame

[Patch] Set default values of iswitchb-default-method to maybe-frame

[Tassilo Horn]

Hello,

I'd advice to set the default value of iswitchb-default-method to
maybe-frame in the TRUNK, because if you use emacs as a server detached
with screen, connect to it with emacsclient and try to switch to the
buffer (with iswitchb) that's displayed in the detached server, nothing
happens in the client frame.

With a default value of maybe-frame the user is queried if he wants to
jump to the other frame.  If he chooses no the buffer is opened in the
current client's frame.

The same has to be done for ido's ido-default-file-method and
ido-default-buffer-method, too.  Here's a patch.

--8<---------------cut here---------------start------------->8---
Index: lisp/iswitchb.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/iswitchb.el,v
retrieving revision 1.70
diff -u -r1.70 iswitchb.el
--- lisp/iswitchb.el	13 Aug 2007 13:40:58 -0000	1.70
+++ lisp/iswitchb.el	6 Sep 2007 11:32:25 -0000
@@ -335,7 +335,7 @@
 ;(setq iswitchb-buffer-ignore '("^ " iswitchb-ignore-c-mode))
 ;(setq iswitchb-buffer-ignore '("^ " "\\.c$" "\\.h$"))
 
-(defcustom iswitchb-default-method  'always-frame
+(defcustom iswitchb-default-method  'maybe-frame
     "*How to switch to new buffer when using `iswitchb-buffer'.
 Possible values:
 `samewindow'	Show new buffer in same window
Index: lisp/ido.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/ido.el,v
retrieving revision 1.133
diff -u -r1.133 ido.el
--- lisp/ido.el	13 Aug 2007 13:40:59 -0000	1.133
+++ lisp/ido.el	6 Sep 2007 11:32:35 -0000
@@ -448,7 +448,7 @@
 ;;; Examples for setting the value of ido-ignore-files
 ;(setq ido-ignore-files '("^ " "\\.c$" "\\.h$"))
 
-(defcustom ido-default-file-method  'raise-frame
+(defcustom ido-default-file-method  'maybe-frame
     "*How to visit a new file when using `ido-find-file'.
 Possible values:
 `selected-window' Show new file in selected window
@@ -468,7 +468,7 @@
 		   (const :tag "Raise frame if already visited" raise-frame))
     :group 'ido)
 
-(defcustom ido-default-buffer-method  'raise-frame
+(defcustom ido-default-buffer-method  'maybe-frame
     "*How to switch to new buffer when using `ido-switch-buffer'.
 See `ido-default-file-method' for details."
     :type '(choice (const :tag "Show in selected window" selected-window)
Index: lisp/ChangeLog
===================================================================
RCS file: /sources/emacs/emacs/lisp/ChangeLog,v
retrieving revision 1.11694
diff -u -r1.11694 ChangeLog
--- lisp/ChangeLog	6 Sep 2007 05:53:04 -0000	1.11694
+++ lisp/ChangeLog	6 Sep 2007 11:32:42 -0000
@@ -1,3 +1,11 @@
+2007-09-06  Tassilo Horn  <tassilo@member.fsf.org>
+
+	* iswitchb.el (iswitchb-default-method): Set default value to
+	maybe-frame.
+
+	* ido.el (ido-default-file-method): likewise.
+	(ido-default-buffer-method): likewise.
+
 2007-09-06  Johan Bockgård  <bojohan@dd.chalmers.se>
 
 	* help-fns.el (describe-variable): Keep doc's text properties.
--8<---------------cut here---------------end--------------->8---

Bye,
Tassilo
-- 
Windows: So easy to admin, even a worm can do it.

Re: [Patch] Set default values of iswitchb-default-method to maybe-frame

[Tassilo Horn]

Hi,

another user has fallen into this trap, so we really should consider
applying my patch.  See

         Message-ID: <m2k5qyjqyi.fsf@cam.ac.uk>

Bye,
Tassilo