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
@@ -41,30 +41,38 @@
 
 //===========================================================================
 
-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 )
 {
     setFixedHeight( AppLnk::smallIconSize() );
     setFixedWidth( AppLnk::smallIconSize() );
 
-    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&) ) );
 }
 
 int IrdaApplet::position()
@@ -94,13 +102,22 @@ void IrdaApplet::popup ( QString message, QString icon )
         if ( icon. isEmpty ( ))
                 m_popup-> insertItem ( message, 0 );
         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
+                             );
 
         QPoint p = mapToGlobal ( QPoint ( 0, 0 ));
         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 ( )
+                             )
+                    );
 
         QTimer::singleShot ( 2000, this, SLOT( popupTimeout()));
 }
@@ -115,7 +132,7 @@ bool IrdaApplet::checkIrdaStatus ( )
         struct ifreq ifr;
         strcpy ( ifr. ifr_name, "irda0" );
 
-        if ( ::ioctl ( m_sockfd, SIOCGIFFLAGS, &ifr ) < 0 )
+    if ( m_sockfd < 0 || ::ioctl ( m_sockfd, SIOCGIFFLAGS, &ifr ) < 0 )
                 return false;
 
         return ( ifr. ifr_flags & IFF_UP );
@@ -126,19 +143,18 @@ bool IrdaApplet::setIrdaStatus ( bool b )
         struct ifreq ifr;
         strcpy ( ifr. ifr_name, "irda0" );
 
-        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 {
                 setIrdaDiscoveryStatus ( 0 );
                 setIrdaReceiveStatus ( 0 );
                 ifr. ifr_flags &= ~IFF_UP;
         }
 
-        if ( ::ioctl ( m_sockfd, SIOCSIFFLAGS, &ifr ) < 0 )
+    if ( m_sockfd < 0 || ::ioctl ( m_sockfd, SIOCSIFFLAGS, &ifr ) < 0 )
                 return false;
 
         return true;
@@ -196,19 +212,27 @@ void IrdaApplet::showDiscovered ( )
                 QString discoveredDevice;
                 QString deviceAddr;
 
-                // 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 ( ));
 
                 QMap <QString, QString>::Iterator it;
 
                 for ( it = m_devices. begin ( ); it != m_devices. end ( ); ++it )
                         it. data ( ). prepend ( "+++" );
 
-                for ( QStringList::Iterator lit = list. begin ( ); lit != list. end ( ); ++lit ) {
+        for ( QStringList::Iterator lit = list. begin ( );
+              lit != list. end ( ); ++lit
+            )
+        {
                         const QString &line = *lit;
 
                         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 );
 
                                 // odebug << discoveredDevice + "(" + deviceAddr + ")" << oendl;
@@ -223,7 +247,8 @@ 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;
 
                         if ( it. data ( ). left ( 3 ) == "+++" ) {
                                 popup ( tr( "Lost:" ) + " " + it. data ( ). mid ( 3 ));
@@ -231,12 +256,13 @@ void IrdaApplet::showDiscovered ( )
 
                                 QMap <QString, QString>::Iterator tmp = it;
                                 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++;
                 }
                 // XXX if( qcopsend ) {
@@ -259,8 +285,11 @@ void IrdaApplet::mousePressEvent ( QMouseEvent * )
         if ( m_irda_active && !m_devices. isEmpty ( )) {
                 menu-> insertItem ( tr( "Discovered Device:" ), 9 );
 
-                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 );
+        }
 
                 menu-> insertSeparator ( );
         }
@@ -268,17 +297,24 @@ void IrdaApplet::mousePressEvent ( QMouseEvent * )
         menu-> insertItem ( m_irda_active ? tr( "Disable IrDA" ) : tr( "Enable IrDA" ), 0 );
 
         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
+                          );
                 
                 /* Only Receive if OBEX is installed */
                 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
+                              );
         }
 
         QPoint p = mapToGlobal ( QPoint ( 0, 0 ) );
         QSize s = menu-> sizeHint ( );
 
-        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 ( ));
 
         switch (  menu-> exec ( p )) {
                 case 0:
@@ -315,9 +351,12 @@ void IrdaApplet::timerEvent ( QTimerEvent * )
         if ( m_irda_discovery_active )
                 showDiscovered ( );
 
-        if (( m_irda_active != oldactive ) || ( m_irda_discovery_active != olddiscovery ) || receiveUpdate )
+    if (( m_irda_active != oldactive ) ||
+        ( m_irda_discovery_active != olddiscovery ) || receiveUpdate )
+    {
                 update ( );
 }
+}
 
 void IrdaApplet::paintEvent ( QPaintEvent * )
 {
@@ -331,12 +370,12 @@ void IrdaApplet::paintEvent ( QPaintEvent * )
         if ( m_receive_active )
                 p.drawPixmap( 0, 1, m_receiveActivePixmap );
 }
-/*
- * 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
  */
 void IrdaApplet::slotMessage( const QCString& str, const QByteArray&  ) {
     if ( str == "enableIrda()") {
@@ -368,3 +407,4 @@ 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
@@ -52,7 +52,6 @@ private:
 
         void showDiscovered();
 
-private:
         QPixmap m_irdaOnPixmap;
         QPixmap m_irdaOffPixmap;
         QPixmap m_irdaDiscoveryOnPixmap;
@@ -69,9 +68,8 @@ private:
 
         QMap <QString, QString> m_devices;
 
-                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__
+