0024841: Remove duplicated and/or unused instances of collections

Removed duplicated instantiations of TCollection classes:
- BRepMAT2d_SequenceOfBasicElt: duplicates MAT_SequenceOfBasicElt
- BRepMesh_NodeHasherOfDataStructureOfDelaun, BRepMesh_IDMapOfNodeOfDataStructureOfDelaun: not used
- QADNaming_DataMapOfShapeOfName: duplicates DNaming_DataMapOfShapeOfName
- TDF_LabelLabelMap: not used
- Graphic3d_SequenceOfAddress: not used
- CDM_StackOfDocument: duplicates ListOfDocument (after fix on #24742)
- Draw_MapOfFunctions: duplicates Plugin_MapOfFunctions
- ExprIntrp_SkackOfNames: duplicates TColStd_ListOfAsciiString (after fix on #24742)
- in package Storage: (H)SeqOfPersistent, MapPSDHasher, (H)SeqOfCallback: not used

In addition, inheritance of MeshVS_ColorHasher from TColStd_MapIntegerHasher is dropped as it is useless but leads to problems if TColStd_MapIntegerHasher is converted to template.
First of the two overloaded functions Draw::Load() removed as it is unused.
Removed unused classes and files from PCollection and related packages.

0024907: Compilation fails on vc12 (VS2013)

Include <algorithm> when needed.

0024913: BRepOffsetAPI_ThruSections::CreateSmoothed contains unused declaration of GeomFill_SectionGenerator

Unused local variable deleted

0024912: Remove unused Graphic3d_Strips

0024914: Micro edge is created during Boolean Operations

class IntAna_QuadQuadGeo
The changes have been done to provide the treatment the cases
when the value of the angle between the planes is small and
the origin of intersection line should be refined.

class IntTools_FaceFace
The changes have been done to provide the treatment of faces
in cases when both are based on planar surfaces.
The UV-bounds have been extended to provide correct solutions
as it has been done for all other cases with plane-based faces.

Added test case bugs/modalg_5/bug24914
Modified length of result shape in test case bugs/modalg_1/bug10232

0024916: Use custom-built Tcl instead of ActiveTcl distribution

Added instructions for building Tcl/Tk from sources
Remove Overview.md (artifact or changing case on Windows)
Correction of misprints in the documentation.

Removing file duplicate

0024873: Draw Harness, ViewerTest - add command vbounding to show presentation bounding box

Test cases for issue CR24873
findPresentation() - fix wrong type of the argument

0024799: [regression] BRepAlgoAPI_Common returns empty result
To track the modification of the source vertices, the new vertices, created in PostTreatFF, have been added to myShapesSD map.

0024893: CLang warnings -Wlogical-not-parentheses for gce_Done comparisons

Incorrect comparison in condition within RiseIf macro fixed

0024892: AIS_TexturedShape::ShowTriangles overloads mismatch

0024829: sprops with precision throws an exception

Reason of exception is reading nonexistent element of array.

Test case for issue CR24829

0024894: TKOpenGl - refactor OpenGl_FrameBuffer and OpenGl_Texture classes.

0024877: GCC 4.4.5 compiler warnings on Debian when OCCT is bult using CMake

Warnings fixed

0024882: Visualization - default orientation is ignored

Restore lines removed from V3d_View constructors.

0024867: Visualization - polygon offsets look broken

If specific layer setting is not enabled, default value extracted from current OpenGl state will be used.
Default value fix.
Updated test case bugs/vis/bug24867
Linux warning fixed.

0024869: OCAF testing framework extending

OCAF testing framework extending.
Test case for issue CR24869

0023804: Visualization - unexpected color modulation during texture mapping

AIS_TexturedShape - use material set to the Drawer, not default undefined one

Test case for issue CR23804

0024742: Remove rarely used collection classes: Set

Classes NCollection_Set and NCollection_HSet removed as unused (along with NCollection_DefineSet.hxx and NCollection_DefineHSet.hxx).

Classes TCollection_Set and TCollection_HSet removed (along with TCollection_SetIterator and TCollection_SetList nested classes).
Code previously using Set classes updated to equivalent use of Sequence (Adaptor3d and Visual3d packages) or TColStd_PackedMapOfInteger (BRepAlgo package).

In Adaptor3d_CurveOnSurface, calculation of continuity intervals refactored so as to build and store sorted sequence of reals, instead of collecting them to set, copying to array, and then sorting.

0024742: Remove rarely used collection classes: Stack

Generic class TCollection_Stack removed (along with TCollection_StackIterator and TCollection_StackNode).

Code using TCollection_Stack changed to equivalent use of TCollection_List (replacing Push -> Prepend, Top -> First, Pop -> RemoveFirst).

0024742: Remove rarely used collection classes: SList

Generic "TCollection_SList" class and nested "TCollection_SListNode" class moved (as non-generic) to the "TopLoc" package (the only place where they were instantiated).
Names of these classes changed to "TopLoc_SListOfItemLocation" and "TopLoc_SListNodeOfItemLocation".

"NCollection_SList" class removed as unused (along with NCollection_DefineSList.hxx).

0024742: Remove rarely used collection classes: Queue

Generic "TCollection_Queue" class removed (along with "TCollection_QueueNode" and three instantiations in TColStd). Template class TCollection_Queue removed.

The code using queue classes converted to use lists for the same purpose (replacing Push -> Append, Front -> First, Pop -> RemoveFirst). In OpenGl_Context, list is used as stack instead of queue, for it looks more natural for release of resources (last allocated - first released).

0024742: Remove rarely used collection classes

"TCollection_Array1Descriptor" and "TCollection_Array2Descriptor" classes were removed as unused.

0024847: CLang warnings -Wdeprecated-writable-strings in OpenGl_Display

Remove obsolete properties, remove OpenGl_Display class.
Fix compilation on Linux

0024858: Convert class V3d_ListOfTransient to non-CDL

Class V3d_ListOfTransient converted to non-CDL form (pure HXX)

0024513: Suspicious code highlighted by 0024510

Blend_Walking_1: value returned by TestArret() deleted.
IntTools_EdgeFace: deleted redundant parameter ff2 and anIsProj2 associated with ff2, since ff2 = !ff1. Refactoring.
ProjLib_ProjectedCurve: treatment of surface of revolution is symmetric for surface of revolution now. Refactoring.

0024157: Parallelization of assembly part of BO

The branch deals with the parallelization of Post-Treatment part of the Builder

0024884: expand the list of short compiler names in CMake meta-project

List of compiler short names expanded (added: gcc, gxx, mingw, clang, intel); generation of install folder name is corrected to include compiler name without spaces even if short name is not recognized.

Tcl search was fixed (tcl8.6 has higher priority than tcl or tcl8.5);
Search for fb2build.h of freetype 2.5.1 was corrected

THRDPARTY_PATH is included into LD_LIBRARY_PATH

0024707: Problem building C# sample win VC 10

Need to check and correct Target Framework in C# projects after conversion to new versions of Visual Studio described in the sample documentation

0024347: problem by tolerance of the distance of Face and Vertex

Added test case bugs/modalg_5/bug24347

0024811: Intersection is insufficient

Added test case bugs/modalg_5/bug24811

0024496: Crash on BRepAlgoAPI_Section

Added test case bugs/modalg_5/bug24496

0024629: Possibility to install binaries in vc*/bin(d), vc*/lib(d) directories

Install occt folder now can contain several occt configurations

- custom.bat/sh was added to include specific (for each occt config) custom.bat/sh files
- specific custom.bat/sh is added for each configuration (e.g., custom_vc9_win32_d.bat), generated by cmake and included within custom.bat/sh
- draw.bat has three argument, e.g., vc9 win32 debug (as occt behavior)
- sample.bat has one or four arguments, e.g., sample_name vc8 win64 release
- PATH is not changed. DRAWEXE.exe is called from specific folder

patched custom.[bat/sh].[main/in] files have higher priority than origin ones

Documentation was updated to describe new features of CMake

Seeking process of OpenCL include folder was fixed

0024810: "Patch" feature should be introduced in OCC CMake building procedure

BUILD_PATCH_DIR variable added
patched source/project files have higher priority to be included than origin source files

Incrementation of  OCCT version up to 6.7.2 dev

Update of testing cases for debug mode and MacOS

0024218: ShapeFix_Face requires double execution to produce valid shape when FixSplitFaceMode is in effect

Test case for issues CR24218

0024834: Allocation of memory for exception message must not throw another exception

- Add QA draw command OCC24834
- Make functions allocate_message/deallocate_message use malloc/free instead of operator new/delete
- Eliminate compilation error on Linux

Test case for issue CR24834

0024376: BMP image cannot be read

Test case for issues CR24376

0024852: Crash on storage of an Ocaf document in XML file format

Blanks are removed.
A check is implemented to correct a GUID in case of presence of the blanks.

Test case for issues CR24852

0024842: Crash on change of HLR-algo in top and bottom views of customer's shape

Test cases for issues CR24842

0024861: Extra solid is in the result of General Fuse Operation

I. New features:
No new features.

II. Changes:
II.1. class BOPAlgo_ShellSplitter
   - statc function:
void RefineShell(TopoDS_Shell& theShell);
has been addded.
The function provides splitting connexity block of faces when the block begins with
the face that is intended to be internal.

II.2. class BOPAlgo_ShellSplitter
   - statc function:
Standard_Boolean IsClosedShell(const TopoDS_Shell& theShell);
has been modified to process empty shells.

II.3. class BOPAlgo_ShellSplitter
   - method:
void BOPAlgo_ShellSplitter::SplitBlock(BOPTools_ConnexityBlock& aCB);
has been modified to process connexity blocks of faces when the block
begins with the face that is intended to be internal [ II.1 ].

III. Modified entities:
packages:
BOPAlgo

Test case for issue CR24861

0024857: CLang warnings -Wunused-function

Unused functions were removed.

0024798: Boolean operation CUT produces incorrect result

Face classifier has been used to check adjustment of 2d curve for the face.

Test case for issue CR24798

0024853: CLang warnings -Wunused-const-variable

Unused variables were removed.

0024840: Exception in BRepOffsetAPI_MakePipe

Test cases for issues CR24840 and CR24849

0024845: CLang warnings -Wunused-variable
Unused variables were removed.

0024800: Point of intersection was not found for 2d offset curve.

For fix this case in method Geom2dInt_Geom2dCurveTool::NBSamples() number of samples for 2d offset and trimmed curve is computed as max value from number of samples for other curve and number of samples for basis curve.

Test case for issue CR24800

0024827: Incorrect result of projection of an edge on a face

Test cases for issue CR24827

0024814: Avoid using explicit names of Handle classes

Sources corrected replacing Handle_XXX by Handle(XXX)

0024856: CLang warnings -Wunused-private-field

Actual warnings were fixed.

0024843: CLang warnings -Wconstant-logical-operand

Logical expression is corrected to take into account input argument

0024823: Hang up in "bopcheck" command

Modifications:
class IntTools_EdgeEdge
 1. method
   Standard_Boolean IntTools_EdgeEdge::FindParameters(
      const BRepAdaptor_Curve& theB
      const Standard_Real aT1,
      const Standard_Real aT2,
      const Standard_Real theRes,
      const Standard_Real thePTol,
      const Bnd_Box& theCBox,
      Standard_Real& aTB1,
      Standard_Real& aTB2)

   Added protection for large values of parameters of the edges.

 2. methods
    static Standard_Boolean BndCommon(const Bnd_Box& theB1,
                                      const Bnd_Box& theB2,
                                      Bnd_Box& theBOut)
   and
    IntTools_EdgeEdge::FindSolutions()

   have been removed as redundant.

Test case for issue CR24823

0024844: Wrong result of Boolean Cut operation.

Small correction in intersection between Cone and Torus with collinear axes.

Test case for issue CR24844

0024793: Documentation of methods using BRepFill_TypeOfContact has to be updated according to changes introduced in OCCT 6.7.0

0024846: CLang warnings -Wunneeded-internal-declaration

Unused function was removed.

0024820: CLang compiler warning -Wreturn-stack-address
The signature of the function BOPDS_DS::CommonBlock(const Handle(BOPDS_PaveBlock)& thePB)
has been changed to return handle, not a reference to handle.

0024818: CLang warnings -Wlogical-not-parentheses

CLang warnings -Wlogical-not-parentheses were fixed.

Deleted TODO in test case de/step_3/E6 regarding to issue 24249.

Deleted unnecessary TODOs in test cases with improvements on MacOS platform. Small corrections of test cases for MacOS.

0024406: Visualization - crash on re-usage of the same primitive array

Redesign Graphic3d_ArrayOfPrimitives
Store vertices data in buffer objects managed using smart-pointers
- no more low-level memory corruption by memory releasing after VBO creation.
Remove broken hasEdgeInfos.

Interleave vertex attributes (position, color, normal, uv) in single buffer.
Remove from Graphic3d_ArrayOfPrimitives methods ::Orientate().
Remove structures Graphic3d_PrimitiveArray, CALL_DEF_PARRAY.
Add support for 2D vertex arrays.

Graphic3d_Group - remove array or primitive arrays.
Introduce more universal method Graphic3d_Group::AddPrimitiveArray().

Fix warning

0024851: Extra vertex is in the result of Boolean Fuse Operation

class IntTools_FaceFace
   - statc function:
void CorrectSurfaceBoundaries(const TopoDS_Face&  theFace,
                              const Standard_Real theTolerance,
                              Standard_Real&      theumin,
                              Standard_Real&      theumax,
                              Standard_Real&      thevmin,
                              Standard_Real&      thevmax)
The faces based on cylidrical surface has been added to the list of candidates to extend the boundaries

Test case for issue CR24851

0024822: Solve selection fails after copying

The method ::Paste() copies myIndex now.
Test case for issue CR24822

0024249: Crash on ShapeFix_Shape

MAIN CHANGES:
FixAddNaturalBound: the boundaries of "natural bound addition" are restricted: a face, that does not contains an outer wire, should not have any infinite UV boundaries due to new face building (with using a surface) requires specified UV boundaries.
FixAddNaturalBound: myResult is updated in FixAddNaturalBound when the method creates a new face with natural boundary. myResult is required to be updated for next "fix small-area wires" algorithm
IsPeriodicConicalLoop: incorrect working BRepTools_WireExplorer was replaced on TopoDS_Iterator.
a natural bound is added to all the faces are constructed with UV-periodical surfaces (not only sphere and torus; e.g., closed b-splines)

other:
ShapeAnalysis: ReverceSeq renamed to ReverseSeq
BRep_Tool Pnt and Tolerance has the check for null TShape
Test cases for issue CR24249

required null checks were added
test cases were corrected according to their new behavior
Correction test case for issue CR24249

0024775: False detecting of intersection in BRepMesh_Classifier class.

Increased checking accuracy of endpoints touching of segments.
Code formatting improvement.
Test case for issue CR24775
Correction test case for issue CR24775

0024848: CLang warnings -Wunused-value

Warning should be fixed.

0024639: Parallelization FillDS part of BO

The contents dealing with:
Parallel computation of Face/Face interferences;
Parallel computation of Edge/Edge interferences (Post-Treatment Level);
Parallel intersection of bounding boxes (BOPDS_Iterator Level).

0024825: Fit Boolean Operation Algorithm to treat multiple arguments.

class BOPTest
class BOPAlgo_BOP
class BOPAlgo_Builder
class BOPTest
class BRepFeat_Builder
class BRepFeat_MakeCylindricalHole

The format of the command has been changed:
bbop r op [-s -t]
The following options has been added:
-s - run in serial mode
-t  - print the CPU time

Test cases for issue CR24825

0024821: CLang warning -Wunsequenced in StdSelect_ViewerSelector3d

0024805: Eliminate unused static functions and methods: ShallowDump(), ShallowCopy(), STANDARD_TYPE(...)

Implementation of global functions STANDARD_TYPE() for types not inheriting Standard_Transient or Standard_Persistent are eliminated.

Global functions and class methods ShallowCopy() are removed; also removed unused classes Visual3d_PickPath and Visual3d_PickDescriptor.

Global functions and class methods ShallowDump() are removed, except for classes Standard_GUID, TopLoc_Datum, and TopLoc_Location as the latter are still used in some Debug printouts.

0024785: Visualization - Modifying z-layers concept to gain more control over OpenGl depth buffer.

Cosmetic fixes.
glDepthFunc fix.

0024649: Update user's guide for Boolean operations

New User's Guide on Boolean operations added.

0024578: Typos in Developer Guide "Building OCCT from sources"

Dev guides for building 3rd party products and "short guide" to WOK have been reviewed.

0024602: Documentation Code Snippets missing sign

Fix for bug 24602 about missing <, >, " and some other signs.
Also a major review of documetation to fix wrong formatting issues.

Added TODO in test case for Debug mode

0024759: Crash on STEP import, when reading a file exported in non-"C" locale

Fix for step reading
Add test case bugs/xde/bug24759

0024534: Improve design of Image_PixMap class

Remove template methods Image_PixMap::EditData(), ::ReadData().
Add template method Image_PixMap::ChangeValue().
Remove redundant parameter for Image_PixMap::Clear() method.

Make Image_PixMapData non-template class. Introduce NCollection_Buffer.

0024538: BRepFill_CompatibleWires.cxx, implementation of function PlaneOfWire() is suboptimal

Implementation improvements and refactoring in PlaneOfWire() function.

0024708: Convertation of the generic classes to the non-generic. Part 2

Generic classes from "AppParCurves" package:
    "AppDef_SmoothCriterion", "AppDef_LinearCriteria" and "AppDef_Variational" moved to the corresponding non-generic classes "AppDef_SmoothCriterion", "AppDef_LinearCriteria" and "AppDef_Variational" to "AppDef" package. Also several "*.cxx" files of "AppDef_Variational" class merged to one ".cxx".
Generic class from "IntImp" package:
    "IntImp_ZerCOnSSParFunc" moved to the corresponding non-generic class "IntPatch_CSFunction" to "IntPatch" package.
Next unused generic classes were removed:

- IntCurveSurface_SurfaceTool
- Intf_InterferencePolygon3d
And some other minor changes.

0024520: Implementing affine transformations in ray-tracing

0024797: TKOpenGl, OpenGl_PointSprite - the assert is absent during incorrect object releasing.

0024639: Parallelization FillDS part of BO

The contents dealing with the parallel computation of Vertex/Face interferences.

0024722: Move functionality of WOK command wgendoc to OCCT tool gendoc

Command gendoc improved to:
- generate Reference Manual documentation (OCCT classes reference) with option -refman; option -overview can be used for generation of overview documentation;
- generate PDF documents for all User Guides automatically (for files listed in FILES_PDF.txt);
- check availability of third-party tools (Doxygen, Inkscape etc.) and properly report warnings and errors.
- use templates of configuration files for third-party tools instead of their generation. These template files are located in dox/resources folder

Tcl scripts are moved from dox folder to adm.
Doxygen warnings are eliminated.
Moved all auxilary functions to occaux.tcl.
Fixed Reference manual generation on *nix platform.
Fixed PDF generation on *nix platforms.

0024809: Exception is raised during perform of "bfillds" command on attached shape

On attached shape command "bfillds -s" hangs in Test Harness on the long time. In debug mode exception is raised.
Vectors are normalized before calculation of an angle.
Added test case bugs/modalg_5/bug24809

0024812: Provide VS2012 debugger visualizer for OCCT types

0024792: Remove unused hacks for compilers without STL

0024804: OSD_PerfMeter documentation is broken

0024808: cmake does not compile mfc sample

OCC_StereoConfigDlg.cxx/h were added into mfcsample CMakeLists.txt meta-project

0024656: Exception on current version of OCC libraries during perform of boolean operation (Regression)

Added protection for null vector.
Test case for issue CR24656

0024766: Wrong result for distmini between a TopoDS_Shell and a TopoDS_Edge on version 6.6.0.

Test case for issue CR24766
Modified test case bugs/modalg_5/bug24766

0024755: TDF_Label::AddAttribute() reverses the order of added attributes

Test case and new draw command for issue CR24755
Fix of regression linked with test case bug24047.
Fix of bug361 regression on Linux.

0024791: Visualization - highlighting is not cleared in local selection on some OpenGL drivers

Force GL commands execution by glFlush().

0023972: Exception thrown when intersecting two cones

DRAW command and test case for issue CR23972

New compiler warnings (vc10 64-bit) fixed in BOPCol_TBB.hxx

0024795: TKOpenGl - new compilation warnings in OpenGl_Workspace_Raytrace.cxx

0024572: Improvement performance of BRepExtrema_DistShapeShape

Fix compilations issues.
Fix compilation issues with GCC.

0024794: Drop unused files Standard_osd_ffi.c and Standard_symlink.c

0024625: Upgrade Bison to version 2.2 or above

Eliminate MSVC warnings in code generated by Bison 2.41 on Linux

Fix for building lex scanners on Windows when scanner is generated by flex 2.5.35 on Linux

0024731: Incomplete section of a surface by plane

Added test case bugs/modalg_5/bug24731