MantisBT - Community
View Issue Details
0025501Community[OCCT] OCCT:Application Frameworkpublic2014-11-19 18:122015-05-14 15:32
Vico Liang 
bugmaster 
normalminor 
closedfixed 
WindowsVC++ 201364 bit
[OCCT] 6.8.0 
[OCCT] 6.9.0[OCCT] 6.9.0 
Not needed
0025501: TNaming::Displace calls itself recursively with default parameter.
void TNaming::Displace (const TDF_Label& L,
            const TopLoc_Location& Loc,
            const Standard_Boolean WithOld)
{
  for (TDF_ChildIterator ciL(L); ciL.More(); ciL.Next()) {
    Displace (ciL.Value(),Loc); // BUG: the recursive call might need the parameter WithOld e.g. Displace (ciL.Value(),Loc, WithOld);
  }
}
Fixed in branch CR25501.
Test case - NA.
No tags attached.
Issue History
2014-11-19 18:12Vico LiangNew Issue
2014-11-19 18:12Vico LiangAssigned To => szy
2015-03-02 15:53szyNote Added: 0038001
2015-03-02 15:53szyAssigned Toszy => Vico Liang
2015-03-02 15:53szyStatusnew => feedback
2015-03-02 16:22Vico LiangNote Added: 0038008
2015-03-02 16:23Vico LiangAssigned ToVico Liang => szy
2015-03-02 16:23Vico LiangStatusfeedback => assigned
2015-03-04 12:35szyStatusassigned => resolved
2015-03-04 12:35szySteps to Reproduce Updatedbug_revision_view_page.php?rev_id=9515#r9515
2015-03-04 16:03gitNote Added: 0038085
2015-03-04 16:14szyNote Added: 0038088
2015-03-04 16:14szyAssigned Toszy => mkv
2015-03-04 16:14szyStatusresolved => reviewed
2015-03-04 16:15szySteps to Reproduce Updatedbug_revision_view_page.php?rev_id=9527#r9527
2015-03-05 13:02mkvNote Added: 0038137
2015-03-05 13:02mkvAssigned Tomkv => bugmaster
2015-03-05 13:02mkvStatusreviewed => tested
2015-03-05 13:02mkvTest case number => Not needed
2015-03-06 15:13bugmasterChangeset attached => occt master 076b125a
2015-03-06 15:13bugmasterStatustested => verified
2015-03-06 15:13bugmasterResolutionopen => fixed
2015-03-10 12:59bugmasterTarget Version7.0.0 => 6.9.0
2015-03-18 13:33gitNote Added: 0038515
2015-05-14 15:29aivStatusverified => closed
2015-05-14 15:32aivFixed in Version => 6.9.0

Notes
(0038001)
szy   
2015-03-02 15:53   
Could you attach a testing script allowing to reproduce the problem.
Thanks.
(0038008)
Vico Liang   
2015-03-02 16:22   
Hi szy,

This bug is very easy to find by static code analysis. I have attached the fix in the comment.

There is a correct version in OCCT QADraw package.
opencascade-6.8.0\src\QANewBRepNaming\QANewBRepNaming.cxx

void QANewBRepNaming::Displace (const TDF_Label& theLabel,
                        const TopLoc_Location& theLoc,
                        const Standard_Boolean theWithOld)
(0038085)
git   
2015-03-04 16:03   
Branch CR25501 has been created by szy.

SHA-1: cf082e89a998303078a814c183cf2b2f7ed30aae


Detailed log of new commits:

Author: szy
Date: Wed Mar 4 14:58:37 2015 +0300

    Patch to fix misprint in method Displace ().
(0038088)
szy   
2015-03-04 16:14   
Reviewed.
(0038137)
mkv   
2015-03-05 13:02   
Dear BugMaster,
Branch CR25501 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: cf082e89a998303078a814c183cf2b2f7ed30aae

Number of compiler warnings:

occt component :
Linux: 18 (18 on master)
Windows: 2 (2 on master)

products component :
Linux: 11 (11 on master)
Windows: 4 (4 on master)

Regressions/Differences:
No regressions/differences

Testing cases:
Not needed

Testing on Linux:
occt component :
Total MEMORY difference: 91907337 / 91875321
Total CPU difference: 59044.25000000029 / 59518.86999999998
products component :
Total MEMORY difference: 23443722 / 23383466
Total CPU difference: 16318.189999999973 / 16551.239999999983

Testing on Windows:
occt component :
Total MEMORY difference: 57946261 / 57944267
Total CPU difference: 30464.90625 / 38926.375
products component :
Total MEMORY difference: 16200522 / 16209637
Total CPU difference: 9786.90625 / 9535.171875

There are no differences in images found by testdiff.
(0038515)
git   
2015-03-18 13:33   
Branch CR25501 has been deleted by inv.

SHA-1: cf082e89a998303078a814c183cf2b2f7ed30aae