Opened 11 years ago

Closed 11 years ago

#6174 closed patch (fixed)

Fix for slow status update in mythwelcome

Reported by: bullestock@… Owned by: paulh
Priority: minor Milestone: unknown
Component: MythTV - Mythwelcome & Mythshutdown Version: 0.21-fixes
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Since the status update runs on a 2 second timer, mythwelcome can sometimes show an incorrect status right after exiting mythfrontend.

This patch forces a status refresh when myth_system() returns. The fix is not perfect, as the incorrect status can still be shown for half a second or so, but is is better than before.

Attachments (1)

mythwelcome.patch (494 bytes) - added by bullestock@… 11 years ago.
Patch

Download all attachments as: .zip

Change History (7)

Changed 11 years ago by bullestock@…

Attachment: mythwelcome.patch added

Patch

comment:1 Changed 11 years ago by Dibblah

Status: newinfoneeded_new

Isn't this fixed by [18671]?

comment:2 in reply to:  1 Changed 11 years ago by paulh

Status: infoneeded_newnew

Replying to Dibblah:

Isn't this fixed by [18671]?

Different issue.

I'm not sure a maximum of 2 seconds is really too long to wait for it to update though?

comment:3 Changed 11 years ago by paulh

Owner: changed from Isaac Richards to paulh
Status: newassigned

comment:4 Changed 11 years ago by bullestock@…

No, this is not at all related to #18671.

In case I did not make myself clear the first time, let me provide a scenario:

1) My MythTV box wakes up to start a scheduled recording. MythWelcome? correctly shows a status of 'recording'. 2) I start mythfrontend and cancel the recording. MythTV is now idle. 3) I exit mythfrontend. 4) Now, I expect MythWelcome? to show a status of 'not recording' - but due to the (up to) 2 s delay, the status will briefly show 'recording'.

I'm not saying that this is a huge issue, but it has annoyed me more than once. And it *is* a one-line patch.

comment:5 Changed 11 years ago by paulh

(In [19898]) Make sure MythWelcome?'s status messages are updated immediately after MythFrontend exits rather than waiting for the update timer to do it. Refs #6174.

comment:6 Changed 11 years ago by paulh

Resolution: fixed
Status: assignedclosed

(In [19900]) Backports [19898] to fixes.

Make sure MythWelcome?'s status messages are updated immediately after MythFrontend exits rather than waiting for the update timer to do it. Fixes #6174.

Note: See TracTickets for help on using tickets.