Use source locations in the diagnostic messages.

1 files changed, 10 insertions, 18 deletions

diff --git a/src/declarative/qml/parser/javascriptparser.cpp b/src/declarative/qml/parser/javascriptparser.cpp
index e1bed5d..0857eef 100644
--- a/src/declarative/qml/parser/javascriptparser.cpp
+++ b/src/declarative/qml/parser/javascriptparser.cpp
@@ -415,9 +415,8 @@ case 51: {
 case 52: {
   bool rx = lexer->scanRegExp(Lexer::NoPrefix);
   if (!rx) {
-    diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Error, lexer->startLineNo(),
-        lexer->startColumnNo(), lexer->errorMessage()));
-      return false;
+    diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Error, location(lexer), lexer->errorMessage()));
+    return false; // ### remove me
   }
   AST::RegExpLiteral *node = makeAstNode<AST::RegExpLiteral> (driver->nodePool(), lexer->pattern, lexer->flags);
   node->literalToken = loc(1);
@@ -427,9 +426,8 @@ case 52: {
 case 53: {
   bool rx = lexer->scanRegExp(Lexer::EqualPrefix);
   if (!rx) {
-    diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Error, lexer->startLineNo(),
-        lexer->startColumnNo(), lexer->errorMessage()));
-      return false;
+    diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Error, location(lexer), lexer->errorMessage()));
+    return false;
   }
   AST::RegExpLiteral *node = makeAstNode<AST::RegExpLiteral> (driver->nodePool(), lexer->pattern, lexer->flags);
   node->literalToken = loc(1);
@@ -1556,10 +1554,8 @@ case 314: {
             yylloc.startColumn += yylloc.length;
             yylloc.length = 0;
 
-            const QString msg = QString::fromUtf8("Missing `;'");
-
-            diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Warning,
-                yylloc.startLine, yylloc.startColumn, msg));
+            //const QString msg = QString::fromUtf8("Missing `;'");
+            //diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Warning, yylloc, msg));
 
             first_token = &token_buffer[0];
             last_token = &token_buffer[1];
@@ -1585,8 +1581,7 @@ case 314: {
         if (t_action(errorState, yytoken)) {
             const QString msg = QString::fromUtf8("Unexpected token `%1'").arg(QLatin1String(spell[token_buffer[0].token]));
 
-            diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Error,
-                token_buffer[0].loc.startLine, token_buffer[0].loc.startColumn, msg));
+            diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Error, token_buffer[0].loc, msg));
 
             action = errorState;
             goto _Lcheck_token;
@@ -1615,8 +1610,7 @@ case 314: {
             if (a > 0 && t_action(a, yytoken)) {
                 const QString msg = QString::fromUtf8("Expected token `%1'").arg(QLatin1String(spell[*tk]));
 
-                diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Error,
-                    token_buffer[0].loc.startLine, token_buffer[0].loc.startColumn, msg));
+                diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Error, token_buffer[0].loc, msg));
 
                 yytoken = *tk;
                 yylval = 0;
@@ -1638,8 +1632,7 @@ case 314: {
             int a = t_action(errorState, tk);
             if (a > 0 && t_action(a, yytoken)) {
                 const QString msg = QString::fromUtf8("Expected token `%1'").arg(QLatin1String(spell[tk]));
-                diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Error,
-                    token_buffer[0].loc.startLine, token_buffer[0].loc.startColumn, msg));
+                diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Error, token_buffer[0].loc, msg));
 
                 yytoken = tk;
                 yylval = 0;
@@ -1652,8 +1645,7 @@ case 314: {
         }
 
         const QString msg = QString::fromUtf8("Syntax error");
-        diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Error,
-            token_buffer[0].loc.startLine, token_buffer[0].loc.startColumn, msg));
+        diagnostic_messages.append(DiagnosticMessage(DiagnosticMessage::Error, token_buffer[0].loc, msg));
     }
 
     return false;