Cleanup #40578: mds: reorganize class members in headers to follow coding guidelines - fs - Ceph

Cleanup #40578

mds: reorganize class members in headers to follow coding guidelines

Added by Patrick Donnelly about 2 months ago. Updated 11 days ago.

Status:

In Progress

Priority:

Normal

Assignee:

Varsha Rao

Category:

Code Hygiene

Target version:

Ceph - v15.0.0

Start date:

Due date:

% Done:

50%

Tags:

Backport:

Reviewed:

Affected Versions:

Component(FS):

MDS

Labels (FS):

task(easy)

Pull request ID:

28860

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:

https://github.com/ceph/ceph/blob/8ba339a41b2178cb27d4421bc512f3e622824d1d/src/mds/MDCache.h#L1108-L1130

Subtasks