# HG changeset patch
# User Stefan Monnier <monnier@iro.umontreal.ca>
# Date 1205268537 0
# Node ID 26e567ab8830260089515b3ef9dc43d3ba1a6042
# Parent  871ad7e118ff44ad87cb13beda97e7b656f9ca7d
(display-buffer-other-frame): Eliminate problematic code.

diff -r 871ad7e118ff -r 26e567ab8830 lisp/ChangeLog
--- a/lisp/ChangeLog	Tue Mar 11 17:44:39 2008 +0000
+++ b/lisp/ChangeLog	Tue Mar 11 20:48:57 2008 +0000
@@ -1,5 +1,7 @@
 2008-03-11  Stefan Monnier  <monnier@iro.umontreal.ca>
 
+	* files.el (display-buffer-other-frame): Eliminate problematic code.
+
 	* menu-bar.el (menu-bar-update-buffers-maxbuf): Remove.
 	(menu-bar-select-buffer): Remove.
 	(menu-bar-select-frame): Make non-interactive and take a frame arg.
diff -r 871ad7e118ff -r 26e567ab8830 lisp/files.el
--- a/lisp/files.el	Tue Mar 11 17:44:39 2008 +0000
+++ b/lisp/files.el	Tue Mar 11 20:48:57 2008 +0000
@@ -1078,9 +1078,18 @@
         (old-window (selected-window))
 	new-window)
     (setq new-window (display-buffer buffer t))
-    (lower-frame (window-frame new-window))
-    (make-frame-invisible (window-frame old-window))
-    (make-frame-visible (window-frame old-window))))
+    ;; This may have been here in order to prevent the new frame from hiding
+    ;; the old frame.  But it does more harm than good.
+    ;; Maybe we should call `raise-window' on the old-frame instead?  --Stef
+    ;;(lower-frame (window-frame new-window))
+
+    ;; This may have been here in order to make sure the old-frame gets the
+    ;; focus.  But not only can it cause an annoying flicker, with some
+    ;; window-managers it just makes the window invisible, with no easy
+    ;; way to recover it.  --Stef
+    ;;(make-frame-invisible (window-frame old-window))
+    ;;(make-frame-visible (window-frame old-window))
+    ))
 
 (defvar find-file-default nil
   "Used within `find-file-read-args'.")