]> git.sesse.net Git - mlt/blobdiff - src/framework/mlt_tractor.c
Fix calloc() parameter ordering
[mlt] / src / framework / mlt_tractor.c
index 2b7616c4d451075b984726c635b757a23710555e..b39b194937b18b0a63f31d7d95e9625da5fc2c2c 100644 (file)
-/*
- * mlt_tractor.c -- tractor service class
- * Copyright (C) 2003-2004 Ushodaya Enterprises Limited
- * Author: Charles Yates <charles.yates@pandora.be>
+/**
+ * \file mlt_tractor.c
+ * \brief tractor service class
+ * \see mlt_tractor_s
  *
- * 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-2009 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_tractor.h"
 #include "mlt_frame.h"
 #include "mlt_multitrack.h"
 #include "mlt_field.h"
+#include "mlt_log.h"
 
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
+#include <ctype.h>
 
-/** Private structure.
-*/
-
-struct mlt_tractor_s
-{
-       struct mlt_producer_s parent;
-       mlt_service producer;
-};
-
-/** Forward references to static methods.
+/* Forward references to static methods.
 */
 
-static int producer_get_frame( mlt_producer this, mlt_frame_ptr frame, int track );
-static void mlt_tractor_listener( mlt_multitrack tracks, mlt_tractor this );
+static int producer_get_frame( mlt_producer parent, mlt_frame_ptr frame, int track );
+static void mlt_tractor_listener( mlt_multitrack tracks, mlt_tractor self );
 
-/** Constructor for the tractor.
-*/
+/** Construct a tractor without a field or multitrack.
+ *
+ * Sets the resource property to "<tractor>", the mlt_type to "mlt_producer",
+ * and mlt_service to "tractor".
+ *
+ * \public \memberof mlt_tractor_s
+ * \return the new tractor
+ */
 
 mlt_tractor mlt_tractor_init( )
 {
-       mlt_tractor this = calloc( sizeof( struct mlt_tractor_s ), 1 );
-       if ( this != NULL )
+       mlt_tractor self = calloc( 1, sizeof( struct mlt_tractor_s ) );
+       if ( self != NULL )
        {
-               mlt_producer producer = &this->parent;
-               if ( mlt_producer_init( producer, this ) == 0 )
+               mlt_producer producer = &self->parent;
+               if ( mlt_producer_init( producer, self ) == 0 )
                {
-                       mlt_properties_set( mlt_producer_properties( producer ), "resource", "<tractor>" );
-                       mlt_properties_set( mlt_producer_properties( producer ), "mlt_type", "mlt_producer" );
-                       mlt_properties_set( mlt_producer_properties( producer ), "mlt_service", "tractor" );
+                       mlt_properties properties = MLT_PRODUCER_PROPERTIES( producer );
+
+                       mlt_properties_set( properties, "resource", "<tractor>" );
+                       mlt_properties_set( properties, "mlt_type", "mlt_producer" );
+                       mlt_properties_set( properties, "mlt_service", "tractor" );
+                       mlt_properties_set_int( properties, "in", 0 );
+                       mlt_properties_set_int( properties, "out", -1 );
+                       mlt_properties_set_int( properties, "length", 0 );
 
                        producer->get_frame = producer_get_frame;
                        producer->close = ( mlt_destructor )mlt_tractor_close;
-                       producer->close_object = this;
+                       producer->close_object = self;
                }
                else
                {
-                       free( this );
-                       this = NULL;
+                       free( self );
+                       self = NULL;
                }
        }
-       return this;
+       return self;
 }
 
+/** Construct a tractor as well as a field and multitrack.
+ *
+ * Sets the resource property to "<tractor>", the mlt_type to "mlt_producer",
+ * and mlt_service to "tractor".
+ *
+ * \public \memberof mlt_tractor_s
+ * \return the new tractor
+ */
+
 mlt_tractor mlt_tractor_new( )
 {
-       mlt_tractor this = calloc( sizeof( struct mlt_tractor_s ), 1 );
-       if ( this != NULL )
+       mlt_tractor self = calloc( 1, sizeof( struct mlt_tractor_s ) );
+       if ( self != NULL )
        {
-               mlt_producer producer = &this->parent;
-               if ( mlt_producer_init( producer, this ) == 0 )
+               mlt_producer producer = &self->parent;
+               if ( mlt_producer_init( producer, self ) == 0 )
                {
                        mlt_multitrack multitrack = mlt_multitrack_init( );
-                       mlt_field field = mlt_field_new( multitrack, this );
-                       mlt_properties props = mlt_producer_properties( producer );
+                       mlt_field field = mlt_field_new( multitrack, self );
+                       mlt_properties props = MLT_PRODUCER_PROPERTIES( producer );
 
                        mlt_properties_set( props, "resource", "<tractor>" );
                        mlt_properties_set( props, "mlt_type", "mlt_producer" );
@@ -93,169 +107,240 @@ mlt_tractor mlt_tractor_new( )
                        mlt_properties_set_data( props, "multitrack", multitrack, 0, ( mlt_destructor )mlt_multitrack_close, NULL );
                        mlt_properties_set_data( props, "field", field, 0, ( mlt_destructor )mlt_field_close, NULL );
 
-                       mlt_events_listen( mlt_multitrack_properties( multitrack ), this, "producer-changed", ( mlt_listener )mlt_tractor_listener );
+                       mlt_events_listen( MLT_MULTITRACK_PROPERTIES( multitrack ), self, "producer-changed", ( mlt_listener )mlt_tractor_listener );
 
                        producer->get_frame = producer_get_frame;
                        producer->close = ( mlt_destructor )mlt_tractor_close;
-                       producer->close_object = this;
+                       producer->close_object = self;
                }
                else
                {
-                       free( this );
-                       this = NULL;
+                       free( self );
+                       self = NULL;
                }
        }
-       return this;
+       return self;
 }
 
 /** Get the service object associated to the tractor.
-*/
+ *
+ * \public \memberof mlt_tractor_s
+ * \param self a tractor
+ * \return the parent service object
+ * \see MLT_TRACTOR_SERVICE
+ */
 
-mlt_service mlt_tractor_service( mlt_tractor this )
+mlt_service mlt_tractor_service( mlt_tractor self )
 {
-       return mlt_producer_service( &this->parent );
+       return MLT_PRODUCER_SERVICE( &self->parent );
 }
 
 /** Get the producer object associated to the tractor.
-*/
+ *
+ * \public \memberof mlt_tractor_s
+ * \param self a tractor
+ * \return the parent producer object
+ * \see MLT_TRACTOR_PRODUCER
+ */
 
-mlt_producer mlt_tractor_producer( mlt_tractor this )
+mlt_producer mlt_tractor_producer( mlt_tractor self )
 {
-       return this != NULL ? &this->parent : NULL;
+       return self != NULL ? &self->parent : NULL;
 }
 
 /** Get the properties object associated to the tractor.
-*/
+ *
+ * \public \memberof mlt_tractor_s
+ * \param self a tractor
+ * \return the tractor's property list
+ * \see MLT_TRACTOR_PROPERTIES
+ */
 
-mlt_properties mlt_tractor_properties( mlt_tractor this )
+mlt_properties mlt_tractor_properties( mlt_tractor self )
 {
-       return mlt_producer_properties( &this->parent );
+       return MLT_PRODUCER_PROPERTIES( &self->parent );
 }
 
-/** Get the field this tractor is harvesting.
-*/
+/** Get the field self tractor is harvesting.
+ *
+ * \public \memberof mlt_tractor_s
+ * \param self a tractor
+ * \return a field or NULL if there is no field for this tractor
+ */
 
-mlt_field mlt_tractor_field( mlt_tractor this )
+mlt_field mlt_tractor_field( mlt_tractor self )
 {
-       return mlt_properties_get_data( mlt_tractor_properties( this ), "field", NULL );
+       return mlt_properties_get_data( MLT_TRACTOR_PROPERTIES( self ), "field", NULL );
 }
 
-/** Get the multitrack this tractor is pulling.
-*/
+/** Get the multitrack a tractor is pulling.
+ *
+ * \public \memberof mlt_tractor_s
+ * \param self a tractor
+ * \return a multitrack or NULL if there is none
+ */
 
-mlt_multitrack mlt_tractor_multitrack( mlt_tractor this )
+mlt_multitrack mlt_tractor_multitrack( mlt_tractor self )
 {
-       return mlt_properties_get_data( mlt_tractor_properties( this ), "multitrack", NULL );
+       return mlt_properties_get_data( MLT_TRACTOR_PROPERTIES( self ), "multitrack", NULL );
 }
 
 /** Ensure the tractors in/out points match the multitrack.
-*/
+ *
+ * \public \memberof mlt_tractor_s
+ * \param self a tractor
+ */
 
-void mlt_tractor_refresh( mlt_tractor this )
+void mlt_tractor_refresh( mlt_tractor self )
 {
-       mlt_multitrack multitrack = mlt_tractor_multitrack( this );
-       mlt_properties properties = mlt_multitrack_properties( multitrack );
-       mlt_properties self = mlt_tractor_properties( this );
-       mlt_events_block( properties, self );
-       mlt_events_block( self, self );
+       mlt_multitrack multitrack = mlt_tractor_multitrack( self );
+       mlt_properties multitrack_props = MLT_MULTITRACK_PROPERTIES( multitrack );
+       mlt_properties properties = MLT_TRACTOR_PROPERTIES( self );
+       mlt_events_block( multitrack_props, properties );
+       mlt_events_block( properties, properties );
        mlt_multitrack_refresh( multitrack );
-       mlt_properties_set_position( self, "in", 0 );
-       mlt_properties_set_position( self, "out", mlt_properties_get_position( properties, "out" ) );
-       mlt_events_unblock( self, self );
-       mlt_events_unblock( properties, self );
-       mlt_properties_set_position( self, "length", mlt_properties_get_position( properties, "length" ) );
+       mlt_properties_set_position( properties, "in", 0 );
+       mlt_properties_set_position( properties, "out", mlt_properties_get_position( multitrack_props, "out" ) );
+       mlt_events_unblock( properties, properties );
+       mlt_events_unblock( multitrack_props, properties );
+       mlt_properties_set_position( properties, "length", mlt_properties_get_position( multitrack_props, "length" ) );
 }
 
-static void mlt_tractor_listener( mlt_multitrack tracks, mlt_tractor this )
+static void mlt_tractor_listener( mlt_multitrack tracks, mlt_tractor self )
 {
-       mlt_tractor_refresh( this );
+       mlt_tractor_refresh( self );
 }
 
 /** Connect the tractor.
-*/
+ *
+ * \public \memberof mlt_tractor_s
+ * \param self a tractor
+ * \param producer a producer
+ * \return true on error
+ */
 
-int mlt_tractor_connect( mlt_tractor this, mlt_service producer )
+int mlt_tractor_connect( mlt_tractor self, mlt_service producer )
 {
-       int ret = mlt_service_connect_producer( mlt_tractor_service( this ), producer, 0 );
+       int ret = mlt_service_connect_producer( MLT_TRACTOR_SERVICE( self ), producer, 0 );
 
        // This is the producer we're going to connect to
        if ( ret == 0 )
-               this->producer = producer;
+               self->producer = producer;
 
        return ret;
 }
 
 /** Set the producer for a specific track.
-*/
+ *
+ * \public \memberof mlt_tractor_s
+ * \param self a tractor
+ * \param producer a producer
+ * \param index the 0-based track index
+ * \return true on error
+ */
 
-int mlt_tractor_set_track( mlt_tractor this, mlt_producer producer, int index )
+int mlt_tractor_set_track( mlt_tractor self, mlt_producer producer, int index )
 {
-       return mlt_multitrack_connect( mlt_tractor_multitrack( this ), producer, index );
+       return mlt_multitrack_connect( mlt_tractor_multitrack( self ), producer, index );
 }
 
 /** Get the producer for a specific track.
-*/
+ *
+ * \public \memberof mlt_tractor_s
+ * \param self a tractor
+ * \param index the 0-based track index
+ * \return the producer for track \p index
+ */
 
-mlt_producer mlt_tractor_get_track( mlt_tractor this, int index )
+mlt_producer mlt_tractor_get_track( mlt_tractor self, int index )
 {
-       return mlt_multitrack_track( mlt_tractor_multitrack( this ), index );
+       return mlt_multitrack_track( mlt_tractor_multitrack( self ), index );
 }
 
-static int producer_get_image( mlt_frame this, uint8_t **buffer, mlt_image_format *format, int *width, int *height, int writable )
+static int producer_get_image( mlt_frame self, uint8_t **buffer, mlt_image_format *format, int *width, int *height, int writable )
 {
        uint8_t *data = NULL;
-       mlt_properties properties = mlt_frame_properties( this );
-       mlt_frame frame = mlt_frame_pop_service( this );
-       mlt_properties frame_properties = mlt_frame_properties( frame );
-       mlt_properties_set_int( frame_properties, "width", mlt_properties_get_int( properties, "width" ) );
-       mlt_properties_set_int( frame_properties, "height", mlt_properties_get_int( properties, "height" ) );
+       int size = 0;
+       mlt_properties properties = MLT_FRAME_PROPERTIES( self );
+       mlt_frame frame = mlt_frame_pop_service( self );
+       mlt_properties frame_properties = MLT_FRAME_PROPERTIES( frame );
        mlt_properties_set( frame_properties, "rescale.interp", mlt_properties_get( properties, "rescale.interp" ) );
-       if ( mlt_properties_get( properties, "distort" ) )
-               mlt_properties_set( frame_properties, "distort", mlt_properties_get( properties, "distort" ) );
-       mlt_properties_set_double( frame_properties, "aspect_ratio", mlt_properties_get_double( properties, "aspect_ratio" ) );
-       mlt_properties_set_double( frame_properties, "consumer_aspect_ratio", mlt_properties_get_double( properties, "consumer_aspect_ratio" ) );
-       mlt_properties_set_int( frame_properties, "consumer_deinterlace", mlt_properties_get_double( properties, "consumer_deinterlace" ) );
+       mlt_properties_set_int( frame_properties, "resize_alpha", mlt_properties_get_int( properties, "resize_alpha" ) );
+       mlt_properties_set_int( frame_properties, "distort", mlt_properties_get_int( properties, "distort" ) );
+       mlt_properties_set_int( frame_properties, "consumer_deinterlace", mlt_properties_get_int( properties, "consumer_deinterlace" ) );
+       mlt_properties_set( frame_properties, "deinterlace_method", mlt_properties_get( properties, "deinterlace_method" ) );
+       mlt_properties_set_int( frame_properties, "consumer_tff", mlt_properties_get_int( properties, "consumer_tff" ) );
        mlt_frame_get_image( frame, buffer, format, width, height, writable );
-       mlt_properties_set_data( properties, "image", *buffer, *width * *height * 2, NULL, NULL );
+       mlt_frame_set_image( self, *buffer, 0, NULL );
        mlt_properties_set_int( properties, "width", *width );
        mlt_properties_set_int( properties, "height", *height );
+       mlt_properties_set_int( properties, "format", *format );
        mlt_properties_set_double( properties, "aspect_ratio", mlt_frame_get_aspect_ratio( frame ) );
        mlt_properties_set_int( properties, "progressive", mlt_properties_get_int( frame_properties, "progressive" ) );
-       if ( mlt_properties_get( frame_properties, "distort" ) )
-               mlt_properties_set( properties, "distort", mlt_properties_get( frame_properties, "distort" ) );
+       mlt_properties_set_int( properties, "distort", mlt_properties_get_int( frame_properties, "distort" ) );
+       mlt_properties_set_int( properties, "colorspace", mlt_properties_get_int( frame_properties, "colorspace" ) );
+       mlt_properties_set_int( properties, "force_full_luma", mlt_properties_get_int( frame_properties, "force_full_luma" ) );
+       mlt_properties_set_int( properties, "top_field_first", mlt_properties_get_int( frame_properties, "top_field_first" ) );
        data = mlt_frame_get_alpha_mask( frame );
-       mlt_properties_set_data( properties, "alpha", data, 0, NULL, NULL );
+       mlt_properties_get_data( frame_properties, "alpha", &size );
+       mlt_frame_set_alpha( self, data, size, NULL );
+       self->convert_image = frame->convert_image;
+       self->convert_audio = frame->convert_audio;
        return 0;
 }
 
-static int producer_get_audio( mlt_frame this, int16_t **buffer, mlt_audio_format *format, int *frequency, int *channels, int *samples )
+static int producer_get_audio( mlt_frame self, void **buffer, mlt_audio_format *format, int *frequency, int *channels, int *samples )
 {
-       mlt_properties properties = mlt_frame_properties( this );
-       mlt_frame frame = mlt_frame_pop_audio( this );
+       mlt_properties properties = MLT_FRAME_PROPERTIES( self );
+       mlt_frame frame = mlt_frame_pop_audio( self );
        mlt_frame_get_audio( frame, buffer, format, frequency, channels, samples );
-       mlt_properties_set_data( properties, "audio", *buffer, 0, NULL, NULL );
-       mlt_properties_set_int( properties, "frequency", *frequency );
-       mlt_properties_set_int( properties, "channels", *channels );
+       mlt_frame_set_audio( self, *buffer, *format, mlt_audio_format_size( *format, *samples, *channels ), NULL );
+       mlt_properties_set_int( properties, "audio_frequency", *frequency );
+       mlt_properties_set_int( properties, "audio_channels", *channels );
+       mlt_properties_set_int( properties, "audio_samples", *samples );
        return 0;
 }
 
+static void destroy_data_queue( void *arg )
+{
+       if ( arg != NULL )
+       {
+               // Assign the correct type
+               mlt_deque queue = arg;
+
+               // Iterate through each item and destroy them
+               while ( mlt_deque_peek_front( queue ) != NULL )
+                       mlt_properties_close( mlt_deque_pop_back( queue ) );
+
+               // Close the deque
+               mlt_deque_close( queue );
+       }
+}
+
 /** Get the next frame.
-*/
+ *
+ * \private \memberof mlt_tractor_s
+ * \param parent the producer interface to the tractor
+ * \param[out] frame a frame by reference
+ * \param track the 0-based track index
+ * \return true on error
+ */
 
 static int producer_get_frame( mlt_producer parent, mlt_frame_ptr frame, int track )
 {
-       mlt_tractor this = parent->child;
+       mlt_tractor self = parent->child;
 
        // We only respond to the first track requests
-       if ( track == 0 && this->producer != NULL )
+       if ( track == 0 && self->producer != NULL )
        {
                int i = 0;
                int done = 0;
                mlt_frame temp = NULL;
                int count = 0;
+               int image_count = 0;
 
                // Get the properties of the parent producer
-               mlt_properties properties = mlt_producer_properties( parent );
+               mlt_properties properties = MLT_PRODUCER_PROPERTIES( parent );
 
                // Try to obtain the multitrack associated to the tractor
                mlt_multitrack multitrack = mlt_properties_get_data( properties, "multitrack", NULL );
@@ -263,9 +348,15 @@ static int producer_get_frame( mlt_producer parent, mlt_frame_ptr frame, int tra
                // Or a specific producer
                mlt_producer producer = mlt_properties_get_data( properties, "producer", NULL );
 
-               // If we don't have one, we're in trouble... 
+               // Determine whether this tractor feeds to the consumer or stops here
+               int global_feed = mlt_properties_get_int( properties, "global_feed" );
+
+               // If we don't have one, we're in trouble...
                if ( multitrack != NULL )
                {
+                       // The output frame will hold the 'global' data feeds (ie: those which are targetted for the final frame)
+                       mlt_deque data_queue = mlt_deque_init( );
+
                        // Used to garbage collect all frames
                        char label[ 30 ];
 
@@ -278,86 +369,161 @@ static int producer_get_frame( mlt_producer parent, mlt_frame_ptr frame, int tra
                        // We'll store audio and video frames to use here
                        mlt_frame audio = NULL;
                        mlt_frame video = NULL;
-
-                       // Determine which data_queue to pass on...
-                       void *data_queue = NULL;
+                       mlt_frame first_video = NULL;
 
                        // Temporary properties
                        mlt_properties temp_properties = NULL;
 
                        // Get the multitrack's producer
-                       mlt_producer target = mlt_multitrack_producer( multitrack );
+                       mlt_producer target = MLT_MULTITRACK_PRODUCER( multitrack );
                        mlt_producer_seek( target, mlt_producer_frame( parent ) );
                        mlt_producer_set_speed( target, mlt_producer_get_speed( parent ) );
 
                        // We will create one frame and attach everything to it
-                       *frame = mlt_frame_init( );
+                       *frame = mlt_frame_init( MLT_PRODUCER_SERVICE( parent ) );
 
                        // Get the properties of the frame
-                       frame_properties = mlt_frame_properties( *frame );
+                       frame_properties = MLT_FRAME_PROPERTIES( *frame );
 
                        // Loop through each of the tracks we're harvesting
                        for ( i = 0; !done; i ++ )
                        {
                                // Get a frame from the producer
-                               mlt_service_get_frame( this->producer, &temp, i );
+                               mlt_service_get_frame( self->producer, &temp, i );
 
                                // Get the temporary properties
-                               temp_properties = mlt_frame_properties( temp );
+                               temp_properties = MLT_FRAME_PROPERTIES( temp );
+
+                               // Pass all unique meta properties from the producer's frame to the new frame
+                               mlt_properties_lock( temp_properties );
+                               int props_count = mlt_properties_count( temp_properties );
+                               int j;
+                               for ( j = 0; j < props_count; j ++ )
+                               {
+                                       char *name = mlt_properties_get_name( temp_properties, j );
+                                       if ( !strncmp( name, "meta.", 5 ) && !mlt_properties_get( frame_properties, name ) )
+                                               mlt_properties_set( frame_properties, name, mlt_properties_get_value( temp_properties, j ) );
+                               }
+                               mlt_properties_unlock( temp_properties );
+
+                               // Copy the format conversion virtual functions
+                               if ( ! (*frame)->convert_image && temp->convert_image )
+                                       (*frame)->convert_image = temp->convert_image;
+                               if ( ! (*frame)->convert_audio && temp->convert_audio )
+                                       (*frame)->convert_audio = temp->convert_audio;
 
                                // Check for last track
                                done = mlt_properties_get_int( temp_properties, "last_track" );
 
+                               // Handle fx only tracks
+                               if ( mlt_properties_get_int( temp_properties, "fx_cut" ) )
+                               {
+                                       int hide = ( video == NULL ? 1 : 0 ) | ( audio == NULL ? 2 : 0 );
+                                       mlt_properties_set_int( temp_properties, "hide", hide );
+                               }
+
                                // We store all frames with a destructor on the output frame
                                sprintf( label, "_%s_%d", id, count ++ );
                                mlt_properties_set_data( frame_properties, label, temp, 0, ( mlt_destructor )mlt_frame_close, NULL );
 
-                               // We want the last data_queue 
+                               // We want to append all 'final' feeds to the global queue
                                if ( !done && mlt_properties_get_data( temp_properties, "data_queue", NULL ) != NULL )
-                                       data_queue = mlt_properties_get_data( mlt_frame_properties( temp ), "data_queue", NULL );
+                               {
+                                       // Move the contents of this queue on to the output frames data queue
+                                       mlt_deque sub_queue = mlt_properties_get_data( MLT_FRAME_PROPERTIES( temp ), "data_queue", NULL );
+                                       mlt_deque temp = mlt_deque_init( );
+                                       while ( global_feed && mlt_deque_count( sub_queue ) )
+                                       {
+                                               mlt_properties p = mlt_deque_pop_back( sub_queue );
+                                               if ( mlt_properties_get_int( p, "final" ) )
+                                                       mlt_deque_push_back( data_queue, p );
+                                               else
+                                                       mlt_deque_push_back( temp, p );
+                                       }
+                                       while( mlt_deque_count( temp ) )
+                                               mlt_deque_push_front( sub_queue, mlt_deque_pop_back( temp ) );
+                                       mlt_deque_close( temp );
+                               }
+
+                               // Now do the same with the global queue but without the conditional behaviour
+                               if ( mlt_properties_get_data( temp_properties, "global_queue", NULL ) != NULL )
+                               {
+                                       mlt_deque sub_queue = mlt_properties_get_data( MLT_FRAME_PROPERTIES( temp ), "global_queue", NULL );
+                                       while ( mlt_deque_count( sub_queue ) )
+                                       {
+                                               mlt_properties p = mlt_deque_pop_back( sub_queue );
+                                               mlt_deque_push_back( data_queue, p );
+                                       }
+                               }
 
                                // Pick up first video and audio frames
                                if ( !done && !mlt_frame_is_test_audio( temp ) && !( mlt_properties_get_int( temp_properties, "hide" ) & 2 ) )
+                               {
+                                       // Order of frame creation is starting to get problematic
+                                       if ( audio != NULL )
+                                       {
+                                               mlt_deque_push_front( MLT_FRAME_AUDIO_STACK( temp ), producer_get_audio );
+                                               mlt_deque_push_front( MLT_FRAME_AUDIO_STACK( temp ), audio );
+                                       }
                                        audio = temp;
+                               }
                                if ( !done && !mlt_frame_is_test_card( temp ) && !( mlt_properties_get_int( temp_properties, "hide" ) & 1 ) )
+                               {
+                                       if ( video != NULL )
+                                       {
+                                               mlt_deque_push_front( MLT_FRAME_IMAGE_STACK( temp ), producer_get_image );
+                                               mlt_deque_push_front( MLT_FRAME_IMAGE_STACK( temp ), video );
+                                       }
                                        video = temp;
+                                       if ( first_video == NULL )
+                                               first_video = temp;
+
+                                       mlt_properties_set_int( MLT_FRAME_PROPERTIES( temp ), "image_count", ++ image_count );
+                                       image_count = 1;
+                               }
                        }
-       
+
                        // Now stack callbacks
                        if ( audio != NULL )
                        {
                                mlt_frame_push_audio( *frame, audio );
-                               ( *frame )->get_audio = producer_get_audio;
+                               mlt_frame_push_audio( *frame, producer_get_audio );
                        }
 
                        if ( video != NULL )
                        {
-                               mlt_properties video_properties = mlt_frame_properties( video );
+                               mlt_properties video_properties = MLT_FRAME_PROPERTIES( first_video );
                                mlt_frame_push_service( *frame, video );
                                mlt_frame_push_service( *frame, producer_get_image );
-                               mlt_properties_set_data( frame_properties, "data_queue", data_queue, 0, NULL, NULL );
+                               if ( global_feed )
+                                       mlt_properties_set_data( frame_properties, "data_queue", data_queue, 0, NULL, NULL );
+                               mlt_properties_set_data( video_properties, "global_queue", data_queue, 0, destroy_data_queue, NULL );
                                mlt_properties_set_int( frame_properties, "width", mlt_properties_get_int( video_properties, "width" ) );
                                mlt_properties_set_int( frame_properties, "height", mlt_properties_get_int( video_properties, "height" ) );
-                               mlt_properties_set_int( frame_properties, "real_width", mlt_properties_get_int( video_properties, "real_width" ) );
-                               mlt_properties_set_int( frame_properties, "real_height", mlt_properties_get_int( video_properties, "real_height" ) );
+                               mlt_properties_pass_list( frame_properties, video_properties, "meta.media.width, meta.media.height" );
                                mlt_properties_set_int( frame_properties, "progressive", mlt_properties_get_int( video_properties, "progressive" ) );
                                mlt_properties_set_double( frame_properties, "aspect_ratio", mlt_properties_get_double( video_properties, "aspect_ratio" ) );
+                               mlt_properties_set_int( frame_properties, "image_count", image_count );
+                       }
+                       else
+                       {
+                               destroy_data_queue( data_queue );
                        }
 
                        mlt_frame_set_position( *frame, mlt_producer_frame( parent ) );
-                       mlt_properties_set_int( mlt_frame_properties( *frame ), "test_audio", audio == NULL );
-                       mlt_properties_set_int( mlt_frame_properties( *frame ), "test_image", video == NULL );
+                       mlt_properties_set_int( MLT_FRAME_PROPERTIES( *frame ), "test_audio", audio == NULL );
+                       mlt_properties_set_int( MLT_FRAME_PROPERTIES( *frame ), "test_image", video == NULL );
                }
                else if ( producer != NULL )
                {
                        mlt_producer_seek( producer, mlt_producer_frame( parent ) );
                        mlt_producer_set_speed( producer, mlt_producer_get_speed( parent ) );
-                       mlt_service_get_frame( this->producer, frame, track );
+                       mlt_service_get_frame( self->producer, frame, track );
                }
                else
                {
-                       fprintf( stderr, "tractor without a multitrack!!\n" );
-                       mlt_service_get_frame( this->producer, frame, track );
+                       mlt_log( MLT_PRODUCER_SERVICE( parent ), MLT_LOG_ERROR, "tractor without a multitrack!!\n" );
+                       mlt_service_get_frame( self->producer, frame, track );
                }
 
                // Prepare the next frame
@@ -369,21 +535,24 @@ static int producer_get_frame( mlt_producer parent, mlt_frame_ptr frame, int tra
        else
        {
                // Generate a test card
-               *frame = mlt_frame_init( );
+               *frame = mlt_frame_init( MLT_PRODUCER_SERVICE( parent ) );
                return 0;
        }
 }
 
-/** Close the tractor.
-*/
+/** Close the tractor and free its resources.
+ *
+ * \public \memberof mlt_tractor_s
+ * \param self a tractor
+ */
 
-void mlt_tractor_close( mlt_tractor this )
+void mlt_tractor_close( mlt_tractor self )
 {
-       if ( this != NULL && mlt_properties_dec_ref( mlt_tractor_properties( this ) ) <= 0 )
+       if ( self != NULL && mlt_properties_dec_ref( MLT_TRACTOR_PROPERTIES( self ) ) <= 0 )
        {
-               this->parent.close = NULL;
-               mlt_producer_close( &this->parent );
-               free( this );
+               self->parent.close = NULL;
+               mlt_producer_close( &self->parent );
+               free( self );
        }
 }