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