Archived issue #0022945
AIS_InteractiveObject: Initialization bug
Description
This is a bug reported on the OCCT forum by Paul Jimenez:
"After spending a few hours trying to track down a strange behavior, I found the reason for it. It only happens when I try to run a Delphi application that uses a DLL which makes heavy use of OCC. When it runs in C++, the strange behavior does not pop up.
In my copy of OCC 6.3, in file Attachment 1 (CXX), lines 56 to 69: myHasTransformation is NOT initialized. It should be set to Standard_False. It caused detected/selected shapes to be given a different location, thus making them impossible to track.
I modified my local copy to include the fix, and I hope you include it also in OCC's code."
"After spending a few hours trying to track down a strange behavior, I found the reason for it. It only happens when I try to run a Delphi application that uses a DLL which makes heavy use of OCC. When it runs in C++, the strange behavior does not pop up.
In my copy of OCC 6.3, in file Attachment 1 (CXX), lines 56 to 69: myHasTransformation is NOT initialized. It should be set to Standard_False. It caused detected/selected shapes to be given a different location, thus making them impossible to track.
I modified my local copy to include the fix, and I hope you include it also in OCC's code."
Public activity
7 archived notes
Participants are labeled by their role within this record.
Fix applied.
Branch http://svn/svn/occt/branches/OCC22945 is ready to be reviewed.
Dear Andrey,
Please review.
Branch http://svn/svn/occt/branches/OCC22945 is ready to be reviewed.
Dear Andrey,
Please review.
Please make sure that (a) other fields are properly initialized (I suspect myCTXPtr, myOwnMaterial, myCurrentFacingModel), and (b) they are initialized in the same order as declared in the class
Fields initialization has been added.
Branch http://svn/svn/occt/branches/OCC22945 is ready to be reviewed.
Dear Andrey,
Please review.
Branch http://svn/svn/occt/branches/OCC22945 is ready to be reviewed.
Dear Andrey,
Please review.
No remarks, please test
Dear Commenter 1,
Workbench KAS:dev:mkv-22945-occt was created from SVN branch http://svn/svn/occt/branches/OCC22945
(and mkv-22945-products from trunk) and compiled on Linux platform.
There are not regressions in mkv-22945-products regarding to KAS:dev:products-20120210-opt
See results in /QADisk/occttests/results/KAS/dev/mkv-22945-products_15022012/lin
See reference results in /QADisk/occttests/results/KAS/dev/products-20120210-opt_10022012/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
Workbench KAS:dev:mkv-22945-occt was created from SVN branch http://svn/svn/occt/branches/OCC22945
(and mkv-22945-products from trunk) and compiled on Linux platform.
There are not regressions in mkv-22945-products regarding to KAS:dev:products-20120210-opt
See results in /QADisk/occttests/results/KAS/dev/mkv-22945-products_15022012/lin
See reference results in /QADisk/occttests/results/KAS/dev/products-20120210-opt_10022012/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
Integrated into trunk of occt repository
Date: 2012-02-17 11:48:50 +0400 (Fri, 17 Feb 2012)
New Revision: 10494
Modified:
trunk/src/AIS/Attachment 1 (CXX)
Date: 2012-02-17 11:48:50 +0400 (Fri, 17 Feb 2012)
New Revision: 10494
Modified:
trunk/src/AIS/Attachment 1 (CXX)
Verified. Please close.