Tidying.

1 files changed, 24 insertions, 24 deletions

diff --git a/src/mono_picture_frame.cc b/src/mono_picture_frame.cc
index 7420193e..3e27bed7 100644
--- a/src/mono_picture_frame.cc
+++ b/src/mono_picture_frame.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012-2014 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of libdcp.
 
@@ -31,10 +31,12 @@
     files in the program, then also delete it here.
 */
 
+
 /** @file  src/mono_picture_frame.cc
- *  @brief MonoPictureFrame class.
+ *  @brief MonoPictureFrame class
  */
 
+
 #include "mono_picture_frame.h"
 #include "exceptions.h"
 #include "util.h"
@@ -46,33 +48,34 @@
 #include <asdcp/KM_fileio.h>
 #include <asdcp/AS_DCP.h>
 
+
+using std::make_shared;
 using std::string;
 using std::shared_ptr;
 using boost::optional;
 using namespace dcp;
 
-/** Make a picture frame from a JPEG2000 file.
- *  @param path Path to JPEG2000 file.
- */
+
 MonoPictureFrame::MonoPictureFrame (boost::filesystem::path path)
 {
-	boost::uintmax_t const size = boost::filesystem::file_size (path);
+	auto const size = boost::filesystem::file_size (path);
 	_buffer.reset(new ASDCP::JP2K::FrameBuffer(size));
-	FILE* f = fopen_boost (path, "rb");
+	auto f = fopen_boost (path, "rb");
 	if (!f) {
-		boost::throw_exception (FileError ("could not open JPEG2000 file", path, errno));
+		boost::throw_exception (FileError("could not open JPEG2000 file", path, errno));
 	}
 
 	size_t n = fread (data(), 1, size, f);
+	fclose (f);
+
 	if (n != size) {
-		boost::throw_exception (FileError ("could not read from JPEG2000 file", path, errno));
+		boost::throw_exception (FileError("could not read from JPEG2000 file", path, errno));
 	}
 
-	fclose (f);
-
 	_buffer->Size (size);
 }
 
+
 /** Make a picture frame from a 2D (monoscopic) asset.
  *  @param reader Reader for the asset's MXF file.
  *  @param n Frame within the asset, not taking EntryPoint into account.
@@ -81,50 +84,47 @@ MonoPictureFrame::MonoPictureFrame (boost::filesystem::path path)
 MonoPictureFrame::MonoPictureFrame (ASDCP::JP2K::MXFReader* reader, int n, shared_ptr<DecryptionContext> c)
 {
 	/* XXX: unfortunate guesswork on this buffer size */
-	_buffer.reset(new ASDCP::JP2K::FrameBuffer(4 * Kumu::Megabyte));
+	_buffer = make_shared<ASDCP::JP2K::FrameBuffer>(4 * Kumu::Megabyte);
 
-	ASDCP::Result_t const r = reader->ReadFrame (n, *_buffer, c->context(), c->hmac());
+	auto const r = reader->ReadFrame (n, *_buffer, c->context(), c->hmac());
 
-	if (ASDCP_FAILURE (r)) {
-		boost::throw_exception (ReadError (String::compose ("could not read video frame %1 (%2)", n, static_cast<int>(r))));
+	if (ASDCP_FAILURE(r)) {
+		boost::throw_exception (ReadError(String::compose ("could not read video frame %1 (%2)", n, static_cast<int>(r))));
 	}
 }
 
+
 MonoPictureFrame::MonoPictureFrame (uint8_t const * data, int size)
 {
-	_buffer.reset(new ASDCP::JP2K::FrameBuffer(size));
+	_buffer = make_shared<ASDCP::JP2K::FrameBuffer>(size);
 	_buffer->Size (size);
 	memcpy (_buffer->Data(), data, size);
 }
 
 
-/** @return Pointer to JPEG2000 data */
 uint8_t const *
 MonoPictureFrame::data () const
 {
 	return _buffer->RoData ();
 }
 
-/** @return Pointer to JPEG2000 data */
+
 uint8_t *
 MonoPictureFrame::data ()
 {
 	return _buffer->Data ();
 }
 
-/** @return Size of JPEG2000 data in bytes */
+
 int
 MonoPictureFrame::size () const
 {
 	return _buffer->Size ();
 }
 
-/** @param reduce a factor by which to reduce the resolution
- *  of the image, expressed as a power of two (pass 0 for no
- *  reduction).
- */
+
 shared_ptr<OpenJPEGImage>
 MonoPictureFrame::xyz_image (int reduce) const
 {
-	return decompress_j2k (const_cast<uint8_t*> (_buffer->RoData()), _buffer->Size(), reduce);
+	return decompress_j2k (const_cast<uint8_t*>(_buffer->RoData()), _buffer->Size(), reduce);
 }