Mon, 12/04/2017 - 21:46
The condition at line 10 of the annexed c++ code:
10: if(nameAtt->ID().IsSame(TDataStd_Name::GetID()))
evaluates to true if the code is built with OCC 6.9.1
while it evaluates to false if the code is built with OCC 7.2.0
The second result is wrong and because of that the following function (which is defined in TDF/TDF_Label.cxx):
void TDF_Label::AddAttribute (const Handle(TDF_Attribute)& anAttribute,
const Standard_Boolean append/* = Standard_False*/) const
does not work in OCCT 7.2.0 when a Handle(TDataStd_Name) is passed to the first argument
Walter
Attachments:
Tue, 12/05/2017 - 11:40
I have the same behaviour here. Isn't the problem that the empty constructor of TDataStd_Name doesn't initialize the member fields correctly?
Tue, 12/05/2017 - 16:30
Since OCCT 7.1.0 same type attributes may have different GUIDs. Because of this the attribute constructor does not initialize GUID. See a chapter "Support of several attributes of one type at the same label" in Release Notes.
So, before AddAttribute you should call
nameAtt->SetID(TDataStd_Name::GetID())
Anyway, the standard way to create and attach attribute to a label is to use "Set" method of the attribute. It works well on any version of OCCT.
I registered an issue 29371 in the bugtracker to avoid such a problem later.
Tue, 12/05/2017 - 17:43
Hello Benjamin, thanks for your hint.
I think you are right. The problem arises because TDataStd_Name::myID is not initialized inside of the empty constructor.
However I have seen that it can be fixed by adding a call to TDataStd_Name::setID() just after the constructor:
Handle(TDataStd_Name) nameAtt=new TDataStd_Name();
nameAtt->setID();
Perhaps it would have been better if the call had been included in the constructor without breaking the compatibility with OCCT 6.9.1.
But it just is a matter of choice and not really a bug.
Walter