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

Unified Diff: webrtc/modules/audio_coding/main/test/APITest.cc

Issue 1353763002: Remove ACM AudioCodingFeedback callback object and derived classes (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@acm-remove-dtmf-api
Patch Set: Created 5 years, 3 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
Index: webrtc/modules/audio_coding/main/test/APITest.cc
diff --git a/webrtc/modules/audio_coding/main/test/APITest.cc b/webrtc/modules/audio_coding/main/test/APITest.cc
index 73f2d2adc746a9ba9b217f1b96b944911e0e4cbe..239fc6bf66214e66b01fca158e9abf7664897c27 100644
--- a/webrtc/modules/audio_coding/main/test/APITest.cc
+++ b/webrtc/modules/audio_coding/main/test/APITest.cc
@@ -86,7 +86,6 @@ APITest::APITest(const Config& config)
_dotMoveDirectionA(1),
_dotPositionB(39),
_dotMoveDirectionB(-1),
- _dtmfCallback(NULL),
_vadCallbackA(NULL),
_vadCallbackB(NULL),
_apiTestRWLock(*RWLockWrapper::CreateRWLock()),
@@ -125,9 +124,7 @@ APITest::~APITest() {
_inFileB.Close();
_outFileB.Close();
- DELETE_POINTER(_dtmfCallback);
DELETE_POINTER(_vadCallbackA);
tlegrand-webrtc 2015/09/18 08:36:55 Why remove DELETE_POINTER(_vadCallbackB);?
hlundin-webrtc 2015/09/18 09:01:13 Oh, bummer. Nice catch! Done.
- DELETE_POINTER(_vadCallbackB);
delete &_apiTestRWLock;
}
@@ -290,9 +287,6 @@ int16_t APITest::SetUp() {
}
}
-#ifdef WEBRTC_DTMF_DETECTION
- _dtmfCallback = new DTMFDetector;
-#endif
_vadCallbackA = new VADCallback;
_vadCallbackB = new VADCallback;
« no previous file with comments | « webrtc/modules/audio_coding/main/test/APITest.h ('k') | webrtc/modules/audio_coding/main/test/TwoWayCommunication.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698