Closed
Bug 1158656
Opened 10 years ago
Closed 8 years ago
Add a .clang-tidy file for misc-use-override
Categories
(Developer Infrastructure :: Source Code Analysis, defect)
Developer Infrastructure
Source Code Analysis
Tracking
(firefox53 fixed)
RESOLVED
FIXED
mozilla53
| Tracking | Status | |
|---|---|---|
| firefox53 | --- | fixed |
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
References
Details
Attachments
(1 file)
|
1020 bytes,
patch
|
jrmuizel
:
review+
|
Details | Diff | Splinter Review |
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.
|
Assignee | |
Comment 1•10 years ago
|
||
Attachment #8597764 -
Flags: review?(jmuizelaar)
|
|
Assignee | |
Updated•10 years ago
|
Assignee: nobody → ehsan
Summary: Add a .clang-tify file for misc-use-override → Add a .clang-tidy file for misc-use-override
|
||
Updated•10 years ago
|
Attachment #8597764 -
Flags: review?(jmuizelaar) → review+
|
|
Assignee | |
Comment 2•10 years ago
|
||
Note to self: Add llvm-namespace-comment (see bug 1182996)
|
||
Comment 4•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/a180081b5cb3f7c65d9f594cc278feafd1bf84db
Bug 1158656 - Add a .clang-tify file for misc-use-override; r=jrmuizel
|
|
||
Updated•8 years ago
|
Keywords: checkin-needed
|
||
Comment 5•8 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
|
||
Updated•7 years ago
|
Product: Core → Firefox Build System
|
|
||
Updated•3 years ago
|
Product: Firefox Build System → Developer Infrastructure
You need to log in
before you can comment on or make changes to this bug.
Description
•