Changeset 35245 in webkit for trunk/JavaScriptCore/kjs/lexer.cpp

Timestamp:

Jul 18, 2008, 6:44:24 PM (17 years ago)

Author:

[email protected]

Message:

Bug 18774: SQUIRRELFISH: print meaningful error messages <​https://bugs.webkit.org/show_bug.cgi?id=18774>
<rdar://problem/5769353> SQUIRRELFISH: JavaScript error messages are missing informative text

Reviewed by Cameron Zwarich

Add support for decent error messages in JavaScript. This patch achieves this by providing
ensuring the common errors and exceptions have messages that provide the text of expression
that trigger the exception. In addition it attaches a number of properties to the exception
object detailing where in the source the expression came from.

File:

• trunk/JavaScriptCore/kjs/lexer.cpp (modified) (5 diffs)

trunk/JavaScriptCore/kjs/lexer.cpp

@@ -75 +75 @@
     , m_next2(0)
     , m_next3(0)
+    , m_currentOffset(0)
+    , m_nextOffset1(0)
+    , m_nextOffset2(0)
+    , m_nextOffset3(0)
     , m_globalData(globalData)
     , m_mainTable(KJS::mainTable)
@@ -120 +124 @@
         m_next1 = m_next2;
         m_next2 = m_next3;
+        m_currentOffset = m_nextOffset1;
+        m_nextOffset1 = m_nextOffset2;
+        m_nextOffset2 = m_nextOffset3;
         do {
             if (m_position >= m_length) {
+                m_nextOffset3 = m_position;
                 m_position++;
                 m_next3 = -1;
                 break;
             }
+            m_nextOffset3 = m_position;
             m_next3 = m_code[m_position++];
         } while (m_next3 == 0xFEFF);
@@ -165 +174 @@
         m_stackToken = 0;
     }
-
+    int startOffset = m_currentOffset;
     while (!m_done) {
         if (m_skipLF && m_current != '\n') // found \r but not \n afterwards
@@ -178 +187 @@
         switch (m_state) {
             case Start:
+                startOffset = m_currentOffset;
                 if (isWhiteSpace()) {
                     // do nothing
@@ -522 +532 @@
     llocp->first_line = yylineno;
     llocp->last_line = yylineno;
-
+    llocp->first_column = startOffset;
+    llocp->last_column = m_currentOffset;
     switch (m_state) {
         case Eof: