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

[Nigel Pearson]

> Hi Nigel,
> 
> I think this may have broken the videocodec statetype?


Sorry about that - I didn't have time to test against any
other theme customisations that you guys might have done.
(I hoped to get in first)



So, we could;

1) go back to the 4 or 5 values that are returned by
get_encoding_type(GetDecoder()->GetVideoCodecID()),
by reverting my change to videocodec, and maybe add a
second value for those who just want a text description:

infoMap["videocodec"]     = GetDecoder()->GetEncodingType();
infoMap["videocodecdesc"] = GetEncodingType();




or

2) the statetype mappings could be expanded to use the new
larger set that are now (currently) iterated in videocodec:

Nuppel/MPEG-4, Nuppel/RTJPEG, Nuppel/unknown,
NONE, MPEG1, MPEG2, H261, H263, RV10, RV20, MJPEG,
MJPEGB, LJPEG, SP5X, MPEG4, RAWVIDEO, MSMPEG4V1,
MSMPEG4V2, MSMPEG4V3, WMV1, WMV2, H263P, H263I,
FLV1, SVQ1, SVQ3, DVVIDEO, HUFFYUV, CYUV, H264,
... XVID ... VC1 ... MPEG2TS ... tr("Unknown")




I guess option 2 is looking a bit too hard?


--
Nigel Pearson, nigel at ind.tansu.com.au|An atheist is a person|
Telstra Net. Eng., Sydney, Australia | with no invisible    |
Office: 9202 3900    Fax:  9212 6348 | means of support!    |
Mobile: 0408 664435  Home: 9792 6998 |    - Sir John Buchan.|