Title: [261544] trunk/Source/_javascript_Core
Revision
261544
Author
[email protected]
Date
2020-05-11 20:52:22 -0700 (Mon, 11 May 2020)

Log Message

Fix build errors after r260992
https://bugs.webkit.org/show_bug.cgi?id=211756

Reviewed by Darin Adler.

Add JSC namespace specifier to NonIntrinsic and PropertyAttribute
in the macros in JSObject.h since those can be used outside of
or without introducing JSC namespace.
* runtime/JSObject.h:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (261543 => 261544)


--- trunk/Source/_javascript_Core/ChangeLog	2020-05-12 03:06:23 UTC (rev 261543)
+++ trunk/Source/_javascript_Core/ChangeLog	2020-05-12 03:52:22 UTC (rev 261544)
@@ -1,3 +1,15 @@
+2020-05-11  Yoshiaki JITSUKAWA  <[email protected]>
+
+        Fix build errors after r260992
+        https://bugs.webkit.org/show_bug.cgi?id=211756
+
+        Reviewed by Darin Adler.
+
+        Add JSC namespace specifier to NonIntrinsic and PropertyAttribute
+        in the macros in JSObject.h since those can be used outside of
+        or without introducing JSC namespace. 
+        * runtime/JSObject.h:
+
 2020-05-11  Ross Kirsling  <[email protected]>
 
         [clang-tidy] Run modernize-use-override over JSC, then ensure as much as possible is final

Modified: trunk/Source/_javascript_Core/runtime/JSObject.h (261543 => 261544)


--- trunk/Source/_javascript_Core/runtime/JSObject.h	2020-05-12 03:06:23 UTC (rev 261543)
+++ trunk/Source/_javascript_Core/runtime/JSObject.h	2020-05-12 03:52:22 UTC (rev 261544)
@@ -1624,10 +1624,10 @@
 // As above, but this assumes that the function you're defining doesn't have an
 // intrinsic.
 #define JSC_NATIVE_FUNCTION(jsName, cppName, attributes, length) \
-    JSC_NATIVE_INTRINSIC_FUNCTION(jsName, cppName, (attributes), (length), NoIntrinsic)
+    JSC_NATIVE_INTRINSIC_FUNCTION(jsName, cppName, (attributes), (length), JSC::NoIntrinsic)
 
 #define JSC_NATIVE_FUNCTION_WITHOUT_TRANSITION(jsName, cppName, attributes, length) \
-    JSC_NATIVE_INTRINSIC_FUNCTION_WITHOUT_TRANSITION(jsName, cppName, (attributes), (length), NoIntrinsic)
+    JSC_NATIVE_INTRINSIC_FUNCTION_WITHOUT_TRANSITION(jsName, cppName, (attributes), (length), JSC::NoIntrinsic)
 
 // Identical helpers but for builtins. Note that currently, we don't support builtins that are
 // also intrinsics, but we probably will do that eventually.
@@ -1641,24 +1641,24 @@
 
 #define JSC_TO_STRING_TAG_WITHOUT_TRANSITION() \
     putDirectWithoutTransition(vm, vm.propertyNames->toStringTagSymbol, \
-        jsNontrivialString(vm, info()->className), PropertyAttribute::DontEnum | PropertyAttribute::ReadOnly)
+        jsNontrivialString(vm, info()->className), JSC::PropertyAttribute::DontEnum | JSC::PropertyAttribute::ReadOnly)
 
 // Helper for defining native getters on properties.
 #define JSC_NATIVE_INTRINSIC_GETTER(jsName, cppName, attributes, intrinsic)  \
     putDirectNativeIntrinsicGetter(\
         vm, globalObject, makeIdentifier(vm, (jsName)), (cppName), \
-        (intrinsic), ((attributes) | PropertyAttribute::Accessor))
+        (intrinsic), ((attributes) | JSC::PropertyAttribute::Accessor))
 
 #define JSC_NATIVE_INTRINSIC_GETTER_WITHOUT_TRANSITION(jsName, cppName, attributes, intrinsic)  \
     putDirectNativeIntrinsicGetterWithoutTransition(\
         vm, globalObject, makeIdentifier(vm, (jsName)), (cppName), \
-        (intrinsic), ((attributes) | PropertyAttribute::Accessor))
+        (intrinsic), ((attributes) | JSC::PropertyAttribute::Accessor))
 
 #define JSC_NATIVE_GETTER(jsName, cppName, attributes) \
-    JSC_NATIVE_INTRINSIC_GETTER((jsName), (cppName), (attributes), NoIntrinsic)
+    JSC_NATIVE_INTRINSIC_GETTER((jsName), (cppName), (attributes), JSC::NoIntrinsic)
 
 #define JSC_NATIVE_GETTER_WITHOUT_TRANSITION(jsName, cppName, attributes) \
-    JSC_NATIVE_INTRINSIC_GETTER_WITHOUT_TRANSITION((jsName), (cppName), (attributes), NoIntrinsic)
+    JSC_NATIVE_INTRINSIC_GETTER_WITHOUT_TRANSITION((jsName), (cppName), (attributes), JSC::NoIntrinsic)
 
 
 #define STATIC_ASSERT_ISO_SUBSPACE_SHARABLE(DerivedClass, BaseClass) \
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to