cosmetic fixes: removed redundant ';' + some whitespaces

nls/fluxbox-nls.hh

@@ -253,6 +253,6 @@ enum {
 
 }; // end enum
 
-}; // end namespace FBNLS
+} // end namespace FBNLS
 
 #endif // FLUXBOX_NLS_HH

src/ButtonTool.hh

@@ -31,7 +31,7 @@ class ButtonTheme;
 namespace FbTk {
 class Button;
 class ImageControl;
-};
+}
 
 class ButtonTool: public GenericTool {
 public:

src/ClientMenu.cc

@@ -77,7 +77,7 @@ private:
     Focusable &m_client;
 };
 
-}; // end anonymous namespace
+} // end anonymous namespace
 
 ClientMenu::ClientMenu(BScreen &screen, Focusables &clients,
                        bool listen_for_iconlist_changes):

src/CurrentWindowCmd.cc

@@ -109,7 +109,7 @@ REGISTER_COMMAND_PARSER(movetabright, createCurrentWindowCmd, void);
 REGISTER_COMMAND_PARSER(detachclient, createCurrentWindowCmd, void);
 REGISTER_COMMAND_PARSER(windowmenu, createCurrentWindowCmd, void);
 
-}; // end anonymous namespace
+} // end anonymous namespace
 
 void WindowHelperCmd::execute() {
     if (WindowCmd<void>::window() || FocusControl::focusedFbWindow())
@@ -201,7 +201,7 @@ FbTk::Command<void> *parseFocusCmd(const string &command, const string &args,
 REGISTER_COMMAND_PARSER(activate, parseFocusCmd, void);
 REGISTER_COMMAND_PARSER(focus, parseFocusCmd, void);
 
-}; // end anonymous namespace
+} // end anonymous namespace
 
 void SetHeadCmd::real_execute() {
     int num = m_head;

src/Ewmh.cc

@@ -288,7 +288,7 @@ void extractNetWmIcon(Atom net_wm_icon, WinClient& winclient) {
     winclient.setIcon(icon);
 }
 
-}; // end anonymous namespace
+} // end anonymous namespace
 
 class Ewmh::EwmhAtoms {
 public:
@@ -1535,3 +1535,4 @@ void Ewmh::updateFrameExtents(FluxboxWindow &win) {
                               (unsigned char *)extents, 4);
     }
 }
+

src/FbCommands.cc

@@ -466,7 +466,7 @@ void SetResourceValueDialogCmd::execute() {
 
     FbTk::FbWindow *win = new CommandDialog(*screen,  "Type resource name and the value", "SetResourceValue ");
     win->show();
-};
+}
 
 REGISTER_UNTRUSTED_COMMAND_WITH_ARGS(bindkey, FbCommands::BindKeyCmd, void);
 

src/FbTk/Color.cc

@@ -37,7 +37,7 @@ inline unsigned char maxValue(unsigned short colval) {
    return colval == 65535 ? 0xFF : static_cast<unsigned char>(colval/0xFF);
 }
 
-};
+}
 
 namespace FbTk {
 
@@ -184,4 +184,4 @@ void Color::setRGB(unsigned short red, unsigned short green, unsigned short blue
     m_blue = blue;
 }
 
-};
+}

src/FbTk/FbPixmap.cc

@@ -66,7 +66,7 @@ void checkAtoms() {
     }
 }
 
-}; // end of anonymous namespace
+} // end of anonymous namespace
 
 FbPixmap::FbPixmap():m_pm(0),
                      m_width(0), m_height(0),

src/FbTk/FbWindow.cc

@@ -87,7 +87,7 @@ FbWindow::FbWindow(int screen_num,
     create(RootWindow(display(), screen_num),
            x, y, width, height, eventmask,
            override_redirect, save_unders, depth, class_type);
-};
+}
 
 FbWindow::FbWindow(const FbWindow &parent,
                    int x, int y, unsigned int width, unsigned int height,
@@ -105,7 +105,7 @@ FbWindow::FbWindow(const FbWindow &parent,
            override_redirect, save_unders, depth, class_type);
 
 
-};
+}
 
 FbWindow::FbWindow(Window client):
     FbDrawable(),
@@ -686,4 +686,4 @@ bool operator == (Window win, const FbWindow &fbwin) {
     return win == fbwin.window();
 }
 
-};
+}

src/FbTk/FileUtil.hh

@@ -58,7 +58,7 @@ namespace FileUtil {
     /// copies file 'from' to 'to'
     bool copyFile(const char* from, const char* to);
 
-}; // end of File namespace
+} // end of File namespace
 
 ///  Wrapper class for DIR * routines
 class Directory : private FbTk::NotCopyable {

src/FbTk/Font.cc

@@ -100,7 +100,7 @@ void resetEffects(FbTk::Font& font) {
     font.setShadowOffX(2);
 }
 
-}; // end nameless namespace
+} // end nameless namespace
 
 
 
@@ -301,5 +301,5 @@ void Font::drawText(const FbDrawable &w, int screen, GC gc,
 
 }
 
-};
+}
 

src/FbTk/Image.cc

@@ -59,7 +59,7 @@ FbTk::ImageXPM xpm_loader;
 #endif
 
 
-}; // end of anonymous namespace
+} // end of anonymous namespace
 
 namespace FbTk {
 

src/FbTk/Image.hh

@@ -50,7 +50,7 @@ namespace Image {
     void removeAllSearchPaths();
     /// locates an image in the search path
     std::string locateFile(const std::string &filename);
-};
+}
 
 /// common interface for all image classes
 class ImageBase {

src/FbTk/ImageControl.cc

@@ -94,7 +94,7 @@ inline unsigned long bsqrt(unsigned long x) {
     }
 }
 
-}; // end anonymous namespace
+} // end anonymous namespace
 
 struct ImageControl::Cache {
     Pixmap pixmap;

src/FbTk/ImageImlib2.cc

@@ -35,7 +35,7 @@ typedef ScreenImlibContextContainer::iterator ScreenImlibContext;
 
 ScreenImlibContextContainer contexts;
 
-}; // anon namespace
+} // anon namespace
 
 
 namespace FbTk {

src/FbTk/KeyUtil.cc

@@ -55,7 +55,7 @@ const struct t_modlist modlist[] = {
     {0, 0}
 };
 
-};
+}
 
 namespace FbTk {
 

src/FbTk/LogicCommands.cc

@@ -77,7 +77,7 @@ REGISTER_COMMAND_PARSER(and, parseLogicCommand, bool);
 REGISTER_COMMAND_PARSER(or, parseLogicCommand, bool);
 REGISTER_COMMAND_PARSER(xor, parseLogicCommand, bool);
 
-}; // end anonymous namespace
+} // end anonymous namespace
 
 Command<void> *IfCommand::parse(const std::string &command, const std::string &args,
                           bool trusted) {

src/FbTk/MacroCommand.cc

@@ -66,7 +66,7 @@ Command<void> *parseMacroCmd(const std::string &command, const std::string &args
 REGISTER_COMMAND_PARSER(macrocmd, parseMacroCmd, void);
 REGISTER_COMMAND_PARSER(togglecmd, parseMacroCmd, void);
 
-}; // end anonymous namespace
+} // end anonymous namespace
 
 void MacroCommand::add(RefCount<Command<void> > &com) {
     m_commandlist.push_back(com);

src/FbTk/MultLayers.hh

@@ -67,5 +67,5 @@ private:
     int m_lock;
 };
 
-};
+}
 #endif // FBTK_MULTLAYERS_HH

src/FbTk/Observer.cc

@@ -28,4 +28,4 @@ Observer::~Observer() {
     Subject::removeObserver(this); // make sure no subject has this observer attached
 }
 
-};
+}

src/FbTk/Parser.cc

@@ -23,4 +23,4 @@
 
 namespace FbTk {
     const Parser::Item Parser::s_empty_item("", "");
-};
+}

src/FbTk/Shape.cc

@@ -103,7 +103,7 @@ struct CornerPixmaps {
 // unfortunately, we need a separate pixmap per screen
 std::vector<CornerPixmaps> s_corners;
 
-}; // end of anonymous namespace
+} // end of anonymous namespace
 
 Shape::Shape(FbWindow &win, int shapeplaces):
     m_win(&win),

src/FbTk/SignalHandler.cc

@@ -36,7 +36,7 @@ SignalHandler &SignalHandler::instance() {
     return singleton;
 }
 
-bool SignalHandler::registerHandler(int signum, SignalEventHandler *eh, 
+bool SignalHandler::registerHandler(int signum, SignalEventHandler *eh,
                                     SignalEventHandler **oldhandler_ret) {
     // must be less than NSIG
     if (signum >= NSIG)
@@ -45,18 +45,18 @@ bool SignalHandler::registerHandler(int signum, SignalEventHandler *eh,
     // get old signal handler for this signum
     if (oldhandler_ret != 0)
         *oldhandler_ret = s_signal_handler[signum];
-	
+
     struct sigaction sa;
     // set callback
     sa.sa_handler = SignalHandler::handleSignal;
     sigemptyset (&sa.sa_mask);
     sa.sa_flags = 0;
-	
+
     if (sigaction(signum, &sa, 0) == -1)
         return false;
-	
+
     s_signal_handler[signum] = eh;
-	
+
     return true;
 }
 
@@ -69,4 +69,5 @@ void SignalHandler::handleSignal(int signum) {
     }
 }
 
-}; 
+}
+

src/FbTk/Transparent.cc

@@ -103,7 +103,7 @@ Picture createAlphaPic(Window drawable, unsigned char alpha) {
     return alpha_pic;
 }
 #endif //  HAVE_XRENDER
-};
+}
 
 namespace FbTk {
 

src/FbTk/XFontImp.cc

@@ -410,4 +410,4 @@ bool XFontImp::validOrientation(FbTk::Orientation orient) {
     return m_rotfonts[orient] != 0;
 }
 
-};
+}

src/FbTk/XmbFontImp.cc

@@ -158,7 +158,8 @@ XFontSet createFontSet(const char *fontname, bool& utf8mode) {
     return fs;
 }
 
-};
+}
+
 namespace FbTk {
 
 XmbFontImp::XmbFontImp(const char *filename, bool utf8) : m_fontset(0), m_setextents(0), m_utf8mode(utf8) {

src/FocusControl.cc

@@ -62,20 +62,20 @@ bool doSkipWindow(const Focusable &win, const ClientPattern *pat) {
     return false; // else don't skip
 }
 
-}; // end anonymous namespace
+} // end anonymous namespace
-    
+
 FocusControl::FocusControl(BScreen &screen):
     m_screen(screen),
-    m_focus_model(screen.resourceManager(), 
+    m_focus_model(screen.resourceManager(),
-                  CLICKFOCUS, 
+                  CLICKFOCUS,
-                  screen.name()+".focusModel", 
+                  screen.name()+".focusModel",
                   screen.altName()+".FocusModel"),
-    m_tab_focus_model(screen.resourceManager(), 
+    m_tab_focus_model(screen.resourceManager(),
-                      CLICKTABFOCUS, 
+                      CLICKTABFOCUS,
-                      screen.name()+".tabFocusModel", 
+                      screen.name()+".tabFocusModel",
                       screen.altName()+".TabFocusModel"),
-    m_focus_new(screen.resourceManager(), true, 
+    m_focus_new(screen.resourceManager(), true,
-                screen.name()+".focusNewWindows", 
+                screen.name()+".focusNewWindows",
                 screen.altName()+".FocusNewWindows"),
     m_focused_list(screen), m_creation_order_list(screen),
     m_focused_win_list(screen), m_creation_order_win_list(screen),
@@ -85,7 +85,7 @@ FocusControl::FocusControl(BScreen &screen):
     m_ignore_mouse_x(-1), m_ignore_mouse_y(-1) {
 
     m_cycling_window = m_focused_list.clientList().end();
-    
+
 }
 
 void FocusControl::cycleFocus(const FocusableList &window_list,
@@ -252,7 +252,7 @@ Focusable *FocusControl::lastFocusedWindow(int workspace) {
     if (workspace < 0 || workspace >= (int) m_screen.numberOfWorkspaces())
         return m_focused_list.clientList().front();
 
-    Focusables::iterator it = m_focused_list.clientList().begin();    
+    Focusables::iterator it = m_focused_list.clientList().begin();
     Focusables::iterator it_end = m_focused_list.clientList().end();
     for (; it != it_end; ++it) {
         if ((*it)->fbwindow() && (*it)->acceptsFocus() &&
@@ -307,11 +307,11 @@ void FocusControl::dirFocus(FluxboxWindow &win, FocusDir dir) {
 
     // we scan through the list looking for the window that is "closest"
     // in the given direction
-    
+
     FluxboxWindow *foundwin = 0;
     int weight = 999999, exposure = 0; // extreme values
     int borderW = win.frame().window().borderWidth(),
-        top = win.y() + borderW, 
+        top = win.y() + borderW,
         bottom = win.y() + win.height() + borderW,
         left = win.x() + borderW,
         right = win.x() + win.width() + borderW;
@@ -319,17 +319,17 @@ void FocusControl::dirFocus(FluxboxWindow &win, FocusDir dir) {
     Workspace::Windows &wins = m_screen.currentWorkspace()->windowList();
     Workspace::Windows::iterator it = wins.begin();
     for (; it != wins.end(); ++it) {
-        if ((*it) == &win 
+        if ((*it) == &win
-            || (*it)->isIconic() 
+            || (*it)->isIconic()
-            || (*it)->isFocusHidden() 
+            || (*it)->isFocusHidden()
-            || !(*it)->acceptsFocus()) 
+            || !(*it)->acceptsFocus())
             continue; // skip self
-        
+
         // we check things against an edge, and within the bounds (draw a picture)
         int edge=0, upper=0, lower=0, oedge=0, oupper=0, olower=0;
 
-        int otop = (*it)->y() + borderW, 
+        int otop = (*it)->y() + borderW,
-            // 2 * border = border on each side 
+            // 2 * border = border on each side
             obottom = (*it)->y() + (*it)->height() + borderW,
             oleft = (*it)->x() + borderW,
             // 2 * border = border on each side
@@ -371,7 +371,7 @@ void FocusControl::dirFocus(FluxboxWindow &win, FocusDir dir) {
             break;
         }
 
-        if (oedge < edge) 
+        if (oedge < edge)
             continue; // not in the right direction
 
         if (olower <= upper || oupper >= lower) {
@@ -396,7 +396,7 @@ void FocusControl::dirFocus(FluxboxWindow &win, FocusDir dir) {
         } // else not improvement
     }
 
-    if (foundwin) 
+    if (foundwin)
         foundwin->focus();
 
 }
@@ -516,7 +516,7 @@ void FocusControl::revertFocus(BScreen &screen) {
  * assumption: client has focus
  */
 void FocusControl::unfocusWindow(WinClient &client,
-                                 bool full_revert, 
+                                 bool full_revert,
                                  bool unfocus_frame) {
     // go up the transient tree looking for a focusable window
 
@@ -547,8 +547,8 @@ void FocusControl::setFocusedWindow(WinClient *client) {
         return;
 
     BScreen *screen = client ? &client->screen() : 0;
-    BScreen *old_screen = 
+    BScreen *old_screen =
-        FocusControl::focusedWindow() ? 
+        FocusControl::focusedWindow() ?
         &FocusControl::focusedWindow()->screen() : 0;
 
 #ifdef DEBUG
@@ -567,10 +567,10 @@ void FocusControl::setFocusedWindow(WinClient *client) {
 
     if (client && client->fbwindow() && !client->fbwindow()->isIconic()) {
         // screen should be ok
-        s_focused_fbwindow = client->fbwindow();        
+        s_focused_fbwindow = client->fbwindow();
         s_focused_window = client;     // update focused window
         s_expecting_focus = 0;
-        s_focused_fbwindow->setCurrentClient(*client, 
+        s_focused_fbwindow->setCurrentClient(*client,
                               false); // don't set inputfocus
         s_focused_fbwindow->setFocusFlag(true); // set focus flag
 
@@ -604,9 +604,9 @@ std::string FbTk::Resource<FocusControl::FocusModel>::getString() const {
 template<>
 void FbTk::Resource<FocusControl::FocusModel>::
 setFromString(char const *strval) {
-    if (strcasecmp(strval, "MouseFocus") == 0) 
+    if (strcasecmp(strval, "MouseFocus") == 0)
         m_value = FocusControl::MOUSEFOCUS;
-    else if (strcasecmp(strval, "ClickToFocus") == 0) 
+    else if (strcasecmp(strval, "ClickToFocus") == 0)
         m_value = FocusControl::CLICKFOCUS;
     else
         setDefaultValue();
@@ -630,9 +630,11 @@ setFromString(char const *strval) {
 
     if (strcasecmp(strval, "SloppyTabFocus") == 0 )
         m_value = FocusControl::MOUSETABFOCUS;
-    else if (strcasecmp(strval, "ClickToTabFocus") == 0) 
+    else if (strcasecmp(strval, "ClickToTabFocus") == 0)
         m_value = FocusControl::CLICKTABFOCUS;
     else
         setDefaultValue();
 }
+
 } // end namespace FbTk
+

src/IconbarTool.cc

@@ -241,7 +241,7 @@ private:
     Focusable &m_win;
 };
 
-}; // end anonymous namespace
+} // end anonymous namespace
 
 IconbarTool::IconbarTool(const FbTk::FbWindow &parent, IconbarTheme &theme,
                          FbTk::ThemeProxy<IconbarTheme> &focused_theme,

src/MenuCreator.cc

@@ -364,7 +364,7 @@ bool getStart(FbMenuParser &parser, string &label, FbTk::StringConvertor &labelc
     return true;
 }
 
-}; // end of anonymous namespace
+} // end of anonymous namespace
 
 FbMenu *MenuCreator::createMenu(const string &label, int screen_number) {
     BScreen *screen = Fluxbox::instance()->findScreen(screen_number);

src/Remember.cc

@@ -337,7 +337,7 @@ FbTk::Menu *createRememberMenu(BScreen &screen) {
 
     menu->updateMenu();
     return menu;
-};
+}
 
 // offset is the offset in the string that we start looking from
 // return true if all ok, false on error
@@ -395,7 +395,7 @@ bool handleStartupItem(const string &line, int offset) {
     tmp_exec_cmd->execute();
     delete tmp_exec_cmd;
     return true;
-};
+}
 
 
 
@@ -609,7 +609,7 @@ Application* findMatchingPatterns(ClientPattern *pat, Remember::Patterns *patlis
     return 0;
 }
 
-}; // end anonymous namespace
+} // end anonymous namespace
 
 /*------------------------------------------------------------------*\
 \*------------------------------------------------------------------*/

src/Remember.hh

@@ -41,7 +41,7 @@ class Application;
 
 namespace FbTk {
 class AutoReloadHelper;
-};
+}
 
 /**
  * Class Remember is an atomhandler to avoid interfering with

src/Slit.cc

@@ -215,7 +215,7 @@ private:
     Slit::Placement m_place;
 };
 
-}; // End anonymous namespace
+} // End anonymous namespace
 
 unsigned int Slit::s_eventmask = SubstructureRedirectMask |  ButtonPressMask |
                                  EnterWindowMask | LeaveWindowMask | ExposureMask;

src/ToolFactory.cc

@@ -66,7 +66,7 @@ private:
     Toolbar &m_tbar;
 };
 
-};
+}
 
 ToolFactory::ToolFactory(BScreen &screen):m_screen(screen),
     m_clock_theme(screen.screenNumber(), "toolbar.clock", "Toolbar.Clock"),

src/ToolFactory.hh

@@ -35,7 +35,7 @@ class Toolbar;
 
 namespace FbTk {
 class FbWindow;
-};
+}
 
 /// creates toolbaritems
 class ToolFactory:private FbTk::NotCopyable {

src/Toolbar.cc

@@ -177,7 +177,7 @@ private:
     Toolbar::Placement m_place;
 };
 
-}; // end anonymous
+} // end anonymous
 
 // toolbar frame
 Toolbar::Frame::Frame(FbTk::EventHandler &evh, int screen_num):

src/Window.cc

@@ -259,7 +259,7 @@ private:
     int m_mode;
 };
 
-};
+}
 
 
 int FluxboxWindow::s_num_grabs = 0;

src/WorkspaceCmd.cc

@@ -189,7 +189,7 @@ REGISTER_COMMAND_PARSER(prevwindow, parseWindowList, void);
 REGISTER_COMMAND_PARSER(prevgroup, parseWindowList, void);
 REGISTER_COMMAND_PARSER(arrangewindows, parseWindowList, void);
 
-}; // end anonymous namespace
+} // end anonymous namespace
 
 void AttachCmd::execute() {
     BScreen *screen = Fluxbox::instance()->keyScreen();
@@ -318,7 +318,7 @@ REGISTER_COMMAND_PARSER(rightworkspace, parseIntCmd, void);
 REGISTER_COMMAND_PARSER(leftworkspace, parseIntCmd, void);
 REGISTER_COMMAND_PARSER(workspace, parseIntCmd, void);
 
-}; // end anonymous namespace
+} // end anonymous namespace
 
 void NextWorkspaceCmd::execute() {
     BScreen *screen = Fluxbox::instance()->mouseScreen();

util/fbsetroot.hh

@@ -27,7 +27,7 @@ namespace FbTk {
 
 class ImageControl;
 
-};
+}
 
 class fbsetroot : public FbTk::App {
 public:

util/fluxbox-update_configs.cc

@@ -463,7 +463,7 @@ set<string> modified_files;
 // we may want to put a size limit on this cache, so it doesn't grow too big
 map<string,string> file_cache;
 
-};
+}
 
 // returns the contents of the file given, either from the cache or by reading
 // the file from disk