11.12.2016, 11:18
Congratulations to your release but
- No idea why you added the cmdid parameter to the command
- Makes it harder to change the command processor
- Isn't even needed, you just could use cid_commandname
- You should change the prefix and macro names, makes it impossible to test in one go
Do it like in ycmd, only add a compatibility macro while using your own namespace - Benchmark script is outdated (on first page) some functions are undefined