[Merge chakra-core/ChakraCore@e0141fb48b] [MERGE #3503 @sigatrev] Fix…

… Issue #3497: move initialization to constructor Merge pull request #3503 from sigatrev:ConstructorInitialization
nodejs · Aug 10, 2017 · a1a64d1 · a1a64d1
1 parent 569128f
commit a1a64d1
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/deps/chakrashim/core/lib/Backend/ValueInfo.h b/deps/chakrashim/core/lib/Backend/ValueInfo.h
@@ -548,7 +548,7 @@ class JsTypeValueInfo : public ValueInfo
 
     JsTypeValueInfo(const JsTypeValueInfo& other)
         : ValueInfo(Uninitialized, ValueStructureKind::JsType),
-        jsType(other.jsType), jsTypeSet(other.jsTypeSet)
+        jsType(other.jsType), jsTypeSet(other.jsTypeSet), isShared(false)
     {
     }
 
@@ -576,9 +576,7 @@ class JsTypeValueInfo : public ValueInfo
 public:
     JsTypeValueInfo * Copy(JitArenaAllocator *const allocator) const
     {
-        JsTypeValueInfo * newInfo = JitAnew(allocator, JsTypeValueInfo, *this);
-        newInfo->isShared = false;
-        return newInfo;
+        return JitAnew(allocator, JsTypeValueInfo, *this);
     }
 
     JITTypeHolder GetJsType() const