FWIW, this removes the always true if and also the never executed else branch. Maybe that unused branch is there for some particular reason, but if not, I think it can be removed. Should I apply?
2011-05-10 Marek Polacek <[email protected]> * nm.c (show_symbols_sysv): Remove unused if branch. --- elfutils/src/nm.c.mp 2011-05-07 18:19:10.355146087 +0200 +++ elfutils/src/nm.c 2011-05-10 22:27:51.838447583 +0200 @@ -762,10 +762,7 @@ show_symbols_sysv (Ebl *ebl, GElf_Word s int digits = length_map[gelf_getclass (ebl->elf) - 1][radix]; /* We always print this prolog. */ - if (prefix == NULL || 1) - printf (gettext ("\n\nSymbols from %s:\n\n"), fullname); - else - printf (gettext ("\n\nSymbols from %s[%s]:\n\n"), prefix, fname); + printf (gettext ("\n\nSymbols from %s:\n\n"), fullname); /* The header line. */ printf (gettext ("%*s%-*s %-*s Class Type %-*s %*s Section\n\n"), _______________________________________________ elfutils-devel mailing list [email protected] https://fedorahosted.org/mailman/listinfo/elfutils-devel
