diff options
| author | Lasse Collin <lasse.collin@tukaani.org> | 2022-10-25 18:30:55 +0300 |
|---|---|---|
| committer | Lasse Collin <lasse.collin@tukaani.org> | 2022-10-25 18:30:55 +0300 |
| commit | 8b46ae8cdeddfd7dc01fec92971b8696e9a96c5d (patch) | |
| tree | 11c755617e662aa047f484f9068c73a8368cd229 /src/xz | |
| parent | xz: If input file cannot be removed, treat it as a warning, not error. (diff) | |
| download | xz-8b46ae8cdeddfd7dc01fec92971b8696e9a96c5d.tar.xz | |
xz: Refactor to remove is_empty_filename().
Long ago it was used in list.c too but nowadays it's needed
only in io_open_src() so it's nicer to avoid a separate function.
Diffstat (limited to 'src/xz')
| -rw-r--r-- | src/xz/file_io.c | 4 | ||||
| -rw-r--r-- | src/xz/util.c | 12 | ||||
| -rw-r--r-- | src/xz/util.h | 4 |
3 files changed, 3 insertions, 17 deletions
diff --git a/src/xz/file_io.c b/src/xz/file_io.c index b559ae35..046ca7e3 100644 --- a/src/xz/file_io.c +++ b/src/xz/file_io.c @@ -748,8 +748,10 @@ error: extern file_pair * io_open_src(const char *src_name) { - if (is_empty_filename(src_name)) + if (src_name[0] == '\0') { + message_error(_("Empty filename, skipping")); return NULL; + } // Since we have only one file open at a time, we can use // a statically allocated structure. diff --git a/src/xz/util.c b/src/xz/util.c index a1339f4f..9f9a8fb0 100644 --- a/src/xz/util.c +++ b/src/xz/util.c @@ -261,18 +261,6 @@ my_snprintf(char **pos, size_t *left, const char *fmt, ...) extern bool -is_empty_filename(const char *filename) -{ - if (filename[0] == '\0') { - message_error(_("Empty filename, skipping")); - return true; - } - - return false; -} - - -extern bool is_tty_stdin(void) { const bool ret = isatty(STDIN_FILENO); diff --git a/src/xz/util.h b/src/xz/util.h index a2516bf9..4a536f52 100644 --- a/src/xz/util.h +++ b/src/xz/util.h @@ -105,10 +105,6 @@ extern void my_snprintf(char **pos, size_t *left, const char *fmt, ...) lzma_attribute((__format__(__printf__, 3, 4))); -/// \brief Check if filename is empty and print an error message -extern bool is_empty_filename(const char *filename); - - /// \brief Test if stdin is a terminal /// /// If stdin is a terminal, an error message is printed and exit status set |