Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix RPMFusion builds for armv7hl/ppc64le/aarch64 #3223

Merged
merged 4 commits into from
Oct 27, 2020
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/effects/builtin/autopaneffect.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ void AutoPanEffect::processChannel(
double sinusoid = 0;

// NOTE: Assuming engine is working in stereo.
for (unsigned int i = 0; i + 1 < bufferParameters.samplesPerBuffer(); i += 2) {
for (SINT i = 0; i + 1 < bufferParameters.samplesPerBuffer(); i += 2) {
const auto periodFraction = static_cast<CSAMPLE>(gs.time) / static_cast<CSAMPLE>(period);

// current quarter in the trigonometric circle
Expand Down
2 changes: 1 addition & 1 deletion src/effects/builtin/balanceeffect.h
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ class BalanceGroupState : public EffectState {

mixxx::SampleBuffer m_pHighBuf;

unsigned int m_oldSampleRate;
mixxx::audio::SampleRate m_oldSampleRate;
int m_freq;

CSAMPLE m_oldBalance;
Expand Down
6 changes: 3 additions & 3 deletions src/effects/builtin/biquadfullkilleqeffect.h
Original file line number Diff line number Diff line change
Expand Up @@ -52,10 +52,10 @@ class BiquadFullKillEQEffectGroupState : public EffectState {
double m_loFreqCorner;
double m_highFreqCorner;

int m_rampHoldOff;
int m_groupDelay;
SINT m_rampHoldOff;
SINT m_groupDelay;

unsigned int m_oldSampleRate;
mixxx::audio::SampleRate m_oldSampleRate;
};

class BiquadFullKillEQEffect : public EffectProcessorImpl<BiquadFullKillEQEffectGroupState> {
Expand Down
2 changes: 1 addition & 1 deletion src/effects/builtin/bitcrushereffect.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ void BitCrusherEffect::processChannel(const ChannelHandle& handle,
// rarely used, to achieve equal loudness and maximum dynamic
const CSAMPLE gainCorrection = (17 - bit_depth) / 8;

for (unsigned int i = 0;
for (SINT i = 0;
i < bufferParameters.samplesPerBuffer();
i += bufferParameters.channelCount()) {
pState->accumulator += downsample;
Expand Down
2 changes: 1 addition & 1 deletion src/effects/builtin/echoeffect.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -189,7 +189,7 @@ void EchoEffect::processChannel(const ChannelHandle& handle, EchoGroupState* pGr
bufferParameters.framesPerBuffer());

//TODO: rewrite to remove assumption of stereo buffer
for (unsigned int i = 0;
for (SINT i = 0;
i < bufferParameters.samplesPerBuffer();
i += bufferParameters.channelCount()) {
CSAMPLE_GAIN send_ramped = send.getNext();
Expand Down
7 changes: 3 additions & 4 deletions src/effects/builtin/flangereffect.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -195,10 +195,9 @@ void FlangerEffect::processChannel(const ChannelHandle& handle,
CSAMPLE* delayLeft = pState->delayLeft;
CSAMPLE* delayRight = pState->delayRight;

for (unsigned int i = 0;
i < bufferParameters.samplesPerBuffer();
i += bufferParameters.channelCount()) {

for (SINT i = 0;
i < bufferParameters.samplesPerBuffer();
i += bufferParameters.channelCount()) {
CSAMPLE_GAIN mix_ramped = mixRamped.getNext();
CSAMPLE_GAIN regen_ramped = regenRamped.getNext();
double width_ramped = widthRamped.getNext();
Expand Down
2 changes: 1 addition & 1 deletion src/effects/builtin/graphiceqeffect.h
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ class GraphicEQEffect : public EffectProcessorImpl<GraphicEQEffectGroupState> {
EngineEffectParameter* m_pPotLow;
QList<EngineEffectParameter*> m_pPotMid;
EngineEffectParameter* m_pPotHigh;
unsigned int m_oldSampleRate;
mixxx::audio::SampleRate m_oldSampleRate;

DISALLOW_COPY_AND_ASSIGN(GraphicEQEffect);
};
Expand Down
2 changes: 1 addition & 1 deletion src/effects/builtin/linkwitzriley8eqeffect.h
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ class LinkwitzRiley8EQEffectGroupState : public EffectState {
CSAMPLE* m_pMidBuf;
CSAMPLE* m_pHighBuf;

unsigned int m_oldSampleRate;
mixxx::audio::SampleRate m_oldSampleRate;
int m_loFreq;
int m_hiFreq;
};
Expand Down
2 changes: 1 addition & 1 deletion src/effects/builtin/loudnesscontoureffect.h
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ class LoudnessContourEffectGroupState final : public EffectState {
CSAMPLE_GAIN m_oldGain;
double m_oldFilterGainDb;
bool m_oldUseGain;
unsigned int m_oldSampleRate;
mixxx::audio::SampleRate m_oldSampleRate;
};

class LoudnessContourEffect
Expand Down
2 changes: 1 addition & 1 deletion src/effects/builtin/lvmixeqbase.h
Original file line number Diff line number Diff line change
Expand Up @@ -253,7 +253,7 @@ class LVMixEQEffectGroupState : public EffectState {
int m_rampHoldOff;
int m_groupDelay;

unsigned int m_oldSampleRate;
mixxx::audio::SampleRate m_oldSampleRate;
double m_loFreq;
double m_hiFreq;

Expand Down
2 changes: 1 addition & 1 deletion src/effects/builtin/metronomeclick.h
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
// This is a new recording from a real metronome
// it was converted from *.wav to plain text using Audacity

const unsigned int kClickSize44100 = 1024;
const SINT kClickSize44100 = 1024;
const CSAMPLE kClick44100[kClickSize44100] = {
0.03459f,
0.09436f,
Expand Down
12 changes: 6 additions & 6 deletions src/effects/builtin/metronomeeffect.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ void MetronomeEffect::processChannel(
return;
}

unsigned int clickSize = kClickSize44100;
SINT clickSize = kClickSize44100;
const CSAMPLE* click = kClick44100;
if (bufferParameters.sampleRate() >= 96000) {
clickSize = kClickSize96000;
Expand All @@ -87,12 +87,12 @@ void MetronomeEffect::processChannel(
click = kClick48000;
}

unsigned int maxFrames;
SINT maxFrames;
if (m_pSyncParameter->toBool() && groupFeatures.has_beat_length_sec) {
maxFrames = static_cast<decltype(maxFrames)>(
bufferParameters.sampleRate() * groupFeatures.beat_length_sec);
if (groupFeatures.has_beat_fraction) {
const auto currentFrame = static_cast<unsigned int>(
const auto currentFrame = static_cast<decltype(maxFrames)>(
maxFrames * groupFeatures.beat_fraction);
uklotzde marked this conversation as resolved.
Show resolved Hide resolved
if (maxFrames > clickSize &&
currentFrame > clickSize &&
Expand All @@ -111,9 +111,9 @@ void MetronomeEffect::processChannel(

if (gs->m_framesSinceClickStart < clickSize) {
// still in click region, write remaining click frames.
const unsigned int copyFrames =
math_min(static_cast<unsigned int>(bufferParameters.framesPerBuffer()),
clickSize - gs->m_framesSinceClickStart);
const SINT copyFrames =
math_min(static_cast<SINT>(bufferParameters.framesPerBuffer()),
uklotzde marked this conversation as resolved.
Show resolved Hide resolved
clickSize - gs->m_framesSinceClickStart);
SampleUtil::addMonoToStereo(pOutput, &click[gs->m_framesSinceClickStart],
copyFrames);
}
Expand Down
2 changes: 1 addition & 1 deletion src/effects/builtin/metronomeeffect.h
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ class MetronomeGroupState final : public EffectState {
~MetronomeGroupState() {
}

unsigned int m_framesSinceClickStart;
SINT m_framesSinceClickStart;
};

class MetronomeEffect : public EffectProcessorImpl<MetronomeGroupState> {
Expand Down
3 changes: 1 addition & 2 deletions src/effects/builtin/parametriceqeffect.h
Original file line number Diff line number Diff line change
Expand Up @@ -61,8 +61,7 @@ class ParametricEQEffect : public EffectProcessorImpl<ParametricEQEffectGroupSta
QList<EngineEffectParameter*> m_pPotQ;
QList<EngineEffectParameter*> m_pPotCenter;


unsigned int m_oldSampleRate;
mixxx::audio::SampleRate m_oldSampleRate;

DISALLOW_COPY_AND_ASSIGN(ParametricEQEffect);
};
Expand Down
2 changes: 1 addition & 1 deletion src/effects/builtin/phasereffect.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,7 @@ void PhaserEffect::processChannel(const ChannelHandle& handle,
const auto stereoCheck = static_cast<int>(m_pStereoParameter->value());
int counter = 0;

for (unsigned int i = 0;
for (SINT i = 0;
i < bufferParameters.samplesPerBuffer();
i += bufferParameters.channelCount()) {
left = pInput[i] + tanh(left * feedback);
Expand Down
2 changes: 1 addition & 1 deletion src/effects/builtin/threebandbiquadeqeffect.h
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ class ThreeBandBiquadEQEffectGroupState final : public EffectState {
double m_loFreqCorner;
double m_highFreqCorner;

unsigned int m_oldSampleRate;
mixxx::audio::SampleRate m_oldSampleRate;
};

class ThreeBandBiquadEQEffect : public EffectProcessorImpl<ThreeBandBiquadEQEffectGroupState> {
Expand Down
2 changes: 1 addition & 1 deletion src/effects/builtin/tremoloeffect.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,7 @@ void TremoloEffect::processChannel(const ChannelHandle& handle,
m_pPhaseParameter->value() * framePerPeriod);
currentFrame = currentFrame % framePerPeriod;

for (unsigned int i = 0;
for (SINT i = 0;
i < bufferParameters.samplesPerBuffer();
i += bufferParameters.channelCount()) {
unsigned int positionFrame = (currentFrame - phaseOffsetFrame);
Expand Down
4 changes: 2 additions & 2 deletions src/sources/soundsourcem4a.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -430,12 +430,12 @@ bool SoundSourceM4A::replaceDecoder(
return false;
}
} else {
if (m_pFaad->Init2(
if (static_cast<signed char>(m_pFaad->Init2(
hNewDecoder,
m_pMP4ESConfigBuffer,
m_sizeofMP4ESConfigBuffer,
&sampleRate,
&channelCount) < 0) {
&channelCount)) < 0) {
free(m_pMP4ESConfigBuffer);
m_pMP4ESConfigBuffer = nullptr;
kLogger.warning() << "Failed to initialize the AAC decoder from "
Expand Down