X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=test%2Faudio_filter_test.cc;h=a258786374e0a22a1a3beea6b50f8017837da967;hb=6a3c03c5eed3cab8fdfdb04fcbaf6cabe8c715e9;hp=0ad91dddeab9bf5ff28fe495f647561f197adbfe;hpb=5a5324ed3a381a86dfe0a6e3932c1d58fdcd596f;p=dcpomatic.git

diff --git a/test/audio_filter_test.cc b/test/audio_filter_test.cc
index 0ad91ddde..a25878637 100644
--- a/test/audio_filter_test.cc
+++ b/test/audio_filter_test.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2014 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2014-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -18,17 +18,21 @@
 
 */
 
+
 /** @file  test/audio_filter_test.cc
- *  @brief Basic tests of audio filters.
+ *  @brief Test AudioFilter, LowPassAudioFilter, HighPassAudioFilter classes.
+ *  @ingroup selfcontained
  */
 
+
+#include <boost/test/unit_test.hpp>
 #include "lib/audio_filter.h"
 #include "lib/audio_buffers.h"
-#include <boost/test/unit_test.hpp>
-#include <boost/make_shared.hpp>
 
-using boost::shared_ptr;
-using boost::make_shared;
+
+using std::make_shared;
+using std::shared_ptr;
+
 
 static void
 audio_filter_impulse_test_one (AudioFilter& f, int block_size, int num_blocks)
@@ -37,12 +41,12 @@ audio_filter_impulse_test_one (AudioFilter& f, int block_size, int num_blocks)
 
 	for (int i = 0; i < num_blocks; ++i) {
 
-		shared_ptr<AudioBuffers> in = make_shared<AudioBuffers> (1, block_size);
+		auto in = make_shared<AudioBuffers>(1, block_size);
 		for (int j = 0; j < block_size; ++j) {
 			in->data()[0][j] = c + j;
 		}
 
-		shared_ptr<AudioBuffers> out = f.run (in);
+		auto out = f.run (in);
 
 		for (int j = 0; j < out->frames(); ++j) {
 			BOOST_CHECK_EQUAL (out->data()[0][j], c + j);
@@ -52,15 +56,15 @@ audio_filter_impulse_test_one (AudioFilter& f, int block_size, int num_blocks)
 	}
 }
 
+
 /** Create a filter with an impulse as a kernel and check that it
  *  passes data through unaltered.
  */
 BOOST_AUTO_TEST_CASE (audio_filter_impulse_kernel_test)
 {
 	AudioFilter f (0.02);
-	delete[] f._ir;
-	f._ir = new float[f._M + 1];
 
+	f._ir.reserve(f._M + 1);
 	f._ir[0] = 1;
 	for (int i = 1; i <= f._M; ++i) {
 		f._ir[i] = 0;
@@ -71,6 +75,7 @@ BOOST_AUTO_TEST_CASE (audio_filter_impulse_kernel_test)
 	audio_filter_impulse_test_one (f, 2048, 1);
 }
 
+
 /** Create filters and pass them impulses as input and check that
  *  the filter kernels comes back.
  */
@@ -78,11 +83,11 @@ BOOST_AUTO_TEST_CASE (audio_filter_impulse_input_test)
 {
 	LowPassAudioFilter lpf (0.02, 0.3);
 
-	shared_ptr<AudioBuffers> in = make_shared<AudioBuffers> (1, 1751);
+	auto in = make_shared<AudioBuffers>(1, 1751);
 	in->make_silent ();
 	in->data(0)[0] = 1;
 
-	shared_ptr<AudioBuffers> out = lpf.run (in);
+	auto out = lpf.run (in);
 	for (int j = 0; j < out->frames(); ++j) {
 		if (j <= lpf._M) {
 			BOOST_CHECK_EQUAL (out->data(0)[j], lpf._ir[j]);
@@ -93,7 +98,7 @@ BOOST_AUTO_TEST_CASE (audio_filter_impulse_input_test)
 
 	HighPassAudioFilter hpf (0.02, 0.3);
 
-	in.reset (new AudioBuffers (1, 9133));
+	in = make_shared<AudioBuffers>(1, 9133);
 	in->make_silent ();
 	in->data(0)[0] = 1;