merge the C branch into HEAD

[chaz/openbox] / otk / screeninfo.cc

diff --git a/otk/screeninfo.cc b/otk/screeninfo.cc
index c1c0c2acc80e563d52e58272241e592660170015..368ae982893f4d90dd65ff57daecd662acfbb38c 100644 (file)
--- a/otk/screeninfo.cc
+++ b/otk/screeninfo.cc
@@ -1,8 +1,6 @@
-// -*- mode: C++; indent-tabs-mode: nil; -*-
+// -*- mode: C++; indent-tabs-mode: nil; c-basic-offset: 2; -*-
 
-#ifdef    HAVE_CONFIG_H
-#  include "../config.h"
-#endif // HAVE_CONFIG_H
+#include "config.h"
 
 extern "C" {
 #include <X11/Xlib.h>
@@ -11,78 +9,34 @@ extern "C" {
 
 #include "screeninfo.hh"
 #include "display.hh"
-#include "src/util.hh"
+#include "util.hh"
 
 using std::string;
 
 namespace otk {
 
-ScreenInfo::ScreenInfo(unsigned int num) {
-  screen_number = num;
-
-  root_window = RootWindow(OBDisplay::display, screen_number);
-
-  rect.setSize(WidthOfScreen(ScreenOfDisplay(OBDisplay::display,
-                                             screen_number)),
-               HeightOfScreen(ScreenOfDisplay(OBDisplay::display,
-                                              screen_number)));
-  /*
-    If the default depth is at least 8 we will use that,
-    otherwise we try to find the largest TrueColor visual.
-    Preference is given to 24 bit over larger depths if 24 bit is an option.
-  */
-
-  depth = DefaultDepth(OBDisplay::display, screen_number);
-  visual = DefaultVisual(OBDisplay::display, screen_number);
-  colormap = DefaultColormap(OBDisplay::display, screen_number);
+ScreenInfo::ScreenInfo(int num) {
+  assert(num >= 0 && num < ScreenCount(**display));
   
-  if (depth < 8) {
-    // search for a TrueColor Visual... if we can't find one...
-    // we will use the default visual for the screen
-    XVisualInfo vinfo_template, *vinfo_return;
-    int vinfo_nitems;
-    int best = -1;
-
-    vinfo_template.screen = screen_number;
-    vinfo_template.c_class = TrueColor;
+  _screen = num;
 
-    vinfo_return = XGetVisualInfo(OBDisplay::display,
-                                  VisualScreenMask | VisualClassMask,
-                                  &vinfo_template, &vinfo_nitems);
-    if (vinfo_return) {
-      int max_depth = 1;
-      for (int i = 0; i < vinfo_nitems; ++i) {
-        if (vinfo_return[i].depth > max_depth) {
-          if (max_depth == 24 && vinfo_return[i].depth > 24)
-            break;          // prefer 24 bit over 32
-          max_depth = vinfo_return[i].depth;
-          best = i;
-        }
-      }
-      if (max_depth < depth) best = -1;
-    }
-
-    if (best != -1) {
-      depth = vinfo_return[best].depth;
-      visual = vinfo_return[best].visual;
-      colormap = XCreateColormap(OBDisplay::display, root_window, visual,
-                                 AllocNone);
-    }
-
-    XFree(vinfo_return);
-  }
+  _root_window = RootWindow(**display, _screen);
 
+  _size = Size(WidthOfScreen(ScreenOfDisplay(**display,
+                                             _screen)),
+               HeightOfScreen(ScreenOfDisplay(**display,
+                                              _screen)));
   // get the default display string and strip the screen number
-  string default_string = DisplayString(OBDisplay::display);
+  string default_string = DisplayString(**display);
   const string::size_type pos = default_string.rfind(".");
   if (pos != string::npos)
     default_string.resize(pos);
 
-  display_string = string("DISPLAY=") + default_string + '.' +
-    ob::itostring(static_cast<unsigned long>(screen_number));
-  
-#ifdef    XINERAMA
-  xinerama_active = False;
+  _display_string = string("DISPLAY=") + default_string + '.' +
+    itostring(static_cast<unsigned long>(_screen));
+
+#if 0 //def    XINERAMA
+  _xinerama_active = False;
 
   if (d->hasXineramaExtensions()) {
     if (d->getXineramaMajorVersion() == 1) {
@@ -93,7 +47,7 @@ ScreenInfo::ScreenInfo(unsigned int num) {
         in future versions we should be able, so the 'activeness' is checked
         on a pre-screen basis anyways.
       */
-      if (XineramaIsActive(OBDisplay::display)) {
+      if (XineramaIsActive(**display)) {
         /*
           If Xinerama is being used, there there is only going to be one screen
           present. We still, of course, want to use the screen class, but that
@@ -101,24 +55,27 @@ ScreenInfo::ScreenInfo(unsigned int num) {
           never be more than one screen present with Xinerama active.
         */
         int num;
-        XineramaScreenInfo *info = XineramaQueryScreens(OBDisplay::display,
-                                                        &num);
+        XineramaScreenInfo *info = XineramaQueryScreens(**display, &num);
         if (num > 0 && info) {
-          xinerama_areas.reserve(num);
+          _xinerama_areas.reserve(num);
           for (int i = 0; i < num; ++i) {
-            xinerama_areas.push_back(Rect(info[i].x_org, info[i].y_org,
-                                          info[i].width, info[i].height));
+            _xinerama_areas.push_back(Rect(info[i].x_org, info[i].y_org,
+                                           info[i].width, info[i].height));
           }
           XFree(info);
 
           // if we can't find any xinerama regions, then we act as if it is not
           // active, even though it said it was
-          xinerama_active = True;
+          _xinerama_active = true;
         }
       }
     }
   }
+#else
+  _xinerama_active = false;
 #endif // XINERAMA
+  if (!_xinerama_active)
+    _xinerama_areas.push_back(Rect(Point(0, 0), _size));
 }
 
 }