0023086: OCC 6.5.2 DNaming bug

The code is corrected to create instances of TNaming_Builder class dynamically. Note that they cannot be created as local variables as they should be instantiated only when needed and then reused for the subshapes of the same type in cycle.
Code around is cleaned from tabs and duplicated fragments.

0023214: BOP section operation produces incorrect result.

0023407: Draw face outlines for XDE objects

FaceOutline aspect and flag added to AIS_Drawer, Prs3d_Drawer.cdl
FaceOutlines computed by StdPrs_ShadedShape and build upon the edge triangulation.
"vshowoutlines" draw command for testing outlines on AIS_Shapes,
"XShowOutlines" draw command for testing outlines on XCAF objects.
remarks corrected:
- FaceOutline renamed to FaceBoundary
- Graphic3d_ArrayOfSegments with edges used instead of Graphic3d_ArrayOfPolylines with bounds.
draw boundaries in separate Graphic3d_Group.
Adding test cases bugs/vis/CR23407_1 CR23407_2

0023430: Tests in xcaf group fail in parallel mode

Corrected integration.

Corrected integration.

Adjusting xcaf test group

0023272: Image comparison algorithm

A new class Image_Diff for comparison of images
and a draw-command "diffimage", which compares 2 images.

Image_PixMap redesigned to provide interface for low-level image operations.
New Image_AlienPixMap class now intended for Save/Load functionality.

Aspect_PixMap class dropped.
Xw_PixMap and WNT_PixMap classes now do not inherit from Aspect_PixMap and deprecated.

ToPixMap methods now retrieve Image_PixMap as argument.

Conflicts:
src/ViewerTest/ViewerTest.cxx
Remarks applied
Fix compilation (correct merging error)

Eliminated Aspect <-> Image cyclic dependency

Fixed GIF dump in case of BGR32 image format

0023196: Porting to the latest version of ftgl library

Use FreeType entities instead of private helper classes from FTGL.
OpenGl package: link freetype.lib and ftgl.lib on Windows with #pragma comment.

0023384: Translate sub-shape names between XDE document and STEP

Sub-shapes naming translation between XDE and STEP implemented as an optional mode of Reader/Writer.
New static variables are now available: write.stepcaf.subshapes.name for Writer and read.stepcaf.subshapes.name for Reader (both have 0 values by default).
XOpen command implemented in scope of XDEDRAW asset.

Added test case bugs xde CR23384

0023403: Crash when parsing an expression with lexical error

Fix double freeing of memory in case when lexical error is detected by parser:
- revert previous fix, instead nullify the global buffer after its freeing in ExprIntrp_stop_string().

Added new QA command CR23403 and test case

0023405: BOP common produces one face instead of a solid

Modified class :IntTools_FaceFace
   - method: void IntTools_FaceFace::ComputeTolReached3d()
The goal is:
to give more precise definition to the value of tolerance of intersection curves in 3D for the case plane/sphere.

Added test case boolean/bopcommon_complex/M7
Added test case boolean/bopcommon_simple/ZP9 for verified bug CR23374

0023409: Tricheck command doesn't report problem when triangulation has unexpected holes

Tricheck command improvement for checking triangulation holes on free links
More obvious error message
Added test case bugs demo CR23409
Modified test case offset wire_closed_outside_0_005 G7

0023410: QA commands don't work properly in new testing system

Comment adding command rename. Add QA test case.
Adjusting bugs test group

0023374: BOP Common between a sphere and a box gives wrong result

0023357: The 'then' statement is equivalent to the 'else' statement.

0023412: Integration of test grid "dec" into the new testing system

Integration of test grid "dec" into the new testing system with "xcaf" name

0023377: Error in importing a rational spline IGES surface [Type 128, PROP3 = 0] into OCCT

Fix to address Rational surface with non-unitary weights at last index
Add new test case folder with test case for bugs

0023205: Patch for Naming improvement

1). Added persistence.
2) Removed not supported evolution - Replace.
QADraw commands were fixed .
Added TKBO to EXTERNLIB.
Modified test case: caf named_shape F6

Integration of adjusted testing cases.

0023393: Improve usability of OSD_MAllocHook::CollectBySize

Usability of OSD_MAllocHook::CollectBySize was improved:
- fields of OSD_MAllocHook::CollectBySize are made public (not private) to enable access for debugging purposes.
- added field myMaxAllocSize to denote maximum tracked size

0023392: Memory leak in OCAF in debug mode

TDF_LabelNode::Destroy() does not free memory for entry strings TCollection_AsciiString myDebugEntry problem was corrected.

0023397: Marker aspect doesn't applied to points drawn by Primitive Arrays

Applying color and point size (marker type currently ignored).

0023341: Wrong result done by 2D classifier algorithm for a point and a face.

Class classIntCurve_IntConicConic has been changed.
The intersection point that is definitely out of both domains is rejected.
Improving expression on lines 1101-1104 of IntCurve_IntConicConic_1.cxx to be more clear with adding comments.

0023373: MSVC++ warnings issued during compilation for 64bits, 'Sparse Arrays'

Replaced 'Standard_Integer' with 'Standard_Size' to avoid compiler warning.
Removed redundant casting to 'Standard_Size'.
Removed code checking if a 'Standard_Size' variable is negative.

0023385: Bug in AIS_TexturedShape::Compute()

Mistake in AIS_TexturedShape::Compute() occurred in the patch for issue 22971 corrected

0023371: Expression which check of change of square/volume/length in end files doesn't work properly

Modified END files
Corrected tests cases due to modification of END files

0023364: Integration of test grid "cpp" into the new testing system

Integration of test grid "cpp" (pipe) into the new testing system

0023382: Integration of test grid "did" into the new testing system

Integration of test grid "did" ( heal tests ) into the new testing system

0023369: MSVC++ warnings issued during compilation for 64bits, Materials.cxx

Replaced 'Standard_Integer' with 'Standard_Size' to avoid compiler warning.

0023370: MSVC++ warnings issued during compilation for 64bits, WNT_FontMapEntry.cxx

Replaced 'int' with 'Standard_Size' to avoid compiler warning.

0023306: Failures reading some attributes of BinOcaf document

Saving OCAF attribute TColStd_Array1OfExtendedString to binary file is corrected (missing data field written).

Corrected integration

0023295: A part of conditional expression is always true

Checking if curve type is GeomAbs_BSplineCurve.

0023349: There are identical sub-expressions 'aS1.ShapeType() != TopAbs_FACE' to the left and to the right of the '||' operator.

The same shape was checked twice if it was a face. The other shape was omitted instead.

0023296: Not verifying method return flag

Checking the returned flag. If the method fails return.

0023294: Identical sub-expression in 'if-clause'

Correcting probable copy-paste bug.
Test case offset shape_type_i D7 was modified

0023365: Name collision of global IsEqual() leads to overriding user's definition

Prevent name collision of global IsEqual()

Corrected integration

0023356: Suspicious assignment inside the condition expression of 'if' operator.

Changed the assignments in the if-clauses to comparisons.

0023347: Expression '(* q == ' ') && (* q == '\t')' is always false. Probably the '||' operator should be used here.

Replacing '&&' with '||'.

0023348: Expression 'ii >= 0' is always true. Unsigned type value is always >= 0.

Redesigned the loop to work correctly with unsigned variables.
Treatment of path to source file is rewritten using OSD_Path for more clarity
Command getsourcefile corrected to return its result as Tcl string, and simplified
Error in treatment of optional arguments in substituted puts is fixed in src/DrawResources/TestCommands.tcl
Test case added: demo draw getsource

0023350: The null pointer is passed into 'strcmp' function.

The method returns if it gets a NULL pointer.

0023353: Graphic3d_NOM_STONE used twice in an if-clause.

Graphic3d_NOM_STONE appeared twice in the if-clause.

0023362: Adding DRAW command vsetinteriorstyle

Adding DRAW command vsetinteriorstyle
- Translated French comments to English

0023363: [Regression] Lost gradient background when switching to the hollow interior style

Calling glPolygonMode() to ensure filled polygon rendering when drawing
the view background
Revert "Calling glPolygonMode() to ensure filled polygon rendering when drawing"

This reverts commit 41d41ab23cb81acb3bcf7e6ad5d10ea0097eecdd.
OpenGl_Workspace::ResetAppliedAspect() method improved in order to
reset GL parameters to default values specified in default aspects.
This method is now called twice during scene rendering:
- At the very beginning by OpenGl_Workspace::Activate()
- After all structures has been drawn but before the auxiliary stuff
(like trihedrons, overlayer, display callbacks)

0023298: Integration of test grid "off" into the new testing system

Corrected integration

0023330: Redundant copying in Geom2d_OffsetCurve::Geom2d_OffsetCurve method

Redundant copying of 'C' in Geom2d_OffsetCurve::Geom2d_OffsetCurve(const Handle(Curve)& C, const Standard_Real Offset) method was removed.

0023346: MSVC++ Error C2664: 'AIS_InteractiveObject::PolygonOffsets': conversion from 'Standard_Real' in 'Standard_ShortReal &'

Changed Standard_Real to Standard_ShortReal.

Modify test case offset shape_type_i D6

0023274: MSVC++ warnings issued during compilation for 64bits

Replacing Standard_Integer with Standard_Size to avoid warnings.

0023332: Expression 'anIndex < 0' is always false. Unsigned type value is never < 0. in Vrmldata_Geometry.cxx

Examining index sign before casting to Standard_Size.

Argument of method VrmlData_ArrayVec3d::Value() changed from Standard_Integer to Standard_Size to be consistent with its use (avoid compiler warnings)

0023333: The variable 'i' is being used for this loop and for the outer loop in Vrmldata_ShapeConvert.cxx

Use separate variable to iterate the inner loop.

0023335: Array overrun is possible. The value of 'i' index could reach 200 in Interface_MSG.cxx

Iterating from 1 to 199 over the 'buf' in order to avoid an overrun.

0023337: The 'strcmp' function returns 0 if corresponding strings are equal. (1)

Checking if the return value from 'strcmp' is '0'.

0023336: The 'strcmp' function returns 0 if corresponding strings are equal.

Checking if the return value from 'strcmp' is '0'.

0023339: The 'strcmp' function returns 0 if corresponding strings are equal. (2)

Checking if the return value from 'strcmp' is '0'.

0023340: The values of different enum types are compared: switch(ENUM_TYPE_A) { case ENUM_TYPE_B: ... }. StepToTopoDS.cxx

Corrected the used enumeration type.

0023325: Always getting the transparency of the back face

Added the 'breaks' in the affected 'switch' statement.

0023319: Typo in AIS_ConcentricRelation.cxx line 156

Checking if the 'Vertex2' (NOT 'Vertex1') is on plane in order to project it.

0023320: The body of 'OpenCollector' function is fully equivalent to the body of 'CloseCollector' function

When opening the collector setting the corresponding flag to Standard_False.

0023326: The 'aSibling' pointer was utilized before it was verified against nullptr. ldom_element.cxx

Verifying pointer against NULL before accessing it.

0023317: Using the iteration variable in the inner and outer loop in GGraphic2d_SetOfCurves.cxx

Replaced the iteration variable for the internal loop.

0023318: If statement equal to else statement in AIS_ConcentricRelation.cxx, lines 108-115

Corrected the else-clause.

0023277: Buffer underflow using memset in osd_path.cxx

The parameter used in memset matches the buffer size.

0023322: Identical sub-expressions 'MyCView.Orientation.ViewReferencePoint.y != float (Y)'

Changed the duplicate 'y' to 'z'.

0023315: Duplicate comarison in an if-clause ib Prs2d_ToleranceFrame.cxx

Testing the length of 'myTxt2'  instead of 'myTxt1' twice.

0023342: Changing of OCCT tests according to integration of 23.07.2012

Tests are changed according to current state of OCCT.

0023280: Pointer to local array is stored outside the scope of this array.

Moved the array 'buff' so it can be visible while 'str' still refers to it.

0023273: Avoid failures due to datadir commend when data files are not present

Fix for #23273 issue.
Fix for #23273 issue (boolean/013/R9).
Modifications of tests according to #23260 issue.
Attempt to protect against hang-ups due to waiting for standard input
change of behaviour of offset/shape_type_i/D7 case.
Attempt to protect against hang-ups during initialization of menu in batch mode.
increase cpulimit for parallel incmesh.
#23260 is fixed. count_parallel is not needed anymore.
Deleted unused code.
change of behaviour of offset/shape_type_i/D7 case.
The fix was incorrect.
Second attempt to protect against hang-ups during initialization of menu in batch mode
correction of tests in order to awoid instability.
Elapsed time changed to CPU user time is performance tests.
Processing of errors on Windows.
Stop cpulimit killer if armed by test, after the test is completed

0023260: Regression: Instability in parallel incmesh on Linux.

Added protection to the function which may have data race (according to the valgrind report).
Added protection to the BRepMesh_FastDiscretFace::RestoreStructureFromTriangulation function
Slight reordering to optimize use of mutex (lock once)
Now Standard_Mutex::SentryNested are created as named object.
Map inside TopTools_MutexForShapeProvider now store Handle_TopoDS_TShape as a key instead of TopoDS_Shape

0023244: Bug of BRepClass_FaceClassifier: it does not take into account the tolerance

0023224: Empty result done by intersection algorithm

Remove the cutting of the original surface from IntCurveSurface_Inter to see how much it affects the correctness of the algorithm.

0023265: cppcheck warning: Mismatching allocation and deallocation

Use pairs: new - delete (new - delete [] for arrays), malloc - free

0023264: cppcheck warning: Logical conjunction always evaluates to false

Replaced the '&&' operator with '||'. Otherwise the logical conjunction always evaluates to false.

0023279: Accessing NULL pointer

Assuring the pointer is not 'NULL'.

0023259: MSVC: C4101 warning - unreferenced local variable

Removed unreferenced variables

0023278: Buffer underflow using memset in osd_host.cxx

Zero memory uses the size of the array.

0023287: IntPolyh_MaillageAffinage.cxx, line 2217: "identical sub-expressions to the left and to the right of the '-' operator"

Changed the redundant curve type Geom(2d)_BSplineCurve to Geom(2d)_BezierCurve.

0023287: IntPolyh_MaillageAffinage.cxx, line 2217: "identical sub-expressions to the left and to the right of the '-' operator"

The expression
Abs(PEP1.U()-PEP1.U())
changed to
Abs(PEP1.U()-PEP2.U())
In this case the points are equal and so the points number can be reduced to '1'.

0023305: One index value verified twice in BRepFill_TrimShellCorner.cxx

Verifying both affected indexes instead of one.

0023308: A typo in if-clause, in TopOpeBRepDS_EXPORT.cxx

Corrected typo in the if-clause.

0023313: Return Standard_False in case of an unsupported case in IntAna_Curve.cxx

Return Standard_False in case of unsupported case.

0023303: Expression always true in ShapeFix_Face.cxx

The two corrected if-clause conditions were always true.

In FixSmallAreaWire where the documentation says: "Detects wires with small area (that is less than 100*Precision::PConfusion(). Removes these wires if they are internal." and the bugfix conforms to this.

For 'NeedCheckSplitWire' option in the Perform method the same assumption is made although no documentation on this could be found.

0023304: Copying constructor fails to copy both 'myIndx' variables.

Copy constructor corrected.

0023311: Duplicated check in if-clause in ChFi2d_Builder_0.cxx

Verifying that the second provided edge is neither a line nor a circle.

0023307: Same variable value verified twice in TopOpeBRepTool_TOOL.cxx

Checking both variable values: onf, onl.

0023300: ShapeFix_EdgeProjAux.cxx: line 283 - identical sub-expression in 'if-clause'

Verifying also the last parameter in the if-clause.

0023299: ShapeCustom_BSplineRestriction.cxx: line 732 - identical sub-expression in 'if-clause'

Verifying both affected variables in the if-clause.

0023301: Comparing variable to itself in ShapeUpgrade_WireDivide.cxx

Fixing self-comparison.

0023276: Add generated files to .gitignore

Directories for generated files (projects, Visual Studio artifacts, Emacs backup files, test results) are added to .gitignore
Suppressed generated files.
Allow to use bat, sh, am, m4 and ac files in subdirectories.

0023268: cppcheck warning: Using size of pointer WIDTHMAP instead of size of its data.

Using the size of the array in bytes.

0023261: Checking handle value against NULL

Checking handles using the IsNull() method.
Minor correction of compilation error (missing parentheses in function call)

0023258: Missing parenthesis

Missing parenthesis added.

0023250: Missing include in OSD_MemInfo.cxx

Function getpid() declared in "unistd.h" header on POSIX-compliant systems.
Fix compilation error with gcc 4.7 on Linux.

0023253: Additional viewer 3d draw-commands vleft, vfront, ...

New draw-commands vleft, vright, vbottom, vback and vfront.

0023248: Wrong result done by solid classifier algorithm for infinite point

023263: Incorrect results displaying of QA command BUC60811 using "printf"

Modified command BUC60811

0023201: Projection algorithm produces wrong results.

0023257: Missing return statement

Added missing return statement