Suppress compression failure message if it is not an error

If the user specifies `--no-error-on-compression-fail`, there is no need
to treat compression failure as an error. Hence, the failure message
should be suppressed along with the non-zero exit code (which already
is suppressed).
This commit is contained in:
Karthik Periagaram 2019-01-31 20:51:30 -08:00 committed by Erik de Castro Lopo
parent 421961f00b
commit cc15b7427a
1 changed files with 2 additions and 3 deletions

View File

@ -1676,7 +1676,7 @@ int EncoderSession_finish_ok(EncoderSession *e, int info_align_carry, int info_a
}
}
if (e->compression_ratio >= 1.0) {
if (e->compression_ratio >= 1.0 && error_on_compression_fail) {
flac__utils_printf(stderr, 1,
"FAILURE: Compression failed (ratio %0.3f, should be < 1.0).\n"
"This happens for some files for one or more of the following reasons:\n"
@ -1684,8 +1684,7 @@ int EncoderSession_finish_ok(EncoderSession *e, int info_align_carry, int info_a
" * Insufficient input data (e.g. very short files, < 10000 frames).\n"
" * The audio data is not compressible (e.g. a full range white noise signal).\n"
, e->compression_ratio);
if (error_on_compression_fail)
ret = 1;
ret = 1;
}
EncoderSession_destroy(e);