Hi Evan,

Evan Layton wrote:
> Hi Jan,
>
> The code changes look fine to me and thanks for adding the great comment
> on why the extra blank line is needed here. That definitely makes this
> more readable. :-)

No problem at all :-)

>
> I also saw Jack's comment on the use of "CLI cmd:" and wondered about 
> that as well. I agree with him that something for example like "exec 
> command:" or "command run:" might be more understandable for this prefix.

Done - the former one was chosen.
I have updated webrev accordingly.

Thanks a lot for review !
Jan


Reply via email to