Review Request: remove one of two copies of KoItemToolTip
Marijn Kruisselbrink
mkruisselbrink at kde.org
Thu Apr 26 18:14:52 BST 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104738/
-----------------------------------------------------------
Review request for Calligra.
Description
-------
Somehow we ended up with two copies of the KoItemToolTip class, one in kowidgets and one in komain. (and worse, some bug fixes were only applied to one of the two copies). This fixes that by only having one copy of the class, in the kowidgets library. I copied the komain version there since that version had some bug fixes that the kowidgets version didn't get.
The only potential downside to this I can see is that now the class needs to be exported (since the only users of it are in kowidgets and komain, the class wasn't exported before and instead somebody decided to have two copies instead).
Diffs
-----
libs/main/CMakeLists.txt f4074ea
libs/main/KoItemToolTip.h 4e35e1e
libs/main/KoItemToolTip.cpp 72b3801
libs/widgets/KoItemToolTip.h 91dd510
libs/widgets/KoItemToolTip.cpp beffbcb
Diff: http://git.reviewboard.kde.org/r/104738/diff/
Testing
-------
Thanks,
Marijn Kruisselbrink
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120426/f3e7085e/attachment.htm>
More information about the calligra-devel
mailing list