[Webkit-unassigned] [Bug 167343] webkit-gtk-2.15.3 fails to build on macOS due to missing declaration of U8_MAX_LENGTH
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Apr 20 12:26:56 PDT 2017
https://bugs.webkit.org/show_bug.cgi?id=167343
--- Comment #11 from Build Bot <buildbot at hotmail.com> ---
Attachment 307612 did not pass style-queue:
ERROR: Source/WTF/wtf/unicode/WTFUTF8.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.h:38: Code inside a namespace should not be indented. [whitespace/indent] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.h:46: One space before end of line comments [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.h:46: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.h:47: One space before end of line comments [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.h:48: One space before end of line comments [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.h:49: One space before end of line comments [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.h:54: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.h:67: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.h:71: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.h:75: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
ERROR: Source/WebCore/ChangeLog:10: 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/WTF/wtf/unicode/WTFUTF8.cpp:65: 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/WTF/wtf/unicode/WTFUTF8.cpp:122: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:221: One line control clauses should not use braces. [whitespace/braces] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:234: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:234: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:235: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:236: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:247: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:254: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:256: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:256: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:257: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:257: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:258: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:262: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:262: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:263: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:264: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:265: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:266: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:270: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:288: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:288: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:289: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:290: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:291: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:292: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:330: An else statement can be removed when the prior "if" concludes with a return, break, continue or goto statement. [readability/control_flow] [4]
ERROR: Source/WTF/wtf/unicode/WTFUTF8.cpp:414: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Total errors found: 41 in 20 files
If any of these errors are false positives, please file a bug against check-webkit-style.
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170420/5117d12f/attachment.html>
More information about the webkit-unassigned
mailing list