fixed size_t issue with prevWorkspace, minor cosmetic changes to Screen.hh

2 files changed, 6 insertions, 6 deletions

diff --git a/src/Screen.cc b/src/Screen.cc
index 4eae1b6..5dd5f9a 100644
--- a/src/Screen.cc
+++ b/src/Screen.cc
@@ -1913,7 +1913,7 @@ void BScreen::nextWorkspace(const int delta) {
  Goes to the workspace "left" of the current
 */
 void BScreen::prevWorkspace(const int delta) {
-    changeWorkspaceID( (currentWorkspaceID() - delta + numberOfWorkspaces()) % numberOfWorkspaces());
+    changeWorkspaceID( (static_cast<signed>(numberOfWorkspaces()) + currentWorkspaceID() - (delta % numberOfWorkspaces())) % numberOfWorkspaces());
 }
 
 /**
diff --git a/src/Screen.hh b/src/Screen.hh
index 13f2fb9..ffac442 100644
--- a/src/Screen.hh
+++ b/src/Screen.hh
@@ -170,15 +170,15 @@ public:
     /// @return true if dock app was added, else false
     bool addKdeDockapp(Window win);
 
-    inline unsigned int width() const { return rootWindow().width(); }
-    inline unsigned int height() const { return rootWindow().height(); }
-    inline int screenNumber() const { return rootWindow().screenNumber(); }
+    unsigned int width() const { return rootWindow().width(); }
+    unsigned int height() const { return rootWindow().height(); }
+    int screenNumber() const { return rootWindow().screenNumber(); }
 
     /// @return number of workspaces
     size_t numberOfWorkspaces() const { return m_workspaces_list.size(); }
 
-    inline const Icons &iconList() const { return m_icon_list; }
-    inline Icons &iconList() { return m_icon_list; }
+    const Icons &iconList() const { return m_icon_list; }
+    Icons &iconList() { return m_icon_list; }
 
     const Workspaces &getWorkspacesList() const { return m_workspaces_list; }
     Workspaces &getWorkspacesList() { return m_workspaces_list; }