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

RESOLVED FIXED in Firefox 53

Status

defect
RESOLVED FIXED
4 years ago
Last year

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

unspecified
mozilla53
Dependency tree / graph

Firefox Tracking Flags

(firefox53 fixed)

Details

Attachments

(1 attachment)

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
https://hg.mozilla.org/mozilla-central/rev/a180081b5cb3
Status: NEW → RESOLVED
Closed: 3 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.