enum всё же изначально ориентирован на создание перечислений, а не набора флагов. Членов enum с именами вида BufferSizeAndScaleChanged не нашёл, зато вижу

stateFlags.check(ViewportIsScaled | ViewportIsCropped)

и

            stateFlags.remove(ViewportIsCropped);
            stateFlags.remove(ViewportIsScaled);

так что вроде подходит и bit-field вместо "голого" enum, если уж так хочется сэкономить место под bool.

С одной стороны автор пишет непонятно зачем обёртку, которая вызовет недоумение сишников, и вот такие геттеры-сеттеры:

                    size.setW(srcRect.w());      // эти два где-то вызываются раздельно?
                    size.setH(srcRect.h());      //
                    stateFlags.add(ViewportIsScaled);

а с другой стороны повсюду голые указатели. Например, мне не хочется вникать, уместны ли были там смартпоинтеры и зачем макрос LPRIVATE_CLASS меняет семантику class на struct (первой строкой там public:). По-моему, плюсы создавались с целью как раз вот такое свести к минимуму. Получается - вот  пример, почему C++ вызывает неодобрение, тогда как дело не в языке.