diff options
| author | Shyam Seshadri | 2010-10-08 17:11:45 -0700 |
|---|---|---|
| committer | Misko Hevery | 2010-10-08 20:41:19 -0700 |
| commit | 8490bb921bd03239f5856cd44b7e21ef6db23eca (patch) | |
| tree | 2bc58b6247ab1f68796ccb0570365e3ae8c48e72 /src | |
| parent | e3ea980c819f62b8c2021f813e3534e4e5731d7d (diff) | |
| download | angular.js-8490bb921bd03239f5856cd44b7e21ef6db23eca.tar.bz2 | |
Fix bug with Lexer not recognizing exponential values and values starting with dots
Diffstat (limited to 'src')
| -rw-r--r-- | src/Parser.js | 21 |
1 files changed, 20 insertions, 1 deletions
diff --git a/src/Parser.js b/src/Parser.js index 74815cd1..2c681be0 100644 --- a/src/Parser.js +++ b/src/Parser.js @@ -53,6 +53,9 @@ function lex(text, parseStrings){ tokens.push({index:index, text:ch}); index++; canStartRegExp = false; + } else if (ch == '.' && isNumber(peek())) { + readNumber(); + canStartRegExp = false; } else if ( ch == ':' || ch == '.' || ch == ',' || ch == ';') { tokens.push({index:index, text:ch}); index++; @@ -104,6 +107,9 @@ function lex(text, parseStrings){ 'A' <= ch && ch <= 'Z' || '_' == ch || ch == '$'; } + function isExpOperator(ch) { + return ch == '-' || ch == '+'; + } function readNumber() { var number = ""; var start = index; @@ -112,7 +118,20 @@ function lex(text, parseStrings){ if (ch == '.' || isNumber(ch)) { number += ch; } else { - break; + var peekCh = peek(); + if (ch == 'E' && isExpOperator(peekCh)) { + number += ch; + } else if (isExpOperator(ch) && + peekCh && isNumber(peekCh) && + number.charAt(number.length - 1) == 'E') { + number += ch; + } else if (isExpOperator(ch) && + (!peekCh || !isNumber(peekCh)) && + number.charAt(number.length - 1) == 'E') { + throw 'Lexer found invalid exponential value "' + text + '"'; + } else { + break; + } } index++; } |
