[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    extragear/multimedia/amarok/src/configdialog
From:       Seb Ruiz <ruiz () kde ! org>
Date:       2008-11-02 10:42:56
Message-ID: 1225622576.730918.4868.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 878975 by seb:

use a QString in preference to a QByteArray


 M  +6 -3      ConfigDialog.cpp  
 M  +1 -1      ConfigDialog.h  


--- trunk/extragear/multimedia/amarok/src/configdialog/ConfigDialog.cpp #878974:878975
@@ -85,10 +85,13 @@
 }
 
 /** Show page by object name */
-void Amarok2ConfigDialog::showPageByName( const QByteArray& page )
+void Amarok2ConfigDialog::showPageByName( const QString& page )
 {
-    for( int index = 0; index < m_pageList.count(); index++ ) {
-        if ( m_pageList[index]->objectName() == page ) {
+    DEBUG_BLOCK
+    for( int index = 0; index < m_pageList.count(); index++ )
+    {
+        if( m_pageList[index]->objectName() == page )
+        {
             KConfigDialog::setCurrentPage( qobject_cast<KPageWidgetItem*>( m_pageList[index] ) );
             return;
         }
--- trunk/extragear/multimedia/amarok/src/configdialog/ConfigDialog.h #878974:878975
@@ -31,7 +31,7 @@
 
         void addPage( ConfigDialogBase *page, const QString &itemName, const QString &pixmapName,
                       const QString &header = QString(), bool manage=true);
-        void showPageByName( const QByteArray& page );
+        void showPageByName( const QString& page );
 
     public slots:
         /**
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic