commit b804dd3e5bd3397fa5bbb3a5313496e9156d4fb5
author Sergey Ponomarev <stokito@gmail.com> Sat Sep 14 21:14:43 2019 +0300
committer Sergey Ponomarev <stokito@gmail.com> Sat Sep 14 21:14:43 2019 +0300
tree 9422f516889d9f87eea9864e7c9d069313217f52
parent a4ca246ecae023469a6c93c0c274f2bc97d7103f

#754 move sufixlist upper and improve error message on missing suffix

programs/fileio.c

diff --git a/programs/fileio.c b/programs/fileio.c
index 8a45563..e5fb1aa 100644
--- a/programs/fileio.c
+++ b/programs/fileio.c
@@ -2169,12 +2169,24 @@
     static size_t dfnbCapacity = 0;
     static char* dstFileNameBuffer = NULL;   /* using static allocation : this function cannot be multi-threaded */
 
+    const char* suffixlist = ZSTD_EXTENSION
+    #ifdef ZSTD_GZDECOMPRESS
+        "/" GZ_EXTENSION
+    #endif
+    #ifdef ZSTD_LZMADECOMPRESS
+        "/" XZ_EXTENSION "/" LZMA_EXTENSION
+    #endif
+    #ifdef ZSTD_LZ4DECOMPRESS
+        "/" LZ4_EXTENSION
+    #endif
+    ;
+
     size_t const sfnSize = strlen(srcFileName);
     size_t suffixSize;
     const char* const suffixPtr = strrchr(srcFileName, '.');
     if (suffixPtr == NULL) {
-        DISPLAYLEVEL(1, "zstd: %s: unknown suffix -- ignored \n",
-                        srcFileName);
+        DISPLAYLEVEL(1, "zstd: %s: missing suffix (%s expected) -- ignored \n",
+                        srcFileName, suffixlist);
         return NULL;
     }
     suffixSize = strlen(suffixPtr);
@@ -2193,17 +2205,6 @@
             && strcmp(suffixPtr, LZ4_EXTENSION)
         #endif
             ) ) {
-        const char* suffixlist = ZSTD_EXTENSION
-        #ifdef ZSTD_GZDECOMPRESS
-            "/" GZ_EXTENSION
-        #endif
-        #ifdef ZSTD_LZMADECOMPRESS
-            "/" XZ_EXTENSION "/" LZMA_EXTENSION
-        #endif
-        #ifdef ZSTD_LZ4DECOMPRESS
-            "/" LZ4_EXTENSION
-        #endif
-        ;
         DISPLAYLEVEL(1, "zstd: %s: unknown suffix (%s expected) -- ignored \n",
                      srcFileName, suffixlist);
         return NULL;