> On 04 Apr 2016, at 15:30, [email protected] wrote:
> 
> From: Max <[email protected]>
> 
> Add command 'show nat bsc' to display number of configured BSCs.

Thank you for moving this to a separate patch.


> +DEFUN(show_nat_bsc, show_nat_bsc_cmd, "show nat bsc",
> +      SHOW_STR "Display NAT configuration details\n"
> +      "BSCs-related\n")
> +{
> +     vty_out(vty, "%d BSCs configured%s", _nat->num_bsc, VTY_NEWLINE);
> +     return CMD_SUCCESS;
> +}


If somebody asked me what "show nat bsc" would do, I would not have guessed it 
prints a number. Can you think of a better name?

        "show nat configured-bsc-count"?
        "show nat bscs-configured"?
        "show nat num-bscs-configured"?

kind regards
        holger

Reply via email to