Replace some raw arrays with std::vectors.

6 files changed, 18 insertions, 24 deletions

diff --git a/src/lib/cross_windows.cc b/src/lib/cross_windows.cc
index a7e7bdd55..f76380836 100644
--- a/src/lib/cross_windows.cc
+++ b/src/lib/cross_windows.cc
@@ -353,10 +353,9 @@ static string
 wchar_to_utf8 (wchar_t const * s)
 {
 	int const length = (wcslen(s) + 1) * 2;
-	char* utf8 = new char[length];
-	WideCharToMultiByte (CP_UTF8, 0, s, -1, utf8, length, 0, 0);
-	string u (utf8);
-	delete[] utf8;
+	std::vector<char> utf8(length);
+	WideCharToMultiByte (CP_UTF8, 0, s, -1, utf8.data(), length, 0, 0);
+	string u (utf8.data());
 	return u;
 }
 
diff --git a/src/lib/ffmpeg_encoder.cc b/src/lib/ffmpeg_encoder.cc
index 8f36bbda1..d4f0b4b47 100644
--- a/src/lib/ffmpeg_encoder.cc
+++ b/src/lib/ffmpeg_encoder.cc
@@ -161,7 +161,7 @@ FFmpegEncoder::go ()
 
 	auto const video_frame = DCPTime::from_frames (1, _film->video_frame_rate ());
 	int const audio_frames = video_frame.frames_round(_film->audio_frame_rate());
-	float* interleaved = new float[_output_audio_channels * audio_frames];
+	std::vector<float> interleaved(_output_audio_channels * audio_frames);
 	auto deinterleaved = make_shared<AudioBuffers>(_output_audio_channels, audio_frames);
 	int const gets_per_frame = _film->three_d() ? 2 : 1;
 	for (DCPTime i; i < _film->length(); i += video_frame) {
@@ -204,9 +204,9 @@ FFmpegEncoder::go ()
 
 		waker.nudge ();
 
-		_butler->get_audio (Butler::Behaviour::BLOCKING, interleaved, audio_frames);
+		_butler->get_audio (Butler::Behaviour::BLOCKING, interleaved.data(), audio_frames);
 		/* XXX: inefficient; butler interleaves and we deinterleave again */
-		float* p = interleaved;
+		float* p = interleaved.data();
 		for (int j = 0; j < audio_frames; ++j) {
 			for (int k = 0; k < _output_audio_channels; ++k) {
 				deinterleaved->data(k)[j] = *p++;
@@ -214,7 +214,6 @@ FFmpegEncoder::go ()
 		}
 		encoder->audio (deinterleaved);
 	}
-	delete[] interleaved;
 
 	for (auto i: file_encoders) {
 		i.flush ();
diff --git a/src/lib/file_log.cc b/src/lib/file_log.cc
index 4d6a0e6ea..a9522bad5 100644
--- a/src/lib/file_log.cc
+++ b/src/lib/file_log.cc
@@ -87,23 +87,22 @@ FileLog::head_and_tail (int amount) const
 
 	string out;
 
-	auto buffer = new char[max(head_amount, tail_amount) + 1];
+	std::vector<char> buffer(max(head_amount, tail_amount) + 1);
 
-	int N = fread (buffer, 1, head_amount, f);
+	int N = fread (buffer.data(), 1, head_amount, f);
 	buffer[N] = '\0';
-	out += string (buffer);
+	out += string (buffer.data());
 
 	if (tail_amount > 0) {
 		out +=  "\n .\n .\n .\n";
 
 		fseek (f, - tail_amount - 1, SEEK_END);
 
-		N = fread (buffer, 1, tail_amount, f);
+		N = fread (buffer.data(), 1, tail_amount, f);
 		buffer[N] = '\0';
-		out += string (buffer) + "\n";
+		out += string (buffer.data()) + "\n";
 	}
 
-	delete[] buffer;
 	fclose (f);
 
 	return out;
diff --git a/src/lib/image_examiner.cc b/src/lib/image_examiner.cc
index 2235c2e69..5a1672204 100644
--- a/src/lib/image_examiner.cc
+++ b/src/lib/image_examiner.cc
@@ -55,16 +55,14 @@ ImageExaminer::ImageExaminer (shared_ptr<const Film> film, shared_ptr<const Imag
 		if (!f) {
 			throw FileError ("Could not open file for reading", path);
 		}
-		auto buffer = new uint8_t[size];
-		checked_fread (buffer, size, f, path);
+		std::vector<uint8_t> buffer(size);
+		checked_fread (buffer.data(), size, f, path);
 		fclose (f);
 		try {
-			_video_size = dcp::decompress_j2k (buffer, size, 0)->size ();
+			_video_size = dcp::decompress_j2k(buffer.data(), size, 0)->size();
 		} catch (dcp::ReadError& e) {
-			delete[] buffer;
 			throw DecodeError (String::compose (_("Could not decode JPEG2000 file %1 (%2)"), path, e.what ()));
 		}
-		delete[] buffer;
 	} else {
 		FFmpegImageProxy proxy(content->path(0));
 		_video_size = proxy.image(Image::Alignment::COMPACT).image->size();
diff --git a/src/lib/update_checker.cc b/src/lib/update_checker.cc
index 3bc02b50a..2c2e23be1 100644
--- a/src/lib/update_checker.cc
+++ b/src/lib/update_checker.cc
@@ -57,7 +57,7 @@ write_callback_wrapper (void* data, size_t size, size_t nmemb, void* user)
  *  do the work.
  */
 UpdateChecker::UpdateChecker ()
-	: _buffer (new char[BUFFER_SIZE])
+	: _buffer (BUFFER_SIZE)
 	, _state (State::NOT_RUN)
 {
 	_curl = curl_easy_init ();
@@ -97,7 +97,6 @@ UpdateChecker::~UpdateChecker ()
 	} catch (...) {}
 
 	curl_easy_cleanup (_curl);
-	delete[] _buffer;
 }
 
 
@@ -142,7 +141,7 @@ UpdateChecker::thread ()
 			/* Parse the reply */
 
 			_buffer[_offset] = '\0';
-			string s (_buffer);
+			string s (_buffer.data());
 			cxml::Document doc ("Update");
 			doc.read_string (s);
 
@@ -181,7 +180,7 @@ size_t
 UpdateChecker::write_callback (void* data, size_t size, size_t nmemb)
 {
 	size_t const t = min (size * nmemb, size_t (BUFFER_SIZE - _offset - 1));
-	memcpy (_buffer + _offset, data, t);
+	memcpy (_buffer.data() + _offset, data, t);
 	_offset += t;
 	return t;
 }
diff --git a/src/lib/update_checker.h b/src/lib/update_checker.h
index aa5f620e6..53ef4457b 100644
--- a/src/lib/update_checker.h
+++ b/src/lib/update_checker.h
@@ -89,7 +89,7 @@ private:
 	void set_state (State);
 	void thread ();
 
-	char* _buffer;
+	std::vector<char> _buffer;
 	int _offset = 0;
 	CURL* _curl = nullptr;