Fixed the clang warning for ApmDataDumper
BUG=
Review-Url: https://codereview.webrtc.org/2296693002
Cr-Commit-Position: refs/heads/master@{#13975}
diff --git a/webrtc/modules/audio_processing/logging/apm_data_dumper.cc b/webrtc/modules/audio_processing/logging/apm_data_dumper.cc
index 491196e..3202006 100644
--- a/webrtc/modules/audio_processing/logging/apm_data_dumper.cc
+++ b/webrtc/modules/audio_processing/logging/apm_data_dumper.cc
@@ -38,6 +38,15 @@
} // namespace
#if WEBRTC_AEC_DEBUG_DUMP == 1
+ApmDataDumper::ApmDataDumper(int instance_index)
+ : instance_index_(instance_index) {}
+#else
+ApmDataDumper::ApmDataDumper(int instance_index) {}
+#endif
+
+ApmDataDumper::~ApmDataDumper() {}
+
+#if WEBRTC_AEC_DEBUG_DUMP == 1
FILE* ApmDataDumper::GetRawFile(const char* name) {
std::string filename =
FormFileName(name, instance_index_, recording_set_index_, ".dat");
diff --git a/webrtc/modules/audio_processing/logging/apm_data_dumper.h b/webrtc/modules/audio_processing/logging/apm_data_dumper.h
index eb3ee88..230c6b3 100644
--- a/webrtc/modules/audio_processing/logging/apm_data_dumper.h
+++ b/webrtc/modules/audio_processing/logging/apm_data_dumper.h
@@ -40,15 +40,12 @@
// Class that handles dumping of variables into files.
class ApmDataDumper {
public:
-// Constructor that takes an instance index that may
-// be used to distinguish data dumped from different
-// instances of the code.
-#if WEBRTC_AEC_DEBUG_DUMP == 1
- explicit ApmDataDumper(int instance_index)
- : instance_index_(instance_index) {}
-#else
- explicit ApmDataDumper(int instance_index) {}
-#endif
+ // Constructor that takes an instance index that may
+ // be used to distinguish data dumped from different
+ // instances of the code.
+ explicit ApmDataDumper(int instance_index);
+
+ ~ApmDataDumper();
// Reinitializes the data dumping such that new versions
// of all files being dumped to are created.
