Commit 612de111 authored by Jean-Baptiste Kempf's avatar Jean-Baptiste Kempf

Qt4: Document and assert when needed.

parent 83e10e11
...@@ -32,6 +32,8 @@ ...@@ -32,6 +32,8 @@
#include <QLabel> #include <QLabel>
#include <QMessageBox> #include <QMessageBox>
#include <assert.h>
VLCProfileSelector::VLCProfileSelector( QWidget *_parent ): QWidget( _parent ) VLCProfileSelector::VLCProfileSelector( QWidget *_parent ): QWidget( _parent )
{ {
QHBoxLayout *layout = new QHBoxLayout( this ); QHBoxLayout *layout = new QHBoxLayout( this );
...@@ -111,17 +113,24 @@ void VLCProfileSelector::editProfile() ...@@ -111,17 +113,24 @@ void VLCProfileSelector::editProfile()
void VLCProfileSelector::editProfile( const QString& qs, const QString& value ) void VLCProfileSelector::editProfile( const QString& qs, const QString& value )
{ {
/* Create the Profile Editor */
VLCProfileEditor *editor = new VLCProfileEditor( qs, value, this ); VLCProfileEditor *editor = new VLCProfileEditor( qs, value, this );
/* Show it */
if( QDialog::Accepted == editor->exec() ) if( QDialog::Accepted == editor->exec() )
{ {
/* New Profile */
if( qs.isEmpty() ) if( qs.isEmpty() )
profileBox->addItem( editor->name, QVariant( editor->transcodeValue() ) ); profileBox->addItem( editor->name, QVariant( editor->transcodeValue() ) );
/* Update old profile */
else else
{ {
/* Look for the profile */
int i_profile = profileBox->findText( qs ); int i_profile = profileBox->findText( qs );
assert( i_profile != -1 );
profileBox->setItemText( i_profile, editor->name ); profileBox->setItemText( i_profile, editor->name );
profileBox->setItemData( i_profile, QVariant( editor->transcodeValue() ) ); profileBox->setItemData( i_profile, QVariant( editor->transcodeValue() ) );
/* Force mrl recreation */
updateOptions( i_profile ); updateOptions( i_profile );
} }
} }
......
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