QByteArray утечка памяти

[ecspertiza]

Есть такой вот незамысловатый пример

Код:

#include <QtCore/QCoreApplication>

#include <QFile>
#include <QByteArray>
#include <QThread>
#include <QByteArray>
#include <QDebug>

class Thread : public QThread
{
    Q_OBJECT

    public :
            Thread(QObject *parent = 0):QThread(parent){}

        void run()
        {
            while (true)
            {
                check_queue();
                usleep(50);
            }
        }

        void addQueue(QByteArray * data)
        {
            listByte << data;
        }

    private :
        QList<QByteArray *> listByte;

        void check_queue()
        {
            if (!listByte.isEmpty())
            {
                if (listByte[0] != NULL)
                {
                    delete listByte[0];
                    listByte[0] = NULL;
                }
            }
        }
};

int main(int argc, char *argv[])
{
    QCoreApplication a(argc, argv);

    int indexFile = 1;

    QString filePath = "D:/MyDocument/Picture/camera/%1.jpg";

    Thread *thread = new Thread();
    thread->start();

    while (true)
    {
        qDebug() << "open file" << filePath.arg(indexFile);
        QFile file(filePath.arg(indexFile));
        file.open(QIODevice::ReadOnly);
        QByteArray *data = new QByteArray(file.readAll());
        thread->addQueue(data);
        file.close();

        ++indexFile;
        if (indexFile == 10)
            indexFile = 1;
    }

    return 0;
}

#include "main.moc"

насколько могу понять, здесь память утекает в QByteArray , но не могу понять почему. Ведь вроде как передается он по указателю ,а потом удаляется. Или я что то упускаю ?

[Пантер]

delete listByte.takeFirst ();

[ecspertiza]

да, спасибо, так гораздо лучше

[SimpleSunny]

1.Также возможна ошибка с гонкой за listByte, если функция "просто вызывается".

Код

C++ (Qt)
Thread t;
...
t.addQueue(b);

В данном случае надо использовать или слот, соединив его с соответствующим сигналом,  или invokeMethod(Qt::QueuedConnection), или Mutex.

2. Может вместо if (!listByte.isEmpty()) подразумевалось while (!listByte.isEmpty()).

[ecspertiza]

1. У меня в оригинальной софтине, слот, здесь просто пример на скорую руку, так сказать 
2. Да while, торопился

Что больше всего мне было непонятно в оригинальной софтине я для удаления из списка объекта использовал removeAt(0) ,ну и перед этим delete конечно, но память почему то утекала, не понимаю почему, но с takeFirst() работает хорошо.

[Igors]

Синхронизацией пока и не пахло (наверное отложил на потом)

[SimpleSunny]

Кстати, так как, сам обьект Thread живет не в своем собственном потоке, а там где он создан, то и слоты будут выполняться в контексте потока в котором он создан. Чтобы слоты выполнялись в потоке Thread, надо его вручную туда переместить.

[ecspertiza]

стало быть в ф-ции run() выполнить moveThread(this) ? я правильно понимаю ? про слоты не знал, спасибо

[SimpleSunny]

В конструкторе
Thread(QObject *parent = 0):QThread(parent){ moveToThread(this); }
Но насколько помню, если задан parent, то конструкция не отработает как ожидается.
На форуме было обсуждение использовать\не_использовать moveToThread(this);
UPD. http://www.prog.org.ru/topic_17090_0.html