Archived issue #0023620
Follow up of #22939 - make Bezier curve/surface evaluation thread-safe
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.
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
[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
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.
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.
Branch [archived branch] has been updated forcibly by Participant.
[revision removed]
[revision removed]
I have reviewed the fix.
Please pass it to test after CR26252 will have been tested.
Please pass it to test after CR26252 will have been tested.
Branch [archived branch] has been updated forcibly by Participant.
[revision removed]
[revision removed]
Branch [archived branch] has been updated forcibly by Participant.
[revision removed]
[revision removed]
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.
Please, test it in cooperation with CR26838_1 after the issue #0026252 will be moved to TESTED state.
Tested together with branch CR26838_1
Branch [archived branch] has been deleted by Commenter 3.
[revision removed]
[revision removed]
Related records