Review Request 123460: No need for Karbon to register a path for resource type "kis_brushes", right?

Friedrich W. H. Kossebau kossebau at kde.org
Tue Apr 21 22:40:43 BST 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123460/
-----------------------------------------------------------

(Updated April 21, 2015, 9:40 nachm.)


Review request for Calligra, Yue Liu and Boudewijn Rempt.


Repository: calligra


Description
-------

Karbon has `s_instance->dirs()->addResourceType("kis_brushes", "data", "krita/brushes/");`. Seems like that registration is there forever. But in current code I cannot see how Karbon could make use of brushes. And at least all code using such resource types is in krita/libbrush/kis_brush_server.cpp, so should be also self-contained.

Might have missed something, so asking the experts if this is really a bogus registration.


Diffs
-----

  karbon/ui/KarbonFactory.cpp 464bb0a 

Diff: https://git.reviewboard.kde.org/r/123460/diff/


Testing
-------


Thanks,

Friedrich W. H. Kossebau

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20150421/76732319/attachment.htm>


More information about the calligra-devel mailing list