Chromium Code Reviews| Index: src/conversions.cc |
| diff --git a/src/conversions.cc b/src/conversions.cc |
| index 19fa7773abf6edfedf9c54e9dfdea4674d3919bd..df92706ba5e4a3cdbe3f1ed560040c09e836695a 100644 |
| --- a/src/conversions.cc |
| +++ b/src/conversions.cc |
| @@ -450,10 +450,10 @@ static double InternalStringToDouble(Iterator current, |
| if (*current == '+') { |
| // Ignore leading sign; skip following spaces. |
| ++current; |
| - if (!AdvanceToNonspace(¤t, end)) return JUNK_STRING_VALUE; |
| + if (current == end) return JUNK_STRING_VALUE; |
|
Karl Klose
2010/11/26 12:40:12
Should the comment not be updated to something lik
Lasse Reichstein
2010/11/26 12:46:01
Good catch.
I'll just remove "; skip following spa
|
| } else if (*current == '-') { |
| ++current; |
| - if (!AdvanceToNonspace(¤t, end)) return JUNK_STRING_VALUE; |
| + if (current == end) return JUNK_STRING_VALUE; |
| sign = true; |
| } |