From 09607c278de056f40f00799796d6c2fd0d0b8e47 Mon Sep 17 00:00:00 2001 From: James Crook Date: Sun, 7 May 2017 19:22:44 +0100 Subject: [PATCH] Fix position of wxASSERTs Moved one ASSERT that was in the wrong place and added a new one. This might give a little more information in mac debug builds about Bug 1636 - (Mac) Equalization: Crash selecting the "RIAA" or "Telephone" curves. This is NOT a fix. --- src/effects/Equalization.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/effects/Equalization.cpp b/src/effects/Equalization.cpp index 055ef4c0b..503083377 100644 --- a/src/effects/Equalization.cpp +++ b/src/effects/Equalization.cpp @@ -1613,8 +1613,8 @@ void EffectEqualization::SaveCurves(const wxString &fileName) void EffectEqualization::setCurve(int currentCurve) { // Set current choice - Select(currentCurve); wxASSERT( currentCurve < (int) mCurves.GetCount() ); + Select(currentCurve); Envelope *env; int numPoints = (int) mCurves[currentCurve].points.GetCount(); @@ -2660,6 +2660,7 @@ void EffectEqualization::OnSliderDBMAX(wxCommandEvent & WXUNUSED(event)) void EffectEqualization::OnCurve(wxCommandEvent & WXUNUSED(event)) { // Select NEW curve + wxASSERT( mCurve != NULL ); setCurve( mCurve->GetCurrentSelection() ); if( !mDrawMode ) UpdateGraphic();