From 4afad4cc62b2db50b12bf476677ef5fb9b7c9458 Mon Sep 17 00:00:00 2001
From: Jedrzej Nowacki <jedrzej.nowacki@nokia.com>
Date: Thu, 4 Aug 2011 11:17:01 +0200
Subject: [PATCH] Code cleanup.

Remove redundant else conditions.

Change-Id: I5795bd931d9204a31e0edfea0d3489c119db1020
Reviewed-on: http://codereview.qt.nokia.com/2700
Reviewed-by: Qt Sanity Bot <qt_sanity_bot@ovi.com>
Reviewed-by: Simon Hausmann <simon.hausmann@nokia.com>
---
 src/declarative/qml/v8/qv8engine.cpp | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/src/declarative/qml/v8/qv8engine.cpp b/src/declarative/qml/v8/qv8engine.cpp
index 3964134..a3dfef1 100644
--- a/src/declarative/qml/v8/qv8engine.cpp
+++ b/src/declarative/qml/v8/qv8engine.cpp
@@ -475,13 +475,13 @@ QVariant QV8Engine::toBasicVariant(v8::Handle<v8::Value> value)
 {
     if (value->IsNull() || value->IsUndefined())
         return QVariant();
-    else if (value->IsBoolean())
+    if (value->IsBoolean())
         return value->ToBoolean()->Value();
-    else if (value->IsInt32())
+    if (value->IsInt32())
         return value->ToInt32()->Value();
-    else if (value->IsNumber())
+    if (value->IsNumber())
         return value->ToNumber()->Value();
-    else if (value->IsString())
+    if (value->IsString())
         return m_stringWrapper.toString(value->ToString());
     if (value->IsDate())
         return qtDateTimeFromJsDate(v8::Handle<v8::Date>::Cast(value)->NumberValue());
@@ -492,7 +492,8 @@ QVariant QV8Engine::toBasicVariant(v8::Handle<v8::Value> value)
     if (value->IsRegExp()) {
         v8::Context::Scope scope(context());
         return QJSConverter::toRegExp(v8::Handle<v8::RegExp>::Cast(value));
-    } else if (value->IsArray()) {
+    }
+    if (value->IsArray()) {
         v8::Context::Scope scope(context());
         QVariantList rv;
 
@@ -502,7 +503,8 @@ QVariant QV8Engine::toBasicVariant(v8::Handle<v8::Value> value)
             rv << toVariant(array->Get(ii), -1);
 
         return rv;
-    } else if (!value->IsFunction()) {
+    }
+    if (!value->IsFunction()) {
         v8::Context::Scope scope(context());
         v8::Handle<v8::Object> object = value->ToObject();
         v8::Local<v8::Array> properties = object->GetPropertyNames();
@@ -2203,9 +2205,9 @@ QVariant QV8Engine::variantFromJS(v8::Handle<v8::Value> value)
         return QVariant();
     if (value->IsBoolean())
         return value->ToBoolean()->Value();
-    else if (value->IsInt32())
+    if (value->IsInt32())
         return value->ToInt32()->Value();
-    else if (value->IsNumber())
+    if (value->IsNumber())
         return value->ToNumber()->Value();
     if (value->IsString())
         return QJSConverter::toString(value->ToString());
-- 
2.7.4