Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(186)

Issue 6392046: Builds for chrome memroy bots (windows only) do not link due to the usage of std::locale, the reaso… (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 9 months ago by Alok Priyadarshi
Modified:
13 years, 9 months ago
Reviewers:
kbr1, dgkoch
CC:
angleproject-review_googlegroups.com
Base URL:
http://angleproject.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Builds for chrome memroy bots (windows only) do not link due to the usage of std::locale, the reason for which is still unknown. This patch avoids the usage of std::locale, while still enforcing "C" locale and checking for overflow. Committed: https://code.google.com/p/angleproject/source/detail?r=1203

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -4 lines) Patch
M src/compiler/preprocessor/new/Token.cpp View 2 chunks +15 lines, -4 lines 0 comments Download

Messages

Total messages: 6
Alok Priyadarshi
Please review ASAP. Chrome memory bots are hosed.
13 years, 9 months ago (2012-07-10 17:27:33 UTC) #1
kbr1
LGTM
13 years, 9 months ago (2012-07-10 17:42:12 UTC) #2
dgkoch
Hrm.. any good reason not to use atof_dot (from compiler/utils.h). That's what we use in ...
13 years, 9 months ago (2012-07-10 18:25:51 UTC) #3
Alok Priyadarshi
On 2012/07/10 18:25:51, dgkoch wrote: > Hrm.. any good reason not to use atof_dot (from ...
13 years, 9 months ago (2012-07-10 18:29:48 UTC) #4
dgkoch
On 2012/07/10 18:29:48, Alok Priyadarshi wrote: > On 2012/07/10 18:25:51, dgkoch wrote: > > Hrm.. ...
13 years, 9 months ago (2012-07-10 18:37:33 UTC) #5
Alok Priyadarshi
13 years, 9 months ago (2012-07-10 18:45:07 UTC) #6
On 2012/07/10 18:37:33, dgkoch wrote:
> On 2012/07/10 18:29:48, Alok Priyadarshi wrote:
> > On 2012/07/10 18:25:51, dgkoch wrote:
> > > Hrm.. any good reason not to use atof_dot (from compiler/utils.h). That's
> what
> > > we use in the old preprocessor.  I can't recall if we'd looked into strtod
> > > earlier and ruled that out or not...
> > 
> > The one in util.cpp does not check for overflow.
> 
> Ok. How is that enforcing the C locale though?  The man page for strtod on the
> mac says: "The decimal point character is defined in the program's locale
> (category LC_NUMERIC)."

You are right. I will take another stab at it.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b