Add missing override specifiers
ClosedPublic

Authored by kfunk on Dec 16 2016, 4:46 PM.

Details

Diff Detail

Repository
R34 Marble
Branch
wip-override
Lint
No Linters Available
Unit
No Unit Test Coverage
kfunk updated this revision to Diff 9090.Dec 16 2016, 4:46 PM
kfunk retitled this revision from to Add missing override specifiers.
kfunk updated this object.
kfunk edited the test plan for this revision. (Show Details)
nienhueser accepted this revision.Dec 16 2016, 7:20 PM
nienhueser added a reviewer: nienhueser.
nienhueser added a subscriber: nienhueser.

Thanks for cleaning up. Did you use some tool / script to automate it?

This revision is now accepted and ready to land.Dec 16 2016, 7:20 PM
kfunk added a comment.Dec 16 2016, 8:16 PM

Thanks for cleaning up. Did you use some tool / script to automate it?

Ran this in the build dir:

$ run-clang-tidy-3.8.py -clang-tidy-binary clang-tidy-3.8 -clang-apply-replacements-binary clang-apply-replacements-3.8 -header-filter='.*' -checks='-*,modernize-use-override' -fix
nienhueser closed this revision.Dec 20 2016, 7:30 AM