NEW185861
Use C++17 message-less static_assert 
https://bugs.webkit.org/show_bug.cgi?id=185861
Summary Use C++17 message-less static_assert
Yusuke Suzuki
Reported 2018-05-22 04:05:59 PDT
Use C++17 message-less static_assert
Attachments
Patch (36.63 KB, patch)
2018-05-22 04:06 PDT, Yusuke Suzuki 		andersca: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2018-05-22 04:06:31 PDT
Created attachment 340969 [details] Patch
EWS Watchlist
Comment 2 2018-05-22 04:09:02 PDT
Attachment 340969 [details] did not pass style-queue: ERROR: Source/WebCore/ChangeLog:8: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5] ERROR: Source/JavaScriptCore/bytecode/ParseHash.cpp:45: Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons. [readability/comparison_to_zero] [5] ERROR: Source/JavaScriptCore/bytecode/CodeBlockHash.cpp:51: Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons. [readability/comparison_to_zero] [5] Total errors found: 3 in 33 files If any of these errors are false positives, please file a bug against check-webkit-style.
Note You need to log in before you can comment on or make changes to this bug.