Changeset 10c717090a22a161c08d4650282d6543c0e49e3d

Show
Ignore:
Timestamp:
12/16/07 14:46:30 (9 months ago)
Author:
Antoine Cellerier <dionoea@videolan.org>
git-committer:
Antoine Cellerier <dionoea@videolan.org> 1197812790 +0000
git-parent:

[415c99dbd1498fbcc078f48693ff90bcbfff4c03]

git-author:
Antoine Cellerier <dionoea@videolan.org> 1197812790 +0000
Message:

Some more vout_intf.c simplification and compile warning fixes.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • src/video_output/vout_intf.c

    r3117844 r10c7170  
    6464static int SnapshotCallback( vlc_object_t *, char const *, 
    6565                             vlc_value_t, vlc_value_t, void * ); 
    66 static int TitleCallback( vlc_object_t *, char const *, 
    67                        vlc_value_t, vlc_value_t, void * ); 
     66 
     67static int TitleShowCallback( vlc_object_t *, char const *, 
     68                              vlc_value_t, vlc_value_t, void * ); 
     69static int TitleTimeoutCallback( vlc_object_t *, char const *, 
     70                                 vlc_value_t, vlc_value_t, void * ); 
     71static int TitlePositionCallback( vlc_object_t *, char const *, 
     72                                  vlc_value_t, vlc_value_t, void * ); 
    6873 
    6974/***************************************************************************** 
     
    8994 
    9095    /* Get requested coordinates */ 
    91     var_Get( p_vout, "video-x", &val ); 
    92     *pi_x_hint = val.i_int ; 
    93     var_Get( p_vout, "video-y", &val ); 
    94     *pi_y_hint = val.i_int; 
     96    *pi_x_hint = var_GetInteger( p_vout, "video-x" ); 
     97    *pi_y_hint = var_GetInteger( p_vout, "video-y" ); 
    9598 
    9699    *pi_width_hint = p_vout->i_window_width; 
     
    287290        var_CreateGetInteger( p_vout, "video-title-position" ); 
    288291 
    289     var_AddCallback( p_vout, "video-title-show", TitleCallback, NULL ); 
    290     var_AddCallback( p_vout, "video-title-timeout", TitleCallback, NULL ); 
    291     var_AddCallback( p_vout, "video-title-position", TitleCallback, NULL ); 
     292    var_AddCallback( p_vout, "video-title-show", TitleShowCallback, NULL ); 
     293    var_AddCallback( p_vout, "video-title-timeout", TitleTimeoutCallback, NULL ); 
     294    var_AddCallback( p_vout, "video-title-position", TitlePositionCallback, NULL ); 
    292295 
    293296    /* Zoom object var */ 
     
    314317 
    315318    /* Crop offset vars */ 
    316     var_Create( p_vout, "crop-left", VLC_VAR_INTEGER ); 
    317     var_Create( p_vout, "crop-top", VLC_VAR_INTEGER ); 
    318     var_Create( p_vout, "crop-right", VLC_VAR_INTEGER ); 
    319     var_Create( p_vout, "crop-bottom", VLC_VAR_INTEGER ); 
     319    var_Create( p_vout, "crop-left", VLC_VAR_INTEGER | VLC_VAR_ISCOMMAND ); 
     320    var_Create( p_vout, "crop-top", VLC_VAR_INTEGER | VLC_VAR_ISCOMMAND ); 
     321    var_Create( p_vout, "crop-right", VLC_VAR_INTEGER | VLC_VAR_ISCOMMAND ); 
     322    var_Create( p_vout, "crop-bottom", VLC_VAR_INTEGER | VLC_VAR_ISCOMMAND ); 
    320323 
    321324    var_AddCallback( p_vout, "crop-left", CropCallback, NULL ); 
     
    325328 
    326329    /* Crop object var */ 
    327     var_Create( p_vout, "crop", VLC_VAR_STRING | 
     330    var_Create( p_vout, "crop", VLC_VAR_STRING | VLC_VAR_ISCOMMAND | 
    328331                VLC_VAR_HASCHOICE | VLC_VAR_DOINHERIT ); 
    329332 
     
    386389 
    387390    /* Aspect-ratio object var */ 
    388     var_Create( p_vout, "aspect-ratio", VLC_VAR_STRING | 
     391    var_Create( p_vout, "aspect-ratio", VLC_VAR_STRING | VLC_VAR_ISCOMMAND | 
    389392                VLC_VAR_HASCHOICE | VLC_VAR_DOINHERIT ); 
    390393 
     
    418421 
    419422    /* Add a variable to indicate if the window should be on top of others */ 
    420     var_Create( p_vout, "video-on-top", VLC_VAR_BOOL | VLC_VAR_DOINHERIT ); 
     423    var_Create( p_vout, "video-on-top", VLC_VAR_BOOL | VLC_VAR_DOINHERIT 
     424                | VLC_VAR_ISCOMMAND ); 
    421425    text.psz_string = _("Always on top"); 
    422426    var_Change( p_vout, "video-on-top", VLC_VAR_SETTEXT, &text, NULL ); 
     
    427431 
    428432    /* Add a fullscreen variable */ 
    429     if( var_CreateGetBool( p_vout, "fullscreen" ) ) 
     433    if( var_CreateGetBoolCommand( p_vout, "fullscreen" ) ) 
    430434    { 
    431435        /* user requested fullscreen */ 
     
    467471    int i_ret; 
    468472 
    469     memset( &fmt_in, 0, sizeof(video_format_t)); 
    470     memset( &fmt_out, 0, sizeof(video_format_t)); 
     473    memset( &fmt_in, 0, sizeof(video_format_t) ); 
     474    memset( &fmt_out, 0, sizeof(video_format_t) ); 
    471475 
    472476    var_Get( p_vout, "snapshot-path", &val ); 
     
    668672            do 
    669673            { 
    670                 asprintf( &psz_filename, "%s" DIR_SEP "%s%05d.%s", val.psz_string, 
    671                           psz_prefix, i_num++, format.psz_string ); 
     674                if( asprintf( &psz_filename, "%s" DIR_SEP "%s%05d.%s", 
     675                              val.psz_string, psz_prefix, i_num++, 
     676                              format.psz_string ) == -1 ) 
     677                { 
     678                    msg_Err( p_vout, "could not create snapshot" ); 
     679                    image_HandlerDelete( p_image ); 
     680                    return VLC_EGENERIC; 
     681                } 
    672682            } 
    673683            while( ( p_file = utf8_fopen( psz_filename, "r" ) ) && !fclose( p_file ) ); 
     
    676686        else 
    677687        { 
    678             asprintf( &psz_filename, "%s" DIR_SEP "%s%u.%s", val.psz_string, 
    679                       psz_prefix, 
    680                       (unsigned int)(p_pic->date / 100000) & 0xFFFFFF, 
    681                       format.psz_string ); 
     688            if( asprintf( &psz_filename, "%s" DIR_SEP "%s%u.%s", 
     689                          val.psz_string, psz_prefix, 
     690                          (unsigned int)(p_pic->date / 100000) & 0xFFFFFF, 
     691                          format.psz_string ) == -1 ) 
     692            { 
     693                msg_Err( p_vout, "could not create snapshot" ); 
     694                image_HandlerDelete( p_image ); 
     695                return VLC_EGENERIC; 
     696            } 
    682697        } 
    683698 
     
    782797        { 
    783798            psz_parser = psz_string; 
    784             asprintf( &psz_string, (*psz_string) ? "%s:%s" : "%s%s", 
    785                             psz_string, psz_name ); 
     799            if( asprintf( &psz_string, (*psz_string) ? "%s:%s" : "%s%s", 
     800                          psz_string, psz_name ) == -1 ) 
     801            { 
     802                free( psz_parser ); 
     803                return; 
     804            } 
    786805            free( psz_parser ); 
    787806        } 
     
    11771196    vout_thread_t *p_vout = (vout_thread_t *)p_this; 
    11781197    vout_Control( p_vout, VOUT_SNAPSHOT ); 
    1179     (void)psz_cmd; (void)oldval; (void)newval; (void)p_data; 
    11801198    return VLC_SUCCESS; 
    11811199} 
    11821200 
    1183 static int TitleCallback( vlc_object_t *p_this, char const *psz_cmd, 
     1201static int TitleShowCallback( vlc_object_t *p_this, char const *psz_cmd, 
    11841202                       vlc_value_t oldval, vlc_value_t newval, void *p_data ) 
    11851203{ 
    11861204    vout_thread_t *p_vout = (vout_thread_t *)p_this; 
    1187  
    1188     if( !strncmp( psz_cmd, "video-title-show", 16 ) ) 
    1189         p_vout->b_title_show = newval.b_bool; 
    1190     else if( !strncmp( psz_cmd, "video-title-timeout", 19 ) ) 
    1191         p_vout->i_title_timeout = (mtime_t) newval.i_int; 
    1192     else if( !strncmp( psz_cmd, "video-title-position", 20 ) ) 
    1193         p_vout->i_title_position = newval.i_int; 
     1205    p_vout->b_title_show = newval.b_bool; 
    11941206    return VLC_SUCCESS; 
    11951207} 
     1208 
     1209static int TitleTimeoutCallback( vlc_object_t *p_this, char const *psz_cmd, 
     1210                       vlc_value_t oldval, vlc_value_t newval, void *p_data ) 
     1211{ 
     1212    vout_thread_t *p_vout = (vout_thread_t *)p_this; 
     1213    p_vout->i_title_timeout = (mtime_t) newval.i_int; 
     1214    return VLC_SUCCESS; 
     1215} 
     1216 
     1217static int TitlePositionCallback( vlc_object_t *p_this, char const *psz_cmd, 
     1218                       vlc_value_t oldval, vlc_value_t newval, void *p_data ) 
     1219{ 
     1220    vout_thread_t *p_vout = (vout_thread_t *)p_this; 
     1221    p_vout->i_title_position = newval.i_int; 
     1222    return VLC_SUCCESS; 
     1223}