Reworked to fit into the Opie style guidelines.

2 files changed, 89 insertions, 51 deletions

diff --git a/core/applets/irdaapplet/irda.cpp b/core/applets/irdaapplet/irda.cpp
index cea836e..d57104b 100644
--- a/core/applets/irdaapplet/irda.cpp
+++ b/core/applets/irdaapplet/irda.cpp
@@ -43,4 +43,11 @@
 
-IrdaApplet::IrdaApplet ( QWidget *parent, const char *name )
-                : QWidget ( parent, name )
+IrdaApplet::IrdaApplet ( QWidget *parent, const char *name ) :
+    QWidget ( parent, name ),
+    m_irda_active( false ),
+    m_irda_discovery_active( false ),
+    m_receive_active( false ),
+    m_receive_state_changed( false ),
+    m_popup( 0 ),
+    m_wasOn( false ),
+    m_wasDiscover( false )
 {
@@ -49,20 +56,21 @@ IrdaApplet::IrdaApplet ( QWidget *parent, const char *name )
 
-    m_sockfd = ::socket ( PF_INET, SOCK_DGRAM, IPPROTO_IP );
-
-    m_irdaOnPixmap = Opie::Core::OResource::loadPixmap( "irdaapplet/irdaon", Opie::Core::OResource::SmallIcon );
-    m_irdaOffPixmap = Opie::Core::OResource::loadPixmap( "irdaapplet/irdaoff", Opie::Core::OResource::SmallIcon );
-    m_irdaDiscoveryOnPixmap = Opie::Core::OResource::loadPixmap( "irdaapplet/magglass", Opie::Core::OResource::SmallIcon );
-    m_receiveActivePixmap = Opie::Core::OResource::loadPixmap( "irdaapplet/receive", Opie::Core::OResource::SmallIcon );
-
-        m_irda_active = false;
-        m_irda_discovery_active = false;
-        m_receive_active = false;
-        m_receive_state_changed = false;
-        m_popup = 0;
-                m_wasOn = false;
-                m_wasDiscover = false;
+    if (m_sockfd = ::socket ( PF_INET, SOCK_DGRAM, IPPROTO_IP ) == -1)
+        perror ( "failed grabbing IrDA socket" );
+
+    m_irdaOnPixmap =
+        Opie::Core::OResource::loadPixmap( "irdaapplet/irdaon",
+                                           Opie::Core::OResource::SmallIcon );
+    m_irdaOffPixmap =
+        Opie::Core::OResource::loadPixmap( "irdaapplet/irdaoff",
+                                           Opie::Core::OResource::SmallIcon );
+    m_irdaDiscoveryOnPixmap =
+        Opie::Core::OResource::loadPixmap( "irdaapplet/magglass",
+                                           Opie::Core::OResource::SmallIcon );
+    m_receiveActivePixmap =
+        Opie::Core::OResource::loadPixmap( "irdaapplet/receive",
+                                           Opie::Core::OResource::SmallIcon );
 
         QCopChannel* chan = new QCopChannel("QPE/IrDaApplet", this );
-        connect(chan, SIGNAL(received(const QCString&,const QByteArray&) ),
-                this, SLOT(slotMessage(const QCString&,const QByteArray&) ) );
+    connect(chan, SIGNAL(received(const QCString&,const QByteArray&) ), this,
+            SLOT(slotMessage(const QCString&,const QByteArray&) ) );
 }
@@ -96,4 +104,10 @@ void IrdaApplet::popup ( QString message, QString icon )
         else
-        m_popup-> insertItem ( QIconSet ( Opie::Core::OResource::loadPixmap ( icon, Opie::Core::OResource::SmallIcon )),
-                               message, 0 );
+        m_popup-> insertItem ( QIconSet (
+                               Opie::Core::OResource::loadPixmap (
+                                                icon,
+                                                Opie::Core::OResource::SmallIcon
+                                                                 )
+                                        ),
+                               message, 0
+                             );
 
@@ -101,4 +115,7 @@ void IrdaApplet::popup ( QString message, QString icon )
         QSize s = m_popup-> sizeHint ( );
-        m_popup-> popup ( QPoint ( p. x ( ) + ( width ( ) / 2 ) - ( s. width ( ) / 2 ),
-                                   p. y ( ) - s. height ( )));
+    m_popup-> popup ( QPoint (
+                            p. x ( ) + ( width ( ) / 2 ) - ( s. width ( ) / 2 ),
+                            p. y ( ) - s. height ( )
+                             )
+                    );
 
@@ -117,3 +134,3 @@ bool IrdaApplet::checkIrdaStatus ( )
 
-        if ( ::ioctl ( m_sockfd, SIOCGIFFLAGS, &ifr ) < 0 )
+    if ( m_sockfd < 0 || ::ioctl ( m_sockfd, SIOCGIFFLAGS, &ifr ) < 0 )
                 return false;
@@ -128,8 +145,7 @@ bool IrdaApplet::setIrdaStatus ( bool b )
 
-        if ( ::ioctl ( m_sockfd, SIOCGIFFLAGS, &ifr ) < 0 )
+    if ( m_sockfd < 0 || ::ioctl ( m_sockfd, SIOCGIFFLAGS, &ifr ) < 0 )
                 return false;
 
-        if ( b ) {
+    if ( b )
                 ifr. ifr_flags |= IFF_UP;
-        }
         else {
@@ -140,3 +156,3 @@ bool IrdaApplet::setIrdaStatus ( bool b )
 
-        if ( ::ioctl ( m_sockfd, SIOCSIFFLAGS, &ifr ) < 0 )
+    if ( m_sockfd < 0 || ::ioctl ( m_sockfd, SIOCSIFFLAGS, &ifr ) < 0 )
                 return false;
@@ -198,4 +214,7 @@ void IrdaApplet::showDiscovered ( )
 
-                // since it is /proc we _must_ use QTextStream
-                QStringList list =  QStringList::split ( "\n", QTextStream ( &discovery ). read ( ));
+        /*!
+         * @note To read /proc, it makes more sense to use QTextStream.
+         */
+        QStringList list =
+            QStringList::split ( "\n", QTextStream ( &discovery ). read ( ));
 
@@ -206,3 +225,6 @@ void IrdaApplet::showDiscovered ( )
 
-                for ( QStringList::Iterator lit = list. begin ( ); lit != list. end ( ); ++lit ) {
+        for ( QStringList::Iterator lit = list. begin ( );
+              lit != list. end ( ); ++lit
+            )
+        {
                         const QString &line = *lit;
@@ -210,3 +232,5 @@ void IrdaApplet::showDiscovered ( )
                         if ( line. startsWith ( "nickname:" )) {
-                                discoveredDevice = line. mid ( line. find ( ':' ) + 2, line. find ( ',' ) - line. find ( ':' ) - 2 );
+                discoveredDevice =
+                    line. mid ( line. find ( ':' ) + 2, line. find ( ',' ) -
+                                line. find ( ':' ) - 2 );
                                 deviceAddr       = line. mid ( line. find ( "daddr:" ) + 9, 8 );
@@ -225,3 +249,4 @@ void IrdaApplet::showDiscovered ( )
                 for ( it = m_devices. begin ( ); it != m_devices. end ( ); ) {
-                        // odebug << "IrdaMon: delete " + it.currentKey() + "=" + *devicesAvailable[it.currentKey()] + "?" << oendl;
+            // odebug << "IrdaMon: delete " + it.currentKey() + "=" +
+            //           *devicesAvailable[it.currentKey()] + "?" << oendl;
 
@@ -233,8 +258,9 @@ void IrdaApplet::showDiscovered ( )
                                 tmp++;
-                                m_devices. remove ( it ); // in contrast to QValueListIterator this remove doesn't return the next Iterator
-                                it = tmp;
+                m_devices. remove ( it );
+                ///< @note In contrast to QValueListIterator this remove doesn't
+                ///< return the next Iterator
 
+                it = tmp;
                                 qcopsend = true;
-                        }
-                        else
+            } else
                                 it++;
@@ -261,4 +287,7 @@ void IrdaApplet::mousePressEvent ( QMouseEvent * )
 
-                for ( QMap<QString, QString>::Iterator it = m_devices. begin ( ); it != m_devices. end ( ); ++it )
+        for ( QMap<QString, QString>::Iterator it = m_devices. begin ( );
+              it != m_devices. end ( ); ++it )
+        {
                         menu-> insertItem ( *it );
+        }
 
@@ -270,3 +299,6 @@ void IrdaApplet::mousePressEvent ( QMouseEvent * )
         if ( m_irda_active ) {
-                menu-> insertItem ( m_irda_discovery_active ? tr( "Disable Discovery" ) : tr( "Enable Discovery" ), 1 );
+        menu-> insertItem (
+            m_irda_discovery_active ? tr( "Disable Discovery" ) :
+                                      tr( "Enable Discovery" ), 1
+                          );
                 
@@ -274,3 +306,6 @@ void IrdaApplet::mousePressEvent ( QMouseEvent * )
                 if( Ir::supported() )
-                        menu-> insertItem ( m_receive_active ? tr( "Disable Receive" ) : tr( "Enable Receive" ), 2 );
+            menu-> insertItem (
+                m_receive_active ? tr( "Disable Receive" ) :
+                                   tr( "Enable Receive" ), 2
+                              );
         }
@@ -280,3 +315,4 @@ void IrdaApplet::mousePressEvent ( QMouseEvent * )
 
-        p = QPoint ( p. x ( ) + ( width ( ) / 2 ) - ( s. width ( ) / 2 ), p. y ( ) - s. height ( ));
+    p = QPoint ( p. x ( ) + ( width ( ) / 2 ) - ( s. width ( ) / 2 ), p. y ( ) -
+                 s. height ( ));
 
@@ -317,5 +353,8 @@ void IrdaApplet::timerEvent ( QTimerEvent * )
 
-        if (( m_irda_active != oldactive ) || ( m_irda_discovery_active != olddiscovery ) || receiveUpdate )
+    if (( m_irda_active != oldactive ) ||
+        ( m_irda_discovery_active != olddiscovery ) || receiveUpdate )
+    {
                 update ( );
 }
+}
 
@@ -333,8 +372,8 @@ void IrdaApplet::paintEvent ( QPaintEvent * )
 }
-/*
- * We know 3 calls
- * a) enable
- * b) disable
- * a and b will temp enable the IrDa device and disable will disable it again if it wasn't on
- * c) listDevices: We will return a list of known devices
+
+/*!
+ * We recognize 3 events:
+ *   - enable: Attempt to enable the IrDA device
+ *   - disable: Attempt to disable the IrDA device if it was already enabled
+ *   - listDevices: Returns a list of known IrDA devices
  */
@@ -370 +409,2 @@ void IrdaApplet::slotMessage( const QCString& str, const QByteArray&  ) {
 EXPORT_OPIE_APPLET_v1( IrdaApplet )
+
diff --git a/core/applets/irdaapplet/irda.h b/core/applets/irdaapplet/irda.h
index f713bbe..2cbf4f7 100644
--- a/core/applets/irdaapplet/irda.h
+++ b/core/applets/irdaapplet/irda.h
@@ -54,3 +54,2 @@ private:
 
-private:
         QPixmap m_irdaOnPixmap;
@@ -71,7 +70,6 @@ private:
 
-                bool m_wasOn; // if IrDa was enabled
+        bool m_wasOn; ///< If IrDA was enabled this is true
         bool m_wasDiscover;
 };
-
-
 #endif // __OPIE_IRDA_APPLET_H__
+