[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
Tue Apr 18 18:04:51 PDT 2017
https://bugs.webkit.org/show_bug.cgi?id=167343
Build Bot <buildbot at hotmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |buildbot at hotmail.com
--- Comment #3 from Build Bot <buildbot at hotmail.com> ---
Attachment 307444 did not pass style-queue:
ERROR: Source/WTF/wtf/unicode/wtf_utf8.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/wtf_utf8.cpp:122: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:221: One line control clauses should not use braces. [whitespace/braces] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:234: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:234: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:235: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:236: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:247: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:254: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:256: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:256: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:257: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:257: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:258: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:262: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:262: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:263: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:264: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:265: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:266: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:270: More than one command on the same line in if [whitespace/parens] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:288: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:288: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:289: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:290: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:291: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.cpp:292: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.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/wtf_utf8.cpp:414: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.h:38: Code inside a namespace should not be indented. [whitespace/indent] [4]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.h:46: One space before end of line comments [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.h:46: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.h:47: One space before end of line comments [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.h:48: One space before end of line comments [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.h:49: One space before end of line comments [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.h:54: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.h:67: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.h:71: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
ERROR: Source/WTF/wtf/unicode/wtf_utf8.h:75: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Total errors found: 40 in 24 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/20170419/2072e9c4/attachment.html>
More information about the webkit-unassigned
mailing list