Commit 53dd2501 authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

variables: anonymize union

parent 2850821a
...@@ -49,7 +49,7 @@ typedef struct callback_entry_t ...@@ -49,7 +49,7 @@ typedef struct callback_entry_t
vlc_callback_t pf_value_callback; vlc_callback_t pf_value_callback;
vlc_list_callback_t pf_list_callback; vlc_list_callback_t pf_list_callback;
void * p_callback; void * p_callback;
} u; };
void * p_data; void * p_data;
} callback_entry_t; } callback_entry_t;
...@@ -270,7 +270,7 @@ static void TriggerCallback(vlc_object_t *obj, variable_t *var, ...@@ -270,7 +270,7 @@ static void TriggerCallback(vlc_object_t *obj, variable_t *var,
vlc_mutex_unlock(&priv->var_lock); vlc_mutex_unlock(&priv->var_lock);
for (size_t i = 0; i < count; i++) for (size_t i = 0; i < count; i++)
entries[i].u.pf_value_callback(obj, name, prev, var->val, entries[i].pf_value_callback(obj, name, prev, var->val,
entries[i].p_data); entries[i].p_data);
vlc_mutex_lock(&priv->var_lock); vlc_mutex_lock(&priv->var_lock);
...@@ -295,7 +295,7 @@ static void TriggerListCallback(vlc_object_t *obj, variable_t *var, ...@@ -295,7 +295,7 @@ static void TriggerListCallback(vlc_object_t *obj, variable_t *var,
vlc_mutex_unlock(&priv->var_lock); vlc_mutex_unlock(&priv->var_lock);
for (size_t i = 0; i < count; i++) for (size_t i = 0; i < count; i++)
entries[i].u.pf_list_callback(obj, name, action, val, entries[i].pf_list_callback(obj, name, action, val,
entries[i].p_data); entries[i].p_data);
vlc_mutex_lock(&priv->var_lock); vlc_mutex_lock(&priv->var_lock);
...@@ -906,7 +906,7 @@ static int AddCallback( vlc_object_t *p_this, const char *psz_name, ...@@ -906,7 +906,7 @@ static int AddCallback( vlc_object_t *p_this, const char *psz_name,
{ {
vlc_mutex_unlock( &p_priv->var_lock ); vlc_mutex_unlock( &p_priv->var_lock );
msg_Err( p_this, "cannot add callback %p to nonexistent variable '%s'", msg_Err( p_this, "cannot add callback %p to nonexistent variable '%s'",
(void *)entry.u.p_callback, psz_name ); entry.p_callback, psz_name );
return VLC_ENOVAR; return VLC_ENOVAR;
} }
...@@ -948,7 +948,7 @@ int var_AddCallback( vlc_object_t *p_this, const char *psz_name, ...@@ -948,7 +948,7 @@ int var_AddCallback( vlc_object_t *p_this, const char *psz_name,
vlc_callback_t pf_callback, void *p_data ) vlc_callback_t pf_callback, void *p_data )
{ {
callback_entry_t entry; callback_entry_t entry;
entry.u.pf_value_callback = pf_callback; entry.pf_value_callback = pf_callback;
entry.p_data = p_data; entry.p_data = p_data;
return AddCallback(p_this, psz_name, entry, vlc_value_callback); return AddCallback(p_this, psz_name, entry, vlc_value_callback);
...@@ -984,13 +984,13 @@ static int DelCallback( vlc_object_t *p_this, const char *psz_name, ...@@ -984,13 +984,13 @@ static int DelCallback( vlc_object_t *p_this, const char *psz_name,
for( i_entry = p_table->i_entries ; i_entry-- ; ) for( i_entry = p_table->i_entries ; i_entry-- ; )
{ {
if( p_table->p_entries[i_entry].u.p_callback == entry.u.p_callback if( p_table->p_entries[i_entry].p_callback == entry.p_callback
&& p_table->p_entries[i_entry].p_data == entry.p_data ) && p_table->p_entries[i_entry].p_data == entry.p_data )
{ {
break; break;
} }
#ifndef NDEBUG #ifndef NDEBUG
else if( p_table->p_entries[i_entry].u.p_callback == entry.u.p_callback ) else if( p_table->p_entries[i_entry].p_callback == entry.p_callback )
b_found_similar = true; b_found_similar = true;
#endif #endif
} }
...@@ -1025,7 +1025,7 @@ int var_DelCallback( vlc_object_t *p_this, const char *psz_name, ...@@ -1025,7 +1025,7 @@ int var_DelCallback( vlc_object_t *p_this, const char *psz_name,
vlc_callback_t pf_callback, void *p_data ) vlc_callback_t pf_callback, void *p_data )
{ {
callback_entry_t entry; callback_entry_t entry;
entry.u.pf_value_callback = pf_callback; entry.pf_value_callback = pf_callback;
entry.p_data = p_data; entry.p_data = p_data;
return DelCallback(p_this, psz_name, entry, vlc_value_callback); return DelCallback(p_this, psz_name, entry, vlc_value_callback);
...@@ -1066,7 +1066,7 @@ int var_AddListCallback( vlc_object_t *p_this, const char *psz_name, ...@@ -1066,7 +1066,7 @@ int var_AddListCallback( vlc_object_t *p_this, const char *psz_name,
vlc_list_callback_t pf_callback, void *p_data ) vlc_list_callback_t pf_callback, void *p_data )
{ {
callback_entry_t entry; callback_entry_t entry;
entry.u.pf_list_callback = pf_callback; entry.pf_list_callback = pf_callback;
entry.p_data = p_data; entry.p_data = p_data;
return AddCallback(p_this, psz_name, entry, vlc_list_callback); return AddCallback(p_this, psz_name, entry, vlc_list_callback);
...@@ -1082,7 +1082,7 @@ int var_DelListCallback( vlc_object_t *p_this, const char *psz_name, ...@@ -1082,7 +1082,7 @@ int var_DelListCallback( vlc_object_t *p_this, const char *psz_name,
vlc_list_callback_t pf_callback, void *p_data ) vlc_list_callback_t pf_callback, void *p_data )
{ {
callback_entry_t entry; callback_entry_t entry;
entry.u.pf_list_callback = pf_callback; entry.pf_list_callback = pf_callback;
entry.p_data = p_data; entry.p_data = p_data;
return DelCallback(p_this, psz_name, entry, vlc_list_callback); return DelCallback(p_this, psz_name, entry, vlc_list_callback);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment