Opened 14 years ago

Closed 14 years ago

#414 closed patch (fixed)

EITScanner, passive scan enhancements

Reported by: Stuart Auchterlonie Owned by: danielk
Priority: minor Milestone: 0.19
Component: mythtv Version: head
Severity: low Keywords: EITscanner
Cc: Ticket locked: no

Description

These changes limit the EITHelper::ProcessEvents? to only inserting 20 events per run. This is called from EITScanner's eventloop every 200ms. This reduces the impact on the database to ensure database accesses don't cause timeouts and pauses elsewhere.

Attachments (1)

eitscanner-passive-easy-on-db.diff (1.2 KB) - added by Stuart Auchterlonie 14 years ago.
The Patch

Download all attachments as: .zip

Change History (2)

Changed 14 years ago by Stuart Auchterlonie

The Patch

comment:1 Changed 14 years ago by danielk

Resolution: fixed
Status: newclosed

(In [7385]) Closes #414.

I didn't use the attached patch.

It turns out the Event class was not being used in a thread-safe way.

This is due to the QString and QStringList classes it contains. These classes use shared data and either all copies must be locked or you must make deep copies. I added a deep copy method to Event and use that to make this safe.

I also changed QList_Events and QListList_Events from QValueList classes to MythDequeue? classes. This lets us convert some of those loops to simple function calls. And makes sense algorithmically since we use them not as vectors but as FIFO queues.

Note: See TracTickets for help on using tickets.