Silence compiler warning in make_huffman_decode_table()

[wimlib] / src / decompress_common.c

diff --git a/src/decompress_common.c b/src/decompress_common.c
index 64e171d9491fa1bc3277045bd5719a503b354d29..66440efe203acaed25f56d6a45ddf07d12c25ab7 100644 (file)
--- a/src/decompress_common.c
+++ b/src/decompress_common.c
@@ -15,7 +15,6 @@
 #endif
 
 #include "wimlib/decompress_common.h"
-#include "wimlib/error.h"
 #include "wimlib/util.h" /* for BUILD_BUG_ON()  */
 
 #include <string.h>
@@ -179,7 +178,6 @@ make_huffman_decode_table(u16 decode_table[const restrict],
                if (unlikely(left < 0)) {
                        /* The lengths overflow the codespace; that is, the code
                         * is over-subscribed.  */
-                       DEBUG("Invalid prefix code (over-subscribed)");
                        return -1;
                }
        }
@@ -201,7 +199,6 @@ make_huffman_decode_table(u16 decode_table[const restrict],
                               table_num_entries * sizeof(decode_table[0]));
                        return 0;
                }
-               DEBUG("Invalid prefix code (incomplete set)");
                return -1;
        }
 
@@ -295,13 +292,7 @@ make_huffman_decode_table(u16 decode_table[const restrict],
 
                        v = MAKE_DIRECT_ENTRY(sorted_syms[sym_idx], codeword_len);
                        v |= v << 16;
-                       if (sizeof(unsigned long) == 8) {
-                               /* This may produce a compiler warning if an
-                                * 'unsigned long' is 32 bits, but this won't be
-                                * executed unless an 'unsigned long' is at
-                                * least 64 bits anyway.  */
-                               v |= v << 32;
-                       }
+                       v |= v << (sizeof(unsigned long) == 8 ? 32 : 0);
 
                        p = (aliased_long_t *)decode_table_ptr;
                        n = stores_per_loop;