Title: [225146] trunk/Source/_javascript_Core
Revision
225146
Author
[email protected]
Date
2017-11-26 07:56:02 -0800 (Sun, 26 Nov 2017)

Log Message

Make ArgList::data() private again when we can remove callWasmFunction().
https://bugs.webkit.org/show_bug.cgi?id=168582

Reviewed by JF Bastien.

Make ArgList::data() private since we already removed callWasmFunction.

* runtime/ArgList.h:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (225145 => 225146)


--- trunk/Source/_javascript_Core/ChangeLog	2017-11-25 16:59:40 UTC (rev 225145)
+++ trunk/Source/_javascript_Core/ChangeLog	2017-11-26 15:56:02 UTC (rev 225146)
@@ -1,3 +1,14 @@
+2017-11-26  Yusuke Suzuki  <[email protected]>
+
+        Make ArgList::data() private again when we can remove callWasmFunction().
+        https://bugs.webkit.org/show_bug.cgi?id=168582
+
+        Reviewed by JF Bastien.
+
+        Make ArgList::data() private since we already removed callWasmFunction.
+
+        * runtime/ArgList.h:
+
 2016-08-05  Darin Adler  <[email protected]>
 
         Fix some minor problems in the StringImpl header

Modified: trunk/Source/_javascript_Core/runtime/ArgList.h (225145 => 225146)


--- trunk/Source/_javascript_Core/runtime/ArgList.h	2017-11-25 16:59:40 UTC (rev 225145)
+++ trunk/Source/_javascript_Core/runtime/ArgList.h	2017-11-26 15:56:02 UTC (rev 225146)
@@ -198,13 +198,9 @@
         
     JS_EXPORT_PRIVATE void getSlice(int startIndex, ArgList& result) const;
 
-    // FIXME: This is only made public as a work around for jsc's test helper function,
-    // callWasmFunction() to use. Make this a private method again once we can remove
-    // callWasmFunction().
-    // https://bugs.webkit.org/show_bug.cgi?id=168582
+private:
     JSValue* data() const { return m_args; }
 
-private:
     JSValue* m_args;
     int m_argCount;
 };
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to