(menu-bar-mode): Make this a global variable.

(menu-bar-mode): Make variable menu-bar-mode and default-frame-alist
consistent.

diff --git a/lisp/menu-bar.el b/lisp/menu-bar.el
index 34bf8e7e0cfe8555d0af951c0967867a8b9bd209..923c776dfac7743726cae5df74cf4d9675b041fd 100644 (file)
--- a/lisp/menu-bar.el
+++ b/lisp/menu-bar.el
@@ -398,6 +398,8 @@ A large number or nil slows down menu responsiveness.")
 ;;;           mode-name
 ;;;           (or (buffer-file-name) ""))))))
 \f
+(defvar menu-bar-mode nil)
+
 (defun menu-bar-mode (flag)
   "Toggle display of a menu bar on each frame.
 This command applies to all frames that exist and frames to be
@@ -406,33 +408,36 @@ With a numeric argument, if the argument is negative,
 turn off menu bars; otherwise, turn on menu bars."
  (interactive "P")
 
- ;; Obtain the current setting by looking at default-frame-alist.
- (let ((menu-bar-mode
-       (not (zerop (let ((assq (assq 'menu-bar-lines default-frame-alist)))
-                     (if assq (cdr assq) 0))))))
-
-   ;; Tweedle it according to the argument.
-   (setq menu-bar-mode (if (null flag) (not menu-bar-mode)
-                        (> (prefix-numeric-value flag) 0)))
-
-   ;; Apply it to default-frame-alist.
-   (let ((parameter (assq 'menu-bar-lines default-frame-alist)))
-     (if (consp parameter)
-        (setcdr parameter (if menu-bar-mode 1 0))
-       (setq default-frame-alist
-            (cons (cons 'menu-bar-lines (if menu-bar-mode 1 0))
-                  default-frame-alist))))
-
-   ;; Apply it to existing frames.
-   (let ((frames (frame-list)))
-     (while frames
-       (let ((height (cdr (assq 'height (frame-parameters (car frames))))))
-        (modify-frame-parameters (car frames)
-                                 (list (cons 'menu-bar-lines
-                                           (if menu-bar-mode 1 0))))
-        (modify-frame-parameters (car frames)
-                                 (list (cons 'height height))))
-       (setq frames (cdr frames))))))
+  ;; Make menu-bar-mode and default-frame-alist consistent.
+  (let ((default (assq 'menu-bar-lines default-frame-alist)))
+    (if default
+       (setq menu-bar-mode (not (eq (cdr default) 0)))
+      (setq default-frame-alist
+           (cons (cons 'menu-bar-lines (if menu-bar-mode 1 0))
+                 default-frame-alist))))
+
+  ;; Toggle or set the mode, according to FLAG.
+ (setq menu-bar-mode (if (null flag) (not menu-bar-mode)
+                      (> (prefix-numeric-value flag) 0)))
+
+ ;; Apply it to default-frame-alist.
+ (let ((parameter (assq 'menu-bar-lines default-frame-alist)))
+   (if (consp parameter)
+       (setcdr parameter (if menu-bar-mode 1 0))
+     (setq default-frame-alist
+          (cons (cons 'menu-bar-lines (if menu-bar-mode 1 0))
+                default-frame-alist))))
+
+ ;; Apply it to existing frames.
+ (let ((frames (frame-list)))
+   (while frames
+     (let ((height (cdr (assq 'height (frame-parameters (car frames))))))
+       (modify-frame-parameters (car frames)
+                               (list (cons 'menu-bar-lines
+                                         (if menu-bar-mode 1 0))))
+       (modify-frame-parameters (car frames)
+                               (list (cons 'height height))))
+     (setq frames (cdr frames)))))
 
 (provide 'menu-bar)