ywkaras commented on pull request #7478: URL: https://github.com/apache/trafficserver/pull/7478#issuecomment-794797303
Since this is a big new module, I suggest we consider organizing the directory structure in a way that makes the scope of each code file more clear. One approach would be to put the .cc files currently in "mgmt2", plus any .h file that is only include within the "mgmt2" directory tree, in a new subdirectory "mgmt2/private". This approach would be applied recursively. So any code currently file in "mgmt2/rpc" that is not used outside the "mgmt2/rpc" directory would be moved to "mgmt2/rpc/private". If a file in "mgmt2/rpc/private" included the file "mgmt2/private/yada.h" it would be included as: ``` #include "../../private/yada.h" ``` ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
