QtDeclarative: Fix compiler warnings.

- QString from ASCII conversion, remove global-static QString
  variable from class.
- Signed/unsigned comparison.
- Unused variable.

Change-Id: I4ea3e72da85ba7ed329196d39085d7bba8bfa389
Reviewed-by: Michael Brasser <michael.brasser@nokia.com>

3 files changed, 3 insertions, 6 deletions

diff --git a/src/qml/qml/qqmlrewrite.cpp b/src/qml/qml/qqmlrewrite.cpp
index bbb17b631d..83ede9456b 100644
--- a/src/qml/qml/qqmlrewrite.cpp
+++ b/src/qml/qml/qqmlrewrite.cpp
@@ -47,12 +47,10 @@
 
 QT_BEGIN_NAMESPACE
 
-DEFINE_BOOL_CONFIG_OPTION(rewriteDump, QML_REWRITE_DUMP);
+DEFINE_BOOL_CONFIG_OPTION(rewriteDump, QML_REWRITE_DUMP)
 
 namespace QQmlRewrite {
 
-QString SharedBindingTester::evalString("eval");
-
 static void rewriteStringLiteral(AST::StringLiteral *ast, const QString *code, int startPosition, TextWriter *writer)
 {
     const unsigned position = ast->firstSourceLocation().begin() - startPosition + 1;
diff --git a/src/qml/qml/qqmlrewrite_p.h b/src/qml/qml/qqmlrewrite_p.h
index efa10ce316..064398115a 100644
--- a/src/qml/qml/qqmlrewrite_p.h
+++ b/src/qml/qml/qqmlrewrite_p.h
@@ -73,8 +73,6 @@ public:
     inline virtual bool visit(AST::FunctionDeclaration *);
     inline virtual bool visit(AST::FunctionExpression *);
     inline virtual bool visit(AST::IdentifierExpression *);
-
-    static QString evalString;
 };
 
 class RewriteBinding: protected AST::Visitor
@@ -159,6 +157,7 @@ bool SharedBindingTester::visit(AST::FunctionExpression *)
 
 bool SharedBindingTester::visit(AST::IdentifierExpression *e)
 {
+    static const QString evalString = QStringLiteral("eval");
     if (e->name == evalString)
         _sharable = false;
 
diff --git a/src/qml/qml/v8/qqmlbuiltinfunctions.cpp b/src/qml/qml/v8/qqmlbuiltinfunctions.cpp
index 9c570d756b..27a339221b 100644
--- a/src/qml/qml/v8/qqmlbuiltinfunctions.cpp
+++ b/src/qml/qml/v8/qqmlbuiltinfunctions.cpp
@@ -1141,7 +1141,7 @@ v8::Handle<v8::Value> createComponent(const v8::Arguments &args)
     // Default to engine parent; this will be removed in the near future (QTBUG-24841)
     QObject *parentArg = engine;
 
-    unsigned consumedCount = 1;
+    int consumedCount = 1;
     if (args.Length() > 1) {
         const v8::Local<v8::Value> &lastArg = args[args.Length()-1];