X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fpicture_asset.cc;h=0019d5622f59a128451c3f6c0f26a541acfc6734;hb=e1e4934599bc244817ae63f4bc9f4a2ba81b1a29;hp=d7036a190fcebf68033ed923bdb035fbcbd83adc;hpb=f6ee90107839e0d750d8678b67c9d6d1d596d739;p=libdcp.git diff --git a/src/picture_asset.cc b/src/picture_asset.cc index d7036a19..0019d562 100644 --- a/src/picture_asset.cc +++ b/src/picture_asset.cc @@ -36,277 +36,501 @@ #include "exceptions.h" #include "picture_frame.h" -using namespace std; -using namespace boost; +using std::string; +using std::ostream; +using std::list; +using std::vector; +using std::max; +using std::pair; +using std::make_pair; +using std::istream; +using std::cout; +using boost::shared_ptr; +using boost::dynamic_pointer_cast; +using boost::lexical_cast; using namespace libdcp; -PictureAsset::PictureAsset ( - sigc::slot get_path, +PictureAsset::PictureAsset (string directory, string mxf_name, boost::signals2::signal* progress, int fps, int intrinsic_duration, Size size) + : MXFAsset (directory, mxf_name, progress, fps, intrinsic_duration) + , _size (size) +{ + +} + +PictureAsset::PictureAsset (string directory, string mxf_name) + : MXFAsset (directory, mxf_name) +{ + +} + +void +PictureAsset::write_to_cpl (ostream& s) const +{ + s << " \n" + << " urn:uuid:" << _uuid << "\n" + << " " << _file_name << "\n" + << " " << _edit_rate << " 1\n" + << " " << _intrinsic_duration << "\n" + << " " << _entry_point << "\n" + << " " << _duration << "\n" + << " " << _edit_rate << " 1\n" + << " " << _size.width << " " << _size.height << "\n" + << " \n"; +} + +bool +PictureAsset::equals (shared_ptr other, EqualityOptions opt, boost::function note) const +{ + if (!MXFAsset::equals (other, opt, note)) { + return false; + } + + ASDCP::JP2K::MXFReader reader_A; + if (ASDCP_FAILURE (reader_A.OpenRead (path().string().c_str()))) { + boost::throw_exception (MXFFileError ("could not open MXF file for reading", path().string())); + } + + ASDCP::JP2K::MXFReader reader_B; + if (ASDCP_FAILURE (reader_B.OpenRead (other->path().string().c_str()))) { + boost::throw_exception (MXFFileError ("could not open MXF file for reading", path().string())); + } + + ASDCP::JP2K::PictureDescriptor desc_A; + if (ASDCP_FAILURE (reader_A.FillPictureDescriptor (desc_A))) { + boost::throw_exception (DCPReadError ("could not read video MXF information")); + } + ASDCP::JP2K::PictureDescriptor desc_B; + if (ASDCP_FAILURE (reader_B.FillPictureDescriptor (desc_B))) { + boost::throw_exception (DCPReadError ("could not read video MXF information")); + } + + if ( + desc_A.EditRate != desc_B.EditRate || + desc_A.ContainerDuration != desc_B.ContainerDuration || + desc_A.SampleRate != desc_B.SampleRate || + desc_A.StoredWidth != desc_B.StoredWidth || + desc_A.StoredHeight != desc_B.StoredHeight || + desc_A.AspectRatio != desc_B.AspectRatio || + desc_A.Rsize != desc_B.Rsize || + desc_A.Xsize != desc_B.Xsize || + desc_A.Ysize != desc_B.Ysize || + desc_A.XOsize != desc_B.XOsize || + desc_A.YOsize != desc_B.YOsize || + desc_A.XTsize != desc_B.XTsize || + desc_A.YTsize != desc_B.YTsize || + desc_A.XTOsize != desc_B.XTOsize || + desc_A.YTOsize != desc_B.YTOsize || + desc_A.Csize != desc_B.Csize +// desc_A.CodingStyleDefault != desc_B.CodingStyleDefault || +// desc_A.QuantizationDefault != desc_B.QuantizationDefault + ) { + + note (ERROR, "video MXF picture descriptors differ"); + return false; + } + +// for (unsigned int j = 0; j < ASDCP::JP2K::MaxComponents; ++j) { +// if (desc_A.ImageComponents[j] != desc_B.ImageComponents[j]) { +// notes.pack_start ("video MXF picture descriptors differ"); +// } +// } + + return true; +} + + +MonoPictureAsset::MonoPictureAsset ( + boost::function get_path, string directory, string mxf_name, - sigc::signal1* progress, + boost::signals2::signal* progress, int fps, - int length, - int width, - int height) - : Asset (directory, mxf_name, progress, fps, length) - , _width (width) - , _height (height) + int intrinsic_duration, + Size size, + MXFMetadata const & metadata + ) + : PictureAsset (directory, mxf_name, progress, fps, intrinsic_duration, size) { - construct (get_path); + construct (get_path, metadata); } -PictureAsset::PictureAsset ( +MonoPictureAsset::MonoPictureAsset ( vector const & files, string directory, string mxf_name, - sigc::signal1* progress, + boost::signals2::signal* progress, int fps, - int length, - int width, - int height) - : Asset (directory, mxf_name, progress, fps, length) - , _width (width) - , _height (height) + int intrinsic_duration, + Size size, + MXFMetadata const & metadata + ) + : PictureAsset (directory, mxf_name, progress, fps, intrinsic_duration, size) { - construct (sigc::bind (sigc::mem_fun (*this, &PictureAsset::path_from_list), files)); + construct (boost::bind (&MonoPictureAsset::path_from_list, this, _1, files), metadata); } -PictureAsset::PictureAsset (string directory, string mxf_name, int fps, int length, int width, int height) - : Asset (directory, mxf_name, 0, fps, length) - , _width (width) - , _height (height) +MonoPictureAsset::MonoPictureAsset (string directory, string mxf_name, int fps, Size size) + : PictureAsset (directory, mxf_name, 0, fps, 0, size) { } -string -PictureAsset::path_from_list (int f, vector const & files) const +MonoPictureAsset::MonoPictureAsset (string directory, string mxf_name) + : PictureAsset (directory, mxf_name) { - return files[f]; + ASDCP::JP2K::MXFReader reader; + if (ASDCP_FAILURE (reader.OpenRead (path().string().c_str()))) { + boost::throw_exception (MXFFileError ("could not open MXF file for reading", path().string())); + } + + ASDCP::JP2K::PictureDescriptor desc; + if (ASDCP_FAILURE (reader.FillPictureDescriptor (desc))) { + boost::throw_exception (DCPReadError ("could not read video MXF information")); + } + + _size.width = desc.StoredWidth; + _size.height = desc.StoredHeight; + _edit_rate = desc.EditRate.Numerator; + assert (desc.EditRate.Denominator == 1); + _intrinsic_duration = desc.ContainerDuration; } void -PictureAsset::construct (sigc::slot get_path) +MonoPictureAsset::construct (boost::function get_path, MXFMetadata const & metadata) { ASDCP::JP2K::CodestreamParser j2k_parser; ASDCP::JP2K::FrameBuffer frame_buffer (4 * Kumu::Megabyte); if (ASDCP_FAILURE (j2k_parser.OpenReadFrame (get_path(0).c_str(), frame_buffer))) { - throw FileError ("could not open JPEG2000 file for reading", get_path (0)); + boost::throw_exception (FileError ("could not open JPEG2000 file for reading", get_path (0))); } ASDCP::JP2K::PictureDescriptor picture_desc; j2k_parser.FillPictureDescriptor (picture_desc); - picture_desc.EditRate = ASDCP::Rational (_fps, 1); + picture_desc.EditRate = ASDCP::Rational (_edit_rate, 1); ASDCP::WriterInfo writer_info; - fill_writer_info (&writer_info); + fill_writer_info (&writer_info, _uuid, metadata); ASDCP::JP2K::MXFWriter mxf_writer; - if (ASDCP_FAILURE (mxf_writer.OpenWrite (mxf_path().string().c_str(), writer_info, picture_desc))) { - throw FileError ("could not open MXF file for writing", mxf_path().string()); + if (ASDCP_FAILURE (mxf_writer.OpenWrite (path().string().c_str(), writer_info, picture_desc, 16384, false))) { + boost::throw_exception (MXFFileError ("could not open MXF file for writing", path().string())); } - for (int i = 0; i < _length; ++i) { + for (int i = 0; i < _intrinsic_duration; ++i) { string const path = get_path (i); - + if (ASDCP_FAILURE (j2k_parser.OpenReadFrame (path.c_str(), frame_buffer))) { - throw FileError ("could not open JPEG2000 file for reading", path); + boost::throw_exception (FileError ("could not open JPEG2000 file for reading", path)); } - /* XXX: passing 0 to WriteFrame ok? */ if (ASDCP_FAILURE (mxf_writer.WriteFrame (frame_buffer, 0, 0))) { - throw MiscError ("error in writing video MXF"); + boost::throw_exception (MXFFileError ("error in writing video MXF", this->path().string())); + } + + if (_progress) { + (*_progress) (0.5 * float (i) / _intrinsic_duration); } - - (*_progress) (0.5 * float (i) / _length); } if (ASDCP_FAILURE (mxf_writer.Finalize())) { - throw MiscError ("error in finalising video MXF"); + boost::throw_exception (MXFFileError ("error in finalising video MXF", path().string())); } } -void -PictureAsset::write_to_cpl (ostream& s) const +string +MonoPictureAsset::path_from_list (int f, vector const & files) const { - s << " \n" - << " urn:uuid:" << _uuid << "\n" - << " " << _mxf_name << "\n" - << " " << _fps << " 1\n" - << " " << _length << "\n" - << " 0\n" - << " " << _length << "\n" - << " " << _fps << " 1\n" - << " " << _width << " " << _height << "\n" - << " \n"; + return files[f]; } -list -PictureAsset::equals (shared_ptr other, EqualityOptions opt) const +shared_ptr +MonoPictureAsset::get_frame (int n) const { - list notes = Asset::equals (other, opt); - - if (opt.flags & MXF_INSPECT) { - ASDCP::JP2K::MXFReader reader_A; - if (ASDCP_FAILURE (reader_A.OpenRead (mxf_path().string().c_str()))) { - throw FileError ("could not open MXF file for reading", mxf_path().string()); - } + return shared_ptr (new MonoPictureFrame (path().string(), n)); +} + + +bool +MonoPictureAsset::equals (shared_ptr other, EqualityOptions opt, boost::function note) const +{ + if (!PictureAsset::equals (other, opt, note)) { + return false; + } - ASDCP::JP2K::MXFReader reader_B; - if (ASDCP_FAILURE (reader_B.OpenRead (other->mxf_path().string().c_str()))) { - throw FileError ("could not open MXF file for reading", mxf_path().string()); + shared_ptr other_picture = dynamic_pointer_cast (other); + assert (other_picture); + + for (int i = 0; i < _intrinsic_duration; ++i) { + note (PROGRESS, "Comparing video frame " + lexical_cast (i) + " of " + lexical_cast (_intrinsic_duration)); + shared_ptr frame_A = get_frame (i); + shared_ptr frame_B = other_picture->get_frame (i); + + if (!frame_buffer_equals ( + i, opt, note, + frame_A->j2k_data(), frame_A->j2k_size(), + frame_B->j2k_data(), frame_B->j2k_size() + )) { + return false; } + } + + return true; +} - ASDCP::JP2K::PictureDescriptor desc_A; - if (ASDCP_FAILURE (reader_A.FillPictureDescriptor (desc_A))) { - throw DCPReadError ("could not read video MXF information"); +bool +StereoPictureAsset::equals (shared_ptr other, EqualityOptions opt, boost::function note) const +{ + if (!PictureAsset::equals (other, opt, note)) { + return false; + } + + shared_ptr other_picture = dynamic_pointer_cast (other); + assert (other_picture); + + for (int i = 0; i < _intrinsic_duration; ++i) { + shared_ptr frame_A = get_frame (i); + shared_ptr frame_B = other_picture->get_frame (i); + + if (!frame_buffer_equals ( + i, opt, note, + frame_A->left_j2k_data(), frame_A->left_j2k_size(), + frame_B->left_j2k_data(), frame_B->left_j2k_size() + )) { + return false; } - ASDCP::JP2K::PictureDescriptor desc_B; - if (ASDCP_FAILURE (reader_B.FillPictureDescriptor (desc_B))) { - throw DCPReadError ("could not read video MXF information"); + + if (!frame_buffer_equals ( + i, opt, note, + frame_A->right_j2k_data(), frame_A->right_j2k_size(), + frame_B->right_j2k_data(), frame_B->right_j2k_size() + )) { + return false; } + } - if ( - desc_A.EditRate != desc_B.EditRate || - desc_A.ContainerDuration != desc_B.ContainerDuration || - desc_A.SampleRate != desc_B.SampleRate || - desc_A.StoredWidth != desc_B.StoredWidth || - desc_A.StoredHeight != desc_B.StoredHeight || - desc_A.AspectRatio != desc_B.AspectRatio || - desc_A.Rsize != desc_B.Rsize || - desc_A.Xsize != desc_B.Xsize || - desc_A.Ysize != desc_B.Ysize || - desc_A.XOsize != desc_B.XOsize || - desc_A.YOsize != desc_B.YOsize || - desc_A.XTsize != desc_B.XTsize || - desc_A.YTsize != desc_B.YTsize || - desc_A.XTOsize != desc_B.XTOsize || - desc_A.YTOsize != desc_B.YTOsize || - desc_A.Csize != desc_B.Csize -// desc_A.CodingStyleDefault != desc_B.CodingStyleDefault || -// desc_A.QuantizationDefault != desc_B.QuantizationDefault - ) { + return true; +} + +bool +PictureAsset::frame_buffer_equals ( + int frame, EqualityOptions opt, boost::function note, + uint8_t const * data_A, unsigned int size_A, uint8_t const * data_B, unsigned int size_B + ) const +{ + if (size_A == size_B && memcmp (data_A, data_B, size_A) == 0) { + note (NOTE, "J2K identical"); + /* Easy result; the J2K data is identical */ + return true; + } - notes.push_back ("video MXF picture descriptors differ"); + /* Decompress the images to bitmaps */ + opj_image_t* image_A = decompress_j2k (const_cast (data_A), size_A, 0); + opj_image_t* image_B = decompress_j2k (const_cast (data_B), size_B, 0); + + /* Compare them */ + + if (image_A->numcomps != image_B->numcomps) { + note (ERROR, "image component counts for frame " + lexical_cast(frame) + " differ"); + return false; + } + + vector abs_diffs (image_A->comps[0].w * image_A->comps[0].h * image_A->numcomps); + int d = 0; + int max_diff = 0; + + for (int c = 0; c < image_A->numcomps; ++c) { + + if (image_A->comps[c].w != image_B->comps[c].w || image_A->comps[c].h != image_B->comps[c].h) { + note (ERROR, "image sizes for frame " + lexical_cast(frame) + " differ"); + return false; } - -// for (unsigned int j = 0; j < ASDCP::JP2K::MaxComponents; ++j) { -// if (desc_A.ImageComponents[j] != desc_B.ImageComponents[j]) { -// notes.pack_start ("video MXF picture descriptors differ"); -// } -// } - - - ASDCP::JP2K::FrameBuffer buffer_A (4 * Kumu::Megabyte); - ASDCP::JP2K::FrameBuffer buffer_B (4 * Kumu::Megabyte); - - for (int i = 0; i < _length; ++i) { - if (ASDCP_FAILURE (reader_A.ReadFrame (i, buffer_A))) { - throw DCPReadError ("could not read video frame"); - } - - if (ASDCP_FAILURE (reader_B.ReadFrame (i, buffer_B))) { - throw DCPReadError ("could not read video frame"); - } - - bool j2k_same = true; - - if (buffer_A.Size() != buffer_B.Size()) { - notes.push_back ("sizes of video data for frame " + lexical_cast(i) + " differ"); - j2k_same = false; - } else if (memcmp (buffer_A.RoData(), buffer_B.RoData(), buffer_A.Size()) != 0) { - notes.push_back ("J2K data for frame " + lexical_cast(i) + " differ"); - j2k_same = false; - } - - if (!j2k_same) { - - if (opt.verbose) { - cout << "J2K images for " << i << " differ; checking by pixel\n"; - } - - /* Decompress the images to bitmaps */ - opj_image_t* image_A = decompress_j2k (const_cast (buffer_A.RoData()), buffer_A.Size ()); - opj_image_t* image_B = decompress_j2k (const_cast (buffer_B.RoData()), buffer_B.Size ()); - - /* Compare them */ - - if (image_A->numcomps != image_B->numcomps) { - notes.push_back ("image component counts for frame " + lexical_cast(i) + " differ"); - } - - vector abs_diffs (image_A->comps[0].w * image_A->comps[0].h * image_A->numcomps); - int d = 0; - int max_diff = 0; - - for (int c = 0; c < image_A->numcomps; ++c) { - - if (image_A->comps[c].w != image_B->comps[c].w || image_A->comps[c].h != image_B->comps[c].h) { - notes.push_back ("image sizes for frame " + lexical_cast(i) + " differ"); - } - - int const pixels = image_A->comps[c].w * image_A->comps[c].h; - for (int j = 0; j < pixels; ++j) { - int const t = abs (image_A->comps[c].data[j] - image_B->comps[c].data[j]); - abs_diffs[d++] = t; - max_diff = max (max_diff, t); - } - } - - uint64_t total = 0; - for (vector::iterator j = abs_diffs.begin(); j != abs_diffs.end(); ++j) { - total += *j; - } - - double const mean = double (total) / abs_diffs.size (); - - uint64_t total_squared_deviation = 0; - for (vector::iterator j = abs_diffs.begin(); j != abs_diffs.end(); ++j) { - total_squared_deviation += pow (*j - mean, 2); - } - - double const std_dev = sqrt (double (total_squared_deviation) / abs_diffs.size()); - - if (mean > opt.max_mean_pixel_error || std_dev > opt.max_std_dev_pixel_error) { - notes.push_back ("mean or standard deviation out of range for " + lexical_cast(i)); - } - - if (opt.verbose) { - cout << "\tmax pixel error " << max_diff << ", mean pixel error " << mean << ", standard deviation " << std_dev << "\n"; - } - - opj_image_destroy (image_A); - opj_image_destroy (image_B); - } + + int const pixels = image_A->comps[c].w * image_A->comps[c].h; + for (int j = 0; j < pixels; ++j) { + int const t = abs (image_A->comps[c].data[j] - image_B->comps[c].data[j]); + abs_diffs[d++] = t; + max_diff = max (max_diff, t); } } + + uint64_t total = 0; + for (vector::iterator j = abs_diffs.begin(); j != abs_diffs.end(); ++j) { + total += *j; + } + + double const mean = double (total) / abs_diffs.size (); + + uint64_t total_squared_deviation = 0; + for (vector::iterator j = abs_diffs.begin(); j != abs_diffs.end(); ++j) { + total_squared_deviation += pow (*j - mean, 2); + } + + double const std_dev = sqrt (double (total_squared_deviation) / abs_diffs.size()); + + note (NOTE, "mean difference " + lexical_cast (mean) + ", deviation " + lexical_cast (std_dev)); + + if (mean > opt.max_mean_pixel_error) { + note (ERROR, "mean " + lexical_cast(mean) + " out of range " + lexical_cast(opt.max_mean_pixel_error) + " in frame " + lexical_cast(frame)); + return false; + } + + if (std_dev > opt.max_std_dev_pixel_error) { + note (ERROR, "standard deviation " + lexical_cast(std_dev) + " out of range " + lexical_cast(opt.max_std_dev_pixel_error) + " in frame " + lexical_cast(frame)); + return false; + } + + opj_image_destroy (image_A); + opj_image_destroy (image_B); + + return true; +} + + +StereoPictureAsset::StereoPictureAsset (string directory, string mxf_name, int fps, int intrinsic_duration) + : PictureAsset (directory, mxf_name, 0, fps, intrinsic_duration, Size (0, 0)) +{ + ASDCP::JP2K::MXFSReader reader; + if (ASDCP_FAILURE (reader.OpenRead (path().string().c_str()))) { + boost::throw_exception (MXFFileError ("could not open MXF file for reading", path().string())); + } + + ASDCP::JP2K::PictureDescriptor desc; + if (ASDCP_FAILURE (reader.FillPictureDescriptor (desc))) { + boost::throw_exception (DCPReadError ("could not read video MXF information")); + } + + _size.width = desc.StoredWidth; + _size.height = desc.StoredHeight; +} + +shared_ptr +StereoPictureAsset::get_frame (int n) const +{ + return shared_ptr (new StereoPictureFrame (path().string(), n)); +} + +shared_ptr +MonoPictureAsset::start_write (bool overwrite, MXFMetadata const & metadata) +{ + /* XXX: can't we use shared_ptr here? */ + return shared_ptr (new MonoPictureAssetWriter (this, overwrite, metadata)); +} + +FrameInfo::FrameInfo (istream& s) +{ + s >> offset >> size >> hash; +} + +void +FrameInfo::write (ostream& s) +{ + s << offset << " " << size << " " << hash; +} + +struct MonoPictureAssetWriter::ASDCPState +{ + ASDCPState() + : frame_buffer (4 * Kumu::Megabyte) + {} + + ASDCP::JP2K::CodestreamParser j2k_parser; + ASDCP::JP2K::FrameBuffer frame_buffer; + ASDCP::JP2K::MXFWriter mxf_writer; + ASDCP::WriterInfo writer_info; + ASDCP::JP2K::PictureDescriptor picture_descriptor; +}; + + +/** @param a Asset to write to. `a' must not be deleted while + * this writer class still exists, or bad things will happen. + */ +MonoPictureAssetWriter::MonoPictureAssetWriter (MonoPictureAsset* a, bool overwrite, MXFMetadata const & m) + : _state (new MonoPictureAssetWriter::ASDCPState) + , _asset (a) + , _frames_written (0) + , _started (false) + , _finalized (false) + , _overwrite (overwrite) + , _metadata (m) +{ + +} + + +void +MonoPictureAssetWriter::start (uint8_t* data, int size) +{ + if (ASDCP_FAILURE (_state->j2k_parser.OpenReadFrame (data, size, _state->frame_buffer))) { + boost::throw_exception (MiscError ("could not parse J2K frame")); + } - return notes; + _state->j2k_parser.FillPictureDescriptor (_state->picture_descriptor); + _state->picture_descriptor.EditRate = ASDCP::Rational (_asset->edit_rate(), 1); + + MXFAsset::fill_writer_info (&_state->writer_info, _asset->uuid(), _metadata); + + if (ASDCP_FAILURE (_state->mxf_writer.OpenWrite ( + _asset->path().string().c_str(), + _state->writer_info, + _state->picture_descriptor, + 16384, + _overwrite) + )) { + + boost::throw_exception (MXFFileError ("could not open MXF file for writing", _asset->path().string())); + } + + _started = true; +} + +FrameInfo +MonoPictureAssetWriter::write (uint8_t* data, int size) +{ + assert (!_finalized); + + if (!_started) { + start (data, size); + } + + if (ASDCP_FAILURE (_state->j2k_parser.OpenReadFrame (data, size, _state->frame_buffer))) { + boost::throw_exception (MiscError ("could not parse J2K frame")); + } + + uint64_t const before_offset = _state->mxf_writer.Tell (); + + string hash; + if (ASDCP_FAILURE (_state->mxf_writer.WriteFrame (_state->frame_buffer, 0, 0, &hash))) { + boost::throw_exception (MXFFileError ("error in writing video MXF", _asset->path().string())); + } + + ++_frames_written; + return FrameInfo (before_offset, _state->mxf_writer.Tell() - before_offset, hash); } -opj_image_t * -PictureAsset::decompress_j2k (uint8_t* data, int64_t size) const +void +MonoPictureAssetWriter::fake_write (int size) { - opj_dinfo_t* decoder = opj_create_decompress (CODEC_J2K); - opj_dparameters_t parameters; - opj_set_default_decoder_parameters (¶meters); - opj_setup_decoder (decoder, ¶meters); - opj_cio_t* cio = opj_cio_open ((opj_common_ptr) decoder, data, size); - opj_image_t* image = opj_decode (decoder, cio); - if (!image) { - opj_destroy_decompress (decoder); - opj_cio_close (cio); - throw DCPReadError ("could not decode JPEG2000 codestream"); + assert (_started); + assert (!_finalized); + + if (ASDCP_FAILURE (_state->mxf_writer.FakeWriteFrame (size))) { + boost::throw_exception (MXFFileError ("error in writing video MXF", _asset->path().string())); } - opj_cio_close (cio); - return image; + ++_frames_written; } -shared_ptr -PictureAsset::get_frame (int n) const +void +MonoPictureAssetWriter::finalize () { - return shared_ptr (new PictureFrame (mxf_path().string(), n)); + assert (!_finalized); + + if (ASDCP_FAILURE (_state->mxf_writer.Finalize())) { + boost::throw_exception (MXFFileError ("error in finalizing video MXF", _asset->path().string())); + } + + _finalized = true; + _asset->set_intrinsic_duration (_frames_written); + _asset->set_duration (_frames_written); }