[SOLVED] How can I used the same thread without having to create a new one ??

[SujaMM]

I'm not sure if I'm even deleting the image correctly.

QDesktopWidget* dw = QApplication::desktop();
       QPixmap pixmap = QPixmap::grabWindow( dw->winId(), 0, 0, dw->width(), dw->height() );

       QDir mDir(QDir::homePath());
       QString format = "jpg";
       QString filename = "/perfq_" + QString::number(QDate::currentDate().year())+ "_"
               +QString::number(QDate::currentDate().month())+ "_"
               +QString::number(QDate::currentDate().day())+ "_"
               +QString::number(QTime::currentTime().hour())+ "_"
               +QString::number(QTime::currentTime().minute())+ "_"
               +QString::number(QTime::currentTime().second());
       QString mPath = QDir::homePath() + "/perfq_id_" + QString::number(this->idu);
       if (!mDir.exists(mPath))
            mDir.mkpath(mPath);
       pixmap.save(mPath+filename+".jpg", format.toLatin1().constData());

however how can I use the QThreadPool ??

[koahnig]

You are using Qt 4.8. The links I had provided are for Qt 5. grabWindow is not defined for QPixMap in Qt 5.
Here is the link for the proper Qt 4.8 QThreadPool documentation.

I though that you might allocate memory for the picture and do not release properly. I cannot see a reason either.

With QThreadPool I have to stay out of discussion. Probably I do not know more about threading than you do.

[SujaMM]

No, I'm ussing Qt 5.

Why did you think I was ussing Qt 4.8 ??

[SujaMM]

I fix the problem I so I didn't have to create a new thread every time, instate I just call the start function.

Thanks.

[koahnig]

Good to know that you have fixed your problem.

My conclusion was that you are using Qt4 since you use grabWindow which I could not find in Qt 5 documentation.
However, grabWindow and grabWidget are marked as obsolete in Qt 5. Apparently, those are still working, but this may be something to be changed, since there is no real support anymore.

[SujaMM]

I didn't know this.

it's there another way to take screenshots ussing non obsolete libraries in Qt5??

Because this was the only way It works with multiple-monitors.

[koahnig]

@SujaMM
Checkout the Qt5 screenshot example.
It uses QGuiApplication::primaryScreen() to take the screenshot. pimaryScreen There is also screens() and other stuff.

[SujaMM]

I try this before I use this code:

QApplication a(argv, argc);

QScreen *screen = a.primaryScreen();

QPixmap screenshot = screen->grabWindow(0);

screenshot.save('screenshot.png', 'png');
QList<QScreen*> screens = a.screens();
QScreen *screen;
QPixmap screenshot;

for(int i = 0; i < screens.length(); i++){
   screen = screens.at(i);
   screenshot = screen->grabWindow(0);
   screenshot.save(QString::number(i) + ".png", 'png');
}

but it will take various screenshots of the primary screen.

[koahnig]

@SujaMM said:

   screenshot = screen->grabWindow(0);

The reason is probably that you are using the same windows identifier on each call.

[SujaMM]

Oooh, should I use i insted of 0 in this line screenshot = screen->grabWindow(0); ??