#include "libvlc.h"
+#include <search.h>
#include <assert.h>
+#include <math.h>
+#include <limits.h>
/*****************************************************************************
* Private types
/*****************************************************************************
* Local comparison functions, returns 0 if v == w, < 0 if v < w, > 0 if v > w
*****************************************************************************/
-static int CmpBool( vlc_value_t v, vlc_value_t w ) { return v.b_bool ? w.b_bool ? 0 : 1 : w.b_bool ? -1 : 0; }
-static int CmpInt( vlc_value_t v, vlc_value_t w ) { return v.i_int == w.i_int ? 0 : v.i_int > w.i_int ? 1 : -1; }
+static int CmpBool( vlc_value_t v, vlc_value_t w )
+{
+ return v.b_bool ? w.b_bool ? 0 : 1 : w.b_bool ? -1 : 0;
+}
+
+static int CmpInt( vlc_value_t v, vlc_value_t w )
+{
+ return v.i_int == w.i_int ? 0 : v.i_int > w.i_int ? 1 : -1;
+}
+
static int CmpTime( vlc_value_t v, vlc_value_t w )
{
return v.i_time == w.i_time ? 0 : v.i_time > w.i_time ? 1 : -1;
}
+
static int CmpString( vlc_value_t v, vlc_value_t w )
{
if( !v.psz_string )
p_val->psz_string = strdup( p_val->psz_string ? p_val->psz_string : "" );
}
-static void DupList( vlc_value_t *p_val )
-{
- int i;
- vlc_list_t *p_list = malloc( sizeof(vlc_list_t) );
-
- p_list->i_count = p_val->p_list->i_count;
- if( p_val->p_list->i_count )
- {
- p_list->p_values = malloc( p_list->i_count * sizeof(vlc_value_t) );
- p_list->pi_types = malloc( p_list->i_count * sizeof(int) );
- }
- else
- {
- p_list->p_values = NULL;
- p_list->pi_types = NULL;
- }
-
- for( i = 0; i < p_list->i_count; i++ )
- {
- p_list->p_values[i] = p_val->p_list->p_values[i];
- p_list->pi_types[i] = p_val->p_list->pi_types[i];
- switch( p_val->p_list->pi_types[i] & VLC_VAR_CLASS )
- {
- case VLC_VAR_STRING:
-
- DupString( &p_list->p_values[i] );
- break;
- default:
- break;
- }
- }
-
- p_val->p_list = p_list;
-}
-
static void FreeDummy( vlc_value_t *p_val ) { (void)p_val; /* unused */ }
static void FreeString( vlc_value_t *p_val ) { free( p_val->psz_string ); }
static void FreeMutex( vlc_value_t *p_val ) { vlc_mutex_destroy( (vlc_mutex_t*)p_val->p_address ); free( p_val->p_address ); }
bool_ops = { CmpBool, DupDummy, FreeDummy, },
float_ops = { CmpFloat, DupDummy, FreeDummy, },
int_ops = { CmpInt, DupDummy, FreeDummy, },
-list_ops = { CmpAddress, DupList, FreeList, },
mutex_ops = { CmpAddress, DupDummy, FreeMutex, },
string_ops = { CmpString, DupString, FreeString, },
-time_ops = { CmpTime, DupDummy, FreeDummy, };
+time_ops = { CmpTime, DupDummy, FreeDummy, },
+coords_ops = { NULL, DupDummy, FreeDummy, };
/*****************************************************************************
* Local prototypes
*****************************************************************************/
-static int GetUnused ( vlc_object_t *, const char * );
-static uint32_t HashString ( const char * );
-static int Insert ( variable_t **, int, const char * );
-static int InsertInner ( variable_t **, int, uint32_t );
-static int Lookup ( variable_t **, size_t, const char * );
+static void WaitUnused ( vlc_object_t *, variable_t * );
static void CheckValue ( variable_t *, vlc_value_t * );
-static int InheritValue( vlc_object_t *, const char *, vlc_value_t *,
- int );
static int TriggerCallback( vlc_object_t *, variable_t *, const char *,
vlc_value_t );
+static int varcmp( const void *a, const void *b )
+{
+ const variable_t *va = a, *vb = b;
+
+ /* psz_name must be first */
+ assert( va == (const void *)&va->psz_name );
+ return strcmp( va->psz_name, vb->psz_name );
+}
+
+static variable_t *Lookup( vlc_object_t *obj, const char *psz_name )
+{
+ vlc_object_internals_t *priv = vlc_internals( obj );
+ variable_t **pp_var;
+
+ vlc_assert_locked( &priv->var_lock );
+ pp_var = tfind( &psz_name, &priv->var_root, varcmp );
+ return (pp_var != NULL) ? *pp_var : NULL;
+}
+
static void Destroy( variable_t *p_var )
{
p_var->ops->pf_free( &p_var->val );
free( p_var );
}
+#undef var_Create
/**
* Initialize a vlc variable
*
* \param i_type The variables type. Must be one of \ref var_type combined with
* zero or more \ref var_flags
*/
-int __var_Create( vlc_object_t *p_this, const char *psz_name, int i_type )
+int var_Create( vlc_object_t *p_this, const char *psz_name, int i_type )
{
- static vlc_list_t dummy_null_list = {0, NULL, NULL};
assert( p_this );
variable_t *p_var = calloc( 1, sizeof( *p_var ) );
if( p_var == NULL )
return VLC_ENOMEM;
- p_var->i_hash = HashString( psz_name );
p_var->psz_name = strdup( psz_name );
p_var->psz_text = NULL;
p_var->ops = &time_ops;
p_var->val.i_time = 0;
break;
+ case VLC_VAR_COORDS:
+ p_var->ops = &coords_ops;
+ p_var->val.coords.x = p_var->val.coords.y = 0;
+ break;
case VLC_VAR_ADDRESS:
p_var->ops = &addr_ops;
p_var->val.p_address = NULL;
p_var->val.p_address = malloc( sizeof(vlc_mutex_t) );
vlc_mutex_init( (vlc_mutex_t*)p_var->val.p_address );
break;
- case VLC_VAR_LIST:
- p_var->ops = &list_ops;
- p_var->val.p_list = &dummy_null_list;
- break;
default:
p_var->ops = &void_ops;
- break;
+#ifndef NDEBUG
+ if( (i_type & VLC_VAR_CLASS) != VLC_VAR_VOID )
+ msg_Err( p_this, "Creating the variable '%s' without a type",
+ psz_name );
+#endif
}
if( i_type & VLC_VAR_DOINHERIT )
{
- if( InheritValue( p_this, psz_name, &p_var->val, p_var->i_type ) )
+ if( var_Inherit( p_this, psz_name, i_type, &p_var->val ) )
msg_Err( p_this, "cannot inherit value for %s", psz_name );
else if( i_type & VLC_VAR_HASCHOICE )
{
}
vlc_object_internals_t *p_priv = vlc_internals( p_this );
- int i_new;
+ variable_t **pp_var, *p_oldvar;
+ int ret = VLC_SUCCESS;
vlc_mutex_lock( &p_priv->var_lock );
- /* FIXME: if the variable already exists, we don't duplicate it. But we
- * duplicate the lookups. It's not that serious, but if anyone finds some
- * time to rework Insert() so that only one lookup has to be done, feel
- * free to do so. */
- i_new = Lookup( p_priv->pp_vars, p_priv->i_vars, psz_name );
-
- if( i_new >= 0 )
+ pp_var = tsearch( p_var, &p_priv->var_root, varcmp );
+ if( unlikely(pp_var == NULL) )
+ ret = VLC_ENOMEM;
+ else if( (p_oldvar = *pp_var) == p_var )
+ p_var = NULL;
+ else if( unlikely((i_type ^ p_oldvar->i_type) & VLC_VAR_CLASS) )
+ { /* If the types differ, variable creation failed. */
+ msg_Err( p_this, "Variable '%s' (0x%04x) already exist "
+ "but with a different type (0x%04x)",
+ psz_name, p_oldvar->i_type, i_type );
+ ret = VLC_EBADVAR;
+ }
+ else
{
- /* If the types differ, variable creation failed. */
- if( (i_type & VLC_VAR_CLASS) != (p_priv->pp_vars[i_new]->i_type & VLC_VAR_CLASS) )
- {
- msg_Err( p_this, "Variable '%s' (0x%04x) already exist but with a different type (0x%04x)",
- psz_name, p_priv->pp_vars[i_new]->i_type, i_type );
- vlc_mutex_unlock( &p_priv->var_lock );
- return VLC_EBADVAR;
- }
-
- p_priv->pp_vars[i_new]->i_usage++;
- p_priv->pp_vars[i_new]->i_type |= ( i_type & VLC_VAR_ISCOMMAND );
- p_priv->pp_vars[i_new]->i_type |= ( i_type & VLC_VAR_HASCHOICE );
- vlc_mutex_unlock( &p_priv->var_lock );
-
- /* We did not need to create a new variable, free everything... */
- Destroy( p_var );
- return VLC_SUCCESS;
+ p_oldvar->i_usage++;
+ p_oldvar->i_type |= i_type & (VLC_VAR_ISCOMMAND|VLC_VAR_HASCHOICE);
}
-
- i_new = Insert( p_priv->pp_vars, p_priv->i_vars, psz_name );
-
- if( (p_priv->i_vars & 15) == 0 )
- p_priv->pp_vars = xrealloc( p_priv->pp_vars,
- (p_priv->i_vars+16) * sizeof(variable_t *) );
-
- memmove( p_priv->pp_vars + i_new + 1,
- p_priv->pp_vars + i_new,
- (p_priv->i_vars - i_new) * sizeof(variable_t *) );
-
- p_priv->i_vars++;
-
- p_priv->pp_vars[i_new] = p_var;
vlc_mutex_unlock( &p_priv->var_lock );
- return VLC_SUCCESS;
+ /* If we did not need to create a new variable, free everything... */
+ if( p_var != NULL )
+ Destroy( p_var );
+ return ret;
}
+#undef var_Destroy
/**
* Destroy a vlc variable
*
* \param p_this The object that holds the variable
* \param psz_name The name of the variable
*/
-int __var_Destroy( vlc_object_t *p_this, const char *psz_name )
+int var_Destroy( vlc_object_t *p_this, const char *psz_name )
{
- int i_var, i;
variable_t *p_var;
assert( p_this );
vlc_mutex_lock( &p_priv->var_lock );
- i_var = GetUnused( p_this, psz_name );
- if( i_var < 0 )
- {
- vlc_mutex_unlock( &p_priv->var_lock );
- return i_var;
- }
-
- p_var = p_priv->pp_vars[i_var];
-
- if( p_var->i_usage > 1 )
+ p_var = Lookup( p_this, psz_name );
+ if( p_var == NULL )
{
- p_var->i_usage--;
vlc_mutex_unlock( &p_priv->var_lock );
- return VLC_SUCCESS;
+ return VLC_ENOVAR;
}
- p_priv->i_vars--;
- memmove( p_priv->pp_vars + i_var,
- p_priv->pp_vars + i_var + 1,
- (p_priv->i_vars - i_var) * sizeof(variable_t *) );
+ WaitUnused( p_this, p_var );
- if( p_priv->i_vars == 0 )
- {
- free( p_priv->pp_vars );
- p_priv->pp_vars = NULL;
- }
+ if( --p_var->i_usage == 0 )
+ tdelete( p_var, &p_priv->var_root, varcmp );
else
- if( (p_priv->i_vars & 15) == 0 )
- {
- variable_t **pp_vars = realloc( p_priv->pp_vars,
- (p_priv->i_vars) * sizeof(variable_t *) );
- if( pp_vars != NULL )
- p_priv->pp_vars = pp_vars;
- }
+ p_var = NULL;
vlc_mutex_unlock( &p_priv->var_lock );
- Destroy( p_var );
+ if( p_var != NULL )
+ Destroy( p_var );
return VLC_SUCCESS;
}
+static void CleanupVar( void *var )
+{
+ Destroy( var );
+}
+
+void var_DestroyAll( vlc_object_t *obj )
+{
+ vlc_object_internals_t *priv = vlc_internals( obj );
+
+ tdestroy( priv->var_root, CleanupVar );
+}
+
+#undef var_Change
/**
* Perform an action on a variable
*
* \param p_val First action parameter
* \param p_val2 Second action parameter
*/
-int __var_Change( vlc_object_t *p_this, const char *psz_name,
- int i_action, vlc_value_t *p_val, vlc_value_t *p_val2 )
+int var_Change( vlc_object_t *p_this, const char *psz_name,
+ int i_action, vlc_value_t *p_val, vlc_value_t *p_val2 )
{
- int i_var, i;
+ int i;
variable_t *p_var;
vlc_value_t oldval;
+ vlc_value_t newval;
assert( p_this );
vlc_mutex_lock( &p_priv->var_lock );
- i_var = Lookup( p_priv->pp_vars, p_priv->i_vars, psz_name );
-
- if( i_var < 0 )
+ p_var = Lookup( p_this, psz_name );
+ if( p_var == NULL )
{
vlc_mutex_unlock( &p_priv->var_lock );
return VLC_ENOVAR;
}
- p_var = p_priv->pp_vars[i_var];
-
switch( i_action )
{
case VLC_VAR_SETMIN:
break;
case VLC_VAR_SETVALUE:
/* Duplicate data if needed */
- p_var->ops->pf_dup( p_val );
+ newval = *p_val;
+ p_var->ops->pf_dup( &newval );
/* Backup needed stuff */
oldval = p_var->val;
/* Check boundaries and list */
- CheckValue( p_var, p_val );
+ CheckValue( p_var, &newval );
/* Set the variable */
- p_var->val = *p_val;
+ p_var->val = newval;
/* Free data if needed */
p_var->ops->pf_free( &oldval );
break;
return VLC_SUCCESS;
}
-
+#undef var_GetAndSet
/**
* Perform a Get and Set on a variable
*
* \param p_val: The action parameter
* \return vlc error codes
*/
-int __var_GetAndSet( vlc_object_t *p_this, const char *psz_name, int i_action,
- vlc_value_t val )
+int var_GetAndSet( vlc_object_t *p_this, const char *psz_name, int i_action,
+ vlc_value_t *p_val )
{
- int i_var;
- int i_ret = VLC_SUCCESS;
+ int i_ret;
variable_t *p_var;
vlc_value_t oldval;
assert( p_this );
+ assert( p_val );
vlc_object_internals_t *p_priv = vlc_internals( p_this );
vlc_mutex_lock( &p_priv->var_lock );
- i_var = GetUnused( p_this, psz_name );
- if( i_var < 0 )
+ p_var = Lookup( p_this, psz_name );
+ if( p_var == NULL )
{
vlc_mutex_unlock( &p_priv->var_lock );
- return i_var;
+ return VLC_ENOVAR;
}
- p_var = p_priv->pp_vars[i_var];
+ WaitUnused( p_this, p_var );
/* Duplicated data if needed */
//p_var->ops->pf_dup( &val );
/* depending of the action requiered */
switch( i_action )
{
- case VLC_VAR_TOGGLE_BOOL:
+ case VLC_VAR_BOOL_TOGGLE:
assert( ( p_var->i_type & VLC_VAR_BOOL ) == VLC_VAR_BOOL );
p_var->val.b_bool = !p_var->val.b_bool;
break;
- case VLC_VAR_INTEGER_INCDEC:
+ case VLC_VAR_INTEGER_ADD:
+ assert( ( p_var->i_type & VLC_VAR_INTEGER ) == VLC_VAR_INTEGER );
+ p_var->val.i_int += p_val->i_int;
+ break;
+ case VLC_VAR_INTEGER_OR:
+ assert( ( p_var->i_type & VLC_VAR_INTEGER ) == VLC_VAR_INTEGER );
+ p_var->val.i_int |= p_val->i_int;
+ break;
+ case VLC_VAR_INTEGER_NAND:
assert( ( p_var->i_type & VLC_VAR_INTEGER ) == VLC_VAR_INTEGER );
- p_var->val.i_int += val.i_int;
+ p_var->val.i_int &= ~p_val->i_int;
break;
default:
vlc_mutex_unlock( &p_priv->var_lock );
/* Check boundaries */
CheckValue( p_var, &p_var->val );
+ *p_val = p_var->val;
/* Deal with callbacks.*/
- if( p_var->i_entries )
- i_ret = TriggerCallback( p_this, p_var, psz_name, oldval );
+ i_ret = TriggerCallback( p_this, p_var, psz_name, oldval );
vlc_mutex_unlock( &p_priv->var_lock );
return i_ret;
}
-
+#undef var_Type
/**
* Request a variable's type
*
* variable could not be found.
* \see \ref var_type
*/
-int __var_Type( vlc_object_t *p_this, const char *psz_name )
+int var_Type( vlc_object_t *p_this, const char *psz_name )
{
- int i_var, i_type;
+ variable_t *p_var;
+ int i_type = 0;
assert( p_this );
vlc_mutex_lock( &p_priv->var_lock );
- i_var = Lookup( p_priv->pp_vars, p_priv->i_vars, psz_name );
-
- if( i_var < 0 )
- {
- vlc_mutex_unlock( &p_priv->var_lock );
- return 0;
- }
-
- i_type = p_priv->pp_vars[i_var]->i_type;
+ p_var = Lookup( p_this, psz_name );
+ if( p_var != NULL )
+ i_type = p_var->i_type;
vlc_mutex_unlock( &p_priv->var_lock );
return i_type;
}
+#undef var_SetChecked
int var_SetChecked( vlc_object_t *p_this, const char *psz_name,
int expected_type, vlc_value_t val )
{
- int i_var;
int i_ret = VLC_SUCCESS;
variable_t *p_var;
vlc_value_t oldval;
vlc_mutex_lock( &p_priv->var_lock );
- i_var = GetUnused( p_this, psz_name );
- if( i_var < 0 )
+ p_var = Lookup( p_this, psz_name );
+ if( p_var == NULL )
{
vlc_mutex_unlock( &p_priv->var_lock );
- return i_var;
+ return VLC_ENOVAR;
}
- p_var = p_priv->pp_vars[i_var];
assert( expected_type == 0 ||
(p_var->i_type & VLC_VAR_CLASS) == expected_type );
+#ifndef NDEBUG
+ /* Alert if the type is VLC_VAR_VOID */
+ if( ( p_var->i_type & VLC_VAR_TYPE ) == VLC_VAR_VOID )
+ msg_Warn( p_this, "Calling var_Set on the void variable '%s' (0x%04x)", psz_name, p_var->i_type );
+#endif
+
+
+ WaitUnused( p_this, p_var );
/* Duplicate data if needed */
p_var->ops->pf_dup( &val );
p_var->val = val;
/* Deal with callbacks */
- if( p_var->i_entries )
- i_ret = TriggerCallback( p_this, p_var, psz_name, oldval );
+ i_ret = TriggerCallback( p_this, p_var, psz_name, oldval );
/* Free data if needed */
p_var->ops->pf_free( &oldval );
return i_ret;
}
-
+#undef var_Set
/**
* Set a variable's value
*
* \param psz_name The name of the variable
* \param val the value to set
*/
-int __var_Set( vlc_object_t *p_this, const char *psz_name, vlc_value_t val )
+int var_Set( vlc_object_t *p_this, const char *psz_name, vlc_value_t val )
{
return var_SetChecked( p_this, psz_name, 0, val );
}
+#undef var_GetChecked
int var_GetChecked( vlc_object_t *p_this, const char *psz_name,
int expected_type, vlc_value_t *p_val )
{
assert( p_this );
vlc_object_internals_t *p_priv = vlc_internals( p_this );
- int i_var, err = VLC_SUCCESS;
+ variable_t *p_var;
+ int err = VLC_SUCCESS;
vlc_mutex_lock( &p_priv->var_lock );
- i_var = Lookup( p_priv->pp_vars, p_priv->i_vars, psz_name );
- if( i_var >= 0 )
+ p_var = Lookup( p_this, psz_name );
+ if( p_var != NULL )
{
- variable_t *p_var = p_priv->pp_vars[i_var];
-
assert( expected_type == 0 ||
(p_var->i_type & VLC_VAR_CLASS) == expected_type );
#ifndef NDEBUG
/* Alert if the type is VLC_VAR_VOID */
if( ( p_var->i_type & VLC_VAR_TYPE ) == VLC_VAR_VOID )
- msg_Warn( p_this, "Calling var_GetVoid on the void variable '%s' (0x%04x)", psz_name, p_var->i_type );
+ msg_Warn( p_this, "Calling var_Get on the void variable '%s' (0x%04x)", psz_name, p_var->i_type );
#endif
/* Duplicate value if needed */
return err;
}
+#undef var_Get
/**
* Get a variable's value
*
* \param p_val Pointer to a vlc_value_t that will hold the variable's value
* after the function is finished
*/
-int __var_Get( vlc_object_t *p_this, const char *psz_name, vlc_value_t *p_val )
+int var_Get( vlc_object_t *p_this, const char *psz_name, vlc_value_t *p_val )
{
return var_GetChecked( p_this, psz_name, 0, p_val );
}
+#undef var_AddCallback
/**
* Register a callback in a variable
*
* the variable. Use proper locking. This thread may not have much
* time to spare, so keep callback functions short.
*/
-int __var_AddCallback( vlc_object_t *p_this, const char *psz_name,
- vlc_callback_t pf_callback, void *p_data )
+int var_AddCallback( vlc_object_t *p_this, const char *psz_name,
+ vlc_callback_t pf_callback, void *p_data )
{
- int i_var;
variable_t *p_var;
callback_entry_t entry;
vlc_mutex_lock( &p_priv->var_lock );
- i_var = GetUnused( p_this, psz_name );
- if( i_var < 0 )
+ p_var = Lookup( p_this, psz_name );
+ if( p_var == NULL )
{
#ifndef NDEBUG
msg_Warn( p_this, "Failed to add a callback to the non-existing "
"variable '%s'", psz_name );
#endif
vlc_mutex_unlock( &p_priv->var_lock );
- return i_var;
+ return VLC_ENOVAR;
}
- p_var = p_priv->pp_vars[i_var];
-
+ WaitUnused( p_this, p_var );
INSERT_ELEM( p_var->p_entries,
p_var->i_entries,
p_var->i_entries,
return VLC_SUCCESS;
}
+#undef var_DelCallback
/**
* Remove a callback from a variable
*
* pf_callback and p_data have to be given again, because different objects
* might have registered the same callback function.
*/
-int __var_DelCallback( vlc_object_t *p_this, const char *psz_name,
- vlc_callback_t pf_callback, void *p_data )
+int var_DelCallback( vlc_object_t *p_this, const char *psz_name,
+ vlc_callback_t pf_callback, void *p_data )
{
- int i_entry, i_var;
+ int i_entry;
variable_t *p_var;
#ifndef NDEBUG
bool b_found_similar = false;
vlc_mutex_lock( &p_priv->var_lock );
- i_var = GetUnused( p_this, psz_name );
- if( i_var < 0 )
+ p_var = Lookup( p_this, psz_name );
+ if( p_var == NULL )
{
vlc_mutex_unlock( &p_priv->var_lock );
- return i_var;
+ return VLC_ENOVAR;
}
- p_var = p_priv->pp_vars[i_var];
+ WaitUnused( p_this, p_var );
for( i_entry = p_var->i_entries ; i_entry-- ; )
{
return VLC_SUCCESS;
}
+#undef var_TriggerCallback
/**
* Trigger callback on a variable
*
* \param p_this The object that hold the variable
* \param psz_name The name of the variable
*/
-int __var_TriggerCallback( vlc_object_t *p_this, const char *psz_name )
+int var_TriggerCallback( vlc_object_t *p_this, const char *psz_name )
{
- int i_var;
- int i_ret = VLC_SUCCESS;
+ int i_ret;
variable_t *p_var;
assert( p_this );
vlc_mutex_lock( &p_priv->var_lock );
- i_var = GetUnused( p_this, psz_name );
- if( i_var < 0 )
+ p_var = Lookup( p_this, psz_name );
+ if( p_var == NULL )
{
vlc_mutex_unlock( &p_priv->var_lock );
- return i_var;
+ return VLC_ENOVAR;
}
- p_var = p_priv->pp_vars[i_var];
+ WaitUnused( p_this, p_var );
/* Deal with callbacks. Tell we're in a callback, release the lock,
* call stored functions, retake the lock. */
- if( p_var->i_entries )
- i_ret = TriggerCallback( p_this, p_var, psz_name, p_var->val );
+ i_ret = TriggerCallback( p_this, p_var, psz_name, p_var->val );
vlc_mutex_unlock( &p_priv->var_lock );
return i_ret;
if( psz_value != NULL )
*psz_value++ = '\0';
- /* FIXME: :programs should be handled generically */
- if( !strcmp( psz_name, "programs" ) )
- i_type = VLC_VAR_LIST;
- else
- i_type = config_GetType( p_obj, psz_name );
-
+ i_type = config_GetType( p_obj, psz_name );
if( !i_type && !psz_value )
{
/* check for "no-foo" or "nofoo" */
/* Create the variable in the input object.
* Children of the input object will be able to retreive this value
* thanks to the inheritance property of the object variables. */
- __var_Create( p_obj, psz_name, i_type );
+ var_Create( p_obj, psz_name, i_type );
switch( i_type )
{
val.psz_string = psz_value;
break;
- case VLC_VAR_LIST:
- {
- char *psz_orig, *psz_var;
- vlc_list_t *p_list = malloc(sizeof(vlc_list_t));
- val.p_list = p_list;
- p_list->i_count = 0;
-
- psz_var = psz_orig = strdup(psz_value);
- while( psz_var && *psz_var )
- {
- char *psz_item = psz_var;
- vlc_value_t val2;
- while( *psz_var && *psz_var != ',' ) psz_var++;
- if( *psz_var == ',' )
- {
- *psz_var = '\0';
- psz_var++;
- }
- val2.i_int = strtol( psz_item, NULL, 0 );
- INSERT_ELEM( p_list->p_values, p_list->i_count,
- p_list->i_count, val2 );
- /* p_list->i_count is incremented twice by INSERT_ELEM */
- p_list->i_count--;
- INSERT_ELEM( p_list->pi_types, p_list->i_count,
- p_list->i_count, VLC_VAR_INTEGER );
- }
- free( psz_orig );
- break;
- }
-
default:
goto cleanup;
}
- __var_Set( p_obj, psz_name, val );
-
- /* If that's a list, remove all elements allocated */
- if( i_type == VLC_VAR_LIST )
- FreeList( &val );
+ var_Set( p_obj, psz_name, val );
cleanup:
free( psz_name );
/* Following functions are local */
-/*****************************************************************************
- * GetUnused: find an unused (not in callback) variable from its name
- *****************************************************************************/
-static int GetUnused( vlc_object_t *p_this, const char *psz_name )
+/**
+ * Waits until the variable is inactive (i.e. not executing a callback)
+ */
+static void WaitUnused( vlc_object_t *p_this, variable_t *p_var )
{
- assert( p_this );
-
vlc_object_internals_t *p_priv = vlc_internals( p_this );
- while( true )
- {
- int i_var;
-
- i_var = Lookup( p_priv->pp_vars, p_priv->i_vars, psz_name );
- if( i_var < 0 )
- {
- return VLC_ENOVAR;
- }
-
- if( ! p_priv->pp_vars[i_var]->b_incallback )
- {
- return i_var;
- }
-
- mutex_cleanup_push( &p_priv->var_lock );
+ mutex_cleanup_push( &p_priv->var_lock );
+ while( p_var->b_incallback )
vlc_cond_wait( &p_priv->var_wait, &p_priv->var_lock );
- vlc_cleanup_pop( );
- }
-}
-
-/*****************************************************************************
- * HashString: our cool hash function
- *****************************************************************************
- * This function is not intended to be crypto-secure, we only want it to be
- * fast and not suck too much. This one is pretty fast and did 0 collisions
- * in wenglish's dictionary.
- *****************************************************************************/
-static uint32_t HashString( const char *psz_string )
-{
- uint32_t i_hash = 0;
-
- while( *psz_string )
- {
- i_hash += *psz_string++;
- i_hash += i_hash << 10;
- i_hash ^= i_hash >> 8;
- }
-
- return i_hash;
-}
-
-/*****************************************************************************
- * Insert: find an empty slot to insert a new variable
- *****************************************************************************
- * We use a recursive inner function indexed on the hash. This function does
- * nothing in the rare cases where a collision may occur, see Lookup()
- * to see how we handle them.
- * XXX: does this really need to be written recursively?
- *****************************************************************************/
-static int Insert( variable_t **pp_vars, int i_count, const char *psz_name )
-{
- if( i_count == 0 )
- {
- return 0;
- }
-
- return InsertInner( pp_vars, i_count, HashString( psz_name ) );
-}
-
-static int InsertInner( variable_t **pp_vars, int i_count, uint32_t i_hash )
-{
- int i_middle;
-
- if( i_hash <= pp_vars[0]->i_hash )
- {
- return 0;
- }
-
- if( i_hash >= pp_vars[i_count - 1]->i_hash )
- {
- return i_count;
- }
-
- i_middle = i_count / 2;
-
- /* We know that 0 < i_middle */
- if( i_hash < pp_vars[i_middle]->i_hash )
- {
- return InsertInner( pp_vars, i_middle, i_hash );
- }
-
- /* We know that i_middle + 1 < i_count */
- if( i_hash > pp_vars[i_middle + 1]->i_hash )
- {
- return i_middle + 1 + InsertInner( pp_vars + i_middle + 1,
- i_count - i_middle - 1,
- i_hash );
- }
-
- return i_middle + 1;
-}
-
-static int u32cmp( const void *key, const void *data )
-{
- const variable_t *const *pp_var = data;
- uint32_t hash = *(const uint32_t *)key ;
-
- if( hash > (*pp_var)->i_hash )
- return 1;
- if( hash < (*pp_var)->i_hash )
- return -1;
- return 0;
-}
-
-/*****************************************************************************
- * Lookup: find an existing variable given its name
- *****************************************************************************
- * We use a recursive inner function indexed on the hash. Care is taken of
- * possible hash collisions.
- *****************************************************************************/
-static int Lookup( variable_t **pp_vars, size_t i_count, const char *psz_name )
-{
- variable_t **pp_var;
- uint32_t i_hash;
-
- i_hash = HashString( psz_name );
- pp_var = bsearch( &i_hash, pp_vars, i_count, sizeof( *pp_var ), u32cmp );
-
- /* Hash not found */
- if( pp_var == NULL )
- return -1;
-
- assert( i_count > 0 );
-
- /* Find the first entry with the right hash */
- while( (pp_var > pp_vars) && (i_hash == pp_var[-1]->i_hash) )
- pp_var--;
-
- assert( (*pp_var)->i_hash == i_hash );
-
- /* Hash collision should be very unlikely, but we cannot guarantee
- * it will never happen. So we do an exhaustive search amongst all
- * entries with the same hash. Typically, there is only one anyway. */
- for( variable_t **p_end = pp_vars + i_count;
- (pp_var < p_end) && (i_hash == (*pp_var)->i_hash);
- pp_var++ )
- {
- if( !strcmp( psz_name, (*pp_var)->psz_name ) )
- return pp_var - pp_vars;
- }
-
- /* Hash found, but entry not found */
- return -1;
+ vlc_cleanup_pop( );
}
/*****************************************************************************
}
}
-/*****************************************************************************
- * InheritValue: try to inherit the value of this variable from the closest
- * ancestor objects or ultimately from the configuration.
- * The function should always be entered with the object var_lock locked.
- *****************************************************************************/
-static int InheritValue( vlc_object_t *p_this, const char *psz_name,
- vlc_value_t *p_val, int i_type )
+/**
+ * Finds the value of a variable. If the specified object does not hold a
+ * variable with the specified name, try the parent object, and iterate until
+ * the top of the tree. If no match is found, the value is read from the
+ * configuration.
+ */
+int var_Inherit( vlc_object_t *p_this, const char *psz_name, int i_type,
+ vlc_value_t *p_val )
{
+#ifndef NDEBUG
+ if (p_this != VLC_OBJECT(p_this->p_libvlc)
+ && unlikely(p_this->p_parent == NULL))
+ {
+ msg_Info (p_this, "%s(%s) on detached object", __func__, psz_name);
+ //vlc_backtrace ();
+ }
+#endif
i_type &= VLC_VAR_CLASS;
- for( vlc_object_t *obj = p_this->p_parent; obj != NULL; obj = obj->p_parent )
+ for( vlc_object_t *obj = p_this; obj != NULL; obj = obj->p_parent )
+ {
if( var_GetChecked( obj, psz_name, i_type, p_val ) == VLC_SUCCESS )
return VLC_SUCCESS;
+#ifndef NDEBUG
+ if (obj != p_this && obj != VLC_OBJECT(p_this->p_libvlc)
+ && unlikely(obj->p_parent == NULL))
+ {
+ msg_Info (p_this, "%s(%s) on detached tree [%p] %s", __func__,
+ psz_name, obj, obj->psz_object_type);
+ //vlc_backtrace ();
+ }
+#endif
+ }
/* else take value from config */
switch( i_type & VLC_VAR_CLASS )
case VLC_VAR_BOOL:
p_val->b_bool = config_GetInt( p_this, psz_name );
break;
- case VLC_VAR_LIST:
- {
- char *psz_orig, *psz_var;
- vlc_list_t *p_list = malloc(sizeof(vlc_list_t));
- p_val->p_list = p_list;
- p_list->i_count = 0;
-
- psz_var = psz_orig = config_GetPsz( p_this, psz_name );
- while( psz_var && *psz_var )
- {
- char *psz_item = psz_var;
- vlc_value_t val;
- while( *psz_var && *psz_var != ',' ) psz_var++;
- if( *psz_var == ',' )
- {
- *psz_var = '\0';
- psz_var++;
- }
- val.i_int = strtol( psz_item, NULL, 0 );
- INSERT_ELEM( p_list->p_values, p_list->i_count,
- p_list->i_count, val );
- /* p_list->i_count is incremented twice by INSERT_ELEM */
- p_list->i_count--;
- INSERT_ELEM( p_list->pi_types, p_list->i_count,
- p_list->i_count, VLC_VAR_INTEGER );
- }
- free( psz_orig );
- break;
- }
+ case VLC_VAR_ADDRESS:
+ return VLC_ENOOBJ;
default:
msg_Warn( p_this, "Could not inherit value for var %s "
"from config. Invalid Type", psz_name );
return VLC_ENOOBJ;
- break;
}
/*msg_Dbg( p_this, "Inherited value for var %s from config", psz_name );*/
return VLC_SUCCESS;
}
+/**
+ * It inherits a string as an unsigned rational number (it also accepts basic
+ * float number).
+ *
+ * It returns an error if the rational number cannot be parsed (0/0 is valid).
+ * The rational is already reduced.
+ */
+int (var_InheritURational)(vlc_object_t *object,
+ unsigned *num, unsigned *den,
+ const char *var)
+{
+ /* */
+ *num = 0;
+ *den = 0;
+
+ /* */
+ char *tmp = var_InheritString(object, var);
+ if (!tmp)
+ goto error;
+
+ char *next;
+ unsigned n = strtol(tmp, &next, 0);
+ unsigned d = strtol(*next ? &next[1] : "0", NULL, 0);
+
+ if (*next == '.') {
+ /* Interpret as a float number */
+ double r = us_atof(tmp);
+ double c = ceil(r);
+ if (c >= UINT_MAX)
+ goto error;
+ unsigned m = c;
+ if (m > 0) {
+ d = UINT_MAX / m;
+ n = r * d;
+ } else {
+ n = 0;
+ d = 0;
+ }
+ }
+
+ if (n > 0 && d > 0)
+ vlc_ureduce(num, den, n, d, 0);
+
+ free(tmp);
+ return VLC_SUCCESS;
+
+error:
+ free(tmp);
+ return VLC_EGENERIC;
+}
+
/**********************************************************************
* Trigger the callbacks.
* Tell we're in a callback, release the lock, call stored functions,
static int TriggerCallback( vlc_object_t *p_this, variable_t *p_var,
const char *psz_name, vlc_value_t oldval )
{
- int i_entries = p_var->i_entries;
- callback_entry_t *p_entries = p_var->p_entries;
-
assert( p_this );
+ int i_entries = p_var->i_entries;
+ if( i_entries == 0 )
+ return VLC_SUCCESS;
+
+ callback_entry_t *p_entries = p_var->p_entries;
vlc_object_internals_t *p_priv = vlc_internals( p_this );
+ assert( !p_var->b_incallback );
p_var->b_incallback = true;
vlc_mutex_unlock( &p_priv->var_lock );
return VLC_SUCCESS;
}
-
+#undef var_Command
/**********************************************************************
* Execute a var command on an object identified by its name
**********************************************************************/
-int __var_Command( vlc_object_t *p_this, const char *psz_name,
- const char *psz_cmd, const char *psz_arg, char **psz_msg )
+int var_Command( vlc_object_t *p_this, const char *psz_name,
+ const char *psz_cmd, const char *psz_arg, char **psz_msg )
{
vlc_object_t *p_obj = vlc_object_find_name( p_this->p_libvlc,
psz_name, FIND_CHILD );
projects / vlc / blobdiff