Changeset 277e48de02dd109619a87796d7e92a3bdc99c157

Show
Ignore:
Timestamp:
12/16/07 18:40:21 (9 months ago)
Author:
Rémi Denis-Courmont <rem@videolan.org>
git-committer:
Rémi Denis-Courmont <rem@videolan.org> 1197826821 +0000
git-parent:

[64ea683969472b60ebc2b4e15e552f7bf23d017a]

git-author:
Rémi Denis-Courmont <rem@videolan.org> 1197826821 +0000
Message:

config_Free simplifications

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • include/vlc_configuration.h

    r64ea683 r277e48d  
    136136typedef union 
    137137{ 
    138     const char *psz; 
     138    char      *psz; 
    139139    int         i; 
    140140    float       f; 

  • src/config/core.c

    r64ea683 r277e48d  
    488488                      size_t n ) 
    489489{ 
    490     int j; 
    491490    const module_config_t *p_item, *p_end = p_orig + n; 
    492491 
    …… 
    561560        module_config_t *p_item = p_module->p_config + j; 
    562561 
    563         free( (char*) p_item->psz_type ); 
    564         free( (char*) p_item->psz_name ); 
    565         free( (char*) p_item->psz_text ); 
    566         free( (char*) p_item->psz_longtext ); 
     562        free( p_item->psz_type ); 
     563        free( p_item->psz_name ); 
     564        free( p_item->psz_text ); 
     565        free( p_item->psz_longtext ); 
    567566 
    568567        if (IsConfigStringType (p_item->i_type)) 
    569568        { 
    570             free ((char *)p_item->value.psz); 
    571             free ((char *)p_item->orig.psz); 
    572             free ((char *)p_item->saved.psz); 
     569            free (p_item->value.psz); 
     570            free (p_item->orig.psz); 
     571            free (p_item->saved.psz); 
    573572        } 
    574573 
    575574        if( p_item->i_list ) 
    576575        { 
    577             for( i = 0; i < p_item->i_list; i++ ) 
    578             { 
    579                 if( p_item->ppsz_list && p_item->ppsz_list[i] ) 
    580                     free( (char*) p_item->ppsz_list[i] ); 
    581                 if( p_item->ppsz_list_text && p_item->ppsz_list_text[i] ) 
    582                     free( (char*) p_item->ppsz_list_text[i] ); 
    583             } 
    584             if( p_item->ppsz_list ) free( p_item->ppsz_list ); 
    585             if( p_item->ppsz_list_text ) free( p_item->ppsz_list_text ); 
    586             if( p_item->pi_list ) free( p_item->pi_list ); 
     576            if( p_item->ppsz_list ) 
     577                for( i = 0; i < p_item->i_list; i++ ) 
     578                    free( p_item->ppsz_list[i] ); 
     579            if( p_item->ppsz_list_text ) 
     580                for( i = 0; i < p_item->i_list; i++ ) 
     581                    free( p_item->ppsz_list[i] ); 
     582            free( p_item->ppsz_list ); 
     583            free( p_item->ppsz_list_text ); 
     584            free( p_item->pi_list ); 
    587585        } 
    588586 
    …… 
    591589            for( i = 0; i < p_item->i_action; i++ ) 
    592590            { 
    593                 free( (char*) p_item->ppsz_action_text[i] ); 
     591                free( p_item->ppsz_action_text[i] ); 
    594592            } 
    595             if( p_item->ppf_action ) free( p_item->ppf_action ); 
    596             if( p_item->ppsz_action_text ) free( p_item->ppsz_action_text ); 
    597         } 
    598     } 
    599  
    600     if (p_module->p_config != NULL) 
    601     { 
    602         free (p_module->p_config); 
    603         p_module->p_config = NULL; 
    604     } 
     593            free( p_item->ppf_action ); 
     594            free( p_item->ppsz_action_text ); 
     595        } 
     596    } 
     597 
     598    free (p_module->p_config); 
     599    p_module->p_config = NULL; 
    605600} 
    606601