Internal error at D:\a\audacity\audacity\libraries\lib-wave-track WaveClip.cpp line 174.

[JohnColket]

Bug description

Reported by Forum user Mickey_McGinness here: https://forum.audacityteam.org/t/internal-error-message/74855
"using Firefox v 112 on Windows 11, Audacity 3.3.0 and I don’t have a drive D:"

Steps to reproduce

attempting to Amplify one clip

Expected behavior

no error

Actual behavior

No response

Audacity Version

latest stable version (from audacityteam.org/download)

Operating system

Windows

Additional context

No response

[JohnColket]

Similar report "“Internal Error at WaveClip.cpp line 161" on the Forum by SeaofRoses 1/16/23: https://forum.audacityteam.org/t/audacity-error-161/66492

[JohnColket]

Similar report "Internal error at WAveClip.cpp line 161" on the Forum by Wendoglen 11/22 on MacOS: https://forum.audacityteam.org/t/internal-error-at-waveclip-cpp-line-161/65969

[JohnColket]

Similar report "Internal error at[...] waveclip.cpp line xxx" on Discord by 42cat on 6/22/23.

[LWinterberg]

I seems that all of these are talking about this THROW_INCONSISTENCY_EXCEPTION, where the exact line moves between versions due to refactorings and whatnot:

audacity/libraries/lib-wave-track/WaveClip.cpp

Lines 167 to 188 in 43981ed

	std::pair<float, float> WaveClip::GetMinMax(
	double t0, double t1, bool mayThrow) const
	{
	t0 = std::max(t0, GetPlayStartTime());
	t1 = std::min(t1, GetPlayEndTime());
	if (t0 > t1) {
	if (mayThrow)
	THROW_INCONSISTENCY_EXCEPTION;
	return {
	0.f, // harmless, but unused since Sequence::GetMinMax does not use these values
	0.f // harmless, but unused since Sequence::GetMinMax does not use these values
	};
	}
	if (t0 == t1)
	return{ 0.f, 0.f };
	auto s0 = TimeToSequenceSamples(t0);
	auto s1 = TimeToSequenceSamples(t1);
	return mSequence->GetMinMax(s0, s1-s0, mayThrow);
	}