Extract string->double and double->string conversions for use in the preparser.

Extract string->double and double->string conversions for use in the preparser.

Committed: http://code.google.com/p/v8/source/detail?r=8534

[Rico, 2011-07-05 08:54:57]

LGTM
As discussed offline, this is simple moved code except the AddDecimalInteger in
utils.cc.

http://codereview.chromium.org/7308004/diff/1/src/cached-powers.cc
File src/cached-powers.cc (right):

http://codereview.chromium.org/7308004/diff/1/src/cached-powers.cc#newcode30
src/cached-powers.cc:30: #include <limits>
limits.h?

http://codereview.chromium.org/7308004/diff/1/src/conversions-inl.h
File src/conversions-inl.h (right):

http://codereview.chromium.org/7308004/diff/1/src/conversions-inl.h#newcode108
src/conversions-inl.h:108: // Returns true if a nonspace found and false if the
end has reached.
nonspace found -> nonspace is found
end has reached -> end is reached

http://codereview.chromium.org/7308004/diff/1/src/strtod.cc
File src/strtod.cc (right):

http://codereview.chromium.org/7308004/diff/1/src/strtod.cc#newcode29
src/strtod.cc:29: #include <limits>
limits.h?

http://codereview.chromium.org/7308004/diff/1/src/utils.cc
File src/utils.cc (right):

http://codereview.chromium.org/7308004/diff/1/src/utils.cc#newcode63
src/utils.cc:63: void SimpleStringBuilder::AddDecimalInteger(int value) {
take an unsigned value as input paramenter

http://codereview.chromium.org/7308004/diff/1/src/v8conversions.cc
File src/v8conversions.cc (right):

http://codereview.chromium.org/7308004/diff/1/src/v8conversions.cc#newcode81
src/v8conversions.cc:81: }
Add comment that we end namespace here

[Lasse Reichstein, 2011-07-05 10:43:18]

http://codereview.chromium.org/7308004/diff/1/src/conversions-inl.h
File src/conversions-inl.h (right):

http://codereview.chromium.org/7308004/diff/1/src/conversions-inl.h#newcode108
src/conversions-inl.h:108: // Returns true if a nonspace found and false if the
end has reached.
rewritten.

http://codereview.chromium.org/7308004/diff/1/src/utils.cc
File src/utils.cc (right):

http://codereview.chromium.org/7308004/diff/1/src/utils.cc#newcode63
src/utils.cc:63: void SimpleStringBuilder::AddDecimalInteger(int value) {
I'd prefer to take it signed, but I'll make it int32_t to make the size
explicit.

http://codereview.chromium.org/7308004/diff/1/src/v8conversions.cc
File src/v8conversions.cc (right):

http://codereview.chromium.org/7308004/diff/1/src/v8conversions.cc#newcode81
src/v8conversions.cc:81: }
Done.