[Uml-devel] kdesdk/umbrello/umbrello

Jonathan Riddell jr at jriddell.org
Mon Jan 10 08:37:25 UTC 2005


CVS commit by jriddell: 

Remove unnecessary check for null, deleteing a null is fine
Patch from Thomas Nagy


  M +5 -6      umlview.cpp   1.201


--- kdesdk/umbrello/umbrello/umlview.cpp  #1.200:1.201
@@ -206,10 +206,9 @@ UMLView::~UMLView() {
         blockSignals( true );
         removeAllWidgets();
-        if ( canvas() != NULL ) {
+
                 // Qt Doc for QCanvasView::~QCanvasView () states:
                 // "Destroys the canvas view. The associated canvas is not deleted."
                 // we should do it now
                 delete canvas();
-        }
 }
 






More information about the umbrello-devel mailing list