From: Peter Penz Date: Mon, 22 Sep 2008 18:00:41 +0000 (+0000) Subject: The else path is never reached -> removed it (thanks to Simon St James for the hint) X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/b5432d60372e2dad6d31cd4e6d28e4da5d47a7e4 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 231f4db15..ad1dbe5d3 100644 --- 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.