Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#9917 closed Patch - Bug Fix (fixed)

softblend OSD is displayed with wrong colors on powerpc

Reported by: kim@… Owned by: beirdo
Priority: minor Milestone:
Component: MythTV - Video/OSD Rendering Version: Unspecified
Severity: medium Keywords: ppc
Cc: Ticket locked: no

Description

Due to an endianness problem, components of the YUVA image are mixed up, which causes the OSD to be displayed wrongly (usually all red/pink). The trivial fix is to change:

mythtv/libs/libmythtv/util-osd.cpp, line 246:

b1 = src1 + B_OI; b2 = b1 + 4; b3 = src2 + 0; b4 = b3 + 4;
g1 = src1 + G_OI; g2 = g1 + 4; g3 = src2 + 1; g4 = g3 + 4;
r1 = src1 + R_OI; r2 = r1 + 4; r3 = src2 + 2; r4 = r3 + 4;
a1 = src1 + A_OI; a2 = a1 + 4; a3 = src2 + 3; a4 = a3 + 4;

into

b1 = src1 + B_OI; b2 = b1 + 4; b3 = src2 + B_OI; b4 = b3 + 4;
g1 = src1 + G_OI; g2 = g1 + 4; g3 = src2 + G_OI; g4 = g3 + 4;
r1 = src1 + R_OI; r2 = r1 + 4; r3 = src2 + R_OI; r4 = r3 + 4;
a1 = src1 + A_OI; a2 = a1 + 4; a3 = src2 + A_OI; a4 = a3 + 4;

Changing this fixes the OSD display problem on linux ppc. Version 0.24, 0.24.1 and git head are affected.

Change History (4)

comment:1 Changed 13 years ago by Raymond Wagner

Owner: changed from markk to beirdo
Status: newassigned

Giving this to you, unless you know if any other dev runs a Linux PPC machine to test on.

comment:2 Changed 13 years ago by Github

Milestone: unknown0.25
Resolution: fixed
Status: assignedclosed

Fix endian issue in OSD color conversion

Fixes #9917

There were already defines available that were setup by endian, just not consistently used.

Signed-off-by: Gavin Hurlbut <ghurlbut@…>

Branch: master Changeset: d8a67ef62c21e23c9ab3a37fa80cc204bc8db14c

comment:3 Changed 13 years ago by beirdo

Milestone: 0.250.24.2

Backported to fixes/0.24 as be29d21c2b13e62de86b

comment:4 Changed 12 years ago by stuartm

Milestone: 0.24.2

Milestone 0.24.2 deleted

Note: See TracTickets for help on using tickets.