+Standard_Boolean Convert_Tools::CreateBoxShape (const gp_Pnt& thePntMin,
+                                                const gp_Pnt& thePntMax,
+                                                TopoDS_Shape& theShape)
...
+  if (((int)aThinOnX + (int)aThinOnY + (int)aThinOnZ) == 3) // thin box in all directions is a point
+  {
+    aBuilder.Add (aCompound, BRepBuilderAPI_MakeVertex (thePntMin));
...
+  else
+  {
+    BRepPrimAPI_MakeBox aBoxBuilder (thePntMin, thePntMax);
+    theShape = aBoxBuilder.Shape();
+    return Standard_True;
+  }

+  void SetModule (DFBrowser_Module* theModule);
...
+  virtual TreeModel_ItemBasePtr createRootItem (const int theColumnId) Standard_OVERRIDE;

+  virtual ~TreeModel_ItemStream() Standard_OVERRIDE {};

-  Standard_EXPORT Handle(AIS_InteractiveObject) GetInteractiveObject() const;
+  Standard_EXPORT Handle(AIS_InteractiveObject) GetInteractiveObject() const
+  { return Handle(AIS_InteractiveObject)::DownCast (Object()); }
...
+  Standard_EXPORT ViewControl_TableModelValues (const Qt::Orientation& theOrientation = Qt::Vertical)
+  { SetOrientation (theOrientation); }
...
+  Standard_EXPORT static TCollection_AsciiString GetPointerPrefix() { return "0x"; }

+  TopTools_IndexedMapOfShape myChildShapes; //! cached container of child shapes
...
+    TreeModel_DimType_Rows,   //! defines number of rows
+    TreeModel_DimType_Columns //! defines number of columns
...
+
+  //!< Returns preview parameters
+  View_PreviewParameters* GetPreviewParameters() const { return myPreviewParameters; }
+
+  //!< Updates visibility of the presentations for the display type
+  Standard_EXPORT void UpdatePreview (const View_DisplayActionType theType,
+                                      const NCollection_List<Handle(Standard_Transient)>& thePresentations,
+                                      int theDisplayMode);
+
+  //!< Sets display mode for all displayed presentations
+  Standard_EXPORT void SetDisplayMode (const int theDisplayMode,
+                                       const bool theToUpdateViewer = true);
+
+  //!< Returns true if preview presentation is shown
+  Standard_EXPORT Standard_Boolean HasPreview() const { return !myPreviewPresentation.IsNull(); }
...
+  //!< Returns true if fit Fit All should be peformed automatically by each Display
+  bool IsFitAllActive() const { return myFitAllActive; }
...
+  //!< Returns current display mode: 0 - AIS_WireFrame, 1 - AIS_Shaded
+  int DisplayMode() const { return myDisplayMode; }
...
+  QSize myDefaultSize; //! default size, empty isze if it should not be used
...
+  QWidget* myMainWidget; //! parent of all controls
...
+  QVBoxLayout* myTableWidgetLayout; //! main view layout where tables or custom widgets are presented
...
+  ViewControl_TableModelValues* myModelValues; //! interface to table values
...
+  QMap<Qt::Orientation, bool> myVisibleHeader; //! table header visibility
+  QMap<Qt::Orientation, int> myDefaultSectionSize; //! table section default size

+  { return myVisibleHeader.contains(theOrientation) ? myVisibleHeader[theOrientation] : true; }

+    else
+      return QColor (Qt::darkGray).darker (150);

+  if (!GetProperties().IsNull())
+    return GetProperties()->RowCount();
+
+  return 0;

--- /dev/null
+++ b/tools/ViewControl/ViewControl_Table.cxx
+#include <limits>

+  //! - visibiblity of columns,

+  //! \return boolean value whether the property is applyed to the tree view

+class ViewControl_Widget : public QWidget

-  Standard_Integer myDragButtonDownX; //!< cached X button down by drag event
+  Standard_Integer myDragButtonDownX; //!< cached X button down by drag eventA

+    { return theActionId == View_ViewActionType_FitAllId ? myFitAllAction->isChecked() : false; }

+  //TODO: processing in local context only
+#if OCC_VERSION_HEX < 0x060901
+  Handle(AIS_LocalContext) aLContext = theContext->LocalContext();
...
 #if OCC_VERSION_HEX > 0x060901
-  myViewer = new V3d_Viewer (aGraphicDriver);
+  Handle(V3d_Viewer) aViewer = new V3d_Viewer (aGraphicDriver);

+    // using default type diring the first setting the external context

+  static void SaveState (View_PreviewParameters* theParameters,
+                         QMap<QString, QString>& theItems,
+                         const QString& thePrefix = QString())
+  { (void)theParameters; (void)theItems; (void)thePrefix; }
   //! Restore state of three view by a container

+  Handle(AIS_Trihedron) myDefaultTrihedron; //!< NULL presentation untill the first display
untill -> until

+Handle(AIS_Trihedron) View_Displayer::defaultTrihedron (const bool toCreate)


const Handle(AIS_Trihedron)&



+void View_DisplayPreview::SetDisplayMode (const int theDisplayMode, const bool theToUpdateViewer)
+{
+  if (!myPreviewPresentation.IsNull())
+     myPreviewPresentation->GetContext()->SetDisplayMode (myPreviewPresentation, theDisplayMode, theToUpdateViewer);
+
+  for (NCollection_List<Handle(AIS_InteractiveObject)>::Iterator aPreviewIt (myPreviewReadyPresentations); aPreviewIt.More(); aPreviewIt.Next())
+  {
+     aPreviewIt.Value()->GetContext()->SetDisplayMode (aPreviewIt.Value(), theDisplayMode, theToUpdateViewer);
+  }


theToUpdateViewer is expected to be processed ones, not for every object.



+    Handle(AIS_Shape)::DownCast (myPreviewPresentation)->Set (aCompound);
+    if (!myPreviewPresentation.IsNull() && !myPreviewPresentation->GetContext().IsNull())


This is illogical calling method without NULL check and check NULL the line after.



+    Handle(Convert_TransientShape) aShapePtr = Handle(Convert_TransientShape)::DownCast (anIterator.Value());
+    if (!aShapePtr.IsNull())
+    {
+      aBuilder.Add (aCompound, aShapePtr->Shape());
+    }


if (Handle(Convert_TransientShape) aShapePtr = Handle(Convert_TransientShape)::DownCast (anIterator.Value()))



+void enableGlobalClipping (const Handle(AIS_InteractiveObject)& theObject, const bool theIsEnable)


static



+    Handle(Graphic3d_SequenceOfHClipPlane) aPlanes = new Graphic3d_SequenceOfHClipPlane;


new Graphic3d_SequenceOfHClipPlane();



+  // seach for objects to be exported


seach 



+  //NCollection_List<TopoDS_Shape> aSelectedShapes = GetSelectedShapes (myTreeView->selectionModel()->selectedIndexes());


unused



+  for (int aTypeId = (int)View_DisplayActionType_DisplayId; aTypeId <= (int)View_DisplayActionType_RemoveId; aTypeId++)
+    aMenu->addAction (ViewControl_Tools::CreateAction (VInspector_Tools::DisplayActionTypeToString ((View_DisplayActionType) aTypeId),
+                      SLOT (onDisplayActionTypeClicked()), GetMainWindow(), this))


{}



+  TopExp_Explorer expS(aShape, TopAbs_EDGE);
+  for (; expS.More(); expS.Next())


for (TopExp_Explorer anEdgeIter (aShape, TopAbs_EDGE); anEdgeIter.More(); anEdgeIter.Next())



+    for (QMap<QString, QString>::const_iterator anItemsIt = anItems.begin(); anItemsIt != anItems.end(); anItemsIt++)
...
+  for (QList<TreeModel_ItemBasePtr>::const_iterator anItemIt = anItems.begin(); anItemIt != anItems.end(); anItemIt++)


++anItemIt



+      if (aSelectedIds.contains ((size_t)aPresentation.operator->()))


.get()



+    else if (View_PreviewParameters::RestoreState (displayer()->DisplayPreview()->GetPreviewParameters(), anItemKey.ToCString(),
+      anItemValue.ToCString(), "preview_parameters_"))
+      continue;


{}



+const int VINSPECTOR_DEFAULT_PROPERTY_VIEW_WIDTH = 300;//600;


Artefact.



+  Standard_EXPORT static Standard_Boolean DisplayActionTypeFromString (const Standard_CString theTypeString,


Double constness of Standard_CString.



+  TCollection_AsciiString aName (theTypeString);


const TCollection_AsciiString



+    std::vector<Handle(AIS_InteractiveObject)> aListOfIOSorted;
+    for (AIS_ListIteratorOfListOfInteractive anIOIt (aListOfIO); anIOIt.More(); anIOIt.Next())
+    {
+      aListOfIOSorted.push_back (anIOIt.Value());
+    }


aListOfIOSorted.reserve (aListOfIO.Size());



+  virtual ~VInspector_ItemHistoryElement() {};
+  virtual ~VInspector_ItemHistoryRoot() {};
+  virtual ~VInspector_ItemHistoryType() {};


Unexpected ;



-  virtual void Reset() Standard_OVERRIDE { TreeModel_ItemBase::Reset(); }
+  virtual void Reset() Standard_OVERRIDE;


Standard_EXPORT.



+  //! Makes the view expanded fron the root till the index


from



+  //! param theParent a parent item


\param



+  //! \param theColumn the item column positition in the parent item


positition 



+  //! Returns stream value of the item to fulfill property panel.
+  //! \return stream value or dummy
+  Standard_EXPORT virtual void initStream (Standard_OStream& theOStream) const;
+
+  //! Initialize the current item. It creates a backup of the specific item information
+  //! Do nothing as context has been already set into item
+  Standard_EXPORT virtual void initItem() const Standard_OVERRIDE;


Inconsistent style within the same class: Initialize->Initializes, Do nothing -> Does nothing.



+    TreeModel_RowValue (const Standard_Integer theValueStartPosition, const QVariant theKey, QVariant theValue)


const QVariant&



+    Standard_Integer myValueStartPosition; //!< start position of the key
+    QVariant myKey; //!< value in the first column
+    QVariant myValue; //!< value in the second column
+    QMap<int, QVariant> myCustomValues; //!< custom values, e.g. key is Background, value is a defined color


Either make public properties private/protected or rename without "my".



+DEFINE_STANDARD_HANDLE (TreeModel_ItemProperties, Standard_Transient)


Useless.



+// function : GetChildStream
+// purpose :
+// =======================================================================
+void TreeModel_ItemProperties::initItem() const


Mismatch.



+  (void)thePresentations;
+
+  const Standard_SStream& aStream = Item()->Stream();
+  Convert_Tools::ConvertStreamToPresentations (aStream, 1, -1, thePresentations);


Why (void)?



   QColor aHColor (229, 243, 255);
-  myViewWindow->GetDisplayer()->SetAttributeColor (Quantity_Color(aHColor.red() / 255., aHColor.green() / 255.,
-                                                   aHColor.blue() / 255., Quantity_TOC_RGB), View_PresentationType_Additional);
+  myViewWindow->Displayer()->SetAttributeColor (Quantity_Color(aHColor.red() / 255., aHColor.green() / 255.,
+                                                aHColor.blue() / 255., Quantity_TOC_RGB), View_PresentationType_Additional);
...
+      Standard_Real aRed, aGreen, aBlue;
+      aColor.Values (aRed, aGreen, aBlue, Quantity_TOC_RGB);
+
+      int aDelta = 255;
+      myRowValues.ChangeFromIndex (1).myCustomValues.insert ((int)Qt::BackgroundRole, QColor((int)(aRed * aDelta),
+        (int)(aGreen * aDelta), (int)(aBlue * aDelta)));


Quantity_TOC_sRGB to convert to/from QColor.



+  Handle(TreeModel_ItemProperties) Properties() const { return myProperties; }


const Handle(TreeModel_ItemProperties)&



+  const_cast<TreeModel_ItemBase*>(this)->myCachedValues.insert (theItemRole, aValueToCache);


mutable



+  //! Returns expode type of the item


expode



+  //! Save state of three view in a container in form: key, value. It saves:
...
+  //! Restore state of three view by a container


Probably 'tree'.



+    #define VIEWCONTROL_EXPORT __declspec(dllexport)
+  #else
+    #define VIEWCONTROL_EXPORT


Unused.	

+//function : CreateShape
+//purpose  :
+//=======================================================================
+void Convert_Tools::ConvertStreamToPresentations (const Standard_SStream&,

+  //! \param thePresentations container to collect new presentation/s

+add_definitions("-D__ViewControl_DLL")

+    Standard_Integer MyValueStartPosition; //!< start position of the key
+    QVariant MyKey; //!< value in the first column
+    QVariant MyValue; //!< value in the second column
+    QMap<int, QVariant> MyCustomValues; //!< custom values, e.g. key is Background, value is a defined color

+  {
+    TreeModel_DimType_Rows,   //! defines number of rows
+    TreeModel_DimType_Columns //! defines number of columns