Bug #282

IsPicked missing const

Added by Jan Dolinár almost 12 years ago. Updated almost 12 years ago.

Status:ApprovedStart date:05/07/2012
Priority:ImmediateDue date:
Assignee:Jan Dolinár% Done:

0%

Category:CoreSpent 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*>*’

BiCont.patch Magnifier (548 Bytes) Jan Dolinár, 05/07/2012 08:38 PM

History

#1 Updated by Jan Dolinár almost 12 years ago

Seems like I forgot to attach the file...

#2 Updated by Miroslav Fidler almost 12 years ago

  • Priority changed from High to Immediate

#3 Updated by Miroslav Fidler almost 12 years ago

  • Status changed from Patch ready to Ready for QA
  • Assignee set to Jan Dolinár

#4 Updated by Jan Dolinár almost 12 years ago

  • Status changed from Ready for QA to Approved

Works well, thank you!

Also available in: Atom PDF