Martin Vahlensieck <open...@academicsolutions.ch> wrote:

>       if (!qflag) {
> -             fprintf(stderr, "Identity removed: %s %s (%s)\n", path,
> -                 sshkey_type(key), comment);
> +             fprintf(stderr, "Identity removed: %s %s%s%s%s\n", path,
> +                 sshkey_type(key), comment ? " (" : "",
> +                 comment ? comment : "", comment ? ")" : "");

this is probably better as something like

> -             fprintf(stderr, "Identity removed: %s %s (%s)\n", path,
> -                 sshkey_type(key), comment ? comment : "no comment");

Which has a minor ambiguity, but probably harms noone.

Reply via email to