----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123347/ -----------------------------------------------------------
(Updated April 13, 2015, 8:11 p.m.) Status ------ This change has been marked as submitted. Review request for Calligra, Dmitry Kazakov and Boudewijn Rempt. Changes ------- Submitted with commit c1ee9e286b5cdf1cabef09279d2cccc77374f32f by Friedrich W. H. Kossebau to branch calligra/2.9. Repository: calligra Description ------- While "kis_shaders" is registered as resource type and that path, any shader is actually fetched via the "data" resource type, from what I found. Which is not a problem, but makes that "kis_shaders" resource type unneeded. Also in Qt5 there is no way to define own resource types (no such concept in QStandardPaths), with KStandardDirs' "data" being mappable to an equivalent in QStandardPaths, so rather future-proof and no sense to change all places to use "kis_shaders". Thus I propose to remove this unused registration, to clean the sources from anything that results only in unneeded thinking during porting. And to save 4 cycles on start-up for 2.9 ;) I might have overlooked something, so going via a RR :) Diffs ----- krita/ui/kis_factory2.cc c24d77f Diff: https://git.reviewboard.kde.org/r/123347/diff/ Testing ------- Thanks, Friedrich W. H. Kossebau
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
