Summary: | check-webkit-style reports false-positive build/include_order warning in WTF C++ source files | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | David Kilzer (:ddkilzer) <ddkilzer> | ||||
Component: | Tools / Tests | Assignee: | David Kilzer (:ddkilzer) <ddkilzer> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | achristensen, bburg, cgarcia, commit-queue, ews-watchlist, glenn, mcatanzaro, thorton, timothy, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
See Also: |
https://bugs.webkit.org/show_bug.cgi?id=198346 https://bugs.webkit.org/show_bug.cgi?id=198351 |
||||||
Attachments: |
|
Description
David Kilzer (:ddkilzer)
2019-05-29 13:23:21 PDT
This fix cuts down on a HUGE amount of false-positives about a missing primary header when running this command: $ ./Tools/Scripts/check-webkit-style --filter=-,+build/include_order Source/WTF 2>&1 | grep -i primary Created attachment 370883 [details]
Patch v1
Comment on attachment 370883 [details]
Patch v1
r=me
This is necessary so that #pragma once can work with WTF headers, which would otherwise sometimes be included from the build directory and sometimes from the source causing duplicate declarations from including two copies of the same header.
(In reply to Alex Christensen from comment #3) > Comment on attachment 370883 [details] > Patch v1 > > r=me > This is necessary so that #pragma once can work with WTF headers, which > would otherwise sometimes be included from the build directory and sometimes > from the source causing duplicate declarations from including two copies of > the same header. Thanks for the explanation! Comment on attachment 370883 [details] Patch v1 Clearing flags on attachment: 370883 Committed r245870: <https://trac.webkit.org/changeset/245870> All reviewed patches have been landed. Closing bug. |