X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=src%2Fvideo_output%2Fvout_intf.c;h=7460763f5e582dc75eeac21912c57e7bd154b5a1;hb=d727c4590f83751a2f28fcdfd8af3886ebeb45d8;hp=5efca5deac8ca5ca1173f7d071c5d461de265308;hpb=c5f75658bf4867d0f3028c74cc5df1ef0e9349a2;p=vlc diff --git a/src/video_output/vout_intf.c b/src/video_output/vout_intf.c index 5efca5deac..7460763f5e 100644 --- a/src/video_output/vout_intf.c +++ b/src/video_output/vout_intf.c @@ -1,8 +1,7 @@ /***************************************************************************** * vout_intf.c : video output interface ***************************************************************************** - * Copyright (C) 2000-2004 the VideoLAN team - * $Id$ + * Copyright (C) 2000-2007 the VideoLAN team * * Authors: Gildas Bazin * @@ -18,21 +17,38 @@ * * 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, USA. + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA. *****************************************************************************/ /***************************************************************************** * Preamble *****************************************************************************/ + +#ifdef HAVE_CONFIG_H +# include "config.h" +#endif + +#include + +#include #include /* free() */ +#include /* opendir() */ +#include /* opendir() */ +#include -#include -#include +#include +#include +#include -#include "vlc_video.h" -#include "video_output.h" -#include "vlc_image.h" -#include "vlc_spu.h" +#include +#include +#include +#include +#include + +#include +#include +#include "../libvlc.h" /***************************************************************************** * Local prototypes @@ -53,6 +69,13 @@ static int FullscreenCallback( vlc_object_t *, char const *, static int SnapshotCallback( vlc_object_t *, char const *, vlc_value_t, vlc_value_t, void * ); +static int TitleShowCallback( vlc_object_t *, char const *, + vlc_value_t, vlc_value_t, void * ); +static int TitleTimeoutCallback( vlc_object_t *, char const *, + vlc_value_t, vlc_value_t, void * ); +static int TitlePositionCallback( vlc_object_t *, char const *, + vlc_value_t, vlc_value_t, void * ); + /***************************************************************************** * vout_RequestWindow: Create/Get a video window if possible. ***************************************************************************** @@ -65,132 +88,183 @@ void *vout_RequestWindow( vout_thread_t *p_vout, unsigned int *pi_width_hint, unsigned int *pi_height_hint ) { - intf_thread_t *p_intf = NULL; - vlc_list_t *p_list; - void *p_window; - vlc_value_t val; - int i; - /* Small kludge */ if( !var_Type( p_vout, "aspect-ratio" ) ) vout_IntfInit( p_vout ); /* Get requested coordinates */ - var_Get( p_vout, "video-x", &val ); - *pi_x_hint = val.i_int ; - var_Get( p_vout, "video-y", &val ); - *pi_y_hint = val.i_int; + *pi_x_hint = var_GetInteger( p_vout, "video-x" ); + *pi_y_hint = var_GetInteger( p_vout, "video-y" ); *pi_width_hint = p_vout->i_window_width; *pi_height_hint = p_vout->i_window_height; /* Check whether someone provided us with a window ID */ - var_Get( p_vout->p_vlc, "drawable", &val ); - if( val.i_int ) return (void *)val.i_int; + int drawable = var_CreateGetInteger( p_vout, "drawable" ); + if( drawable ) return (void *)(intptr_t)drawable; - /* Find if the main interface supports embedding */ - p_list = vlc_list_find( p_vout, VLC_OBJECT_INTF, FIND_ANYWHERE ); - if( !p_list ) return NULL; + vout_window_t *wnd = vlc_custom_create (VLC_OBJECT(p_vout), sizeof (*wnd), + VLC_OBJECT_GENERIC, "window"); + if (wnd == NULL) + return NULL; - for( i = 0; i < p_list->i_count; i++ ) - { - p_intf = (intf_thread_t *)p_list->p_values[i].p_object; - if( p_intf->b_block && p_intf->pf_request_window ) break; - p_intf = NULL; - } + wnd->vout = p_vout; + wnd->width = *pi_width_hint; + wnd->height = *pi_height_hint; + wnd->pos_x = *pi_x_hint; + wnd->pos_y = *pi_y_hint; + vlc_object_attach (wnd, p_vout); - if( !p_intf ) + wnd->module = module_Need (wnd, "vout window", 0, 0); + if (wnd->module == NULL) { - vlc_list_release( p_list ); + msg_Dbg (wnd, "no window provider available"); + vlc_object_release (wnd); return NULL; } - - vlc_object_yield( p_intf ); - vlc_list_release( p_list ); - - p_window = p_intf->pf_request_window( p_intf, p_vout, pi_x_hint, pi_y_hint, - pi_width_hint, pi_height_hint ); - - if( !p_window ) vlc_object_release( p_intf ); - else p_vout->p_parent_intf = p_intf; - - return p_window; + p_vout->p_window = wnd; + *pi_width_hint = wnd->width; + *pi_height_hint = wnd->height; + *pi_x_hint = wnd->pos_x; + *pi_y_hint = wnd->pos_y; + return wnd->handle; } -void vout_ReleaseWindow( vout_thread_t *p_vout, void *p_window ) +void vout_ReleaseWindow( vout_thread_t *p_vout, void *dummy ) { - intf_thread_t *p_intf = p_vout->p_parent_intf; - - if( !p_intf ) return; - - vlc_mutex_lock( &p_intf->object_lock ); - if( p_intf->b_dead ) - { - vlc_mutex_unlock( &p_intf->object_lock ); - return; - } + vout_window_t *wnd = p_vout->p_window; - if( !p_intf->pf_release_window ) - { - msg_Err( p_vout, "no pf_release_window"); - vlc_mutex_unlock( &p_intf->object_lock ); - vlc_object_release( p_intf ); + if (wnd == NULL) return; - } + p_vout->p_window = NULL; - p_intf->pf_release_window( p_intf, p_window ); + assert (wnd->module); + module_Unneed (wnd, wnd->module); - p_vout->p_parent_intf = NULL; - vlc_mutex_unlock( &p_intf->object_lock ); - vlc_object_release( p_intf ); + vlc_object_release (wnd); + (void)dummy; } -int vout_ControlWindow( vout_thread_t *p_vout, void *p_window, +int vout_ControlWindow( vout_thread_t *p_vout, void *dummy, int i_query, va_list args ) { - intf_thread_t *p_intf = p_vout->p_parent_intf; - int i_ret; - - if( !p_intf ) return VLC_EGENERIC; + vout_window_t *wnd = p_vout->p_window; - vlc_mutex_lock( &p_intf->object_lock ); - if( p_intf->b_dead ) - { - vlc_mutex_unlock( &p_intf->object_lock ); + if (wnd == NULL) return VLC_EGENERIC; - } - if( !p_intf->pf_control_window ) - { - msg_Err( p_vout, "no pf_control_window"); - vlc_mutex_unlock( &p_intf->object_lock ); - return VLC_EGENERIC; - } - - i_ret = p_intf->pf_control_window( p_intf, p_window, i_query, args ); - vlc_mutex_unlock( &p_intf->object_lock ); - return i_ret; + assert (wnd->control); + return wnd->control (wnd, i_query, args); } /***************************************************************************** * vout_IntfInit: called during the vout creation to initialise misc things. *****************************************************************************/ +static const struct +{ + double f_value; + const char *psz_label; +} p_zoom_values[] = { + { 0.25, N_("1:4 Quarter") }, + { 0.5, N_("1:2 Half") }, + { 1, N_("1:1 Original") }, + { 2, N_("2:1 Double") }, + { 0, NULL } }; + +static const struct +{ + const char *psz_value; + const char *psz_label; +} p_crop_values[] = { + { "", N_("Default") }, + { "16:10", "16:10" }, + { "16:9", "16:9" }, + { "185:100", "1.85:1" }, + { "221:100", "2.21:1" }, + { "235:100", "2.35:1" }, + { "239:100", "2.39:1" }, + { "5:3", "5:3" }, + { "4:3", "4:3" }, + { "5:4", "5:4" }, + { "1:1", "1:1" }, + { NULL, NULL } }; + +static const struct +{ + const char *psz_value; + const char *psz_label; +} p_aspect_ratio_values[] = { + { "", N_("Default") }, + { "1:1", "1:1" }, + { "4:3", "4:3" }, + { "16:9", "16:9" }, + { "16:10", "16:10" }, + { "221:100", "2.21:1" }, + { "5:4", "5:4" }, + { NULL, NULL } }; + +static void AddCustomRatios( vout_thread_t *p_vout, const char *psz_var, + char *psz_list ) +{ + if( psz_list && *psz_list ) + { + char *psz_cur = psz_list; + char *psz_next; + while( psz_cur && *psz_cur ) + { + vlc_value_t val, text; + psz_next = strchr( psz_cur, ',' ); + if( psz_next ) + { + *psz_next = '\0'; + psz_next++; + } + val.psz_string = psz_cur; + text.psz_string = psz_cur; + var_Change( p_vout, psz_var, VLC_VAR_ADDCHOICE, &val, &text); + psz_cur = psz_next; + } + } +} + void vout_IntfInit( vout_thread_t *p_vout ) { vlc_value_t val, text, old_val; - vlc_bool_t b_force_par = VLC_FALSE; + bool b_force_par = false; + char *psz_buf; + int i; /* Create a few object variables we'll need later on */ var_Create( p_vout, "snapshot-path", VLC_VAR_STRING | VLC_VAR_DOINHERIT ); + var_Create( p_vout, "snapshot-prefix", VLC_VAR_STRING | VLC_VAR_DOINHERIT ); var_Create( p_vout, "snapshot-format", VLC_VAR_STRING | VLC_VAR_DOINHERIT ); + var_Create( p_vout, "snapshot-preview", VLC_VAR_BOOL | VLC_VAR_DOINHERIT ); + var_Create( p_vout, "snapshot-sequential", + VLC_VAR_BOOL | VLC_VAR_DOINHERIT ); + var_Create( p_vout, "snapshot-num", VLC_VAR_INTEGER ); + var_SetInteger( p_vout, "snapshot-num", 1 ); + var_Create( p_vout, "snapshot-width", VLC_VAR_INTEGER | VLC_VAR_DOINHERIT ); + var_Create( p_vout, "snapshot-height", VLC_VAR_INTEGER | VLC_VAR_DOINHERIT ); + var_Create( p_vout, "width", VLC_VAR_INTEGER | VLC_VAR_DOINHERIT ); var_Create( p_vout, "height", VLC_VAR_INTEGER | VLC_VAR_DOINHERIT ); - var_Create( p_vout, "align", VLC_VAR_INTEGER | VLC_VAR_DOINHERIT ); - var_Get( p_vout, "align", &val ); - p_vout->i_alignment = val.i_int; + p_vout->i_alignment = var_CreateGetInteger( p_vout, "align" ); var_Create( p_vout, "video-x", VLC_VAR_INTEGER | VLC_VAR_DOINHERIT ); var_Create( p_vout, "video-y", VLC_VAR_INTEGER | VLC_VAR_DOINHERIT ); + var_Create( p_vout, "mouse-hide-timeout", + VLC_VAR_INTEGER | VLC_VAR_DOINHERIT ); + + p_vout->b_title_show = var_CreateGetBool( p_vout, "video-title-show" ); + p_vout->i_title_timeout = + (mtime_t)var_CreateGetInteger( p_vout, "video-title-timeout" ); + p_vout->i_title_position = + var_CreateGetInteger( p_vout, "video-title-position" ); + + var_AddCallback( p_vout, "video-title-show", TitleShowCallback, NULL ); + var_AddCallback( p_vout, "video-title-timeout", TitleTimeoutCallback, NULL ); + var_AddCallback( p_vout, "video-title-position", TitlePositionCallback, NULL ); + /* Zoom object var */ var_Create( p_vout, "zoom", VLC_VAR_FLOAT | VLC_VAR_ISCOMMAND | VLC_VAR_HASCHOICE | VLC_VAR_DOINHERIT ); @@ -199,52 +273,61 @@ void vout_IntfInit( vout_thread_t *p_vout ) var_Change( p_vout, "zoom", VLC_VAR_SETTEXT, &text, NULL ); var_Get( p_vout, "zoom", &old_val ); - if( old_val.f_float == 0.25 || - old_val.f_float == 0.5 || - old_val.f_float == 1 || - old_val.f_float == 2 ) + + for( i = 0; p_zoom_values[i].f_value; i++ ) { - var_Change( p_vout, "zoom", VLC_VAR_DELCHOICE, &old_val, NULL ); + if( old_val.f_float == p_zoom_values[i].f_value ) + var_Change( p_vout, "zoom", VLC_VAR_DELCHOICE, &old_val, NULL ); + val.f_float = p_zoom_values[i].f_value; + text.psz_string = _( p_zoom_values[i].psz_label ); + var_Change( p_vout, "zoom", VLC_VAR_ADDCHOICE, &val, &text ); } - val.f_float = 0.25; text.psz_string = _("1:4 Quarter"); - var_Change( p_vout, "zoom", VLC_VAR_ADDCHOICE, &val, &text ); - val.f_float = 0.5; text.psz_string = _("1:2 Half"); - var_Change( p_vout, "zoom", VLC_VAR_ADDCHOICE, &val, &text ); - val.f_float = 1; text.psz_string = _("1:1 Original"); - var_Change( p_vout, "zoom", VLC_VAR_ADDCHOICE, &val, &text ); - val.f_float = 2; text.psz_string = _("2:1 Double"); - var_Change( p_vout, "zoom", VLC_VAR_ADDCHOICE, &val, &text ); - - var_Set( p_vout, "zoom", old_val ); + var_Set( p_vout, "zoom", old_val ); /* Is this really needed? */ var_AddCallback( p_vout, "zoom", ZoomCallback, NULL ); + /* Crop offset vars */ + var_Create( p_vout, "crop-left", VLC_VAR_INTEGER | VLC_VAR_ISCOMMAND ); + var_Create( p_vout, "crop-top", VLC_VAR_INTEGER | VLC_VAR_ISCOMMAND ); + var_Create( p_vout, "crop-right", VLC_VAR_INTEGER | VLC_VAR_ISCOMMAND ); + var_Create( p_vout, "crop-bottom", VLC_VAR_INTEGER | VLC_VAR_ISCOMMAND ); + + var_AddCallback( p_vout, "crop-left", CropCallback, NULL ); + var_AddCallback( p_vout, "crop-top", CropCallback, NULL ); + var_AddCallback( p_vout, "crop-right", CropCallback, NULL ); + var_AddCallback( p_vout, "crop-bottom", CropCallback, NULL ); + /* Crop object var */ - var_Create( p_vout, "crop", VLC_VAR_STRING | + var_Create( p_vout, "crop", VLC_VAR_STRING | VLC_VAR_ISCOMMAND | VLC_VAR_HASCHOICE | VLC_VAR_DOINHERIT ); text.psz_string = _("Crop"); var_Change( p_vout, "crop", VLC_VAR_SETTEXT, &text, NULL ); - val.psz_string = ""; + val.psz_string = (char*)""; var_Change( p_vout, "crop", VLC_VAR_DELCHOICE, &val, 0 ); - val.psz_string = ""; text.psz_string = _("Default"); - var_Change( p_vout, "crop", VLC_VAR_ADDCHOICE, &val, &text ); - val.psz_string = "001:1"; text.psz_string = _("1:1"); - var_Change( p_vout, "crop", VLC_VAR_ADDCHOICE, &val, &text ); - val.psz_string = "004:3"; text.psz_string = _("4:3"); - var_Change( p_vout, "crop", VLC_VAR_ADDCHOICE, &val, &text ); - val.psz_string = "16:9"; text.psz_string = _("16:9"); - var_Change( p_vout, "crop", VLC_VAR_ADDCHOICE, &val, &text ); - val.psz_string = "221:100"; text.psz_string = _("221:100"); - var_Change( p_vout, "crop", VLC_VAR_ADDCHOICE, &val, &text ); + + for( i = 0; p_crop_values[i].psz_value; i++ ) + { + val.psz_string = (char*)p_crop_values[i].psz_value; + text.psz_string = _( p_crop_values[i].psz_label ); + var_Change( p_vout, "crop", VLC_VAR_ADDCHOICE, &val, &text ); + } + + /* update triggered every time the vout's crop parameters are changed */ + var_Create( p_vout, "crop-update", VLC_VAR_VOID ); + + /* Add custom crop ratios */ + psz_buf = config_GetPsz( p_vout, "custom-crop-ratios" ); + AddCustomRatios( p_vout, "crop", psz_buf ); + free( psz_buf ); var_AddCallback( p_vout, "crop", CropCallback, NULL ); var_Get( p_vout, "crop", &old_val ); if( old_val.psz_string && *old_val.psz_string ) var_Change( p_vout, "crop", VLC_VAR_TRIGGER_CALLBACKS, 0, 0 ); - if( old_val.psz_string ) free( old_val.psz_string ); + free( old_val.psz_string ); /* Monitor pixel aspect-ratio */ var_Create( p_vout, "monitor-par", VLC_VAR_STRING | VLC_VAR_DOINHERIT ); @@ -256,8 +339,8 @@ void vout_IntfInit( vout_thread_t *p_vout ) float i_aspect = 0; if( psz_parser ) { - i_aspect_num = strtol( val.psz_string, 0, 0 ); - i_aspect_den = strtol( ++psz_parser, 0, 0 ); + i_aspect_num = strtol( val.psz_string, 0, 10 ); + i_aspect_den = strtol( ++psz_parser, 0, 10 ); } else { @@ -273,44 +356,47 @@ void vout_IntfInit( vout_thread_t *p_vout ) vlc_ureduce( &p_vout->i_par_num, &p_vout->i_par_den, p_vout->i_par_num, p_vout->i_par_den, 0 ); - msg_Dbg( p_vout, "monitor pixel aspect-ratio overriding: %i:%i", + msg_Dbg( p_vout, "overriding monitor pixel aspect-ratio: %i:%i", p_vout->i_par_num, p_vout->i_par_den ); - b_force_par = VLC_TRUE; + b_force_par = true; } - if( val.psz_string ) free( val.psz_string ); + free( val.psz_string ); /* Aspect-ratio object var */ - var_Create( p_vout, "aspect-ratio", VLC_VAR_STRING | + var_Create( p_vout, "aspect-ratio", VLC_VAR_STRING | VLC_VAR_ISCOMMAND | VLC_VAR_HASCHOICE | VLC_VAR_DOINHERIT ); text.psz_string = _("Aspect-ratio"); var_Change( p_vout, "aspect-ratio", VLC_VAR_SETTEXT, &text, NULL ); - val.psz_string = ""; + val.psz_string = (char*)""; var_Change( p_vout, "aspect-ratio", VLC_VAR_DELCHOICE, &val, 0 ); - val.psz_string = ""; text.psz_string = _("Default"); - var_Change( p_vout, "aspect-ratio", VLC_VAR_ADDCHOICE, &val, &text ); - val.psz_string = "001:1"; text.psz_string = _("1:1"); - var_Change( p_vout, "aspect-ratio", VLC_VAR_ADDCHOICE, &val, &text ); - val.psz_string = "004:3"; text.psz_string = _("4:3"); - var_Change( p_vout, "aspect-ratio", VLC_VAR_ADDCHOICE, &val, &text ); - val.psz_string = "16:9"; text.psz_string = _("16:9"); - var_Change( p_vout, "aspect-ratio", VLC_VAR_ADDCHOICE, &val, &text ); - val.psz_string = "221:100"; text.psz_string = _("221:100"); - var_Change( p_vout, "aspect-ratio", VLC_VAR_ADDCHOICE, &val, &text ); + + for( i = 0; p_aspect_ratio_values[i].psz_value; i++ ) + { + val.psz_string = (char*)p_aspect_ratio_values[i].psz_value; + text.psz_string = _( p_aspect_ratio_values[i].psz_label ); + var_Change( p_vout, "aspect-ratio", VLC_VAR_ADDCHOICE, &val, &text ); + } + + /* Add custom aspect ratios */ + psz_buf = config_GetPsz( p_vout, "custom-aspect-ratios" ); + AddCustomRatios( p_vout, "aspect-ratio", psz_buf ); + free( psz_buf ); var_AddCallback( p_vout, "aspect-ratio", AspectCallback, NULL ); var_Get( p_vout, "aspect-ratio", &old_val ); if( (old_val.psz_string && *old_val.psz_string) || b_force_par ) var_Change( p_vout, "aspect-ratio", VLC_VAR_TRIGGER_CALLBACKS, 0, 0 ); - if( old_val.psz_string ) free( old_val.psz_string ); + free( old_val.psz_string ); /* Initialize the dimensions of the video window */ InitWindowSize( p_vout, &p_vout->i_window_width, &p_vout->i_window_height ); /* Add a variable to indicate if the window should be on top of others */ - var_Create( p_vout, "video-on-top", VLC_VAR_BOOL | VLC_VAR_DOINHERIT ); + var_Create( p_vout, "video-on-top", VLC_VAR_BOOL | VLC_VAR_DOINHERIT + | VLC_VAR_ISCOMMAND ); text.psz_string = _("Always on top"); var_Change( p_vout, "video-on-top", VLC_VAR_SETTEXT, &text, NULL ); var_AddCallback( p_vout, "video-on-top", OnTopCallback, NULL ); @@ -319,15 +405,13 @@ void vout_IntfInit( vout_thread_t *p_vout ) var_Create( p_vout, "video-deco", VLC_VAR_BOOL | VLC_VAR_DOINHERIT ); /* Add a fullscreen variable */ - var_Create( p_vout, "fullscreen", VLC_VAR_BOOL ); - text.psz_string = _("Fullscreen"); - var_Change( p_vout, "fullscreen", VLC_VAR_SETTEXT, &text, NULL ); - var_Change( p_vout, "fullscreen", VLC_VAR_INHERITVALUE, &val, NULL ); - if( val.b_bool ) + if( var_CreateGetBoolCommand( p_vout, "fullscreen" ) ) { /* user requested fullscreen */ p_vout->i_changes |= VOUT_FULLSCREEN_CHANGE; } + text.psz_string = _("Fullscreen"); + var_Change( p_vout, "fullscreen", VLC_VAR_SETTEXT, &text, NULL ); var_AddCallback( p_vout, "fullscreen", FullscreenCallback, NULL ); /* Add a snapshot variable */ @@ -344,8 +428,7 @@ void vout_IntfInit( vout_thread_t *p_vout ) var_Create( p_vout, "mouse-clicked", VLC_VAR_INTEGER ); var_Create( p_vout, "intf-change", VLC_VAR_BOOL ); - val.b_bool = VLC_TRUE; - var_Set( p_vout, "intf-change", val ); + var_SetBool( p_vout, "intf-change", true ); } /***************************************************************************** @@ -354,13 +437,17 @@ void vout_IntfInit( vout_thread_t *p_vout ) int vout_Snapshot( vout_thread_t *p_vout, picture_t *p_pic ) { image_handler_t *p_image = image_HandlerCreate( p_vout ); - video_format_t fmt_in = {0}, fmt_out = {0}; - char *psz_filename; + video_format_t fmt_in, fmt_out; + char *psz_filename = NULL; subpicture_t *p_subpic; picture_t *p_pif; vlc_value_t val, format; + DIR *path; int i_ret; + memset( &fmt_in, 0, sizeof(video_format_t) ); + memset( &fmt_out, 0, sizeof(video_format_t) ); + var_Get( p_vout, "snapshot-path", &val ); if( val.psz_string && !*val.psz_string ) { @@ -368,21 +455,107 @@ int vout_Snapshot( vout_thread_t *p_vout, picture_t *p_pic ) val.psz_string = 0; } -#if defined(SYS_DARWIN) || defined(SYS_BEOS) - if( !val.psz_string && p_vout->p_vlc->psz_homedir ) + /* Embedded snapshot : if snapshot-path == object:object-id, then + create a snapshot_t* and store it in + object(object-id)->p_private, then unlock and signal the + waiting object. + */ + if( val.psz_string && !strncmp( val.psz_string, "object:", 7 ) ) { - asprintf( &val.psz_string, "%s/Desktop", - p_vout->p_vlc->psz_homedir ); + int i_id; + vlc_object_t* p_dest; + block_t *p_block; + snapshot_t *p_snapshot; + int i_size; + + /* Destination object-id is following object: */ + i_id = atoi( &val.psz_string[7] ); + p_dest = ( vlc_object_t* )vlc_object_get( i_id ); + if( !p_dest ) + { + msg_Err( p_vout, "Cannot find calling object" ); + image_HandlerDelete( p_image ); + return VLC_EGENERIC; + } + /* Object must be locked. We will unlock it once we get the + snapshot and written it to p_private */ + p_dest->p_private = NULL; + + /* Save the snapshot to a memory zone */ + fmt_in = p_vout->fmt_in; + fmt_out.i_sar_num = fmt_out.i_sar_den = 1; + fmt_out.i_width = var_GetInteger( p_vout, "snapshot-width" ); + fmt_out.i_height = var_GetInteger( p_vout, "snapshot-height" ); + fmt_out.i_chroma = VLC_FOURCC( 'p','n','g',' ' ); + + p_block = ( block_t* ) image_Write( p_image, p_pic, &fmt_in, &fmt_out ); + if( !p_block ) + { + msg_Err( p_vout, "Could not get snapshot" ); + image_HandlerDelete( p_image ); + vlc_object_signal_maybe( VLC_OBJECT(p_dest) ); + vlc_object_release( p_dest ); + return VLC_EGENERIC; + } + + /* Copy the p_block data to a snapshot structure */ + /* FIXME: get the timestamp */ + p_snapshot = ( snapshot_t* ) malloc( sizeof( snapshot_t ) ); + if( !p_snapshot ) + { + block_Release( p_block ); + image_HandlerDelete( p_image ); + vlc_object_signal_maybe( VLC_OBJECT(p_dest) ); + vlc_object_release( p_dest ); + return VLC_ENOMEM; + } + + i_size = p_block->i_buffer; + + p_snapshot->i_width = fmt_out.i_width; + p_snapshot->i_height = fmt_out.i_height; + p_snapshot->i_datasize = i_size; + p_snapshot->date = p_block->i_pts; /* FIXME ?? */ + p_snapshot->p_data = ( char* ) malloc( i_size ); + if( !p_snapshot->p_data ) + { + block_Release( p_block ); + free( p_snapshot ); + image_HandlerDelete( p_image ); + vlc_object_signal_maybe( VLC_OBJECT(p_dest) ); + vlc_object_release( p_dest ); + return VLC_ENOMEM; + } + memcpy( p_snapshot->p_data, p_block->p_buffer, p_block->i_buffer ); + + p_dest->p_private = p_snapshot; + + block_Release( p_block ); + + /* Unlock the object */ + vlc_object_signal_maybe( VLC_OBJECT(p_dest) ); + vlc_object_release( p_dest ); + + image_HandlerDelete( p_image ); + return VLC_SUCCESS; + } + +#if defined(__APPLE__) || defined(SYS_BEOS) + if( !val.psz_string ) + { + if( asprintf( &val.psz_string, "%s/Desktop", + config_GetHomeDir() ) == -1 ) + val.psz_string = NULL; } #elif defined(WIN32) && !defined(UNDER_CE) - if( !val.psz_string && p_vout->p_vlc->psz_homedir ) + if( !val.psz_string ) { /* Get the My Pictures folder path */ char *p_mypicturesdir = NULL; typedef HRESULT (WINAPI *SHGETFOLDERPATH)( HWND, int, HANDLE, DWORD, - LPSTR ); + LPWSTR ); #ifndef CSIDL_FLAG_CREATE # define CSIDL_FLAG_CREATE 0x8000 #endif @@ -399,67 +572,137 @@ int vout_Snapshot( vout_thread_t *p_vout, picture_t *p_pic ) /* load the shfolder dll to retrieve SHGetFolderPath */ if( ( shfolder_dll = LoadLibrary( _T("SHFolder.dll") ) ) != NULL ) { - SHGetFolderPath = (void *)GetProcAddress( shfolder_dll, - _T("SHGetFolderPathA") ); - if( SHGetFolderPath != NULL ) - { - p_mypicturesdir = (char *)malloc( MAX_PATH ); - if( p_mypicturesdir ) - { + wchar_t wdir[PATH_MAX]; + SHGetFolderPath = (void *)GetProcAddress( shfolder_dll, + _T("SHGetFolderPathW") ); + if ((SHGetFolderPath != NULL ) + && SUCCEEDED (SHGetFolderPath (NULL, + CSIDL_MYPICTURES | CSIDL_FLAG_CREATE, + NULL, SHGFP_TYPE_CURRENT, + wdir))) + p_mypicturesdir = FromWide (wdir); - if( S_OK != SHGetFolderPath( NULL, - CSIDL_MYPICTURES | CSIDL_FLAG_CREATE, - NULL, SHGFP_TYPE_CURRENT, - p_mypicturesdir ) ) - { - free( p_mypicturesdir ); - p_mypicturesdir = NULL; - } - } - } FreeLibrary( shfolder_dll ); } if( p_mypicturesdir == NULL ) { - asprintf( &val.psz_string, "%s/" CONFIG_DIR, - p_vout->p_vlc->psz_homedir ); + if( asprintf( &val.psz_string, "%s", config_GetHomeDir() ) == -1 ) + val.psz_string = NULL; } else { - asprintf( &val.psz_string, p_mypicturesdir ); + if( asprintf( &val.psz_string, p_mypicturesdir ) == -1 ) + val.psz_string = NULL; free( p_mypicturesdir ); } } #else - if( !val.psz_string && p_vout->p_vlc->psz_homedir ) + /* XXX: This saves in the data directory. Shouldn't we try saving + * to psz_homedir/Desktop or something nicer ? */ + char *psz_datadir = config_GetUserDataDir(); + if( !val.psz_string && psz_datadir ) { - asprintf( &val.psz_string, "%s/" CONFIG_DIR, - p_vout->p_vlc->psz_homedir ); + if( asprintf( &val.psz_string, "%s", psz_datadir ) == -1 ) + val.psz_string = NULL; } + free( psz_datadir ); #endif if( !val.psz_string ) { - msg_Err( p_vout, "no directory specified for snapshots" ); + msg_Err( p_vout, "no path specified for snapshots" ); + image_HandlerDelete( p_image ); return VLC_EGENERIC; } var_Get( p_vout, "snapshot-format", &format ); if( !format.psz_string || !*format.psz_string ) { - if( format.psz_string ) free( format.psz_string ); + free( format.psz_string ); format.psz_string = strdup( "png" ); } - asprintf( &psz_filename, "%s/vlcsnap-%u.%s", val.psz_string, - (unsigned int)(p_pic->date / 100000) & 0xFFFFFF, - format.psz_string ); + /* + * Did the user specify a directory? If not, path = NULL. + */ + path = utf8_opendir ( (const char *)val.psz_string ); + if( path != NULL ) + { + char *psz_prefix = var_GetNonEmptyString( p_vout, "snapshot-prefix" ); + if( psz_prefix == NULL ) + psz_prefix = strdup( "vlcsnap-" ); + else + { + char *psz_tmp = str_format( p_vout, psz_prefix ); + filename_sanitize( psz_tmp ); + free( psz_prefix ); + psz_prefix = psz_tmp; + } + + closedir( path ); + if( var_GetBool( p_vout, "snapshot-sequential" ) == true ) + { + int i_num = var_GetInteger( p_vout, "snapshot-num" ); + FILE *p_file; + do + { + if( asprintf( &psz_filename, "%s" DIR_SEP "%s%05d.%s", + val.psz_string, psz_prefix, i_num++, + format.psz_string ) == -1 ) + { + msg_Err( p_vout, "could not create snapshot" ); + image_HandlerDelete( p_image ); + return VLC_EGENERIC; + } + } + while( ( p_file = utf8_fopen( psz_filename, "r" ) ) && !fclose( p_file ) ); + var_SetInteger( p_vout, "snapshot-num", i_num ); + } + else + { + if( asprintf( &psz_filename, "%s" DIR_SEP "%s%u.%s", + val.psz_string, psz_prefix, + (unsigned int)(p_pic->date / 100000) & 0xFFFFFF, + format.psz_string ) == -1 ) + { + msg_Err( p_vout, "could not create snapshot" ); + image_HandlerDelete( p_image ); + return VLC_EGENERIC; + } + } + + free( psz_prefix ); + } + else // The user specified a full path name (including file name) + { + psz_filename = str_format( p_vout, val.psz_string ); + path_sanitize( psz_filename ); + } + free( val.psz_string ); free( format.psz_string ); - /* Save the snapshot */ + fmt_out.i_width = var_GetInteger( p_vout, "snapshot-width" ); + fmt_out.i_height = var_GetInteger( p_vout, "snapshot-height" ); + fmt_in = p_vout->fmt_in; + + if( fmt_out.i_width == 0 && fmt_out.i_height > 0 ) + { + fmt_out.i_width = (fmt_in.i_width * fmt_out.i_height) / fmt_in.i_height; + } + else if( fmt_out.i_height == 0 && fmt_out.i_width > 0 ) + { + fmt_out.i_height = (fmt_in.i_height * fmt_out.i_width) / fmt_in.i_width; + } + else + { + fmt_out.i_width = fmt_in.i_width; + fmt_out.i_height = fmt_in.i_height; + } + + /* Save the snapshot */ fmt_out.i_sar_num = fmt_out.i_sar_den = 1; i_ret = image_WriteUrl( p_image, p_pic, &fmt_in, &fmt_out, psz_filename ); if( i_ret != VLC_SUCCESS ) @@ -471,37 +714,104 @@ int vout_Snapshot( vout_thread_t *p_vout, picture_t *p_pic ) } msg_Dbg( p_vout, "snapshot taken (%s)", psz_filename ); + vout_OSDMessage( VLC_OBJECT( p_vout ), DEFAULT_CHAN, + "%s", psz_filename ); free( psz_filename ); - /* Inject a subpicture with the snapshot */ - memset( &fmt_out, 0, sizeof(fmt_out) ); - fmt_out.i_chroma = VLC_FOURCC('Y','U','V','A'); - p_pif = image_Convert( p_image, p_pic, &fmt_in, &fmt_out ); - image_HandlerDelete( p_image ); - if( !p_pif ) return VLC_EGENERIC; + if( var_GetBool( p_vout, "snapshot-preview" ) ) + { + /* Inject a subpicture with the snapshot */ + memset( &fmt_out, 0, sizeof(fmt_out) ); + fmt_out.i_chroma = VLC_FOURCC('Y','U','V','A'); + p_pif = image_Convert( p_image, p_pic, &fmt_in, &fmt_out ); + image_HandlerDelete( p_image ); + if( !p_pif ) return VLC_EGENERIC; + + p_subpic = spu_CreateSubpicture( p_vout->p_spu ); + if( p_subpic == NULL ) + { + p_pif->pf_release( p_pif ); + return VLC_EGENERIC; + } + + p_subpic->i_channel = 0; + p_subpic->i_start = mdate(); + p_subpic->i_stop = mdate() + 4000000; + p_subpic->b_ephemer = true; + p_subpic->b_fade = true; + p_subpic->i_original_picture_width = p_vout->render.i_width * 4; + p_subpic->i_original_picture_height = p_vout->render.i_height * 4; - p_subpic = spu_CreateSubpicture( p_vout->p_spu ); - if( p_subpic == NULL ) + p_subpic->p_region = spu_CreateRegion( p_vout->p_spu, &fmt_out ); + vout_CopyPicture( p_image->p_parent, &p_subpic->p_region->picture, + p_pif ); + p_pif->pf_release( p_pif ); + + spu_DisplaySubpicture( p_vout->p_spu, p_subpic ); + } + else { - p_pif->pf_release( p_pif ); - return VLC_EGENERIC; + image_HandlerDelete( p_image ); } - p_subpic->i_channel = 0; - p_subpic->i_start = mdate(); - p_subpic->i_stop = mdate() + 4000000; - p_subpic->b_ephemer = VLC_TRUE; - p_subpic->b_fade = VLC_TRUE; - p_subpic->i_original_picture_width = p_vout->render.i_width * 4; - p_subpic->i_original_picture_height = p_vout->render.i_height * 4; + return VLC_SUCCESS; +} - p_subpic->p_region = spu_CreateRegion( p_vout->p_spu, &fmt_out ); - vout_CopyPicture( p_image->p_parent, &p_subpic->p_region->picture, p_pif ); - p_pif->pf_release( p_pif ); +/***************************************************************************** + * Handle filters + *****************************************************************************/ - spu_DisplaySubpicture( p_vout->p_spu, p_subpic ); +void vout_EnableFilter( vout_thread_t *p_vout, char *psz_name, + bool b_add, bool b_setconfig ) +{ + char *psz_parser; + char *psz_string = config_GetPsz( p_vout, "vout-filter" ); - return VLC_SUCCESS; + /* Todo : Use some generic chain manipulation functions */ + if( !psz_string ) psz_string = strdup(""); + + psz_parser = strstr( psz_string, psz_name ); + if( b_add ) + { + if( !psz_parser ) + { + psz_parser = psz_string; + if( asprintf( &psz_string, (*psz_string) ? "%s:%s" : "%s%s", + psz_string, psz_name ) == -1 ) + { + free( psz_parser ); + return; + } + free( psz_parser ); + } + else + return; + } + else + { + if( psz_parser ) + { + memmove( psz_parser, psz_parser + strlen(psz_name) + + (*(psz_parser + strlen(psz_name)) == ':' ? 1 : 0 ), + strlen(psz_parser + strlen(psz_name)) + 1 ); + + /* Remove trailing : : */ + if( *(psz_string+strlen(psz_string ) -1 ) == ':' ) + { + *(psz_string+strlen(psz_string ) -1 ) = '\0'; + } + } + else + { + free( psz_string ); + return; + } + } + if( b_setconfig ) + config_PutPsz( p_vout, "vout-filter", psz_string ); + + var_SetString( p_vout, "vout-filter", psz_string ); + free( psz_string ); } /***************************************************************************** @@ -509,27 +819,22 @@ int vout_Snapshot( vout_thread_t *p_vout, picture_t *p_pic ) *****************************************************************************/ int vout_vaControlDefault( vout_thread_t *p_vout, int i_query, va_list args ) { + (void)args; switch( i_query ) { case VOUT_REPARENT: case VOUT_CLOSE: - if( p_vout->p_parent_intf ) - { - vlc_object_release( p_vout->p_parent_intf ); - p_vout->p_parent_intf = NULL; - } + vout_ReleaseWindow( p_vout, NULL ); return VLC_SUCCESS; - break; case VOUT_SNAPSHOT: - p_vout->b_snapshot = VLC_TRUE; + p_vout->b_snapshot = true; return VLC_SUCCESS; - break; default: msg_Dbg( p_vout, "control query not supported" ); - return VLC_EGENERIC; } + return VLC_EGENERIC; } /***************************************************************************** @@ -577,23 +882,28 @@ static void InitWindowSize( vout_thread_t *p_vout, unsigned *pi_width, goto initwsize_end; } - if( p_vout->fmt_in.i_sar_num >= p_vout->fmt_in.i_sar_den ) + if( p_vout->fmt_in.i_sar_num == 0 || p_vout->fmt_in.i_sar_den == 0 ) { + msg_Warn( p_vout, "fucked up aspect" ); + *pi_width = (int)( p_vout->fmt_in.i_visible_width * ll_zoom / FP_FACTOR ); + *pi_height = (int)( p_vout->fmt_in.i_visible_height * ll_zoom /FP_FACTOR); + } + else if( p_vout->fmt_in.i_sar_num >= p_vout->fmt_in.i_sar_den ) { *pi_width = (int)( p_vout->fmt_in.i_visible_width * ll_zoom * p_vout->fmt_in.i_sar_num / p_vout->fmt_in.i_sar_den / FP_FACTOR ); - *pi_height = (int)( p_vout->fmt_in.i_visible_height * ll_zoom + *pi_height = (int)( p_vout->fmt_in.i_visible_height * ll_zoom / FP_FACTOR ); } else { - *pi_width = (int)( p_vout->fmt_in.i_visible_width * ll_zoom + *pi_width = (int)( p_vout->fmt_in.i_visible_width * ll_zoom / FP_FACTOR ); *pi_height = (int)( p_vout->fmt_in.i_visible_height * ll_zoom * p_vout->fmt_in.i_sar_den / p_vout->fmt_in.i_sar_num / FP_FACTOR ); } initwsize_end: - msg_Dbg( p_vout, "window size: %dx%d", p_vout->i_window_width, + msg_Dbg( p_vout, "window size: %dx%d", p_vout->i_window_width, p_vout->i_window_height ); #undef FP_FACTOR @@ -606,9 +916,11 @@ static int ZoomCallback( vlc_object_t *p_this, char const *psz_cmd, vlc_value_t oldval, vlc_value_t newval, void *p_data ) { vout_thread_t *p_vout = (vout_thread_t *)p_this; + (void)psz_cmd; (void)oldval; (void)newval; (void)p_data; InitWindowSize( p_vout, &p_vout->i_window_width, &p_vout->i_window_height ); - vout_Control( p_vout, VOUT_SET_ZOOM ); + vout_Control( p_vout, VOUT_SET_SIZE, p_vout->i_window_width, + p_vout->i_window_height ); return VLC_SUCCESS; } @@ -619,7 +931,7 @@ static int CropCallback( vlc_object_t *p_this, char const *psz_cmd, int64_t i_aspect_num, i_aspect_den; unsigned int i_width, i_height; - char *psz_end, *psz_parser = strchr( newval.psz_string, ':' ); + (void)oldval; (void)p_data; /* Restore defaults */ p_vout->fmt_in.i_x_offset = p_vout->fmt_render.i_x_offset; @@ -627,30 +939,124 @@ static int CropCallback( vlc_object_t *p_this, char const *psz_cmd, p_vout->fmt_in.i_y_offset = p_vout->fmt_render.i_y_offset; p_vout->fmt_in.i_visible_height = p_vout->fmt_render.i_visible_height; - if( !psz_parser ) goto crop_end; + if( !strcmp( psz_cmd, "crop" ) ) + { + char *psz_end = NULL, *psz_parser = strchr( newval.psz_string, ':' ); + if( psz_parser ) + { + /* We're using the 3:4 syntax */ + i_aspect_num = strtol( newval.psz_string, &psz_end, 10 ); + if( psz_end == newval.psz_string || !i_aspect_num ) goto crop_end; + + i_aspect_den = strtol( ++psz_parser, &psz_end, 10 ); + if( psz_end == psz_parser || !i_aspect_den ) goto crop_end; + + i_width = p_vout->fmt_in.i_sar_den*p_vout->fmt_render.i_visible_height * + i_aspect_num / i_aspect_den / p_vout->fmt_in.i_sar_num; + i_height = p_vout->fmt_render.i_visible_width*p_vout->fmt_in.i_sar_num * + i_aspect_den / i_aspect_num / p_vout->fmt_in.i_sar_den; + + if( i_width < p_vout->fmt_render.i_visible_width ) + { + p_vout->fmt_in.i_x_offset = p_vout->fmt_render.i_x_offset + + (p_vout->fmt_render.i_visible_width - i_width) / 2; + p_vout->fmt_in.i_visible_width = i_width; + } + else + { + p_vout->fmt_in.i_y_offset = p_vout->fmt_render.i_y_offset + + (p_vout->fmt_render.i_visible_height - i_height) / 2; + p_vout->fmt_in.i_visible_height = i_height; + } + } + else + { + psz_parser = strchr( newval.psz_string, 'x' ); + if( psz_parser ) + { + /* Maybe we're using the x++ syntax */ + unsigned int i_crop_width, i_crop_height, i_crop_top, i_crop_left; + + i_crop_width = strtol( newval.psz_string, &psz_end, 10 ); + if( psz_end != psz_parser ) goto crop_end; - i_aspect_num = strtol( newval.psz_string, &psz_end, 0 ); - if( psz_end == newval.psz_string || !i_aspect_num ) goto crop_end; + psz_parser = strchr( ++psz_end, '+' ); + i_crop_height = strtol( psz_end, &psz_end, 10 ); + if( psz_end != psz_parser ) goto crop_end; - i_aspect_den = strtol( ++psz_parser, &psz_end, 0 ); - if( psz_end == psz_parser || !i_aspect_den ) goto crop_end; + psz_parser = strchr( ++psz_end, '+' ); + i_crop_left = strtol( psz_end, &psz_end, 10 ); + if( psz_end != psz_parser ) goto crop_end; - i_width = p_vout->fmt_in.i_sar_den * p_vout->fmt_render.i_visible_height * - i_aspect_num / i_aspect_den / p_vout->fmt_in.i_sar_num; - i_height = p_vout->fmt_render.i_visible_width * p_vout->fmt_in.i_sar_num * - i_aspect_den / i_aspect_num / p_vout->fmt_in.i_sar_den; + psz_end++; + i_crop_top = strtol( psz_end, &psz_end, 10 ); + if( *psz_end != '\0' ) goto crop_end; - if( i_width < p_vout->fmt_render.i_visible_width ) - { - p_vout->fmt_in.i_x_offset = p_vout->fmt_render.i_x_offset + - (p_vout->fmt_render.i_visible_width - i_width) / 2; - p_vout->fmt_in.i_visible_width = i_width; + i_width = i_crop_width; + p_vout->fmt_in.i_visible_width = i_width; + + i_height = i_crop_height; + p_vout->fmt_in.i_visible_height = i_height; + + p_vout->fmt_in.i_x_offset = i_crop_left; + p_vout->fmt_in.i_y_offset = i_crop_top; + } + else + { + /* Maybe we're using the +++ syntax */ + unsigned int i_crop_top, i_crop_left, i_crop_bottom, i_crop_right; + + psz_parser = strchr( newval.psz_string, '+' ); + i_crop_left = strtol( newval.psz_string, &psz_end, 10 ); + if( psz_end != psz_parser ) goto crop_end; + + psz_parser = strchr( ++psz_end, '+' ); + i_crop_top = strtol( psz_end, &psz_end, 10 ); + if( psz_end != psz_parser ) goto crop_end; + + psz_parser = strchr( ++psz_end, '+' ); + i_crop_right = strtol( psz_end, &psz_end, 10 ); + if( psz_end != psz_parser ) goto crop_end; + + psz_end++; + i_crop_bottom = strtol( psz_end, &psz_end, 10 ); + if( *psz_end != '\0' ) goto crop_end; + + i_width = p_vout->fmt_render.i_visible_width + - i_crop_left - i_crop_right; + p_vout->fmt_in.i_visible_width = i_width; + + i_height = p_vout->fmt_render.i_visible_height + - i_crop_top - i_crop_bottom; + p_vout->fmt_in.i_visible_height = i_height; + + p_vout->fmt_in.i_x_offset = i_crop_left; + p_vout->fmt_in.i_y_offset = i_crop_top; + } + } } - else + else if( !strcmp( psz_cmd, "crop-top" ) + || !strcmp( psz_cmd, "crop-left" ) + || !strcmp( psz_cmd, "crop-bottom" ) + || !strcmp( psz_cmd, "crop-right" ) ) { - p_vout->fmt_in.i_y_offset = p_vout->fmt_render.i_y_offset + - (p_vout->fmt_render.i_visible_height - i_height) / 2; + unsigned int i_crop_top, i_crop_left, i_crop_bottom, i_crop_right; + + i_crop_top = var_GetInteger( p_vout, "crop-top" ); + i_crop_left = var_GetInteger( p_vout, "crop-left" ); + i_crop_right = var_GetInteger( p_vout, "crop-right" ); + i_crop_bottom = var_GetInteger( p_vout, "crop-bottom" ); + + i_width = p_vout->fmt_render.i_visible_width + - i_crop_left - i_crop_right; + p_vout->fmt_in.i_visible_width = i_width; + + i_height = p_vout->fmt_render.i_visible_height + - i_crop_top - i_crop_bottom; p_vout->fmt_in.i_visible_height = i_height; + + p_vout->fmt_in.i_x_offset = i_crop_left; + p_vout->fmt_in.i_y_offset = i_crop_top; } crop_end: @@ -665,6 +1071,8 @@ static int CropCallback( vlc_object_t *p_this, char const *psz_cmd, p_vout->fmt_in.i_visible_width, p_vout->fmt_in.i_visible_height ); + var_SetVoid( p_vout, "crop-update" ); + return VLC_SUCCESS; } @@ -676,6 +1084,7 @@ static int AspectCallback( vlc_object_t *p_this, char const *psz_cmd, vlc_value_t val; char *psz_end, *psz_parser = strchr( newval.psz_string, ':' ); + (void)psz_cmd; (void)oldval; (void)p_data; /* Restore defaults */ p_vout->fmt_in.i_sar_num = p_vout->fmt_render.i_sar_num; @@ -685,10 +1094,10 @@ static int AspectCallback( vlc_object_t *p_this, char const *psz_cmd, if( !psz_parser ) goto aspect_end; - i_aspect_num = strtol( newval.psz_string, &psz_end, 0 ); + i_aspect_num = strtol( newval.psz_string, &psz_end, 10 ); if( psz_end == newval.psz_string || !i_aspect_num ) goto aspect_end; - i_aspect_den = strtol( ++psz_parser, &psz_end, 0 ); + i_aspect_den = strtol( ++psz_parser, &psz_end, 10 ); if( psz_end == psz_parser || !i_aspect_den ) goto aspect_end; i_sar_num = i_aspect_num * p_vout->fmt_render.i_visible_height; @@ -717,29 +1126,25 @@ static int AspectCallback( vlc_object_t *p_this, char const *psz_cmd, i_aspect_num, i_aspect_den, p_vout->fmt_in.i_sar_num, p_vout->fmt_in.i_sar_den ); - var_Get( p_vout, "crop", &val ); - return CropCallback( p_this, 0, val, val, 0 ); + if( var_Get( p_vout, "crop", &val ) ) + return VLC_EGENERIC; - return VLC_SUCCESS; + int i_ret = CropCallback( p_this, "crop", val, val, 0 ); + free( val.psz_string ); + return i_ret; } static int OnTopCallback( vlc_object_t *p_this, char const *psz_cmd, vlc_value_t oldval, vlc_value_t newval, void *p_data ) { vout_thread_t *p_vout = (vout_thread_t *)p_this; - playlist_t *p_playlist; vout_Control( p_vout, VOUT_SET_STAY_ON_TOP, newval.b_bool ); + (void)psz_cmd; (void)oldval; (void)p_data; - p_playlist = (playlist_t *)vlc_object_find( p_this, VLC_OBJECT_PLAYLIST, - FIND_PARENT ); - if( p_playlist ) - { - /* Modify playlist as well because the vout might have to be restarted */ - var_Create( p_playlist, "video-on-top", VLC_VAR_BOOL ); - var_Set( p_playlist, "video-on-top", newval ); + /* Modify libvlc as well because the vout might have to be restarted */ + var_Create( p_vout->p_libvlc, "video-on-top", VLC_VAR_BOOL ); + var_Set( p_vout->p_libvlc, "video-on-top", newval ); - vlc_object_release( p_playlist ); - } return VLC_SUCCESS; } @@ -747,35 +1152,16 @@ static int FullscreenCallback( vlc_object_t *p_this, char const *psz_cmd, vlc_value_t oldval, vlc_value_t newval, void *p_data ) { vout_thread_t *p_vout = (vout_thread_t *)p_this; - playlist_t *p_playlist; vlc_value_t val; + (void)psz_cmd; (void)oldval; (void)p_data; p_vout->i_changes |= VOUT_FULLSCREEN_CHANGE; - p_playlist = (playlist_t *)vlc_object_find( p_this, VLC_OBJECT_PLAYLIST, - FIND_PARENT ); - if( p_playlist ) - { - /* Modify playlist as well because the vout might have to be restarted */ - var_Create( p_playlist, "fullscreen", VLC_VAR_BOOL ); - var_Set( p_playlist, "fullscreen", newval ); - - vlc_object_release( p_playlist ); - } + /* Modify libvlc as well because the vout might have to be restarted */ + var_Create( p_vout->p_libvlc, "fullscreen", VLC_VAR_BOOL ); + var_Set( p_vout->p_libvlc, "fullscreen", newval ); - /* Disable "always on top" in fullscreen mode */ - var_Get( p_vout, "video-on-top", &val ); - if( newval.b_bool && val.b_bool ) - { - val.b_bool = VLC_FALSE; - vout_Control( p_vout, VOUT_SET_STAY_ON_TOP, val.b_bool ); - } - else if( !newval.b_bool && val.b_bool ) - { - vout_Control( p_vout, VOUT_SET_STAY_ON_TOP, val.b_bool ); - } - - val.b_bool = VLC_TRUE; + val.b_bool = true; var_Set( p_vout, "intf-change", val ); return VLC_SUCCESS; } @@ -783,7 +1169,38 @@ static int FullscreenCallback( vlc_object_t *p_this, char const *psz_cmd, static int SnapshotCallback( vlc_object_t *p_this, char const *psz_cmd, vlc_value_t oldval, vlc_value_t newval, void *p_data ) { + VLC_UNUSED(psz_cmd); VLC_UNUSED(oldval); + VLC_UNUSED(newval); VLC_UNUSED(p_data); vout_thread_t *p_vout = (vout_thread_t *)p_this; vout_Control( p_vout, VOUT_SNAPSHOT ); return VLC_SUCCESS; } + +static int TitleShowCallback( vlc_object_t *p_this, char const *psz_cmd, + vlc_value_t oldval, vlc_value_t newval, void *p_data ) +{ + VLC_UNUSED(psz_cmd); VLC_UNUSED(oldval); + VLC_UNUSED(p_data); + vout_thread_t *p_vout = (vout_thread_t *)p_this; + p_vout->b_title_show = newval.b_bool; + return VLC_SUCCESS; +} + +static int TitleTimeoutCallback( vlc_object_t *p_this, char const *psz_cmd, + vlc_value_t oldval, vlc_value_t newval, void *p_data ) +{ + VLC_UNUSED(psz_cmd); VLC_UNUSED(oldval); VLC_UNUSED(p_data); + vout_thread_t *p_vout = (vout_thread_t *)p_this; + p_vout->i_title_timeout = (mtime_t) newval.i_int; + return VLC_SUCCESS; +} + +static int TitlePositionCallback( vlc_object_t *p_this, char const *psz_cmd, + vlc_value_t oldval, vlc_value_t newval, void *p_data ) +{ + VLC_UNUSED(psz_cmd); VLC_UNUSED(oldval); + VLC_UNUSED(p_data); + vout_thread_t *p_vout = (vout_thread_t *)p_this; + p_vout->i_title_position = newval.i_int; + return VLC_SUCCESS; +}