Fix end pops when trimming wav files

-Added trim limit constant at top of file, defining at which db trimming should occur (moved from being in the code itself)
-Added fade out frames constant at top of file, defining how many frames should have fade out applied (to avoid pops at the end of trim)
-Rewrote parts of the trimming logic to use an average of volume across all channels instead of any particular channel
-Added fade-out to trimming

(cherry picked from commit 26cc521d55)
This commit is contained in:
Theis Egeberg 2019-04-03 19:11:08 +02:00 committed by Rémi Verschelde
parent 21d3ad2e52
commit 330c7a2fb4
1 changed files with 27 additions and 13 deletions

View File

@ -35,6 +35,9 @@
#include "core/os/file_access.h" #include "core/os/file_access.h"
#include "scene/resources/audio_stream_sample.h" #include "scene/resources/audio_stream_sample.h"
const float TRIM_DB_LIMIT = -50;
const int TRIM_FADE_OUT_FRAMES = 500;
String ResourceImporterWAV::get_importer_name() const { String ResourceImporterWAV::get_importer_name() const {
return "wav"; return "wav";
@ -393,31 +396,42 @@ Error ResourceImporterWAV::import(const String &p_source_file, const String &p_s
if (trim && !loop && format_channels > 0) { if (trim && !loop && format_channels > 0) {
int first = 0; int first = 0;
int last = (frames * format_channels) - 1; int last = (frames / format_channels) - 1;
bool found = false; bool found = false;
float limit = Math::db2linear((float)-30); float limit = Math::db2linear(TRIM_DB_LIMIT);
for (int i = 0; i < data.size(); i++) {
float amp = Math::abs(data[i]); for (int i = 0; i < data.size() / format_channels; i++) {
float ampChannelSum = 0;
for (int j = 0; j < format_channels; j++) {
ampChannelSum += Math::abs(data[(i * format_channels) + j]);
}
float amp = Math::abs(ampChannelSum / (float)format_channels);
if (!found && amp > limit) { if (!found && amp > limit) {
first = i; first = i / format_channels;
found = true; found = true;
} }
if (found && amp > limit) { if (found && amp > limit) {
last = i; last = i / format_channels;
} }
} }
first /= format_channels;
last /= format_channels;
if (first < last) { if (first < last) {
Vector<float> new_data; Vector<float> new_data;
new_data.resize((last - first + 1) * format_channels); new_data.resize((last - first) * format_channels);
for (int i = first * format_channels; i < (last + 1) * format_channels; i++) { for (int i = first; i < last; i++) {
new_data.write[i - first * format_channels] = data[i];
float fadeOutMult = 1;
if (last - i < TRIM_FADE_OUT_FRAMES) {
fadeOutMult = ((float)(last - i - 1) / (float)TRIM_FADE_OUT_FRAMES);
}
for (int j = 0; j < format_channels; j++) {
new_data.write[((i - first) * format_channels) + j] = data[(i * format_channels) + j] * fadeOutMult;
}
} }
data = new_data; data = new_data;