-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100520/
-----------------------------------------------------------

Review request for KDE Base Apps and KDE Runtime.


Summary
-------

The base splashimage pixmap/qimage is never initialized, so if for some reason 
the splashscreen doesn't cover the entire screen, you get garbage. 

That reason can be a ksplashx theme that does not use multiple monitors on a 
multi-monitor system, or if it picks a background that is smaller than the 
current resolution (for example the Horos splash 1280x1024 version appears to 
be 1278x1024).


This addresses bug 264444.
    http://bugs.kde.org/show_bug.cgi?id=264444


Diffs
-----

  ksplash/ksplashx/splash.cpp d6a992a 

Diff: http://git.reviewboard.kde.org/r/100520/diff


Testing
-------

I tested this by both replacing the background that ksplashx was using with a 
smaller one, and on two multi-monitor systems (one of which uses a gentoo 
ksplash screen that only covered one of the monitors).

The splash_image.fill( 0 ) can be changed to another color for debugging 
purposes, clearly showing the region of the monitor that is not covered by the 
splashscreen (and where before would be garbage from graphics memory).


Thanks,

Ivo

Reply via email to