Review Request 123340: Remove unneeded resource type setting for s_componentData->componentName() + "data"
Friedrich W. H. Kossebau
kossebau at kde.org
Sun Apr 12 11:47:07 BST 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123340/
-----------------------------------------------------------
(Updated April 12, 2015, 10:47 a.m.)
Status
------
This change has been marked as submitted.
Review request for Calligra, David Faure and Boudewijn Rempt.
Changes
-------
Submitted with commit 92bc60aa40fe3277ecb058215ae1a62c6afd66a6 by Friedrich W. H. Kossebau to branch calligra/2.9.
Repository: calligra
Description
-------
When copying over KPart code into Calligra, during the dekpartification, also lines for registering a resourcetype componentName() + "data" were copied.
>From what I understood this behaviour is not needed at all, at least in Calligra.
And from reading http://marc.info/?l=kde-commits&m=98328741017209&w=2 it seems it really was just a random idea that never turned into something really useful, which is also why it now traceless seems to have disappeared from KParts in KF5.
Can you confirm, David?
Diffs
-----
krita/ui/kis_factory2.cc 2f5ba41
libs/main/KoPart.cpp 0cc5658
Diff: https://git.reviewboard.kde.org/r/123340/diff/
Testing
-------
All Calligra apps still start fine, no problems seen.
Thanks,
Friedrich W. H. Kossebau
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20150412/9343bf08/attachment.htm>
More information about the calligra-devel
mailing list