Opened 13 years ago

Closed 12 years ago

#3636 closed defect (wontfix)

MythMainWindow::TranslateKeyPress bug?

Reported by: anonymous Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: mythtv Version: head
Severity: medium Keywords: TranslateKeyPress jump point
Cc: Ticket locked: no

Description

When watching live tv and with the "Program Guide" jump point and "GUIDE" in TV Playback context are both set to 'S' I see a flash of the live tv in guide screen before the "Program Guide" page renders?

I have placed some debug VERBOSE statements in tv_play.cpp and mythmainwindow.cpp and have noticed that:

  • When watching live tv and I hit 'S' that ignoreKeys is false when TV::ProcessKeypress?() is called.
  • The jump point is scheduled by TranslateKeyPress?("ITV Menu",...)@2257 and the return is false and program flow continues.
  • It is slightly after the GUIDE is displayed (the video isn't painted yet) that the jump point kicks in and switches to the Program Guide.
  • This could happen for any jumppoint and local action that shares the same keybinding as far as I can tell.

Two questions:

Q1/ Should MythMainWindow::TranslateKeyPress? first check to see if a jump point is currently queued (!d->exitmenucallback == NULL) and simply return false if it is?

Q2/ Should there be a test for jump points at the start of TV::ProcessKeypress?() so they are handled before all the other stuff?

Attached is the patch to insert the debug statements I used.

PS: I posted on the -dev list but it didn't get any response so I created the ticket...

Attachments (1)

translate-keypresses-debug.diff (1.9 KB) - added by Roo 13 years ago.
Debug statements patch

Download all attachments as: .zip

Change History (3)

Changed 13 years ago by Roo

Debug statements patch

comment:1 Changed 13 years ago by mythtv@…

Just to bring this ticket up (again), when I had the jumpoint for guide set to S my LiveTV went into an endless loop of flash-guide/flash-tv/flash-guide/flash-tv. Don't know if it helps, but meh.

Ticket was #2668.

comment:2 Changed 12 years ago by stuartm

Resolution: wontfix
Status: newclosed
Note: See TracTickets for help on using tickets.