Bug #282
IsPicked missing const
| Status: | Approved | Start date: | 05/07/2012 | |
|---|---|---|---|---|
| Priority: | Immediate | Due date: | ||
| Assignee: | Jan Dolinár | % Done: | 0% | |
| Category: | Core | Spent time: | - | |
| Target version: | - |
Description
IsPicked() method in BiVector is missing "const". GCC 4.7.0 complains about it - 4.6.x didn't for some reason :-)
In file included from /home/h/upp-production/uppsrc/Core/Core.h:273:0,
from /home/h/upp-production/uppsrc/Draw/Draw.h:6,
from /home/h/upp-production/uppsrc/RichText/RichText.h:4,
from /home/h/upp-production/uppsrc/CtrlCore/CtrlCore.h:4,
from /home/h/upp-production/uppsrc/CtrlLib/CtrlLib.h:4,
from /home/h/upp-production/uppsrc/CtrlLib/CtrlLib.icpp:1:
/home/h/upp-production/uppsrc/Core/BiCont.h: In member function ‘bool Upp::BiArray<T>::IsPicked() const’:
/home/h/upp-production/uppsrc/Core/BiCont.h:122:45: error: no matching function for call to ‘Upp::BiVector<void*>::IsPicked() const’
/home/h/upp-production/uppsrc/Core/BiCont.h:122:45: note: candidate is:
In file included from /home/h/upp-production/uppsrc/Core/Core.h:273:0,
from /home/h/upp-production/uppsrc/Draw/Draw.h:6,
from /home/h/upp-production/uppsrc/RichText/RichText.h:4,
from /home/h/upp-production/uppsrc/CtrlCore/CtrlCore.h:4,
from /home/h/upp-production/uppsrc/CtrlLib/CtrlLib.h:4,
from /home/h/upp-production/uppsrc/CtrlLib/CtrlLib.icpp:1:
/home/h/upp-production/uppsrc/Core/BiCont.h:51:7: note: bool Upp::BiVector<T>::IsPicked() [with T = void*] <near match>
/home/h/upp-production/uppsrc/Core/BiCont.h:51:7: note: no known conversion for implicit ‘this’ parameter from ‘const Upp::BiVector<void*>*’ to ‘Upp::BiVector<void*>*’
History
#1 Updated by Jan Dolinár over 13 years ago
- File BiCont.patch
added
Seems like I forgot to attach the file...
#2 Updated by Miroslav Fidler over 13 years ago
- Priority changed from High to Immediate
#3 Updated by Miroslav Fidler over 13 years ago
- Status changed from Patch ready to Ready for QA
- Assignee set to Jan Dolinár
#4 Updated by Jan Dolinár over 13 years ago
- Status changed from Ready for QA to Approved
Works well, thank you!