MantisBT - Open CASCADE
View Issue Details
0026628Open CASCADE[OCCT] OCCT:Samplespublic2015-09-02 12:292015-10-23 20:49
apn 
bugmaster 
normalminor 
closedfixed 
WindowsVC++ 201064 bit
 
[OCCT] 6.9.1[OCCT] 6.9.1 
Not needed
0026628: Button Erase (Delete where erase algorythm is used) works incorrect
There are two types of behavior of Erase in samples:
- crash trying to erase one highlight object;
- erase only first selected object trying to erase several shapes.
It caused by adding AddOrRemoveCurrentObject in function EraseGlobal in AIS_InteractiveContext.cxx.
Samples crashes on function NextCurrent which is called by OnObjectErase in OCC_3dBaseDoc.cpp.
To avoid this problem it's possible to change NextCurrent to InitCurrent in function erase in all samples or add some check in function NextCurrent in AIS_InteractiveContext_1.cxx.
Run mfc sample (e.g. modeling), build few shapes, select one or several objects and try to erase.
No tags attached.
related to 0026153closed bugmaster Draw Harness, ViewerTest - "verase" does not remove selection highlight for shaded object 
related to 0026726closed bugmaster Draw Harness, ViewerTest - AIS_InteractiveContext::EraseSelected() lacks test case 
Issue History
2015-09-02 12:29apnNew Issue
2015-09-02 12:29apnAssigned To => abv
2015-09-02 12:29apnStatusnew => assigned
2015-09-02 12:29apnRelationship addedrelated to 0026153
2015-09-02 13:12abvAssigned Toabv => kgv
2015-09-02 13:39kgvAssigned Tokgv => isk
2015-09-02 13:39kgvTarget Version => 7.0.0
2015-09-02 14:21abvTarget Version7.0.0 => 6.9.1
2015-09-02 16:34gitNote Added: 0044969
2015-09-03 11:01gitNote Added: 0044990
2015-09-03 11:33gitNote Added: 0044993
2015-09-04 12:09iskNote Added: 0045043
2015-09-04 12:09iskAssigned Toisk => kgv
2015-09-04 12:09iskStatusassigned => resolved
2015-09-04 13:07kgvNote Added: 0045053
2015-09-04 13:07kgvAssigned Tokgv => bugmaster
2015-09-04 13:07kgvStatusresolved => reviewed
2015-09-04 13:07kgvProduct Version6.9.1 =>
2015-09-04 13:29mkvAssigned Tobugmaster => mkv
2015-09-04 14:18gitNote Added: 0045065
2015-09-07 19:10mkvNote Added: 0045185
2015-09-07 19:10mkvNote Added: 0045186
2015-09-10 15:07mkvNote Added: 0045423
2015-09-10 15:07mkvNote Added: 0045424
2015-09-10 15:07mkvAssigned Tomkv => bugmaster
2015-09-10 15:07mkvStatusreviewed => tested
2015-09-10 15:07mkvTest case number => Not needed
2015-09-11 14:09bugmasterChangeset attached => occt master 7140edaf
2015-09-11 14:09bugmasterStatustested => verified
2015-09-11 14:09bugmasterResolutionopen => fixed
2015-09-28 12:45vpaRelationship addedrelated to 0026726
2015-10-16 14:55aivStatusverified => closed
2015-10-16 16:37gitNote Added: 0047008
2015-10-23 20:49aivFixed in Version => 6.9.1

Notes
(0044969)
git   
2015-09-02 16:34   
Branch CR26628 has been created by isk.

SHA-1: 6c2073897813f4eedc9f464beb71f8f4bdf70602


Detailed log of new commits:

Author: isk
Date: Wed Sep 2 16:33:58 2015 +0300

    0026628: Button Erase (Delete where erase algorythm is used) works incorrect
    Update AIS_InteractiveContext::EraseSelected() method.
(0044990)
git   
2015-09-03 11:01   
Branch CR26628 has been updated by isk.

SHA-1: 0805d1a48f2716ea7ed38eeefec282bc6ff0d8d6


Detailed log of new commits:

Author: isk
Date: Thu Sep 3 11:01:19 2015 +0300

    Fix HLR sample

(0044993)
git   
2015-09-03 11:33   
Branch CR26628 has been updated by isk.

SHA-1: a8f281c77df2ea14a8bbc80f1a6ed0fd6e335045


Detailed log of new commits:

Author: isk
Date: Thu Sep 3 11:33:32 2015 +0300

    Fix samples.

(0045043)
isk   
2015-09-04 12:09   
Dear Kirill,
please review the branch CR26628.
(0045053)
kgv   
2015-09-04 13:07   
Please test the patch and samples.
(0045065)
git   
2015-09-04 14:18   
Branch CR26628 has been updated forcibly by mkv.

SHA-1: b8662487cf53ddaa5ad180eb7c00fc00899e3f2e
(0045185)
mkv   
2015-09-07 19:10   
Dear BugMaster,
Branch CR26628 was rebased on current master of occt git-repository.
SHA-1: b8662487cf53ddaa5ad180eb7c00fc00899e3f2e
(0045186)
mkv   
2015-09-07 19:10   
Dear BugMaster,
Branch CR26628 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: b8662487cf53ddaa5ad180eb7c00fc00899e3f2e

Number of compiler warnings:

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

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

There is new additional compilation warning on Linux platform:
There are compilation errors in Qt samples products on windows platform:

Regressions/Differences/Improvements:
No regressions/differences

Testing cases:
Not needed

Testing on Linux:
occt component :
Total MEMORY difference: 92330512 / 91937745 [+0.43%]
Total CPU difference: 17797.78999999896 / 17713.55999999897 [+0.48%]
products component :
Total MEMORY difference: 25227524 / 25220029 [+0.03%]
Total CPU difference: 7072.640000000005 / 6936.800000000004 [+1.96%]

Testing on Windows:
occt component :
Total MEMORY difference: 57108857 / 57101025 [+0.01%]
Total CPU difference: 16456.919892399328 / 16228.628028999388 [+1.41%]
products component :
Total MEMORY difference: 16219687 / 16219655 [+0.00%]
Total CPU difference: 5411.159886699978 / 5397.61899989997 [+0.25%]

There are no differences in images found by testdiff.
(0045423)
mkv   
2015-09-10 15:07   
Dear BugMaster,
Erasing one or several objects in MFC Modeling sample was tested, it work correct.
(0045424)
mkv   
2015-09-10 15:07   
Dear BugMaster,
Branch CR26628 is TESTED.
(0047008)
git   
2015-10-16 16:37   
Branch CR26628 has been deleted by kgv.

SHA-1: b8662487cf53ddaa5ad180eb7c00fc00899e3f2e