MantisBT - Community
View Issue Details
0027234Community[OCCT] OCCT:Modeling Algorithmspublic2016-03-08 19:342016-12-09 16:37
abv 
bugmaster 
normalminor 
closedfixed 
 
[OCCT] 7.1.0[OCCT] 7.1.0 
Not needed
0027234: Code duplication: Convert_CompBezierCurvesToBSplineCurve* in ShapeConstruct
Code duplication of classes Convert_CompBezierCurvesToBSplineCurve and ShapeConstruct_CompBezierCurvesToBSplineCurve has been reported at http://dev.opencascade.org/index.php?q=node/1149 [^]

Actually classes from ShapeConstruct are not used in OCCT; classes from Convert have been evolving more since duplication. However, classes in ShapeConstruct contain one fix not present in Convert: protection against joining segments when degree is 1.

Naturally duplicate classes from ShapeConstruct shall be removed.
N/A
No tags attached.
Issue History
2016-03-08 19:34abvNew Issue
2016-03-08 19:34abvAssigned To => msv
2016-03-08 19:41gitNote Added: 0051436
2016-03-08 19:41abvRelationship addedrelated to 0025256
2016-03-08 19:45abvNote Added: 0051437
2016-03-08 19:45abvStatusnew => resolved
2016-03-08 20:21gitNote Added: 0051438
2016-03-09 10:15msvNote Added: 0051451
2016-03-09 10:15msvAssigned Tomsv => abv
2016-03-09 10:15msvStatusresolved => assigned
2016-03-11 17:02gitNote Added: 0051561
2016-03-11 17:02abvNote Added: 0051562
2016-03-11 17:02abvAssigned Toabv => msv
2016-03-11 17:02abvStatusassigned => resolved
2016-03-11 17:55msvNote Added: 0051565
2016-03-11 17:55msvAssigned Tomsv => bugmaster
2016-03-11 17:55msvStatusresolved => reviewed
2016-03-11 17:56mkvAssigned Tobugmaster => mkv
2016-03-15 13:18mkvNote Added: 0051628
2016-03-15 13:18mkvNote Added: 0051629
2016-03-15 13:18mkvAssigned Tomkv => abv
2016-03-15 13:18mkvStatusreviewed => assigned
2016-03-15 13:18mkvTest case number => Not needed
2016-03-15 13:43abvNote Added: 0051631
2016-03-15 13:43abvStatusassigned => feedback
2016-03-15 17:29abvAssigned Toabv => mkv
2016-03-16 14:38mkvNote Added: 0051700
2016-03-16 14:38mkvNote Added: 0051701
2016-03-16 14:38mkvAssigned Tomkv => bugmaster
2016-03-16 14:38mkvStatusfeedback => tested
2016-04-22 16:48bugmasterChangeset attached => occt master 9a9a3edf
2016-04-22 16:48bugmasterStatustested => verified
2016-04-22 16:48bugmasterResolutionopen => fixed
2016-05-20 12:23gitNote Added: 0054229
2016-12-09 16:31aivStatusverified => closed
2016-12-09 16:37aivFixed in Version => 7.1.0

Notes
(0051436)
git   
2016-03-08 19:41   
Branch CR27234 has been created by abv.

SHA-1: cbdd59b334c2fdde34fe1fe77a48402bf67e2c18


Detailed log of new commits:

Author: abv
Date: Tue Mar 8 19:39:15 2016 +0300

    0027234: Code duplication: Convert_CompBezierCurvesToBSplineCurve* in ShapeConstruct
    
    Classes from ShapeConstruct duplicating the ones from Convert package are removed.
    Protection against joining segments when degree is 1 is introduced in Convert classes.
    Optimization previously made in Convert_CompBezierCurvesToBSplineCurve class (within #25256) is applied to 2d equivalent.
(0051437)
abv   
2016-03-08 19:45   
Fix pushed to CR27234, please review
(0051438)
git   
2016-03-08 20:21   
Branch CR27234 has been updated forcibly by abv.

SHA-1: eb312d301a5db60589a97719eaf9a6d60dea1767
(0051451)
msv   
2016-03-09 10:15   
src\Convert\Convert_CompBezierCurvesToBSplineCurve.cxx

- Please comment out the output to cout in the line 56.
(0051561)
git   
2016-03-11 17:02   
Branch CR27234 has been updated by abv.

SHA-1: cb596cfad41b1e8919e8857a6be3e9c52edefabc


Detailed log of new commits:

Author: abv
Date: Fri Mar 11 17:01:52 2016 +0300

    // cout disabled unless OCCT_DEBUG is defined

(0051562)
abv   
2016-03-11 17:02   
Done, please review again
(0051565)
msv   
2016-03-11 17:55   
Reviewed.
(0051628)
mkv   
2016-03-15 13:18   
Dear BugMaster,
Branch CR27234 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested on Release mode.
SHA-1: cb596cfad41b1e8919e8857a6be3e9c52edefabc

There are following compilation errors:
Windows:
http://jenkins-test-01.nnov.opencascade.com:8080/view/CR27234-master/job/CR27234-master_build_products_csharp_samples_windows/1/parsed_console/ [^]
occtypes.i(367) : Error: Unable to find 'ShapeConstruct_CompBezierCurves2dToBSplineCurve2d.hxx'
occtypes.i(367) : Error: Unable to find 'ShapeConstruct_CompBezierCurvesToBSplineCurve.hxx'

http://jenkins-test-01.nnov.opencascade.com:8080/view/CR27234-master/job/CR27234-master_build_products_java_samples_windows/1/parsed_console/ [^]
occtypes.i(310) : Error: Unable to find 'ShapeConstruct_CompBezierCurves2dToBSplineCurve2d.hxx'
occtypes.i(310) : Error: Unable to find 'ShapeConstruct_CompBezierCurvesToBSplineCurve.hxx'

Number of compiler warnings:

occt component :
Linux: 0 (0 on master)
Windows: 0 (0 on master)
MacOS : 0 (0 on master)

products component :
Linux: 41 (41 on master)
Windows: 0 (0 on master)

Regressions/Differences/Improvements:
No regressions/differences

Testing cases:
Not needed

Testing on Linux:
occt component :
Total MEMORY difference: 90004625 / 90985305 [-1.08%]
Total CPU difference: 18964.12999999975 / 19490.33000000016 [-2.70%]
products component :
Total MEMORY difference: 25997916 / 25908972 [+0.34%]
Total CPU difference: 5334.429999999985 / 5284.629999999986 [+0.94%]

Testing on Windows:
occt component :
Total MEMORY difference: 58139020 / 57324399 [+1.42%]
Total CPU difference: 17382.39582489897 / 18302.94212579877 [-5.03%]
products component :
Total MEMORY difference: 17971259 / 17515721 [+2.60%]
Total CPU difference: 5018.708170999953 / 5141.340557099954 [-2.39%]

There are no differences in images found by testdiff.
(0051629)
mkv   
2016-03-15 13:18   
Dear abv,
Branch CR27234 has been rejected due to:
- compilation errors in Products Csharp Samples and Products Java Samples
(0051631)
abv   
2016-03-15 13:43   
Fix for C# wrapper is pushed to CR27234 (in Products), please check wrapper
(0051700)
mkv   
2016-03-16 14:38   
Dear BugMaster,
Branch CR27234 from occt git-repository (and CR27234 from products git-repository) was recompiled on Linux, MacOS and Windows platforms on Release mode.
SHA-1: dd59edcf080cecebf50781e07eebfcfa13024c5e

There are not compilation errors.
(0051701)
mkv   
2016-03-16 14:38   
Dear BugMaster,
Branch CR27234 is TESTED.
(0054229)
git   
2016-05-20 12:23   
Branch CR27234 has been deleted by inv.

SHA-1: cb596cfad41b1e8919e8857a6be3e9c52edefabc