X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=modules%2Fvideo_filter%2Fgradient.c;h=b0585dcdb2a39a5b81f2fe29b116ce55787c8d4b;hb=96067e4922878b125f86509a83c3495670bcb450;hp=87573123d361b3f0c01b34e0423450461a15f776;hpb=27d483e9ef7a451397d7857251c8d67097661f1d;p=vlc diff --git a/modules/video_filter/gradient.c b/modules/video_filter/gradient.c index 87573123d3..b0585dcdb2 100644 --- a/modules/video_filter/gradient.c +++ b/modules/video_filter/gradient.c @@ -1,7 +1,7 @@ /***************************************************************************** * gradient.c : Gradient and edge detection video effects plugin for vlc ***************************************************************************** - * Copyright (C) 2000-2006 the VideoLAN team + * Copyright (C) 2000-2008 the VideoLAN team * $Id$ * * Authors: Samuel Hocevar @@ -26,18 +26,18 @@ * Preamble *****************************************************************************/ -#include /* sin(), cos() */ - #ifdef HAVE_CONFIG_H # include "config.h" #endif -#include +#include /* sin(), cos() */ + +#include #include #include -#include -#include "vlc_filter.h" +#include +#include "filter_picture.h" enum { GRADIENT, EDGE, HOUGH }; @@ -70,43 +70,47 @@ static void FilterHough ( filter_t *, picture_t *, picture_t * ); #define CARTOON_LONGTEXT N_("Apply cartoon effect. It is only used by " \ "\"gradient\" and \"edge\".") -static const char *mode_list[] = { "gradient", "edge", "hough" }; -static const char *mode_list_text[] = { N_("Gradient"), N_("Edge"), N_("Hough") }; +#define GRADIENT_HELP N_("Apply color gradient or edge detection effects") + +static const char *const mode_list[] = { "gradient", "edge", "hough" }; +static const char *const mode_list_text[] = { N_("Gradient"), N_("Edge"), N_("Hough") }; #define FILTER_PREFIX "gradient-" -vlc_module_begin(); - set_description( _("Gradient video filter") ); - set_shortname( _( "Gradient" )); - set_capability( "video filter2", 0 ); - set_category( CAT_VIDEO ); - set_subcategory( SUBCAT_VIDEO_VFILTER ); +vlc_module_begin () + set_description( N_("Gradient video filter") ) + set_shortname( N_( "Gradient" )) + set_help(GRADIENT_HELP) + set_capability( "video filter2", 0 ) + set_category( CAT_VIDEO ) + set_subcategory( SUBCAT_VIDEO_VFILTER ) add_string( FILTER_PREFIX "mode", "gradient", NULL, - MODE_TEXT, MODE_LONGTEXT, false ); - change_string_list( mode_list, mode_list_text, 0 ); + MODE_TEXT, MODE_LONGTEXT, false ) + change_string_list( mode_list, mode_list_text, 0 ) add_integer_with_range( FILTER_PREFIX "type", 0, 0, 1, NULL, - GRADIENT_TEXT, GRADIENT_LONGTEXT, false ); - add_bool( FILTER_PREFIX "cartoon", 1, NULL, - CARTOON_TEXT, CARTOON_LONGTEXT, false ); + GRADIENT_TEXT, GRADIENT_LONGTEXT, false ) + add_bool( FILTER_PREFIX "cartoon", true, NULL, + CARTOON_TEXT, CARTOON_LONGTEXT, false ) - add_shortcut( "gradient" ); - set_callbacks( Create, Destroy ); -vlc_module_end(); + add_shortcut( "gradient" ) + set_callbacks( Create, Destroy ) +vlc_module_end () -static const char *ppsz_filter_options[] = { +static const char *const ppsz_filter_options[] = { "mode", "type", "cartoon", NULL }; /***************************************************************************** - * vout_sys_t: Distort video output method descriptor + * filter_sys_t: Distort video output method descriptor ***************************************************************************** * This structure is part of the video output thread descriptor. * It describes the Distort specific properties of an output thread. *****************************************************************************/ struct filter_sys_t { + vlc_mutex_t lock; int i_mode; /* For the gradient mode */ @@ -131,13 +135,21 @@ static int Create( vlc_object_t *p_this ) filter_t *p_filter = (filter_t *)p_this; char *psz_method; + switch( p_filter->fmt_in.video.i_chroma ) + { + CASE_PLANAR_YUV + break; + + default: + msg_Err( p_filter, "Unsupported input chroma (%4.4s)", + (char*)&(p_filter->fmt_in.video.i_chroma) ); + return VLC_EGENERIC; + } + /* Allocate structure */ p_filter->p_sys = malloc( sizeof( filter_sys_t ) ); if( p_filter->p_sys == NULL ) - { - msg_Err( p_filter, "out of memory" ); return VLC_ENOMEM; - } p_filter->pf_video_filter = Filter; @@ -180,6 +192,7 @@ static int Create( vlc_object_t *p_this ) p_filter->p_sys->b_cartoon = var_CreateGetBoolCommand( p_filter, FILTER_PREFIX "cartoon" ); + vlc_mutex_init( &p_filter->p_sys->lock ); var_AddCallback( p_filter, FILTER_PREFIX "mode", GradientCallback, p_filter->p_sys ); var_AddCallback( p_filter, FILTER_PREFIX "type", @@ -202,13 +215,22 @@ static int Create( vlc_object_t *p_this ) static void Destroy( vlc_object_t *p_this ) { filter_t *p_filter = (filter_t *)p_this; - - free( p_filter->p_sys->p_buf32 ); - free( p_filter->p_sys->p_buf32_bis ); - free( p_filter->p_sys->p_buf8 ); - free( p_filter->p_sys->p_pre_hough ); - - free( p_filter->p_sys ); + filter_sys_t *p_sys = p_filter->p_sys; + + var_DelCallback( p_filter, FILTER_PREFIX "mode", + GradientCallback, p_sys ); + var_DelCallback( p_filter, FILTER_PREFIX "type", + GradientCallback, p_sys ); + var_DelCallback( p_filter, FILTER_PREFIX "cartoon", + GradientCallback, p_sys ); + vlc_mutex_destroy( &p_sys->lock ); + + free( p_sys->p_buf32 ); + free( p_sys->p_buf32_bis ); + free( p_sys->p_buf8 ); + free( p_sys->p_pre_hough ); + + free( p_sys ); } /***************************************************************************** @@ -224,15 +246,14 @@ static picture_t *Filter( filter_t *p_filter, picture_t *p_pic ) if( !p_pic ) return NULL; - p_outpic = p_filter->pf_vout_buffer_new( p_filter ); + p_outpic = filter_NewPicture( p_filter ); if( !p_outpic ) { - msg_Warn( p_filter, "can't get output picture" ); - if( p_pic->pf_release ) - p_pic->pf_release( p_pic ); + picture_Release( p_pic ); return NULL; } + vlc_mutex_lock( &p_filter->p_sys->lock ); switch( p_filter->p_sys->i_mode ) { case EDGE: @@ -250,17 +271,9 @@ static picture_t *Filter( filter_t *p_filter, picture_t *p_pic ) default: break; } + vlc_mutex_unlock( &p_filter->p_sys->lock ); - p_outpic->date = p_pic->date; - p_outpic->b_force = p_pic->b_force; - p_outpic->i_nb_fields = p_pic->i_nb_fields; - p_outpic->b_progressive = p_pic->b_progressive; - p_outpic->b_top_field_first = p_pic->b_top_field_first; - - if( p_pic->pf_release ) - p_pic->pf_release( p_pic ); - - return p_outpic; + return CopyInfoAndRelease( p_outpic, p_pic ); } /***************************************************************************** @@ -637,16 +650,27 @@ static void FilterHough( filter_t *p_filter, picture_t *p_inpic, double d_sin; double d_cos; uint32_t *p_smooth; + int *p_hough = malloc( i_diag * i_nb_steps * sizeof(int) ); if( ! p_hough ) return; + p_smooth = (uint32_t *)malloc( i_num_lines*i_src_visible*sizeof(uint32_t)); - if( !p_smooth ) return; + if( !p_smooth ) + { + free( p_hough ); + return; + } if( ! p_pre_hough ) { msg_Dbg(p_filter, "Starting precalculation"); p_pre_hough = malloc( i_num_lines*i_src_visible*i_nb_steps*sizeof(int)); - if( ! p_pre_hough ) return; + if( ! p_pre_hough ) + { + free( p_smooth ); + free( p_hough ); + return; + } for( i = 0 ; i < i_nb_steps ; i++) { d_sin = sin(d_step * i); @@ -748,6 +772,8 @@ static int GradientCallback( vlc_object_t *p_this, char const *psz_var, { VLC_UNUSED(oldval); filter_sys_t *p_sys = (filter_sys_t *)p_data; + + vlc_mutex_lock( &p_sys->lock ); if( !strcmp( psz_var, FILTER_PREFIX "mode" ) ) { if( !strcmp( newval.psz_string, "gradient" ) ) @@ -776,5 +802,7 @@ static int GradientCallback( vlc_object_t *p_this, char const *psz_var, { p_sys->b_cartoon = newval.b_bool; } + vlc_mutex_unlock( &p_sys->lock ); + return VLC_SUCCESS; }