0023586: The test execution process should correctly stop on user demand

Possibility to break DRAW commands by user break (Control-C) is implemented as follows:

- Treatment of Control-C (SIGINT signal) on UNIX in OSD::SetSignal() is made coherent with Windows implementation: instead of attempt to raise exception (simulated by longjump, does not work anyway), signal handler just sets a flag which can be later checked by OSD::ControlBreak()

- Call to OSD::ControlBreak() is added to common entry point for OCCT DRAW commands; this causes command interruption if Control-C has been pressed before its start.

- Command "dbreak" added allowing to check Control-Break status from Tcl script (raises Tcl exception if break was signaled)

- Command "dversion" added printing information on OCCT version, used build options, compiler, etc.

- Test system modified to properly handle and report user breaks and add version info in the summary log

Fix compiler error on Linux

0021985: Vista/WIndows 7 compatibility issues reported by the community

WNT_Window::Dump method calls were replaced with V3dView::Dump calls
Type casts were removed
Added supported image formats in file filters of Export methods, edited documentation for Image_AlienPixMap  class
Supported image formats filter for 2dsample was extended

0023546: Test report failure in Debug mode due to debug messages

Adding IGNORE for debug output messages in test groups heal, pipe, mesh, offset, xcaf

0023535: Exception in BSplCLib::BuildCache()

Preliminary result. It was tested with spl_2.brep file only.
Add "const"
Adding test cases for this fix

0023599: Some MFC samples are crached during launching

Fixing compiler warnings in MFC samples (MSVC compiler)
File .gitignore modified to ignore files generated by VS 2012

0023544: Texture management in TKOpenGl should be redesigned

Structures Graphic3d_CView, Graphic3d_CStructure, Graphic3d_CGroup become classes and their definitions moved from InterfaceGraphic to Graphic3d.
Introduced new class OpenGl_Texture as replacement for OpenGl_ResourceTexture class and static functions in OpenGl_TextureBox.
Graphic3d_TextureRoot now no more communicate within Graphic3d_GraphicalDriver.
Instead class returns image through GetImage() method.
OpenGl_AspectFace - avoid possible NULL-dereference
OpenGl_Texture::Init() - check gluBuild2DMipmaps() return value
OpenGl_Texture - check GL_BGRA_EXT for compatibility
OpenGl_Texture - scale NPOT image when required
Added more description to Graphic3d_TextureRoot class
OpenGl_Texture - added missing break statement for ImgBGR32 case
OpenGl_Workspace::setTextureParams() - fixed local variable aFilterMin overrides visibility of early declared variable
OpenGl_Workspace::DisableTexture() - reset texture matrix
FTGL do not reset texture matrix and corrupt text could be rendered if custom texture has not identity texture matrix.

0023546: Test report failure in Debug mode due to debug messages

Adding IGNORE for debug output messages in test groups heal, pipe, mesh, offset

0023561: stepwrite and brepiges dont work with big path (over 150 simbols) of writing file

DRAW commands stepwrite and brepiges modified to never ask for interactive input of file name or write mode; the behavior is completely defined by input arguments. String buffers causing access violation due to overflow are eliminated.

Test case bugs xde bug23561 added for this issue; TODO 23651 in related test cases removed.

Compiler warning (unrelated) is fixed in QABugs_11.cxx

0023550: Variable imagedir is not defined in interactive run of tests

When test is run interactively, imagedir is set to temporary directory (search order: environment variables TempDir, Temp, Tmp, upper and lower case, in  this order, then $HOME/tmp)
Update testing case with imagedir variable

0023415: OSD_FontMgr can't idenify aspect for fonts with names dependant on system locale.

Added function DetectFontsAspects to Font_FontMgr class. This function uses workaround from OpenGl_FontMgr with FreeType for detecting font aspect.
Removed font name parsing from Font_FontMgr::InitFontDataBase(). Now the font name and font style we get through the FreeType.
Fixed Unix part of Font_FontMgr::InitFontDataBase() method. Font name and font style now detected through the FreeType.
Remarks fix. Added recursive default font directories scanning .
Fixed adding fonts folders recursively from configuration files.
Moved fonts aliases map from OpenGl_Display_1 to Font_FontMgr.
Moved fonts name definition from Graphic3d_NameOfFont.hxx to Font_NameOfFont.hxx.
Added new methods to Font_FontMgr: GetAvailableFontsNames, GetFont and FindFont.
Modified Font_SystemFont creation from XLFD. Added method IsEqual to Font_SystemFont.
Modified methods OpenGl_Display::FindFont, OpenGl_FontMgr::request_font in accordance to the new functionality of the Font_FontMgr.
OpenGl_FontMgr now stores only generated fonts instead of duplication of available fonts list.
Removed method OpenGl_FontMgr::requestFontList. Its function now performs Font_FontMgr::GetAvailableFontsNames.
Documentation was fixed
Adjusting testing cases for current state of OCCT

0023609: Tests should handle output containing word error as failure regardless of case

File parse.rules modified to handle word "error" case insensitive

0023597: Failed to export ellipse into STEP with units other than mm
Test case for this bug

CR23589: performance improvements in curve-curve intersection

0023601: Member fields of BRepTools_ReShape made protected

0023596: Extending XCAFPrs_AISObject to enable customization of default color

0023594: XCAFDoc_LayerTool extended to work with layer defined by its label, not string

0023593: XCAFDoc_DocumentTool label must be removed from internal map upon closing the XDE document

0023592: Enabling TBB allocator by default for OCC built with -DHAVE_TBB
Implementation updated based on discussion with OCC team

CR23590: IGESCAFControl_Writer extended to accept TDF_LabelSequence

Reorganization of bugs testing group

Adjusting testing cases

Adjusting testing cases for current state of OCCT

Integration part of tests of grid chl

0023237: OSD_PerfMeter reports wrong (zero) times

Commit details:
1) in OSD_PerfMeter, use static functions of OSD_Chronometer class for time measurements instead of specific code to avoid incorrect results on CentOS (due to wrong valus of CLK_TCK);
2) changed definition of OSD_PerfMeter from .c to .cxx to avoid problems with C-functions;
3) fixed OSD_PerfMeter.h for building on Unix systems;
4) removed platform-specific #defines;
5) added test case for OSD_PerfMeter as bugs fclasses bug23237;
6) Removed DebugTools package (duplicates OSD_PerfMeter)
7) Avoid compiler (GCC) error casting BRepPrimAPI_Make* instances to TopoDS_Shape

0023569: Adding NCollection_StdAllocator

Adding NCollection_StdAllocator
Adding NCollection_StdAllocator (correcting previous commit)
- addressed code review comment by kgv with using 2 (vs 4) spaces and amending max_size()

Add new draw-commands
Adding test case for this bug

0023241: DRAWEXE crash on startup in Mac OS X

Fixing problem with Draw starting
Added problem description into the sources.
Cosmetic fix. Replaced tabs with spaces.

0023584: Tests in group geometry generate gif images instead of png

File end in group geometry modified to produce PNG images instead of GIF

23563: Introduce support of MS Visual Studio 2012

unused Mapiwi.h commented
Update of OCCT samples for vc11
Update of target folders for vc11
Update of generation-vc-projects.bat files

0023418: Crash on the object displaying when running DRAW on remote station. OpenGL version determination was corrected

Small style changes
Comments were corrected

0023532: Mismatching allocation and deallocation: anAsciiString, CDM_COutMessageDriver.cxx

Replaced 'delete' with 'delete []' when deallocating array.

0023498: Comparison always true in Units_Lexicon::Creates()

Method Units_Lexicon::Creates refactored
Further refactoring of Units_Lexicon::Creates() and Units_UnitsDictionary::Creates()

Adjusting testing cases for current state of OCCT

0023556: Identical sub-expressions to the left and to the right of the '||' operator

The second condition changed to: (allMovePenToY->Lower() != 1 || allMovePenToY->Length() != Len)

0023560: Redundant copying in Geom_OffsetCurve::Geom_OffsetCurve constructor

Eliminating redundant copying of 'C' in Geom_OffsetCurve::Geom_OffsetCurve
  const Handle(Curve)& C,
  const Standard_Real Offset,
  const Dir & V) constructor was removed.

0023567: Wrong STEP unit is returned by STEPControl_Reader.FileUnits() function

Fix bug with using of uninitialized variable.
New version of patch: pass unit object to the getSiName() function directly.
Test case for this bug

0023132: Suspicious code snippets

1) Warning in Aspect.cxx couldn't be reproduced
2) Description of changes: added 'return' statements into ReadStep(...) functions of RWStepBasic_* classes.
3) BRepFill_OffsetWire.cxx - removed dead code;
4) IFSelect_WorkSession.cxx - swapped arguments;
5) TopOpeBRep_ShapeIntersector2d.cxx - removed 'brake' statement and changed loop to if-statement because of
     void TopOpeBRep_ShapeIntersector2d::FindFFIntersection() function's call features.
6) V2d_RectangularGraphicGrid.cxx - left constants instead of functions beacuse of faster perfomance.
7) Commented unreachable code in files GeomFill_LocationGuide.cxx and TopOpeBRepTool_mkTondgE.cxx

0023286: Standard_Mutex behavior depends on platform

Implemented recursive POSIX mutex instead of non-recursive,
Removed SentryNested class, implemented it's features into Sentry class
Added second constructor to Sentry class

0023130: command "maxtolerance" doesn't work on Windows

maxtol command returns incorrect values of variables nbFaces, nbEdges and nbVtx.
Test case for this bug

0023487: Remove obsolete BRepTools::OuterShell() function

All corresponding calls to BRepTools::OuterShell() replaced by function BRepClass3d::OuterShell()
Changed function BRepClass3d::OuterShell

0023379: Exporting single point into STEP produces a compound with two points

Adding single vertex for second time was removed.
Adding single vertex in first time was removed, but for second time was returned.
Duplication of vertex compound in STEPControl_GeometricCurveSet mode was removed.
Some cosmetic changes
Test case for this bug

0023120: Transform persistence is not restored

Updating myTransPers field in any case in OpenGl_View::BeginTransformPersistence()
Test case for this bug

0023572: Wrong line endings in *.yacc and *.lex files on Linux

Adjusting testing cases for current state of OCCT

0023473: Top face of imported STEP part is missing

Fix for constraints for case of complex internal wires
Test case for this bug

0023548: Boolean operation between two faces fails

0023290: IntCurve_IntPolyPolyGen.gxx, suspicious if/for
unnecessary code was deleted

0022835: IncrementalMesh generates FLT exceptions when running in parallel mode

Adding test case

0023475: Wrong result of Geom_BSpline_Surface::IsVClosed()

Modifications:
Definition closure surface by U and V by distance between isolines on bounds
Comparison to isolines on identity
Implementation of short dump of surface and curves
Change order of comparison

Fix and test script for bug 0023475
Modification of script to restore surface
Modification for correction regression ( UIso and VIso were confused)
Modified test case and test surface for bug 0023475

0022927: Regression with OCCT652 on Windows. DRAWEXE crashes during offsetcomshape.

Check was added to avoid a crash if result is not reachable.
Modified test case

0023291: GccAna_Circ2d3Tan_8.cxx, suspicious if

one of the duplicate conditions has been removed
Code was changed to be more obvious.
Unite if blocks into one block
Small correction of coding style

0022506: Wrong factor for elapsed time mesaured with GetTickCount() in OSD_Timer

Commit details: set the proper factor value.

0022522: Small BUG in DRAWEXE - vtrihedron command

Add condition to check if shape name is already bound
Used VDisplayAISObject() to display an object
vtrihedron - fixed parsing of arguments with Z direction definition

0023329: Edit comment of function Epsilon (Standard_Real.hxx)

0022970: Incorrect array use in Graphic3d_StructureManager.cxx

Corrected bounds of loops and initialisation error message.

0022832: Not documented limitation of Graphic3d_StructureManager

Added information about viewers count limitation in CDL documentation.

0023432: Connected Interactive Objects computed without Interactive Context

Setup interactive context for reference(s) in Compute() method of classes AIS_ConnectedInteractive and AIS_MultipleConnectedInteractive
Extend vconnectsh and vconnect to process several input objects, added displaying color for result object
Fixed errors in QAGetPixelColor DRAW command
Edit QAGetPixelColor command and test case
ViewerTest::GetColorFromName() - fixed iteration bounds
QAAISGetPixelColor() - removed redundant conversions
Turn back QAGetPixelColor output syntax in case of 3 arguments
Integration of test script to new test system

0023361: Bug in gp_Trsf::Multiply

Bug in gp_Trsf::Multiply fixed.
If we multiply 2 gp_Trsf objects (shape == gp_Rotation), for example, t1 * t2, and t2.loc = 0
(t2.loc.x == 0 and t2.loc.y == 0 and t2.loc.z == 0)
then t1.loc will not change (you can verify it from the matrix multiplication)
Adding test case

0023292:The variable 'k' is being used for this loop and for the outer loop.

New variable 'm' was defined for the inner loop in line 212 of gccana_circ2d3tan_2.cxx.

Rebased on current master.

0022545: Improved exception handling

0023141: Suspicious if (2)

Fixed condition which was always true

0023144: Suspicious if (3)

Fixed mistake in Standard_Boolean IntCoCo ( ... ) function of IntPatch_ImpImpIntersection_5.gxx file.
Fixed this condition because of the opposite parametrisation direction of the intersection line.

0023143: Suspicious else

Changes description:
In src\IntPatch\IntPatch_ImpPrmIntersection.cxx , in ForcedPurgePoints function changed
ParametersOnS2() to ParametersOnS1() in cases of
Standard_Boolean IsReversed == false in lines 1522 & 1540
Got rid from duplicated code in function static void ForcedPurgePoints(...)

0022591: Migration to FreeImage: texture management

Graphic3d_TextureRoot - migrate to Image_AlienPixMap
OpenGl_View::CreateBackgroundTexture() - migrate to Image_AlienPixMap
Fixed Graphic3d_TextureRoot::LoadTexture() method.

0023142: GccAna : suspicious else

the else should be signe = +1

0023281: Removed unused OpenGl_ImageBox

0023312: Suspicious for loop in BiTgte_Blend.cxx

Deleted line 'NbS=0;' to allow next loop be executable

0023161: Select publicly accessible data files for OCCT tests

Duplicated data files are removed.
New command testfile added for automation of analysis of the new data files before adding to the data base.

0023500: MFC texture example is crashed when view closed

x64 sample build directory has been changed to win64

Update version for OCCT 6.5.4 official release

0023489: Memory leak in TNaming_NamedShape

Added test for memory leak (bugs/ocaf/bug23489)
Missing return added in QANewBRepNaming_BooleanOperationFeat::IsWRCase2()
Adding test case and general procedure for trend checking for detection of memory leaks

0023372: "diffimage" using in tests commands

Comparison of images is implemented in testdiff command; output in HTML form generated
Add measurement of memory (working set) delta in test case execution
Add memory statistics to output of testdiff command
Treatment of parameter -outdir of command testgrid corrected.
Function locate_data_file and code to run tests in parallel revised to be usable with Tcl 8.4.
Added function testsummarize: regenerate summary log of tests from test case logs
Command testdiff is protected against exception raised by diffimage if images have different formats

0023193: Some triangles are inverted when writing an STL file

Adding test case

0004894: Problems with shading in vdisplay

Adding test case

0023289: IntCurve_IntPolyPolyGen.gxx, suspicious else

if-else blocks in lines 805 and 822 were removed.
Corrected initialization of 'nbsamples' variable

0023468: Include current branch name into default name of directory for test results

Default name for results directory is generated as "results_<branch>_<timestamp>".

In the HTML log of test execution references to script files are made HTML links to relevant files

Aded possibility to put data file for use by test script into subdirectory data of the script folder (function locate_data_file is extended to find such files).

Test demo/testsystem/A1 renamed to locate_data_file and corrected to account for the recent changes.

Default value of CSF_TestDataPath is defined pointing to $CASROOT/data

File psrse.rules corrected in accordance with changed message on missing data file

Interface of command testgrid changed:
- output directory (formerly required argument) is now defined by optional parameter -outdir
- by default tests are run in parallel mode with number of processes equal to number of system CPUs
- check for input arguments is made more strict
Treatment of parameter -outdir of command testgrid corrected.
Function locate_data_file and code to run tests in parallel revised to be usable with Tcl 8.4.
Adjusting testing case demo draw getsource for current state of master

0023500: MFC samples are crached on 3D visualisation

void FreeTexture() in OpenGl_TextureBox.cxx - avoid possible NULL-pointer dereference

after debug x64 compilation the executable viewer3d had been moved to the bin folder. now executable viewer3d located in bind folder.

0023139: BRepFilletAPI_MakeFillet algorithm builds edges with wrong first and last parameters: out of pcurve

- Revert previous integration

0023275: restore command set name of result variable incorrectly if absolute path is used

Command restore corrected to set default name of the shape equal to filename without extension
Test case added (demo draw restore)

0022779: Pixel format should be chosen to support stencil buffer

Missing code in find_pixel_format() function was restored.
Coding rules applied
Added changes in GLX part

0023383: Bugs in CDL documentation of visualization classes

Added lines in CDL documentation for SetDisplayPriority and SetViewingVolume.

0023488: Make video recording functionality (OpenGl_AVIWriter) optional

CSF_AviLibs missing in EXTERNLIB of TKOpenGl toolkit. This cause incorrect project generation.
HAVE_VIDEOCAPTURE now should be defined to enable video recording capabilities on Windows.

0023116: Add Standard_EXPORT declaration to method Poly_MakeLoops::GetHangingLinks

0022754: Add forgotten include to BRepBlend_BlendTool.lxx

0023147: Suspicious if (5)

Removed non-executed parts of code because

   Standard_Boolean s1Point = Standard_False;
   Standard_Boolean s2Point = Standard_False;
   Standard_Boolean vClosed = Standard_False;

and never changed thus some parts of code are non-reachable.
Fixed mistake with 'else' without corresponding 'if'.
Commented unreachable code part.

0023140: Suspicious if

Condition that is always true was replaced on assigning a variable 'Tol' to Precision::Confusion.

0023145: Suspicious else (2)

else statement was incorrect, is has been changed.

0023288: IntCurve_IntConicConic_1.cxx: if(A) {...} else if (A){...} pattern detected.

Unrealizable condition ( second condition block 'else if(d0102>AbsR1mR2-Tol)' ) was deleted.

0023476: Integration of test grid "cge" into the new testing system

0023480: New compiler warnings on MS VC 9

Unused local variables eliminated

0022871: Step Reader raises exception on invalid entity (null swept curve)

checking for null swept curve was added
Adding test case

0023448: Fix QT samples for Mac OS X

Fixed QT samples so they can be built on Mac OS X with qt-x11

0023464: Projection algorithm produces wrong results.

including of .hxx file added
Correction of computation of intersection point
Integration of test cases for this issue

Adjusting testing cases for current state of OCCT.

0023472: BRepAlgoAPI_Section algorithm fails with exception while intersecting two faces

0023338: The 'then' statement is equivalent to the 'else' statement. IGEStoBRep_TopoSurface.cxx

0023367: New functionality restoring the middle path of pipe-like shape

0023470: Boolean Fuse between two edges fails

0023354: The use of 'if (A) {...} else if (A) {...}' pattern was detected.

second else was deleted, now "D" is hotkey for reset view and removing selected object is impossible by hotkeys
help message was rewrited according to these changes, also hotkeys "R","L" and "B" added to help
Minor remarks: viewer commands help fulfilled with U; Z; ","; "." . Comments revised.

0023434: Generate images in lossless format (PNG) instead of lossy GIF currently used in test log

0023351: The use of 'if (A) {...} else if (A) {...}' pattern was detected.
Got rid of duplicated code.
Got rid of redundant conditions' checks in Standard_Integer TOPOC(Draw_Interpretor& interpretor,Standard_Integer na,const char** a) function.