Archived issue #0023040
Annoying warnings in NCollection_Vector
Description
Compiler notify about wrong initialization order of class members in constructors:
> NCollection_BaseVector.hxx: In constructor
> 'NCollection_BaseVector::MemBlock::MemBlock(NCollection_BaseAllocator*)':
> 'NCollection_BaseVector::MemBlock::myAlloc' will be initialized after
> 'Standard_Integer NCollection_BaseVector::MemBlock::myFirstInd'
This is regression after #22143
This is suggested either to fix constructors or members order in class definition.
> NCollection_BaseVector.hxx: In constructor
> 'NCollection_BaseVector::MemBlock::MemBlock(NCollection_BaseAllocator*)':
> 'NCollection_BaseVector::MemBlock::myAlloc' will be initialized after
> 'Standard_Integer NCollection_BaseVector::MemBlock::myFirstInd'
This is regression after #22143
This is suggested either to fix constructors or members order in class definition.
Public activity
4 archived notes
Participants are labeled by their role within this record.
I suppose the problem could be reproduced with GCC compiler...
Dear abv, please review the patch in branch CR23040.
No remarks, please test
Dear Commenter 1,
The workbenches
KAS:dev:mkv-23040-occt (GIT branch CR23040)
KAS:dev:mkv-23040-products (GIT master)
were compiled on Linux platform and tested.
There are not warnings in NCollection_BaseVector during compilation on Linux.
Regression:
Not detected
Improvements:
Not detected
Testing case:
Not needed
See results in /QADisk/occttests/results/KAS/dev/mkv-23040-products_13062012/lin
See reference results in /QADisk/occttests/results/KAS/dev/products-20120601-opt_02062012/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
The workbenches
KAS:dev:mkv-23040-occt (GIT branch CR23040)
KAS:dev:mkv-23040-products (GIT master)
were compiled on Linux platform and tested.
There are not warnings in NCollection_BaseVector during compilation on Linux.
Regression:
Not detected
Improvements:
Not detected
Testing case:
Not needed
See results in /QADisk/occttests/results/KAS/dev/mkv-23040-products_13062012/lin
See reference results in /QADisk/occttests/results/KAS/dev/products-20120601-opt_02062012/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