xz: Remove unused code and simplify

opt_mode == MODE_COMPRESS isn't possible when HAVE_ENCODERS isn't
defined. Thus, when *encoding*, the message about *decoder* memory
usage is possible to show only when both encoder and decoder have
been built.

Since the message is shown only at V_DEBUG, skip the memusage
calculation if verbosity level isn't high enough.

Fixes: 5f0c5a0438
This commit is contained in:
Lasse Collin 2024-05-10 20:33:08 +03:00
parent 31358c057c
commit 347b412a93
1 changed files with 4 additions and 12 deletions

View File

@ -532,18 +532,10 @@ coder_set_compression_settings(void)
// NOTE: If only encoder support was built, we cannot show the // NOTE: If only encoder support was built, we cannot show the
// what the decoder memory usage will be. // what the decoder memory usage will be.
message_mem_needed(V_DEBUG, memory_usage); message_mem_needed(V_DEBUG, memory_usage);
#ifdef HAVE_DECODERS
if (opt_mode == MODE_COMPRESS) { #if defined(HAVE_ENCODERS) && defined(HAVE_DECODERS)
#ifdef HAVE_ENCODERS if (opt_mode == MODE_COMPRESS && message_verbosity_get() >= V_DEBUG) {
const uint64_t decmem = const uint64_t decmem = filters_memusage_max(NULL, false);
filters_memusage_max(NULL, false);
#else
// If encoders are not enabled, then --block-list is never
// usable and the filter chains 1-9 are never used.
// So there is no need to find the maximum decoder memory
// required in this case.
const uint64_t decmem = lzma_raw_decoder_memusage(filters[0]);
#endif
if (decmem != UINT64_MAX) if (decmem != UINT64_MAX)
message(V_DEBUG, _("Decompression will need " message(V_DEBUG, _("Decompression will need "
"%s MiB of memory."), uint64_to_str( "%s MiB of memory."), uint64_to_str(