Fix build warnings

Thanks to Bruce Sass for the patch!

11 files changed, 37 insertions, 36 deletions

diff --git a/tools/designer/designer/command.cpp b/tools/designer/designer/command.cpp
index 74a6b19..8c6b791 100644
--- a/tools/designer/designer/command.cpp
+++ b/tools/designer/designer/command.cpp
@@ -511,8 +511,8 @@ bool SetPropertyCommand::canMerge( Command *c )
 	return FALSE;
     }
     QVariant::Type t = QVariant::nameToType( p->type() );
-    return ( cmd->propName == propName &&
-	     t == QVariant::String || t == QVariant::CString || t == QVariant::Int || t == QVariant::UInt );
+    return ( ( cmd->propName == propName &&
+	     t == QVariant::String ) || t == QVariant::CString || t == QVariant::Int || t == QVariant::UInt );
 }
 
 void SetPropertyCommand::merge( Command *c )
diff --git a/tools/designer/designer/formwindow.cpp b/tools/designer/designer/formwindow.cpp
index ec620e7..9daeaae 100644
--- a/tools/designer/designer/formwindow.cpp
+++ b/tools/designer/designer/formwindow.cpp
@@ -1632,7 +1632,7 @@ QWidget *FormWindow::designerWidget( QObject *o ) const
     if ( !o || !o->isWidgetType() )
 	return 0;
     QWidget *w = (QWidget*)o;
-    while ( w && !isMainContainer( w ) && !insertedWidgets[ (void*)w ] || isCentralWidget( w ) )
+    while ( ( w && !isMainContainer( w ) && !insertedWidgets[ (void*)w ] ) || isCentralWidget( w ) )
 	w = (QWidget*)w->parent();
     return w;
 }
diff --git a/tools/designer/designer/layout.cpp b/tools/designer/designer/layout.cpp
index 2701023..3422311 100644
--- a/tools/designer/designer/layout.cpp
+++ b/tools/designer/designer/layout.cpp
@@ -964,11 +964,12 @@ int Spacer::alignment() const
 QSize Spacer::minimumSize() const
 {
     QSize s = QSize( 20,20 );
-    if ( sizeType() == Expanding )
+    if ( sizeType() == Expanding ) {
 	if ( orient == Vertical )
 	    s.rheight() = 0;
 	else
 	    s.rwidth() = 0;
+	}
     return s;
 }
 
diff --git a/tools/designer/designer/mainwindow.cpp b/tools/designer/designer/mainwindow.cpp
index 0474b2f..d4f8af4 100644
--- a/tools/designer/designer/mainwindow.cpp
+++ b/tools/designer/designer/mainwindow.cpp
@@ -989,12 +989,12 @@ bool MainWindow::eventFilter( QObject *o, QEvent *e )
 	     ( ::qt_cast<MenuBarEditor*>(o) ||
 	       ::qt_cast<PopupMenuEditor*>(o) ||
 	       ::qt_cast<QDesignerToolBar*>(o) ||
-	       ( ::qt_cast<QComboBox*>(o) ||
+	       ( ( ::qt_cast<QComboBox*>(o) ||
 		 ::qt_cast<QToolButton*>(o) ||
 		 ::qt_cast<QDesignerToolBarSeparator*>(o) ) &&
 	       o->parent()
 	       && ( ::qt_cast<QDesignerToolBar*>(o->parent())
-		   || ::qt_cast<QDesignerWidgetStack*>(o->parent())) ) ) {
+		   || ::qt_cast<QDesignerWidgetStack*>(o->parent())) ) ) ) {
 	    QWidget *w = (QWidget*)o;
 	    if ( ::qt_cast<QToolButton*>(w) ||
 		 ::qt_cast<QComboBox*>(w) ||
@@ -1570,10 +1570,10 @@ void MainWindow::setupRMBProperties( QValueList<uint> &ids, QMap<QString, int> &
     if ( pixmap && qstrcmp( pixmap->type(), "QPixmap") != 0 )
 	pixmap = 0;
 
-    if ( text && text->designable(w) ||
-	 title && title->designable(w) ||
-	 pagetitle && pagetitle->designable(w) ||
-	 pixmap && pixmap->designable(w) ) {
+    if ( ( text && text->designable(w) ) ||
+	 ( title && title->designable(w) ) ||
+	 ( pagetitle && pagetitle->designable(w) ) ||
+	 ( pixmap && pixmap->designable(w) ) ) {
 	int id = 0;
 	if ( ids.isEmpty() )
 	    ids << rmbWidgets->insertSeparator(0);
@@ -3337,7 +3337,7 @@ void MainWindow::showSourceLine( QObject *o, int line, LineMode lm )
 	    if ( fw->project() != currentProject )
 		continue;
 	    if ( qstrcmp( fw->name(), o->name() ) == 0 ||
-		 fw->isFake() && currentProject->objectForFakeForm( fw ) == o ) {
+		 ( fw->isFake() && currentProject->objectForFakeForm( fw ) == o ) ) {
 		if ( se ) {
 		    switch ( lm ) {
 		    case Error:
diff --git a/tools/designer/designer/mainwindowactions.cpp b/tools/designer/designer/mainwindowactions.cpp
index d8d99aa..94a115f 100644
--- a/tools/designer/designer/mainwindowactions.cpp
+++ b/tools/designer/designer/mainwindowactions.cpp
@@ -1248,7 +1248,7 @@ void MainWindow::fileOpen( const QString &filter, const QString &extension, cons
 			setCurrentProject( eProject );
 		    openFormWindow( filename );
 		    addRecentlyOpened( filename, recentlyFiles );
-		} else if ( !extension.isEmpty() && extension.find( ";" + fi.extension( FALSE ) ) != -1 ||
+		} else if ( ( !extension.isEmpty() && extension.find( ";" + fi.extension( FALSE ) ) != -1 ) ||
 			    additionalSources.find( fi.extension( FALSE ) ) != additionalSources.end() ) {
 		    SourceFile *sf = project->findSourceFile( project->makeRelative( filename ) );
 		    if ( !sf )
@@ -1687,14 +1687,14 @@ void MainWindow::editBreakLayout()
     if ( formWindow()->currentWidget() )
 	w = formWindow()->currentWidget();
     if ( WidgetFactory::layoutType( w ) != WidgetFactory::NoLayout ||
-	 w->parentWidget() && WidgetFactory::layoutType( w->parentWidget() ) != WidgetFactory::NoLayout ) {
+	 ( w->parentWidget() && WidgetFactory::layoutType( w->parentWidget() ) != WidgetFactory::NoLayout ) ) {
 	formWindow()->breakLayout( w );
 	return;
     } else {
 	QWidgetList widgets = formWindow()->selectedWidgets();
 	for ( w = widgets.first(); w; w = widgets.next() ) {
 	    if ( WidgetFactory::layoutType( w ) != WidgetFactory::NoLayout ||
-		 w->parentWidget() && WidgetFactory::layoutType( w->parentWidget() ) != WidgetFactory::NoLayout )
+		 ( w->parentWidget() && WidgetFactory::layoutType( w->parentWidget() ) != WidgetFactory::NoLayout ) )
 		break;
 	}
 	if ( w ) {
@@ -1705,7 +1705,7 @@ void MainWindow::editBreakLayout()
 
     w = formWindow()->mainContainer();
     if ( WidgetFactory::layoutType( w ) != WidgetFactory::NoLayout ||
-	 w->parentWidget() && WidgetFactory::layoutType( w->parentWidget() ) != WidgetFactory::NoLayout )
+	 ( w->parentWidget() && WidgetFactory::layoutType( w->parentWidget() ) != WidgetFactory::NoLayout ) )
 	formWindow()->breakLayout( w );
 }
 
diff --git a/tools/designer/designer/metadatabase.cpp b/tools/designer/designer/metadatabase.cpp
index cd36cc0..48422b1 100644
--- a/tools/designer/designer/metadatabase.cpp
+++ b/tools/designer/designer/metadatabase.cpp
@@ -355,7 +355,7 @@ void MetaDataBase::setMargin( QObject *o, int margin )
     QWidget *widget = (QWidget*)o;
     if ( widget && !::qt_cast<QLayoutWidget*>(widget) &&
 	( WidgetDatabase::isContainer( WidgetDatabase::idFromClassName( WidgetFactory::classNameOf( widget ) ) ) ||
-	widget && widget->parentWidget() && ::qt_cast<FormWindow*>(widget->parentWidget()) ) )
+	  ( widget && widget->parentWidget() && ::qt_cast<FormWindow*>(widget->parentWidget()) ) ) )
 	isInnerLayout = FALSE;
 
 
diff --git a/tools/designer/designer/project.cpp b/tools/designer/designer/project.cpp
index 3eae1c4..2c0912d 100644
--- a/tools/designer/designer/project.cpp
+++ b/tools/designer/designer/project.cpp
@@ -628,7 +628,7 @@ static void remove_multiline_contents( QString &contents, const QString &s, int
 	    if ( contents[ i ] == '\n' && !lastWasBackspash )
 		break;
 	    lastWasBackspash = ( contents[ i ] == '\\' ||
-				 lastWasBackspash && ( contents[ i ] == ' ' || contents[ i ] == '\t' ) );
+				 ( lastWasBackspash && ( contents[ i ] == ' ' || contents[ i ] == '\t' ) ) );
 	}
 	contents.remove( start, i - start + 1 );
     }
diff --git a/tools/designer/designer/propertyeditor.cpp b/tools/designer/designer/propertyeditor.cpp
index 708f779..98d299c 100644
--- a/tools/designer/designer/propertyeditor.cpp
+++ b/tools/designer/designer/propertyeditor.cpp
@@ -553,9 +553,9 @@ QString PropertyItem::currentItemFromObject() const
 void PropertyItem::setFocus( QWidget *w )
 {
     if ( !qApp->focusWidget() ||
-	 listview->propertyEditor()->formWindow() &&
+	 ( listview->propertyEditor()->formWindow() &&
 	 ( !MainWindow::self->isAFormWindowChild( qApp->focusWidget() ) &&
-	   !qApp->focusWidget()->inherits( "Editor" ) ) )
+	   !qApp->focusWidget()->inherits( "Editor" ) ) ) )
 	w->setFocus();
 }
 
@@ -3151,7 +3151,7 @@ void PropertyList::setupProperties()
 	bool isPropertyObject = w->isA( "PropertyObject" );
 
 	if ( ( p->designable(w) ||
-	       isPropertyObject && p->designable( ( (PropertyObject*)w )->widgetList().first() ) ) &&
+	       ( isPropertyObject && p->designable( ( (PropertyObject*)w )->widgetList().first() ) ) ) &&
 	     ( !isPropertyObject || qstrcmp( p->name(), "name" ) != 0 ) ) {
 	    if ( p->isSetType() ) {
 		if ( QString( p->name() ) == "alignment" ) {
@@ -3733,9 +3733,9 @@ void PropertyList::setPropertyValue( PropertyItem *i )
 
 void PropertyList::setCurrentProperty( const QString &n )
 {
-    if ( currentItem() && currentItem()->text( 0 ) == n ||
-	 currentItem() && ( (PropertyItem*)currentItem() )->propertyParent() &&
-	 ( (PropertyItem*)currentItem() )->propertyParent()->text( 0 ) == n )
+    if ( ( currentItem() && currentItem()->text( 0 ) == n ) ||
+	 ( currentItem() && ( (PropertyItem*)currentItem() )->propertyParent() &&
+	 ( (PropertyItem*)currentItem() )->propertyParent()->text( 0 ) == n ) )
 	return;
 
     QListViewItemIterator it( this );
diff --git a/tools/designer/designer/qcompletionedit.cpp b/tools/designer/designer/qcompletionedit.cpp
index a293135..e9e65c0 100644
--- a/tools/designer/designer/qcompletionedit.cpp
+++ b/tools/designer/designer/qcompletionedit.cpp
@@ -115,8 +115,8 @@ void QCompletionEdit::updateListBox()
     if ( compList.isEmpty() )
 	return;
     for ( QStringList::Iterator it = compList.begin(); it != compList.end(); ++it ) {
-	if ( caseSensitive && (*it).left( text().length() ) == text() ||
-	     !caseSensitive && (*it).left( text().length() ).lower() == text().lower() )
+	if ( ( caseSensitive && (*it).left( text().length() ) == text() ) ||
+	     ( !caseSensitive && (*it).left( text().length() ).lower() == text().lower() ) )
 	    listbox->insertItem( *it );
     }
 }
diff --git a/tools/designer/designer/resource.cpp b/tools/designer/designer/resource.cpp
index 257f4e6..7fed3ca 100644
--- a/tools/designer/designer/resource.cpp
+++ b/tools/designer/designer/resource.cpp
@@ -1220,8 +1220,8 @@ void Resource::saveItems( QObject *obj, QTextStream &ts, int indent )
         bool isDataTable = false;
 #  endif
 	for ( i = 0; i < table->horizontalHeader()->count(); ++i ) {
-	    if ( !table->horizontalHeader()->label( i ).isNull() &&
-		 table->horizontalHeader()->label( i ).toInt() != i + 1 ||
+	    if ( ( !table->horizontalHeader()->label( i ).isNull() &&
+		 table->horizontalHeader()->label( i ).toInt() != i + 1 ) ||
 		 table->horizontalHeader()->iconSet( i ) ||
 		  isDataTable ) {
 		ts << makeIndent( indent ) << "<column>" << endl;
@@ -1245,8 +1245,8 @@ void Resource::saveItems( QObject *obj, QTextStream &ts, int indent )
 	    }
 	}
 	for ( i = 0; i < table->verticalHeader()->count(); ++i ) {
-	    if ( !table->verticalHeader()->label( i ).isNull() &&
-		 table->verticalHeader()->label( i ).toInt() != i + 1 ||
+	    if ( ( !table->verticalHeader()->label( i ).isNull() &&
+		 table->verticalHeader()->label( i ).toInt() != i + 1 ) ||
 		 table->verticalHeader()->iconSet( i ) ) {
 		ts << makeIndent( indent ) << "<row>" << endl;
 		indent++;
@@ -1547,7 +1547,7 @@ void Resource::saveProperty( QObject *w, const QString &name, const QVariant &va
     uint unum;
     double dob;
     QString comment;
-    if ( w && formwindow->widgets()->find( (QWidget*)w ) || formwindow->actionList().find( (QAction*)w ) )
+    if ( ( w && formwindow->widgets()->find( (QWidget*)w ) ) || formwindow->actionList().find( (QAction*)w ) )
 	comment = MetaDataBase::propertyComment( w, name );
     switch ( t ) {
     case QVariant::String:
diff --git a/tools/designer/designer/widgetfactory.cpp b/tools/designer/designer/widgetfactory.cpp
index e168b4f..3757f5c 100644
--- a/tools/designer/designer/widgetfactory.cpp
+++ b/tools/designer/designer/widgetfactory.cpp
@@ -568,7 +568,7 @@ QLayout *WidgetFactory::createLayout( QWidget *widget, QLayout *layout, LayoutTy
 
     if ( ::qt_cast<QLayoutWidget*>(widget) &&
 	 ( WidgetDatabase::isContainer( WidgetDatabase::idFromClassName( WidgetFactory::classNameOf( widget ) ) ) ||
-	   widget && ::qt_cast<FormWindow*>(widget->parentWidget()) ) )
+	   ( widget && ::qt_cast<FormWindow*>(widget->parentWidget()) ) ) )
 	margin = MainWindow::self->currentLayoutDefaultMargin();
 
     if ( !layout && ::qt_cast<QTabWidget*>(widget) )
@@ -883,7 +883,7 @@ QWidget *WidgetFactory::createWidget( const QString &className, QWidget *parent,
 	MetaDataBase::setPropertyChanged( s, "sizeType", TRUE );
 	if ( !r )
 	    return s;
-	if ( !r->isValid() || r->width() < 2 && r->height() < 2 )
+	if ( !r->isValid() || ( r->width() < 2 && r->height() < 2 ) )
 	    s->setOrientation( orient );
 	else if ( r->width() < r->height() )
 	    s->setOrientation( Qt::Vertical );
@@ -904,7 +904,7 @@ QWidget *WidgetFactory::createWidget( const QString &className, QWidget *parent,
 	QSlider *s = new QSlider( parent, name );
 	if ( !r )
 	    return s;
-	if ( !r->isValid() || r->width() < 2 && r->height() < 2 )
+	if ( !r->isValid() || ( r->width() < 2 && r->height() < 2 ) )
 	    s->setOrientation( orient );
 	else if ( r->width() > r->height() )
 	    s->setOrientation( Qt::Horizontal );
@@ -915,7 +915,7 @@ QWidget *WidgetFactory::createWidget( const QString &className, QWidget *parent,
 	QScrollBar *s = new QScrollBar( parent, name );
 	if ( !r )
 	    return s;
-	if ( !r->isValid() || r->width() < 2 && r->height() < 2 )
+	if ( !r->isValid() || ( r->width() < 2 && r->height() < 2 ) )
 	    s->setOrientation( orient );
 	else if ( r->width() > r->height() )
 	    s->setOrientation( Qt::Horizontal );
@@ -936,7 +936,7 @@ QWidget *WidgetFactory::createWidget( const QString &className, QWidget *parent,
 	MetaDataBase::setPropertyChanged( l, "frameShape", TRUE );
 	if ( !r )
 	    return l;
-	if ( !r->isValid() || r->width() < 2 && r->height() < 2 )
+	if ( !r->isValid() || ( r->width() < 2 && r->height() < 2 ) )
 	    l->setOrientation( orient );
 	else if ( r->width() < r->height() )
 	    l->setOrientation( Qt::Vertical );
@@ -1143,7 +1143,7 @@ QWidget* WidgetFactory::widgetOfContainer( QWidget *w )
     while ( w ) {
 	int id = WidgetDatabase::idFromClassName( WidgetFactory::classNameOf( w ) );
 	if ( WidgetDatabase::isContainer( id ) ||
-	     w && ::qt_cast<FormWindow*>(w->parentWidget()) )
+	     ( w && ::qt_cast<FormWindow*>(w->parentWidget()) ) )
 	    return w;
 	w = w->parentWidget();
     }