Skip to content

CmdBuilder has inconsistent naming for its methods #135

@SimonCROS

Description

@SimonCROS

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.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions