> Maybe the naming would be nicer if we did: > > target_init_cmd_cdb > and > target_parse_cmd_cdb > > This would match each other's pattern and also match the style of the > other cmd related function naming where its "target or transport" > prefix, verb, cmd then optionally something extra. > > Or maybe: > > target_cmd_init_cdb > and > target_cmd_parse_cdb > > so they at least match each other and you get an idea they pair together. > > Just a suggestion and not a requirement, because I'm pretty bad at > naming, so I have no idea if its better or not. > Thank you Mike, Nice suggestion. I will incorporate the name change and send out another patch shortly. Thanks Sudhakar
