[cmaster-next] [PATCH] tools: teach xml2cli.pl the "{}" notation for optional arguments

[Renato Westphal]

This patch is wrong, please ignore it. I'll send a new version now.

On Mon, Dec 19, 2016 at 6:35 PM, Renato Westphal
<renato at opensourcerouting.org> wrote:
> Requested by Daniel Walton.
>
> Signed-off-by: Renato Westphal <renato at opensourcerouting.org>
> ---
>  tools/xml2cli.pl | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/tools/xml2cli.pl b/tools/xml2cli.pl
> index 1b45e7f..b84ab67 100755
> --- a/tools/xml2cli.pl
> +++ b/tools/xml2cli.pl
> @@ -196,7 +196,11 @@ sub generate_code {
>                         $helpstr .= $::options{$options_name}{'help'};
>                 } else {
>                         $funcname .= $node{'name'} . " ";
> -                       $cmdstr .= $node{'name'} . " ";
> +                       if ($node{'optional'}) {
> +                               $cmdstr .= "{" . $node{'name'} . "} ";
> +                       } else {
> +                               $cmdstr .= $node{'name'} . " ";
> +                       }
>                         $helpstr .= "\n       \"" . $node{'help'} . "\\n\"";
>                 }
>
> @@ -279,6 +283,7 @@ sub parse_tree {
>         $node{'help'} = $xml_node->findvalue('./@help');
>         $node{'function'} = $xml_node->findvalue('./@function');
>         $node{'ifdef'} = $xml_node->findvalue('./@ifdef');
> +       $node{'optional'} = $xml_node->findvalue('./@optional');
>
>         # push node to stack
>         push (@nodes, \%node);
> --
> 1.9.1
>



-- 
Renato Westphal