Page MenuHomePhabricator

[LLDB] Fix Clang-tidy misc-use-override warnings in some files in include/lldb/Interpreter and Host; other minor fixes.
ClosedPublic

Authored by Eugene.Zelenko on Oct 9 2015, 1:20 PM.

Details

Summary

This is second attempt based on current code.

I checked this patch on my own build on RHEL 6.

Please check it in if it's OK, because I don't have SVN write access.

Diff Detail

Event Timeline

Eugene.Zelenko retitled this revision from to [LLDB] Fix Clang-tidy misc-use-override warnings in some files in include/lldb/Interpreter and Host; other minor fixes..
Eugene.Zelenko updated this object.
Eugene.Zelenko added reviewers: brucem, labath, clayborg.
Eugene.Zelenko set the repository for this revision to rL LLVM.
Eugene.Zelenko added a subscriber: lldb-commits.

Ping. Please review this patch, since these files is largest source of modernize-use-override warnings warnings in LLDB code.

labath accepted this revision.Oct 22 2015, 2:47 AM
labath edited edge metadata.

looks reasonable

This revision is now accepted and ready to land.Oct 22 2015, 2:47 AM
clayborg accepted this revision.Oct 22 2015, 9:27 AM
clayborg edited edge metadata.

Looks good. I would venture to say just checkin any clang tidy things that aren't changing code functionality without need for review.

This revision was automatically updated to reflect the committed changes.