PascalCase and camelCase are mixed in this snippet
|
CmdBuilder &addChecker(bool (*ft)(CmdSpec &cmd, size_t idx)); |
|
CmdBuilder &CmExecutor(void (*ft)(CmdSpec &cmd)); |
|
CmdBuilder &Name(const std::string &name); |
|
CmdBuilder &addParam(e_param type, CmdParam *param); |
|
CmdBuilder &Registration(int stage); |
|
CmdSpec *build(); |
Also, CmExecutor is using a prefix which is not used somewhere else, cmdExecutor or simply executor might look better.
PascalCase and camelCase are mixed in this snippet
irc/server/headers/builder/command-validation/CmdSpec.hpp
Lines 88 to 93 in 22ee811
Also,
CmExecutoris using a prefix which is not used somewhere else,cmdExecutoror simplyexecutormight look better.