QImage: Как правильно уничтожить указатель на данные пикселей?

Вы можете удалить данные, когда QImage будет уничтожен; это, как показано выше @JeremyFriesner, можно просто сделать, определив экземпляр QImage во внутренней области и выделив/освободив данные вне его.

Однако зачем возиться со всей этой работой? Этот конструктор QImage предназначен для сложных случаев использования, когда у вас уже есть пиксельные данные из какого-то другого источника или вам нужно сохранить их на «странное» время жизни.

Вместо этого ваш случай намного проще, данные пикселей имеют точно такое же время жизни, что и QImage, поэтому лучше позволить ему выделять память для пикселей самостоятельно; таким образом вы можете быть уверены, что он имеет правильный размер и выровнен, и что он принадлежит/автоматически уничтожается экземпляром QImage; после создания объекта QImage вы можете получить указатель на данные пикселя, используя метод bits, и делать с ним все, что захотите.

Таким образом, ваш код просто станет:

void updateDisplay() {
    QImage image = QImage(width, height,
                          width, QImage::Format_Indexed8);
    uchar *data = image.bits();
    ... 

    pixmapItem->setPixmap(QPixmap::fromImage(image));
}

проще, безопаснее и с нулевым риском утечки памяти.

Из документации Qt

Создает изображение с заданной шириной, высотой и форматом, которое использует существующий буфер памяти data. Ширина и высота должны быть указаны в пикселях, данные должны быть выровнены по 32 битам, и каждая строка развертки данных в изображении также должна быть выровнена по 32 битам.

Буфер должен оставаться действительным в течение всего срока службы QImage. Образ не удаляет буфер при уничтожении.

Вам нужно реализовать свой собственный способ удаления этого буфера

Таким образом, вам нужно удалить data самостоятельно, но, конечно, хитрость заключается в том, чтобы не удалять его слишком рано — в частности, вы не хотите удалять его, пока объект QImage все еще может его использовать. Самый простой способ убедиться, что в вашем случае это удалить только после уничтожения объекта QImage:

void updateDisplay() {
   uchar *data = new ...; // array of pixel data
   ...
   {
      QImage image = QImage(data, width, height,
                      width, QImage::Format_Indexed8);

      pixmapItem->setPixmap(QPixmap::fromImage(image));
   }
   delete [] data;
}

(обратите внимание на использование фигурных скобок для создания внутренней области видимости и тем самым убедитесь, что объект QImage извлекается из стека и уничтожается до выполнения строки delete [] data!)

Конечно, более простым и безопасным подходом было бы вообще избежать вопроса о том, когда удалять, никогда не выделяя массив вручную. Вместо этого позвольте объекту QImage выделить свой собственный массив данных и вместо этого просто напишите в него:

QImage image(width, height, QImage::Format_Indexed8);
char * data = image.bits();
// write into (data) here if you want to, but don't delete [] data, ever!  
// instead, the QImage destructor will do any necessary deletes for you

Вы можете создать подкласс QImage со своим собственным деструктором, который выполняет очистку.

class NativeBufferQImage : public QImage {
    uchar *data;

    // other stuff

public:
    ~NativeBufferQImage() {
        delete[] this->data; 
    }
};

Также обратите внимание, что поскольку QImage является производным от QPaintDevice, а QPaintDevice имеет виртуальный деструктор, поэтому, если кто-то delete ваш NativeBufferQImage указателем на его базовый класс (т.е. QImage), ваш деструктор тоже будет вызван.

Лучший способ сделать это — не делать этого:

void updateDisplay() {
    QImage image(width, height, QImage::Format_Indexed8);
    auto const data = image.bits();
    // you can modify the data here
    pixmapItem->setPixmap(QPixmap::fromImage(image));
}