Opened 16 years ago
Closed 14 years ago
#5318 closed task (fixed)
QTimer audit
Reported by: | danielk | Owned by: | danielk |
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | MythTV - General | Version: | head |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
Due to changes in how Qt handles QTimers in Qt4 all QTimer instances in MythTV need to be audited to ensure they still work.
Change History (10)
comment:1 Changed 15 years ago by
Status: | new → assigned |
---|---|
Type: | defect → task |
comment:2 Changed 15 years ago by
comment:4 Changed 15 years ago by
(In [21977]) Refs #5318. QTimer fixes.
Fixes an unsafe QTimer delete in httpcomms. I don't know if the QTimer usage is safe here, but this whole class should probably be phased out as it appears to be a hack around the lack of an event thread in individual QThread instances. Which was fixed in Qt4.
Fixes a double delete of the m_updateTimer in guidegrid.
Fixes a Qt4 out-of-create-thread start/stop problem with previewVideoRefreshTimer.
comment:6 Changed 15 years ago by
Milestone: | 0.22 → 0.23 |
---|
I have not looked at LCD timer usage nor at the plugins, but I looked for glaring errors elsewhere. I marked the QTimer's that I'm pretty sure are ok with an "audited #5318" comment in the header.
comment:7 Changed 15 years ago by
Component: | mythtv → MythTV - General |
---|
comment:8 Changed 14 years ago by
Milestone: | 0.23 → unknown |
---|
comment:10 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
#5766 is an example of the problem (I think).