]> git.sesse.net Git - mlt/blobdiff - src/framework/mlt_producer.c
src/framework/*: improve the doxygen documentation (work in progress). This also...
[mlt] / src / framework / mlt_producer.c
index d84a0eb0571aeee2609230ad58c994222c86cb88..6f00734fa67d9ff59ea618df611c2ec0a57c22fc 100644 (file)
@@ -1,49 +1,57 @@
-/*
- * mlt_producer.c -- abstraction for all producer services
- * Copyright (C) 2003-2004 Ushodaya Enterprises Limited
- * Author: Charles Yates <charles.yates@pandora.be>
+/**
+ * \file mlt_producer.c
+ * \brief abstraction for all producer services
  *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
+ * Copyright (C) 2003-2008 Ushodaya Enterprises Limited
+ * \author Charles Yates <charles.yates@pandora.be>
  *
- * This program is distributed in the hope that it will be useful,
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
  *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software Foundation,
- * Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
  */
 
-#include "config.h"
 #include "mlt_producer.h"
 #include "mlt_factory.h"
 #include "mlt_frame.h"
 #include "mlt_parser.h"
+#include "mlt_profile.h"
+
 #include <stdio.h>
 #include <string.h>
 #include <stdlib.h>
 #include <math.h>
 
-/** Forward references.
-*/
+/* Forward references. */
 
 static int producer_get_frame( mlt_service this, mlt_frame_ptr frame, int index );
 static void mlt_producer_property_changed( mlt_service owner, mlt_producer this, char *name );
 static void mlt_producer_service_changed( mlt_service owner, mlt_producer this );
 
+/* for debugging */
 //#define _MLT_PRODUCER_CHECKS_ 1
-
 #ifdef _MLT_PRODUCER_CHECKS_
 static int producers_created = 0;
 static int producers_destroyed = 0;
 #endif
 
-/** Constructor
-*/
+/** Initialize a producer service.
+ *
+ * \public \memberof mlt_producer_s
+ * \param this the producer structure to initialize
+ * \param child a pointer to the child object for the subclass
+ * \return true if there was an error
+ * \todo Document the special properties and events.
+ */
 
 int mlt_producer_init( mlt_producer this, void *child )
 {
@@ -59,19 +67,16 @@ int mlt_producer_init( mlt_producer this, void *child )
 
                // Initialise the producer
                memset( this, 0, sizeof( struct mlt_producer_s ) );
-       
+
                // Associate with the child
                this->child = child;
 
                // Initialise the service
                if ( mlt_service_init( &this->parent, this ) == 0 )
                {
-                       // Get the normalisation preference
-                       char *normalisation = mlt_environment( "MLT_NORMALISATION" );
-
                        // The parent is the service
                        mlt_service parent = &this->parent;
-       
+
                        // Define the parent close
                        parent->close = ( mlt_destructor )mlt_producer_close;
                        parent->close_object = this;
@@ -81,21 +86,12 @@ int mlt_producer_init( mlt_producer this, void *child )
 
                        // Get the properties of the parent
                        mlt_properties properties = MLT_SERVICE_PROPERTIES( parent );
-       
+
                        // Set the default properties
                        mlt_properties_set( properties, "mlt_type", "mlt_producer" );
                        mlt_properties_set_position( properties, "_position", 0.0 );
                        mlt_properties_set_double( properties, "_frame", 0 );
-                       if ( normalisation == NULL || strcmp( normalisation, "NTSC" ) )
-                       {
-                               mlt_properties_set_double( properties, "fps", 25.0 );
-                               mlt_properties_set_double( properties, "aspect_ratio", 59.0 / 54.0 );
-                       }
-                       else
-                       {
-                               mlt_properties_set_double( properties, "fps", 30000.0 / 1001.0 );
-                               mlt_properties_set_double( properties, "aspect_ratio", 10.0 / 11.0 );
-                       }
+                       mlt_properties_set_double( properties, "aspect_ratio", mlt_profile_sar( NULL ) );
                        mlt_properties_set_double( properties, "_speed", 1.0 );
                        mlt_properties_set_position( properties, "in", 0 );
                        mlt_properties_set_position( properties, "out", 14999 );
@@ -116,7 +112,14 @@ int mlt_producer_init( mlt_producer this, void *child )
 }
 
 /** Listener for property changes.
-*/
+ *
+ * If the in, out, or length properties changed, fire a "producer-changed" event.
+ *
+ * \private \memberof mlt_producer_s
+ * \param owner a service (ignored)
+ * \param this the producer
+ * \param name the property that changed
+ */
 
 static void mlt_producer_property_changed( mlt_service owner, mlt_producer this, char *name )
 {
@@ -125,15 +128,24 @@ static void mlt_producer_property_changed( mlt_service owner, mlt_producer this,
 }
 
 /** Listener for service changes.
-*/
+ *
+ * Fires the "producer-changed" event.
+ *
+ * \private \memberof mlt_producer_s
+ * \param owner a service (ignored)
+ * \param this the producer
+ */
 
 static void mlt_producer_service_changed( mlt_service owner, mlt_producer this )
 {
        mlt_events_fire( MLT_PRODUCER_PROPERTIES( mlt_producer_cut_parent( this ) ), "producer-changed", NULL );
 }
 
-/** Create a new producer.
-*/
+/** Create and initialize a new producer.
+ *
+ * \public \memberof mlt_producer_s
+ * \return the new producer
+ */
 
 mlt_producer mlt_producer_new( )
 {
@@ -143,7 +155,12 @@ mlt_producer mlt_producer_new( )
 }
 
 /** Determine if producer is a cut.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return true if \p this is a "cut" producer
+ * \see mlt_producer_cut
+ */
 
 int mlt_producer_is_cut( mlt_producer this )
 {
@@ -151,7 +168,12 @@ int mlt_producer_is_cut( mlt_producer this )
 }
 
 /** Determine if producer is a mix.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return true if this is a "mix" producer
+ * \todo Define a mix producer.
+ */
 
 int mlt_producer_is_mix( mlt_producer this )
 {
@@ -160,8 +182,14 @@ int mlt_producer_is_mix( mlt_producer this )
        return tractor != NULL;
 }
 
-/** Determine if the producer is a blank [from a playlist].
-*/
+/** Determine if the producer is a blank.
+ *
+ * Blank producers should only appear as an item in a playlist.
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return true if this is a "blank" producer
+ * \see mlt_playlist_insert_blank
+ */
 
 int mlt_producer_is_blank( mlt_producer this )
 {
@@ -169,7 +197,11 @@ int mlt_producer_is_blank( mlt_producer this )
 }
 
 /** Obtain the parent producer.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return either the parent producer if this is a "cut" producer or \p this otherwise.
+ */
 
 mlt_producer mlt_producer_cut_parent( mlt_producer this )
 {
@@ -180,8 +212,17 @@ mlt_producer mlt_producer_cut_parent( mlt_producer this )
                return this;
 }
 
-/** Create a cut of this producer
-*/
+/** Create a cut of this producer.
+ *
+ * A "cut" is a portion of another (parent) producer.
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \param in the beginning
+ * \param out the end
+ * \return the new producer
+ * \todo Expand on the value of a cut.
+ */
 
 mlt_producer mlt_producer_cut( mlt_producer this, int in, int out )
 {
@@ -194,26 +235,26 @@ mlt_producer mlt_producer_cut( mlt_producer this, int in, int out )
        // Special case - allow for a cut of the entire producer (this will squeeze all other cuts to 0)
        if ( in <= 0 )
                in = 0;
-       if ( ( out < 0 || out >= mlt_producer_get_playtime( parent ) ) && !mlt_producer_is_blank( this ) )
-               out = mlt_producer_get_playtime( parent ) - 1;
+       if ( ( out < 0 || out >= mlt_producer_get_length( parent ) ) && !mlt_producer_is_blank( this ) )
+               out = mlt_producer_get_length( parent ) - 1;
 
        mlt_properties_inc_ref( parent_props );
        mlt_properties_set_int( properties, "_cut", 1 );
        mlt_properties_set_data( properties, "_cut_parent", parent, 0, ( mlt_destructor )mlt_producer_close, NULL );
        mlt_properties_set_position( properties, "length", mlt_properties_get_position( parent_props, "length" ) );
+       mlt_properties_set_double( properties, "aspect_ratio", mlt_properties_get_double( parent_props, "aspect_ratio" ) );
        mlt_producer_set_in_and_out( result, in, out );
 
-       // Mini fezzik :-/
-       mlt_filter filter = mlt_factory_filter( "data_feed", "attr_check" );
-       mlt_properties_set_int( MLT_FILTER_PROPERTIES( filter ), "_fezzik", 1 );
-       mlt_producer_attach( result, filter );
-       mlt_filter_close( filter );
-
        return result;
 }
 
 /** Get the parent service object.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return the service parent class
+ * \see MLT_PRODUCER_SERVICE
+ */
 
 mlt_service mlt_producer_service( mlt_producer this )
 {
@@ -221,7 +262,11 @@ mlt_service mlt_producer_service( mlt_producer this )
 }
 
 /** Get the producer properties.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return the producer's property list
+ */
 
 mlt_properties mlt_producer_properties( mlt_producer this )
 {
@@ -229,7 +274,13 @@ mlt_properties mlt_producer_properties( mlt_producer this )
 }
 
 /** Seek to a specified position.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \param position set the "play head" position of the producer
+ * \return false
+ * \todo Document how the properties affect behavior.
+ */
 
 int mlt_producer_seek( mlt_producer this, mlt_position position )
 {
@@ -244,18 +295,18 @@ int mlt_producer_seek( mlt_producer this, mlt_position position )
                mlt_producer_seek( mlt_producer_cut_parent( this ), position + mlt_producer_get_in( this ) );
 
        // Check bounds
-       if ( position < 0 )
+       if ( position < 0 || mlt_producer_get_playtime( this ) == 0 )
        {
                position = 0;
        }
-       else if ( use_points && !strcmp( eof, "pause" ) && position >= mlt_producer_get_playtime( this ) )
+       else if ( use_points && ( eof == NULL || !strcmp( eof, "pause" ) ) && position >= mlt_producer_get_playtime( this ) )
        {
                mlt_producer_set_speed( this, 0 );
                position = mlt_producer_get_playtime( this ) - 1;
        }
        else if ( use_points && !strcmp( eof, "loop" ) && position >= mlt_producer_get_playtime( this ) )
        {
-               position = position % mlt_producer_get_playtime( this );
+               position = (int)position % (int)mlt_producer_get_playtime( this );
        }
 
        // Set the position
@@ -268,7 +319,11 @@ int mlt_producer_seek( mlt_producer this, mlt_position position )
 }
 
 /** Get the current position (relative to in point).
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return the position of the "play head" relative to its beginning
+ */
 
 mlt_position mlt_producer_position( mlt_producer this )
 {
@@ -276,7 +331,11 @@ mlt_position mlt_producer_position( mlt_producer this )
 }
 
 /** Get the current position (relative to start of producer).
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return the position of the "play head" regardless of the in point
+ */
 
 mlt_position mlt_producer_frame( mlt_producer this )
 {
@@ -284,7 +343,12 @@ mlt_position mlt_producer_frame( mlt_producer this )
 }
 
 /** Set the playing speed.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \param speed the new speed as a relative factor (1.0 = normal)
+ * \return
+ */
 
 int mlt_producer_set_speed( mlt_producer this, double speed )
 {
@@ -292,7 +356,11 @@ int mlt_producer_set_speed( mlt_producer this, double speed )
 }
 
 /** Get the playing speed.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return the speed as a relative factor (1.0 = normal)
+ */
 
 double mlt_producer_get_speed( mlt_producer this )
 {
@@ -300,15 +368,34 @@ double mlt_producer_get_speed( mlt_producer this )
 }
 
 /** Get the frames per second.
-*/
+ *
+ * This is determined by the producer's profile.
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return the video refresh rate
+ */
 
 double mlt_producer_get_fps( mlt_producer this )
 {
-       return mlt_properties_get_double( MLT_PRODUCER_PROPERTIES( this ), "fps" );
+       mlt_profile profile = mlt_service_profile( MLT_PRODUCER_SERVICE( this ) );
+       return mlt_profile_fps( profile );
 }
 
 /** Set the in and out points.
-*/
+ *
+ * The in point is where play out should start relative to the natural start
+ * of the underlying file. The out point is where play out should end, also
+ * relative to the start of the underlying file. If the underlying resource is
+ * a live stream, then the in point is an offset relative to first usable
+ * sample.
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \param in the relative starting time
+ * \param out the relative ending time
+ * \return false
+ */
 
 int mlt_producer_set_in_and_out( mlt_producer this, mlt_position in, mlt_position out )
 {
@@ -345,8 +432,12 @@ int mlt_producer_set_in_and_out( mlt_producer this, mlt_position in, mlt_positio
 }
 
 /** Physically reduce the producer (typically a cut) to a 0 length.
-       Essentially, all 0 length cuts should be immediately removed by containers.
-*/
+ *  Essentially, all 0 length cuts should be immediately removed by containers.
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return false
+ */
 
 int mlt_producer_clear( mlt_producer this )
 {
@@ -362,7 +453,11 @@ int mlt_producer_clear( mlt_producer this )
 }
 
 /** Get the in point.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return the in point
+ */
 
 mlt_position mlt_producer_get_in( mlt_producer this )
 {
@@ -370,7 +465,11 @@ mlt_position mlt_producer_get_in( mlt_producer this )
 }
 
 /** Get the out point.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return the out point
+ */
 
 mlt_position mlt_producer_get_out( mlt_producer this )
 {
@@ -378,7 +477,11 @@ mlt_position mlt_producer_get_out( mlt_producer this )
 }
 
 /** Get the total play time.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return the playable (based on in and out points) duration
+ */
 
 mlt_position mlt_producer_get_playtime( mlt_producer this )
 {
@@ -386,7 +489,13 @@ mlt_position mlt_producer_get_playtime( mlt_producer this )
 }
 
 /** Get the total length of the producer.
-*/
+ *
+ * The value returned by a live streaming producer is unknown.
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return the duration of the producer regardless of in and out points
+ */
 
 mlt_position mlt_producer_get_length( mlt_producer this )
 {
@@ -394,7 +503,13 @@ mlt_position mlt_producer_get_length( mlt_producer this )
 }
 
 /** Prepare for next frame.
-*/
+ *
+ * Advance the play out position. If the speed is less than zero, it will
+ * move the play out position in the reverse direction.
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ */
 
 void mlt_producer_prepare_next( mlt_producer this )
 {
@@ -403,7 +518,20 @@ void mlt_producer_prepare_next( mlt_producer this )
 }
 
 /** Get a frame.
-*/
+ *
+ * This is the implementation of the \p get_frame virtual function.
+ * It requests a new frame object from the actual producer for the current
+ * play out position. The producer and its filters can add information and
+ * operations to the frame object in their get_frame handlers.
+ *
+ * \private \memberof mlt_producer_s
+ * \param service a service
+ * \param[out] frame a frame by reference
+ * \param index as determined by the actual producer
+ * \return true if there was an error
+ * \todo Learn more about the details and document how certain properties affect
+ * its behavior.
+ */
 
 static int producer_get_frame( mlt_service service, mlt_frame_ptr frame, int index )
 {
@@ -431,7 +559,7 @@ static int producer_get_frame( mlt_service service, mlt_frame_ptr frame, int ind
                if ( this->get_frame == NULL || ( !strcmp( eof, "continue" ) && mlt_producer_position( this ) > mlt_producer_get_out( this ) ) )
                {
                        // Generate a test frame
-                       *frame = mlt_frame_init( );
+                       *frame = mlt_frame_init( service );
 
                        // Set the position
                        result = mlt_frame_set_position( *frame, mlt_producer_position( this ) );
@@ -452,7 +580,6 @@ static int producer_get_frame( mlt_service service, mlt_frame_ptr frame, int ind
                // Copy the fps and speed of the producer onto the frame
                properties = MLT_FRAME_PROPERTIES( *frame );
                mlt_properties_set_double( properties, "_speed", speed );
-               mlt_properties_set_double( properties, "fps", mlt_producer_get_fps( this ) );
                mlt_properties_set_int( properties, "test_audio", mlt_frame_is_test_audio( *frame ) );
                mlt_properties_set_int( properties, "test_image", mlt_frame_is_test_card( *frame ) );
                if ( mlt_properties_get_data( properties, "_producer", NULL ) == NULL )
@@ -503,12 +630,16 @@ static int producer_get_frame( mlt_service service, mlt_frame_ptr frame, int ind
                // We're done with the clone now
                mlt_properties_set_data( parent_properties, "use_clone", NULL, 0, NULL, NULL );
 
+               // This is useful and required by always_active transitions to determine in/out points of the cut
+               if ( mlt_properties_get_data( MLT_FRAME_PROPERTIES( *frame ), "_producer", NULL ) == MLT_PRODUCER_SERVICE( parent ) )
+                       mlt_properties_set_data( MLT_FRAME_PROPERTIES( *frame ), "_producer", this, 0, NULL, NULL );
+
                mlt_properties_set_double( MLT_FRAME_PROPERTIES( *frame ), "_speed", speed );
                mlt_producer_prepare_next( this );
        }
        else
        {
-               *frame = mlt_frame_init( );
+               *frame = mlt_frame_init( service );
                result = 0;
        }
 
@@ -524,6 +655,8 @@ static int producer_get_frame( mlt_service service, mlt_frame_ptr frame, int ind
                        char *name = mlt_properties_get_name( p_props, i );
                        if ( !strncmp( name, "meta.", 5 ) )
                                mlt_properties_set( f_props, name, mlt_properties_get( p_props, name ) );
+                       else if ( !strncmp( name, "set.", 4 ) )
+                               mlt_properties_set( f_props, name + 4, mlt_properties_get( p_props, name ) );
                }
        }
 
@@ -531,7 +664,12 @@ static int producer_get_frame( mlt_service service, mlt_frame_ptr frame, int ind
 }
 
 /** Attach a filter.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \param filter the filter to attach
+ * \return true if there was an error
+ */
 
 int mlt_producer_attach( mlt_producer this, mlt_filter filter )
 {
@@ -539,7 +677,12 @@ int mlt_producer_attach( mlt_producer this, mlt_filter filter )
 }
 
 /** Detach a filter.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a service
+ * \param filter the filter to detach
+ * \return true if there was an error
+ */
 
 int mlt_producer_detach( mlt_producer this, mlt_filter filter )
 {
@@ -547,7 +690,12 @@ int mlt_producer_detach( mlt_producer this, mlt_filter filter )
 }
 
 /** Retrieve a filter.
-*/
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a service
+ * \param index which filter to retrieve
+ * \return the filter or null if there was an error
+ */
 
 mlt_filter mlt_producer_filter( mlt_producer this, int index )
 {
@@ -555,7 +703,12 @@ mlt_filter mlt_producer_filter( mlt_producer this, int index )
 }
 
 /** Clone this producer.
-*/
+ *
+ * \private \memberof mlt_producer_s
+ * \param this a producer
+ * \return a new producer that is a copy of \p this
+ * \see mlt_producer_set_clones
+ */
 
 static mlt_producer mlt_producer_clone( mlt_producer this )
 {
@@ -563,14 +716,15 @@ static mlt_producer mlt_producer_clone( mlt_producer this )
        mlt_properties properties = MLT_PRODUCER_PROPERTIES( this );
        char *resource = mlt_properties_get( properties, "resource" );
        char *service = mlt_properties_get( properties, "mlt_service" );
+       mlt_profile profile = mlt_service_profile( MLT_PRODUCER_SERVICE( this ) );
 
        mlt_events_block( mlt_factory_event_object( ), mlt_factory_event_object( ) );
 
        if ( service != NULL )
-               clone = mlt_factory_producer( service, resource );
+               clone = mlt_factory_producer( profile, service, resource );
 
        if ( clone == NULL && resource != NULL )
-               clone = mlt_factory_producer( "fezzik", resource );
+               clone = mlt_factory_producer( profile, mlt_environment( "MLT_PRODUCER" ), resource );
 
        if ( clone != NULL )
                mlt_properties_inherit( MLT_PRODUCER_PROPERTIES( clone ), properties );
@@ -581,7 +735,12 @@ static mlt_producer mlt_producer_clone( mlt_producer this )
 }
 
 /** Create clones.
-*/
+ *
+ * \private \memberof mlt_producer_s
+ * \param this a producer
+ * \param clones the number of copies to make
+ * \see mlt_producer_optimise
+ */
 
 static void mlt_producer_set_clones( mlt_producer this, int clones )
 {
@@ -627,10 +786,9 @@ static void mlt_producer_set_clones( mlt_producer this, int clones )
        mlt_properties_set_int( properties, "_clones", clones );
 }
 
-/** Optimise for overlapping cuts from the same clip.
-*/
+/** \brief private to mlt_producer_s, used by mlt_producer_optimise() */
 
-typedef struct 
+typedef struct
 {
        int multitrack;
        int track;
@@ -640,6 +798,8 @@ typedef struct
 }
 track_info;
 
+/** \brief private to mlt_producer_s, used by mlt_producer_optimise() */
+
 typedef struct
 {
        mlt_producer cut;
@@ -747,6 +907,14 @@ static int on_end_multitrack( mlt_parser this, mlt_multitrack object )
        return 0;
 }
 
+/** Optimise for overlapping cuts from the same clip.
+ *
+ * \todo learn more about this
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ * \return true if there was an error
+ */
+
 int mlt_producer_optimise( mlt_producer this )
 {
        int error = 1;
@@ -818,7 +986,15 @@ int mlt_producer_optimise( mlt_producer this )
 }
 
 /** Close the producer.
-*/
+ *
+ * Destroys the producer and deallocates its resources managed by its
+ * properties list. This will call the close virtual function. Therefore, a
+ * subclass that defines its own close function should set its virtual close
+ * function to NULL prior to calling this to avoid circular calls.
+ *
+ * \public \memberof mlt_producer_s
+ * \param this a producer
+ */
 
 void mlt_producer_close( mlt_producer this )
 {