Archived issue #0025391
Graphic3d_MaterialAspect: getting/setting color discrepancy.
Description
Graphic3d_MaterialAspect::SetColor stores the provided color in 'myDiffuseColor' variable and modifes 'myAmbientColor' using a hard-coded coefficient.
Graphic3d_MaterialAspect::Color() returns the value of 'myAmbientColor'.
If an application relies on the combination of SetColor() / Color() calls undesired effects would occur therefore I suggest to return 'myDiffuseColor' in Graphic3d_MaterialAspect::Color().
Graphic3d_MaterialAspect::Color() returns the value of 'myAmbientColor'.
If an application relies on the combination of SetColor() / Color() calls undesired effects would occur therefore I suggest to return 'myDiffuseColor' in Graphic3d_MaterialAspect::Color().
Public activity
5 archived notes
Participants are labeled by their role within this record.
Branch [archived branch] has been created by Author.
[revision removed]
Detailed log of new commits:
Author: Author
Date: Fri Oct 17 13:55:16 2014 +0200
Graphic3d_MaterialAspect::Color() returns the value of 'myDiffuseColor' in order to be consistent with Graphic3d_MaterialAspect::SetColor.
[revision removed]
Detailed log of new commits:
Author: Author
Date: Fri Oct 17 13:55:16 2014 +0200
Graphic3d_MaterialAspect::Color() returns the value of 'myDiffuseColor' in order to be consistent with Graphic3d_MaterialAspect::SetColor.
CR25391 has been pushed. Please review.
Dear Author,
the issue is duplicate of #0025349, which has already waiting for testing the similar patch.
the issue is duplicate of #0025349, which has already waiting for testing the similar patch.
Dear Commenter 1,
please close the issue as duplicate.
please close the issue as duplicate.
Branch [archived branch] has been deleted by Participant.
[revision removed]
[revision removed]
Related records