Am 27.12.15 um 21:58 schrieb Gert Doering:
>   push-remove route-ipv6 2001:
I think this is missing " " in the commit message, it is okay later in
the manpage

> +  else if (streq (p[0], "push-remove"))

I would add at least a && p[1] and possible a !p[2]

I think multiple push-remove are better than a single one taking
multiple arguments.

Otherwise are you more likely to end up with push-remove a b instead of
push-remove "a b"

Arne






Reply via email to