Fix a potential crash in imagecolors

Unlike std::vector, erase an iterator can invalidate the iterator. QList may optionally move the data away from beginning.

See: https://github.com/qt/qtbase/blob/40143c189b7c1bf3c2058b77d00ea5c4e3be8b28/src/corelib/tools/qlist.cpp#L295