- combo->addItem( qfu(p_item->ppsz_list_text ?
- p_item->ppsz_list_text[i_index] :
- p_item->ppsz_list[i_index] ),
- QVariant( p_item->ppsz_list[i_index] ) );
- if( p_item->value.psz && !strcmp( p_item->value.psz,
- p_item->ppsz_list[i_index] ) )
+ vlc_value_t val;
+ val.psz_string = strdup(p_module_config->value.psz);
+
+ p_module_config->pf_update_list(p_this, p_item->psz_name, val, val, NULL);
+
+ // assume in any case that dirty was set to true
+ // because lazy programmes will use the same callback for
+ // this, like the one behind the refresh push button?
+ p_module_config->b_dirty = false;
+
+ free( val.psz_string );
+ }
+
+ for( int i_index = 0; i_index < p_module_config->i_list; i_index++ )
+ {
+ if( !p_module_config->ppsz_list[i_index] )
+ {
+ combo->addItem( "", QVariant(""));
+ if( !p_item->value.psz )
+ combo->setCurrentIndex( combo->count() - 1 );
+ continue;
+ }
+ combo->addItem( qfu((p_module_config->ppsz_list_text &&
+ p_module_config->ppsz_list_text[i_index])?
+ p_module_config->ppsz_list_text[i_index] :
+ p_module_config->ppsz_list[i_index] ),
+ QVariant( qfu(p_module_config->ppsz_list[i_index] )) );
+ if( p_item->value.psz && !strcmp( p_module_config->value.psz,
+ p_module_config->ppsz_list[i_index] ) )