Don't use wxArrayPtrVoid with casts, use std::vector

src/effects/Effect.cpp

@@ -2046,8 +2046,8 @@ void Effect::CopyInputTracks()
 void Effect::CopyInputTracks(int trackType)
 {
    // Reset map
-   mIMap.Clear();
+   mIMap.clear();
-   mOMap.Clear();
+   mOMap.clear();
 
    mOutputTracks = std::make_unique<TrackList>();
    mOutputTracksType = trackType;
@@ -2063,16 +2063,16 @@ void Effect::CopyInputTracks(int trackType)
             (trackType == Track::All && aTrack->IsSyncLockSelected()))
       {
          Track *o = mOutputTracks->Add(aTrack->Duplicate());
-         mIMap.Add(aTrack);
+         mIMap.push_back(aTrack);
-         mOMap.Add(o);
+         mOMap.push_back(o);
       }
    }
 }
 
 Track *Effect::AddToOutputTracks(std::unique_ptr<Track> &&t)
 {
-   mIMap.Add(NULL);
+   mIMap.push_back(NULL);
-   mOMap.Add(t.get());
+   mOMap.push_back(t.get());
    return mOutputTracks->Add(std::move(t));
 }
 
@@ -2178,8 +2178,8 @@ void Effect::ReplaceProcessedTracks(const bool bGoodResult)
       mOutputTracks->Clear();
 
       // Reset map
-      mIMap.Clear();
+      mIMap.clear();
-      mOMap.Clear();
+      mOMap.clear();
 
       //TODO:undo the non-gui ODTask transfer
       return;
@@ -2187,7 +2187,7 @@ void Effect::ReplaceProcessedTracks(const bool bGoodResult)
 
    auto iterOut = mOutputTracks->begin(), iterEnd = mOutputTracks->end();
 
-   size_t cnt = mOMap.GetCount();
+   size_t cnt = mOMap.size();
    size_t i = 0;
 
    for (; iterOut != iterEnd; ++i) {
@@ -2195,7 +2195,7 @@ void Effect::ReplaceProcessedTracks(const bool bGoodResult)
       // If tracks were removed from mOutputTracks, then there will be
       // tracks in the map that must be removed from mTracks.
       while (i < cnt && mOMap[i] != o.get()) {
-         Track *t = (Track *) mIMap[i];
+         const auto t = mIMap[i];
          if (t) {
             mTracks->Remove(t);
          }
@@ -2208,7 +2208,7 @@ void Effect::ReplaceProcessedTracks(const bool bGoodResult)
       // Remove the track from the output list...don't DELETE it
       iterOut = mOutputTracks->erase(iterOut);
 
-      Track *t = (Track *) mIMap[i];
+      const auto  t = mIMap[i];
       if (t == NULL)
       {
          // This track is a NEW addition to output tracks; add it to mTracks
@@ -2232,16 +2232,16 @@ void Effect::ReplaceProcessedTracks(const bool bGoodResult)
    // If tracks were removed from mOutputTracks, then there may be tracks
    // left at the end of the map that must be removed from mTracks.
    while (i < cnt) {
-      Track *t = (Track *) mIMap[i];
+      const auto t = mIMap[i];
       if (t) {
-         mTracks->Remove((Track *)mIMap[i]);
+         mTracks->Remove(t);
       }
       i++;
    }
 
    // Reset map
-   mIMap.Clear();
+   mIMap.clear();
-   mOMap.Clear();
+   mOMap.clear();
 
    // Make sure we processed everything
    wxASSERT(mOutputTracks->empty());

src/effects/Effect.h

@@ -494,8 +494,8 @@ private:
 
    bool mUIDebug;
 
-   wxArrayPtrVoid mIMap;
+   std::vector<Track*> mIMap;
-   wxArrayPtrVoid mOMap;
+   std::vector<Track*> mOMap;
 
    int mNumTracks; //v This is really mNumWaveTracks, per CountWaveTracks() and GetNumWaveTracks().
    int mNumGroups;

src/effects/Equalization48x.cpp

@@ -329,10 +329,11 @@ bool EffectEqualization48x::TrackCompare()
       mEffectEqualization->mM=(mEffectEqualization->mM&(~15))+1;
    AllocateBuffersWorkers(sMathPath&MATH_FUNCTION_THREADED);
    // Reset map
-   wxArrayPtrVoid SecondIMap;
+   // PRL:  These two maps aren't really used
-   wxArrayPtrVoid SecondOMap;
+   std::vector<Track*> SecondIMap;
-   SecondIMap.Clear();
+   std::vector<Track*> SecondOMap;
-   SecondOMap.Clear();
+   SecondIMap.clear();
+   SecondOMap.clear();
    
    TrackList      SecondOutputTracks;
 
@@ -346,8 +347,8 @@ bool EffectEqualization48x::TrackCompare()
          (mEffectEqualization->mOutputTracksType == Track::All && aTrack->IsSyncLockSelected()))
       {
          auto o = aTrack->Duplicate();
-         SecondIMap.Add(aTrack);
+         SecondIMap.push_back(aTrack);
-         SecondIMap.Add(o.get());
+         SecondIMap.push_back(o.get());
          SecondOutputTracks.Add(std::move(o));
       }
    }

src/export/ExportMultiple.cpp

@@ -925,9 +925,8 @@ int ExportMultiple::ExportMultipleByTrack(bool byName,
    }
 
    // Restore the selection states
-   for (size_t i = 0; i < mSelected.GetCount(); i++) {
+   for (auto pTrack : mSelected)
-      selected[i]->SetSelected(true);
+      pTrack->SetSelected(true);
-   }
 
    return ok ;
 }

src/export/ExportMultiple.h

@@ -29,6 +29,7 @@ class wxTextCtrl;
 class AudacityProject;
 class LabelTrack;
 class ShuttleGui;
+class Track;
 
 class ExportMultiple final : public wxDialogWrapper
 {
@@ -111,7 +112,10 @@ private:
    LabelTrack *mLabels;
    int mNumLabels;
    int mNumWaveTracks;
-   wxArrayPtrVoid mSelected;
+
+   // PRL:  This is never populated anywhere?
+   std::vector<Track*> mSelected;
+
    int mFilterIndex;          /**< The index in the drop-down list of export
                                 formats (mFormat) of the selected export format.
                                 This list includes all possible

src/toolbars/ToolDock.h

@@ -19,7 +19,6 @@
 
 #include "ToolBar.h"
 
-class wxArrayPtrVoid;
 class wxCommandEvent;
 class wxEraseEvent;
 class wxSizeEvent;

src/widgets/FileHistory.cpp

@@ -100,19 +100,27 @@ size_t FileHistory::GetCount()
 
 void FileHistory::UseMenu(wxMenu *menu)
 {
-   wxASSERT(mMenus.Index(menu) == wxNOT_FOUND);
+   auto end = mMenus.end();
+   auto iter = std::find(mMenus.begin(), end, menu);
+   auto found = (iter != end);
 
-   if (mMenus.Index(menu) == wxNOT_FOUND) {
+   if (!found)
-      mMenus.Add(menu);
+      mMenus.push_back(menu);
+   else {
+      wxASSERT(false);
    }
 }
 
 void FileHistory::RemoveMenu(wxMenu *menu)
 {
-   wxASSERT(mMenus.Index(menu) != wxNOT_FOUND);
+   auto end = mMenus.end();
+   auto iter = std::find(mMenus.begin(), end, menu);
+   auto found = (iter != end);
 
-   if (mMenus.Index(menu) != wxNOT_FOUND) {
+   if (found)
-      mMenus.Remove(menu);
+      mMenus.erase(iter);
+   else {
+      wxASSERT(false);
    }
 }
 
@@ -151,9 +159,8 @@ void FileHistory::Save(wxConfigBase & config, const wxString & group)
 
 void FileHistory::AddFilesToMenu()
 {
-   for (size_t i = 0; i < mMenus.GetCount(); i++) {
+   for (auto pMenu : mMenus)
-      AddFilesToMenu((wxMenu *) mMenus[i]);
+      AddFilesToMenu(pMenu);
-   }
 }
 
 void FileHistory::AddFilesToMenu(wxMenu *menu)

src/widgets/FileHistory.h

@@ -11,6 +11,7 @@
 #ifndef __AUDACITY_WIDGETS_FILEHISTORY__
 #define __AUDACITY_WIDGETS_FILEHISTORY__
 
+#include <vector>
 #include <wx/defs.h>
 #include <wx/choice.h>
 #include <wx/dynarray.h>
@@ -43,7 +44,7 @@ class AUDACITY_DLL_API FileHistory
    size_t mMaxFiles;
    wxWindowID mIDBase;
 
-   wxArrayPtrVoid mMenus;
+   std::vector<wxMenu*> mMenus;
    wxArrayString mHistory;
 
 };