Summary: | [Qt][Mips] r150484 broke the build | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Zoltan Arvai <zarvai> | ||||
Component: | New Bugs | Assignee: | Allan Sandfeld Jensen <allan.jensen> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | allan.jensen, andersca, gergely, jturcotte, kadam, kilvadyb, ossy | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 108664, 116578 | ||||||
Attachments: |
|
Description
Zoltan Arvai
2013-05-22 06:45:18 PDT
It seems WTF_COMPILER_SUPPORTS_CXX_NULLPTR isn't defined in Source/WTF/wtf/Compiler.h , because you use too old compiler. But now c++11 is mandatory for WebKit2, so you probably need newer compiler. (In reply to comment #1) > It seems WTF_COMPILER_SUPPORTS_CXX_NULLPTR isn't defined in > Source/WTF/wtf/Compiler.h , because you use too old compiler. > > But now c++11 is mandatory for WebKit2, so you probably need newer compiler. Tamk you for the info. We are working on it. (In reply to comment #2) > (In reply to comment #1) > > It seems WTF_COMPILER_SUPPORTS_CXX_NULLPTR isn't defined in > > Source/WTF/wtf/Compiler.h , because you use too old compiler. > > > > But now c++11 is mandatory for WebKit2, so you probably need newer compiler. > > Tamk you for the info. We are working on it. Sorry, I wanted to write: Thank you The compiler support most of C++11, but is 'nullptr' really necessary? We can replace it with a 0, and it would work. (In reply to comment #4) > The compiler support most of C++11, but is 'nullptr' really necessary? We can replace it with a 0, and it would work. C++11 in WebKit2 is mandatory now... ... but feel free to find a WebKit2 owner for this fix. :) Created attachment 202969 [details]
Patch
(In reply to comment #5) > (In reply to comment #4) > > The compiler support most of C++11, but is 'nullptr' really necessary? We can replace it with a 0, and it would work. > > C++11 in WebKit2 is mandatory now... > > ... but feel free to find a WebKit2 owner for this fix. :) All of C++11 is hardly mandatory, or we would depend on unreleased compilers. There is has to be an expected subset of featuers. (In reply to comment #7) > (In reply to comment #5) > > (In reply to comment #4) > > > The compiler support most of C++11, but is 'nullptr' really necessary? We can replace it with a 0, and it would work. > > > > C++11 in WebKit2 is mandatory now... > > > > ... but feel free to find a WebKit2 owner for this fix. :) > > All of C++11 is hardly mandatory, or we would depend on unreleased compilers. There is has to be an expected subset of featuers. I’ve generally only used features that’s available on compilers that have been out for at least two years. That includes MSVC 2010 as well as GCC 4.6. The bot is green again :) Comment on attachment 202969 [details]
Patch
Clearing review flag.
|