Pass containers by ref to import plugin factories, use STL idiom

src/import/Import.cpp

@@ -101,20 +101,20 @@ bool Importer::Initialize()
    // build the list of import plugin and/or unusableImporters.
    // order is significant.  If none match, they will all be tried
    // in the order defined here.
-   GetPCMImportPlugin(mImportPluginList, mUnusableImportPluginList);
-   GetOGGImportPlugin(mImportPluginList, mUnusableImportPluginList);
-   GetFLACImportPlugin(mImportPluginList, mUnusableImportPluginList);
-   GetMP3ImportPlugin(mImportPluginList, mUnusableImportPluginList);
-   GetLOFImportPlugin(mImportPluginList, mUnusableImportPluginList);
+   GetPCMImportPlugin(*mImportPluginList, *mUnusableImportPluginList);
+   GetOGGImportPlugin(*mImportPluginList, *mUnusableImportPluginList);
+   GetFLACImportPlugin(*mImportPluginList, *mUnusableImportPluginList);
+   GetMP3ImportPlugin(*mImportPluginList, *mUnusableImportPluginList);
+   GetLOFImportPlugin(*mImportPluginList, *mUnusableImportPluginList);
 
    #if defined(USE_FFMPEG)
-   GetFFmpegImportPlugin(mImportPluginList, mUnusableImportPluginList);
+   GetFFmpegImportPlugin(*mImportPluginList, *mUnusableImportPluginList);
    #endif
    #ifdef USE_QUICKTIME
-   GetQTImportPlugin(mImportPluginList, mUnusableImportPluginList);
+   GetQTImportPlugin(*mImportPluginList, *mUnusableImportPluginList);
    #endif
    #if defined(USE_GSTREAMER)
-   GetGStreamerImportPlugin(mImportPluginList, mUnusableImportPluginList);
+   GetGStreamerImportPlugin(*mImportPluginList, *mUnusableImportPluginList);
    #endif
 
    ReadImportItems();

src/import/ImportFFmpeg.cpp

@@ -283,10 +283,10 @@ private:
 };
 
 
-void GetFFmpegImportPlugin(ImportPluginList *importPluginList,
-                           UnusableImportPluginList *WXUNUSED(unusableImportPluginList))
+void GetFFmpegImportPlugin(ImportPluginList &importPluginList,
+                           UnusableImportPluginList &WXUNUSED(unusableImportPluginList))
 {
-   importPluginList->Append(new FFmpegImportPlugin);
+   importPluginList.push_back( new FFmpegImportPlugin );
 }
 
 

src/import/ImportFFmpeg.h

@@ -13,7 +13,7 @@ LRN
 
 #include "ImportForwards.h"
 
-void GetFFmpegImportPlugin(ImportPluginList *importPluginList,
-                           UnusableImportPluginList *unusableImportPluginList);
+void GetFFmpegImportPlugin(ImportPluginList &importPluginList,
+                           UnusableImportPluginList &unusableImportPluginList);
 
 #endif

src/import/ImportFLAC.cpp

@@ -56,13 +56,13 @@ static const wxChar *exts[] =
 
 #ifndef USE_LIBFLAC
 
-void GetFLACImportPlugin(ImportPluginList *importPluginList,
-                        UnusableImportPluginList *unusableImportPluginList)
+void GetFLACImportPlugin(ImportPluginList &importPluginList,
+                        UnusableImportPluginList &unusableImportPluginList)
 {
    UnusableImportPlugin* flacIsUnsupported =
       new UnusableImportPlugin(DESC, wxArrayString(WXSIZEOF(exts), exts));
 
-   unusableImportPluginList->Append(flacIsUnsupported);
+   unusableImportPluginList.push_back( flacIsUnsupported );
 }
 
 #else /* USE_LIBFLAC */
@@ -282,10 +282,10 @@ FLAC__StreamDecoderWriteStatus MyFLACFile::write_callback(const FLAC__Frame *fra
 }
 
 
-void GetFLACImportPlugin(ImportPluginList *importPluginList,
-                         UnusableImportPluginList *WXUNUSED(unusableImportPluginList))
+void GetFLACImportPlugin(ImportPluginList &importPluginList,
+                         UnusableImportPluginList &WXUNUSED(unusableImportPluginList))
 {
-   importPluginList->Append(new FLACImportPlugin);
+   importPluginList.push_back( new FLACImportPlugin );
 }
 
 

src/import/ImportFLAC.h

@@ -13,8 +13,8 @@
 
 #include "ImportForwards.h"
 
-void GetFLACImportPlugin(ImportPluginList *importPluginList,
-                         UnusableImportPluginList *unusableImportPluginList);
+void GetFLACImportPlugin(ImportPluginList &importPluginList,
+                         UnusableImportPluginList &unusableImportPluginList);
 
 
 #endif

src/import/ImportGStreamer.cpp

@@ -261,8 +261,8 @@ public:
 // ----------------------------------------------------------------------------
 // Instantiate GStreamerImportPlugin and add to the list of known importers
 void
-GetGStreamerImportPlugin(ImportPluginList *importPluginList,
-                         UnusableImportPluginList * WXUNUSED(unusableImportPluginList))
+GetGStreamerImportPlugin(ImportPluginList &importPluginList,
+                         UnusableImportPluginList & WXUNUSED(unusableImportPluginList))
 {
    wxLogMessage(wxT("Audacity is built against GStreamer version %d.%d.%d-%d"),
                 GST_VERSION_MAJOR,
@@ -300,7 +300,7 @@ GetGStreamerImportPlugin(ImportPluginList *importPluginList,
       return;
 
    // Add to list of importers
-   importPluginList->push_back(std::move(plug));
+   importPluginList.push_back( plug.release() );
 }
 
 // ============================================================================

src/import/ImportGStreamer.h

@@ -13,7 +13,7 @@ LRN
 
 #include "ImportPlugin.h"
 
-void GetGStreamerImportPlugin(ImportPluginList *importPluginList,
-                           UnusableImportPluginList *unusableImportPluginList);
+void GetGStreamerImportPlugin(ImportPluginList &importPluginList,
+                           UnusableImportPluginList &unusableImportPluginList);
 
 #endif

src/import/ImportLOF.cpp

@@ -175,10 +175,10 @@ LOFImportFileHandle::LOFImportFileHandle
    scrollOffset = 0;
 }
 
-void GetLOFImportPlugin(ImportPluginList *importPluginList,
-                        UnusableImportPluginList * WXUNUSED(unusableImportPluginList))
+void GetLOFImportPlugin(ImportPluginList &importPluginList,
+                        UnusableImportPluginList & WXUNUSED(unusableImportPluginList))
 {
-   importPluginList->Append(new LOFImportPlugin);
+   importPluginList.push_back( new LOFImportPlugin );
 }
 
 wxString LOFImportPlugin::GetPluginFormatDescription()

src/import/ImportLOF.h

@@ -56,7 +56,7 @@
 
 #include "ImportForwards.h"
 
-void GetLOFImportPlugin(ImportPluginList *importPluginList,
-                        UnusableImportPluginList *unusableImportPluginList);
+void GetLOFImportPlugin(ImportPluginList &importPluginList,
+                        UnusableImportPluginList &unusableImportPluginList);
 
 #endif

src/import/ImportMP3.cpp

@@ -56,13 +56,13 @@ static const wxChar *exts[] =
 
 #ifndef USE_LIBMAD
 
-void GetMP3ImportPlugin(ImportPluginList *importPluginList,
-                        UnusableImportPluginList *unusableImportPluginList)
+void GetMP3ImportPlugin(ImportPluginList &importPluginList,
+                        UnusableImportPluginList &unusableImportPluginList)
 {
    UnusableImportPlugin* mp3IsUnsupported =
       new UnusableImportPlugin(DESC, wxArrayString(WXSIZEOF(exts), exts));
 
-   unusableImportPluginList->Append(mp3IsUnsupported);
+   unusableImportPluginList.push_back( mp3IsUnsupported );
 }
 
 #else /* USE_LIBMAD */
@@ -153,10 +153,10 @@ private:
    mad_decoder mDecoder;
 };
 
-void GetMP3ImportPlugin(ImportPluginList *importPluginList,
-                        UnusableImportPluginList * WXUNUSED(unusableImportPluginList))
+void GetMP3ImportPlugin(ImportPluginList &importPluginList,
+                        UnusableImportPluginList & WXUNUSED(unusableImportPluginList))
 {
-   importPluginList->Append(new MP3ImportPlugin);
+   importPluginList.push_back( new MP3ImportPlugin );
 }
 
 /* The MAD callbacks */

src/import/ImportMP3.h

@@ -13,7 +13,7 @@
 
 #include "ImportForwards.h"
 
-void GetMP3ImportPlugin(ImportPluginList *importPluginList,
-                        UnusableImportPluginList *unusableImportPluginList);
+void GetMP3ImportPlugin(ImportPluginList &importPluginList,
+                        UnusableImportPluginList &unusableImportPluginList);
 
 #endif

src/import/ImportOGG.cpp

@@ -55,13 +55,13 @@ static const wxChar *exts[] =
 /* BPF There is no real reason to compile without LIBVORBIS, but if you do, you will needs this header */
 #include "ImportPlugin.h"
 
-void GetOGGImportPlugin(ImportPluginList *importPluginList,
-                        UnusableImportPluginList *unusableImportPluginList)
+void GetOGGImportPlugin(ImportPluginList &importPluginList,
+                        UnusableImportPluginList &unusableImportPluginList)
 {
    UnusableImportPlugin* oggIsUnsupported =
       new UnusableImportPlugin(DESC, wxArrayString(WXSIZEOF(exts), exts));
 
-   unusableImportPluginList->Append(oggIsUnsupported);
+   unusableImportPluginList.push_back( oggIsUnsupported );
 }
 
 #else /* USE_LIBVORBIS */
@@ -160,10 +160,10 @@ private:
    sampleFormat   mFormat;
 };
 
-void GetOGGImportPlugin(ImportPluginList *importPluginList,
-                        UnusableImportPluginList * WXUNUSED(unusableImportPluginList))
+void GetOGGImportPlugin(ImportPluginList &importPluginList,
+                        UnusableImportPluginList & WXUNUSED(unusableImportPluginList))
 {
-   importPluginList->Append(new OggImportPlugin);
+   importPluginList.push_back( new OggImportPlugin );
 }
 
 wxString OggImportPlugin::GetPluginFormatDescription()

src/import/ImportOGG.h

@@ -13,7 +13,7 @@
 
 #include "ImportForwards.h"
 
-void GetOGGImportPlugin(ImportPluginList *importPluginList,
-                        UnusableImportPluginList *unusableImportPluginList);
+void GetOGGImportPlugin(ImportPluginList &importPluginList,
+                        UnusableImportPluginList &unusableImportPluginList);
 
 #endif

src/import/ImportPCM.cpp

@@ -113,10 +113,10 @@ private:
    sampleFormat          mFormat;
 };
 
-void GetPCMImportPlugin(ImportPluginList * importPluginList,
-                        UnusableImportPluginList * WXUNUSED(unusableImportPluginList))
+void GetPCMImportPlugin(ImportPluginList & importPluginList,
+                        UnusableImportPluginList & WXUNUSED(unusableImportPluginList))
 {
-   importPluginList->Append(new PCMImportPlugin);
+   importPluginList.push_back( new PCMImportPlugin );
 }
 
 wxString PCMImportPlugin::GetPluginFormatDescription()

src/import/ImportPCM.h

@@ -13,8 +13,8 @@
 
 #include "ImportForwards.h"
 
-void GetPCMImportPlugin(ImportPluginList *importPluginList,
-                        UnusableImportPluginList *unusableImportPluginList);
+void GetPCMImportPlugin(ImportPluginList &importPluginList,
+                        UnusableImportPluginList &unusableImportPluginList);
 
 
 #endif

src/import/ImportQT.cpp

@@ -27,13 +27,13 @@ static const wxChar *exts[] =
 
 #ifndef USE_QUICKTIME
 
-void GetQTImportPlugin(ImportPluginList *importPluginList,
-                       UnusableImportPluginList *unusableImportPluginList)
+void GetQTImportPlugin(ImportPluginList &importPluginList,
+                       UnusableImportPluginList &unusableImportPluginList)
 {
    UnusableImportPlugin* qtIsUnsupported =
       new UnusableImportPlugin(DESC, wxArrayString(WXSIZEOF(exts), exts));
 
-   unusableImportPluginList->Append(qtIsUnsupported);
+   unusableImportPluginList.push_back( qtIsUnsupported );
 }
 
 #else /* USE_QUICKTIME */
@@ -162,10 +162,10 @@ class QTImportFileHandle final : public ImportFileHandle
    Movie mMovie;
 };
 
-void GetQTImportPlugin(ImportPluginList *importPluginList,
-                       UnusableImportPluginList *unusableImportPluginList)
+void GetQTImportPlugin(ImportPluginList &importPluginList,
+                       UnusableImportPluginList &unusableImportPluginList)
 {
-   importPluginList->Append(new QTImportPlugin);
+   importPluginList.push_back( new QTImportPlugin );
 }
 
 wxString QTImportPlugin::GetPluginFormatDescription()

src/import/ImportQT.h

@@ -13,7 +13,7 @@
 
 #include "ImportForwards.h"
 
-void GetQTImportPlugin(ImportPluginList *importPluginList,
-                       UnusableImportPluginList *unusableImportPluginList);
+void GetQTImportPlugin(ImportPluginList &importPluginList,
+                       UnusableImportPluginList &unusableImportPluginList);
 
 #endif