Why do you think so?

— 
Damjan



> On 09.05.2022., at 03:43, jiangxiaom...@outlook.com wrote:
> 
> Hi Damjan,
>  With your patch, most of current clis have to be modified, it may be a large 
> work. I think the suitable way is changing the meaning of VLIB_CLI_COMMAND 
> function's second params from input to args.
> That's:
>  
> /* CLI command callback function. */
> 
> typedef clib_error_t *(vlib_cli_command_function_t)
> 
>   (struct vlib_main_t * vm,
> 
>    unformat_input_t * input (input changed to args), struct 
> vlib_cli_command_t * cmd)
> 
> 
> vlib_cli_dispatch_sub_commands(...) {
> ....
> 
>         c_error = c->function (vm, si (changed to args), c); => si changed to 
> args
> ...
> }
> 
> 
> Every command just parse it's own args.
> 
> 
> 
> 
> 
> 

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#21389): https://lists.fd.io/g/vpp-dev/message/21389
Mute This Topic: https://lists.fd.io/mt/90974441/21656
Group Owner: vpp-dev+ow...@lists.fd.io
Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to