WebKit nightly fails to build on macOS Sierra
Created attachment 283951 [details] Patch
Attachment 283951 [details] did not pass style-queue: ERROR: Source/WebCore/Modules/applepay/cocoa/PaymentMerchantSessionCocoa.mm:32: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:7: #ifndef header guard has wrong style, please use: ucurr_h [build/header_guard] [5] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:10: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:54: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:56: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:99: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:100: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:113: The parameter name "key" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:135: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:136: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:137: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:138: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:136: The parameter name "nameStyle" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:139: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:139: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:159: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:163: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:176: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:176: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:189: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:189: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:274: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:275: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:276: The parameter name "errorCode" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:276: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:294: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:294: The parameter name "date" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:295: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:295: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:318: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:318: The parameter name "date" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:322: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/mac/icu/unicode/ucurr.h:342: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/WebCore/platform/spi/cocoa/PassKitSPI.h:43: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Source/WebCore/platform/spi/cocoa/PassKitSPI.h:49: Missing spaces around | [whitespace/operators] [3] ERROR: Source/WebCore/platform/spi/cocoa/PassKitSPI.h:52: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Source/WTF/icu/unicode/ucurr.h:7: #ifndef header guard has wrong style, please use: ucurr_h [build/header_guard] [5] ERROR: Source/WTF/icu/unicode/ucurr.h:10: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WTF/icu/unicode/ucurr.h:54: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:56: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WTF/icu/unicode/ucurr.h:99: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:100: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:113: The parameter name "key" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WTF/icu/unicode/ucurr.h:135: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:136: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:137: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:138: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:136: The parameter name "nameStyle" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WTF/icu/unicode/ucurr.h:139: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WTF/icu/unicode/ucurr.h:139: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:159: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:163: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WTF/icu/unicode/ucurr.h:176: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WTF/icu/unicode/ucurr.h:176: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:189: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WTF/icu/unicode/ucurr.h:189: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:274: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:275: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:276: The parameter name "errorCode" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WTF/icu/unicode/ucurr.h:276: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:294: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:294: The parameter name "date" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WTF/icu/unicode/ucurr.h:295: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WTF/icu/unicode/ucurr.h:295: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:318: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/WTF/icu/unicode/ucurr.h:318: The parameter name "date" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WTF/icu/unicode/ucurr.h:322: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WTF/icu/unicode/ucurr.h:342: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:7: #ifndef header guard has wrong style, please use: ucurr_h [build/header_guard] [5] ERROR: Source/WebCore/icu/unicode/ucurr.h:10: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebCore/icu/unicode/ucurr.h:54: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:56: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/icu/unicode/ucurr.h:99: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:100: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:113: The parameter name "key" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/icu/unicode/ucurr.h:135: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:136: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:137: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:138: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:136: The parameter name "nameStyle" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/icu/unicode/ucurr.h:139: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/icu/unicode/ucurr.h:139: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:159: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:163: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/icu/unicode/ucurr.h:176: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/icu/unicode/ucurr.h:176: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:189: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/icu/unicode/ucurr.h:189: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:274: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:275: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:276: The parameter name "errorCode" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/icu/unicode/ucurr.h:276: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:294: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:294: The parameter name "date" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/icu/unicode/ucurr.h:295: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/icu/unicode/ucurr.h:295: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:318: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/WebCore/icu/unicode/ucurr.h:318: The parameter name "date" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/icu/unicode/ucurr.h:322: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebCore/icu/unicode/ucurr.h:342: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:7: #ifndef header guard has wrong style, please use: ucurr_h [build/header_guard] [5] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:10: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:54: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:56: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:99: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:100: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:113: The parameter name "key" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:135: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:136: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:137: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:138: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:136: The parameter name "nameStyle" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:139: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:139: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:159: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/JavaScrip tCore/icu/unicode/ucurr.h:163: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:176: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:176: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:189: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:189: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:274: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:275: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:276: The parameter name "errorCode" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:276: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:294: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:294: The parameter name "date" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:295: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:295: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:318: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:318: The parameter name "date" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:322: The parameter name "ec" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/JavaScriptCore/icu/unicode/ucurr.h:342: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3] Total errors found: 132 in 19 files If any of these errors are false positives, please file a bug against check-webkit-style.
Committed r203381: <http://trac.webkit.org/changeset/203381>