The else path is never reached -> removed it (thanks to Simon St James for the hint)

CCMAIL: kdedevel@etotheipiplusone.com

svn path=/trunk/KDE/kdebase/apps/; revision=863623

diff --git a/src/tooltipmanager.cpp b/src/tooltipmanager.cpp
index 231f4db15fe39491f2c1a85d1f24a542928f9592..ad1dbe5d3d7776ee6cdb640402a619abcd174c05 100644 (file)
--- a/src/tooltipmanager.cpp
+++ b/src/tooltipmanager.cpp
@@ -230,14 +230,8 @@ void ToolTipManager::showToolTip(KToolTipItem* tip)
         }
         y = hasRoomBelow ? m_itemRect.bottom() : m_itemRect.top() - size.height();
     } else {
-        if (hasRoomToLeft || hasRoomToRight) {
-            x = hasRoomToRight ? m_itemRect.right() : m_itemRect.left() - size.width();
-        } else {
-            // Put the tooltip at the far right of the screen. The item will be overlapped
-            // horizontally, but the y-coordinate will be adjusted afterwards so that no overlapping
-            // occurs vertically.
-            x = desktop.right() - size.width();
-        }
+        Q_ASSERT(hasRoomToLeft || hasRoomToRight);
+        x = hasRoomToRight ? m_itemRect.right() : m_itemRect.left() - size.width();
         
         // Put the tooltip at the bottom of the screen. The x-coordinate has already
         // been adjusted, so that no overlapping with m_itemRect occurs.