Attempt at an aspect ratio clean up

[mlt] / src / framework / mlt_consumer.c

diff --git a/src/framework/mlt_consumer.c b/src/framework/mlt_consumer.c
index 2ba290133d63d4445b529b05f57fe27bc6376766..ae79cf9db2cbeb28ca0a8b2adba64170078d74f2 100644 (file)
--- a/src/framework/mlt_consumer.c
+++ b/src/framework/mlt_consumer.c
@@ -55,7 +55,7 @@ int mlt_consumer_init( mlt_consumer this, void *child )
                        mlt_properties_set_int( properties, "width", 720 );
                        mlt_properties_set_int( properties, "height", 576 );
                        mlt_properties_set_int( properties, "progressive", 0 );
-                       mlt_properties_set_double( properties, "aspect_ratio", 128.0 / 117.0 );
+                       mlt_properties_set_double( properties, "aspect_ratio", 59.0 / 54.0 );
                }
                else
                {
@@ -64,7 +64,7 @@ int mlt_consumer_init( mlt_consumer this, void *child )
                        mlt_properties_set_int( properties, "width", 720 );
                        mlt_properties_set_int( properties, "height", 480 );
                        mlt_properties_set_int( properties, "progressive", 0 );
-                       mlt_properties_set_double( properties, "aspect_ratio", 72.0 / 79.0 );
+                       mlt_properties_set_double( properties, "aspect_ratio", 10.0 / 11.0 );
                }
 
                // Default rescaler for all consumers
@@ -88,6 +88,11 @@ int mlt_consumer_init( mlt_consumer this, void *child )
 
                mlt_events_register( properties, "consumer-frame-show", ( mlt_transmitter )mlt_consumer_frame_show );
                mlt_events_register( properties, "consumer-stopped", NULL );
+
+               // Create the push mutex and condition
+               pthread_mutex_init( &this->put_mutex, NULL );
+               pthread_cond_init( &this->put_cond, NULL );
+
        }
        return error;
 }
@@ -188,6 +193,37 @@ int mlt_consumer_start( mlt_consumer this )
        return 0;
 }
 
+/** An alternative method to feed frames into the consumer - only valid if
+       the consumer itself is not connected.
+*/
+
+int mlt_consumer_put_frame( mlt_consumer this, mlt_frame frame )
+{
+       int error = 1;
+
+       // Get the service assoicated to the consumer
+       mlt_service service = mlt_consumer_service( this );
+
+       if ( mlt_service_producer( service ) == NULL )
+       {
+               pthread_mutex_lock( &this->put_mutex );
+               if ( this->put != NULL )
+                       pthread_cond_wait( &this->put_cond, &this->put_mutex );
+               if ( this->put == NULL )
+                       this->put = frame;
+               else
+                       mlt_frame_close( frame );
+               pthread_cond_broadcast( &this->put_cond );
+               pthread_mutex_unlock( &this->put_mutex );
+       }
+       else
+       {
+               mlt_frame_close( frame );
+       }
+
+       return error;
+}
+
 /** Protected method for consumer to get frames from connected service
 */
 
@@ -199,12 +235,25 @@ mlt_frame mlt_consumer_get_frame( mlt_consumer this )
        // Get the service assoicated to the consumer
        mlt_service service = mlt_consumer_service( this );
 
+       // Get the consumer properties
+       mlt_properties properties = mlt_consumer_properties( this );
+
        // Get the frame
-       if ( mlt_service_get_frame( service, &frame, 0 ) == 0 )
+       if ( mlt_service_producer( service ) == NULL && mlt_properties_get_int( properties, "put_mode" ) )
        {
-               // Get the consumer properties
-               mlt_properties properties = mlt_consumer_properties( this );
+               pthread_mutex_lock( &this->put_mutex );
+               if ( this->put == NULL )
+                       pthread_cond_wait( &this->put_cond, &this->put_mutex );
+               frame = this->put;
+               this->put = NULL;
+               pthread_cond_broadcast( &this->put_cond );
+               pthread_mutex_unlock( &this->put_mutex );
+               if ( frame != NULL )
+                       mlt_service_apply_filters( service, frame, 0 );
+       }
 
+       if ( frame != NULL || mlt_service_get_frame( service, &frame, 0 ) == 0 )
+       {
                // Get the frame properties
                mlt_properties frame_properties = mlt_frame_properties( frame );
 
@@ -415,6 +464,11 @@ static void consumer_read_ahead_stop( mlt_consumer this )
                pthread_cond_broadcast( &this->cond );
                pthread_mutex_unlock( &this->mutex );
 
+               // Broadcast to the put condition in case it's waiting
+               pthread_mutex_lock( &this->put_mutex );
+               pthread_cond_broadcast( &this->put_cond );
+               pthread_mutex_unlock( &this->put_mutex );
+
                // Join the thread
                pthread_join( this->ahead_thread, NULL );
 
@@ -513,6 +567,11 @@ int mlt_consumer_stop( mlt_consumer this )
        if ( mlt_properties_get_int( properties, "real_time" ) )
                consumer_read_ahead_stop( this );
 
+       // Just in case...
+       pthread_mutex_lock( &this->put_mutex );
+       pthread_cond_broadcast( &this->put_cond );
+       pthread_mutex_unlock( &this->put_mutex );
+
        // Kill the test card
        mlt_properties_set_data( properties, "test_card_producer", NULL, 0, NULL, NULL );
 
@@ -520,6 +579,12 @@ int mlt_consumer_stop( mlt_consumer this )
        if ( mlt_properties_get( properties, "post" ) )
                system( mlt_properties_get( properties, "post" ) );
 
+       if ( this->put != NULL )
+       {
+               mlt_frame_close( this->put );
+               this->put = NULL;
+       }
+
        return 0;
 }
 
@@ -549,6 +614,11 @@ void mlt_consumer_close( mlt_consumer this )
                this->close = NULL;
                this->parent.close = NULL;
 
+               // Destroy the push mutex and condition
+               pthread_cond_broadcast( &this->put_cond );
+               pthread_mutex_destroy( &this->put_mutex );
+               pthread_cond_destroy( &this->put_cond );
+
                // Call the childs close if available
                if ( consumer_close != NULL )
                        consumer_close( this );