Review Request: remove one of two copies of KoItemToolTip

Boudewijn Rempt boud at valdyas.org
Thu Apr 26 18:30:02 BST 2012


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

Ship it!


Ship It!

- Boudewijn Rempt


On April 26, 2012, 5:14 p.m., Marijn Kruisselbrink wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104738/
> -----------------------------------------------------------
> 
> (Updated April 26, 2012, 5:14 p.m.)
> 
> 
> 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/b90e7f7d/attachment.htm>


More information about the calligra-devel mailing list