Qtopia1-6 merge

still to test
bic changes to be resolved
more changes to be made?

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,12 +1,12 @@
 /**********************************************************************
-** 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.
 **
 ** This file is provided AS IS with NO WARRANTY OF ANY KIND, INCLUDING THE
 ** WARRANTY OF DESIGN, MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE.
@@ -242,29 +242,33 @@ bool QPEMenuToolFocusManager::eventFilter( QObject *object, QEvent *event )
             // Deaticvate if a menu/tool has been hidden
             QWidget *w = (QWidget *)object;
             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;
 }
 
 void QPEMenuToolFocusManager::deactivate()
 {
@@ -274,17 +278,16 @@ void QPEMenuToolFocusManager::deactivate()
 /*!
   \class QPEMenuBar qpemenubar.h
   \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.
 */
 QPEMenuBar::QPEMenuBar( QWidget *parent, const char *name )
     : QMenuBar( parent, name )
@@ -301,24 +304,22 @@ QPEMenuBar::~QPEMenuBar()
 /*!
   \internal
 */
 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();
-        }
     }
 }