]> Dogcows Code - chaz/openbox/blobdiff - src/Basemenu.cc
sync with the 2.0 branch
[chaz/openbox] / src / Basemenu.cc
index ef6cf4d5f7e786acfea3b06999ed5097a1ed4256..3556d00a9034bf30fecdf7c7176296f5bd69d197 100644 (file)
@@ -1,5 +1,6 @@
-// Basemenu.cc for Openbox
-// Copyright (c) 2001 Sean 'Shaleh' Perry <shaleh@debian.org>
+// -*- mode: C++; indent-tabs-mode: nil; c-basic-offset: 2; -*-
+// Basemenu.cc for Blackbox - an X11 Window manager
+// Copyright (c) 2001 - 2002 Sean 'Shaleh' Perry <shaleh@debian.org>
 // Copyright (c) 1997 - 2000 Brad Hughes (bhughes@tcac.net)
 //
 // Permission is hereby granted, free of charge, to any person obtaining a
 // FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
 // DEALINGS IN THE SOFTWARE.
 
-// stupid macros needed to access some functions in version 2 of the GNU C
-// library
-#ifndef   _GNU_SOURCE
-#define   _GNU_SOURCE
-#endif // _GNU_SOURCE
-
 #ifdef    HAVE_CONFIG_H
 #  include "../config.h"
 #endif // HAVE_CONFIG_H
 
+extern "C" {
 #ifdef    HAVE_STDIO_H
 #  include <stdio.h>
 #endif // HAVE_STDIO_H
 
-#ifdef    STDC_HEADERS
+#ifdef HAVE_STDLIB_H
 #  include <stdlib.h>
+#endif // HAVE_STDLIB_H
+
+#ifdef HAVE_STRING_H
 #  include <string.h>
-#endif // STDC_HEADERS
+#endif // HAVE_STRING_H
+}
 
 #include <algorithm>
+#include <assert.h>
 using namespace std;
 
-#include "i18n.h"
-#include "openbox.h"
-#include "Basemenu.h"
-#include "Screen.h"
+#include "i18n.hh"
+#include "blackbox.hh"
+#include "Basemenu.hh"
+#include "Font.hh"
+#include "GCCache.hh"
+#include "Image.hh"
+#include "Screen.hh"
+#include "Util.hh"
+
 
 static Basemenu *shown = (Basemenu *) 0;
 
-Basemenu::Basemenu(BScreen &scrn) : openbox(scrn.getOpenbox()), screen(scrn) {
-  image_ctrl = screen.getImageControl();
-  display = openbox.getXDisplay();
+Basemenu::Basemenu(BScreen *scrn) {
+  screen = scrn;
+  blackbox = screen->getBlackbox();
+  image_ctrl = screen->getImageControl();
+  display = blackbox->getXDisplay();
   parent = (Basemenu *) 0;
   alignment = AlignDontCare;
 
@@ -76,75 +84,59 @@ Basemenu::Basemenu(BScreen &scrn) : openbox(scrn.getOpenbox()), screen(scrn) {
     which_press =
     which_sbl = -1;
 
+  menu.sublevels =
+    menu.persub =
+    menu.minsub = 0;
+
   menu.frame_pixmap =
     menu.title_pixmap =
-    menu.hilite_pixmap =
-    menu.sel_pixmap = None;
+    menu.hilite_pixmap = None;
 
-  menu.bevel_w = screen.getBevelWidth();
+  menu.bevel_w = screen->getBevelWidth();
+
+  const MenuStyle* const style = screen->getMenuStyle();
+  menu.width = menu.title_h = menu.item_w = menu.frame_h =
+    style->t_font->height() + (menu.bevel_w  * 2);
+
+  menu.item_h = style->f_font->height() + menu.bevel_w;
+
+  menu.height = menu.title_h + screen->getBorderWidth() + menu.frame_h;
 
-  if (i18n->multibyte())
-    menu.width = menu.title_h = menu.item_w = menu.frame_h =
-      screen.getMenuStyle()->t_fontset_extents->max_ink_extent.height +
-       (menu.bevel_w  * 2);
-  else
-    menu.width = menu.title_h = menu.item_w = menu.frame_h =
-      screen.getMenuStyle()->t_font->ascent +
-        screen.getMenuStyle()->t_font->descent + (menu.bevel_w * 2);
-  
-  menu.label = 0;
-  
-  menu.sublevels =
-    menu.persub =
-    menu.minsub = 0;
-  
-  MenuStyle *style = screen.getMenuStyle();
-  if (i18n->multibyte()) {
-    menu.item_h = style->f_fontset_extents->max_ink_extent.height +
-      (menu.bevel_w);
-  } else {
-    menu.item_h = style->f_font->ascent + style->f_font->descent +
-                 (menu.bevel_w);
-  }
-  
-  menu.height = menu.title_h + screen.getBorderWidth() + menu.frame_h;
-  
   unsigned long attrib_mask = CWBackPixmap | CWBackPixel | CWBorderPixel |
-                             CWColormap | CWOverrideRedirect | CWEventMask;
+    CWColormap | CWOverrideRedirect | CWEventMask;
   XSetWindowAttributes attrib;
   attrib.background_pixmap = None;
   attrib.background_pixel = attrib.border_pixel =
-                           screen.getBorderColor()->getPixel();
-  attrib.colormap = screen.getColormap();
+    screen->getBorderColor()->pixel();
+  attrib.colormap = screen->getColormap();
   attrib.override_redirect = True;
   attrib.event_mask = ButtonPressMask | ButtonReleaseMask |
-                      ButtonMotionMask | ExposureMask;
+    ButtonMotionMask | ExposureMask;
 
   menu.window =
-    XCreateWindow(display, screen.getRootWindow(), menu.x, menu.y, menu.width,
-                 menu.height, screen.getBorderWidth(), screen.getDepth(),
-                  InputOutput, screen.getVisual(), attrib_mask, &attrib);
-  openbox.saveMenuSearch(menu.window, this);
+    XCreateWindow(display, screen->getRootWindow(),
+                  menu.x, menu.y, menu.width, menu.height,
+                  screen->getBorderWidth(), screen->getDepth(),
+                  InputOutput, screen->getVisual(), attrib_mask, &attrib);
+  blackbox->saveMenuSearch(menu.window, this);
 
   attrib_mask = CWBackPixmap | CWBackPixel | CWBorderPixel | CWEventMask;
-  attrib.background_pixel = screen.getBorderColor()->getPixel();
+  attrib.background_pixel = screen->getBorderColor()->pixel();
   attrib.event_mask |= EnterWindowMask | LeaveWindowMask;
 
   menu.title =
     XCreateWindow(display, menu.window, 0, 0, menu.width, menu.height, 0,
-                 screen.getDepth(), InputOutput, screen.getVisual(),
-                 attrib_mask, &attrib);
-  openbox.saveMenuSearch(menu.title, this);
+                  screen->getDepth(), InputOutput, screen->getVisual(),
+                  attrib_mask, &attrib);
+  blackbox->saveMenuSearch(menu.title, this);
 
   attrib.event_mask |= PointerMotionMask;
   menu.frame = XCreateWindow(display, menu.window, 0,
-                            menu.title_h + screen.getBorderWidth(),
-                            menu.width, menu.frame_h, 0,
-                            screen.getDepth(), InputOutput,
-                            screen.getVisual(), attrib_mask, &attrib);
-  openbox.saveMenuSearch(menu.frame, this);
-
-  menuitems = new LinkedList<BasemenuItem>;
+                             menu.title_h + screen->getBorderWidth(),
+                             menu.width, menu.frame_h, 0,
+                             screen->getDepth(), InputOutput,
+                             screen->getVisual(), attrib_mask, &attrib);
+  blackbox->saveMenuSearch(menu.frame, this);
 
   // even though this is the end of the constructor the menu is still not
   // completely created.  items must be inserted and it must be update()'d
@@ -157,14 +149,21 @@ Basemenu::~Basemenu(void) {
   if (shown && shown->getWindowID() == getWindowID())
     shown = (Basemenu *) 0;
 
-  int n = menuitems->count();
-  for (int i = 0; i < n; ++i)
-    remove(0);
-
-  delete menuitems;
+  MenuItems::const_iterator it = menuitems.begin(),
+    end = menuitems.end();
+  for (; it != end; ++it) {
+    if (! internal_menu) {
+      Basemenu *tmp = (*it)->submenu();
+      if (tmp) {
+        if (! tmp->internal_menu)
+          delete tmp;
+        else
+          tmp->internal_hide();
+      }
+    }
+  }
 
-  if (menu.label)
-    delete [] menu.label;
+  std::for_each(menuitems.begin(), menuitems.end(), PointerAssassin());
 
   if (menu.title_pixmap)
     image_ctrl->removeImage(menu.title_pixmap);
@@ -175,154 +174,113 @@ Basemenu::~Basemenu(void) {
   if (menu.hilite_pixmap)
     image_ctrl->removeImage(menu.hilite_pixmap);
 
-  if (menu.sel_pixmap)
-    image_ctrl->removeImage(menu.sel_pixmap);
-
-  openbox.removeMenuSearch(menu.title);
+  blackbox->removeMenuSearch(menu.title);
   XDestroyWindow(display, menu.title);
 
-  openbox.removeMenuSearch(menu.frame);
+  blackbox->removeMenuSearch(menu.frame);
   XDestroyWindow(display, menu.frame);
 
-  openbox.removeMenuSearch(menu.window);
+  blackbox->removeMenuSearch(menu.window);
   XDestroyWindow(display, menu.window);
 }
 
 
-int Basemenu::insert(const char *l, int function, const char *e, int pos) {
-  char *label = 0, *exec = 0;
+BasemenuItem *Basemenu::find(int index) {
+  if (index < 0 || index >= static_cast<signed>(menuitems.size()))
+    return (BasemenuItem*) 0;
 
-  if (l) label = bstrdup(l);
-  if (e) exec = bstrdup(e);
+  return menuitems[index];
+}
 
-  BasemenuItem *item = new BasemenuItem(label, function, exec);
-  menuitems->insert(item, pos);
 
-  return menuitems->count();
+int Basemenu::insert(BasemenuItem *item, int pos) {
+  if (pos < 0) {
+    menuitems.push_back(item);
+  } else {
+    assert(pos <= static_cast<signed>(menuitems.size()));
+    menuitems.insert((menuitems.begin() + pos), item);
+  }
+  return menuitems.size();
 }
 
 
-int Basemenu::insert(const char *l, Basemenu *submenu, int pos) {
-  char *label = 0;
+int Basemenu::insert(const string& label, int function,
+                     const string& exec, int pos) {
+  BasemenuItem *item = new BasemenuItem(label, function, exec);
+  return insert(item, pos);
+}
 
-  if (l) label = bstrdup(l);
 
+int Basemenu::insert(const string& label, Basemenu *submenu, int pos) {
   BasemenuItem *item = new BasemenuItem(label, submenu);
-  menuitems->insert(item, pos);
-
   submenu->parent = this;
 
-  return menuitems->count();
-}
-
-
-int Basemenu::insert(const char **ulabel, int pos, int function) {
-  BasemenuItem *item = new BasemenuItem(ulabel, function);
-  menuitems->insert(item, pos);
-
-  return menuitems->count();
+  return insert(item, pos);
 }
 
 
 int Basemenu::remove(int index) {
-  if (index < 0 || index > menuitems->count()) return -1;
-
-  BasemenuItem *item = menuitems->remove(index);
-
-  if (item) {
-    if ((! internal_menu) && (item->submenu())) {
-      Basemenu *tmp = (Basemenu *) item->submenu();
+  BasemenuItem *item = find(index);
+  if (! item) return -1;
 
-      if (! tmp->internal_menu) {
-       delete tmp;
-      } else {
-       tmp->internal_hide();
-      }
+  if (! internal_menu) {
+    Basemenu *tmp = item->submenu();
+    if (tmp) {
+      if (! tmp->internal_menu)
+        delete tmp;
+      else
+        tmp->internal_hide();
     }
-
-    if (item->label())
-      delete [] item->label();
-    
-    if (item->exec())
-      delete [] item->exec();
-
-    delete item;
   }
 
+  delete item;
+
   if (which_sub == index)
     which_sub = -1;
   else if (which_sub > index)
     which_sub--;
 
-  return menuitems->count();
+  menuitems.erase(menuitems.begin() + index);
+
+  return menuitems.size();
 }
 
 
 void Basemenu::update(void) {
-  MenuStyle *style = screen.getMenuStyle();
-  if (i18n->multibyte()) {
-    menu.item_h = style->f_fontset_extents->max_ink_extent.height +
-                 menu.bevel_w;
-    menu.title_h = style->t_fontset_extents->max_ink_extent.height +
-                   (menu.bevel_w * 2);
-  } else {
-    menu.item_h = style->f_font->ascent + style->f_font->descent +
-                 menu.bevel_w;
-    menu.title_h = style->t_font->ascent + style->t_font->descent +
-                  (menu.bevel_w * 2);
-  }
-    
-  if (title_vis) {
-    const char *s = (menu.label) ? menu.label :
-                   i18n->getMessage(BasemenuSet, BasemenuOpenboxMenu,
-                                    "Openbox Menu");
-    int l = strlen(s);
-    
+  const MenuStyle* const style = screen->getMenuStyle();
+  menu.item_h = (style->f_font->height() < 9 ? 9 : style->f_font->height()) +
+                menu.bevel_w;  // 9 for the menu pixmaps (checkmarks)
+  menu.title_h = style->t_font->height() + menu.bevel_w * 2;
 
-    if (i18n->multibyte()) {
-      XRectangle ink, logical;
-      XmbTextExtents(screen.getMenuStyle()->t_fontset, s, l, &ink, &logical);
-      menu.item_w = logical.width;
-    } else {
-      menu.item_w = XTextWidth(screen.getMenuStyle()->t_font, s, l);
-    }
-    
-    menu.item_w += (menu.bevel_w * 2);
+  if (title_vis) {
+    menu.item_w = screen->getMenuStyle()->t_font->measureString(menu.label) +
+      menu.bevel_w * 2;
   }  else {
     menu.item_w = 1;
   }
 
-  int ii = 0;
-  LinkedListIterator<BasemenuItem> it(menuitems);
-  for (BasemenuItem *tmp = it.current(); tmp; it++, tmp = it.current()) {
-    const char *s = ((tmp->u && *tmp->u) ? *tmp->u :
-                    ((tmp->l) ? tmp->l : (const char *) 0));
-    int l = strlen(s);
-
-    if (i18n->multibyte()) {
-      XRectangle ink, logical;
-      XmbTextExtents(screen.getMenuStyle()->f_fontset, s, l, &ink, &logical);
-      ii = logical.width;
-    } else
-      ii = XTextWidth(screen.getMenuStyle()->f_font, s, l);
-
-    ii += (menu.bevel_w * 2) + (menu.item_h * 2);
+  unsigned int ii = 0;
+  MenuItems::iterator it = menuitems.begin(), end = menuitems.end();
+  for (; it != end; ++it) {
+    ii = screen->getMenuStyle()->f_font->measureString((*it)->l) +
+      (menu.bevel_w * 2) + (menu.item_h * 2);
 
-    menu.item_w = ((menu.item_w < (unsigned int) ii) ? ii : menu.item_w);
+    menu.item_w = ((menu.item_w < ii) ? ii : menu.item_w);
   }
 
-  if (menuitems->count()) {
+  if (! menuitems.empty()) {
     menu.sublevels = 1;
 
-    while (((menu.item_h * (menuitems->count() + 1) / menu.sublevels)
-           + menu.title_h + screen.getBorderWidth()) >
-          screen.getHeight())
+    const unsigned int menu_size = menuitems.size();
+    while (((menu.item_h * (menu_size + 1) / menu.sublevels)
+            + menu.title_h + screen->getBorderWidth()) >
+           screen->getHeight())
       menu.sublevels++;
 
     if (menu.sublevels < menu.minsub) menu.sublevels = menu.minsub;
 
-    menu.persub = menuitems->count() / menu.sublevels;
-    if (menuitems->count() % menu.sublevels) menu.persub++;
+    menu.persub = menu_size / menu.sublevels;
+    if (menu_size % menu.sublevels) menu.persub++;
   } else {
     menu.sublevels = 0;
     menu.persub = 0;
@@ -332,8 +290,8 @@ void Basemenu::update(void) {
   if (! menu.width) menu.width = menu.item_w;
 
   menu.frame_h = (menu.item_h * menu.persub);
-  menu.height = ((title_vis) ? menu.title_h + screen.getBorderWidth() : 0) +
-               menu.frame_h;
+  menu.height = ((title_vis) ? menu.title_h + screen->getBorderWidth() : 0) +
+    menu.frame_h;
   if (! menu.frame_h) menu.frame_h = 1;
   if (menu.height < 1) menu.height = 1;
 
@@ -341,14 +299,14 @@ void Basemenu::update(void) {
   BTexture *texture;
   if (title_vis) {
     tmp = menu.title_pixmap;
-    texture = &(screen.getMenuStyle()->title);
-    if (texture->getTexture() == (BImage_Flat | BImage_Solid)) {
+    texture = &(screen->getMenuStyle()->title);
+    if (texture->texture() == (BTexture::Flat | BTexture::Solid)) {
       menu.title_pixmap = None;
       XSetWindowBackground(display, menu.title,
-                          texture->getColor()->getPixel());
+                           texture->color().pixel());
     } else {
       menu.title_pixmap =
-        image_ctrl->renderImage(menu.width, menu.title_h, texture);
+        image_ctrl->renderImage(menu.width, menu.title_h, *texture);
       XSetWindowBackgroundPixmap(display, menu.title, menu.title_pixmap);
     }
     if (tmp) image_ctrl->removeImage(tmp);
@@ -356,35 +314,25 @@ void Basemenu::update(void) {
   }
 
   tmp = menu.frame_pixmap;
-  texture = &(screen.getMenuStyle()->frame);
-  if (texture->getTexture() == (BImage_Flat | BImage_Solid)) {
+  texture = &(screen->getMenuStyle()->frame);
+  if (texture->texture() == (BTexture::Flat | BTexture::Solid)) {
     menu.frame_pixmap = None;
     XSetWindowBackground(display, menu.frame,
-                        texture->getColor()->getPixel());
+                         texture->color().pixel());
   } else {
     menu.frame_pixmap =
-      image_ctrl->renderImage(menu.width, menu.frame_h, texture);
+      image_ctrl->renderImage(menu.width, menu.frame_h, *texture);
     XSetWindowBackgroundPixmap(display, menu.frame, menu.frame_pixmap);
   }
   if (tmp) image_ctrl->removeImage(tmp);
 
   tmp = menu.hilite_pixmap;
-  texture = &(screen.getMenuStyle()->hilite);
-  if (texture->getTexture() == (BImage_Flat | BImage_Solid)) {
+  texture = &(screen->getMenuStyle()->hilite);
+  if (texture->texture() == (BTexture::Flat | BTexture::Solid)) {
     menu.hilite_pixmap = None;
   } else {
     menu.hilite_pixmap =
-      image_ctrl->renderImage(menu.item_w, menu.item_h, texture);
-  }
-  if (tmp) image_ctrl->removeImage(tmp);
-
-  tmp = menu.sel_pixmap;
-  if (texture->getTexture() == (BImage_Flat | BImage_Solid)) {
-    menu.sel_pixmap = None;
-  } else {
-    int hw = menu.item_h / 2;
-    menu.sel_pixmap =
-      image_ctrl->renderImage(hw, hw, texture);
+      image_ctrl->renderImage(menu.item_w, menu.item_h, *texture);
   }
   if (tmp) image_ctrl->removeImage(tmp);
 
@@ -394,9 +342,9 @@ void Basemenu::update(void) {
     XResizeWindow(display, menu.title, menu.width, menu.title_h);
 
   XMoveResizeWindow(display, menu.frame, 0,
-                   ((title_vis) ? menu.title_h +
-                    screen.getBorderWidth() : 0), menu.width,
-                   menu.frame_h);
+                    ((title_vis) ? menu.title_h +
+                     screen->getBorderWidth() : 0), menu.width,
+                    menu.frame_h);
 
   XClearWindow(display, menu.window);
   XClearWindow(display, menu.title);
@@ -404,7 +352,8 @@ void Basemenu::update(void) {
 
   if (title_vis && visible) redrawTitle();
 
-  for (int i = 0; visible && i < menuitems->count(); i++) {
+  const int menu_size = menuitems.size();
+  for (int i = 0; visible && i < menu_size; i++) {
     if (i == which_sub) {
       drawItem(i, True, 0);
       drawSubmenu(i);
@@ -427,7 +376,7 @@ void Basemenu::show(void) {
 
   if (! parent) {
     if (shown && (! shown->torn))
-       shown->hide();
+      shown->hide();
 
     shown = this;
   }
@@ -435,10 +384,10 @@ void Basemenu::show(void) {
 
 
 void Basemenu::hide(void) {
-  if ((! torn) && hide_tree && parent && parent->isVisible()) {
+  if (! torn && hide_tree && parent && parent->isVisible()) {
     Basemenu *p = parent;
 
-    while (p->isVisible() && (! p->torn) && p->parent) p = p->parent;
+    while (p->isVisible() && ! p->torn && p->parent) p = p->parent;
     p->internal_hide();
   } else {
     internal_hide();
@@ -447,12 +396,11 @@ void Basemenu::hide(void) {
 
 
 void Basemenu::internal_hide(void) {
-  if (which_sub != -1) {
-    BasemenuItem *tmp = menuitems->find(which_sub);
+  BasemenuItem *tmp = find(which_sub);
+  if (tmp)
     tmp->submenu()->internal_hide();
-  }
 
-  if (parent && (! torn)) {
+  if (parent && ! torn) {
     parent->drawItem(parent->which_sub, False, True);
 
     parent->which_sub = -1;
@@ -477,184 +425,163 @@ void Basemenu::move(int x, int y) {
 
 
 void Basemenu::redrawTitle(void) {
-  char *text = (char *) ((menu.label) ? menu.label :
-                        i18n->getMessage(BasemenuSet, BasemenuOpenboxMenu,
-                                         "Openbox Menu"));
-  int dx = menu.bevel_w, len = strlen(text);
+  const char *text = (! menu.label.empty()) ? getLabel() :
+    i18n(BasemenuSet, BasemenuBlackboxMenu, "Blackbox Menu");
+  int dx = menu.bevel_w;
   unsigned int l;
+  const MenuStyle* const style = screen->getMenuStyle();
 
-  if (i18n->multibyte()) {
-    XRectangle ink, logical;
-    XmbTextExtents(screen.getMenuStyle()->t_fontset, text, len, &ink, &logical);
-    l = logical.width;
-  } else {
-    l = XTextWidth(screen.getMenuStyle()->t_font, text, len);
-  }
-
-  l +=  (menu.bevel_w * 2);
+  l = style->t_font->measureString(text) + menu.bevel_w * 2;
 
-  switch (screen.getMenuStyle()->t_justify) {
-  case BScreen::RightJustify:
+  switch (screen->getMenuStyle()->t_justify) {
+  case RightJustify:
     dx += menu.width - l;
     break;
 
-  case BScreen::CenterJustify:
+  case CenterJustify:
     dx += (menu.width - l) / 2;
     break;
+
+  case LeftJustify:
+  default:
+    break;
   }
 
-  MenuStyle *style = screen.getMenuStyle();
-  if (i18n->multibyte())
-    XmbDrawString(display, menu.title, style->t_fontset, style->t_text_gc, dx,
-                 (menu.bevel_w - style->t_fontset_extents->max_ink_extent.y),
-                 text, len);
-  else
-    XDrawString(display, menu.title, style->t_text_gc, dx,
-               (style->t_font->ascent + menu.bevel_w), text, len);
+  style->t_font->drawString(menu.title, dx, menu.bevel_w,
+                            style->t_text, text);
 }
 
 
 void Basemenu::drawSubmenu(int index) {
-  if (which_sub != -1 && which_sub != index) {
-    BasemenuItem *itmp = menuitems->find(which_sub);
+  BasemenuItem *item = find(which_sub);
+  if (item && item->submenu() && ! item->submenu()->isTorn() &&
+      which_sub != index)
+    item->submenu()->internal_hide();
+
+  item = find(index);
+  if (! item)
+    return;
+
+  Basemenu *submenu = item->submenu();
+
+  if (submenu && visible && ! submenu->isTorn() && item->isEnabled()) {
+    if (submenu->parent != this) submenu->parent = this;
+    const int sbl = index / menu.persub, i = index - (sbl * menu.persub);
+    int x = menu.x + ((menu.item_w * (sbl + 1)) + screen->getBorderWidth()), y;
+
+    if (alignment == AlignTop) {
+      y = (((shifted) ? menu.y_shift : menu.y) +
+           ((title_vis) ? menu.title_h + screen->getBorderWidth() : 0) -
+           ((submenu->title_vis) ?
+            submenu->menu.title_h + screen->getBorderWidth() : 0));
+    } else {
+      y = (((shifted) ? menu.y_shift : menu.y) +
+           (menu.item_h * i) +
+           ((title_vis) ? menu.title_h + screen->getBorderWidth() : 0) -
+           ((submenu->title_vis) ?
+            submenu->menu.title_h + screen->getBorderWidth() : 0));
+    }
 
-    if (! itmp->submenu()->isTorn())
-      itmp->submenu()->internal_hide();
-  }
+    if (alignment == AlignBottom &&
+        (y + submenu->menu.height) > ((shifted) ? menu.y_shift :
+                                              menu.y) + menu.height)
+      y = (((shifted) ? menu.y_shift : menu.y) +
+           menu.height - submenu->menu.height);
 
-  if (index >= 0 && index < menuitems->count()) {
-    BasemenuItem *item = menuitems->find(index);
-    if (item->submenu() && visible && (! item->submenu()->isTorn()) &&
-       item->isEnabled()) {
-      if (item->submenu()->parent != this) item->submenu()->parent = this;
-      int sbl = index / menu.persub, i = index - (sbl * menu.persub),
-           x = menu.x +
-               ((menu.item_w * (sbl + 1)) + screen.getBorderWidth()), y;
-    
-      if (alignment == AlignTop)
-       y = (((shifted) ? menu.y_shift : menu.y) +
-            ((title_vis) ? menu.title_h + screen.getBorderWidth() : 0) -
-            ((item->submenu()->title_vis) ?
-             item->submenu()->menu.title_h + screen.getBorderWidth() : 0));
-      else
-       y = (((shifted) ? menu.y_shift : menu.y) +
-            (menu.item_h * i) +
-            ((title_vis) ? menu.title_h + screen.getBorderWidth() : 0) -
-            ((item->submenu()->title_vis) ?
-             item->submenu()->menu.title_h + screen.getBorderWidth() : 0));
-
-      if (alignment == AlignBottom &&
-         (y + item->submenu()->menu.height) > ((shifted) ? menu.y_shift :
-                                               menu.y) + menu.height)
-       y = (((shifted) ? menu.y_shift : menu.y) +
-            menu.height - item->submenu()->menu.height);
-
-      if ((x + item->submenu()->getWidth()) > screen.getWidth()) {
-       x = ((shifted) ? menu.x_shift : menu.x) -
-           item->submenu()->getWidth() - screen.getBorderWidth();
-      }
-      
-      if (x < 0) x = 0;
+    if ((x + submenu->getWidth()) > screen->getWidth())
+      x = ((shifted) ? menu.x_shift : menu.x) -
+        submenu->getWidth() - screen->getBorderWidth();
 
-      if ((y + item->submenu()->getHeight()) > screen.getHeight())
-       y = screen.getHeight() - item->submenu()->getHeight() -
-           (screen.getBorderWidth() * 2);
-      if (y < 0) y = 0;
-      
-      item->submenu()->move(x, y);
-      if (! moving) drawItem(index, True);
-    
-      if (! item->submenu()->isVisible())
-       item->submenu()->show();
-      item->submenu()->moving = moving;
-      which_sub = index;
-    } else {
-      which_sub = -1;
-    }
+    if (x < 0) x = 0;
+
+    if ((y + submenu->getHeight()) > screen->getHeight())
+      y = screen->getHeight() - submenu->getHeight() -
+        (screen->getBorderWidth() * 2);
+    if (y < 0) y = 0;
+
+    submenu->move(x, y);
+    if (! moving) drawItem(index, True);
+
+    if (! submenu->isVisible())
+      submenu->show();
+    submenu->moving = moving;
+    which_sub = index;
+  } else {
+    which_sub = -1;
   }
 }
 
 
-Bool Basemenu::hasSubmenu(int index) {
-  if ((index >= 0) && (index < menuitems->count()))
-    if (menuitems->find(index)->submenu())
-      return True;
-
+bool Basemenu::hasSubmenu(int index) {
+  BasemenuItem *item = find(index);
+  if (item && item->submenu())
+    return True;
   return False;
 }
 
 
-void Basemenu::drawItem(int index, Bool highlight, Bool clear,
-                       int x, int y, unsigned int w, unsigned int h)
-{
-  if (index < 0 || index > menuitems->count()) return;
-
-  BasemenuItem *item = menuitems->find(index);
+void Basemenu::drawItem(int index, bool highlight, bool clear,
+                        int x, int y, unsigned int w, unsigned int h) {
+  BasemenuItem *item = find(index);
   if (! item) return;
-  
-  Bool dotext = True, dohilite = True, dosel = True;
-  const char *text = (item->ulabel()) ? *item->ulabel() : item->label();
-  int sbl = index / menu.persub, i = index - (sbl * menu.persub);
+
+  bool dotext = True, dohilite = True, dosel = True, dooppsel = True;
+  const char *text = item->label();
+  const int sbl = index / menu.persub, i = index - (sbl * menu.persub);
+  const unsigned int half_w = menu.item_h / 2, quarter_w = menu.item_h / 4;
   int item_x = (sbl * menu.item_w), item_y = (i * menu.item_h);
   int hilite_x = item_x, hilite_y = item_y, hoff_x = 0, hoff_y = 0;
-  int text_x = 0, text_y = 0, len = strlen(text), sel_x = 0, sel_y = 0;
+  int text_x = 0, text_y = 0, sel_x = 0, oppsel_x = 0, sel_y = 0;
   unsigned int hilite_w = menu.item_w, hilite_h = menu.item_h, text_w = 0,
     text_h = 0;
-  unsigned int half_w = menu.item_h / 2, quarter_w = menu.item_h / 4;
-  
+
   if (text) {
-    if (i18n->multibyte()) {
-      XRectangle ink, logical;
-      XmbTextExtents(screen.getMenuStyle()->f_fontset,
-                    text, len, &ink, &logical);
-      text_w = logical.width;
-      text_y = item_y + (menu.bevel_w / 2) -
-              screen.getMenuStyle()->f_fontset_extents->max_ink_extent.y;
-    } else {
-      text_w = XTextWidth(screen.getMenuStyle()->f_font, text, len);
-      text_y =  item_y +
-               screen.getMenuStyle()->f_font->ascent +
-               (menu.bevel_w / 2);
-    }
-    
-    switch(screen.getMenuStyle()->f_justify) {
-    case BScreen::LeftJustify:
+    text_w = screen->getMenuStyle()->f_font->measureString(text);
+    text_y = item_y + menu.bevel_w / 2;
+
+    switch(screen->getMenuStyle()->f_justify) {
+    case LeftJustify:
       text_x = item_x + menu.bevel_w + menu.item_h + 1;
       break;
-      
-    case BScreen::RightJustify:
+
+    case RightJustify:
       text_x = item_x + menu.item_w - (menu.item_h + menu.bevel_w + text_w);
       break;
-      
-    case BScreen::CenterJustify:
+
+    case CenterJustify:
       text_x = item_x + ((menu.item_w + 1 - text_w) / 2);
       break;
     }
-    
+
     text_h = menu.item_h - menu.bevel_w;
   }
-  
-  GC gc =
-    ((highlight || item->isSelected()) ? screen.getMenuStyle()->h_text_gc :
-     screen.getMenuStyle()->f_text_gc),
-    tgc =
-    ((highlight) ? screen.getMenuStyle()->h_text_gc :
-     ((item->isEnabled()) ? screen.getMenuStyle()->f_text_gc :
-      screen.getMenuStyle()->d_text_gc));
-  
+
+  const MenuStyle* const style = screen->getMenuStyle();
+  const BPen hipen(style->hilite.color());
+  // match the text color
+  const BPen pen((highlight ? style->h_text :
+                  (item->isEnabled() ? style->f_text :
+                   style->d_text)));
+
+
   sel_x = item_x;
-  if (screen.getMenuStyle()->bullet_pos == Right)
+  if (screen->getMenuStyle()->bullet_pos == Right)
     sel_x += (menu.item_w - menu.item_h - menu.bevel_w);
   sel_x += quarter_w;
+  oppsel_x = sel_x;
+  if (screen->getMenuStyle()->bullet_pos == Right)
+    oppsel_x -= (menu.item_w - menu.item_h - menu.bevel_w);
+  else
+    oppsel_x += (menu.item_w - menu.item_h - menu.bevel_w);
   sel_y = item_y + quarter_w;
-  
+
   if (clear) {
     XClearArea(display, menu.frame, item_x, item_y, menu.item_w, menu.item_h,
-              False);
+               False);
   } else if (! (x == y && y == -1 && w == h && h == 0)) {
     // calculate the which part of the hilite to redraw
-    if (! (max(item_x, x) <= (signed) min(item_x + menu.item_w, x + w) &&
-          max(item_y, y) <= (signed) min(item_y + menu.item_h, y + h))) {
+    if (! (max(item_x, x) <= min<signed>(item_x + menu.item_w, x + w) &&
+           max(item_y, y) <= min<signed>(item_y + menu.item_h, y + h))) {
       dohilite = False;
     } else {
       hilite_x = max(item_x, x);
@@ -664,90 +591,114 @@ void Basemenu::drawItem(int index, Bool highlight, Bool clear,
       hoff_x = hilite_x % menu.item_w;
       hoff_y = hilite_y % menu.item_h;
     }
-    
-    // check if we need to redraw the text    
-    int text_ry = item_y + (menu.bevel_w / 2);
-    if (! (max(text_x, x) <= (signed) min(text_x + text_w, x + w) &&
-          max(text_ry, y) <= (signed) min(text_ry + text_h, y + h)))
+
+    // check if we need to redraw the text
+    const int text_ry = item_y + (menu.bevel_w / 2);
+    if (! (max(text_x, x) <= min<signed>(text_x + text_w, x + w) &&
+           max(text_ry, y) <= min<signed>(text_ry + text_h, y + h)))
       dotext = False;
-    
+
     // check if we need to redraw the select pixmap/menu bullet
-    if (! (max(sel_x, x) <= (signed) min(sel_x + half_w, x + w) &&
-          max(sel_y, y) <= (signed) min(sel_y + half_w, y + h)))
+    if (! (max(sel_x, x) <= min<signed>(sel_x + half_w, x + w) &&
+           max(sel_y, y) <= min<signed>(sel_y + half_w, y + h)))
       dosel = False;
+    
+    // check if we need to redraw the select pixmap/menu bullet
+    // on the opposite side of the menu
+    if (! (max(oppsel_x, x) <= min<signed>(oppsel_x + half_w, x + w) &&
+           max(sel_y, y) <= min<signed>(sel_y + half_w, y + h)))
+      dooppsel = False;
   }
-  
+
   if (dohilite && highlight && (menu.hilite_pixmap != ParentRelative)) {
     if (menu.hilite_pixmap)
       XCopyArea(display, menu.hilite_pixmap, menu.frame,
-               screen.getMenuStyle()->hilite_gc, hoff_x, hoff_y,
-               hilite_w, hilite_h, hilite_x, hilite_y);
+                hipen.gc(), hoff_x, hoff_y,
+                hilite_w, hilite_h, hilite_x, hilite_y);
     else
-      XFillRectangle(display, menu.frame,
-                    screen.getMenuStyle()->hilite_gc,
-                    hilite_x, hilite_y, hilite_w, hilite_h);
-  } else if (dosel && item->isSelected() &&
-                        (menu.sel_pixmap != ParentRelative)) {
-    if (menu.sel_pixmap)
-      XCopyArea(display, menu.sel_pixmap, menu.frame,
-               screen.getMenuStyle()->hilite_gc, 0, 0,
-               half_w, half_w, sel_x, sel_y);
-    else
-      XFillRectangle(display, menu.frame,
-                    screen.getMenuStyle()->hilite_gc,
-                    sel_x, sel_y, half_w, half_w);
+      XFillRectangle(display, menu.frame, hipen.gc(),
+                     hilite_x, hilite_y, hilite_w, hilite_h);
   }
   
+  if (dooppsel && item->isSelected()) {
+      XPoint pts[6];
+
+      pts[0].x = oppsel_x + 0;
+      pts[0].y = sel_y + 2;
+      
+      pts[1].x = 0;
+      pts[1].y = 3;
+      
+      pts[2].x = 2;
+      pts[2].y = 3;
+      
+      pts[3].x = 5;
+      pts[3].y = -5;
+      
+      
+      pts[4].x = 0;
+      pts[4].y = -3;
+      
+      pts[5].x = -5;
+      pts[5].y = 5;
+
+      XFillPolygon(display, menu.frame, pen.gc(), pts, 6, Nonconvex,
+                   CoordModePrevious);
+  }
+
   if (dotext && text) {
-    if (i18n->multibyte())
-      XmbDrawString(display, menu.frame, screen.getMenuStyle()->f_fontset,
-                   tgc, text_x, text_y, text, len);
-    else
-      XDrawString(display, menu.frame, tgc, text_x, text_y, text, len);
+    style->f_font->drawString(menu.frame, text_x, text_y,
+                              (highlight ? style->h_text :
+                               (item->isEnabled() ? style->f_text :
+                                style->d_text)),
+                              text);
   }
 
   if (dosel && item->submenu()) {
-    switch (screen.getMenuStyle()->bullet) {
+    const int bullet_size = 3;
+
+    switch (screen->getMenuStyle()->bullet) {
     case Square:
-      XDrawRectangle(display, menu.frame, gc, sel_x, sel_y, half_w, half_w);
+      XDrawRectangle(display, menu.frame, pen.gc(), sel_x, sel_y,
+                     bullet_size * 2, bullet_size * 2);
       break;
 
     case Triangle:
       XPoint tri[3];
 
-      if (screen.getMenuStyle()->bullet_pos == Right) {
-        tri[0].x = sel_x + quarter_w - 2;
-       tri[0].y = sel_y + quarter_w - 2;
-        tri[1].x = 4;
-       tri[1].y = 2;
-        tri[2].x = -4;
-       tri[2].y = 2;
+      if (screen->getMenuStyle()->bullet_pos == Right) {
+        tri[0].x = sel_x + quarter_w - bullet_size;
+        tri[0].y = sel_y + quarter_w - bullet_size;
+        tri[1].x = 2 * bullet_size;
+        tri[1].y = bullet_size;
+        tri[2].x = -(2 * bullet_size);
+        tri[2].y = bullet_size;
       } else {
-        tri[0].x = sel_x + quarter_w - 2;
-       tri[0].y = item_y + half_w;
-        tri[1].x = 4;
-       tri[1].y = 2;
+        tri[0].x = sel_x + quarter_w - bullet_size;
+        tri[0].y = item_y + half_w;
+        tri[1].x = 2 * bullet_size;
+        tri[1].y = bullet_size;
         tri[2].x = 0;
-       tri[2].y = -4;
+        tri[2].y = -(2 * bullet_size);
       }
-      
-      XFillPolygon(display, menu.frame, gc, tri, 3, Convex,
+
+      XFillPolygon(display, menu.frame, pen.gc(), tri, 3, Convex,
                    CoordModePrevious);
       break;
-      
+
     case Diamond:
       XPoint dia[4];
 
-      dia[0].x = sel_x + quarter_w - 3;
+      dia[0].x = sel_x + quarter_w - bullet_size;
       dia[0].y = item_y + half_w;
-      dia[1].x = 3;
-      dia[1].y = -3;
-      dia[2].x = 3;
-      dia[2].y = 3;
-      dia[3].x = -3;
-      dia[3].y = 3;
-
-      XFillPolygon(display, menu.frame, gc, dia, 4, Convex,
+      dia[1].x = bullet_size;
+      dia[1].y = -bullet_size;
+      dia[2].x = bullet_size;
+      dia[2].y = bullet_size;
+      dia[3].x = -bullet_size;
+      dia[3].y = bullet_size;
+
+      XFillPolygon(display, menu.frame, pen.gc(), dia, 4, Convex,
                    CoordModePrevious);
       break;
     }
@@ -755,18 +706,13 @@ void Basemenu::drawItem(int index, Bool highlight, Bool clear,
 }
 
 
-void Basemenu::setLabel(const char *l) {
-  if (menu.label)
-    delete [] menu.label;
-
-  if (l) menu.label = bstrdup(l);
-  else menu.label = 0;
+void Basemenu::setLabel(const string& label) {
+  menu.label = label;
 }
 
 
-void Basemenu::setItemSelected(int index, Bool sel) {
-  if (index < 0 || index >= menuitems->count()) return;
-
+void Basemenu::setItemSelected(int index, bool sel) {
+  assert(index >= 0);
   BasemenuItem *item = find(index);
   if (! item) return;
 
@@ -775,9 +721,8 @@ void Basemenu::setItemSelected(int index, Bool sel) {
 }
 
 
-Bool Basemenu::isItemSelected(int index) {
-  if (index < 0 || index >= menuitems->count()) return False;
-
+bool Basemenu::isItemSelected(int index) {
+  assert(index >= 0);
   BasemenuItem *item = find(index);
   if (! item) return False;
 
@@ -785,9 +730,8 @@ Bool Basemenu::isItemSelected(int index) {
 }
 
 
-void Basemenu::setItemEnabled(int index, Bool enable) {
-  if (index < 0 || index >= menuitems->count()) return;
-
+void Basemenu::setItemEnabled(int index, bool enable) {
+  assert(index >= 0);
   BasemenuItem *item = find(index);
   if (! item) return;
 
@@ -796,9 +740,8 @@ void Basemenu::setItemEnabled(int index, Bool enable) {
 }
 
 
-Bool Basemenu::isItemEnabled(int index) {
-  if (index < 0 || index >= menuitems->count()) return False;
-
+bool Basemenu::isItemEnabled(int index) {
+  assert(index >= 0);
   BasemenuItem *item = find(index);
   if (! item) return False;
 
@@ -808,19 +751,19 @@ Bool Basemenu::isItemEnabled(int index) {
 
 void Basemenu::buttonPressEvent(XButtonEvent *be) {
   if (be->window == menu.frame) {
-    int sbl = (be->x / menu.item_w), i = (be->y / menu.item_h);
-    int w = (sbl * menu.persub) + i;
-
-    if (w < menuitems->count() && w >= 0) {
+    const int sbl = (be->x / menu.item_w), i = (be->y / menu.item_h),
+      w = (sbl * menu.persub) + i;
+    
+    BasemenuItem *item = find(w);
+    if (item) {
       which_press = i;
       which_sbl = sbl;
 
-      BasemenuItem *item = menuitems->find(w);
 
       if (item->submenu())
-       drawSubmenu(w);
+        drawSubmenu(w);
       else
-       drawItem(w, (item->isEnabled()), True);
+        drawItem(w, (item->isEnabled()), True);
     }
   } else {
     menu.x_move = be->x_root - menu.x;
@@ -833,37 +776,38 @@ void Basemenu::buttonReleaseEvent(XButtonEvent *re) {
   if (re->window == menu.title) {
     if (moving) {
       moving = False;
-      
+
       if (which_sub != -1)
-       drawSubmenu(which_sub);
+        drawSubmenu(which_sub);
     }
-    
-    if (re->x >= 0 && re->x <= (signed) menu.width &&
-       re->y >= 0 && re->y <= (signed) menu.title_h)
+
+    if (re->x >= 0 && re->x <= static_cast<signed>(menu.width) &&
+        re->y >= 0 && re->y <= static_cast<signed>(menu.title_h))
       if (re->button == 3)
-       hide();
+        hide();
   } else if (re->window == menu.frame &&
-            re->x >= 0 && re->x < (signed) menu.width &&
-            re->y >= 0 && re->y < (signed) menu.frame_h) {
+             re->x >= 0 && re->x < static_cast<signed>(menu.width) &&
+             re->y >= 0 && re->y < static_cast<signed>(menu.frame_h)) {
     if (re->button == 3) {
       hide();
     } else {
-      int sbl = (re->x / menu.item_w), i = (re->y / menu.item_h),
-          ix = sbl * menu.item_w, iy = i * menu.item_h,
-           w = (sbl * menu.persub) + i,
-           p = (which_sbl * menu.persub) + which_press;
+      const int sbl = (re->x / menu.item_w), i = (re->y / menu.item_h),
+        ix = sbl * menu.item_w, iy = i * menu.item_h,
+        w = (sbl * menu.persub) + i,
+        p = (which_sbl * menu.persub) + which_press;
 
-      if (w < menuitems->count() && w >= 0) {
-       drawItem(p, (p == which_sub), True);
+      if (w >= 0 && w < static_cast<signed>(menuitems.size())) {
+        drawItem(p, (p == which_sub), True);
 
         if  (p == w && isItemEnabled(w)) {
-         if (re->x > ix && re->x < (signed) (ix + menu.item_w) &&
-             re->y > iy && re->y < (signed) (iy + menu.item_h)) {
-           itemSelected(re->button, w);
-         }
+          if (re->x > ix && re->x < static_cast<signed>(ix + menu.item_w) &&
+              re->y > iy && re->y < static_cast<signed>(iy + menu.item_h)) {
+            itemSelected(re->button, w);
+          }
         }
-      } else
+      } else {
         drawItem(p, False, True);
+      }
     }
   }
 }
@@ -873,55 +817,57 @@ void Basemenu::motionNotifyEvent(XMotionEvent *me) {
   if (me->window == menu.title && (me->state & Button1Mask)) {
     if (movable) {
       if (! moving) {
-       if (parent && (! torn)) {
-         parent->drawItem(parent->which_sub, False, True);
-         parent->which_sub = -1;
-       }
+        if (parent && ! torn) {
+          parent->drawItem(parent->which_sub, False, True);
+          parent->which_sub = -1;
+        }
 
         moving = torn = True;
 
-       if (which_sub != -1)
-         drawSubmenu(which_sub);
+        if (which_sub != -1)
+          drawSubmenu(which_sub);
       } else {
-       menu.x = me->x_root - menu.x_move,
-       menu.y = me->y_root - menu.y_move;
-       
-       XMoveWindow(display, menu.window, menu.x, menu.y);
-         
-       if (which_sub != -1)
-         drawSubmenu(which_sub);
+        menu.x = me->x_root - menu.x_move,
+          menu.y = me->y_root - menu.y_move;
+
+        XMoveWindow(display, menu.window, menu.x, menu.y);
+
+        if (which_sub != -1)
+          drawSubmenu(which_sub);
       }
     }
-  } else if ((! (me->state & Button1Mask)) && me->window == menu.frame &&
-            me->x >= 0 && me->x < (signed) menu.width &&
-            me->y >= 0 && me->y < (signed) menu.frame_h) {
-    int sbl = (me->x / menu.item_w), i = (me->y / menu.item_h),
-         w = (sbl * menu.persub) + i;
+  } else if (me->window == menu.frame &&
+             me->x >= 0 && me->x < static_cast<signed>(menu.width) &&
+             me->y >= 0 && me->y < static_cast<signed>(menu.frame_h)) {
+    const int sbl = (me->x / menu.item_w), i = (me->y / menu.item_h),
+      w = (sbl * menu.persub) + i;
 
     if ((i != which_press || sbl != which_sbl) &&
-       (w < menuitems->count() && w >= 0)) {
+        (w >= 0 && w < static_cast<signed>(menuitems.size()))) {
       if (which_press != -1 && which_sbl != -1) {
-       int p = (which_sbl * menu.persub) + which_press;
-       BasemenuItem *item = menuitems->find(p);
-
-       drawItem(p, False, True);
-       if (item->submenu())
-         if (item->submenu()->isVisible() &&
-             (! item->submenu()->isTorn())) {
-           item->submenu()->internal_hide();
-           which_sub = -1;
-         }
+        const int p = (which_sbl * menu.persub) + which_press;
+        BasemenuItem *item = find(p);
+        if (! item) return;
+
+        drawItem(p, False, True);
+        if (item->submenu() &&
+            item->submenu()->isVisible() &&
+            ! item->submenu()->isTorn()) {
+          item->submenu()->internal_hide();
+          which_sub = -1;
+        }
       }
 
       which_press = i;
       which_sbl = sbl;
 
-      BasemenuItem *itmp = menuitems->find(w);
-
+      BasemenuItem *itmp = find(w);
+      if (! itmp) return;
+      
       if (itmp->submenu())
-       drawSubmenu(w);
+        drawSubmenu(w);
       else
-       drawItem(w, (itmp->isEnabled()), True);
+        drawItem(w, (itmp->isEnabled()), True);
     }
   }
 }
@@ -931,29 +877,29 @@ void Basemenu::exposeEvent(XExposeEvent *ee) {
   if (ee->window == menu.title) {
     redrawTitle();
   } else if (ee->window == menu.frame) {
-    // this is a compilicated algorithm... lets do it step by step...
+    // this is a complicated algorithm... lets do it step by step...
     // first... we see in which sub level the expose starts... and how many
     // items down in that sublevel
 
-    int sbl = (ee->x / menu.item_w), id = (ee->y / menu.item_h),
+    const int sbl = (ee->x / menu.item_w), id = (ee->y / menu.item_h),
       // next... figure out how many sublevels over the redraw spans
-      sbl_d = ((ee->x + ee->width) / menu.item_w),
+      sbl_d = ((ee->x + ee->width) / menu.item_w);
       // then we see how many items down to redraw
-      id_d = ((ee->y + ee->height) / menu.item_h);
+    int id_d = ((ee->y + ee->height) / menu.item_h);
 
     if (id_d > menu.persub) id_d = menu.persub;
 
     // draw the sublevels and the number of items the exposure spans
-    LinkedListIterator<BasemenuItem> it(menuitems);
-    int i, ii;
-    for (i = sbl; i <= sbl_d; i++) {
+    MenuItems::iterator it,
+      end = menuitems.end();
+    for (int i = sbl; i <= sbl_d; i++) {
       // set the iterator to the first item in the sublevel needing redrawing
-      it.set(id + (i * menu.persub));
-      for (ii = id; ii <= id_d && it.current(); it++, ii++) {
-       int index = ii + (i * menu.persub);
-       // redraw the item
-       drawItem(index, (which_sub == index), False,
-                ee->x, ee->y, ee->width, ee->height);
+      it = menuitems.begin() + (id + (i * menu.persub));
+      for (int ii = id; ii <= id_d && it != end; ++it, ii++) {
+        const int index = ii + (i * menu.persub);
+        // redraw the item
+        drawItem(index, (which_sub == index), False,
+                 ee->x, ee->y, ee->width, ee->height);
       }
     }
   }
@@ -963,21 +909,21 @@ void Basemenu::exposeEvent(XExposeEvent *ee) {
 void Basemenu::enterNotifyEvent(XCrossingEvent *ce) {
   if (ce->window == menu.frame) {
     menu.x_shift = menu.x, menu.y_shift = menu.y;
-    if (menu.x + menu.width > screen.getWidth()) {
-      menu.x_shift = screen.getWidth() - menu.width -
-        screen.getBorderWidth();
+    if (menu.x + menu.width > screen->getWidth()) {
+      menu.x_shift = screen->getWidth() - menu.width -
+        screen->getBorderWidth();
       shifted = True;
     } else if (menu.x < 0) {
-      menu.x_shift = -screen.getBorderWidth();
+      menu.x_shift = -screen->getBorderWidth();
       shifted = True;
     }
 
-    if (menu.y + menu.height > screen.getHeight()) {
-      menu.y_shift = screen.getHeight() - menu.height -
-        screen.getBorderWidth();
+    if (menu.y + menu.height > screen->getHeight()) {
+      menu.y_shift = screen->getHeight() - menu.height -
+        screen->getBorderWidth();
       shifted = True;
-    } else if (menu.y + (signed) menu.title_h < 0) {
-      menu.y_shift = -screen.getBorderWidth();
+    } else if (menu.y + static_cast<signed>(menu.title_h) < 0) {
+      menu.y_shift = -screen->getBorderWidth();
       shifted = True;
     }
 
@@ -985,17 +931,17 @@ void Basemenu::enterNotifyEvent(XCrossingEvent *ce) {
       XMoveWindow(display, menu.window, menu.x_shift, menu.y_shift);
 
     if (which_sub != -1) {
-      BasemenuItem *tmp = menuitems->find(which_sub);
+      BasemenuItem *tmp = find(which_sub);
       if (tmp->submenu()->isVisible()) {
-       int sbl = (ce->x / menu.item_w), i = (ce->y / menu.item_h),
-         w = (sbl * menu.persub) + i;
+        const int sbl = (ce->x / menu.item_w), i = (ce->y / menu.item_h),
+          w = (sbl * menu.persub) + i;
 
-       if (w != which_sub && (! tmp->submenu()->isTorn())) {
-         tmp->submenu()->internal_hide();
+        if (w != which_sub && ! tmp->submenu()->isTorn()) {
+          tmp->submenu()->internal_hide();
 
-         drawItem(which_sub, False, True);
-         which_sub = -1;
-       }
+          drawItem(which_sub, False, True);
+          which_sub = -1;
+        }
       }
     }
   }
@@ -1004,8 +950,8 @@ void Basemenu::enterNotifyEvent(XCrossingEvent *ce) {
 
 void Basemenu::leaveNotifyEvent(XCrossingEvent *ce) {
   if (ce->window == menu.frame) {
-    if (which_press != -1 && which_sbl != -1 && menuitems->count() > 0) {
-      int p = (which_sbl * menu.persub) + which_press;
+    if (which_press != -1 && which_sbl != -1 && menuitems.size() > 0) {
+      const int p = (which_sbl * menu.persub) + which_press;
 
       drawItem(p, (p == which_sub), True);
 
@@ -1024,11 +970,18 @@ void Basemenu::leaveNotifyEvent(XCrossingEvent *ce) {
 
 void Basemenu::reconfigure(void) {
   XSetWindowBackground(display, menu.window,
-                      screen.getBorderColor()->getPixel());
+                       screen->getBorderColor()->pixel());
   XSetWindowBorder(display, menu.window,
-                  screen.getBorderColor()->getPixel());
-  XSetWindowBorderWidth(display, menu.window, screen.getBorderWidth());
+                   screen->getBorderColor()->pixel());
+  XSetWindowBorderWidth(display, menu.window, screen->getBorderWidth());
 
-  menu.bevel_w = screen.getBevelWidth();
+  menu.bevel_w = screen->getBevelWidth();
   update();
 }
+
+
+void Basemenu::changeItemLabel(unsigned int index, const string& label) {
+  BasemenuItem *item = find(index);
+  if (item)
+    item->newLabel(label);
+}
This page took 0.06614 seconds and 4 git commands to generate.