0028728: unifysamedomain creates unnecessary copies of the vertices

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0028728

Open CASCADE

[OCCT] OCCT:Modeling Algorithms

public

2017-05-11 12:15

2017-12-05 17:09

Reporter

isn

Assigned To

isn

Priority

normal

Severity

minor

Status

assigned

Resolution

open

Platform

OS Version

Product Version

[OCCT] 7.1.0

Target Version

[OCCT] 7.4.0*

Fixed in Version

Summary

0028728: unifysamedomain creates unnecessary copies of the vertices

Description

ShapeUpgrade_UnifySameDomain class, MergeSubSeq() function:

    if (isSafeInputMode) {
      for (int k = 0; k < 2; k++) {
        if (!theContext->IsRecorded(V[k])) {
          TopoDS_Vertex Vcopy = TopoDS::Vertex(V[k].EmptyCopied());
          theContext->Replace(V[k], Vcopy);
          V[k] = Vcopy;
        }
        else
          V[k] = TopoDS::Vertex(theContext->Apply(V[k]));
      }
    }
    Handle(Geom_Line) L = new Geom_Line(gp_Ax1(PV1,Vec));
    Standard_Real dist = PV1.Distance(PV2);
    Handle(Geom_TrimmedCurve) tc = new Geom_TrimmedCurve(L,0.0,dist);
    TopoDS_Edge E;
    B.MakeEdge (E, tc ,Precision::Confusion());
    B.Add (E,V[0]); B.Add (E,V[1]);
    B.UpdateVertex(V[0], 0., E, 0.);
    B.UpdateVertex(V[1], dist, E, 0.);

If isSafeInputMode is true => empty-copies of vertices will be created to prevent modifying of the original shapes. However, UpdateVertex(..) will never modify the vertex properties in this context (0-tolerance + V[0]/V[1] can be only FORWARD/REVERSED).
I guess some of the code (related to the copying/replacing of original vertices) should be dropped. Plus, UpdateVertex(..) should be changed (now it will throw an exception anyway even if a locked vertex is not going to be modified).

Tags

No tags attached.

Test case number

Attached Files

Relationships

Notes
There are no notes attached to this issue.

Issue History
Date Modified	Username	Field	Change
2017-05-11 12:15	isn	New Issue
2017-05-11 12:15	isn	Assigned To	=> msv
2017-05-11 12:15	isn	Assigned To	msv => isn
2017-05-11 12:16	isn	Description Updated	View Revisions
2017-05-11 12:24	msv	Status	new => assigned
2017-05-11 12:24	msv	Product Version	=> 7.1.0
2017-07-24 09:22	msv	Target Version	7.2.0 => 7.3.0
2017-12-05 17:09	msv	Target Version	7.3.0 => 7.4.0*