DiscussionsIssue archiveOCCT:Modeling Data

Archived issue #0023620

Follow up of #22939 - make Bezier curve/surface evaluation thread-safe

CommunityOCCT:Modeling Dataclosed9 public notes

Search issues

Description

Apparently Bezier surface/curve/2d curve were overlooked in #0022939, although they have the same approach (caching of calculated value).

Steps to reproduce

Not required

Public activity

9 archived notes

Participants are labeled by their role within this record.

01Commenter 2
Branch [archived branch] has been created by Participant.

[revision removed]


Detailed log of new commits:

Author: azv
Date: Mon Nov 2 09:33:04 2015 +0300

    0023620: Follow up of 0022939 - make Bezier curve/surface evaluation thread-safe
    
    1. Remove cache from Geom_BezierCurve, Geom2d_BezierCurve and Geom_BezierSurface
    2. Add cache for Bezier curves into GeomAdaptor_Curve, Geom2dAdaptor_Curve and GeomAdaptor_Surface
    3. Update comments in corresponding cache classes
    4. Avoid frequent down-casting to B-splines in adaptors
02Commenter 2
Dear Mikhail,

Please, review branch CR23620.

Cache for Bezier curves and surfaces was moved into corresponding adaptors as it was done for B-splines before. So, direct using of Bezier geometry is thread-safe now.
03Commenter 2
Branch [archived branch] has been updated forcibly by Participant.

[revision removed]
04Commenter 4
I have reviewed the fix.
Please pass it to test after CR26252 will have been tested.
05Commenter 2
Branch [archived branch] has been updated forcibly by Participant.

[revision removed]
06Commenter 2
Branch [archived branch] has been updated forcibly by Participant.

[revision removed]
07Commenter 7
Branch [archived branch] has been rebased on CR26252_4.

Please, test it in cooperation with CR26838_1 after the issue #0026252 will be moved to TESTED state.
08Commenter 8
Tested together with branch CR26838_1
09Commenter 2
Branch [archived branch] has been deleted by Commenter 3.

[revision removed]

Related records