@@ -1,6 +1,10 @@
 (Format: Year/Month/Day)
 Changes for 0.9.16:
 *06/04/25:
+   * Fix up comments for doxygen (Thanks Jonas Koelker)
+     Screen.cc CommandParser.cc Container.hh fluxbox.hh Workspace.hh 
+     ArrowButton.hh FbTk/Texture.cc FbTk/ThemeItems.cc FbTk/Texture.hh 
+     FbTk/Theme.hh FbTk/EventHandler.hh fluxbox.cc Remember.cc
    * Output the error message on fallback restart execs (sf patch#1475213)
      (Thanks Jonas Kölker, jonaskoelker at users.sourceforge.net).
      main.cc     

@@ -31,10 +31,10 @@ class ArrowButton: public FbTk::Button {
 public:
     enum Type { LEFT, RIGHT, UP, DOWN};
 
-    ArrowButton(Type arrow_type, const FbTk::FbWindow &parent,
+    ArrowButton(ArrowButton::Type arrow_type, const FbTk::FbWindow &parent,
                 int x, int y, 
                 unsigned int width, unsigned int height);
-    ArrowButton(Type arrow_type, int screen_num,
+    ArrowButton(ArrowButton::Type arrow_type, int screen_num,
                 int x, int y,
                 unsigned int width, unsigned int height);
     void clear();

@@ -45,7 +45,7 @@ CommandParser::instance().removeAssociation(*this);
 
 }
 
-void CommandFactory::addCommand(const string &command_name) {
+void CommandFactory::addCommand(const std::string &command_name) {
     CommandParser::instance().associateCommand(command_name, *this);
 }
 
@@ -62,7 +62,7 @@
     return *s_singleton;
 }
 
-FbTk::Command *CommandParser::parseLine(const string &line) {
+FbTk::Command *CommandParser::parseLine(const std::string &line) {
 
     // parse arguments and command
     string command = line;

@@ -97,7 +97,7 @@ unsigned int maxTotalSize() const { return m_max_total_size; }
     inline unsigned int maxHeightPerClient() const { return (empty() ? height() : height()/size()); }    
     inline bool updateLock() const { return m_update_lock; }
 
-    void for_each(std::mem_fun_t<void, FbWindow> function);
+    void for_each(std::mem_fun_t<void, FbTk::FbWindow> function);
     void setAlpha(unsigned char alpha); // set alpha on all windows
 
     ItemList::iterator begin() { return m_item_list.begin(); }

@@ -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 @@ // DEALINGS IN THE SOFTWARE.
 
 // $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

@@ -77,15 +77,19 @@ else if (strstr(ts, "vertical"))
                 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"))

@@ -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 @@ CROSSDIAGONAL =  0x00200,
         RECTANGLE =      0x00400,
         PYRAMID =        0x00800,
         PIPECROSS =      0x01000,
-        ELLIPTIC =       0x02000		
+        ELLIPTIC =       0x02000
     };
 	
     enum {

@@ -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

@@ -21,7 +21,7 @@ // DEALINGS IN THE SOFTWARE.
 
 // $Id$
 
-/// @file implements common theme items
+/// @file ThemeItems.cc implements common theme items
 
 #ifndef THEMEITEMS_HH
 #define THEMEITEMS_HH
@@ -71,7 +71,7 @@ *(*this) = (str ? 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 @@ }
 
 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
 }
 

@@ -297,7 +297,7 @@ m_pats->push_back(make_pair(p, app));
     return app;
 }
 
-int Remember::parseApp(ifstream &file, Application &app, string *first_line) {
+int Remember::parseApp(std::ifstream &file, Application &app, std::string *first_line) {
     string line;
     _FB_USES_NLS;
     int row = 0;

@@ -293,7 +293,8 @@
 }
 
 BScreen::BScreen(FbTk::ResourceManager &rm,
-                 const string &screenname, const string &altscreenname,
+                 const std::string &screenname,
+                 const std::string &altscreenname,
                  int scrn, int num_layers) : 
     m_clientlist_sig(*this),  // client signal
     m_iconlist_sig(*this), // icon list signal

@@ -42,8 +42,8 @@ class FluxboxWindow;
 class WinClient;
 
 /**
-    Handles a single workspace
-*/
+ * Handles a single workspace
+ */
 class Workspace:private FbTk::NotCopyable, private FbTk::Observer {
 public:
     typedef std::vector<FluxboxWindow *> Windows;
@@ -54,13 +54,17 @@ ~Workspace();
 
     void setLastFocusedWindow(FluxboxWindow *w);
 
-    ///   Set workspace name
+    /// Set workspace name
     void setName(const std::string &name);
+    /// Deiconify all windows on this workspace
     void showAll();
     void hideAll(bool interrupt_moving);
+    /// Iconify all windows on this workspace
     void removeAll();
     void reconfigure();
     void shutdown();
+
+    /// Add @a win to this workspace, placing it if @a place is true
     void addWindow(FluxboxWindow &win, bool place = false);
     int removeWindow(FluxboxWindow *win, bool still_alive);
     void updateClientmenu();

@@ -1511,7 +1511,7 @@ return m_rc_file;
 }
 
 /// Provides default filename of data file
-void Fluxbox::getDefaultDataFilename(char *name, string &filename) {
+void Fluxbox::getDefaultDataFilename(char *name, std::string &filename) {
     filename = string(getenv("HOME") + string("/.") + m_RC_PATH + string("/") + name);
 }
 

@@ -199,7 +199,7 @@ /// @return whether the timestamps on the menu changed
     bool menuTimestampsChanged() const;
     bool haveShape() const { return m_have_shape; }
     int shapeEventbase() const { return m_shape_eventbase; }
-    void getDefaultDataFilename(char *, std::string &);
+    void getDefaultDataFilename(char *name, std::string &);
     // screen mouse was in at last key event
     BScreen *mouseScreen() { return m_mousescreen; }
     // screen of window that last key event (i.e. focused window) went to