mirror of
https://github.com/claunia/flac.git
synced 2025-12-16 18:54:26 +00:00
Fixed bug where using --replay-gain without any padding option caused only a small PADDING block to be created (SF#1760790: https://sourceforge.net/tracker/index.php?func=detail&aid=1760790&group_id=13478&atid=113478)
This commit is contained in:
@@ -80,6 +80,7 @@
|
|||||||
flac:
|
flac:
|
||||||
<ul>
|
<ul>
|
||||||
<li>Added a new option <span class="argument"><a href="documentation_tools_flac.html#flac_options_no_utf8_convert">--no-utf8-convert</a></span> which works like it does in <span class="commandname">metaflac</span> (<a href="https://sourceforge.net/tracker/index.php?func=detail&aid=973740&group_id=13478&atid=363478">SF #973740</a>).</li>
|
<li>Added a new option <span class="argument"><a href="documentation_tools_flac.html#flac_options_no_utf8_convert">--no-utf8-convert</a></span> which works like it does in <span class="commandname">metaflac</span> (<a href="https://sourceforge.net/tracker/index.php?func=detail&aid=973740&group_id=13478&atid=363478">SF #973740</a>).</li>
|
||||||
|
<li>Fixed bug where using <span class="argument">--replay-gain</span> without any padding option caused only a small PADDING block to be created (<a href="https://sourceforge.net/tracker/index.php?func=detail&aid=1760790&group_id=13478&atid=113478">SF #1760790</a>).</li>
|
||||||
</ul>
|
</ul>
|
||||||
</li>
|
</li>
|
||||||
<li>
|
<li>
|
||||||
|
|||||||
@@ -28,6 +28,7 @@
|
|||||||
#include <stdlib.h>
|
#include <stdlib.h>
|
||||||
#include <string.h>
|
#include <string.h>
|
||||||
#include <time.h>
|
#include <time.h>
|
||||||
|
#include <unistd.h>
|
||||||
|
|
||||||
#if !defined _MSC_VER && !defined __MINGW32__
|
#if !defined _MSC_VER && !defined __MINGW32__
|
||||||
/* unlink is in stdio.h in VC++ */
|
/* unlink is in stdio.h in VC++ */
|
||||||
@@ -432,7 +433,10 @@ int do_it(void)
|
|||||||
* tags that we will set later, to avoid rewriting the
|
* tags that we will set later, to avoid rewriting the
|
||||||
* whole file.
|
* whole file.
|
||||||
*/
|
*/
|
||||||
if(option_values.padding <= 0) {
|
if(
|
||||||
|
(option_values.padding >= 0 && option_values.padding < GRABBAG__REPLAYGAIN_MAX_TAG_SPACE_REQUIRED) ||
|
||||||
|
(option_values.padding < 0 && FLAC_ENCODE__DEFAULT_PADDING < GRABBAG__REPLAYGAIN_MAX_TAG_SPACE_REQUIRED)
|
||||||
|
) {
|
||||||
flac__utils_printf(stderr, 1, "NOTE: --replay-gain may leave a small PADDING block even with --no-padding\n");
|
flac__utils_printf(stderr, 1, "NOTE: --replay-gain may leave a small PADDING block even with --no-padding\n");
|
||||||
option_values.padding = GRABBAG__REPLAYGAIN_MAX_TAG_SPACE_REQUIRED;
|
option_values.padding = GRABBAG__REPLAYGAIN_MAX_TAG_SPACE_REQUIRED;
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user