Review Request 110138: missing checks for size(0, 0) and missing check before freeing pixmap

[Ömer Fadıl Usta]

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

(Updated April 27, 2013, 12:04 p.m.)


Review request for kdelibs, Aaron J. Seigo, Martin Gräßlin, and Marco Martin.


Changes
-------

In this patch there is 2 patches 1 from : https://git.reviewboard.kde.org/r/110208/
and 1 is from mine. I have to combine 2 of them because even first one get a "ship it"
it didn't committed yet so i have to add it to show you complete one. Otherwise i need
to wait until it committed than make my diff on that one.

So mine part is almost useless but gets away lots of warnings which makes me sick of them.


Description
-------

I was getting lots of errors while moving my mouse over system tray before this patch.( https://git.reviewboard.kde.org/r/110029/ ) 
Also this patch fixes a forgetten check before freeing a already freed pixmap


Diffs (updated)
-----

  plasma/private/dialogshadows.cpp e787fd4 
  plasma/private/dialogshadows_p.h a2d62cc 

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


Testing
-------

Looks like it is working fine


Thanks,

Ömer Fadıl Usta

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130427/ef6f2632/attachment.htm>