]> git.sesse.net Git - mlt/blobdiff - src/framework/mlt_producer.c
mlt_factory.c: guard against accessing mlt_environment before it is ready
[mlt] / src / framework / mlt_producer.c
index 55f85ada23c7d1c04ba040b1b033c661f480a71b..7f95e8584bc3be9113f030a169534e1fa71f55aa 100644 (file)
@@ -3,19 +3,19 @@
  * Copyright (C) 2003-2004 Ushodaya Enterprises Limited
  * Author: Charles Yates <charles.yates@pandora.be>
  *
- * 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.
+ * 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 program is distributed in the hope that it will be useful,
+ * 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"
@@ -23,6 +23,8 @@
 #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>
@@ -66,9 +68,6 @@ int mlt_producer_init( mlt_producer this, void *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;
        
@@ -86,16 +85,7 @@ int mlt_producer_init( mlt_producer this, void *child )
                        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 );
@@ -194,21 +184,16 @@ 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;
 }
 
@@ -244,18 +229,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
@@ -304,7 +289,8 @@ double mlt_producer_get_speed( mlt_producer this )
 
 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.
@@ -431,7 +417,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 +438,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 )
@@ -512,7 +497,7 @@ static int producer_get_frame( mlt_service service, mlt_frame_ptr frame, int ind
        }
        else
        {
-               *frame = mlt_frame_init( );
+               *frame = mlt_frame_init( service );
                result = 0;
        }
 
@@ -528,6 +513,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 ) );
                }
        }
 
@@ -567,14 +554,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 );