9 years ago0023392: Memory leak in OCAF in debug mode
Roman Lygin [Fri, 24 Aug 2012 11:08:50 +0000 (15:08 +0400)]
0023392: Memory leak in OCAF in debug mode

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

9 years ago0023397: Marker aspect doesn't applied to points drawn by Primitive Arrays
kgv [Wed, 22 Aug 2012 07:36:50 +0000 (11:36 +0400)]
0023397: Marker aspect doesn't applied to points drawn by Primitive Arrays

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

9 years ago0023341: Wrong result done by 2D classifier algorithm for a point and a face.
pkv [Fri, 24 Aug 2012 10:32:10 +0000 (14:32 +0400)]
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.

9 years ago0023373: MSVC++ warnings issued during compilation for 64bits, 'Sparse Arrays'
Pawel [Fri, 24 Aug 2012 10:18:55 +0000 (14:18 +0400)]
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.

9 years ago0023385: Bug in AIS_TexturedShape::Compute()
san [Thu, 16 Aug 2012 15:43:26 +0000 (19:43 +0400)]
0023385: Bug in AIS_TexturedShape::Compute()

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

9 years ago0023371: Expression which check of change of square/volume/length in end files doesn...
ski [Mon, 20 Aug 2012 09:33:29 +0000 (13:33 +0400)]
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

9 years ago0023364: Integration of test grid "cpp" into the new testing system
ski [Tue, 31 Jul 2012 14:10:09 +0000 (18:10 +0400)]
0023364: Integration of test grid "cpp" into the new testing system

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

9 years ago0023382: Integration of test grid "did" into the new testing system
ski [Wed, 15 Aug 2012 13:16:53 +0000 (17:16 +0400)]
0023382: Integration of test grid "did" into the new testing system

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

9 years ago0023369: MSVC++ warnings issued during compilation for 64bits, Materials.cxx
Pawel [Tue, 7 Aug 2012 09:07:28 +0000 (11:07 +0200)]
0023369: MSVC++ warnings issued during compilation for 64bits, Materials.cxx

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

9 years ago0023370: MSVC++ warnings issued during compilation for 64bits, WNT_FontMapEntry.cxx
Pawel [Tue, 7 Aug 2012 12:12:02 +0000 (14:12 +0200)]
0023370: MSVC++ warnings issued during compilation for 64bits, WNT_FontMapEntry.cxx

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

9 years ago0023306: Failures reading some attributes of BinOcaf document
szy [Mon, 20 Aug 2012 08:56:47 +0000 (12:56 +0400)]
0023306: Failures reading some attributes of BinOcaf document

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

9 years agoCorrected integration
bugmaster [Mon, 13 Aug 2012 11:34:43 +0000 (15:34 +0400)]
Corrected integration

9 years ago0023295: A part of conditional expression is always true
Pawel [Tue, 17 Jul 2012 09:34:42 +0000 (11:34 +0200)]
0023295: A part of conditional expression is always true

Checking if curve type is GeomAbs_BSplineCurve.

9 years ago0023349: There are identical sub-expressions 'aS1.ShapeType() != TopAbs_FACE' to...
Pawel [Thu, 26 Jul 2012 09:08:26 +0000 (11:08 +0200)]
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.

9 years ago0023296: Not verifying method return flag
Pawel [Tue, 17 Jul 2012 09:37:28 +0000 (11:37 +0200)]
0023296: Not verifying method return flag

Checking the returned flag. If the method fails return.

9 years ago0023294: Identical sub-expression in 'if-clause'
Pawel Kowalski [Fri, 10 Aug 2012 06:45:42 +0000 (10:45 +0400)]
0023294: Identical sub-expression in 'if-clause'

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

9 years ago0023365: Name collision of global IsEqual() leads to overriding user's definition
Roman Lygin [Wed, 1 Aug 2012 20:14:05 +0000 (00:14 +0400)]
0023365: Name collision of global IsEqual() leads to overriding user's definition

Prevent name collision of global IsEqual()

9 years agoCorrected integration
Pawel Kowalski [Mon, 6 Aug 2012 12:25:17 +0000 (16:25 +0400)]
Corrected integration

9 years ago0023356: Suspicious assignment inside the condition expression of 'if' operator.
Pawel [Thu, 26 Jul 2012 11:06:55 +0000 (13:06 +0200)]
0023356: Suspicious assignment inside the condition expression of 'if' operator.

Changed the assignments in the if-clauses to comparisons.

9 years ago0023347: Expression '(* q == ' ') && (* q == '\t')' is always false. Probably the...
Pawel [Wed, 25 Jul 2012 15:35:26 +0000 (17:35 +0200)]
0023347: Expression '(* q == ' ') && (* q == '\t')' is always false. Probably the '||' operator should be used here.

Replacing '&&' with '||'.

9 years ago0023348: Expression 'ii >= 0' is always true. Unsigned type value is always >= 0.
Pawel Kowalski [Fri, 3 Aug 2012 09:28:27 +0000 (13:28 +0400)]
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

9 years ago0023350: The null pointer is passed into 'strcmp' function.
Pawel [Thu, 26 Jul 2012 09:18:42 +0000 (11:18 +0200)]
0023350: The null pointer is passed into 'strcmp' function.

The method returns if it gets a NULL pointer.

9 years ago0023353: Graphic3d_NOM_STONE used twice in an if-clause.
Pawel [Thu, 26 Jul 2012 09:54:13 +0000 (11:54 +0200)]
0023353: Graphic3d_NOM_STONE used twice in an if-clause.

Graphic3d_NOM_STONE appeared twice in the if-clause.

9 years ago0023362: Adding DRAW command vsetinteriorstyle
san [Fri, 3 Aug 2012 07:35:36 +0000 (11:35 +0400)]
0023362: Adding DRAW command vsetinteriorstyle

Adding DRAW command vsetinteriorstyle
- Translated French comments to English

9 years ago0023363: [Regression] Lost gradient background when switching to the hollow interior...
san [Fri, 3 Aug 2012 07:16:58 +0000 (11:16 +0400)]
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)

9 years ago0023298: Integration of test grid "off" into the new testing system
ski [Mon, 30 Jul 2012 12:09:29 +0000 (16:09 +0400)]
0023298: Integration of test grid "off" into the new testing system

9 years agoCorrected integration
bugmaster [Fri, 27 Jul 2012 13:59:22 +0000 (17:59 +0400)]
Corrected integration

9 years ago0023330: Redundant copying in Geom2d_OffsetCurve::Geom2d_OffsetCurve method
abk [Thu, 19 Jul 2012 13:51:32 +0000 (17:51 +0400)]
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.

9 years ago0023346: MSVC++ Error C2664: 'AIS_InteractiveObject::PolygonOffsets': conversion...
Pawel Kowalski [Fri, 27 Jul 2012 12:52:37 +0000 (16:52 +0400)]
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

9 years ago0023274: MSVC++ warnings issued during compilation for 64bits
Pawel [Thu, 12 Jul 2012 15:24:03 +0000 (17:24 +0200)]
0023274: MSVC++ warnings issued during compilation for 64bits

Replacing Standard_Integer with Standard_Size to avoid warnings.

9 years ago0023332: Expression 'anIndex < 0' is always false. Unsigned type value is never ...
Pawel Kowalski [Fri, 27 Jul 2012 12:08:57 +0000 (16:08 +0400)]
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)

9 years ago0023333: The variable 'i' is being used for this loop and for the outer loop in Vrmld...
Pawel [Fri, 20 Jul 2012 11:07:26 +0000 (13:07 +0200)]
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.

9 years ago0023335: Array overrun is possible. The value of 'i' index could reach 200 in Interfa...
Pawel [Fri, 20 Jul 2012 12:16:50 +0000 (14:16 +0200)]
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.

9 years ago0023337: The 'strcmp' function returns 0 if corresponding strings are equal. (1)
Pawel [Fri, 20 Jul 2012 13:59:02 +0000 (15:59 +0200)]
0023337: The 'strcmp' function returns 0 if corresponding strings are equal. (1)

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

9 years ago0023336: The 'strcmp' function returns 0 if corresponding strings are equal.
Pawel [Fri, 20 Jul 2012 12:56:44 +0000 (14:56 +0200)]
0023336: The 'strcmp' function returns 0 if corresponding strings are equal.

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

9 years ago0023339: The 'strcmp' function returns 0 if corresponding strings are equal. (2)
Pawel [Fri, 20 Jul 2012 14:28:44 +0000 (16:28 +0200)]
0023339: The 'strcmp' function returns 0 if corresponding strings are equal. (2)

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

9 years ago0023340: The values of different enum types are compared: switch(ENUM_TYPE_A) { case...
Pawel [Fri, 20 Jul 2012 14:56:01 +0000 (16:56 +0200)]
0023340: The values of different enum types are compared: switch(ENUM_TYPE_A) { case ENUM_TYPE_B: ... }. StepToTopoDS.cxx

Corrected the used enumeration type.

9 years ago0023325: Always getting the transparency of the back face
Pawel [Wed, 18 Jul 2012 10:53:47 +0000 (12:53 +0200)]
0023325: Always getting the transparency of the back face

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

9 years ago0023319: Typo in AIS_ConcentricRelation.cxx line 156
Pawel [Wed, 18 Jul 2012 10:42:02 +0000 (12:42 +0200)]
0023319: Typo in AIS_ConcentricRelation.cxx line 156

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

9 years ago0023320: The body of 'OpenCollector' function is fully equivalent to the body of...
Pawel [Wed, 18 Jul 2012 10:43:54 +0000 (12:43 +0200)]
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.

9 years ago0023326: The 'aSibling' pointer was utilized before it was verified against nullptr...
Pawel [Wed, 18 Jul 2012 12:50:45 +0000 (14:50 +0200)]
0023326: The 'aSibling' pointer was utilized before it was verified against nullptr. ldom_element.cxx

Verifying pointer against NULL before accessing it.

9 years ago0023317: Using the iteration variable in the inner and outer loop in GGraphic2d_SetOf...
Pawel [Wed, 18 Jul 2012 08:48:50 +0000 (10:48 +0200)]
0023317: Using the iteration variable in the inner and outer loop in GGraphic2d_SetOfCurves.cxx

Replaced the iteration variable for the internal loop.

9 years ago0023318: If statement equal to else statement in AIS_ConcentricRelation.cxx, lines...
Pawel [Wed, 18 Jul 2012 10:40:06 +0000 (12:40 +0200)]
0023318: If statement equal to else statement in AIS_ConcentricRelation.cxx, lines 108-115

Corrected the else-clause.

9 years ago0023277: Buffer underflow using memset in osd_path.cxx
Pawel [Fri, 13 Jul 2012 13:40:22 +0000 (15:40 +0200)]
0023277: Buffer underflow using memset in osd_path.cxx

The parameter used in memset matches the buffer size.

9 years ago0023322: Identical sub-expressions 'MyCView.Orientation.ViewReferencePoint.y != float...
Pawel [Wed, 18 Jul 2012 10:48:11 +0000 (12:48 +0200)]
0023322: Identical sub-expressions 'MyCView.Orientation.ViewReferencePoint.y != float (Y)'

Changed the duplicate 'y' to 'z'.

9 years ago0023315: Duplicate comarison in an if-clause ib Prs2d_ToleranceFrame.cxx
Pawel [Wed, 18 Jul 2012 08:44:57 +0000 (10:44 +0200)]
0023315: Duplicate comarison in an if-clause ib Prs2d_ToleranceFrame.cxx

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

9 years ago0023342: Changing of OCCT tests according to integration of 23.07.2012
vtn [Mon, 23 Jul 2012 11:53:47 +0000 (15:53 +0400)]
0023342: Changing of OCCT tests according to integration of 23.07.2012

Tests are changed according to current state of OCCT.

9 years ago0023280: Pointer to local array is stored outside the scope of this array.
Pawel [Fri, 13 Jul 2012 14:29:20 +0000 (16:29 +0200)]
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.

9 years ago0023273: Avoid failures due to datadir commend when data files are not present
vtn [Fri, 20 Jul 2012 13:37:21 +0000 (17:37 +0400)]
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

9 years ago0023260: Regression: Instability in parallel incmesh on Linux.
dbv [Fri, 20 Jul 2012 13:18:29 +0000 (17:18 +0400)]
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

9 years ago0023244: Bug of BRepClass_FaceClassifier: it does not take into account the tolerance
jgv [Fri, 20 Jul 2012 13:12:28 +0000 (17:12 +0400)]
0023244: Bug of BRepClass_FaceClassifier: it does not take into account the tolerance

9 years ago0023224: Empty result done by intersection algorithm
ama [Mon, 16 Jul 2012 15:13:20 +0000 (19:13 +0400)]
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.

9 years ago0023265: cppcheck warning: Mismatching allocation and deallocation
Pawel [Thu, 12 Jul 2012 09:43:42 +0000 (11:43 +0200)]
0023265: cppcheck warning: Mismatching allocation and deallocation

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

9 years ago0023264: cppcheck warning: Logical conjunction always evaluates to false
Pawel [Thu, 12 Jul 2012 08:17:38 +0000 (10:17 +0200)]
0023264: cppcheck warning: Logical conjunction always evaluates to false

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

9 years ago0023279: Accessing NULL pointer
Pawel [Fri, 13 Jul 2012 14:14:38 +0000 (16:14 +0200)]
0023279: Accessing NULL pointer

Assuring the pointer is not 'NULL'.

9 years ago0023259: MSVC: C4101 warning - unreferenced local variable
Pawel [Wed, 11 Jul 2012 12:11:56 +0000 (14:11 +0200)]
0023259: MSVC: C4101 warning - unreferenced local variable

Removed unreferenced variables

9 years ago0023278: Buffer underflow using memset in osd_host.cxx
Pawel [Fri, 13 Jul 2012 14:00:52 +0000 (16:00 +0200)]
0023278: Buffer underflow using memset in osd_host.cxx

Zero memory uses the size of the array.

9 years ago0023287: IntPolyh_MaillageAffinage.cxx, line 2217: "identical sub-expressions to...
Pawel [Mon, 16 Jul 2012 10:34:22 +0000 (12:34 +0200)]
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.

9 years ago0023287: IntPolyh_MaillageAffinage.cxx, line 2217: "identical sub-expressions to...
Pawel [Mon, 16 Jul 2012 11:22:07 +0000 (13:22 +0200)]
0023287: IntPolyh_MaillageAffinage.cxx, line 2217: "identical sub-expressions to the left and to the right of the '-' operator"

The expression
changed to
In this case the points are equal and so the points number can be reduced to '1'.

9 years ago0023305: One index value verified twice in BRepFill_TrimShellCorner.cxx
Pawel [Tue, 17 Jul 2012 13:31:40 +0000 (15:31 +0200)]
0023305: One index value verified twice in BRepFill_TrimShellCorner.cxx

Verifying both affected indexes instead of one.

9 years ago0023308: A typo in if-clause, in TopOpeBRepDS_EXPORT.cxx
Pawel [Tue, 17 Jul 2012 14:08:11 +0000 (16:08 +0200)]
0023308: A typo in if-clause, in TopOpeBRepDS_EXPORT.cxx

Corrected typo in the if-clause.

9 years ago0023313: Return Standard_False in case of an unsupported case in IntAna_Curve.cxx
Pawel [Tue, 17 Jul 2012 15:38:18 +0000 (17:38 +0200)]
0023313: Return Standard_False in case of an unsupported case in IntAna_Curve.cxx

Return Standard_False in case of unsupported case.

9 years ago0023303: Expression always true in ShapeFix_Face.cxx
Pawel [Tue, 17 Jul 2012 12:43:45 +0000 (14:43 +0200)]
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.

9 years ago0023304: Copying constructor fails to copy both 'myIndx' variables.
Pawel [Tue, 17 Jul 2012 13:07:03 +0000 (15:07 +0200)]
0023304: Copying constructor fails to copy both 'myIndx' variables.

Copy constructor corrected.

9 years ago0023311: Duplicated check in if-clause in ChFi2d_Builder_0.cxx
Pawel [Tue, 17 Jul 2012 14:58:53 +0000 (16:58 +0200)]
0023311: Duplicated check in if-clause in ChFi2d_Builder_0.cxx

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

9 years ago0023307: Same variable value verified twice in TopOpeBRepTool_TOOL.cxx
Pawel [Tue, 17 Jul 2012 13:46:18 +0000 (15:46 +0200)]
0023307: Same variable value verified twice in TopOpeBRepTool_TOOL.cxx

Checking both variable values: onf, onl.

9 years ago0023300: ShapeFix_EdgeProjAux.cxx: line 283 - identical sub-expression in 'if-clause'
Pawel [Tue, 17 Jul 2012 12:23:53 +0000 (14:23 +0200)]
0023300: ShapeFix_EdgeProjAux.cxx: line 283 - identical sub-expression in 'if-clause'

Verifying also the last parameter in the if-clause.

9 years ago0023299: ShapeCustom_BSplineRestriction.cxx: line 732 - identical sub-expression...
Pawel [Tue, 17 Jul 2012 12:21:01 +0000 (14:21 +0200)]
0023299: ShapeCustom_BSplineRestriction.cxx: line 732 - identical sub-expression in 'if-clause'

Verifying both affected variables in the if-clause.

9 years ago0023301: Comparing variable to itself in ShapeUpgrade_WireDivide.cxx
Pawel [Tue, 17 Jul 2012 12:26:22 +0000 (14:26 +0200)]
0023301: Comparing variable to itself in ShapeUpgrade_WireDivide.cxx

Fixing self-comparison.

9 years ago0023276: Add generated files to .gitignore
abv [Tue, 17 Jul 2012 13:26:25 +0000 (17:26 +0400)]
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.

9 years ago0023268: cppcheck warning: Using size of pointer WIDTHMAP instead of size of its...
Pawel [Thu, 12 Jul 2012 11:13:58 +0000 (13:13 +0200)]
0023268: cppcheck warning: Using size of pointer WIDTHMAP instead of size of its data.

Using the size of the array in bytes.

9 years ago0023261: Checking handle value against NULL
Pawel [Fri, 13 Jul 2012 13:31:19 +0000 (17:31 +0400)]
0023261: Checking handle value against NULL

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

9 years ago0023258: Missing parenthesis
Pawel [Wed, 11 Jul 2012 10:43:07 +0000 (12:43 +0200)]
0023258: Missing parenthesis

Missing parenthesis added.

9 years ago0023250: Missing include in OSD_MemInfo.cxx
kgv [Wed, 11 Jul 2012 06:35:19 +0000 (10:35 +0400)]
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.

9 years ago0023253: Additional viewer 3d draw-commands vleft, vfront, ...
vro [Tue, 10 Jul 2012 07:21:58 +0000 (11:21 +0400)]
0023253: Additional viewer 3d draw-commands vleft, vfront, ...

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

9 years ago0023248: Wrong result done by solid classifier algorithm for infinite point
pkv [Fri, 6 Jul 2012 09:52:43 +0000 (13:52 +0400)]
0023248: Wrong result done by solid classifier algorithm for infinite point

9 years ago023263: Incorrect results displaying of QA command BUC60811 using "printf"
apn [Thu, 12 Jul 2012 14:00:17 +0000 (18:00 +0400)]
023263: Incorrect results displaying of QA command BUC60811 using "printf"

Modified command BUC60811

9 years ago0023201: Projection algorithm produces wrong results.
jgv [Thu, 5 Jul 2012 21:35:34 +0000 (01:35 +0400)]
0023201: Projection algorithm produces wrong results.

9 years ago0023257: Missing return statement
Pawel [Wed, 11 Jul 2012 10:12:29 +0000 (12:12 +0200)]
0023257: Missing return statement

Added missing return statement

9 years ago0023251: Crash during reading materials from STEP file for case when name of material...
gka [Mon, 9 Jul 2012 13:52:35 +0000 (17:52 +0400)]
0023251: Crash during reading materials from STEP file for case when name of material is not set.

Fix for case when materials in STEP file was written invalid way

9 years ago0023003: Photos of AIS Viewer are corrupted on Windows if tests are executed remotely
abv [Fri, 13 Jul 2012 12:09:09 +0000 (16:09 +0400)]
0023003: Photos of AIS Viewer are corrupted on Windows if tests are executed remotely

Default position of 3d viewer window created by vinit command is set to the right-left corner of the screen (0,0) to avoid problems on small screens, e.g. dummy remote desktops

Default position of 3d viewer window created by vinit command is set to be close to the right-left corner of the screen (0,0) to avoid problems on small screens, e.g. dummy screens of remote desktops

9 years ago0023226: Extend OpenGl_Context to store map of shared GPU resources
kgv [Fri, 13 Jul 2012 11:51:16 +0000 (15:51 +0400)]
0023226: Extend OpenGl_Context to store map of shared GPU resources

OpenGl_Resource was slightly corrected and OpenGl_Element was extended
with Release method to manage GPU resources.

OpenGl_PrimitiveArray now uses new OpenGl_VertexBuffer class (requires OpenGL 1.5+).
Strange workarounds for feedback mode were removed.

OpenGl_Context now provides access to shared GPU resources
and manages resources queue for delayed release
(replaces functionality of removed OpenGl_ResourceCleaner).
Loaded GL_ARB_texture_buffer_object and GL_ARB_draw_instanced extensions.

Global maps of views, workspaces and structures
were moved to OpenGl_GraphicDriver members.
UserDrawCallback() function moved to OpenGl_GraphicDriver methods.

Aspect_GraphicCallbackStruct now holds handle of OpenGl_Context
instead of system-dependent pointers to GL context definition.

New classes NCollection_Vec2, NCollection_Vec3 and NCollection_Vec4
implements interface to low-level data (points, vertices, colors) in GLSL-style.
Removed EnableVBO argument from vdrawparray Draw Harness command
Corrected compilation errors
Fixed wrong argument in Index VBO initialization
Fixed several cases of incorrect memory management in TKV3d

Destroy structures before last view removed for correct GPU resources management.

Remove groups to avoid usage of dead OpenGl_Group pointers.

Fixed mistake in #0000280 patch.
Small correction
Fixed OCC280 test command

Replace removed view within created one in ViewerTest EventManager.
ViewerTest, do not create unused 3D view

In current design NIS_View always created and used for both - NIS objects and AIS objects.

9 years ago0023087: Upgrade of the OCCT test system
vtn [Fri, 13 Jul 2012 07:42:23 +0000 (11:42 +0400)]
0023087: Upgrade of the OCCT test system

Integration of new occt testing system

9 years ago0023123: Bug/Regression in visualization of XDE documents in shaded mode
dbv [Tue, 3 Jul 2012 14:55:57 +0000 (18:55 +0400)]
0023123: Bug/Regression in visualization of XDE documents in shaded mode

Added new group creation before adding styled item in XCAFPrs_AISObject::Compute

9 years ago0022850: Not stable fix 22735
dbv [Fri, 6 Jul 2012 12:08:21 +0000 (16:08 +0400)]
0022850: Not stable fix 22735

Class TopTools_MutexForShapeProvider has been created
Class contain methods:
TopTools_MutexForShapeProvider::CreateMutexesForSubShapes - Creates and associates mutexes with each sub-shape of type theType in theShape.
TopTools_MutexForShapeProvider::CreateMutexForShape - Creates and associates mutex with theShape
TopTools_MutexForShapeProvider::GetMutex - Returns pointer to mutex associated with theShape. In case when mutex not found returns NULL.

Added method RemoveAllMutexes to TopTools_MutexForShapeProvider
Assign operator in MutexProvider, constructor and operator and assign operator in Standard_Mutex now private
Replaced TopExp_Explorer with TopoDS_Iterator to avoid cyclic dependence

9 years ago0023157: Exception reading STEP file in Debug mode
szv [Tue, 15 May 2012 10:38:08 +0000 (14:38 +0400)]
0023157: Exception reading STEP file in Debug mode

Unsafe Debug printout was removed.

9 years ago0023219: Triangulation is not built on a planar face
oan [Fri, 6 Jul 2012 11:59:17 +0000 (15:59 +0400)]
0023219: Triangulation is not built on a planar face

9 years ago0023218: Wrong value of tolerance for the intersection curve.
pkv [Fri, 6 Jul 2012 11:55:23 +0000 (15:55 +0400)]
0023218: Wrong value of tolerance for the intersection curve.

9 years ago0023072: Eliminate compiler warnings (level 3) on Windows / MSVC++
dbv [Mon, 25 Jun 2012 07:31:40 +0000 (11:31 +0400)]
0023072: Eliminate compiler warnings (level 3) on Windows / MSVC++

9 years ago0023175: Failed to segment a periodic B-Spline when parameter coincides with existing...
dbv [Thu, 21 Jun 2012 13:17:14 +0000 (17:17 +0400)]
0023175: Failed to segment a periodic B-Spline when parameter coincides with existing knot

9 years ago0023238: TKOpenGl glext header conflicts with system headers
kgv [Tue, 3 Jul 2012 12:45:17 +0000 (16:45 +0400)]
0023238: TKOpenGl glext header conflicts with system headers

9 years ago0023227: New Draw Harness command to estimate current geometry complexity of OpenGL...
kgv [Wed, 27 Jun 2012 12:41:03 +0000 (16:41 +0400)]
0023227: New Draw Harness command to estimate current geometry complexity of OpenGL scene

New command vfeedback proposed to capture GL info using GL_FEEDBACK functionality

9 years ago0023195: A method ShapeFix_Face::FixPeriodicDegeneratedMode() is not implemented
vro [Thu, 7 Jun 2012 12:08:37 +0000 (16:08 +0400)]
0023195: A method ShapeFix_Face::FixPeriodicDegeneratedMode() is not implemented

Implementation of the method ShapeFix_Face::FixPeriodicDegeneratedMode().

9 years ago0023197: Draw executable do not detect update of environment variables on Windows.
abv [Thu, 28 Jun 2012 13:48:50 +0000 (17:48 +0400)]
0023197: Draw executable do not detect update of environment variables on Windows.

New DRAW commands dgetenv and dsetenv are added, allowing to query and set environment variables as defined in C subsystem, from Tcl.

On Windows, special handler is armed in DrawDefaults providing automatic update of C environment when Tcl environment (array env) is modified. Note that this is not needed on Linux (Tcl does this internally).

Fixed problem with "array get env" command.

Added comments to _update_c_env proc.

9 years ago0023152: Possibility to have echo of DRAW commands in log file
abv [Thu, 28 Jun 2012 13:46:43 +0000 (17:46 +0400)]
0023152: Possibility to have echo of DRAW commands in log file

Two commands are added in DRAW:

decho: allows switch on/off echo of commands and their results. When echo is on, all commands implemented as OCCT DRAW C procedures will be echoed to standard output, along with their result. This can be useful to trace process of execution of script evaluated by 'source' command.

dlog: implements off-screen log for recording DRAW commands and their output for further processing in Tcl script (mainly for use in automatic tests). Run this command without arguments to get help.
Added ios::sync_with_stdio() call to Draw::BasicCommands.
Correction for compilation on Linux

9 years ago0023186: Unable to display Graphic3d_ArrayOfPoints after migrating from OCCT 6.5.2
san [Fri, 22 Jun 2012 07:33:53 +0000 (11:33 +0400)]
0023186: Unable to display Graphic3d_ArrayOfPoints after migrating from OCCT 6.5.2

GL_NONE replaced with PARRAY_DRAW_MODE_NONE to disambiguate with GL_POINTS
DRAW_MODE_NONE constant moved into OpenGl_PrimitiveArray class to avoid potential name collisions

9 years ago0023156: Image_PixMap::PixelColor() extended to return alpha value
vro [Fri, 22 Jun 2012 07:29:55 +0000 (11:29 +0400)]
0023156: Image_PixMap::PixelColor() extended to return alpha value

Implemented new method to provide access to alpha value in image.

Implemented new vreadpixel Draw Harness command to read
specified pixel value from 3D view.
vdump command - added result checks
Corrected misprint

9 years ago0023185: DrawAppliInit must be loaded from current directory
abv [Fri, 22 Jun 2012 07:19:15 +0000 (11:19 +0400)]
0023185: DrawAppliInit must be loaded from current directory

1. File src/DrawResources/DrawAppliInit removed;
   code loading QA commands moved from DrawAppliInit to DrawDefaults (temporarily)

2. In src/DrawResources/DrawDefaults:
   - change of tcl_precision removed (following advise in Tcl 8.5 documentation)
   - code loading file init.tcl removed as obsolete
   - if environment variable CSF_DrawAppliInit is defined, it is assumed to
     be the name of the application-defined file to load; otherwise
     it is searched as file DrawAppliInit in current directory

DrawAppliInit removed from FILES

9 years ago0023162: BRepOffsetAPI_MakePipeShell works wrong
eap [Fri, 22 Jun 2012 07:01:13 +0000 (11:01 +0400)]
0023162: BRepOffsetAPI_MakePipeShell works wrong
In ComputeOrigin(), superpose wire origins before checking distance between vertices
Fix for wires of one edge

9 years agoWrong bounding box for edge on periodic B-Spline with target
Roman Lygin [Sun, 20 May 2012 05:33:02 +0000 (09:33 +0400)]
Wrong bounding box for edge on periodic B-Spline with target
range beyond own range.
Improved consistency of 2D case with 3D