as pixeliterator is no longer derived from the std::iterator class ( was a bit pointless and had side effects) for some reason the for_each needs now std:for_each. It is correct but I wonder why it previously didnt need it.