diff options
author | Lasse Collin <lasse.collin@tukaani.org> | 2009-09-19 09:47:30 +0300 |
---|---|---|
committer | Lasse Collin <lasse.collin@tukaani.org> | 2009-09-19 09:47:30 +0300 |
commit | e599bba4216c0edb8cc8f40adad3a6dba88685f4 (patch) | |
tree | d52c699f599e276ff65302e5854a26998da1a928 /src/lzmainfo | |
parent | 49cfc8d392cf535f8dd10233225b1fc726fec9ef (diff) | |
download | xz-e599bba4216c0edb8cc8f40adad3a6dba88685f4.tar.gz |
Various changes.
Separate a few reusable components from XZ Utils specific
code. The reusable code is now in "tuklib" modules. A few
more could be separated still, e.g. bswap.h.
Fix some bugs in lzmainfo.
Fix physmem and cpucores code on OS/2. Thanks to Elbert Pol
for help.
Add OpenVMS support into physmem. Add a few #ifdefs to ease
building XZ Utils on OpenVMS. Thanks to Jouk Jansen for the
original patch.
Diffstat (limited to 'src/lzmainfo')
-rw-r--r-- | src/lzmainfo/Makefile.am | 5 | ||||
-rw-r--r-- | src/lzmainfo/lzmainfo.c | 65 |
2 files changed, 24 insertions, 46 deletions
diff --git a/src/lzmainfo/Makefile.am b/src/lzmainfo/Makefile.am index 22295ee..cbc7e5c 100644 --- a/src/lzmainfo/Makefile.am +++ b/src/lzmainfo/Makefile.am @@ -7,7 +7,10 @@ bin_PROGRAMS = lzmainfo -lzmainfo_SOURCES = lzmainfo.c +lzmainfo_SOURCES = \ + lzmainfo.c \ + $(top_srcdir)/src/common/tuklib_progname.c \ + $(top_srcdir)/src/common/tuklib_exit.c lzmainfo_CPPFLAGS = \ -DLOCALEDIR=\"$(localedir)\" \ diff --git a/src/lzmainfo/lzmainfo.c b/src/lzmainfo/lzmainfo.c index d9ae311..006a6b0 100644 --- a/src/lzmainfo/lzmainfo.c +++ b/src/lzmainfo/lzmainfo.c @@ -23,39 +23,9 @@ #include "lzma.h" #include "getopt.h" - - -/// Name of the program from argv[0] -static const char *argv0; - - -/// Close stdout unless we are already going to exit with EXIT_FAILURE. -/// If closing stdout fails, set exit status to EXIT_FAILURE and print -/// an error message to stderr. We don't care about closing stderr, -/// because we don't print anything to stderr unless we are going to -/// use EXIT_FAILURE anyway. -static void lzma_attribute((noreturn)) -my_exit(int status) -{ - if (status != EXIT_FAILURE) { - const int ferror_err = ferror(stdout); - const int fclose_err = fclose(stdout); - - if (ferror_err || fclose_err) { - // If it was fclose() that failed, we have the reason - // in errno. If only ferror() indicated an error, - // we have no idea what the reason was. - fprintf(stderr, "%s: %s: %s\n", argv0, - _("Writing to standard output " - "failed"), - fclose_err ? strerror(errno) - : _("Unknown error")); - status = EXIT_FAILURE; - } - } - - exit(status); -} +#include "tuklib_gettext.h" +#include "tuklib_progname.h" +#include "tuklib_exit.h" static void lzma_attribute((noreturn)) @@ -63,7 +33,7 @@ help(void) { printf( _("Usage: %s [--help] [--version] [FILE]...\n" -"Show information stored in the .lzma file header"), argv0); +"Show information stored in the .lzma file header"), progname); printf(_( "\nWith no FILE, or when FILE is -, read standard input.\n")); @@ -73,7 +43,7 @@ _("Usage: %s [--help] [--version] [FILE]...\n" PACKAGE_BUGREPORT); printf(_("%s home page: <%s>\n"), PACKAGE_NAME, PACKAGE_HOMEPAGE); - my_exit(EXIT_SUCCESS); + tuklib_exit(EXIT_SUCCESS, EXIT_FAILURE, true); } @@ -81,7 +51,7 @@ static void lzma_attribute((noreturn)) version(void) { puts("lzmainfo (" PACKAGE_NAME ") " PACKAGE_VERSION); - my_exit(EXIT_SUCCESS); + tuklib_exit(EXIT_SUCCESS, EXIT_FAILURE, true); } @@ -135,7 +105,7 @@ lzmainfo(const char *name, FILE *f) uint8_t buf[13]; const size_t size = fread(buf, 1, sizeof(buf), f); if (size != 13) { - fprintf(stderr, "%s: %s: %s\n", argv0, name, + fprintf(stderr, "%s: %s: %s\n", progname, name, ferror(f) ? strerror(errno) : _("File is too small to be a .lzma file")); return true; @@ -149,16 +119,17 @@ lzmainfo(const char *name, FILE *f) break; case LZMA_OPTIONS_ERROR: - fprintf(stderr, "%s: %s: %s\n", argv0, name, + fprintf(stderr, "%s: %s: %s\n", progname, name, _("Not a .lzma file")); return true; case LZMA_MEM_ERROR: - fprintf(stderr, "%s: %s\n", argv0, strerror(ENOMEM)); + fprintf(stderr, "%s: %s\n", progname, strerror(ENOMEM)); exit(EXIT_FAILURE); default: - fprintf(stderr, "%s: %s\n", argv0, _("Internal error (bug)")); + fprintf(stderr, "%s: %s\n", progname, + _("Internal error (bug)")); exit(EXIT_FAILURE); } @@ -202,16 +173,19 @@ lzmainfo(const char *name, FILE *f) extern int main(int argc, char **argv) { - int ret = EXIT_SUCCESS; - argv0 = argv[0]; + tuklib_progname_init(argv); + tuklib_gettext_init(PACKAGE, LOCALEDIR); parse_args(argc, argv); + int ret = EXIT_SUCCESS; + // We print empty lines around the output only when reading from // files specified on the command line. This is due to how // LZMA Utils did it. if (optind == argc) { - lzmainfo("(stdin)", stdin); + if (lzmainfo("(stdin)", stdin)) + ret = EXIT_FAILURE; } else { printf("\n"); @@ -224,7 +198,8 @@ main(int argc, char **argv) if (f == NULL) { ret = EXIT_FAILURE; fprintf(stderr, "%s: %s: %s\n", - argv0, argv[optind], + progname, + argv[optind], strerror(errno)); continue; } @@ -238,5 +213,5 @@ main(int argc, char **argv) } while (++optind < argc); } - my_exit(ret); + tuklib_exit(ret, EXIT_FAILURE, true); } |