Changeset 959c5560883360dff21958fbf79abce04baec84c

Show
Ignore:
Timestamp:
03/30/08 16:15:09 (5 months ago)
Author:
Rémi Denis-Courmont <rem@videolan.org>
git-committer:
Rémi Denis-Courmont <rem@videolan.org> 1206886509 +0300
git-parent:

[e4e9182903c4915dd0c1e73eff6b325f0d3be54d]

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

Use var_AcquireMutex for the "libvlc" lock

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • src/libvlc-common.c

    re4e9182 r959c556  
    152152    int i_ret; 
    153153    libvlc_int_t * p_libvlc = NULL; 
    154     vlc_value_t lockval; 
    155154    char *psz_env = NULL; 
    156155 
    …… 
    161160 
    162161    /* Now that the thread system is initialized, we don't have much, but 
    163      * at least we have var_Create */ 
    164     var_Create( p_libvlc_global, "libvlc", VLC_VAR_MUTEX ); 
    165     var_Get( p_libvlc_global, "libvlc", &lockval ); 
    166     vlc_mutex_lock( lockval.p_address ); 
     162     * at least we have variables */ 
     163    vlc_mutex_t *lock = var_AcquireMutex( "libvlc" ); 
    167164 
    168165    i_instances++; 
    …… 
    177174        p_libvlc_global->b_ready = VLC_TRUE; 
    178175    } 
    179     vlc_mutex_unlock( lockval.p_address ); 
     176    vlc_mutex_unlock( lock ); 
    180177 
    181178    /* Allocate a libvlc instance object */ 
    …… 
    10451042    FREENULL( p_libvlc->p_hotkeys ); 
    10461043 
    1047     var_Create( p_libvlc_global, "libvlc", VLC_VAR_MUTEX ); 
    1048     var_Get( p_libvlc_global, "libvlc", &lockval ); 
    1049     vlc_mutex_lock( lockval.p_address ); 
     1044    vlc_mutex_t *lock = var_AcquireMutex( "libvlc" ); 
    10501045    i_instances--; 
    10511046 
    …… 
    10551050        system_End( p_libvlc ); 
    10561051    } 
    1057     vlc_mutex_unlock( lockval.p_address ); 
     1052    vlc_mutex_unlock( lock ); 
    10581053 
    10591054    msg_Flush( p_libvlc ); 

  • src/modules/modules.c

    re4e9182 r959c556  
    116116{ 
    117117    module_bank_t *p_bank = NULL; 
    118     vlc_value_t  lockval; 
    119118    libvlc_global_data_t *p_libvlc_global = vlc_global(); 
    120119 
    121     var_Create( p_libvlc_global, "libvlc", VLC_VAR_MUTEX ); 
    122     var_Get( p_libvlc_global, "libvlc", &lockval ); 
    123     vlc_mutex_lock( lockval.p_address ); 
     120    vlc_mutex_t *lock = var_AcquireMutex( "libvlc" ); 
    124121 
    125122    if( p_libvlc_global->p_module_bank == NULL ) 
    …… 
    147144        p_libvlc_global->p_module_bank->i_usage++; 
    148145 
    149     vlc_mutex_unlock( lockval.p_address ); 
     146    vlc_mutex_unlock( lock ); 
    150147} 
    151148 
    …… 
    162159{ 
    163160    module_t * p_next = NULL; 
    164     vlc_value_t lockval; 
    165161    libvlc_global_data_t *p_libvlc_global = vlc_global(); 
    166162 
    167     var_Create( p_libvlc_global, "libvlc", VLC_VAR_MUTEX ); 
    168     var_Get( p_libvlc_global, "libvlc", &lockval ); 
    169     vlc_mutex_lock( lockval.p_address ); 
     163    vlc_mutex_t *lock = var_AcquireMutex( "libvlc" ); 
    170164    if( !p_libvlc_global->p_module_bank ) 
    171165    { 
    172         vlc_mutex_unlock( lockval.p_address ); 
     166        vlc_mutex_unlock( lock ); 
    173167        return; 
    174168    } 
    175169    if( --p_libvlc_global->p_module_bank->i_usage ) 
    176170    { 
    177         vlc_mutex_unlock( lockval.p_address ); 
     171        vlc_mutex_unlock( lock ); 
    178172        return; 
    179173    } 
    180     vlc_mutex_unlock( lockval.p_address ); 
     174    vlc_mutex_unlock( lock ); 
    181175 
    182176    /* Save the configuration */ 
    …… 
    248242void __module_LoadBuiltins( vlc_object_t * p_this ) 
    249243{ 
    250     vlc_value_t lockval; 
    251244    libvlc_global_data_t *p_libvlc_global = vlc_global(); 
    252245 
    253     var_Create( p_libvlc_global, "libvlc", VLC_VAR_MUTEX ); 
    254     var_Get( p_libvlc_global, "libvlc", &lockval ); 
    255     vlc_mutex_lock( lockval.p_address ); 
     246    vlc_mutex_t *lock = var_AcquireMutex( "libvlc" ); 
    256247    if( p_libvlc_global->p_module_bank->b_builtins ) 
    257248    { 
    258         vlc_mutex_unlock( lockval.p_address ); 
     249        vlc_mutex_unlock( lock ); 
    259250        return; 
    260251    } 
    261252    p_libvlc_global->p_module_bank->b_builtins = VLC_TRUE; 
    262     vlc_mutex_unlock( lockval.p_address ); 
     253    vlc_mutex_unlock( lock ); 
    263254 
    264255    msg_Dbg( p_this, "checking builtin modules" ); 
    …… 
    277268{ 
    278269#ifdef HAVE_DYNAMIC_PLUGINS 
    279     vlc_value_t lockval; 
    280270    libvlc_global_data_t *p_libvlc_global = vlc_global(); 
    281271 
    282     var_Create( p_libvlc_global, "libvlc", VLC_VAR_MUTEX ); 
    283     var_Get( p_libvlc_global, "libvlc", &lockval ); 
    284     vlc_mutex_lock( lockval.p_address ); 
     272    vlc_mutex_t *lock = var_AcquireMutex( "libvlc" ); 
    285273    if( p_libvlc_global->p_module_bank->b_plugins ) 
    286274    { 
    287         vlc_mutex_unlock( lockval.p_address ); 
     275        vlc_mutex_unlock( lock ); 
    288276        return; 
    289277    } 
    290278    p_libvlc_global->p_module_bank->b_plugins = VLC_TRUE; 
    291     vlc_mutex_unlock( lockval.p_address ); 
     279    vlc_mutex_unlock( lock ); 
    292280 
    293281    msg_Dbg( p_this, "checking plugin modules" );