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.