[Uml-devel] KDE/kdesdk/umbrello/umbrello

Oliver Kellogg okellogg at users.sourceforge.net
Tue Dec 4 18:26:32 UTC 2007


SVN commit 744856 by okellogg:

dashboard recommends curly braces

 M  +2 -1      codegenerators/cppwriter.cpp  
 M  +2 -1      codegenobjectwithtextblocks.cpp  
 M  +2 -1      dialogs/codeeditor.cpp  
 M  +2 -1      linepath.cpp  


--- trunk/KDE/kdesdk/umbrello/umbrello/codegenerators/cppwriter.cpp #744855:744856
@@ -1158,11 +1158,12 @@
 
         // as header doc for this method indicates, we need to be a bit sophisticated on
         // how to declare some associations.
-        if( current )
+        if (current) {
             if( !isFirstClass && !a->getRoleName(Uml::A).isEmpty() && !a->getRoleName(Uml::B).isEmpty())
                 stream << "class " << current->getName() << ";" << m_endl; // special case: use forward declaration
             else
                 stream << "#include \"" << current->getName() << ".h\"" << m_endl; // just the include statement
+        }
     }
 }
 
--- trunk/KDE/kdesdk/umbrello/umbrello/codegenobjectwithtextblocks.cpp #744855:744856
@@ -253,11 +253,12 @@
     }
 
     codeComment->setText(text);
-    if(createdCodeComment)
+    if (createdCodeComment) {
         if(!text.isEmpty())
             codeComment->setWriteOutText(true); // set to visible, if we created
         else
             codeComment->setWriteOutText(false); // set to not visible, if we created
+    }
 
     codeComment->setIndentationLevel(indentationLevel);
 
--- trunk/KDE/kdesdk/umbrello/umbrello/dialogs/codeeditor.cpp #744855:744856
@@ -1068,7 +1068,7 @@
         QList<ParaInfo*> list = info->m_paraList;
         foreach (ParaInfo * item, list)
         {
-            if((item->start+pstart) <= para && (item->start+pstart+item->size) >= para)
+            if (item->start+pstart <= para && item->start+pstart+item->size >= para) {
                 if(item->isEditable && hasEditableRange)
                 {
                     if ( relativeLine >= editStart && relativeLine <= (item->size + editEnd) )
@@ -1077,6 +1077,7 @@
                         return false;
                 } else
                     return item->isEditable;
+            }
         }
     }
     return false;
--- trunk/KDE/kdesdk/umbrello/umbrello/linepath.cpp #744855:744856
@@ -381,10 +381,11 @@
         line -> setPen( QPen( color, linewidth ) );
     }
 
-    if( getAssocType() == Uml::at_Aggregation )
+    if (getAssocType() == Uml::at_Aggregation) {
         if (m_pClearPoly) m_pClearPoly -> setBrush( QBrush( Qt::white ) );
         else if( getAssocType() == Uml::at_Composition )
             if (m_pClearPoly) m_pClearPoly -> setBrush( QBrush( color ) );
+    }
 
     if( m_pCircle ) {
         linewidth = m_pCircle->pen().width();




More information about the umbrello-devel mailing list