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.

Adjusting testing case for current state of OCCT

Adjusting testing cases for current state of OCCT

0023367: New functionality restoring the middle path of pipe-like shape
Version 2
Small correction
Adding test cases

Update OCCT version up to 6.5.4beta1

0023345: Crash when destroying OpenGl_Element

OpenGl_PrimitiveArray::Release() - avoid possible NULL-pointer dereference
OpenGl_GraphicDriver::RemoveView() - release GL resources within removing last view
Fixed OCC280 test command - do not remove old view until new one is initialized

0023466: Move OSD_FontMgr class outside TKernel

Created new package Font in TKService.
Classes FontMgr and SystemFont have been moved from OSD package to Font package.

0023098: Cppcheck warnings/errors in QA files

Fix for cppcheck warnings
Warning fixes

0023266: cppcheck warning: Logical disjunction always evaluates to true

Logical disjunction changed to  Logical conjunction, it evaluates true only then number of parameters isn't right.

0023467: BRepOffsetAPI_MakeOffset throws StdFail_NotDone and not Standard_ConstructionError

0023428: Extend OpenGl_Context to use Geometry Shaders extension
Additional corrective integration

Update vc10 project for Animation MVC sample.

0023450: Test bugs vis CR23407_1 fails
Files parse.rules corrected (point 2 in bug description)
Adjusting testing cases for current state of master.
Modified test cases CR23407_1 and CR23407_2

0023438: Update test cases for new organization of data files

Modifications:
1.Test cases
2. locate_data_file from DrawResources/TestCommands.tcl
Fix for mesh end
Adjusting testing cases for current state of master using new organization of data files

0021189: Clean up KAS:dev:ros and Products

VoxelDemo is modified to be compliant with OCC 6.5.4
Update of environment files and VS projects.

Adjusting test cases for current state of OCCT

0023456: Update MFC samples for OCCT 6.5.4
Update About dialog
Update reading files in OCAF sample

0023137: Class BRepAlgoAPI_Cut does not made correct result on attached shapes.

class ShellFaceClassifier;
   - method:
void BOP_ShellFaceClassifier::ResetElement(const TopoDS_Shape& theElement)

The order of choice a testing point for the element has been changed.
The following order is:
-point inside an edge
-point as an vertex-point
-point inside UV-range of a surface

Adding test cases
Expected sduare is corrected

0021189: Clean up KAS:dev:ros and Products

Removal of VoxelClient
VoxelDemo sample is redesigned so that it doesn't require compilation of OpenGl classes. It refers to TKOpenGl.dll as to an external library.
Some minor bugs are fixed in OCAF and Viewer 3d standard MFC samples

0023234: Incorrect behavior of AIS_Trihedron

AIS_InteractiveContext::myLocalContexts map accessed before newly created AIS_LocalContext bound to it
Minor remarks: comments revised.
Adding test case
Adding new draw command
Small correction of tests

0023453: Infinite loop on cut operation

In IntTools_FClass2d.cxx, do {} while {} cycle is replaced by for () cycle to avoid possible infinite loop.
Check for degeneration is made with Precision::Confusion() precision instead of comparison with 0.
Correction of misprint

0023404: Create SquareConfusion function in Precision package for speed and convenience

SquareConfusion function was created in FoundationClasses module -> TKMath toolkit -> Precision package.
The function returns square of Precision::Confusion().
SquareConfusion function was corrected in FoundationClasses module -> TKMath toolkit -> Precision package.
Squares of Precision::Confusion() were replaced by Precision::SquareConfusion().

0023431: BOP Cut produces invalid shape

0023125: Wrong results done by classifier algorithm for a point and a solid.

class :BRepClass3d_SolidExplorer;
   - method:
Standard_Integer BRepClass3d_SolidExplorer::OtherSegment(const gp_Pnt& P,
                               gp_Lin& L,
                               Standard_Real& _Par)

The object of the type Extrema_ExtPS is created using real parametric ranges (U,V) of the surface.

0023442: Provide the access to the functionality of Geom2dHatch_Hacher via IntTools_Context

class IntTools_Context
- the method:
Geom2dHatch_Hatcher& IntTools_Context::Hatcher(const TopoDS_Face& aF)
has been added.
The method returns the reference to 2D hatcher for given face <aF>

class IntTools_Context
- the method: IntTools_Context::~IntTools_Context()
has been modified in terms of I.1.

0023423: Test system does not get log if DRAW crashes or gets killed

Avoid using dlog in grid tests (command testgrid), in order to get some output even if DRAW crashes.
Command test now accepts argument echo to get echo immediately (but no log processing).
Test case for checking log processing in case of premature exit of DRAW is added (demo testsystem premature_exit).

0023428: Extend OpenGl_Context to use Geometry Shaders extension

0023388: Boolean operations hang up trying to build section of two customer's shapes

0023425: diffimage generate OSD_Exception during execution

diffimage out-of-image access during border filter execution

Verify neighbor pixel position for out-of-image access
A list of referenced articles and books is added to Image_Diff.hxx file.

Adding test case bugs/vis/CR23425

Image_Diff - fix compilation error on x86_64 target using msvc
There no std::abs for ptrdiff_t in Microsoft STL implementation

Image_PixMap - fixed compilation on gcc 3

Removed extra newlines in TCL script
Image_Diff, move back extra modifications

0023316: OpenGl package can not be compiled on RedHat40-64

OpenGl_Window avoid early unprotected glx.h inclusion
which may cause system glext.h inclusion

0023394: Problem with BRepOffset_MakeOffset on a cylindrical face

0023358: Unitialized variables used.

Draw command was fixed

0023129: BRepTools::OuterShell() works wrong - it always returns the first shell

0023252: Fillet regression

0023433: Compiler warnings in Graphic3d_ArrayOfPrimitives.lxx

Conversion of color components from real [0,1] to char [0,255] corrected to avoid compiler warnings