Conversational Speech tool, WavReaderAdaptor and unit test

webrtc/modules/audio_processing/test/conversational_speech/wavreader_factory.cc

Index: webrtc/modules/audio_processing/test/conversational_speech/wavreader_factory.cc
diff --git a/webrtc/modules/audio_processing/test/conversational_speech/wavreader_factory.cc b/webrtc/modules/audio_processing/test/conversational_speech/wavreader_factory.cc
index f9d43f80702b53876e42e6a1146f5d1dacc0e190..d6665369630fa21c66826758f127be75c5f6b5e5 100644
--- a/webrtc/modules/audio_processing/test/conversational_speech/wavreader_factory.cc
+++ b/webrtc/modules/audio_processing/test/conversational_speech/wavreader_factory.cc
@@ -22,7 +22,7 @@ WavReaderFactory::~WavReaderFactory() = default;
 
 std::unique_ptr<WavReaderInterface> WavReaderFactory::Create(
     const std::string& filepath) const {
-  return std::unique_ptr<WavReaderAdaptor>(new WavReaderAdaptor(filepath));
+  return CreateWavReaderAdaptor(filepath);
 }

[kwiberg-webrtc, 2017/03/29 09:08:28]

Hmm. Can you eliminate either WavReaderFactory::Create() or
CreateWavReaderAdaptor()? They seem to do exactly the same thing...

[AleBzk, 2017/03/29 09:34:43]

Right. I can move everything into the factory.

 
 }  // namespace conversational_speech