Subject: Re: bug#1754: 23.0.60; tool-bar is shown with tool-bar-mode off

From: martin rudalics <[email-masked]>
Date: Fri, 09 Jan 2009 20:37:08 +0100

> No, I only said that it shouldn't *set* the value in
 > initial-frame-alist.  Of course, it needs to remove any previous setting
 > in order for the setting in default-frame-alist not to be overridden.

So shall I apply the attached patch?

martin
*** frame.el.~1.291.~	2009-01-05 09:58:26.812500000 +0100
--- frame.el	2009-01-09 20:09:15.750000000 +0100
***************
*** 593,600 ****
      (dolist (w window-system-default-frame-alist)
        (setcdr w (assq-delete-all (car pair) (cdr w)))))
  
!   (and frame-notice-user-settings
!        (setq initial-frame-alist (append initial-frame-alist alist)))
    (setq default-frame-alist (append default-frame-alist alist)))
  
  (defun get-other-frame ()
--- 593,602 ----
      (dolist (w window-system-default-frame-alist)
        (setcdr w (assq-delete-all (car pair) (cdr w)))))
  
!   ;; There's no need to set the value in initial-frame-alist since
!   ;; frame-notice-user-settings will take it from default-frame-alist.
!   ;; (and frame-notice-user-settings
!   ;;     (setq initial-frame-alist (append initial-frame-alist alist)))
    (setq default-frame-alist (append default-frame-alist alist)))
  
  (defun get-other-frame ()