Hi all,

Looking at a timewarp producer issue in Kdenlive, I noticed that the MLT 
properties method "mlt_properties_pass_list" does not trigger a "property-
changed" event.

In Kdenlive, we use this "mlt_properties_pass_list" method to pass some 
properties on the producer. However, since the "property-changed" event is not 
triggered, the properties are not applied on the Timewarp producer that 
applies changes to the original avformat producer on the "property-changed" 
event.

Is there any reason to not trigger a property-changed event in 
mlt_properties_pass_list ?

If this is meant to be, I can change Kdenlive to use a property set method, 
but otherwise, it seems reasonable to fix it with this 1 line patch :
________________________________________________________________________________

diff --git a/src/framework/mlt_properties.c b/src/framework/mlt_properties.c
index 94afe45f..f3e419b5 100644
--- a/src/framework/mlt_properties.c
+++ b/src/framework/mlt_properties.c
@@ -614,6 +614,7 @@ void mlt_properties_pass_property( mlt_properties self, 
mlt_properties that, con
                return;
 
        mlt_property_pass( mlt_properties_fetch( self, name ), that_prop );
+       mlt_events_fire( self, "property-changed", name, NULL );
 }
 
 /** Copy all properties specified in a comma-separated list to another 
properties list.
________________________________________________________________________________

Thanks in advance for your opinion on this.

Regards,
Jean-Baptiste







_______________________________________________
Mlt-devel mailing list
Mlt-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mlt-devel

Reply via email to