diff options
author | Jedrzej Nowacki <jedrzej.nowacki@nokia.com> | 2011-08-04 11:17:01 +0200 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2011-08-10 12:49:30 +0200 |
commit | 4afad4cc62b2db50b12bf476677ef5fb9b7c9458 (patch) | |
tree | 96673061b7dc5b17ee6f2f91e93340e539c38dba | |
parent | 26f31e663f56ab60f620ccf50005e6c85b654242 (diff) |
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>
-rw-r--r-- | src/declarative/qml/v8/qv8engine.cpp | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/src/declarative/qml/v8/qv8engine.cpp b/src/declarative/qml/v8/qv8engine.cpp index 3964134201..a3dfef1dd9 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()); |