Try to add correct namespace for 3D CPLs.
[libdcp.git] / src / picture_asset.cc
index c3b496687a89aaeb436eb97de63e88940194e673..07cc195dd1ca4ba5334b98e66ab459752b0a7b6d 100644 (file)
 #include <boost/filesystem.hpp>
 #include <boost/lexical_cast.hpp>
 #include <openjpeg.h>
+#include <libxml++/nodes/element.h>
 #include "AS_DCP.h"
 #include "KM_fileio.h"
 #include "picture_asset.h"
 #include "util.h"
 #include "exceptions.h"
 #include "picture_frame.h"
+#include "xyz_frame.h"
+#include "picture_asset_writer.h"
 
 using std::string;
 using std::ostream;
 using std::list;
 using std::vector;
 using std::max;
+using std::stringstream;
+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 (string directory, string mxf_name, boost::signals2::signal<void (float)>* progress, int fps, int intrinsic_duration, Size size)
-       : MXFAsset (directory, mxf_name, progress, fps, intrinsic_duration)
+PictureAsset::PictureAsset (string directory, string mxf_name, boost::signals2::signal<void (float)>* progress, int fps, int intrinsic_duration, bool encrypted, Size size)
+       : MXFAsset (directory, mxf_name, progress, fps, intrinsic_duration, encrypted)
        , _size (size)
 {
 
@@ -59,50 +67,91 @@ PictureAsset::PictureAsset (string directory, string mxf_name)
 
 }
 
+string
+MonoPictureAsset::cpl_node_name () const
+{
+       return "MainPicture";
+}
+
+int
+MonoPictureAsset::edit_rate_factor () const
+{
+       return 1;
+}
+
+string
+StereoPictureAsset::cpl_node_name () const
+{
+       return "MainStereoscopicPicture";
+}
+
+string
+StereoPictureAsset::cpl_node_namespace (bool interop) const
+{
+       if (interop) {
+               return "http://www.digicine.com/schemas/437-Y/2007/Main-Stereo-Picture-CPL";
+       } else {
+               return "http://www.smpte-ra.org/schemas/429-10/2008/Main-Stereo-Picture-CPL";
+       }
+
+       return "";
+}
+
+int
+StereoPictureAsset::edit_rate_factor () const
+{
+       return 2;
+}
+
 void
-PictureAsset::write_to_cpl (ostream& s) const
+PictureAsset::write_to_cpl (xmlpp::Node* node, bool interop) const
 {
-       s << "        <MainPicture>\n"
-         << "          <Id>urn:uuid:" << _uuid << "</Id>\n"
-         << "          <AnnotationText>" << _file_name << "</AnnotationText>\n"
-         << "          <EditRate>" << _edit_rate << " 1</EditRate>\n"
-         << "          <IntrinsicDuration>" << _intrinsic_duration << "</IntrinsicDuration>\n"
-         << "          <EntryPoint>" << _entry_point << "</EntryPoint>\n"
-         << "          <Duration>" << _duration << "</Duration>\n"
-         << "          <FrameRate>" << _edit_rate << " 1</FrameRate>\n"
-         << "          <ScreenAspectRatio>" << _size.width << " " << _size.height << "</ScreenAspectRatio>\n"
-         << "        </MainPicture>\n";
+       MXFAsset::write_to_cpl (node, interop);
+       
+       xmlpp::Node::NodeList c = node->get_children ();
+       xmlpp::Node::NodeList::iterator i = c.begin();
+       while (i != c.end() && (*i)->get_name() != cpl_node_name ()) {
+               ++i;
+       }
+
+       assert (i != c.end ());
+
+       (*i)->add_child ("FrameRate")->add_child_text (lexical_cast<string> (_edit_rate * edit_rate_factor ()) + " 1");
+       if (interop) {
+               (*i)->add_child ("ScreenAspectRatio")->add_child_text (lexical_cast<string> (float (_size.width) / _size.height));
+       } else {
+               (*i)->add_child ("ScreenAspectRatio")->add_child_text (lexical_cast<string> (_size.width) + " " + lexical_cast<string> (_size.height));
+       }
 }
 
 bool
-PictureAsset::equals (shared_ptr<const Asset> other, EqualityOptions opt, list<string>& notes) const
+PictureAsset::equals (shared_ptr<const Asset> other, EqualityOptions opt, boost::function<void (NoteType, string)> note) const
 {
-       if (!MXFAsset::equals (other, opt, notes)) {
+       if (!MXFAsset::equals (other, opt, note)) {
                return false;
        }
                     
        ASDCP::JP2K::MXFReader reader_A;
        if (ASDCP_FAILURE (reader_A.OpenRead (path().string().c_str()))) {
-               throw MXFFileError ("could not open MXF file for reading", path().string());
+               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()))) {
-               throw MXFFileError ("could not open MXF file for reading", path().string());
+               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))) {
-               throw DCPReadError ("could not read video MXF information");
+               boost::throw_exception (DCPReadError ("could not read video MXF information"));
        }
        ASDCP::JP2K::PictureDescriptor desc_B;
        if (ASDCP_FAILURE (reader_B.FillPictureDescriptor (desc_B))) {
-               throw DCPReadError ("could not read video MXF information");
+               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 ||
@@ -121,10 +170,14 @@ PictureAsset::equals (shared_ptr<const Asset> other, EqualityOptions opt, list<s
 //             desc_A.QuantizationDefault != desc_B.QuantizationDefault
                ) {
                
-               notes.push_back ("video MXF picture descriptors differ");
+               note (ERROR, "video MXF picture descriptors differ");
                return false;
        }
 
+       if (desc_A.ContainerDuration != desc_B.ContainerDuration) {
+               note (ERROR, "video container durations differ");
+       }
+       
 //             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");
@@ -142,10 +195,14 @@ MonoPictureAsset::MonoPictureAsset (
        boost::signals2::signal<void (float)>* progress,
        int fps,
        int intrinsic_duration,
-       Size size)
-       : PictureAsset (directory, mxf_name, progress, fps, intrinsic_duration, size)
+       bool encrypted,
+       Size size,
+       bool interop,
+       MXFMetadata const & metadata
+       )
+       : PictureAsset (directory, mxf_name, progress, fps, intrinsic_duration, encrypted, size)
 {
-       construct (get_path);
+       construct (get_path, interop, metadata);
 }
 
 MonoPictureAsset::MonoPictureAsset (
@@ -155,14 +212,18 @@ MonoPictureAsset::MonoPictureAsset (
        boost::signals2::signal<void (float)>* progress,
        int fps,
        int intrinsic_duration,
-       Size size)
-       : PictureAsset (directory, mxf_name, progress, fps, intrinsic_duration, size)
+       bool encrypted,
+       Size size,
+       bool interop,
+       MXFMetadata const & metadata
+       )
+       : PictureAsset (directory, mxf_name, progress, fps, intrinsic_duration, encrypted, size)
 {
-       construct (boost::bind (&MonoPictureAsset::path_from_list, this, _1, files));
+       construct (boost::bind (&MonoPictureAsset::path_from_list, this, _1, files), interop, metadata);
 }
 
 MonoPictureAsset::MonoPictureAsset (string directory, string mxf_name, int fps, Size size)
-       : PictureAsset (directory, mxf_name, 0, fps, 0, size)
+       : PictureAsset (directory, mxf_name, 0, fps, 0, false, size)
 {
 
 }
@@ -172,12 +233,12 @@ MonoPictureAsset::MonoPictureAsset (string directory, string mxf_name)
 {
        ASDCP::JP2K::MXFReader reader;
        if (ASDCP_FAILURE (reader.OpenRead (path().string().c_str()))) {
-               throw MXFFileError ("could not open MXF file for reading", path().string());
+               boost::throw_exception (MXFFileError ("could not open MXF file for reading", path().string()));
        }
        
        ASDCP::JP2K::PictureDescriptor desc;
        if (ASDCP_FAILURE (reader.FillPictureDescriptor (desc))) {
-               throw DCPReadError ("could not read video MXF information");
+               boost::throw_exception (DCPReadError ("could not read video MXF information"));
        }
 
        _size.width = desc.StoredWidth;
@@ -188,12 +249,12 @@ MonoPictureAsset::MonoPictureAsset (string directory, string mxf_name)
 }
 
 void
-MonoPictureAsset::construct (boost::function<string (int)> get_path)
+MonoPictureAsset::construct (boost::function<string (int)> get_path, bool interop, 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;
@@ -201,11 +262,11 @@ MonoPictureAsset::construct (boost::function<string (int)> get_path)
        picture_desc.EditRate = ASDCP::Rational (_edit_rate, 1);
        
        ASDCP::WriterInfo writer_info;
-       fill_writer_info (&writer_info, _uuid);
+       fill_writer_info (&writer_info, _uuid, interop, metadata);
        
        ASDCP::JP2K::MXFWriter mxf_writer;
-       if (ASDCP_FAILURE (mxf_writer.OpenWrite (path().string().c_str(), writer_info, picture_desc))) {
-               throw MXFFileError ("could not open MXF file for writing", 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 < _intrinsic_duration; ++i) {
@@ -213,12 +274,11 @@ MonoPictureAsset::construct (boost::function<string (int)> get_path)
                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");
+               if (ASDCP_FAILURE (mxf_writer.WriteFrame (frame_buffer, _encryption_context, 0))) {
+                       boost::throw_exception (MXFFileError ("error in writing video MXF", this->path().string()));
                }
 
                if (_progress) {
@@ -227,7 +287,7 @@ MonoPictureAsset::construct (boost::function<string (int)> get_path)
        }
        
        if (ASDCP_FAILURE (mxf_writer.Finalize())) {
-               throw MiscError ("error in finalising video MXF");
+               boost::throw_exception (MXFFileError ("error in finalising video MXF", path().string()));
        }
 }
 
@@ -240,14 +300,14 @@ MonoPictureAsset::path_from_list (int f, vector<string> const & files) const
 shared_ptr<const MonoPictureFrame>
 MonoPictureAsset::get_frame (int n) const
 {
-       return shared_ptr<const MonoPictureFrame> (new MonoPictureFrame (path().string(), n + _entry_point));
+       return shared_ptr<const MonoPictureFrame> (new MonoPictureFrame (path().string(), n, _decryption_context));
 }
 
 
 bool
-MonoPictureAsset::equals (shared_ptr<const Asset> other, EqualityOptions opt, list<string>& notes) const
+MonoPictureAsset::equals (shared_ptr<const Asset> other, EqualityOptions opt, boost::function<void (NoteType, string)> note) const
 {
-       if (!PictureAsset::equals (other, opt, notes)) {
+       if (!PictureAsset::equals (other, opt, note)) {
                return false;
        }
 
@@ -255,13 +315,18 @@ MonoPictureAsset::equals (shared_ptr<const Asset> other, EqualityOptions opt, li
        assert (other_picture);
 
        for (int i = 0; i < _intrinsic_duration; ++i) {
+               if (i >= other_picture->intrinsic_duration()) {
+                       return false;
+               }
+               
+               note (PROGRESS, "Comparing video frame " + lexical_cast<string> (i) + " of " + lexical_cast<string> (_intrinsic_duration));
                shared_ptr<const MonoPictureFrame> frame_A = get_frame (i);
                shared_ptr<const MonoPictureFrame> frame_B = other_picture->get_frame (i);
                
                if (!frame_buffer_equals (
-                           i, opt, notes,
-                           frame_A->j2k_frame()->RoData(), frame_A->j2k_frame()->Size(),
-                           frame_B->j2k_frame()->RoData(), frame_B->j2k_frame()->Size()
+                           i, opt, note,
+                           frame_A->j2k_data(), frame_A->j2k_size(),
+                           frame_B->j2k_data(), frame_B->j2k_size()
                            )) {
                        return false;
                }
@@ -271,9 +336,9 @@ MonoPictureAsset::equals (shared_ptr<const Asset> other, EqualityOptions opt, li
 }
 
 bool
-StereoPictureAsset::equals (shared_ptr<const Asset> other, EqualityOptions opt, list<string>& notes) const
+StereoPictureAsset::equals (shared_ptr<const Asset> other, EqualityOptions opt, boost::function<void (NoteType, string)> note) const
 {
-       if (!PictureAsset::equals (other, opt, notes)) {
+       if (!PictureAsset::equals (other, opt, note)) {
                return false;
        }
        
@@ -285,17 +350,17 @@ StereoPictureAsset::equals (shared_ptr<const Asset> other, EqualityOptions opt,
                shared_ptr<const StereoPictureFrame> frame_B = other_picture->get_frame (i);
                
                if (!frame_buffer_equals (
-                           i, opt, notes,
-                           frame_A->j2k_frame()->Left.RoData(), frame_A->j2k_frame()->Left.Size(),
-                           frame_B->j2k_frame()->Left.RoData(), frame_B->j2k_frame()->Left.Size()
+                           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;
                }
                
                if (!frame_buffer_equals (
-                           i, opt, notes,
-                           frame_A->j2k_frame()->Right.RoData(), frame_A->j2k_frame()->Right.Size(),
-                           frame_B->j2k_frame()->Right.RoData(), frame_B->j2k_frame()->Right.Size()
+                           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;
                }
@@ -306,39 +371,36 @@ StereoPictureAsset::equals (shared_ptr<const Asset> other, EqualityOptions opt,
 
 bool
 PictureAsset::frame_buffer_equals (
-       int frame, EqualityOptions opt, list<string>& notes, uint8_t const * data_A, unsigned int size_A, uint8_t const * data_B, unsigned int size_B
+       int frame, EqualityOptions opt, boost::function<void (NoteType, string)> 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;
        }
                
        /* Decompress the images to bitmaps */
-       opj_image_t* image_A = decompress_j2k (const_cast<uint8_t*> (data_A), size_A, 0);
-       opj_image_t* image_B = decompress_j2k (const_cast<uint8_t*> (data_B), size_B, 0);
+       shared_ptr<XYZFrame> image_A = decompress_j2k (const_cast<uint8_t*> (data_A), size_A, 0);
+       shared_ptr<XYZFrame> image_B = decompress_j2k (const_cast<uint8_t*> (data_B), size_B, 0);
        
        /* Compare them */
        
-       if (image_A->numcomps != image_B->numcomps) {
-               notes.push_back ("image component counts for frame " + lexical_cast<string>(frame) + " differ");
-               return false;
-       }
-       
-       vector<int> abs_diffs (image_A->comps[0].w * image_A->comps[0].h * image_A->numcomps);
+       vector<int> abs_diffs (image_A->size().width * image_A->size().height * 3);
        int d = 0;
        int max_diff = 0;
        
-       for (int c = 0; c < image_A->numcomps; ++c) {
+       for (int c = 0; c < 3; ++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<string>(frame) + " differ");
+               if (image_A->size() != image_B->size()) {
+                       note (ERROR, "image sizes for frame " + lexical_cast<string>(frame) + " differ");
                        return false;
                }
                
-               int const pixels = image_A->comps[c].w * image_A->comps[c].h;
+               int const pixels = image_A->size().width * image_A->size().height;
                for (int j = 0; j < pixels; ++j) {
-                       int const t = abs (image_A->comps[c].data[j] - image_B->comps[c].data[j]);
+                       int const t = abs (image_A->data(c)[j] - image_B->data(c)[j]);
                        abs_diffs[d++] = t;
                        max_diff = max (max_diff, t);
                }
@@ -358,29 +420,33 @@ PictureAsset::frame_buffer_equals (
        
        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<string>(frame));
+       note (NOTE, "mean difference " + lexical_cast<string> (mean) + ", deviation " + lexical_cast<string> (std_dev));
+       
+       if (mean > opt.max_mean_pixel_error) {
+               note (ERROR, "mean " + lexical_cast<string>(mean) + " out of range " + lexical_cast<string>(opt.max_mean_pixel_error) + " in frame " + lexical_cast<string>(frame));
+               return false;
+       }
+
+       if (std_dev > opt.max_std_dev_pixel_error) {
+               note (ERROR, "standard deviation " + lexical_cast<string>(std_dev) + " out of range " + lexical_cast<string>(opt.max_std_dev_pixel_error) + " in frame " + lexical_cast<string>(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))
+       : PictureAsset (directory, mxf_name, 0, fps, intrinsic_duration, false, Size (0, 0))
 {
        ASDCP::JP2K::MXFSReader reader;
        if (ASDCP_FAILURE (reader.OpenRead (path().string().c_str()))) {
-               throw MXFFileError ("could not open MXF file for reading", path().string());
+               boost::throw_exception (MXFFileError ("could not open MXF file for reading", path().string()));
        }
        
        ASDCP::JP2K::PictureDescriptor desc;
        if (ASDCP_FAILURE (reader.FillPictureDescriptor (desc))) {
-               throw DCPReadError ("could not read video MXF information");
+               boost::throw_exception (DCPReadError ("could not read video MXF information"));
        }
 
        _size.width = desc.StoredWidth;
@@ -390,84 +456,32 @@ StereoPictureAsset::StereoPictureAsset (string directory, string mxf_name, int f
 shared_ptr<const StereoPictureFrame>
 StereoPictureAsset::get_frame (int n) const
 {
-       return shared_ptr<const StereoPictureFrame> (new StereoPictureFrame (path().string(), n + _entry_point));
+       return shared_ptr<const StereoPictureFrame> (new StereoPictureFrame (path().string(), n));
 }
 
-shared_ptr<MonoPictureAssetWriter>
-MonoPictureAsset::start_write ()
+shared_ptr<PictureAssetWriter>
+MonoPictureAsset::start_write (bool overwrite, bool interop, MXFMetadata const & metadata)
 {
-       /* XXX: can't we use shared_ptr here? */
-       return shared_ptr<MonoPictureAssetWriter> (new MonoPictureAssetWriter (this));
+       /* XXX: can't we use shared_ptr here? */
+       return shared_ptr<MonoPictureAssetWriter> (new MonoPictureAssetWriter (this, overwrite, interop, metadata));
 }
 
-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)
-       : _state (new MonoPictureAssetWriter::ASDCPState)
-       , _asset (a)
-       , _frames_written (0)
-       , _finalized (false)
+string
+PictureAsset::key_type () const
 {
-
+       return "MDIK";
 }
 
-void
-MonoPictureAssetWriter::write (uint8_t* data, int size)
+StereoPictureAsset::StereoPictureAsset (string directory, string mxf_name, int fps, Size size)
+       : PictureAsset (directory, mxf_name, 0, fps, 0, false, size)
 {
-       assert (!_finalized);
-       
-       if (ASDCP_FAILURE (_state->j2k_parser.OpenReadFrame (data, size, _state->frame_buffer))) {
-               throw MiscError ("could not parse J2K frame");
-       }
-
-       if (_frames_written == 0) {
-               /* This is our first frame; set up the writer */
-               
-               _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());
-               
-               if (ASDCP_FAILURE (_state->mxf_writer.OpenWrite (_asset->path().string().c_str(), _state->writer_info, _state->picture_descriptor))) {
-                       throw MXFFileError ("could not open MXF file for writing", _asset->path().string());
-               }
-       }
-
-       if (ASDCP_FAILURE (_state->mxf_writer.WriteFrame (_state->frame_buffer, 0, 0))) {
-               throw MiscError ("error in writing video MXF");
-       }
 
-       _frames_written++;
 }
 
-void
-MonoPictureAssetWriter::finalize ()
+shared_ptr<PictureAssetWriter>
+StereoPictureAsset::start_write (bool overwrite, bool interop, MXFMetadata const & metadata)
 {
-       if (ASDCP_FAILURE (_state->mxf_writer.Finalize())) {
-               throw MiscError ("error in finalizing video MXF");
-       }
-
-       _finalized = true;
-       _asset->set_intrinsic_duration (_frames_written);
-       _asset->set_duration (_frames_written);
+       /* XXX: can't we use shared_ptr here? */
+       return shared_ptr<StereoPictureAssetWriter> (new StereoPictureAssetWriter (this, overwrite, interop, metadata));
 }
 
-MonoPictureAssetWriter::~MonoPictureAssetWriter ()
-{
-       assert (_finalized);
-}