Code analysis & compiller warnings removed:

* XGUI_ViewWindow, XGUI_ViewPort uninitialized variables in constructor
* PyConsole_EnhEditor signed/unsigned comparsion
* XGUI_Module virtual destructor added

diff --git a/src/PyConsole/PyConsole_EnhEditor.cxx b/src/PyConsole/PyConsole_EnhEditor.cxx
index 2dd8207abcc75e98991bdb7150544548f9fa85a5..d5c47dff3728c94632ec788f0635beb25c3191a2 100644 (file)
--- a/src/PyConsole/PyConsole_EnhEditor.cxx
+++ b/src/PyConsole/PyConsole_EnhEditor.cxx
@@ -342,7 +342,7 @@ void PyConsole_EnhEditor::extractCommon(const std::vector<QString> & matches, QS
       if (charIdx >= matches[0].size())
         return;
       QChar ch = matches[0][charIdx];
-      for (int j = 1; j < matches.size(); j++)
+      for (size_t j = 1; j < matches.size(); j++)
         if (charIdx >= matches[j].size() || matches[j][charIdx] != ch)
           return;
       result += ch;

diff --git a/src/XGUI/XGUI_Module.h b/src/XGUI/XGUI_Module.h
index 1860462e26c29212406d7378a2d84d617bf56cf0..374e057eac601e58a43c6fed73001c402d79aa68 100644 (file)
--- a/src/XGUI/XGUI_Module.h
+++ b/src/XGUI/XGUI_Module.h
@@ -10,6 +10,8 @@ class XGUI_Module
 public:\r
   virtual void createFeatures() = 0;\r
   virtual void featureCreated(XGUI_Command*) = 0;\r
+\r
+  virtual ~XGUI_Module() {};\r
 };\r
 \r
 //! This function must return a new module instance.\r

diff --git a/src/XGUI/XGUI_ViewPort.cpp b/src/XGUI/XGUI_ViewPort.cpp
index 4398cabaa5d6c97c5f69ca0aacced541c6e19678..21552a55dc41197dec1bef8d9072edff1a050cf0 100644 (file)
--- a/src/XGUI/XGUI_ViewPort.cpp
+++ b/src/XGUI/XGUI_ViewPort.cpp
@@ -249,10 +249,15 @@ void OpenGLUtils_FrameBuffer::unbind()
 //************************************************************************
 //************************************************************************
 //************************************************************************
-XGUI_ViewPort::XGUI_ViewPort(XGUI_ViewWindow* theParent, const Handle(V3d_Viewer)& theViewer,
+XGUI_ViewPort::XGUI_ViewPort(XGUI_ViewWindow* theParent,
+                             const Handle(V3d_Viewer)& theViewer,
                              V3d_TypeOfView theType)
-    : QWidget(theParent), myPaintersRedrawing(false), myScale(1.0), myIsAdvancedZoomingEnabled(
-        false)
+    : QWidget(theParent),
+      myPaintersRedrawing(false),
+      myScale(1.0),
+      myIsAdvancedZoomingEnabled(false),
+      myBgImgHeight(0),
+      myBgImgWidth(0)
 {
   setMouseTracking(true);
   setBackgroundRole(QPalette::NoRole);

diff --git a/src/XGUI/XGUI_ViewWindow.cpp b/src/XGUI/XGUI_ViewWindow.cpp
index f68ca95c32e9043844a9c71992cc57b15e4859bc..fd8f1dc9e19149df50af3e01ac84694438a9ca2b 100644 (file)
--- a/src/XGUI/XGUI_ViewWindow.cpp
+++ b/src/XGUI/XGUI_ViewWindow.cpp
@@ -133,7 +133,10 @@ XGUI_ViewWindow::XGUI_ViewWindow(XGUI_Viewer* theViewer, V3d_TypeOfView theType)
     myCurrPointType(XGUI::GRAVITY), 
     myPrevPointType(XGUI::GRAVITY), 
     myRotationPointSelection(false),
-    myClosable(true)
+    myClosable(true),
+    myStartX(0), myStartY(0), myCurrX(0), myCurrY(0), myCurScale(0.0), myCurSketch(0),
+    myDrawRect(false), myEnableDrawMode(false), myCursorIsHand(false), myEventStarted(false),
+    myLastState(NormalState), myOperation(NOTHING)
 {
   mySelectedPoint = gp_Pnt(0., 0., 0.);
   setFrameStyle(QFrame::Raised);