Fix warning about returning address of local variable by MSVC2015 in qv4engine.cpp.
authorFriedemann Kleint <Friedemann.Kleint@theqtcompany.com>
Mon, 6 Jul 2015 10:11:19 +0000 (12:11 +0200)
committerFriedemann Kleint <Friedemann.Kleint@theqtcompany.com>
Fri, 17 Jul 2015 13:44:21 +0000 (13:44 +0000)
qtdeclarative\src\qml\jsruntime\qv4engine.cpp(179) : warning C4172: returning address of local variable or temporary: dummy

Disable warning as using the address is intended.

Change-Id: Ide894a8dc2fb94f11d0455723c46567c84d91f8d
Reviewed-by: Simon Hausmann <simon.hausmann@theqtcompany.com>
src/qml/jsruntime/qv4engine.cpp

index 0f2b44fac6dd20683658580d2fbbbd2372098c9d..e1282eeca99082098febfdbceeb3148017af5fc6 100644 (file)
@@ -108,6 +108,9 @@ static ReturnedValue throwTypeError(CallContext *ctx)
 
 const int MinimumStackSize = 256; // in kbytes
 
+QT_WARNING_PUSH
+QT_WARNING_DISABLE_MSVC(4172) // MSVC 2015: warning C4172: returning address of local variable or temporary: dummy
+
 quintptr getStackLimit()
 {
     quintptr stackLimit;
@@ -172,6 +175,7 @@ quintptr getStackLimit()
     int dummy;
     // this is inexact, as part of the stack is used when being called here,
     // but let's simply default to 1MB from where the stack is right now
+    // (Note: triggers warning C4172 as of MSVC 2015, returning address of local variable)
     stackLimit = reinterpret_cast<qintptr>(&dummy) - 1024*1024;
 #endif
 
@@ -179,6 +183,7 @@ quintptr getStackLimit()
     return stackLimit + MinimumStackSize*1024;
 }
 
+QT_WARNING_POP
 
 QJSEngine *ExecutionEngine::jsEngine() const
 {