// Override the get_frame method
service->get_frame = filter_get_frame;
+ // Define the destructor
+ service->close = ( mlt_destructor )mlt_filter_close;
+ service->close_object = this;
+
// Default in, out, track properties
mlt_properties_set_position( properties, "in", 0 );
mlt_properties_set_position( properties, "out", 0 );
mlt_service mlt_filter_service( mlt_filter this )
{
- return &this->parent;
+ return this != NULL ? &this->parent : NULL;
}
/** Get the properties associated to this filter.
if ( ret == 0 )
{
mlt_properties properties = mlt_service_properties( &this->parent );
- this->producer = producer;
mlt_properties_set_position( properties, "in", 0 );
mlt_properties_set_position( properties, "out", 0 );
mlt_properties_set_int( properties, "track", index );
int in = mlt_filter_get_in( this );
int out = mlt_filter_get_out( this );
+ // Get the producer this is connected to
+ mlt_service producer = mlt_service_producer( &this->parent );
+
// If the frame request is for this filters track, we need to process it
if ( index == track )
{
- int ret = mlt_service_get_frame( this->producer, frame, index );
+ int ret = mlt_service_get_frame( producer, frame, index );
if ( ret == 0 )
{
mlt_position position = mlt_frame_get_position( *frame );
}
else
{
- return mlt_service_get_frame( this->producer, frame, index );
+ return mlt_service_get_frame( producer, frame, index );
}
}
void mlt_filter_close( mlt_filter this )
{
- if ( this->close != NULL )
- this->close( this );
- else
- mlt_service_close( &this->parent );
- free( this );
+ if ( this != NULL && mlt_properties_dec_ref( mlt_filter_properties( this ) ) <= 0 )
+ {
+ if ( this->close != NULL )
+ {
+ this->close( this );
+ }
+ else
+ {
+ this->parent.close = NULL;
+ mlt_service_close( &this->parent );
+ }
+ free( this );
+ }
}