Add using namespace std statement.
Committed r199868: <http://trac.webkit.org/changeset/199868>
(In reply to comment #1) > Committed r199868: <http://trac.webkit.org/changeset/199868> And Darin landed one more fix: (I don't konw which build it fixed.) http://trac.webkit.org/changeset/199870 But now the Apple Mac cloop build is broken: /Volumes/Data/slave/elcapitan-cloop-debug/build/Source/JavaScriptCore/runtime/MathCommon.h:61:28: error: expected unqualified-id if (!constant || !std::isnormal(constant)) ^ /Volumes/Data/slave/elcapitan-cloop-debug/build/Source/JavaScriptCore/runtime/MathCommon.h:65:9: error: no member named 'frexp' in namespace 'std'; did you mean simply 'frexp'? if (std::frexp(constant, &exponent) != 0.5) ^~~~~~~~~~ frexp /Volumes/Data/slave/elcapitan-cloop-debug/build/Source/JavaScriptCore/runtime/MathCommon.h:77:25: error: no member named 'ldexp' in namespace 'std'; did you mean simply 'ldexp'? double reciprocal = std::ldexp(1, -exponent); ^~~~~~~~~~ ldexp /Volumes/Data/slave/elcapitan-cloop-debug/build/Source/JavaScriptCore/runtime/MathCommon.h:78:17: error: expected unqualified-id ASSERT(std::isnormal(reciprocal)); ^ Common, nobody watches bots after landing random patches? Why ???? The original patch broke all Linux builds, the buildfixes broke the Apple Mac cloop build.
(In reply to comment #2) > (In reply to comment #1) > > Committed r199868: <http://trac.webkit.org/changeset/199868> > > And Darin landed one more fix: (I don't konw which build it fixed.) > http://trac.webkit.org/changeset/199870 > > But now the Apple Mac cloop build is broken: > > /Volumes/Data/slave/elcapitan-cloop-debug/build/Source/JavaScriptCore/ > runtime/MathCommon.h:61:28: error: expected unqualified-id > if (!constant || !std::isnormal(constant)) > ^ > /Volumes/Data/slave/elcapitan-cloop-debug/build/Source/JavaScriptCore/ > runtime/MathCommon.h:65:9: error: no member named 'frexp' in namespace > 'std'; did you mean simply 'frexp'? > if (std::frexp(constant, &exponent) != 0.5) > ^~~~~~~~~~ > frexp > > /Volumes/Data/slave/elcapitan-cloop-debug/build/Source/JavaScriptCore/ > runtime/MathCommon.h:77:25: error: no member named 'ldexp' in namespace > 'std'; did you mean simply 'ldexp'? > double reciprocal = std::ldexp(1, -exponent); > ^~~~~~~~~~ > ldexp > > /Volumes/Data/slave/elcapitan-cloop-debug/build/Source/JavaScriptCore/ > runtime/MathCommon.h:78:17: error: expected unqualified-id > ASSERT(std::isnormal(reciprocal)); > ^ > > Common, nobody watches bots after landing random patches? Why ???? > The original patch broke all Linux builds, the buildfixes broke the > Apple Mac cloop build. I'm sorry, Ossy. If it is not fixed soon, let me check it out again. My mac is at home, so I couldn't check mac cloop port in advance.
Reopening to attach new patch.
Created attachment 277039 [details] To check EWS build status
(In reply to comment #5) > Created attachment 277039 [details] > To check EWS build status FYI: there is no EWS for cloop build.
Created attachment 277040 [details] Fix a typo ignoring style check warning. cloop build was successful
Attachment 277040 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/runtime/MathCommon.h:36: Do not use 'using namespace std;'. [build/using_namespace] [4] Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 277041 [details] Change commit log to let this patch be reviewed.
Attachment 277041 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/runtime/MathCommon.h:36: Do not use 'using namespace std;'. [build/using_namespace] [4] Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 277041 [details] Change commit log to let this patch be reviewed. r=me
Comment on attachment 277041 [details] Change commit log to let this patch be reviewed. View in context: https://bugs.webkit.org/attachment.cgi?id=277041&action=review > Source/JavaScriptCore/runtime/MathCommon.h:36 > +using namespace std; I think the correct fix is to add an include so we get the "std::" versions of these rather than doing this.
Comment on attachment 277041 [details] Change commit log to let this patch be reviewed. r- per Darin's comment.
Quick fix: http://trac.webkit.org/changeset/199913 Let's see if the bots like that better.
Created attachment 277105 [details] Follow up to Bejamin's quick fix
Comment on attachment 277105 [details] Follow up to Bejamin's quick fix Clearing flags on attachment: 277105 Committed r199943: <http://trac.webkit.org/changeset/199943>
All reviewed patches have been landed. Closing bug.