Archived issue #0022704
A method defining TKOpenGl.dll for Open CASCADE visualization toolkit not using GraphicShr variable.
Description
Different modules of an application may use different redefined TkOpenGl libraries (redefined because of usage of user-draw technology, for example). But the environment variable CSF_GraphicShr points to only one such a library...
The problem is easily solved by additional of a new constructor of Graphic3d_WNTGraphicDevice, which obtains a reference (path) to the redefined library.
The problem is easily solved by additional of a new constructor of Graphic3d_WNTGraphicDevice, which obtains a reference (path) to the redefined library.
Steps to reproduce
It requires specific environment (at least 2 modules, which use 2 different redefined TKOpenGl libraries).
Additional information
A new constructor is added to Graphic3d_WNTGraphicDevice, which obtains a path to visualization plug-in library (TKOpenGl.dll or a redefined one).
Public activity
9 archived notes
Participants are labeled by their role within this record.
Revise the bug please.
2 files were modified:
Graphic3d_WNTGraphicDevice.cdl (added a constructor and a method SetGraphicDevice())
Graphic3d_WNTGraphicDevice.cxx (implementation of added methods).
2 files were modified:
Graphic3d_WNTGraphicDevice.cdl (added a constructor and a method SetGraphicDevice())
Graphic3d_WNTGraphicDevice.cxx (implementation of added methods).
Sergey,
Fix is revised from 2011-08-18 16:58 (1 Month)
Fix is revised from 2011-08-18 16:58 (1 Month)
It would be more reasonable if Kirill or Sergey (SAN) would reviewed it.
Branch http://svn/svn/occt/branches/0022704 was reviewed.
>>void Graphic3d_WNTGraphicDevice::SetGraphicDriver (const Standard_CString graphicLib);
This function - is a code duplication. Please reuse the code - move common functionality into ::SetGraphicDriver (const Standard_CString graphicLib) method and reuse it in ::SetGraphicDriver() without arguments.
Please fix this.
>>void Graphic3d_WNTGraphicDevice::SetGraphicDriver (const Standard_CString graphicLib);
This function - is a code duplication. Please reuse the code - move common functionality into ::SetGraphicDriver (const Standard_CString graphicLib) method and reuse it in ::SetGraphicDriver() without arguments.
Please fix this.
Vlad,
Issue was reviewed with remarks 2011-09-20.
Please update branch in accoardance with remarks of KGV
Issue was reviewed with remarks 2011-09-20.
Please update branch in accoardance with remarks of KGV
The code duplication is removed. Thank you! :-)
2vro, French->English translation of comments was lost (due to initial code duplication). This issue was fixed and integrated to Bug branch with coding rules cosmetics applied.
Dear Commenter 1, please perform tests of bug branch and integrate.
Dear Commenter 1, please perform tests of bug branch and integrate.
Dear Commenter 1,
Workbench KAS:dev:aan-22704-occt from SVN branch http://svn/svn/occt/branches/0022704
(and aan-22704-products from trunk) was successfully compiled on Linux and Windows platforms, and tested on Linux and Windows platforms.
There are not regressions in aan-22704-products regarding to KAS:dev:products-20120127-opt
See results in /QADisk/occttests/results/KAS/dev/aan-22704-products_02022012/lin
See reference results in /QADisk/occttests/results/KAS/dev/products-20120127-opt_28012012/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:aan-22704-occt from SVN branch http://svn/svn/occt/branches/0022704
(and aan-22704-products from trunk) was successfully compiled on Linux and Windows platforms, and tested on Linux and Windows platforms.
There are not regressions in aan-22704-products regarding to KAS:dev:products-20120127-opt
See results in /QADisk/occttests/results/KAS/dev/aan-22704-products_02022012/lin
See reference results in /QADisk/occttests/results/KAS/dev/products-20120127-opt_28012012/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-03 15:16:44 +0400 (Fri, 03 Feb 2012)
New Revision: 10294
Modified:
trunk/src/Graphic3d/Graphic3d_WNTGraphicDevice.cdl
trunk/src/Graphic3d/Graphic3d_WNTGraphicDevice.cxx
Date: 2012-02-03 15:16:44 +0400 (Fri, 03 Feb 2012)
New Revision: 10294
Modified:
trunk/src/Graphic3d/Graphic3d_WNTGraphicDevice.cdl
trunk/src/Graphic3d/Graphic3d_WNTGraphicDevice.cxx