MantisBT - Community
View Issue Details
0023285Community[OCCT] OCCT:Modeling Datapublic2012-07-16 14:282012-11-16 13:17
Pawel 
Pawel 
normalmajor 
closedfixed 
WindowsVC++ 200832 bit
[OCCT] 6.5.3 
[OCCT] 6.5.4[OCCT] 6.5.4 
Not needed
0023285: Ignoring Bezier curves
In GeomLib_Tool.cxx, in lines 298 and 548, Geom(2d)_BSplineCurve is specified twice. Geom(2d)_BezierCurve is not specified at all. Prbably a copy-paste bug.
No tags attached.
Issue History
2012-07-16 14:28PawelNew Issue
2012-07-16 14:28PawelAssigned To => Pawel
2012-07-16 14:35PawelNote Added: 0020986
2012-07-16 14:35PawelAssigned ToPawel => bugmaster
2012-07-16 14:35PawelStatusnew => resolved
2012-07-16 18:20abvNote Added: 0020994
2012-07-16 18:20abvStatusresolved => reviewed
2012-07-16 18:43mkvAssigned Tobugmaster => mkv
2012-07-19 11:27mkvNote Added: 0021059
2012-07-19 11:27mkvTest case number => Not needed
2012-07-19 11:27mkvAssigned Tomkv => bugmaster
2012-07-19 11:27mkvStatusreviewed => tested
2012-07-23 13:22abvChangeset attached => occt master 0564092b
2012-07-23 14:39bugmasterChangeset removedocct master 0564092b =>
2012-07-23 14:40bugmasterChangeset attached => occt master 0564092b
2012-07-23 14:42bugmasterStatustested => verified
2012-07-23 14:42bugmasterResolutionopen => fixed
2012-07-23 14:42bugmasterAssigned Tobugmaster => Pawel
2012-07-23 16:22PawelAssigned ToPawel => bugmaster
2012-07-23 16:22PawelNote Added: 0021120
2012-07-23 16:22PawelStatusverified => feedback
2012-07-23 16:22PawelResolutionfixed => reopened
2012-07-23 16:31abvNote Added: 0021123
2012-07-23 16:37PawelNote Added: 0021124
2012-07-23 16:37PawelStatusfeedback => assigned
2012-07-23 16:39PawelNote Added: 0021125
2012-07-23 16:39PawelStatusassigned => resolved
2012-07-23 17:08bugmasterStatusresolved => reviewed
2012-07-23 17:09bugmasterStatusreviewed => verified
2012-07-23 17:09bugmasterResolutionreopened => fixed
2012-07-23 17:09bugmasterAssigned Tobugmaster => Pawel
2012-11-16 13:14bugmasterFixed in Version => 6.5.4
2012-11-16 13:17bugmasterStatusverified => closed

Notes
(0020986)
Pawel   
2012-07-16 14:35   
Corresponding git branch pushed. Please review.
(0020994)
abv   
2012-07-16 18:20   
No remarks, please test
(0021059)
mkv   
2012-07-19 11:27   
Dear BugMaster,
The workbenches
KAS:dev:mkv-23285-occt (GIT branch CR23285)
KAS:dev:mkv-23285-products (GIT master)
were compiled on Linux platform and tested.

Regression:
Not detected

Improvements:
Not detected

Testing case:
Not needed

See results in /QADisk/occttests/results/KAS/dev/mkv-23285-products_17072012/lin
See reference results in /QADisk/occttests/results/KAS/dev/products-20120706-opt_06072012/lin
See test cases in /QADisk/occttests/tests/ED
N.B. In order to launch testing case you can make use the following instructions
http://doc/doku.php?id=occt:certification [^]
(0021120)
Pawel   
2012-07-23 16:22   
Hello Andrey,

the fix for this issue was integrated under wrong name (0023287: IntPolyh_MaillageAffinage.cxx, line 2217: "identical sub-expressions to the left and to the right of the '-' operator") into master.

Please correct if possible.
(0021123)
abv   
2012-07-23 16:31   
Hello Pawel,

Sorry for this mistake!

We have noticed the problem, but decided not to revert the commit, as it would cause troubles to all people who have already checked out the updated master branch. We have corrected the information in Mantis (you should see the link to correct commit below), thus the inconsistency remains only in the Git history.

Hope this should not be a big problem.
(0021124)
Pawel   
2012-07-23 16:37   
Hello Andrey,

no problem. Thanks for clarifying.

BTW, it's a pity git does not offer the possibility to change the commit text only (like Subversion does).
(0021125)
Pawel   
2012-07-23 16:39   
Please change the status back to 'verified'.