summaryrefslogtreecommitdiff
authorzecke <zecke>2002-07-08 18:46:40 (UTC)
committer zecke <zecke>2002-07-08 18:46:40 (UTC)
commit7d82c654e6a1df83f70b2d81aa05a1175799b66b (patch) (side-by-side diff)
tree079e87ac4dcd96c304a4e8eb7f928194fffe63f1
parent35d0ca5cc59b82af7b7ddb343ec092c3171b11ef (diff)
downloadopie-7d82c654e6a1df83f70b2d81aa05a1175799b66b.zip
opie-7d82c654e6a1df83f70b2d81aa05a1175799b66b.tar.gz
opie-7d82c654e6a1df83f70b2d81aa05a1175799b66b.tar.bz2
API clean ups
Diffstat (more/less context) (ignore whitespace changes)
-rw-r--r--noncore/net/opietooth/manager/bluebase.cpp7
-rw-r--r--noncore/net/opietooth/manager/btlistitem.cpp22
-rw-r--r--noncore/net/opietooth/manager/btlistitem.h16
-rw-r--r--noncore/net/opietooth/manager/hciconfwrapper.cpp4
-rw-r--r--noncore/net/opietooth/manager/hciconfwrapper.h4
-rw-r--r--noncore/net/opietooth/manager/main.cpp4
-rw-r--r--noncore/net/opietooth/manager/scandialog.cpp10
-rw-r--r--noncore/net/opietooth/manager/scandialog.h2
8 files changed, 31 insertions, 38 deletions
diff --git a/noncore/net/opietooth/manager/bluebase.cpp b/noncore/net/opietooth/manager/bluebase.cpp
index ff6981b..8ddc161 100644
--- a/noncore/net/opietooth/manager/bluebase.cpp
+++ b/noncore/net/opietooth/manager/bluebase.cpp
@@ -433,7 +433,8 @@ void BlueBase::startServiceActionHold( QListViewItem * item, const QPoint & poin
*/
void BlueBase::startScan() {
- ScanDialog *scan = new ScanDialog( this, "", true);
- QObject::connect( scan, SIGNAL( selectedDevices( QList<RemoteDevice>& ) ),
- this, SLOT( addSearchedDevices( QList<RemoteDevice>& ) ) );
+ ScanDialog *scan = new ScanDialog( this, "ScanDialog",
+ true, WDestructiveClose );
+ QObject::connect( scan, SIGNAL( selectedDevices( QValueList<RemoteDevice>& ) ),
+ this, SLOT( addSearchedDevices( QValueList<RemoteDevice>& ) ) );
scan->showMaximized();
diff --git a/noncore/net/opietooth/manager/btlistitem.cpp b/noncore/net/opietooth/manager/btlistitem.cpp
index ce5b141..82e7c00 100644
--- a/noncore/net/opietooth/manager/btlistitem.cpp
+++ b/noncore/net/opietooth/manager/btlistitem.cpp
@@ -6,18 +6,12 @@ namespace OpieTooth {
BTListItem::BTListItem( QListView * parent ) : QListViewItem( parent ) {
- m_name = "";
- m_mac ="";
- m_type ="";
}
BTListItem::BTListItem( QListViewItem * parent ) : QListViewItem( parent ) {
- m_name = "";
- m_mac ="";
- m_type ="";
}
// name, and then mac and then servicetype
- BTListItem::BTListItem( QListView * parent, QString name , QString mac, QString type )
+ BTListItem::BTListItem( QListView * parent, const QString &name , const QString& mac, const QString& type )
: QListViewItem( parent, name ){
@@ -28,5 +22,5 @@ namespace OpieTooth {
}
- BTListItem::BTListItem( QListViewItem * parent , QString name, QString mac, QString type )
+ BTListItem::BTListItem( QListViewItem * parent , const QString& name, const QString& mac, const QString& type )
: QListViewItem( parent, name ){
@@ -37,25 +31,25 @@ namespace OpieTooth {
- void BTListItem::setMac( QString mac ) {
+ void BTListItem::setMac( const QString& mac ) {
m_mac = mac;
}
- QString BTListItem::mac() {
+ QString BTListItem::mac() const {
return m_mac;
}
- void BTListItem::setName( QString name ) {
+ void BTListItem::setName( const QString& name ) {
m_name = name;
}
- QString BTListItem::name() {
+ QString BTListItem::name() const {
return m_name;
}
- void BTListItem::setType( QString type ) {
+ void BTListItem::setType( const QString& type ) {
m_type = type;
}
- QString BTListItem::type() {
+ QString BTListItem::type() const {
return m_type;
}
diff --git a/noncore/net/opietooth/manager/btlistitem.h b/noncore/net/opietooth/manager/btlistitem.h
index 8963caa..d7ce03b 100644
--- a/noncore/net/opietooth/manager/btlistitem.h
+++ b/noncore/net/opietooth/manager/btlistitem.h
@@ -13,14 +13,14 @@ namespace OpieTooth {
// name, and then mac and then servicetype
- BTListItem( QListView * , QString, QString, QString );
- BTListItem( QListViewItem * parent , QString, QString , QString );
+ BTListItem( QListView * , const QString&, const QString&, const QString& );
+ BTListItem( QListViewItem * parent , const QString&, const QString& , const QString& );
~BTListItem();
- void setMac( QString );
- QString mac();
- void setName( QString );
- QString name();
- void setType( QString );
- QString type();
+ void setMac( const QString& );
+ QString mac()const;
+ void setName( const QString& );
+ QString name()const;
+ void setType( const QString& );
+ QString type()const;
private:
QString m_name;
diff --git a/noncore/net/opietooth/manager/hciconfwrapper.cpp b/noncore/net/opietooth/manager/hciconfwrapper.cpp
index 0b90e87..5f74fee 100644
--- a/noncore/net/opietooth/manager/hciconfwrapper.cpp
+++ b/noncore/net/opietooth/manager/hciconfwrapper.cpp
@@ -17,9 +17,9 @@ namespace OpieTooth {
- void HciConfWrapper::setPinHelper( QString app ) {
+ void HciConfWrapper::setPinHelper( const QString& app ) {
setValue( "pin_helper" , app );
}
- void HciConfWrapper::setName( QString name ) {
+ void HciConfWrapper::setName( const QString &name ) {
qDebug ("NAME : " + name);
setValue( "name" , "\"" + name + "\"" );
diff --git a/noncore/net/opietooth/manager/hciconfwrapper.h b/noncore/net/opietooth/manager/hciconfwrapper.h
index 10738c0..de8c9a8 100644
--- a/noncore/net/opietooth/manager/hciconfwrapper.h
+++ b/noncore/net/opietooth/manager/hciconfwrapper.h
@@ -12,6 +12,6 @@ namespace OpieTooth {
~HciConfWrapper();
- void setPinHelper( QString app );
- void setName( QString name );
+ void setPinHelper( const QString& app );
+ void setName( const QString& name );
void setIscan( bool enable );
void setPscan( bool enable );
diff --git a/noncore/net/opietooth/manager/main.cpp b/noncore/net/opietooth/manager/main.cpp
index d60ef1c..a86e36d 100644
--- a/noncore/net/opietooth/manager/main.cpp
+++ b/noncore/net/opietooth/manager/main.cpp
@@ -28,8 +28,8 @@ int main( int argc, char ** argv ) {
BluetoothApp=&a;
- OpieTooth::BlueBase *t = new OpieTooth::BlueBase();
+ OpieTooth::BlueBase t;
// t.setCaption( OpieTooth::BlueBase::tr("Bluetooth Manager") );
- a.showMainWidget(t);
+ a.showMainWidget(&t);
return a.exec();
diff --git a/noncore/net/opietooth/manager/scandialog.cpp b/noncore/net/opietooth/manager/scandialog.cpp
index 0e6e93b..3d6c928 100644
--- a/noncore/net/opietooth/manager/scandialog.cpp
+++ b/noncore/net/opietooth/manager/scandialog.cpp
@@ -159,17 +159,15 @@ namespace OpieTooth {
}
- QList<RemoteDevice> *deviceList = new QList<RemoteDevice>;
- deviceList->setAutoDelete(true);
+ QValueList<RemoteDevice> deviceList;
QListViewItemIterator it( ListView1 );
for ( ; it.current(); ++it ) {
if ( ((QCheckListItem*)it.current())->isOn() ) {
- RemoteDevice * device = new RemoteDevice( it.current()->text(1), it.current()->text(0));
- deviceList->append( device );
+ RemoteDevice device( it.current()->text(1), it.current()->text(0));
+ deviceList.append( device );
}
}
qDebug("vor emit");
- emit selectedDevices( *deviceList );
- delete deviceList;
+ emit selectedDevices( deviceList );
}
diff --git a/noncore/net/opietooth/manager/scandialog.h b/noncore/net/opietooth/manager/scandialog.h
index 7c84ce2..cc890f3 100644
--- a/noncore/net/opietooth/manager/scandialog.h
+++ b/noncore/net/opietooth/manager/scandialog.h
@@ -71,5 +71,5 @@ public slots:
signals:
- void selectedDevices(QList<RemoteDevice>&);
+ void selectedDevices(QValueList<RemoteDevice>&);
};