Details
Description
In src/slave/paths.cpp a number of string constants are defined to describe the formats of various paths. However, given there is a 1:1 mapping between the string constant and the functions that build the paths, the code would be more readable if the format strings were inline in the functions.
In the cases where one constant depends on another (see the EXECUTOR_INFO_PATH, EXECUTOR_PATH, FRAMEWORK_PATH, SLAVE_PATH, ROOT_PATH chain, for example) the function calls can just be chained together.
This will have the added benefit of removing some statically constructed string constants, which are dangerous.
Attachments
Issue Links
- blocks
-
MESOS-1023 Replace all static/global variables with non-POD type
- Accepted