Fix enabling menu bar multiple times on Haiku

* src/haiku_support.cc (FrameResized): Adjust width and height
appropriately when sending menu bar resize event.
* src/haikufns.c (haiku_set_menu_bar_lines): Don't always reset
FRAME_MENU_BAR_LINES and FRAME_MENU_BAR_HEIGHT.
* src/haikumenu.c (free_frame_menubar): Clear Haiku menu bar.
(set_frame_menubar): Fix coding style.
* src/haikuterm.c (haiku_read_socket): Don't adjust height here
anymore.

diff --git a/src/haiku_support.cc b/src/haiku_support.cc
index 290b2541f4cb71fab72bd7537b2d71009f4a31a7..af37fe7a62086726b8b41656d956b5d0096ec982 100644 (file)
--- a/src/haiku_support.cc
+++ b/src/haiku_support.cc
@@ -1272,8 +1272,8 @@ public:
   {
     struct haiku_menu_bar_resize_event rq;
     rq.window = this->Window ();
-    rq.height = std::lrint (newHeight);
-    rq.width = std::lrint (newWidth);
+    rq.height = std::lrint (newHeight + 1);
+    rq.width = std::lrint (newWidth + 1);
 
     haiku_write (MENU_BAR_RESIZE, &rq);
     BMenuBar::FrameResized (newWidth, newHeight);

diff --git a/src/haikufns.c b/src/haikufns.c
index 151874e26fd0829a80b7e6be3a8230dd10061732..b19fdd5488a21d6f43fa3f59d6f4557a92c69ce3 100644 (file)
--- a/src/haikufns.c
+++ b/src/haikufns.c
@@ -1278,9 +1278,6 @@ haiku_set_menu_bar_lines (struct frame *f, Lisp_Object value, Lisp_Object oldval
 
   fset_redisplay (f);
 
-  FRAME_MENU_BAR_LINES (f) = 0;
-  FRAME_MENU_BAR_HEIGHT (f) = 0;
-
   if (nlines)
     {
       FRAME_EXTERNAL_MENU_BAR (f) = 1;
@@ -1289,11 +1286,14 @@ haiku_set_menu_bar_lines (struct frame *f, Lisp_Object value, Lisp_Object oldval
     }
   else
     {
+      FRAME_MENU_BAR_LINES (f) = 0;
+      FRAME_MENU_BAR_HEIGHT (f) = 0;
+
       if (FRAME_EXTERNAL_MENU_BAR (f))
        free_frame_menubar (f);
+
       FRAME_EXTERNAL_MENU_BAR (f) = 0;
-      if (FRAME_HAIKU_P (f))
-       FRAME_HAIKU_MENU_BAR (f) = 0;
+      FRAME_HAIKU_MENU_BAR (f) = 0;
     }
 
   adjust_frame_glyphs (f);

diff --git a/src/haikumenu.c b/src/haikumenu.c
index 54ee3f160f69a53c9be893a3f0efbebc86fd8854..9779c34a9987563a688ec235d23aca2529d63681 100644 (file)
--- a/src/haikumenu.c
+++ b/src/haikumenu.c
@@ -486,14 +486,19 @@ haiku_menu_show (struct frame *f, int x, int y, int menuflags,
 void
 free_frame_menubar (struct frame *f)
 {
+  void *mbar;
+
   FRAME_MENU_BAR_LINES (f) = 0;
   FRAME_MENU_BAR_HEIGHT (f) = 0;
   FRAME_EXTERNAL_MENU_BAR (f) = 0;
 
   block_input ();
-  void *mbar = FRAME_HAIKU_MENU_BAR (f);
+  mbar = FRAME_HAIKU_MENU_BAR (f);
+  FRAME_HAIKU_MENU_BAR (f) = NULL;
+
   if (mbar)
     BMenuBar_delete (mbar);
+
   if (FRAME_OUTPUT_DATA (f)->menu_bar_open_p)
     --popup_activated_p;
   FRAME_OUTPUT_DATA (f)->menu_bar_open_p = 0;
@@ -516,8 +521,7 @@ set_frame_menubar (struct frame *f, bool deep_p)
 {
   void *mbar = FRAME_HAIKU_MENU_BAR (f);
   void *view = FRAME_HAIKU_VIEW (f);
-
-  int first_time_p = 0;
+  bool first_time_p = false;
 
   if (!mbar)
     {
@@ -529,6 +533,7 @@ set_frame_menubar (struct frame *f, bool deep_p)
         initial dimensions of that menu bar.  */
       if (FRAME_VISIBLE_P (f))
        haiku_wait_for_event (f, MENU_BAR_RESIZE);
+
       unblock_input ();
     }
 

diff --git a/src/haikuterm.c b/src/haikuterm.c
index 9cd1d0f0158cb2679933ace8e312861d9584998d..64c657fef565f3e43ae8ea9289b95ae6a7434ae5 100644 (file)
--- a/src/haikuterm.c
+++ b/src/haikuterm.c
@@ -3514,11 +3514,11 @@ haiku_read_socket (struct terminal *terminal, struct input_event *hold_quit)
 
            int old_height = FRAME_MENU_BAR_HEIGHT (f);
 
-           FRAME_MENU_BAR_HEIGHT (f) = b->height + 1;
+           FRAME_MENU_BAR_HEIGHT (f) = b->height;
            FRAME_MENU_BAR_LINES (f)
              = (b->height + FRAME_LINE_HEIGHT (f)) / FRAME_LINE_HEIGHT (f);
 
-           if (old_height != b->height + 1)
+           if (old_height != b->height)
              {
                adjust_frame_size (f, -1, -1, 3, true, Qmenu_bar_lines);
                haiku_clear_under_internal_border (f);