[mythtv] [mythtv-commits] Ticket #8656: Display video format in INFO OSD

[Robert McNamara]

On Fri, Aug 27, 2010 at 12:19 AM, MythTV <mythtv at cvs.mythtv.org> wrote:
> #8656: Display video format in INFO OSD
> -------------------------------------+--------------------------------------
>  Reporter:  nigel                    |           Owner:  markk
>     Type:  patch                    |          Status:  closed
>  Priority:  minor                    |       Milestone:  0.24
> Component:  MythTV - Video Playback  |         Version:  Trunk Head
>  Severity:  low                      |      Resolution:  fixed
>  Keywords:                           |   Ticket locked:  0
> -------------------------------------+--------------------------------------
>
> Comment (by nigel):
>
>  (In [25889]) Better codec descriptions (than MPEG2) in the themed INFO
>  OSD. Refs #8656.
>
>  Currently, MythPlayer::GetEncodingType() uses GetVideoCodecID(),
>  which only iterates some of the FFmpeg codecs (plus, AVF's video_codec_id
>  is often mapped to MPEG2 in ScanStream()). Basically, a more opaque way of
>  getting codec descriptions was needed, and what is more opaque than a
>  string?
>
>  Technically, this changes the library API, but since only libmythtv is
>  calling the extra method, no version change - to save a full recompile.

Hi Nigel,

I think this may have broken the videocodec statetype?  MPEG-2 video
is no longer matching in the statetype, so OSDs that use the statetype
(Arclight) end up with nothing displayed when using the statetype.

Robert