0032429: Coding - Warnings during compilation on macosx arm64 with option BUILD_Inspe...
authorkgv <kgv@opencascade.com>
Tue, 6 Jul 2021 13:46:31 +0000 (16:46 +0300)
committerkgv <kgv@opencascade.com>
Tue, 6 Jul 2021 13:46:31 +0000 (16:46 +0300)
18 files changed:
tools/DFBrowser/DFBrowser_ItemBase.hxx
tools/DFBrowser/DFBrowser_SearchView.cxx
tools/DFBrowserPane/DFBrowserPane_AttributePaneModel.hxx
tools/DFBrowserPane/DFBrowserPane_TDataStdTreeNodeItem.hxx
tools/DFBrowserPane/DFBrowserPane_TNamingNamedShape.cxx
tools/DFBrowserPane/DFBrowserPane_TNamingNamedShape.hxx
tools/DFBrowserPane/DFBrowserPane_TNamingUsedShapes.hxx
tools/DFBrowserPane/DFBrowserPane_TableView.cxx
tools/ShapeView/ShapeView_ItemRoot.hxx
tools/ShapeView/ShapeView_ItemShape.hxx
tools/ShapeView/ShapeView_OpenFileDialog.cxx
tools/ShapeView/ShapeView_Window.cxx
tools/TInspector/TInspector_OpenFileDialog.cxx
tools/TInspector/TInspector_Preferences.cxx
tools/TreeModel/TreeModel_ItemStream.hxx
tools/TreeModel/TreeModel_ModelBase.cxx
tools/ViewControl/ViewControl_ColorSelector.cxx
tools/ViewControl/ViewControl_TableModel.hxx

index c87ca1d..44f9cae 100644 (file)
@@ -85,7 +85,7 @@ protected:
   virtual TreeModel_ItemBasePtr createChild (int theRow, int theColumn) Standard_OVERRIDE;
 
   //! Initializes the current item. It creates a backup of the specific item information
-  virtual void initItem() const {}
+  virtual void initItem() const Standard_OVERRIDE {}
 
 protected:
 
index 08ae8f6..5ca5182 100644 (file)
@@ -35,8 +35,7 @@
 #include <QWidget>
 #include <Standard_WarningsRestore.hxx>
 
-const int DEFAULT_COLUMN_WIDTH = 500;
-const int DEFAULT_ICON_SIZE = 40;
+static const int DEFAULT_ICON_SIZE = 40;
 
 // =======================================================================
 // function : Constructor
index 804372c..e9b9d6d 100644 (file)
@@ -104,7 +104,7 @@ public:
   //! Returns flags for the item: ItemIsEnabled | Qt::ItemIsSelectable
   //! \param theIndex a model index
   //! \return flags
-  Qt::ItemFlags flags (const QModelIndex& theIndex) const
+  virtual Qt::ItemFlags flags (const QModelIndex& theIndex) const Standard_OVERRIDE
   { return theIndex.isValid() ? Qt::ItemIsEnabled | Qt::ItemIsSelectable : Qt::NoItemFlags; }
 
 private:
index 9570aea..3c4e0a1 100644 (file)
@@ -97,7 +97,7 @@ protected:
     : TreeModel_ItemBase (theParent, theRow, theColumn), myIsCurrentItem (false) {}
 
   //! Initializes the current item. It creates a backup of the specific item information
-  void initItem() const;
+  virtual void initItem() const Standard_OVERRIDE;
 
   //! Returns number of children attributes, initializes item is necessary
   int getRowCount() const;
index c93f7bf..781c6d6 100644 (file)
 #include <QWidget>
 #include <Standard_WarningsRestore.hxx>
 
-const int COLUMN_EXPORT_WIDTH = 20;
-const int COLUMN_TYPE_WIDTH = 70;
-const int COLUMN_POINTER_WIDTH = 110;
-const int COLUMN_REFERENCE_WIDTH = 90;
-const int COLUMN_EVOLUTION_WIDTH = 90;
+static const int COLUMN_EXPORT_WIDTH = 20;
 
 // =======================================================================
 // function : Constructor
index ccffd18..2d934d2 100644 (file)
@@ -51,7 +51,7 @@ public:
 
   //! Initializes the content of the pane by the parameter attribute
   //! \param theAttribute an OCAF attribute
-  Standard_EXPORT void Init (const Handle(TDF_Attribute)& theAttribute);
+  Standard_EXPORT virtual void Init (const Handle(TDF_Attribute)& theAttribute) Standard_OVERRIDE;
 
   //! Returns information for the given attribute
   //! \param theAttribute a current attribute
index d7b0466..095b955 100644 (file)
@@ -58,7 +58,7 @@ public:
 protected:
   //! Defines widths of table columns
   //! \return container of widths
-  Standard_EXPORT virtual QMap<int, int> getTableColumnWidths() const;
+  Standard_EXPORT virtual QMap<int, int> getTableColumnWidths() const Standard_OVERRIDE;
 
 };
 
index 5bbf228..514b46d 100644 (file)
@@ -27,8 +27,6 @@
 #include <QWidget>
 #include <Standard_WarningsRestore.hxx>
 
-static const int DEFAULT_ROW_HEIGHT = 30;
-
 // =======================================================================
 // function : Constructor
 // purpose :
index c05495d..af31b26 100644 (file)
@@ -55,7 +55,7 @@ protected:
   //! Returns data value for the role.
   //! \param theItemRole a value role
   //! \return the value
-  virtual QVariant initValue(const int theItemRole) const;
+  virtual QVariant initValue(const int theItemRole) const Standard_OVERRIDE;
 
   //! \return number of children.
   virtual int initRowCount() const Standard_OVERRIDE { return myShapes.Size(); }
index 6615691..888a0e6 100644 (file)
@@ -82,7 +82,7 @@ public:
   //! Returns data value for the role.
   //! \param theRole a value role
   //! \return the value
-  Standard_EXPORT virtual QVariant initValue(const int theRole) const;
+  Standard_EXPORT virtual QVariant initValue(const int theRole) const Standard_OVERRIDE;
 
   //! \return number of children.
   Standard_EXPORT virtual int initRowCount() const Standard_OVERRIDE;
index 3ea9b1f..b3588d4 100644 (file)
 #include <QToolButton>
 #include <Standard_WarningsRestore.hxx>
 
-const int FONT_POINT_SIZE = 18;
-const int ICON_SIZE = 40;
+static const int ICON_SIZE = 40;
 
-const int OPEN_DIALOG_WIDTH = 550;
-const int OPEN_DIALOG_HEIGHT = 200;
+static const int OPEN_DIALOG_WIDTH = 550;
+static const int OPEN_DIALOG_HEIGHT = 200;
 
-const int MARGIN_DIALOG = 4;
-const int SPACING_DIALOG = 2;
+static const int MARGIN_DIALOG = 4;
+static const int SPACING_DIALOG = 2;
 
 // =======================================================================
 // function : StartButton
index d6f54d7..d53ad47 100644 (file)
 #include <QVBoxLayout>
 #include <Standard_WarningsRestore.hxx>
 
-const int DEFAULT_TEXT_VIEW_WIDTH = 800;
-const int DEFAULT_TEXT_VIEW_HEIGHT = 700;
-const int DEFAULT_TEXT_VIEW_POSITION_X = 430;
-const int DEFAULT_TEXT_VIEW_POSITION_Y = 30;
-const int DEFAULT_TEXT_VIEW_DELTA = 100;
-
-const int DEFAULT_SHAPE_VIEW_WIDTH = 900;
-const int DEFAULT_SHAPE_VIEW_HEIGHT = 450;
-const int DEFAULT_SHAPE_VIEW_POSITION_X = 60;
-const int DEFAULT_SHAPE_VIEW_POSITION_Y = 60;
-
-const int SHAPEVIEW_DEFAULT_TREE_VIEW_WIDTH = 600;
-const int SHAPEVIEW_DEFAULT_TREE_VIEW_HEIGHT = 500;
-
-const int SHAPEVIEW_DEFAULT_VIEW_WIDTH = 300;
-const int SHAPEVIEW_DEFAULT_VIEW_HEIGHT = 1000;
+static const int DEFAULT_SHAPE_VIEW_WIDTH = 900;
+static const int DEFAULT_SHAPE_VIEW_HEIGHT = 450;
+static const int DEFAULT_SHAPE_VIEW_POSITION_X = 60;
+static const int DEFAULT_SHAPE_VIEW_POSITION_Y = 60;
+
+static const int SHAPEVIEW_DEFAULT_TREE_VIEW_WIDTH = 600;
+static const int SHAPEVIEW_DEFAULT_TREE_VIEW_HEIGHT = 500;
+
+static const int SHAPEVIEW_DEFAULT_VIEW_WIDTH = 300;
+static const int SHAPEVIEW_DEFAULT_VIEW_HEIGHT = 1000;
 
 // =======================================================================
 // function : Constructor
index 622818a..04473fa 100644 (file)
 #include <QWidget>
 #include <Standard_WarningsRestore.hxx>
 
-const int FONT_POINT_SIZE = 18;
-const int ICON_SIZE = 40;
+static const int ICON_SIZE = 40;
 
-const int OPEN_DIALOG_WIDTH = 550;
-const int OPEN_DIALOG_HEIGHT = 200;
+static const int OPEN_DIALOG_WIDTH = 550;
+static const int OPEN_DIALOG_HEIGHT = 200;
 
-const int MARGIN_DIALOG = 4;
-const int SPACING_DIALOG = 2;
+static const int MARGIN_DIALOG = 4;
+static const int SPACING_DIALOG = 2;
 
-const int RECENT_FILES_CACHE_SIZE = 10;
+static const int RECENT_FILES_CACHE_SIZE = 10;
 
 TInspector_Communicator* MyCommunicator;
 
index c335fab..00d6593 100644 (file)
@@ -79,7 +79,9 @@ void TInspector_Preferences::StorePreferences()
   QTextStream aTextStream (&aFile);
   QStringList aDocumentStr = aDomDocument.toString().split ("\n");
   for (QStringList::ConstIterator aContentIt = aDocumentStr.begin(); aContentIt != aDocumentStr.end(); ++aContentIt)
-    aTextStream << *aContentIt << endl;
+  {
+    aTextStream << *aContentIt << '\n';
+  }
   aFile.close();
 }
 
index 800e014..65fa16c 100644 (file)
@@ -69,7 +69,7 @@ public:
 protected:
   //! Returns stream value of the item to fulfill property panel.
   //! \return stream value or dummy
-  Standard_EXPORT virtual void initStream (Standard_OStream& theOStream) const;
+  Standard_EXPORT virtual void initStream (Standard_OStream& theOStream) const Standard_OVERRIDE;
 
   //! Initializes the current item. It creates a backup of the specific item information
   //! Does nothing as context has been already set into item
index 5dd4055..30b3202 100644 (file)
@@ -162,7 +162,9 @@ QModelIndex TreeModel_ModelBase::parent (const QModelIndex& theIndex) const
 Qt::ItemFlags TreeModel_ModelBase::flags (const QModelIndex& theIndex) const
 {
   if (!theIndex.isValid())
-    return 0;
+  {
+    return Qt::ItemFlags();
+  }
   return Qt::ItemIsEnabled | Qt::ItemIsSelectable;
 }
 
index 67fff81..2e480bc 100644 (file)
@@ -105,7 +105,7 @@ public:
   //! \param theColumn a model index column
   //! \param theRole a view role
   //! \return true if the value is changed
-  virtual bool SetData (const int theRow, const int theColumn, const QVariant& theValue, int)
+  virtual bool SetData (const int theRow, const int theColumn, const QVariant& theValue, int) Standard_OVERRIDE
   {
     if (theColumn != 1 || theRow < 2 || theRow > 5)
       return false;
@@ -138,7 +138,7 @@ public:
 
   //! Returns editable flag for color RGB and alpha rows
   //! \return flags
-  Qt::ItemFlags Flags (const QModelIndex& theIndex) const
+  virtual Qt::ItemFlags Flags (const QModelIndex& theIndex) const Standard_OVERRIDE
   {
     Qt::ItemFlags aFlags = ViewControl_TableModelValues::Flags (theIndex);
 
index 0255187..aeadc44 100644 (file)
@@ -90,7 +90,7 @@ public:
   //! Returns flags for the item: ItemIsEnabled | Qt::ItemIsSelectable
   //! \param theIndex a model index
   //! \return flags
-  Qt::ItemFlags flags (const QModelIndex& theIndex) const
+  virtual Qt::ItemFlags flags (const QModelIndex& theIndex) const Standard_OVERRIDE
   { return myModelValues ? myModelValues->Flags (theIndex) : Qt::NoItemFlags; }
 
 private: