gecko-dev/tools/lint/cpp-virtual-final.yml
Ed Lee 144374ea8b Bug 1440421 part 1. Ignore activity-stream and node_modules for linting. r=ahal
MozReview-Commit-ID: EEua2y790cW

--HG--
extra : rebase_source : ce98dcbd82eb1f59594d6fe40e34648b7bd639de
2018-06-06 10:32:43 -07:00

33 lines
1.0 KiB
YAML

---
cpp-virtual-final:
description: "Virtual function declarations should specify only one of
`virtual`, `final`, or `override`"
level: warning
include: ['.']
exclude:
# Third-party code with cpp-virtual-final warnings:
- '**/node_modules'
- gfx/angle
- gfx/skia
- media/webrtc/trunk
- security/sandbox/chromium
extensions: ['cpp', 'h', 'mm']
type: regex
#
# This lint warns about:
#
# virtual void Bad1() final
# void Bad2() final override
# void Bad3() override final
#
# Caveats: This lint ...
#
# * Doesn't warn about `virtual void NotBad() override` at this time
# because there are 8000+ instances.
#
# * Doesn't warn about function declarations that span multiple lines
# because the regex can't match across line breaks.
#
# virtual ) final | final override | override final
payload: ^ *virtual .+\).+\bfinal\b|\bfinal +override\b|\boverride +final\b