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

Issue 6843083: Add explicit std:: namespace to code from <cXYZ> includes. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years ago by milian.wolff
Modified:
11 years, 12 months ago
CC:
angleproject-review_googlegroups.com
Base URL:
http://angleproject.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Add explicit std:: namespace to code from <cXYZ> includes. Some platforms seem to implicitly include the <XYZ.h> headers which also add some types and functions (like strlen, size_t,...) into the global namespace. On other platforms though, this can result in compile errors, which is noticeable in WebKit on e.g. QNX. See also: https://bugs.webkit.org/show_bug.cgi?id=95468

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -10 lines) Patch
M AUTHORS View 1 chunk +1 line, -0 lines 0 comments Download
M src/compiler/preprocessor/Input.cpp View 2 chunks +2 lines, -2 lines 0 comments Download
M src/compiler/preprocessor/MacroExpander.h View 1 chunk +1 line, -1 line 0 comments Download
M src/compiler/preprocessor/MacroExpander.cpp View 5 chunks +6 lines, -6 lines 0 comments Download
M src/compiler/preprocessor/Tokenizer.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 8
milian.wolff
Please review this in order to get https://bugs.webkit.org/show_bug.cgi?id=95468 sorted out.
12 years ago (2012-11-19 16:50:48 UTC) #1
apatrick1
Explicitly qualifying these names is a bit verbose. Someone suggested including the C-style headers, like ...
12 years ago (2012-11-19 18:19:19 UTC) #2
dgkoch
Please note that you must also sign the CLA and email me the required information ...
12 years ago (2012-11-19 18:30:03 UTC) #3
milian.wolff
Hey there, I can also try to adapt the patch to use the <foo.h> style ...
12 years ago (2012-11-19 18:44:30 UTC) #4
Alok Priyadarshi
I prefer keeping cFoo headers and do not mind std:: qualification. But if the consensus ...
12 years ago (2012-11-19 18:56:37 UTC) #5
dgkoch
On 2012/11/19 18:44:30, milian.wolff wrote: > Hey there, > > I can also try to ...
12 years ago (2012-11-19 19:04:03 UTC) #6
milian.wolff
On 2012/11/19 19:04:03, dgkoch wrote: > On 2012/11/19 18:44:30, milian.wolff wrote: > > Hey there, ...
12 years ago (2012-11-26 11:49:45 UTC) #7
dgkoch
11 years, 12 months ago (2012-11-29 20:55:33 UTC) #8
> 
> I've send you the CLA information, is anything else required to get this
fixed?
> 
Committed in r1556.  Please close the issue.  Thanks!
Sign in to reply to this message.

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