#define CONFIG_ITEM 0x00F0
+/* Item types that use a string value (i.e. serialized in the module cache) */
+#define CONFIG_STRING_TYPES \
+ { \
+ CONFIG_ITEM_STRING, CONFIG_ITEM_FILE, CONFIG_ITEM_MODULE, \
+ CONFIG_ITEM_DIRECTORY, CONFIG_ITEM_MODULE_CAT, \
+ CONFIG_ITEM_MODULE_LIST, CONFIG_ITEM_MODULE_LIST_CAT \
+ }
+
+static inline int IsConfigStringType (int type)
+{
+ const unsigned char config_string_types[] = CONFIG_STRING_TYPES;
+
+ /* NOTE: this needs to be changed if we ever get more than 255 types */
+ return memchr (config_string_types, type, sizeof (config_string_types))
+ != NULL;
+}
+
+static inline int IsConfigIntegerType (int type)
+{
+ return (type == CONFIG_ITEM_INTEGER) || (type == CONFIG_ITEM_KEY)
+ || (type == CONFIG_ITEM_BOOL);
+}
+
+static inline int IsConfigFloatType (int type)
+{
+ return type == CONFIG_ITEM_FLOAT;
+}
+
/*******************************************************************
* All predefined categories and subcategories
*******************************************************************/
const char *psz_help;
};
+typedef union
+{
+ const char *psz;
+ int i;
+ float f;
+} module_value_t;
+
+typedef union
+{
+ int i;
+ float f;
+} module_nvalue_t;
+
struct module_config_t
{
int i_type; /* Configuration type */
char i_short; /* Optional short option name */
const char *psz_text; /* Short comment on the configuration option */
const char *psz_longtext; /* Long comment on the configuration option */
- const char *psz_value; /* Option value */
- int i_value; /* Option value */
- float f_value; /* Option value */
- int i_min; /* Option minimum value */
- int i_max; /* Option maximum value */
- float f_min; /* Option minimum value */
- float f_max; /* Option maximum value */
+ module_value_t value; /* Option value */
+ module_value_t orig;
+ module_value_t saved;
+ module_nvalue_t min;
+ module_nvalue_t max;
/* Function to call when commiting a change */
vlc_callback_t pf_callback;
const char **ppsz_action_text; /* Friendly names for actions */
int i_action; /* actions list size */
- /* Deprecated */
- const char *psz_current; /* Good option name */
- vlc_bool_t b_strict; /* Transitionnal or strict */
/* Misc */
vlc_mutex_t *p_lock; /* Lock to use when modifying the config */
vlc_bool_t b_dirty; /* Dirty flag to indicate a config change */
vlc_bool_t b_restart; /* Flag to indicate the option need a restart */
/* to take effect */
- /* Original option values */
- const char *psz_value_orig;
- int i_value_orig;
- float f_value_orig;
+ /* Deprecated */
+ const char *psz_current; /* Good option name */
+ vlc_bool_t b_strict; /* Transitionnal or strict */
/* Option values loaded from config file */
- const char *psz_value_saved;
- int i_value_saved;
- float f_value_saved;
vlc_bool_t b_autosave; /* Config will be auto-saved at exit time */
vlc_bool_t b_unsaveable; /* confg should be saved*/
};
p_config[i_config].psz_name = name; \
p_config[i_config].pf_callback = cb
-#define add_string_inner( type, name, text, longtext, advc, cb, value ) \
+#define add_string_inner( type, name, text, longtext, advc, cb, v ) \
add_typename_inner( type, name, text, longtext, advc, cb ); \
- p_config[i_config].psz_value = value
+ p_config[i_config].value.psz = v
-#define add_int_inner( type, name, text, longtext, advc, cb, value ) \
+#define add_int_inner( type, name, text, longtext, advc, cb, v ) \
add_typename_inner( type, name, text, longtext, advc, cb ); \
- p_config[i_config].i_value = value
+ p_config[i_config].value.i = v
#define set_category( i_id ) \
add_type_inner( CONFIG_CATEGORY ); \
- p_config[i_config].i_value = i_id
+ p_config[i_config].value.i = i_id
#define set_subcategory( i_id ) \
add_type_inner( CONFIG_SUBCATEGORY ); \
- p_config[i_config].i_value = i_id
+ p_config[i_config].value.i = i_id
#define set_section( text, longtext ) \
add_typedesc_inner( CONFIG_SECTION, text, longtext )
#define add_module_cat( name, i_subcategory, value, p_callback, text, longtext, advc ) \
add_string_inner( CONFIG_ITEM_MODULE_CAT, name, text, longtext, advc, p_callback, value ); \
- p_config[i_config].i_min = i_subcategory /* gruik */
+ p_config[i_config].min.i = i_subcategory /* gruik */
#define add_module_list( name, psz_caps, value, p_callback, text, longtext, advc ) \
add_string_inner( CONFIG_ITEM_MODULE_LIST, name, text, longtext, advc, p_callback, value ); \
#define add_module_list_cat( name, i_subcategory, value, p_callback, text, longtext, advc ) \
add_string_inner( CONFIG_ITEM_MODULE_LIST_CAT, name, text, longtext, advc, p_callback, value ); \
- p_config[i_config].i_min = i_subcategory /* gruik */
+ p_config[i_config].min.i = i_subcategory /* gruik */
#define add_integer( name, value, p_callback, text, longtext, advc ) \
add_int_inner( CONFIG_ITEM_INTEGER, name, text, longtext, advc, p_callback, value )
add_integer( name, value, p_callback, text, longtext, advc ); \
change_integer_range( i_min, i_max )
-#define add_float( name, value, p_callback, text, longtext, advc ) \
+#define add_float( name, v, p_callback, text, longtext, advc ) \
add_typename_inner( CONFIG_ITEM_FLOAT, name, text, longtext, advc, p_callback ); \
- p_config[i_config].f_value = value
+ p_config[i_config].value.f = v
#define add_float_with_range( name, value, f_min, f_max, p_callback, text, longtext, advc ) \
add_float( name, value, p_callback, text, longtext, advc ); \
change_float_range( f_min, f_max )
-#define add_bool( name, value, p_callback, text, longtext, advc ) \
+#define add_bool( name, v, p_callback, text, longtext, advc ) \
add_typename_inner( CONFIG_ITEM_BOOL, name, text, longtext, advc, p_callback ); \
- p_config[i_config].i_value = value
+ p_config[i_config].value.i = v
/* For renamed option */
#define add_deprecated( name, strict ) \
p_config[i_config].pi_list = (int *)list; \
p_config[i_config].ppsz_list_text = list_text;
-#define change_integer_range( min, max ) \
- p_config[i_config].i_min = min; \
- p_config[i_config].i_max = max;
+#define change_integer_range( minv, maxv ) \
+ p_config[i_config].min.i = minv; \
+ p_config[i_config].max.i = maxv;
-#define change_float_range( min, max ) \
- p_config[i_config].f_min = min; \
- p_config[i_config].f_max = max;
+#define change_float_range( minv, maxv ) \
+ p_config[i_config].min.f = minv; \
+ p_config[i_config].max.f = maxv;
#define change_action_add( pf_action, action_text ) \
if( !p_config[i_config].i_action ) \
static int ConfigStringToKey( const char * );
static char *ConfigKeyToString( int );
+static inline void freenull (const void *p)
+{
+ if (p != NULL)
+ free ((void *)p);
+}
+
+static inline char *strdupnull (const char *src)
+{
+ if (src == NULL)
+ return NULL;
+ return strdup (src);
+}
+
+static inline char *_strdupnull (const char *src)
+{
+ if (src == NULL)
+ return NULL;
+ return strdup (_(src));
+}
+
+
/*****************************************************************************
* config_GetType: get the type of a variable (bool, int, float, string)
*****************************************************************************
msg_Err( p_this, "option %s does not exist", psz_name );
return -1;
}
- if( (p_config->i_type!=CONFIG_ITEM_INTEGER) &&
- (p_config->i_type!=CONFIG_ITEM_KEY) &&
- (p_config->i_type!=CONFIG_ITEM_BOOL) )
+
+ if (!IsConfigIntegerType (p_config->i_type))
{
msg_Err( p_this, "option %s does not refer to an int", psz_name );
return -1;
}
- return p_config->i_value;
+ return p_config->value.i;
}
/*****************************************************************************
msg_Err( p_this, "option %s does not exist", psz_name );
return -1;
}
- if( p_config->i_type != CONFIG_ITEM_FLOAT )
+
+ if (!IsConfigFloatType (p_config->i_type))
{
msg_Err( p_this, "option %s does not refer to a float", psz_name );
return -1;
}
- return p_config->f_value;
+ return p_config->value.f;
}
/*****************************************************************************
char * __config_GetPsz( vlc_object_t *p_this, const char *psz_name )
{
module_config_t *p_config;
- char *psz_value = NULL;
p_config = config_FindConfig( p_this, psz_name );
msg_Err( p_this, "option %s does not exist", psz_name );
return NULL;
}
- if( (p_config->i_type!=CONFIG_ITEM_STRING) &&
- (p_config->i_type!=CONFIG_ITEM_FILE) &&
- (p_config->i_type!=CONFIG_ITEM_DIRECTORY) &&
- (p_config->i_type!=CONFIG_ITEM_MODULE_LIST) &&
- (p_config->i_type!=CONFIG_ITEM_MODULE_LIST_CAT) &&
- (p_config->i_type!=CONFIG_ITEM_MODULE_CAT) &&
- (p_config->i_type!=CONFIG_ITEM_MODULE) )
+
+ if (!IsConfigStringType (p_config->i_type))
{
msg_Err( p_this, "option %s does not refer to a string", psz_name );
return NULL;
}
-
/* return a copy of the string */
vlc_mutex_lock( p_config->p_lock );
- if( p_config->psz_value ) psz_value = strdup( p_config->psz_value );
+ char *psz_value = strdupnull (p_config->value.psz);
vlc_mutex_unlock( p_config->p_lock );
return psz_value;
msg_Warn( p_this, "option %s does not exist", psz_name );
return;
}
- if( (p_config->i_type!=CONFIG_ITEM_STRING) &&
- (p_config->i_type!=CONFIG_ITEM_FILE) &&
- (p_config->i_type!=CONFIG_ITEM_DIRECTORY) &&
- (p_config->i_type!=CONFIG_ITEM_MODULE_LIST) &&
- (p_config->i_type!=CONFIG_ITEM_MODULE_CAT) &&
- (p_config->i_type!=CONFIG_ITEM_MODULE_LIST_CAT) &&
- (p_config->i_type!=CONFIG_ITEM_MODULE) )
+
+ if (!IsConfigStringType (p_config->i_type))
{
msg_Err( p_this, "option %s does not refer to a string", psz_name );
return;
vlc_mutex_lock( p_config->p_lock );
/* backup old value */
- oldval.psz_string = p_config->psz_value;
+ oldval.psz_string = (char *)p_config->value.psz;
- if( psz_value && *psz_value ) p_config->psz_value = strdup( psz_value );
- else p_config->psz_value = NULL;
+ if ((psz_value != NULL) && *psz_value)
+ p_config->value.psz = strdup (psz_value);
+ else
+ p_config->value.psz = NULL;
p_config->b_dirty = VLC_TRUE;
- val.psz_string = p_config->psz_value;
+ val.psz_string = (char *)p_config->value.psz;
vlc_mutex_unlock( p_config->p_lock );
msg_Warn( p_this, "option %s does not exist", psz_name );
return;
}
- if( (p_config->i_type!=CONFIG_ITEM_INTEGER) &&
- (p_config->i_type!=CONFIG_ITEM_KEY) &&
- (p_config->i_type!=CONFIG_ITEM_BOOL) )
+
+ if (!IsConfigIntegerType (p_config->i_type))
{
msg_Err( p_this, "option %s does not refer to an int", psz_name );
return;
}
/* backup old value */
- oldval.i_int = p_config->i_value;
+ oldval.i_int = p_config->value.i;
/* if i_min == i_max == 0, then do not use them */
- if ((p_config->i_min == 0) && (p_config->i_max == 0))
+ if ((p_config->min.i == 0) && (p_config->max.i == 0))
{
- p_config->i_value = i_value;
+ p_config->value.i = i_value;
}
- else if (i_value < p_config->i_min)
+ else if (i_value < p_config->min.i)
{
- p_config->i_value = p_config->i_min;
+ p_config->value.i = p_config->min.i;
}
- else if (i_value > p_config->i_max)
+ else if (i_value > p_config->max.i)
{
- p_config->i_value = p_config->i_max;
+ p_config->value.i = p_config->max.i;
}
else
{
- p_config->i_value = i_value;
+ p_config->value.i = i_value;
}
p_config->b_dirty = VLC_TRUE;
- val.i_int = p_config->i_value;
+ val.i_int = p_config->value.i;
if( p_config->pf_callback )
{
msg_Warn( p_this, "option %s does not exist", psz_name );
return;
}
- if( p_config->i_type != CONFIG_ITEM_FLOAT )
+
+ if (!IsConfigFloatType (p_config->i_type))
{
msg_Err( p_this, "option %s does not refer to a float", psz_name );
return;
}
/* backup old value */
- oldval.f_float = p_config->f_value;
+ oldval.f_float = p_config->value.f;
/* if f_min == f_max == 0, then do not use them */
- if ((p_config->f_min == 0) && (p_config->f_max == 0))
+ if ((p_config->min.f == 0) && (p_config->max.f == 0))
{
- p_config->f_value = f_value;
+ p_config->value.f = f_value;
}
- else if (f_value < p_config->f_min)
+ else if (f_value < p_config->min.f)
{
- p_config->f_value = p_config->f_min;
+ p_config->value.f = p_config->min.f;
}
- else if (f_value > p_config->f_max)
+ else if (f_value > p_config->max.f)
{
- p_config->f_value = p_config->f_max;
+ p_config->value.f = p_config->max.f;
}
else
{
- p_config->f_value = f_value;
+ p_config->value.f = f_value;
}
p_config->b_dirty = VLC_TRUE;
- val.f_float = p_config->f_value;
+ val.f_float = p_config->value.f;
if( p_config->pf_callback )
{
void config_Duplicate( module_t *p_module, const module_config_t *p_orig )
{
int i, j, i_lines = 1;
- module_config_t *p_item;
+ const module_config_t *p_item;
/* Calculate the structure length */
p_module->i_config_items = 0;
{
p_module->p_config[i] = p_orig[i];
- p_module->p_config[i].i_value_orig = p_orig[i].i_value;
- p_module->p_config[i].f_value_orig = p_orig[i].f_value;
- p_module->p_config[i].i_value_saved = p_orig[i].i_value;
- p_module->p_config[i].f_value_saved = p_orig[i].f_value;
- p_module->p_config[i].psz_value_saved = 0;
-
- p_module->p_config[i].psz_type = p_orig[i].psz_type ?
- strdup( p_orig[i].psz_type ) : NULL;
- p_module->p_config[i].psz_name = p_orig[i].psz_name ?
- strdup( p_orig[i].psz_name ) : NULL;
- p_module->p_config[i].psz_current = p_orig[i].psz_current?
- strdup( p_orig[i].psz_current ) : NULL;
- p_module->p_config[i].psz_text = p_orig[i].psz_text ?
- strdup( _(p_orig[i].psz_text) ) : NULL;
- p_module->p_config[i].psz_longtext = p_orig[i].psz_longtext ?
- strdup( _(p_orig[i].psz_longtext) ) : NULL;
- p_module->p_config[i].psz_value = p_orig[i].psz_value ?
- strdup( p_orig[i].psz_value ) : NULL;
- p_module->p_config[i].psz_value_orig = p_orig[i].psz_value ?
- strdup( p_orig[i].psz_value ) : NULL;
+ if (IsConfigIntegerType (p_module->p_config[i].i_type))
+ {
+ p_module->p_config[i].orig.i = p_orig[i].value.i;
+ p_module->p_config[i].saved.i = p_orig[i].value.i;
+ }
+ else
+ if (IsConfigFloatType (p_module->p_config[i].i_type))
+ {
+ p_module->p_config[i].orig.f = p_orig[i].value.f;
+ p_module->p_config[i].saved.f = p_orig[i].value.f;
+ }
+ else
+ if (IsConfigStringType (p_module->p_config[i].i_type))
+ {
+ p_module->p_config[i].value.psz = strdupnull (p_orig[i].value.psz);
+ p_module->p_config[i].orig.psz = strdupnull (p_orig[i].value.psz);
+ p_module->p_config[i].saved.psz = NULL;
+ }
+
+ p_module->p_config[i].psz_type = strdupnull (p_orig[i].psz_type);
+ p_module->p_config[i].psz_name = strdupnull (p_orig[i].psz_name);
+ p_module->p_config[i].psz_current = strdupnull (p_orig[i].psz_current);
+ p_module->p_config[i].psz_text = _strdupnull (p_orig[i].psz_text);
+ p_module->p_config[i].psz_longtext = _strdupnull (p_orig[i].psz_longtext);
p_module->p_config[i].p_lock = &p_module->object_lock;
if( p_module->p_config[i].ppsz_list )
{
for( j = 0; j < p_orig[i].i_list; j++ )
- p_module->p_config[i].ppsz_list[j] = p_orig[i].ppsz_list[j] ?
- strdup( p_orig[i].ppsz_list[j] ) : NULL ;
+ p_module->p_config[i].ppsz_list[j] =
+ strdupnull (p_orig[i].ppsz_list[j]);
p_module->p_config[i].ppsz_list[j] = NULL;
}
}
if( p_orig[i].ppsz_list_text )
{
p_module->p_config[i].ppsz_list_text =
- malloc( (p_orig[i].i_list + 1) * sizeof(char *) );
+ calloc( (p_orig[i].i_list + 1), sizeof(char *) );
if( p_module->p_config[i].ppsz_list_text )
{
for( j = 0; j < p_orig[i].i_list; j++ )
- p_module->p_config[i].ppsz_list_text[j] = _(p_orig[i].ppsz_list_text[j]) ?
- strdup( _(p_orig[i].ppsz_list_text[j] ) ) : NULL ;
+ p_module->p_config[i].ppsz_list_text[j] =
+ strdupnull (_(p_orig[i].ppsz_list_text[j]));
p_module->p_config[i].ppsz_list_text[j] = NULL;
}
}
p_module->p_config[i].ppf_action[j] =
p_orig[i].ppf_action[j];
p_module->p_config[i].ppsz_action_text[j] =
- p_orig[i].ppsz_action_text[j] ?
- strdup( p_orig[i].ppsz_action_text[j] ) : NULL;
+ strdupnull (p_orig[i].ppsz_action_text[j]);
}
}
}
}
+
/*****************************************************************************
* config_Free: frees a duplicated module's configuration data.
*****************************************************************************
if( p_item->psz_longtext )
free( p_item->psz_longtext );
- if( p_item->psz_value )
- free( p_item->psz_value );
-
- if( p_item->psz_value_orig )
- free( p_item->psz_value_orig );
-
- if( p_item->psz_value_saved )
- free( p_item->psz_value_saved );
+ if (IsConfigStringType (p_item->i_type))
+ {
+ freenull (p_item->value.psz);
+ freenull (p_item->orig.psz);
+ freenull (p_item->saved.psz);
+ }
if( p_item->i_list )
{
for( i = 0; p_module->p_config[i].i_type != CONFIG_HINT_END; i++ )
{
- p_module->p_config[i].i_value = p_module->p_config[i].i_value_orig;
- p_module->p_config[i].f_value = p_module->p_config[i].f_value_orig;
- if( p_module->p_config[i].psz_value )
- free( p_module->p_config[i].psz_value );
- p_module->p_config[i].psz_value =
- p_module->p_config[i].psz_value_orig ?
- strdup( p_module->p_config[i].psz_value_orig ) : NULL;
+ if (IsConfigIntegerType (p_module->p_config[i].i_type))
+ p_module->p_config[i].value.i = p_module->p_config[i].orig.i;
+ else
+ if (IsConfigFloatType (p_module->p_config[i].i_type))
+ p_module->p_config[i].value.f = p_module->p_config[i].orig.f;
+ else
+ if (IsConfigStringType (p_module->p_config[i].i_type))
+ {
+ freenull (p_module->p_config[i].value.psz);
+ p_module->p_config[i].value.psz =
+ strdupnull (p_module->p_config[i].orig.psz);
+ }
}
}
if ((fread (bom, 1, 3, file) == 3)
&& (memcmp (bom, "\xEF\xBB\xBF", 3) == 0))
- convert = strdup;
+ convert = strdupnull;
else
rewind (file); // no BOM, rewind
case CONFIG_ITEM_INTEGER:
if( !*psz_option_value )
break; /* ignore empty option */
- p_item->i_value = strtol( psz_option_value, 0, 0 );
- p_item->i_value_saved = p_item->i_value;
+ p_item->value.i = strtol( psz_option_value, 0, 0 );
+ p_item->saved.i = p_item->value.i;
#if 0
msg_Dbg( p_this, "option \"%s\", value %i",
p_item->psz_name, p_item->i_value );
case CONFIG_ITEM_FLOAT:
if( !*psz_option_value )
break; /* ignore empty option */
- p_item->f_value = (float)i18n_atof( psz_option_value);
- p_item->f_value_saved = p_item->f_value;
+ p_item->value.f = (float)i18n_atof( psz_option_value);
+ p_item->saved.f = p_item->value.f;
#if 0
msg_Dbg( p_this, "option \"%s\", value %f",
p_item->psz_name, (double)p_item->f_value );
case CONFIG_ITEM_KEY:
if( !*psz_option_value )
break; /* ignore empty option */
- p_item->i_value = ConfigStringToKey(psz_option_value);
- p_item->i_value_saved = p_item->i_value;
+ p_item->value.i = ConfigStringToKey(psz_option_value);
+ p_item->saved.i = p_item->value.i;
break;
default:
vlc_mutex_lock( p_item->p_lock );
/* free old string */
- if( p_item->psz_value )
- free( p_item->psz_value );
+ freenull (p_item->value.psz);
+
+ p_item->value.psz = convert (psz_option_value);
- p_item->psz_value = *psz_option_value ?
- convert( psz_option_value ) : NULL;
+ freenull (p_item->saved.psz);
+ p_item->saved.psz = NULL;
- if( p_item->psz_value_saved )
- free( p_item->psz_value_saved );
- p_item->psz_value_saved = NULL;
- if( !p_item->psz_value || !p_item->psz_value_orig ||
- (p_item->psz_value && p_item->psz_value_orig &&
- strcmp(p_item->psz_value,p_item->psz_value_orig)))
- p_item->psz_value_saved = p_item->psz_value ?
- convert( p_item->psz_value ) : NULL;
+ if( !p_item->value.psz || !p_item->orig.psz ||
+ (p_item->value.psz && p_item->orig.psz &&
+ strcmp(p_item->value.psz, p_item->orig.psz)))
+ p_item->saved.psz = convert (p_item->value.psz);
vlc_mutex_unlock( p_item->p_lock );
p_item++ )
{
char *psz_key;
- int i_value = p_item->i_value;
- float f_value = p_item->f_value;
- const char *psz_value = p_item->psz_value;
+ int i_value = p_item->value.i;
+ float f_value = p_item->value.f;
+ const char *psz_value = p_item->value.psz;
if( p_item->i_type & CONFIG_HINT )
/* ignore hints */
if( b_autosave && !p_item->b_autosave )
{
- i_value = p_item->i_value_saved;
- f_value = p_item->f_value_saved;
- psz_value = p_item->psz_value_saved;
- if( !psz_value ) psz_value = p_item->psz_value_orig;
+ i_value = p_item->saved.i;
+ f_value = p_item->saved.f;
+ psz_value = p_item->saved.psz;
+ if( !psz_value ) psz_value = p_item->orig.psz;
}
else
{
fprintf( file, "# %s (%s)\n", p_item->psz_text,
(p_item->i_type == CONFIG_ITEM_BOOL) ?
_("boolean") : _("integer") );
- if( i_value == p_item->i_value_orig )
+ if( i_value == p_item->orig.i )
fputc ('#', file);
fprintf( file, "%s=%i\n", p_item->psz_name, i_value );
- p_item->i_value_saved = i_value;
+ p_item->saved.i = i_value;
break;
case CONFIG_ITEM_KEY:
if( p_item->psz_text )
fprintf( file, "# %s (%s)\n", p_item->psz_text,
_("key") );
- if( i_value == p_item->i_value_orig )
+ if( i_value == p_item->orig.i )
fputc ('#', file);
psz_key = ConfigKeyToString( i_value );
fprintf( file, "%s=%s\n", p_item->psz_name,
psz_key ? psz_key : "" );
- if ( psz_key ) free( psz_key );
+ freenull (psz_key);
- p_item->i_value_saved = i_value;
+ p_item->saved.i = i_value;
break;
case CONFIG_ITEM_FLOAT:
if( p_item->psz_text )
fprintf( file, "# %s (%s)\n", p_item->psz_text,
_("float") );
- if( f_value == p_item->f_value_orig )
+ if( f_value == p_item->orig.f )
fputc ('#', file);
fprintf( file, "%s=%f\n", p_item->psz_name, (double)f_value );
- p_item->f_value_saved = f_value;
+ p_item->saved.f = f_value;
break;
default:
if( p_item->psz_text )
fprintf( file, "# %s (%s)\n", p_item->psz_text,
_("string") );
- if( (!psz_value && !p_item->psz_value_orig) ||
- (psz_value && p_item->psz_value_orig &&
- !strcmp( psz_value, p_item->psz_value_orig )) )
+ if( (!psz_value && !p_item->orig.psz) ||
+ (psz_value && p_item->orig.psz &&
+ !strcmp( psz_value, p_item->orig.psz )) )
fputc ('#', file);
fprintf( file, "%s=%s\n", p_item->psz_name,
- psz_value ? psz_value : "" );
+ psz_value ?: "" );
if( b_autosave && !p_item->b_autosave ) break;
- if( p_item->psz_value_saved ) free( p_item->psz_value_saved );
- p_item->psz_value_saved = 0;
- if( (psz_value && p_item->psz_value_orig &&
- strcmp( psz_value, p_item->psz_value_orig )) ||
- !psz_value || !p_item->psz_value_orig)
- p_item->psz_value_saved = psz_value ? strdup(psz_value):0;
+ freenull (p_item->saved.psz);
+ if( (psz_value && p_item->orig.psz &&
+ strcmp( psz_value, p_item->orig.psz )) ||
+ !psz_value || !p_item->orig.psz)
+ p_item->saved.psz = strdupnull (psz_value);
+ else
+ p_item->saved.psz = NULL;
}
}