Bug 1420366 - remove modernize-use-override from clang-bot config. r?sylvestre draft
authorAndi-Bogdan Postelnicu <bpostelnicu@mozilla.com>
Fri, 24 Nov 2017 12:26:32 +0200
changeset 703146 a63d5306da8428bd62e716e76cc03520b87582a1
parent 702882 3f5d48c08903475b5f556f3d5906773978b30489
child 741677 5b6787c5e50ee550713923f12ef283f9d8097681
push id90718
push userbmo:bpostelnicu@mozilla.com
push dateFri, 24 Nov 2017 10:26:50 +0000
reviewerssylvestre
bugs1420366
milestone59.0a1
Bug 1420366 - remove modernize-use-override from clang-bot config. r?sylvestre MozReview-Commit-ID: IEZ7NxP9pXe
tools/clang-tidy/config.yaml
--- a/tools/clang-tidy/config.yaml
+++ b/tools/clang-tidy/config.yaml
@@ -32,17 +32,18 @@ clang_checkers:
    publish: !!bool yes
  - name: modernize-use-equals-default
    publish: !!bool yes
  - name: modernize-use-equals-delete
    publish: !!bool yes
  - name: modernize-use-nullptr
    publish: !!bool yes
  - name: modernize-use-override
-   publish: !!bool yes
+   # Too noisy because of the way how we implement NS_IMETHOD. See Bug 1420366.
+   publish: !!bool no
  - name: mozilla-*
    publish: !!bool yes
  - name: performance-*
    publish: !!bool yes
  - name: readability-container-size-empty
    publish: !!bool yes
  - name: readability-else-after-return
    publish: !!bool yes