Russian Qt Forum

Просто запускаю/приостанавливаю поток но память увеличивается, надо удалять поток?

надо ли удалять объект, чтобы освободить память?

Тонкий намек? ;)

красноглазик, выложи, плиз, код, как ты с потоком работаешь. старт, пауза, удаление.

Та ничего особого, просто в функции run() запускаю процес отрисовки QPainter-ом на QImage текст + картинки. Пока даже до паузи недошел, просто стартую поток когда поступает новая задачка MyThread->start(). В итоге получаю +2МБ за отрисовку в оперативную память и таким образом она засоряется... :(

C++ (Qt)
void PdfPreview::run(){
    while(!obj->pageList.isEmpty()){
        if(!m_stopped){
            obj->displayPage(obj->pageList.at(0),m_dpi);
            obj->pageList.removeAt(0);
        }//if m_stoped
        else{break;}
        }//while
   }//run

Методом тика нашел что вот эта зараза во всем виновата:

C++ (Qt)
if ( fsize > 0){m_painter-> drawText (QPointF(x1,y1), str );}

Причем если нарисовать сразу 10 QImage (страниц) то все в порядке, а если по одной то получаю +2МБ в оперативную память :(

Что делать с m_painter-> drawText ?

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

При каждом последующем вызове потока соответственно будет утекать память.

PS код пжалста предъявите, гражданин.

deep copy QImage?

Почему это?

Помоему он не стопит поток, а просто каждый раз вызывает новый, хотя и не видел кода, но предполагаю (что он вызывает каждый раз все заного и заного).

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

Помоему рождается новый цикл, который соответственно работает параллельно со старым.

Уж не буду ручаться за эти рассуждения, но эта версия вкупе с цитатой кажется наиболее вероятным развитием хода его программы.

Таким образов вызывая и вызывая функцию он и засоряет память кучей циклов со своими элементами.

PS поправьте меня, если ошибаюсь.

To andrew.k:
Память будет не теряться(ибо можно остановить эти циклы), но соответственно каждая новая копия требует снова и снова ресурсов/мощности. И получается так называемая утечка памяти. Возможно и контролируемая, но утечка.

Ты ошибаешься. Если поток уже работает, то метод start() ничего не делает.
Смотри ассистент.

Кхм. повторюсь тогда (просил поправить - поправили, спасибо :) )


Почитал, действительно старт ничего не сделает, но тогда как ув.Ubuntu_linux стартует поток? :)

К сожалению мой мозг уже заражен идеей дублирования потоков и предлагает вариант - он все же создает новые и новые потоки и стартует их :)

PS ну код тут нужен, а спорить можно до посинения :)

Ясно, что по отдельному экземпляру на отдельный поток. Иначе бы у него просто не было потоков.
Так же ясно, что удалять поток не нужно после завершения, а можно запустить повторно.
У него проблема в другом, это уже выяснено и обсуждается в другой теме.

 И в той и в этой ни 1 куска кода :) Он что тренирует интуицию у вас? :)

Проблема решена!
Я не делал новый поток каждый раз, а просто стартовал поток, после того как он завершыт свое дело. А проблема оказалась в повторяющемся вызове старта до завершения, как я полагаю.
Код приведен пару постов выше...

В этом не могло быть проблемы.
О чем я писал выше.
Повторный вызов старт ничего не делает.

"If the thread is already running, this function does nothing."
QThread (http://doc.qt.nokia.com/latest/qthread.html#start)

Так у меня хитрая система и может в список отображения добавиться уже существующая страничка pageList.append(page);

Какая хитрая? ты потоки через QThread делал? значит start работает как описано.

Страничка в список может добавиться в любой момент вне потока.

Не понятно, как это связано с запуском потока.

Я тоже не понял до конца.
 И вот проблема вернулась!
Картинки ресует нормально но при рендере текста память засерается опять.

Аррр!!! *рвет поседевшие от Ubuntu_linux волосы*

КОООД!
МАЯ НУЖНА КОООД!

скинь ты блин файл, полностью, где ты загружаешь текст/отрисовываешь/создаешь потоки.

Ну если жалко - так и скажи, тебе ж люди популярно объяснять пытаются, а ты кусок кода выложить ленишься.

Код - это когда я могу его скопировать в *.cpp, добавить переменные в *.h  т скомпилировать и таки чтоб все заработало, а не кусок бесполезного бессвязного текста.

У тебя блин все плохо, но код то покажи, тут люди разумные сидят - посмотрят, проверят и скажут тебе, что и как, баг или так, ты сделал или Qt подсобила.

ХЛЕБА И КОД НАРОДУ!!! ;D

Извините за снобизм, сам безграмотный, но очень режет глаза.

То еще цветочки. Один раз этот автор написал
Я никак не мог сообразить что "пробивать"-то? И только потом дошло - это "попробывать" с перепутанным "ы". А ведь вполне грамотный, культурный человек - просто дурачится. Поскольку С++ и русский - практически одно и то же, то подозреваю что поэтому исходники выложить стесняется. В общем, стал жертвой собственного остроумия  :)
 

http://lurkmore.ru/Grammar_nazi x2 *FUUUUUUUUU*

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

ты не путай жопу с пальцем.
мелкие ошибки бывают, от усталости, от новнимательности и других факторов.
"а кагда чиловек делает невозможные ошыбки." это другое.

// Извините за снобизм, сам безграмотный, но очень режет глаза.

меня тошнит от этой фразы. уверен на 99% - это лицемерие в чистом виде.

если бы эт был форум задротов грамотности, тогда бы это было уместно. а так лицемерие, ИМХО. а от лицемеров с их "принципами" блювать хочется  :D

// пипл таких не любит
а чо эт ты о себе во множественном числе заговорил?  :D

заканчивай.

Ну поймите, кода очень много... где-то на 5-10 минут компиляции и там много ненужного хлама...
Я вот думаю, у меня на каждую страничку свой qimage на котором рисует пейнтер, мож в этом
загвоздка с drawText ?

Загвоздка в том, что ты задаешь слишком неопределенные вопросы без подкрепления их участками кода.

может те QtConcurrent лучше заюзать, кстати говоря.

C++ (Qt)
#include <QtGui/QApplication>
#include <QFuture>
#include <QtConcurrentRun>
 
void hello()
{
    qDebug() << "Hello from thread " << QThread::currentThread();
}
 
int main(int argc, char *argv[])
{
    QApplication a(argc, argv);
 
    QFuture<void> future = QtConcurrent::run(hello);
    qDebug() << "hello from GUI thread " << QThread::currentThread();
    future.waitForFinished();
    return 0;   
}

в hello() реализуешь
и не надо еб с потоками  :D

Сделал все в одном потоке, ситуация повторилась :( Значит дело не в потоках...

Стартую новую страничку:

C++ (Qt)
void QOutputDev::startPage (int pageNum, GfxState *state)
{         m_pageNum=pageNum;
         m_image = new QImage((int)state-> getPageWidth( ),(int)state-> getPageHeight( ),QImage::Format_ARGB32);
         m_painter = new QPainter (m_image );
        m_painter->setBrush(QBrush(Qt::white));
        QPainterPath path;
        //path.addRoundedRect(0,0,m_image->width(),m_image->height(),20,20);
        path.addRect(0,0,m_image->width(),m_image->height());
        m_painter->drawPath(path);
}
 

Конец странички:

C++ (Qt)
void QOutputDev::endPage()
{
        paint();
        m_painter->end();
        delete m_image;
        delete m_painter;
}

проблема в m_painter-> drawText (QPointF(x1,y1), str ); , если закоментировать, то с памятью все нормально!

Что тут не так?

Так а в чем фишка?

Вы бы на личности не переходили бы. Звучит как оскорбление. Это неприятно и  неуместно.

P.S.Извините за оффтоп. Он первый начал.  :'(

// Вы бы на личности не переходили бы. Звучит как оскорбление. Это неприятно и  неуместно.
а то что ты написал, приятно ТС читать? хотя ТС какой-то странный и походу, что ему глубоко пофик на твое мнение, но все же, оно неуместно!
и я лично про тебя ничего не говорил, а написал в общем, что не люблю лицемеров.

// P.S.Извините за оффтоп. Он первый начал.
детский сад :D

Ну в приведенном Вами фрагменте drawText вообще не просматривается  :) На всякий случай я бы поменял местами delete m_image/m_painter (чтобы было типа push/pop). Также обнулил бы эти указатели после удаления. Шансов что это поможет очень мало, но почиститься никогда не мешает.

Дальше стал бы  выкидывать из paint куски и смотреть - ну Вы так и делаете.

попробую...
Но в принципе там же стоит m_paiter->end()  перед удалением.

Я сделал замечание по поводу формы. Я читаю этот форум и хочу, чтоб люди старались писать грамотно. А Вы сказали, что так делают только лицемеры. Это оскорбление.

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

чечино, у тебя что аватар, что большинство твоих сообщений на грани фола.
ты сейчас не прав, но продолжаешь что-то писать не по делу.
себя так не ведут. ты портишь этим форум.

хотелось бы узнать, помог ли совет igors

НЕ ПОМОГЛО!

на 70 страничек ~ 45MБ памяти, пипец.
но если закоментировать m_painter-> drawText (QPointF(x1,y1), str ); и вместо текста рисовать прямоугольники то все в порядке ~13МБ  памяти. ( с самой пргой)
Что за херня?

// Я читаю этот форум и хочу, чтоб люди старались писать грамотно
ну я ж грю лицемер.

andrew.k
на свою посмотри  :D
а так это оффтоп. иногда забавно его почитать.
особенно, когда SASA извеняется, а потом оффтопит. просто перлы выдает ;D
ТС все равно код зажал и хочет, чтобы ему телепаты из пальца высосали решение :D
так что это вся тема, сплошной оффтоп, поэтому я не считаю себя неправым или виноватым.
но оффтопить больше не буду :D

Ubuntu_linux - хватит пложить печаль и багопроизводство - с тебя твой проект, или урезанная его версия, создающая/отрисовывающая/рисующая текст. И мы тогда уж 100% разберем.

А так - Кабы если бы кабы, да в лесу да по грибы, да по маленькой опушке шли неведомы зверушки, а в дебагерах ошибки, добавляются так шибко, а при запуске экран, синий весь парам там там.

скоро

Молодец что читаешь на иностранных языках. Хорошая книга - желание помочь отпало враз.