Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1473)

Unified Diff: webrtc/modules/audio_coding/test/PCMFile.cc

Issue 2291503002: Fix Chromium clang plugin warnings (Closed)
Patch Set: Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/modules/audio_coding/test/PCMFile.h ('k') | webrtc/modules/audio_coding/test/utility.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/audio_coding/test/PCMFile.cc
diff --git a/webrtc/modules/audio_coding/test/PCMFile.cc b/webrtc/modules/audio_coding/test/PCMFile.cc
index 9289d73baa7e044b14fe3e8a03c7f51b8e9e04db..b4acf350055def237584cccd45153536b2114ebe 100644
--- a/webrtc/modules/audio_coding/test/PCMFile.cc
+++ b/webrtc/modules/audio_coding/test/PCMFile.cc
@@ -46,6 +46,12 @@ PCMFile::PCMFile(uint32_t timestamp)
timestamp_ = timestamp;
}
+PCMFile::~PCMFile() {
+ if (pcm_file_) {
+ fclose(pcm_file_);
+ }
+}
+
int16_t PCMFile::ChooseFile(std::string* file_name, int16_t max_len,
uint16_t* frequency_hz) {
char tmp_name[MAX_FILE_NAME_LENGTH_BYTE];
« no previous file with comments | « webrtc/modules/audio_coding/test/PCMFile.h ('k') | webrtc/modules/audio_coding/test/utility.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698