RESOLVED FIXED80971
Remove obsolete constructors of CSSPrimitiveValue class. 
https://bugs.webkit.org/show_bug.cgi?id=80971
Summary Remove obsolete constructors of CSSPrimitiveValue class.
Kwonjin Jeong
Reported 2012-03-13 00:32:30 PDT
In CSSPrimitiveValueMappings.h file, there are many specialized template functions for CSSPrimitiveValue class. but the following constructors can't be reached. * template<> inline CSSPrimitiveValue::CSSPrimitiveValue(int i) * template<> inline CSSPrimitiveValue::CSSPrimitiveValue(unsigned i) For the each constructors, there exists overloaded constructor that has the same parameter types as corresponding constructor. Therefore, the overloaded constructor is always called instead of template specialized one. Moreover, they cause compile errors on the Intel compiler(icpc version 12.1.0) because icpc doesn't allow template function specialization with same parameter types as overloaded function.
Attachments
Patch (1.11 KB, patch)
2012-03-13 00:39 PDT, Kwonjin Jeong 		no flags
DetailsFormatted DiffDiff
Patch (1.79 KB, patch)
2012-03-13 00:47 PDT, Kwonjin Jeong 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Kwonjin Jeong
Comment 1 2012-03-13 00:39:16 PDT
Created attachment 131566 [details] Patch
Kwonjin Jeong
Comment 2 2012-03-13 00:47:31 PDT
Created attachment 131567 [details] Patch
Kwonjin Jeong
Comment 3 2012-03-13 22:06:45 PDT
I create a new bug for Intel compiler support on Linux QT port. https://bugs.webkit.org/show_bug.cgi?id=81077 You can build using Intel compiler referring to the bug.
Kwonjin Jeong
Comment 4 2012-03-14 18:06:45 PDT
Would you give me some review about this patch?
WebKit Review Bot
Comment 5 2012-03-14 19:09:41 PDT
Comment on attachment 131567 [details] Patch Clearing flags on attachment: 131567 Committed r110807: <http://trac.webkit.org/changeset/110807>
WebKit Review Bot
Comment 6 2012-03-14 19:09:46 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.