Remove the locale parameter from the StringToDouble and

Remove the locale parameter from the StringToDouble and
DoubleToString methods and only leave the
locale independent methods.  It turns out this code is
only used in the json code and if people want locale
formatting, they should use ICU.

[Evan Martin, 2008-11-17 22:37:51]

http://codereview.chromium.org/11423/diff/15/213
File base/string_util.cc (right):

http://codereview.chromium.org/11423/diff/15/213#newcode1002
Line 1002: char buffer[32];
Is this guaranteed?  It makes me nervous.

http://codereview.chromium.org/11423/diff/15/214
File base/string_util.h (right):

http://codereview.chromium.org/11423/diff/15/214#newcode384
Line 384: // locale (ie, uses the C locale).  If your input is locale specific,
use ICU
I kinda suspect people who don't know Unix won't know what "C locale" means. 
(?)

[tony, 2008-11-17 23:27:52]

On 2008/11/17 22:37:51, Evan Martin wrote:
> http://codereview.chromium.org/11423/diff/15/213
> File base/string_util.cc (right):
> 
> http://codereview.chromium.org/11423/diff/15/213#newcode1002
> Line 1002: char buffer[32];
> Is this guaranteed?  It makes me nervous.

Yes, the g_fmt.cc promises that 32 bytes is enough.  I've updated the comments
to say this.

> http://codereview.chromium.org/11423/diff/15/214
> File base/string_util.h (right):
> 
> http://codereview.chromium.org/11423/diff/15/214#newcode384
> Line 384: // locale (ie, uses the C locale).  If your input is locale
specific,
> use ICU
> I kinda suspect people who don't know Unix won't know what "C locale" means. 
> (?)

Ok, removed the text.

[Evan Martin, 2008-11-17 23:31:46]

LGTM