1 files changed, 24 insertions, 23 deletions

diff --git a/library/qpemenubar.cpp b/library/qpemenubar.cpp
index bb3ad8d..f0171ba 100644
--- a/library/qpemenubar.cpp
+++ b/library/qpemenubar.cpp
@@ -1,10 +1,10 @@
 /**********************************************************************
-** Copyright (C) 2001 Trolltech AS.  All rights reserved.
+** Copyright (C) 2000-2002 Trolltech AS.  All rights reserved.
 **
-** This file is part of Qtopia Environment.
+** This file is part of the Qtopia Environment.
 **
 ** This file may be distributed and/or modified under the terms of the
 ** GNU General Public License version 2 as published by the Free Software
 ** Foundation and appearing in the file LICENSE.GPL included in the
 ** packaging of this file.
 **
@@ -244,25 +244,29 @@ bool QPEMenuToolFocusManager::eventFilter( QObject *object, QEvent *event )
             if ( !w->isPopup() && !list.contains( GuardedWidget( w ) ) ) {
                 setActive( FALSE );
             }
         }
     } else if ( event->type() == QEvent::ChildInserted ) {
         QChildEvent *ce = (QChildEvent *)event;
-        if ( ce->child()->inherits( "QMenuBar" ) ) {
-            addWidget( (QWidget *)ce->child() );
-            ce->child()->installEventFilter( this );
-        } else if ( object->inherits( "QToolBar" ) && ce->child()->isWidgetType() ) {
-            addWidget( (QWidget *)ce->child() );
+        if ( ce->child()->isWidgetType() ) {
+            if ( ce->child()->inherits( "QMenuBar" ) ) {
+                addWidget( (QWidget *)ce->child() );
+                ce->child()->installEventFilter( this );
+            } else if ( object->inherits( "QToolBar" ) ) {
+                addWidget( (QWidget *)ce->child() );
+            }
         }
     } else if ( event->type() == QEvent::ChildRemoved ) {
         QChildEvent *ce = (QChildEvent *)event;
-        if ( ce->child()->inherits( "QMenuBar" ) ) {
-            removeWidget( (QWidget *)ce->child() );
-            ce->child()->removeEventFilter( this );
-        } else if ( object->inherits( "QToolBar" ) && ce->child()->isWidgetType() ) {
-            removeWidget( (QWidget *)ce->child() );
+        if ( ce->child()->isWidgetType() ) {
+            if ( ce->child()->inherits( "QMenuBar" ) ) {
+                removeWidget( (QWidget *)ce->child() );
+                ce->child()->removeEventFilter( this );
+            } else if ( object->inherits( "QToolBar" ) ) {
+                removeWidget( (QWidget *)ce->child() );
+            }
         }
     }
 
     return FALSE;
 }
 
@@ -276,13 +280,12 @@ void QPEMenuToolFocusManager::deactivate()
   \brief The QPEMenuBar class is obsolete.  Use QMenuBar instead.
 
   \obsolete
 
   This class is obsolete.  Use QMenuBar instead.
 
-  \sa QMenuBar
 */
 
 /*!
   Constructs a QPEMenuBar just as you would construct
   a QMenuBar, passing \a parent and \a name.
 */
@@ -303,22 +306,20 @@ QPEMenuBar::~QPEMenuBar()
 */
 void QPEMenuBar::keyPressEvent( QKeyEvent *e )
 {
     QMenuBar::keyPressEvent( e );
 }
 
-
-void QPEMenuBar::activateItem( int index )
-{
+/*!
+  \internal
+*/
+void QPEMenuBar::activateItem( int index ) {
     activateItemAt( index );
 }
-
-void QPEMenuBar::goodbye()
-{
+void QPEMenuBar::goodbye() {
     activateItemAt(-1);
-    for ( unsigned int i = 0; i < count(); i++ ) {
-        QMenuItem *mi = findItem( idAt(i) );
-        if ( mi->popup() ) {
+    for ( uint i = 0; i < count(); i++ ) {
+        QMenuItem* mi = findItem( idAt(i) );
+        if (mi->popup() )
             mi->popup()->hide();
-        }
     }
 }