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