Opened 20 months ago

Last modified 7 hours ago

#13026 assigned Patch - Bug Fix

Clean up technical debt from Qt4 changes

Reported by: David Hampton <mythtv@…> Owned by: David Hampton
Priority: minor Milestone: 30.0
Component: MythTV - General Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Changes from Qt3 to Qt4 removed the need to explicitly call the detach or copy functions on qt classes. These will be called internally as necessary. See http://doc.qt.io/qt-4.8/porting4.html

Change History (8)

comment:1 Changed 20 months ago by David Hampton <mythtv@…>

comment:2 Changed 20 months ago by stuartm

This particular area should be treated with caution, although there were areas with QT4 where we apparently didn't need to call detach according to the manual, failing to do so lead to lots of subtle and not so subtle bugs.

QT just didn't always do the right thing without being explicit about it. Daniel K was the real expert on this topic, having spent a lot time debugging the problem, so my understanding is limited but he did write an in-depth guide on the subject, I just can't find it at the present.

Last edited 20 months ago by stuartm (previous) (diff)

comment:3 Changed 14 months ago by David Hampton

Owner: set to David Hampton
Status: newaccepted

comment:4 Changed 13 months ago by Stuart Auchterlonie

Milestone: 29.029.1

comment:5 Changed 11 months ago by Stuart Auchterlonie

Milestone: 29.10.28.2

Moving remaining open tickets to 0.28.2 milestone

comment:6 Changed 11 months ago by Stuart Auchterlonie

Milestone: 0.28.229.2

Moving remaining open tickets to 29.2 milestone

comment:7 Changed 7 months ago by David Hampton

Owner: changed from David Hampton to David Hampton
Status: acceptedassigned

comment:8 Changed 7 hours ago by David Hampton

Milestone: 29.230.0
Note: See TracTickets for help on using tickets.