QT update video frame cause stuck?

[eyllanesc]

@Puppy-Bear Using processEvent and sleep is unnecessary. Send the signal only once and you should not have problems, if the slot associated with the signal is not called then there is something wrong with your program.

[JonB]

@Puppy-Bear said in QT update video frame cause stuck?:

slot{
while(1)
{
qApp->processEvents();
msleep()
}
}

Even if using processEvents()/msleep() were the right thing to do --- which is not the case --- you should realise that any while (1) would mean the slot never exits. Which would be disastrous, and you should understand why.

As @eyllanesc have said, just one plain send of signal. Anything then wrong is another matter.

[Puppy Bear]

This post is deleted!

[Puppy Bear]

This post is deleted!

[Puppy Bear]

@JonB Hi I'm actually trying to crop image from camera capture thread, so it would be a infinite loop, and keep sending it to a preprocess thread.
my code looks like this and it seems running normal right now,
If it's not a nice way to run with, would you please show me how am I able to execute a loop inside a thread's slot?
My code is below:
promoted QLabel:

void MyLabel::mouseReleaseEvent(QMouseEvent *e){
if(mouse_released==true){
emit updaterate(m_roi, this->width(), this->height());
}

SLOT in mainwindow.cpp

void app0::updateroi(QRectF roi, double label_w, double label_h)
{
	thread2.start();//sending cropped img to this thread, using movetoThread
	while(1){
		qApp->processEvents();
		crop_image_function();
		if(image.isNull()==false){
			emit sendcropimage(image); //I'm connecting it with my preprocess thread's slot
			cout << "send crop image" << "\n";
			ui->videoviewer2->setPixmap(QPixmap::fromImage(image));//crop from member QImage and showing it
			}
		QThread::msleep(30);
	}
}

[jsulm]

@Puppy-Bear said in QT update video frame cause stuck?:

void app0::updateroi(QRectF roi, double label_w, double label_h)
{
thread2.start();//sending cropped img to this thread, using movetoThread
while(1){

Do you understand that updateroi(...) will hang forever because you have an infinite loop inside without any break condition?
Use a QTimer to trigger actions on a regular basis instead of doing strange hacks.

[JonB]

@Puppy-Bear

@JonB said in QT update video frame cause stuck?:

you should realise that any while (1) would mean the slot never exits. Which would be disastrous, and you should understand why.

[J.Hilk]

@Puppy-Bear said in QT update video frame cause stuck?:

thread2.start();//sending cropped img to this thread, using movetoThread

I kind of doubt you're doing any "threading" at all.

[Puppy Bear]

@JonB Hi Thanks for your reminding, as my limit of experience, I have updated my code below.

timer = new QTimer(this);
timer->setInterval(20);
timer->start(); 

void app0::updateroi(QRectF roi, double label_w, double label_h)
{
	timer->setInterval(20);
	timer->start();}
void app0::stop_timer() {
	timer->stop();
}
void app0::cropimg() {
	qApp->processEvents(QEventLoop::AllEvents, 100);
	//Do some crop img preprocess here
}

@eyllanesc said in QT update video frame cause stuck?:

@Puppy-Bear Using processEvent and sleep is unnecessary. Send the signal only once and you should not have problems, if the slot associated with the signal is not called then there is something wrong with your program.

It's been mentioned that processEvent is unecessary which made me a little confused earlier.

[jsulm]

@Puppy-Bear said in QT update video frame cause stuck?:

void app0::cropimg() {
qApp->processEvents(QEventLoop::AllEvents, 100);

processEvents() is not needed