Opened 18 years ago

Closed 16 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 by Dibblah, 18 years ago

Status: new → assigned
Type: defect → task

comment:2 by Roo, 18 years ago

#5766 is an example of the problem (I think).

comment:3 by paulh, 17 years ago

Is this still a problem? Not heard of any QTimer problems.

comment:4 by danielk, 17 years ago

(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:5 by danielk, 17 years ago

(In [21978]) Refs #5318. QTimer fixes.

comment:6 by danielk, 17 years ago

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 by stuartm, 17 years ago

Component: mythtv → MythTV - General

comment:8 by danielk, 16 years ago

Milestone: 0.23 → unknown

comment:9 by danielk, 16 years ago

AFAIK all these Qt4 port QTimer issues are fixed in trunk.

comment:10 by danielk, 16 years ago

Resolution: → fixed
Status: assigned → closed
Note: See TracTickets for help on using tickets.