Reviewers: Mike Belshe,

Description:
Split the RUNTIME_FUNCTION_LIST_ALWAYS macro into two as this macro is
very large for Visual Studio's
intellisense operation thus causing it to crash.

Fixes bug http://code.google.com/p/chromium/issues/detail?id=11837

Bug=11837


Please review this at http://codereview.chromium.org/113310

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
   M     src/runtime.h


Index: src/runtime.h
===================================================================
--- src/runtime.h       (revision 1918)
+++ src/runtime.h       (working copy)
@@ -37,7 +37,7 @@
  // release and debug mode.
  // This macro should only be used by the macro RUNTIME_FUNCTION_LIST.

-#define RUNTIME_FUNCTION_LIST_ALWAYS(F) \
+#define RUNTIME_FUNCTION_LIST_ALWAYS_1(F) \
    /* Property access */ \
    F(GetProperty, 2) \
    F(KeyedGetProperty, 2) \
@@ -153,8 +153,9 @@
    F(NumberToRadixString, 2) \
    F(NumberToFixed, 2) \
    F(NumberToExponential, 2) \
-  F(NumberToPrecision, 2) \
-  \
+  F(NumberToPrecision, 2)
+
+#define RUNTIME_FUNCTION_LIST_ALWAYS_2(F) \
    /* Reflection */ \
    F(FunctionSetInstanceClassName, 2) \
    F(FunctionSetLength, 2) \
@@ -183,7 +184,6 @@
    F(GetTemplateField, 2) \
    F(DisableAccessChecks, 1) \
    F(EnableAccessChecks, 1) \
-  \
    /* Dates */ \
    F(DateCurrentTime, 0) \
    F(DateParseString, 2) \
@@ -320,7 +320,8 @@
  // via a native call by name (from within JS code).

  #define RUNTIME_FUNCTION_LIST(F) \
-  RUNTIME_FUNCTION_LIST_ALWAYS(F) \
+  RUNTIME_FUNCTION_LIST_ALWAYS_1(F) \
+  RUNTIME_FUNCTION_LIST_ALWAYS_2(F) \
    RUNTIME_FUNCTION_LIST_DEBUG(F) \
    RUNTIME_FUNCTION_LIST_DEBUGGER_SUPPORT(F)




--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to