Reviewed by Maciej.
authorggaren <ggaren@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 12 Jun 2006 19:02:59 +0000 (19:02 +0000)
committerggaren <ggaren@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 12 Jun 2006 19:02:59 +0000 (19:02 +0000)
        - Have *.lut.h files #include lookup.h to eliminate surprising header
        include order dependency.

        * DerivedSources.make:
        * kjs/array_object.cpp:
        * kjs/date_object.cpp:
        * kjs/date_object.h:
        (KJS::DateProtoFunc::):
        * kjs/lexer.cpp:
        * kjs/math_object.cpp:
        * kjs/number_object.cpp:
        * kjs/regexp_object.cpp:
        * kjs/string_object.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@14821 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JavaScriptCore/ChangeLog
JavaScriptCore/DerivedSources.make
JavaScriptCore/kjs/array_object.cpp
JavaScriptCore/kjs/date_object.cpp
JavaScriptCore/kjs/date_object.h
JavaScriptCore/kjs/lexer.cpp
JavaScriptCore/kjs/math_object.cpp
JavaScriptCore/kjs/number_object.cpp
JavaScriptCore/kjs/regexp_object.cpp
JavaScriptCore/kjs/string_object.cpp

index 0342649..79590e6 100644 (file)
@@ -1,3 +1,21 @@
+2006-06-12  Geoffrey Garen  <ggaren@apple.com>
+
+        Reviewed by Maciej.
+        
+        - Have *.lut.h files #include lookup.h to eliminate surprising header
+        include order dependency.
+
+        * DerivedSources.make:
+        * kjs/array_object.cpp:
+        * kjs/date_object.cpp:
+        * kjs/date_object.h:
+        (KJS::DateProtoFunc::):
+        * kjs/lexer.cpp:
+        * kjs/math_object.cpp:
+        * kjs/number_object.cpp:
+        * kjs/regexp_object.cpp:
+        * kjs/string_object.cpp:
+
 2006-06-10  Geoffrey Garen  <ggaren@apple.com>
 
         - http://bugzilla.opendarwin.org/show_bug.cgi?id=8515
index d75e575..9a61c56 100644 (file)
@@ -44,7 +44,7 @@ all : \
 # lookup tables for classes
 
 %.lut.h: create_hash_table %.cpp
-       $^ > $@
+       $^ -i > $@
 lexer.lut.h: create_hash_table keywords.table
        $^ > $@
 
index 455c648..89c7f89 100644 (file)
 
 #include "config.h"
 #include "array_object.h"
+#include "array_object.lut.h"
 
 #include "error_object.h"
-#include "lookup.h"
 #include "operations.h"
 #include "reference_list.h"
 #include <wtf/HashSet.h>
 #include <stdio.h>
 
-#include "array_object.lut.h"
 
 using namespace KJS;
 
index 8776119..3363a6d 100644 (file)
@@ -21,6 +21,7 @@
 
 #include "config.h"
 #include "date_object.h"
+#include "date_object.lut.h"
 
 #if HAVE(ERRNO_H)
 #include <errno.h>
@@ -49,7 +50,6 @@
 #include <time.h>
 
 #include "error_object.h"
-#include "lookup.h"
 #include "operations.h"
 
 #if PLATFORM(MAC)
@@ -80,31 +80,6 @@ namespace KJS {
  * @internal
  *
  * Class to implement all methods that are properties of the
- * Date.prototype object
- */
-class DateProtoFunc : public InternalFunctionImp {
-public:
-    DateProtoFunc(ExecState *, int i, int len, const Identifier& date);
-
-    virtual JSValue *callAsFunction(ExecState *, JSObject *thisObj, const List &args);
-
-    enum { ToString, ToDateString, ToTimeString, ToLocaleString,
-           ToLocaleDateString, ToLocaleTimeString, ValueOf, GetTime,
-           GetFullYear, GetMonth, GetDate, GetDay, GetHours, GetMinutes,
-           GetSeconds, GetMilliSeconds, GetTimezoneOffset, SetTime,
-           SetMilliSeconds, SetSeconds, SetMinutes, SetHours, SetDate,
-           SetMonth, SetFullYear, ToUTCString,
-           // non-normative properties (Appendix B)
-           GetYear, SetYear, ToGMTString };
-private:
-    int id;
-    bool utc;
-};
-
-/**
- * @internal
- *
- * Class to implement all methods that are properties of the
  * Date object
  */
 class DateObjectFuncImp : public InternalFunctionImp {
@@ -119,12 +94,6 @@ private:
     int id;
 };
 
-}
-
-#include "date_object.lut.h"
-
-namespace KJS {
-
 // some constants
 const double hoursPerDay = 24;
 const double minutesPerHour = 60;
index 906b80d..3c8847b 100644 (file)
@@ -56,6 +56,31 @@ namespace KJS {
     };
 
     /**
+        * @internal
+     *
+     * Class to implement all methods that are properties of the
+     * Date.prototype object
+     */
+    class DateProtoFunc : public InternalFunctionImp {
+    public:
+        DateProtoFunc(ExecState *, int i, int len, const Identifier& date);
+        
+        virtual JSValue *callAsFunction(ExecState *, JSObject *thisObj, const List &args);
+        
+        enum { ToString, ToDateString, ToTimeString, ToLocaleString,
+            ToLocaleDateString, ToLocaleTimeString, ValueOf, GetTime,
+            GetFullYear, GetMonth, GetDate, GetDay, GetHours, GetMinutes,
+            GetSeconds, GetMilliSeconds, GetTimezoneOffset, SetTime,
+            SetMilliSeconds, SetSeconds, SetMinutes, SetHours, SetDate,
+            SetMonth, SetFullYear, ToUTCString,
+            // non-normative properties (Appendix B)
+            GetYear, SetYear, ToGMTString };
+    private:
+        int id;
+        bool utc;
+    };
+    
+    /**
      * @internal
      *
      * The initial value of the the global variable's "Date" property
index 954e85e..62b9cf6 100644 (file)
@@ -28,7 +28,6 @@
 
 #include "interpreter.h"
 #include "nodes.h"
-#include "lookup.h"
 #include <wtf/unicode/Unicode.h>
 
 static bool isDecimalDigit(unsigned short c);
@@ -42,6 +41,7 @@ static Lexer *currLexer = 0;
 #include "grammar.h"
 #endif
 
+#include "lookup.h"
 #include "lexer.lut.h"
 
 extern YYLTYPE kjsyylloc; // global bison variable holding token info
index 26712f1..9bd0ab0 100644 (file)
 
 #include "config.h"
 #include "math_object.h"
+#include "math_object.lut.h"
 
-#include "lookup.h"
 #include "operations.h"
 #include <math.h>
 #include <time.h>
 
-#include "math_object.lut.h"
-
 #if PLATFORM(WIN_OS)
 
 #include <float.h>
index e4090b9..c544925 100644 (file)
 
 #include "config.h"
 #include "number_object.h"
+#include "number_object.lut.h"
 
 #include "dtoa.h"
 #include "error_object.h"
-#include "lookup.h"
 #include "operations.h"
 #include <wtf/Vector.h>
 #include <math.h>
 
-#include "number_object.lut.h"
-
 using namespace KJS;
 
 // ------------------------------ NumberInstance ----------------------------
index 87aee53..5a5ace6 100644 (file)
@@ -22,8 +22,8 @@
 
 #include "config.h"
 #include "regexp_object.h"
+#include "regexp_object.lut.h"
 
-#include <stdio.h>
 #include "value.h"
 #include "object.h"
 #include "types.h"
@@ -32,9 +32,8 @@
 #include "internal.h"
 #include "regexp.h"
 #include "error_object.h"
-#include "lookup.h"
 
-#include "regexp_object.lut.h"
+#include <stdio.h>
 
 using namespace KJS;
 
index 6ee78dc..fa97f76 100644 (file)
 
 #include "config.h"
 #include "string_object.h"
+#include "string_object.lut.h"
 
 #include "error_object.h"
-#include "lookup.h"
 #include "operations.h"
 #include "reference_list.h"
 #include "regexp_object.h"
 #include <wtf/unicode/Unicode.h>
 
-#include "string_object.lut.h"
-
 using namespace KJS;
 
 // ------------------------------ StringInstance ----------------------------