Modify

Ticket #9271 (closed Developer Task: Fixed)

Opened 2 years ago

Last modified 2 years ago

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


Attachments

Change History

comment:1 Changed 2 years ago by kenni

  • Status changed from new to assigned

comment:2 Changed 2 years ago by markk

ref r27367

comment:3 Changed 2 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 2 years ago by markk

  • Milestone changed from unknown to 0.24.1

comment:5 Changed 2 years ago by markk

  • Status changed from assigned to accepted

comment:6 Changed 2 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 2 years ago by markk

  • Status changed from accepted to closed
  • Resolution set to Fixed

(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

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.