From d3496b12310d9fec0e52e537c341e87fcaa2f8b5 Mon Sep 17 00:00:00 2001 From: Peter Penz Date: Wed, 9 Feb 2011 19:21:58 +0100 Subject: Coding style update for pointer comparison Most developers seem to prefer if (ptr) ... if (!ptr) ... in comparison to if (ptr != 0) ... if (ptr == 0) ... Adjusted the Dolphin-code to use the "most-prefered style" to make contributors happy. --- src/views/dolphinviewautoscroller.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/views/dolphinviewautoscroller.cpp') diff --git a/src/views/dolphinviewautoscroller.cpp b/src/views/dolphinviewautoscroller.cpp index 4f645d026..5b338cc37 100644 --- a/src/views/dolphinviewautoscroller.cpp +++ b/src/views/dolphinviewautoscroller.cpp @@ -131,13 +131,13 @@ void DolphinViewAutoScroller::scrollViewport() } QScrollBar* verticalScrollBar = m_itemView->verticalScrollBar(); - if (verticalScrollBar != 0) { + if (verticalScrollBar) { const int value = verticalScrollBar->value(); verticalScrollBar->setValue(value + m_verticalScrollInc); } QScrollBar* horizontalScrollBar = m_itemView->horizontalScrollBar(); - if (horizontalScrollBar != 0) { + if (horizontalScrollBar) { const int value = horizontalScrollBar->value(); horizontalScrollBar->setValue(value + m_horizontalScrollInc); @@ -156,9 +156,9 @@ void DolphinViewAutoScroller::scrollViewport() void DolphinViewAutoScroller::triggerAutoScroll() { - const bool verticalScrolling = (m_itemView->verticalScrollBar() != 0) && + const bool verticalScrolling = m_itemView->verticalScrollBar() && m_itemView->verticalScrollBar()->isVisible(); - const bool horizontalScrolling = (m_itemView->horizontalScrollBar() != 0) && + const bool horizontalScrolling = m_itemView->horizontalScrollBar() && m_itemView->horizontalScrollBar()->isVisible(); if (!verticalScrolling && !horizontalScrolling) { // no scrollbars are shown at all, so no autoscrolling is necessary -- cgit v1.3