summaryrefslogtreecommitdiff
authormickeyl <mickeyl>2005-07-09 13:42:36 (UTC)
committer mickeyl <mickeyl>2005-07-09 13:42:36 (UTC)
commit5dbfa604bcc5dd32401d862372af806bfe674e89 (patch) (side-by-side diff)
tree38abbfa6cf68bca36ce397e5e1e69c12f8674dc2
parent7bccb3143fbfb4dd70cc22d5233e6275245d0285 (diff)
downloadopie-5dbfa604bcc5dd32401d862372af806bfe674e89.zip
opie-5dbfa604bcc5dd32401d862372af806bfe674e89.tar.gz
opie-5dbfa604bcc5dd32401d862372af806bfe674e89.tar.bz2
- sanitize 'ret' scoping
- fix orthography in popupmenu - remove 'add to group', since groups seem unfinished :/
Diffstat (more/less context) (ignore whitespace changes)
-rw-r--r--noncore/net/opietooth/manager/bluebase.cpp58
1 files changed, 27 insertions, 31 deletions
diff --git a/noncore/net/opietooth/manager/bluebase.cpp b/noncore/net/opietooth/manager/bluebase.cpp
index 7954cc3..af1cd23 100644
--- a/noncore/net/opietooth/manager/bluebase.cpp
+++ b/noncore/net/opietooth/manager/bluebase.cpp
@@ -158,5 +158,5 @@ void BlueBase::writeConfig()
void BlueBase::writeToHciConfig()
{
- owarn << "writeToHciConfig" << oendl;
+ owarn << "writeToHciConfig" << oendl;
HciConfWrapper hciconf ( "/etc/bluetooth/hcid.conf" );
hciconf.load();
@@ -319,34 +319,30 @@ void BlueBase::startServiceActionHold( QListViewItem * item, const QPoint & poin
QPopupMenu *menu = new QPopupMenu();
- int ret=0;
- if ( ((BTListItem*)item)->type() == "device")
+ if ( static_cast<BTListItem*>( item )->type() == "device")
{
-
QPopupMenu *groups = new QPopupMenu();
- menu->insertItem( ((BTDeviceItem*)item)->name(),0 );
- menu->insertSeparator(1);
- menu->insertItem( tr("rescan sevices"), 2);
- menu->insertItem( tr("to group"), groups , 3);
- menu->insertItem( tr("delete"), 4);
-
- ret = menu->exec( point , 0);
+ menu->insertItem( static_cast<BTDeviceItem*>( item )->name(), 0 );
+ menu->insertSeparator( 1 );
+ menu->insertItem( tr( "&Rescan services" ), 2);
+ // menu->insertItem( tr( "&Add to group" ), groups, 3);
+ menu->insertItem( tr( "&Delete"), 4);
+ int ret = menu->exec( point, 0);
switch(ret)
{
- case -1:
- break;
- case 2:
- addServicesToDevice( (BTDeviceItem*)item );
- break;
-
- case 4:
- // deletes childs too
- delete item;
- break;
+ case -1:
+ break;
+ case 2:
+ addServicesToDevice( static_cast<BTDeviceItem*>( item ) );
+ break;
+
+ case 4:
+ // deletes childs too
+ delete item;
+ break;
}
- delete groups;
-
+ // delete groups;
}
@@ -370,5 +366,5 @@ void BlueBase::startServiceActionHold( QListViewItem * item, const QPoint & poin
if ( it != list.end() )
{
- owarn << "Searching id " << it.key() << " " << it.data().latin1() << "" << oendl;
+ owarn << "Searching id " << it.key() << " " << it.data().latin1() << "" << oendl;
popup = m_popHelper.find( it.key(),
service->services(),
@@ -377,16 +373,16 @@ void BlueBase::startServiceActionHold( QListViewItem * item, const QPoint & poin
else
{
- owarn << "Empty" << oendl;
+ owarn << "Empty" << oendl;
}
if ( popup == 0l )
{
- owarn << "factory returned 0l" << oendl;
+ owarn << "factory returned 0l" << oendl;
popup = new QPopupMenu();
}
int test1 = popup->insertItem( tr("Test1:"), 2);
- ret = popup->exec( point );
- owarn << "returned from exec() " << oendl;
+ int ret = popup->exec( point );
+ owarn << "returned from exec() " << oendl;
if ( ret == -1 )
{
@@ -409,5 +405,5 @@ void BlueBase::startServiceActionHold( QListViewItem * item, const QPoint & poin
void BlueBase::addServicesToDevice( BTDeviceItem * item )
{
- odebug << "addServicesToDevice" << oendl;
+ odebug << "addServicesToDevice" << oendl;
// row of mac adress text(3)
RemoteDevice device = item->remoteDevice();
@@ -426,5 +422,5 @@ void BlueBase::addServicesToDevice( BTDeviceItem * item )
void BlueBase::addServicesToDevice( const QString& device, Services::ValueList servicesList )
{
- odebug << "fill services list" << oendl;
+ odebug << "fill services list" << oendl;
QMap<QString,BTDeviceItem*>::Iterator it;
@@ -610,5 +606,5 @@ void BlueBase::deviceActive( const RemoteDevice &device )
void BlueBase::deviceActive( const QString& device, bool connected )
{
- odebug << "deviceActive slot" << oendl;
+ odebug << "deviceActive slot" << oendl;
QMap<QString,BTDeviceItem*>::Iterator it;