fix some 17-06 build breaks and CI copyright check failure

chakra-core · Jun 15, 2017 · 86b66ee · 86b66ee
1 parent bb95963
commit 86b66ee
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 8 deletions.
diff --git a/lib/Common/CommonDefines.h b/lib/Common/CommonDefines.h
@@ -8,13 +8,6 @@
 #include "Warnings.h"
 #include "ChakraCoreVersion.h"
 
-#if defined(NTBUILD) && defined(_CHAKRACOREBUILD)
-#error NTBUILD and _CHAKRACOREBUILD cannot be both defined
-#endif
-#if !defined(NTBUILD) && !defined(_CHAKRACOREBUILD)
-#error Either NTBUILD or _CHAKRACOREBUILD must be defined
-#endif
-
 //----------------------------------------------------------------------------------------------------
 // Default debug/fretest/release flags values
 //  - Set the default values of debug/fretest/release flags if it is not set by the command line

diff --git a/lib/Runtime/Library/JavascriptArray.cpp b/lib/Runtime/Library/JavascriptArray.cpp
@@ -5253,6 +5253,7 @@ namespace Js
 
         if (hasInlineSegment)
         {
+            AnalysisAssert(array->head);
             SparseArraySegment<T>* newHeadSeg = array->ReallocNonLeafSegment((SparseArraySegment<T>*)PointerValue(array->head), array->head->next);
             array->head = newHeadSeg;
         }
@@ -5274,6 +5275,7 @@ namespace Js
 
         if (SparseArraySegmentBase::IsLeafSegment(lastSeg, recycler))
         {
+            AnalysisAssert(lastSeg);
             arr->ReallocNonLeafSegment((SparseArraySegment<T>*)lastSeg, lastSeg->next, true /*forceNonLeaf*/);
         }
     }

diff --git a/lib/Runtime/Library/JavascriptArray.inl b/lib/Runtime/Library/JavascriptArray.inl
@@ -110,7 +110,7 @@ namespace Js
         Recycler *recycler = this->GetScriptContext()->GetRecycler();
         if (forceNonLeaf)
         {
-            newSeg = SparseArraySegment<T>::AllocateSegmentImpl<false /*isLeaf*/>(recycler, seg->left, seg->length, nextSeg);
+            newSeg = SparseArraySegment<T>::template AllocateSegmentImpl<false /*isLeaf*/>(recycler, seg->left, seg->length, nextSeg);
         }
         else
         {

diff --git a/test/LetConst/shadowedsetter.js b/test/LetConst/shadowedsetter.js
@@ -1,3 +1,8 @@
+//-------------------------------------------------------------------------------------------------------
+// Copyright (C) Microsoft. All rights reserved.
+// Licensed under the MIT license. See LICENSE.txt file in the project root for full license information.
+//-------------------------------------------------------------------------------------------------------
+
 evaluate = WScript.LoadScript;
 
 __defineSetter__("x", function () { });