X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=modules%2Fvideo_filter%2Fgradient.c;h=809e53703b5f143766cace420a7a676a9b4735eb;hb=314d91aaf1fcd5807caf3be7ab2c7da78751f837;hp=975c509ef7534286e50cf4bc607fcbd41cdd6dfb;hpb=fa4bde0b26a6c7a2a617362ea0b17144686e39fe;p=vlc diff --git a/modules/video_filter/gradient.c b/modules/video_filter/gradient.c index 975c509ef7..809e53703b 100644 --- a/modules/video_filter/gradient.c +++ b/modules/video_filter/gradient.c @@ -35,9 +35,8 @@ #include #include #include -#include -#include "vlc_filter.h" +#include #include "filter_picture.h" enum { GRADIENT, EDGE, HOUGH }; @@ -71,6 +70,8 @@ static void FilterHough ( filter_t *, picture_t *, picture_t * ); #define CARTOON_LONGTEXT N_("Apply cartoon effect. It is only used by " \ "\"gradient\" and \"edge\".") +#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") }; @@ -79,17 +80,18 @@ static const char *const mode_list_text[] = { N_("Gradient"), N_("Edge"), N_("Ho 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 ); + 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, + add_bool( FILTER_PREFIX "cartoon", true, NULL, CARTOON_TEXT, CARTOON_LONGTEXT, false ) add_shortcut( "gradient" ) @@ -108,6 +110,7 @@ static const char *const ppsz_filter_options[] = { *****************************************************************************/ struct filter_sys_t { + vlc_mutex_t lock; int i_mode; /* For the gradient mode */ @@ -138,7 +141,7 @@ static int Create( vlc_object_t *p_this ) break; default: - msg_Err( p_filter, "Unsupported input chroma (%4s)", + msg_Err( p_filter, "Unsupported input chroma (%4.4s)", (char*)&(p_filter->fmt_in.video.i_chroma) ); return VLC_EGENERIC; } @@ -189,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", @@ -211,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 ); } /***************************************************************************** @@ -240,6 +253,7 @@ static picture_t *Filter( filter_t *p_filter, picture_t *p_pic ) return NULL; } + vlc_mutex_lock( &p_filter->p_sys->lock ); switch( p_filter->p_sys->i_mode ) { case EDGE: @@ -257,6 +271,7 @@ static picture_t *Filter( filter_t *p_filter, picture_t *p_pic ) default: break; } + vlc_mutex_unlock( &p_filter->p_sys->lock ); return CopyInfoAndRelease( p_outpic, p_pic ); } @@ -345,12 +360,8 @@ static void FilterGradient( filter_t *p_filter, picture_t *p_inpic, if( p_filter->p_sys->b_cartoon ) { - vlc_memcpy( p_outpic->p[U_PLANE].p_pixels, - p_inpic->p[U_PLANE].p_pixels, - p_outpic->p[U_PLANE].i_lines * p_outpic->p[U_PLANE].i_pitch ); - vlc_memcpy( p_outpic->p[V_PLANE].p_pixels, - p_inpic->p[V_PLANE].p_pixels, - p_outpic->p[V_PLANE].i_lines * p_outpic->p[V_PLANE].i_pitch ); + plane_CopyPixels( &p_outpic->p[U_PLANE], &p_inpic->p[U_PLANE] ); + plane_CopyPixels( &p_outpic->p[V_PLANE], &p_inpic->p[V_PLANE] ); } else { @@ -489,12 +500,8 @@ static void FilterEdge( filter_t *p_filter, picture_t *p_inpic, if( p_filter->p_sys->b_cartoon ) { - vlc_memcpy( p_outpic->p[U_PLANE].p_pixels, - p_inpic->p[U_PLANE].p_pixels, - p_outpic->p[U_PLANE].i_lines * p_outpic->p[U_PLANE].i_pitch ); - vlc_memcpy( p_outpic->p[V_PLANE].p_pixels, - p_inpic->p[V_PLANE].p_pixels, - p_outpic->p[V_PLANE].i_lines * p_outpic->p[V_PLANE].i_pitch ); + plane_CopyPixels( &p_outpic->p[U_PLANE], &p_inpic->p[U_PLANE] ); + plane_CopyPixels( &p_outpic->p[V_PLANE], &p_inpic->p[V_PLANE] ); } else { @@ -672,15 +679,9 @@ static void FilterHough( filter_t *p_filter, picture_t *p_inpic, vlc_memset( p_hough, 0, i_diag * i_nb_steps * sizeof(int) ); - vlc_memcpy( - p_outpic->p[Y_PLANE].p_pixels, p_inpic->p[Y_PLANE].p_pixels, - p_outpic->p[Y_PLANE].i_lines * p_outpic->p[Y_PLANE].i_pitch ); - vlc_memcpy( - p_outpic->p[U_PLANE].p_pixels, p_inpic->p[U_PLANE].p_pixels, - p_outpic->p[U_PLANE].i_lines * p_outpic->p[U_PLANE].i_pitch ); - vlc_memcpy( - p_outpic->p[V_PLANE].p_pixels, p_inpic->p[V_PLANE].p_pixels, - p_outpic->p[V_PLANE].i_lines * p_outpic->p[V_PLANE].i_pitch ); + plane_CopyPixels( &p_outpic->p[Y_PLANE], &p_inpic->p[Y_PLANE] ); + plane_CopyPixels( &p_outpic->p[U_PLANE], &p_inpic->p[U_PLANE] ); + plane_CopyPixels( &p_outpic->p[V_PLANE], &p_inpic->p[V_PLANE] ); GaussianConvolution( p_inpic, p_smooth ); @@ -757,6 +758,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" ) ) @@ -785,5 +788,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; }