fix a bunch of comments, make default textures obvious

Thanks Jonas Koelker

5 files changed, 19 insertions, 12 deletions

diff --git a/src/FbTk/EventHandler.hh b/src/FbTk/EventHandler.hh
index 4ae1aa5..62e2cb9 100644
--- a/src/FbTk/EventHandler.hh
+++ b/src/FbTk/EventHandler.hh
@@ -1,4 +1,4 @@
-// EventHandler.cc for Fluxbox Window Manager
+// EventHandler.hh for Fluxbox Window Manager
 // Copyright (c) 2002-2003 Henrik Kinnunen (fluxgen at fluxbox dot org)
 //
 // Permission is hereby granted, free of charge, to any person obtaining a
@@ -21,7 +21,7 @@
 
 // $Id$
 
-/// @file holds EventHandler interface for X events
+/// @file EventHandler.hh holds EventHandler interface for X events
 
 #ifndef FBTK_EVENTHANDLER_HH
 #define FBTK_EVENTHANDLER_HH
diff --git a/src/FbTk/Texture.cc b/src/FbTk/Texture.cc
index e06c328..0395cd5 100644
--- a/src/FbTk/Texture.cc
+++ b/src/FbTk/Texture.cc
@@ -77,15 +77,19 @@ void Texture::setFromString(const char * const texture_str) {
                 addType(Texture::VERTICAL);
             else
                 addType(Texture::DIAGONAL);
-        } else // default is "solid", according to ThemeItems.cc
+        } else if (strstr(ts, "solid"))
             addType(Texture::SOLID);
+        else
+            addType(Texture::DEFAULT_BEVEL);
 
         if (strstr(ts, "raised"))
             addType(Texture::RAISED);
         else if (strstr(ts, "sunken"))
             addType(Texture::SUNKEN);
-        else // default us "flat", according to ThemeItems.cc
+        else if (strstr(ts, "flat"))
             addType(Texture::FLAT);
+        else
+            addType(Texture::DEFAULT_TEXTURE);
 
         if (! (type() & Texture::FLAT))
             if (strstr(ts, "bevel2"))
diff --git a/src/FbTk/Texture.hh b/src/FbTk/Texture.hh
index 162cf44..35b628b 100644
--- a/src/FbTk/Texture.hh
+++ b/src/FbTk/Texture.hh
@@ -39,15 +39,18 @@ class Texture {
 public:
 
     enum Bevel {
+        // why are we not using the lowest-order bit?
         FLAT =     0x00002,
-        SUNKEN =   0x00004, 
-        RAISED =   0x00008
+        SUNKEN =   0x00004,
+        RAISED =   0x00008,
+        DEFAULT_BEVEL = FLAT,
     };
 
     enum Textures {
         NONE =     0x00000,
         SOLID =    0x00010,
-        GRADIENT = 0x00020
+        GRADIENT = 0x00020,
+        DEFAULT_TEXTURE = SOLID,
     };
 
     enum Gradients {
@@ -58,7 +61,7 @@ public:
         RECTANGLE =      0x00400,
         PYRAMID =        0x00800,
         PIPECROSS =      0x01000,
-        ELLIPTIC =       0x02000                
+        ELLIPTIC =       0x02000
     };
         
     enum {
diff --git a/src/FbTk/Theme.hh b/src/FbTk/Theme.hh
index 5edc840..e3fc5b4 100644
--- a/src/FbTk/Theme.hh
+++ b/src/FbTk/Theme.hh
@@ -22,7 +22,7 @@
 // $Id$
 
 /**
- @file holds ThemeItem<T>, Theme and ThemeManager which is the base for any theme
+ @file Theme.hh holds ThemeItem<T>, Theme and ThemeManager which is the base for any theme
 */
 
 #ifndef FBTK_THEME_HH
diff --git a/src/FbTk/ThemeItems.cc b/src/FbTk/ThemeItems.cc
index 41d05e8..2f58b2a 100644
--- a/src/FbTk/ThemeItems.cc
+++ b/src/FbTk/ThemeItems.cc
@@ -21,7 +21,7 @@
 
 // $Id$
 
-/// @file implements common theme items
+/// @file ThemeItems.cc implements common theme items
 
 #ifndef THEMEITEMS_HH
 #define THEMEITEMS_HH
@@ -71,7 +71,7 @@ void ThemeItem<string>::setFromString(const char *str) {
 }
 
 template <>
-void ThemeItem<int>::load(const string *name, const string *altname) { }
+void ThemeItem<int>::load(const std::string *name, const std::string *altname) { }
 
 template<>
 void ThemeItem<int>::setDefaultValue() {
@@ -233,7 +233,7 @@ void ThemeItem<Texture>::load(const string *o_name, const string *o_altname) {
 
 template <>
 void ThemeItem<Texture>::setDefaultValue() {
-    m_value.setType(Texture::FLAT | Texture::SOLID);
+    m_value.setType(Texture::DEFAULT_BEVEL | Texture::DEFAULT_TEXTURE);
     load(); // one might forget to add line something:  so we try to load something.*:  too
 }