Closed Bug 1158656 Opened 6 years ago Closed 5 years ago

Add a .clang-tidy file for misc-use-override

Categories

(Firefox Build System :: Source Code Analysis, defect)

defect
Not set
normal

Tracking

(firefox53 fixed)

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

References

Details

Attachments

(1 file)

My first attempt at using clang-tidy on the Mozilla code base.

For now we just enable the misc-use-override check, and specify the
KeepVirtual option that I have implemented for this check so that
it doesn't complain about usage of the virtual keyword alongside
override/final.
Attachment #8597764 - Flags: review?(jmuizelaar)
Assignee: nobody → ehsan
Summary: Add a .clang-tify file for misc-use-override → Add a .clang-tidy file for misc-use-override
Attachment #8597764 - Flags: review?(jmuizelaar) → review+
Note to self: Add llvm-namespace-comment (see bug 1182996)
Depends on: 1183143
I will take care of the followup
Keywords: checkin-needed
Depends on: 1319393
https://hg.mozilla.org/mozilla-central/rev/a180081b5cb3
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.