[fixed in 1.8 & 1.7.4]list::Iterator compile bug in 1.7.3?

[wqking]

Code: Select all

 
    class Iterator
    {
        Iterator& operator +=(s32 num)
        {
            if(num > 0)
            {
                while (num-- && this->Current != 0) ++(*this);
            }
            else
            {
                while(num++ && this->Current != 0) --(*this);
            }
            return *this;
        }
 
        Iterator& operator -=(s32 num) const { return (*this)+=(-num); }
    };
 
 

Above is a snippet of list::Iterator.
Note the -= operator is a const operator, but it invokes the non-const += and failed compile in VC.
There is no other overloading for +=

Is this a bug? Did it fix in 1.8.0?

[CuteAlien]

Two times yes - it is a bug and it is fixed in Irrlicht 1.8.

[hybrid]

'I guess it's even fixed for 1.7.4!?

[CuteAlien]

No, it's not. I suspect maybe because it would change the interface? Or simply someone was lazy? Don't know right now ... is changing const ok for minor library versions or will that already mess up linking? I wouldn't expect so.

[hybrid]

Well, this method wouldn't link anyway, so fixing should be ok.

[CuteAlien]

Ok, I've backported it to 1.7