mattrussmill/qcvTouchUp

Work Event Loop

Closed this issue · 1 comments

Remove the WorkEventLoop worker thread event loop controller. It is only being used as a portal sending slots between the mainwindow and the worker thread. Eliminate this extraneous code and use mainwindow (the communicating object) as the thread controller.

See: https://doc.qt.io/qt-5.10/qthread.html

WorkerEventLoop was removed from the project including all SIGNAL to SIGNAL connections (bad programming practice). The MainWindow class now operates as the worker thread event loop controller. Since this is the class that interfaces with the worker thread, it is appropriate for the controller to reside here. The application now correctly follows Qt's QThread event loop model.