Opened 8 years ago

Closed 8 years ago

#9271 closed Developer Task (Fixed)

Improved teletext text rendering

Reported by: markk Owned by: markk
Priority: minor Milestone: 0.24.1
Component: MythTV - Video Playback Version: 0.24-fixes
Severity: medium Keywords:
Cc: Ticket locked: no

Description


Change History (7)

comment:1 Changed 8 years ago by Kenni Lund [kenni a kelu dot dk]

Status: newassigned

comment:2 Changed 8 years ago by markk

ref r27367

comment:3 Changed 8 years ago by markk

(In [27381]) Refactor TeletextDecoder? and TeletextScreen? and add a new TeletextReader? class.

This follows the approach used in the other subtitle and caption classes whereby the screen object handles OSD representation of the data, the reader object holds the state and data and the decoder object feeds new data into the reader.

This simplifies setup and initialisation of the teletext objects and removes the requirement to lock the OSD from the decoder, which was causing endless problems.

The previous behaviour of always decoding teletext packets is also reinstated which radically improves responsiveness. Actually decoding and parsing the packets is trivial for a modern processor and hence should have no impact on performance.

I will backport to release-0-24-fixes in a few days if there are no reported problems.

Refs #9271

comment:4 Changed 8 years ago by markk

Milestone: unknown0.24.1

comment:5 Changed 8 years ago by markk

Status: assignedaccepted

comment:6 Changed 8 years ago by markk

(In [27418]) Backport r27367 from trunk to release-0-24-fixes

Improved teletext display.

  • fix some font stretch adjustments for double height text by basing

it on the actual stretch value.

  • be a little more agressive in determining the font height.
  • ignore the UI determined font stretch value and select the largest

possible value that will fill the width available for each character.

Collectively these ensure the font used is as large and as visible as possible, especially on widescreen displays with VDPAU/OpenGL/Direct3D and widescreen video sources (XVideo).

Use of a fixed space font is still highly recommended.

Refs #9271

comment:7 Changed 8 years ago by markk

Resolution: Fixed
Status: acceptedclosed

(In [27419]) Backport r27381 from trunk to release-0-24-fixes

Refactor TeletextDecoder? and TeletextScreen? and add a new TeletextReader? class.

This follows the approach used in the other subtitle and caption classes whereby the screen object handles OSD representation of the data, the reader object holds the state and data and the decoder object feeds new data into the reader.

This simplifies setup and initialisation of the teletext objects and removes the requirement to lock the OSD from the decoder, which was causing endless problems.

The previous behaviour of always decoding teletext packets is also reinstated which radically improves responsiveness. Actually decoding and parsing the packets is trivial for a modern processor and hence should have no impact on performance.

Closes #9271

Note: See TracTickets for help on using tickets.