From 203fb838038d4c6c33d9e6ed3a45600c6e064e5a Mon Sep 17 00:00:00 2001
From: Robert Szefner <r.szefner@hydro-partner.pl>
Date: Sat, 25 Nov 2017 20:00:18 +0100
Subject: QSqlQuery: Remove temporary variable in navigation functions

Change-Id: I33836a75e1d2e5663f81a33a195d0cb21760e1f8
Reviewed-by: Andy Shaw <andy.shaw@qt.io>
---
 src/sql/kernel/qsqlquery.cpp | 17 +++++------------
 1 file changed, 5 insertions(+), 12 deletions(-)

(limited to 'src/sql/kernel')

diff --git a/src/sql/kernel/qsqlquery.cpp b/src/sql/kernel/qsqlquery.cpp
index f9cc07daa1..b89d20976f 100644
--- a/src/sql/kernel/qsqlquery.cpp
+++ b/src/sql/kernel/qsqlquery.cpp
@@ -649,11 +649,10 @@ bool QSqlQuery::next()
 {
     if (!isSelect() || !isActive())
         return false;
-    bool b = false;
+
     switch (at()) {
     case QSql::BeforeFirstRow:
-        b = d->sqlResult->fetchFirst();
-        return b;
+        return d->sqlResult->fetchFirst();
     case QSql::AfterLastRow:
         return false;
     default:
@@ -703,13 +702,11 @@ bool QSqlQuery::previous()
         return false;
     }
 
-    bool b = false;
     switch (at()) {
     case QSql::BeforeFirstRow:
         return false;
     case QSql::AfterLastRow:
-        b = d->sqlResult->fetchLast();
-        return b;
+        return d->sqlResult->fetchLast();
     default:
         if (!d->sqlResult->fetchPrevious()) {
             d->sqlResult->setAt(QSql::BeforeFirstRow);
@@ -737,9 +734,7 @@ bool QSqlQuery::first()
         qWarning("QSqlQuery::seek: cannot seek backwards in a forward only query");
         return false;
     }
-    bool b = false;
-    b = d->sqlResult->fetchFirst();
-    return b;
+    return d->sqlResult->fetchFirst();
 }
 
 /*!
@@ -758,9 +753,7 @@ bool QSqlQuery::last()
 {
     if (!isSelect() || !isActive())
         return false;
-    bool b = false;
-    b = d->sqlResult->fetchLast();
-    return b;
+    return d->sqlResult->fetchLast();
 }
 
 /*!
-- 
cgit v1.2.3