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

[Oliver Kellogg]

SVN commit 691088 by okellogg:

slotMenuSelection(case ListPopupMenu::mt_EnumLiteral): No need to call parent.

 M  +6 -3      enumwidget.cpp  


--- trunk/KDE/kdesdk/umbrello/umbrello/enumwidget.cpp #691087:691088
@@ -168,13 +168,16 @@
 }
 
 void EnumWidget::slotMenuSelection(int sel) {
-    switch(sel) {
-    case ListPopupMenu::mt_EnumLiteral:
+    if (sel == ListPopupMenu::mt_EnumLiteral) {
         if (Object_Factory::createChildObject(static_cast<UMLClassifier*>(m_pObject),
                                               Uml::ot_EnumLiteral) )  {
+            /* I don't know why it works without these calls:
+            updateComponentSize();
+            update();
+             */
             UMLApp::app()->getDocument()->setModified();
         }
-        break;
+        return;
     }
     UMLWidget::slotMenuSelection(sel);
 }