Review Request: Fix crash when scrolling over overlay buttons and tidy up that code

[Commit Hook]

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


This review has been submitted with commit 8348dda8de679adc91ff3ea63a34b0221d0e7978 by Martin Klapetek to branch master.

- Commit Hook


On May 8, 2012, 8:27 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104887/
> -----------------------------------------------------------
> 
> (Updated May 8, 2012, 8:27 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> The overlay classes were massive mess, this severely tidies it up by merging several classes together as they only derived themselves for no real purpose and removes lots of unused or useless methods. Overall the code is much simpler and cleaner now. And it fixes the crash.
> 
> 
> This addresses bug 293886.
>     http://bugs.kde.org/show_bug.cgi?id=293886
> 
> 
> Diffs
> -----
> 
>   contact-delegate-overlay.h 2307044 
>   contact-delegate-overlay.cpp 4733cdd 
>   contact-delegate.cpp 8c6aebe 
>   contact-list-widget.cpp 73be93c 
>   contact-overlays.h f232c5c 
>   contact-overlays.cpp 4200b46 
>   contact-view-hover-button.h 4248987 
>   contact-view-hover-button.cpp fd411b2 
> 
> Diff: http://git.reviewboard.kde.org/r/104887/diff/
> 
> 
> Testing
> -------
> 
> Everything still works.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120512/fff43c76/attachment.html>