> -char *program_name;
> +const char *argp_program_bug_address = "<" PACKAGE_BUGREPORT ">";
>
> Maybe this should be a macro, or define in one place?
Looks reasonable. I'd propose this:
#define ARGP_PROGRAM_DATA(name) \
const char *argp_program_bug_address = "<" PACKAGE_BUGREPORT ">"; \
const char *argp_program_version = \
#name " (" PACKAGE_NAME ") " PACKAGE_VERSION \
"\nThis is free software. You may redistribute copies of it under the terms
of\" \
"\nthe GNU General Public License <http://www.gnu.org/licenses/gpl.html>." \
"\nThere is NO WARRANTY, to the extent permitted by law."
Then the appropriate initailization for ping would be:
ARGP_PROGRAM_DATA(ping);
This begs the question, should we have the `Written by FOO' string?
I'm either way on that, I asked Debarshi to keep that stuff.
_______________________________________________
bug-inetutils mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/bug-inetutils
