Cleanup #40578
mds: reorganize class members in headers to follow coding guidelines
% Done:
100%
Tags:
Backport:
Reviewed:
Affected Versions:
Description
Guide here: https://google.github.io/styleguide/cppguide.html#Declaration_Order
A past commit that has improved this: https://github.com/ceph/ceph/commit/36f3bc067f0971d490ab4aabe4feab2996138275
All of the MDS headers should follow these guidelines. This is especially helpful in identifying the public interface to a MDS class. Right now many MDS headers mix private/public in ways that makes understanding the class very difficult. For example:
Subtasks
History
#1 Updated by Jos Collin almost 2 years ago
- Pull request ID set to 28860
#2 Updated by Patrick Donnelly almost 2 years ago
- Status changed from New to In Progress
#3 Updated by Patrick Donnelly almost 2 years ago
- Tracker changed from Bug to Cleanup
#4 Updated by Patrick Donnelly over 1 year ago
- Status changed from In Progress to Resolved