Opened 5 years ago

Closed 4 years ago

Last modified 4 years ago

#12489 closed Bug Report - General (fixed)

masking images delays mythfrontend startup

Reported by: lomion@… Owned by: stuartm
Priority: minor Milestone: 0.28
Component: MythTV - User Interface Library Version: 0.27.4
Severity: medium Keywords:
Cc: Ticket locked: no

Description

fphilips found out that masked background images in menu_ui.xml dramatically delay startup of mythfrontend (see https://github.com/fphillips/blue-abstract-wide/commit/b13747811da89c13903ea531a20919609027d738)

With every <mask> tag the mask images is loaded (and resized in trunk) which causes the delay.

Attached is a fix for it

Attachments (3)

image_mask.patch (2.4 KB) - added by lomion@… 5 years ago.
mythui.diff (3.4 KB) - added by lomion@… 5 years ago.
Fixed copy constructor
12489_0_28.patch (3.9 KB) - added by lomion@… 4 years ago.
Updated patch for fixes/0.28

Download all attachments as: .zip

Change History (11)

Changed 5 years ago by lomion@…

Attachment: image_mask.patch added

comment:1 Changed 5 years ago by stuartm

Owner: set to stuartm
Status: newaccepted

Changed 5 years ago by lomion@…

Attachment: mythui.diff added

Fixed copy constructor

comment:2 Changed 5 years ago by Stuart Auchterlonie

appears to be an extraneous change to mythtv/libs/libmythui/mythuitext.cpp in that last patch

comment:3 Changed 5 years ago by lomion@…

starta: Correct, the last change in mythuitext.cpp is not related to this issue. When using a theme with scrolling text the CPU usage goes to 100%! That's another bug I guess. I don't understand the code well enough to fix this bug so I tried to disable scrolling whenever the screensafer is active to safe some CPU cycles (and power). As a result mythfrontend went down to 30% but X11 went up to 70% :-P Maybe somebody might want to look into it ... in the meantime I switch to a screen without scrolling ;-)

comment:4 Changed 4 years ago by paulh

Component: MythTV - GeneralMythTV - User Interface Library

comment:5 Changed 4 years ago by Karl Egly

Milestone: 0.27.60.27.7

Reschedule all tickets planned for, but not solved in time for, 0.27.6 to 0.27.7.

Changed 4 years ago by lomion@…

Attachment: 12489_0_28.patch added

Updated patch for fixes/0.28

comment:6 Changed 4 years ago by Lomion0815 <lomion@…>

Resolution: fixed
Status: acceptedclosed

In e049905721c148c27ffc0766b8d387ac7ab97343/mythtv:

Reduce screen/frontend load times by moving the load of mask images. Fixes #12489

comment:7 Changed 4 years ago by Stuart Auchterlonie

Milestone: 0.27.70.28

comment:8 Changed 4 years ago by Lomion0815 <lomion@…>

In ea5a50b6555e34e97219ca43475d67307e4621c2/mythtv:

Reduce screen/frontend load times by moving the load of mask images. Fixes #12489

Note: See TracTickets for help on using tickets.