Simon's last changes lead to segfaults when the users specified <Sticky> or

<Workspace> in apps-file. Reason is, that an unmanaged window is not yet
assigned to an workspace and thus the returned workspacenumber is -1 ... which
leads in combination with getWorkspace(nr) to an invalid workspace (0x0) ...
This also fixes the problem with several deiconify-modes of the iconbar, when
the iconbar is in icons-only mode (same reason)

The other changes in the files are just cosmetic, i throw out the comments
about the changes Simon did coz they make sense and are correct imho.

1 files changed, 1 insertions, 4 deletions

diff --git a/src/IconButton.cc b/src/IconButton.cc
index 24bdddf..40310e6 100644
--- a/src/IconButton.cc
+++ b/src/IconButton.cc
@@ -81,20 +81,17 @@ public:
                     m_win.screen().sendToWorkspace(m_win.screen().currentWorkspaceID(), &m_win);
                 } else {
                     m_win.screen().changeWorkspaceID(m_win.workspaceNumber());
-                    m_win.raiseAndFocus();
                 }
                 break;
             case IconbarTool::CURRENT:
                 m_win.screen().sendToWorkspace(m_win.screen().currentWorkspaceID(), &m_win);
-                m_win.raiseAndFocus();
                 break;
             case IconbarTool::FOLLOW:
             default:
                 m_win.screen().changeWorkspaceID(m_win.workspaceNumber());
-                m_win.raiseAndFocus();
                 break;
             };
-
+            m_win.raiseAndFocus();
        } else
            m_win.iconify();
     }