Review Request 119033: Replace KServiceTypeTrader with KoServiceLocator
Boudewijn Rempt
boud at valdyas.org
Tue Jul 8 17:03:52 BST 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119033/
-----------------------------------------------------------
(Updated July 8, 2014, 4:03 p.m.)
Status
------
This change has been marked as submitted.
Review request for Calligra.
Repository: calligra
Description
-------
Use KoServiceLocator instead of KServiceTypeTrader
KServiceTypeTrader is dependent on the sysoca -- and on OSX and Windows that is a big chore. KoServiceLocator just loads all desktop files in services/calligra and applications/kde4/calligra and sorts them by type.
Loading the desktop files and creating KService entries for them takes less than 100ms on my system, so it's not a huge performance penalty.
Diffs
-----
krita/plugins/extensions/dockers/lut/CMakeLists.txt 28e043c
krita/plugins/extensions/dockers/overview/CMakeLists.txt 3cef9b1
krita/plugins/extensions/dockers/imagedocker/CMakeLists.txt 86cea99
krita/plugins/extensions/colorrange/CMakeLists.txt f678c19
kexi/plugins/forms/CMakeLists.txt 3dbd3b7
krita/plugins/extensions/colorspaceconversion/CMakeLists.txt 8337a48
krita/plugins/extensions/dockers/advancedcolorselector/CMakeLists.txt f3ca701
krita/plugins/extensions/dockers/artisticcolorselector/CMakeLists.txt 6af6ad8
krita/plugins/extensions/dockers/channeldocker/CMakeLists.txt f21828c
krita/plugins/extensions/dockers/compositiondocker/CMakeLists.txt 8beb794
krita/plugins/extensions/dockers/defaultdockers/CMakeLists.txt 8273262
krita/plugins/extensions/dockers/digitalmixer/CMakeLists.txt c2bca8f
kexi/plugins/forms/widgets/mapbrowser/CMakeLists.txt fd29579
kexi/plugins/forms/widgets/webbrowser/CMakeLists.txt d8c61a6
kexi/plugins/importexport/csv/CMakeLists.txt 59df853
kexi/plugins/migration/CMakeLists.txt dcd625a
kexi/plugins/queries/CMakeLists.txt 02220fa
kexi/plugins/relations/CMakeLists.txt 821eba9
kexi/plugins/reports/CMakeLists.txt 8bff375
kexi/plugins/scripting/kexiscripting/CMakeLists.txt 17d02c8
kexi/plugins/shapes/relationdesign/CMakeLists.txt 357a3c7
kexi/plugins/tables/CMakeLists.txt b0eca15
krita/CMakeLists.txt d78217d
krita/crashreporter/main.cpp 3ee0e78
krita/gemini/CMakeLists.txt f597885
krita/plugins/assistants/RulerAssistant/CMakeLists.txt 9a08253
krita/plugins/colorspaces/extensions/CMakeLists.txt f96e1f5
krita/plugins/extensions/bigbrother/CMakeLists.txt da02298
krita/plugins/extensions/clonesarray/CMakeLists.txt 23f3e6c
kexi/migration/txt/CMakeLists.txt c3514f7
kexi/migration/xbase/CMakeLists.txt c915b92
kexi/plugins/autoforms/CMakeLists.txt b703bef
krita/plugins/formats/bmp/CMakeLists.txt b7f16a3
krita/plugins/formats/eps/CMakeLists.txt 33a0784
krita/plugins/formats/exr/CMakeLists.txt 70d4ada
krita/plugins/formats/flipbook/CMakeLists.txt 0f54991
krita/plugins/formats/heightmap/CMakeLists.txt e883968
krita/plugins/formats/jp2/CMakeLists.txt fc5915f
krita/plugins/formats/jpeg/CMakeLists.txt 2acf912
krita/plugins/formats/odg/CMakeLists.txt c362dbf
krita/plugins/formats/oiio/CMakeLists.txt 5d7ca14
krita/plugins/filters/colors/CMakeLists.txt 4ee8d3f
krita/plugins/filters/colorsfilters/CMakeLists.txt 2f71712
krita/plugins/filters/convolutionfilters/CMakeLists.txt 4abcdff
krita/plugins/filters/dodgeburn/CMakeLists.txt 54684a2
krita/plugins/filters/embossfilter/CMakeLists.txt d1f4a53
krita/plugins/filters/example/CMakeLists.txt 73f39ca
krita/plugins/filters/fastcolortransfer/CMakeLists.txt c7712b4
krita/plugins/filters/imageenhancement/CMakeLists.txt 917d9e8
krita/plugins/filters/indexcolors/CMakeLists.txt 2e74c95
krita/plugins/filters/levelfilter/CMakeLists.txt d4e2297
krita/plugins/filters/noisefilter/CMakeLists.txt 35022b3
krita/plugins/filters/oilpaintfilter/CMakeLists.txt 5687740
krita/plugins/filters/phongbumpmap/CMakeLists.txt b263313
krita/plugins/filters/pixelizefilter/CMakeLists.txt 5fb58de
krita/plugins/filters/posterize/CMakeLists.txt 84e56ac
krita/plugins/filters/raindropsfilter/CMakeLists.txt a5994b1
krita/plugins/filters/randompickfilter/CMakeLists.txt 327327f
krita/plugins/filters/roundcorners/CMakeLists.txt d70ee53
krita/plugins/filters/smalltilesfilter/CMakeLists.txt ee6966d
krita/plugins/filters/sobelfilter/CMakeLists.txt 3a82020
krita/plugins/filters/unsharp/CMakeLists.txt 5ac34f8
krita/plugins/filters/wavefilter/CMakeLists.txt 1f2ea48
words/plugins/scripting/CMakeLists.txt 27bd499
sheets/plugins/calendar/CMakeLists.txt 3e2e009
sheets/plugins/scripting/CMakeLists.txt 594d87d
sheets/plugins/solver/CMakeLists.txt 83b097a
sheets/shape/CMakeLists.txt f16b1a3
plan/plugins/filters/kplato/import/CMakeLists.txt 37796fc
plan/plugins/schedulers/rcps/CMakeLists.txt 57b5f10
plan/plugins/schedulers/tj/CMakeLists.txt 59d799c
plan/plugins/scripting/CMakeLists.txt 9d22fb2
plan/workpackage/CMakeLists.txt 845e137
plugins/artistictextshape/CMakeLists.txt 7669312
plugins/chartshape/CMakeLists.txt ad3649c
plugins/colorengines/lcms2/CMakeLists.txt 23426e3
plugins/commentshape/CMakeLists.txt 48444c8
plugins/defaultTools/CMakeLists.txt 8705c1e
krita/plugins/tools/tool_transform2/CMakeLists.txt 57058f2
krita/sketch/CMakeLists.txt 3e8bcba
krita/ui/kis_view2.cpp 096436d
libs/basicflakes/plugin/CMakeLists.txt bd3c950
libs/db/drivers/sqlite/CMakeLists.txt c4deb6b
libs/db/pluginloader.h PRE-CREATION
libs/db/pluginloader.cpp PRE-CREATION
libs/flake/KoShapeFactoryBase.cpp 1fb57fb
libs/kopageapp/tools/backgroundTool/CMakeLists.txt 748bde7
libs/koplugin/CMakeLists.txt 8d18cf2
libs/koplugin/KoPluginLoader.cpp e6eb342
libs/koplugin/KoServiceLocator.h PRE-CREATION
libs/koplugin/KoServiceLocator.cpp PRE-CREATION
stage/app/CMakeLists.txt 9370b9c
stage/part/CMakeLists.txt f79eea9
stage/part/tools/animationtool/CMakeLists.txt c6bd045
stage/plugins/eventactions/CMakeLists.txt 5e615c3
stage/plugins/pageeffects/barwipe/CMakeLists.txt a1120bf
stage/plugins/pageeffects/clockwipe/CMakeLists.txt f868291
stage/plugins/pageeffects/edgewipe/CMakeLists.txt e404ea0
stage/plugins/pageeffects/example/CMakeLists.txt aa979df
stage/plugins/pageeffects/fade/CMakeLists.txt a3a5715
plugins/dockers/CMakeLists.txt 75f007e
plugins/formulashape/CMakeLists.txt 015cb8f
plugins/kexi/spreadsheet/CMakeLists.txt 1a7d1d5
plugins/musicshape/CMakeLists.txt ae5f5eb
plugins/pathshapes/CMakeLists.txt 177faf5
plugins/pictureshape/CMakeLists.txt 64aa998
plugins/pluginshape/CMakeLists.txt e29b2cc
plugins/reporting/barcode/CMakeLists.txt 145781e
plugins/reporting/chart/CMakeLists.txt 348c9e5
plugins/reporting/maps/CMakeLists.txt 7993ca6
plugins/reporting/web/CMakeLists.txt 7fd8c0c
plugins/semanticitems/contact/CMakeLists.txt f23492e
plugins/semanticitems/event/CMakeLists.txt 407826e
plugins/semanticitems/location/CMakeLists.txt 5b9c233
libs/main/KoApplication.cpp 0b8965d
libs/main/KoDocumentEntry.h e9884d0
libs/main/KoDocumentEntry.cpp 12b4117
libs/main/KoFilterEntry.cpp 75ae32d
libs/main/KoFilterGraph.cpp 59177d9
libs/main/KoFilterManager.cpp f6cdec9
krita/plugins/paintops/chalk/CMakeLists.txt b3ba824
krita/plugins/paintops/colorsmudge/CMakeLists.txt 6874326
krita/plugins/paintops/curvebrush/CMakeLists.txt 779e569
krita/plugins/paintops/defaultpaintops/CMakeLists.txt 7618ecb
krita/plugins/paintops/deform/CMakeLists.txt e511d68
krita/plugins/paintops/dynadraw/CMakeLists.txt 7f599ac
krita/plugins/paintops/experiment/CMakeLists.txt 6b05915
krita/plugins/paintops/filterop/CMakeLists.txt fca6436
krita/plugins/paintops/gridbrush/CMakeLists.txt b550bdb
krita/plugins/paintops/hairy/CMakeLists.txt 547e2d3
krita/plugins/paintops/hatching/CMakeLists.txt 872ccd3
krita/plugins/paintops/mypaint/CMakeLists.txt 7977f26
krita/plugins/paintops/particle/CMakeLists.txt 1dd0538
krita/plugins/formats/qml/CMakeLists.txt 756852e
krita/plugins/formats/raw/CMakeLists.txt fd9d0f2
krita/plugins/formats/tga/CMakeLists.txt c677691
krita/plugins/formats/tiff/CMakeLists.txt ce5af68
krita/plugins/formats/xcf/CMakeLists.txt b639734
krita/plugins/generators/pattern/CMakeLists.txt cc92150
krita/plugins/formats/ora/CMakeLists.txt 0ea7784
krita/plugins/formats/pdf/CMakeLists.txt 6bee050
krita/plugins/formats/png/CMakeLists.txt 6f46370
krita/plugins/formats/ppm/CMakeLists.txt 34b499b
krita/plugins/formats/psd/CMakeLists.txt b4032c2
stage/plugins/pageeffects/iriswipe/CMakeLists.txt 81d51c6
stage/plugins/pageeffects/matrixwipe/CMakeLists.txt 7223f6d
stage/plugins/pageeffects/slidewipe/CMakeLists.txt e24ce6b
stage/plugins/pageeffects/spacerotation/CMakeLists.txt 7e1b110
stage/plugins/pageeffects/swapeffect/CMakeLists.txt c974f35
stage/plugins/shapeanimations/example/CMakeLists.txt 7cf03f4
stage/plugins/variable/CMakeLists.txt a09a8ae
words/app/CMakeLists.txt 61d5180
words/part/CMakeLists.txt 41f675b
plugins/spacenavigator/CMakeLists.txt 9859962
plugins/staging/pivottables/CMakeLists.txt c1b15b5
plugins/staging/templateshape/CMakeLists.txt 36af7d2
plugins/staging/threedshape/CMakeLists.txt c7e113a
plugins/textediting/autocorrection/CMakeLists.txt 6e8ce4f
plugins/textediting/changecase/CMakeLists.txt 93fa63c
plugins/textediting/spellcheck/CMakeLists.txt 0c7a324
plugins/textediting/thesaurus/CMakeLists.txt 5f4c1dd
plugins/textshape/CMakeLists.txt f7fcdb4
plugins/variables/CMakeLists.txt 4a50e04
plugins/vectorshape/CMakeLists.txt b2526eb
plugins/videoshape/CMakeLists.txt ccf5013
sheets/CMakeLists.txt 71c6159
sheets/functions/CMakeLists.txt 27f8c8f
libs/main/tests/filter_graph.cpp 87ee024
plan/CMakeLists.txt d9e24d8
plan/plugins/filters/icalendar/export/CMakeLists.txt a20e2b0
krita/plugins/paintops/resourcemanager/CMakeLists.txt 56db1bb
krita/plugins/paintops/sketch/CMakeLists.txt 9263b02
krita/plugins/paintops/spray/CMakeLists.txt 63b0618
krita/plugins/tools/defaulttools/CMakeLists.txt a33a1b1
krita/plugins/tools/selectiontools/CMakeLists.txt bbb8c88
krita/plugins/tools/tool_crop/CMakeLists.txt 5c7bd9b
krita/plugins/tools/tool_dyna/CMakeLists.txt 80c5025
krita/plugins/tools/tool_grid/CMakeLists.txt d068671
krita/plugins/tools/tool_perspectivegrid/CMakeLists.txt 968d0e0
krita/plugins/tools/tool_polygon/CMakeLists.txt bd4eaf1
krita/plugins/tools/tool_polyline/CMakeLists.txt e124221
krita/plugins/tools/tool_text/CMakeLists.txt 8ac17d1
krita/plugins/generators/solid/CMakeLists.txt 8c4b7b8
krita/plugins/kimgio/CMakeLists.txt 15187a2
krita/plugins/extensions/imagesplit/CMakeLists.txt b1d28e1
krita/plugins/extensions/layergroupswitcher/CMakeLists.txt 31847bc
krita/plugins/extensions/metadataeditor/CMakeLists.txt bdde0fa
krita/plugins/extensions/modify_selection/CMakeLists.txt f788780
krita/plugins/extensions/offsetimage/CMakeLists.txt 2538911
krita/plugins/extensions/rotateimage/CMakeLists.txt f6a52ef
krita/plugins/extensions/separate_channels/CMakeLists.txt 5a122ec
krita/plugins/extensions/shearimage/CMakeLists.txt a7536be
krita/plugins/filters/blur/CMakeLists.txt 85ca597
krita/plugins/extensions/dockers/palettedocker/CMakeLists.txt 3b088d2
krita/plugins/extensions/dockers/patterndocker/CMakeLists.txt 0148dcd
krita/plugins/extensions/dockers/presetdocker/CMakeLists.txt c0d2654
krita/plugins/extensions/dockers/smallcolorselector/CMakeLists.txt 8957b33
krita/plugins/extensions/dockers/specificcolorselector/CMakeLists.txt dc7a333
krita/plugins/extensions/dockers/tasksetdocker/CMakeLists.txt 8234972
krita/plugins/extensions/dropshadow/CMakeLists.txt 9337d7d
krita/plugins/extensions/gmic/CMakeLists.txt 34c4fec
krita/plugins/extensions/histogram/CMakeLists.txt fd05940
krita/plugins/extensions/imagesize/CMakeLists.txt 70a5371
kexi/CMakeLists.txt 01ea697
kexi/formeditor/factories/CMakeLists.txt 635852d
kexi/kexidb/drivers/mysql/CMakeLists.txt c10e847
kexi/kexidb/drivers/odbc/CMakeLists.txt d554be9
kexi/kexidb/drivers/oracle/CMakeLists.txt 08ca310
kexi/kexidb/drivers/pqxx/CMakeLists.txt 2df3374
kexi/kexidb/drivers/sybase/CMakeLists.txt 1a23da9
kexi/kexidb/drivers/xbase/CMakeLists.txt 10f3933
kexi/migration/mdb/src/CMakeLists.txt 827b19f
kexi/migration/mysql/CMakeLists.txt b04acf8
kexi/migration/pqxx/CMakeLists.txt 2136d17
kexi/migration/sybase/CMakeLists.txt 684da55
active/CMakeLists.txt d6faa0a
braindump/data/CMakeLists.txt b67d164
braindump/plugins/quickstates/CMakeLists.txt cb97fff
braindump/plugins/stateshape/CMakeLists.txt 25f086e
braindump/plugins/webshape/CMakeLists.txt b823d15
braindump/src/View.cpp 6421fac
doc/CMakeLists.txt 0b4e238
extras/calligra/main.cpp 051a955
extras/properties/CMakeLists.txt 6eb2559
extras/quickprint/CMakeLists.txt 659b8ed
filters/flow/imageexport/CMakeLists.txt e11ee80
filters/flow/visio/import/CMakeLists.txt dde9ec2
filters/flow/wpg/import/CMakeLists.txt 34eab50
filters/karbon/eps/CMakeLists.txt d2f2192
filters/karbon/image/CMakeLists.txt 112a5cd
filters/karbon/karbon1.x/CMakeLists.txt 482cc65
filters/karbon/pdf/CMakeLists.txt 6128938
filters/karbon/svg/CMakeLists.txt e87ae77
filters/karbon/wmf/CMakeLists.txt 13532ed
filters/karbon/wpg/CMakeLists.txt 5f0c060
filters/karbon/xaml/CMakeLists.txt cd4a494
filters/karbon/xfig/CMakeLists.txt 26038f8
filters/plan/mpxj/import/CMakeLists.txt 942d460
filters/sheets/applixspread/CMakeLists.txt b77c678
filters/sheets/csv/CMakeLists.txt 0a8ea31
filters/sheets/dbase/CMakeLists.txt ac39a5b
filters/sheets/excel/export/CMakeLists.txt a9944a5
filters/sheets/excel/import/CMakeLists.txt c9b1efd
filters/sheets/gnumeric/CMakeLists.txt 887f7c3
filters/sheets/html/CMakeLists.txt 3b5d89b
filters/sheets/latex/export/CMakeLists.txt 8c69ed4
filters/sheets/opencalc/CMakeLists.txt 2809b64
filters/sheets/qpro/CMakeLists.txt e4f8b7e
filters/sheets/xlsx/CMakeLists.txt e819631
filters/stage/keynote/import/CMakeLists.txt a1e0b4d
filters/stage/kpr2odf/CMakeLists.txt 8809b261
filters/stage/powerpoint/CMakeLists.txt 90b55c7
filters/stage/pptx/CMakeLists.txt b46e5e3
filters/words/applixword/CMakeLists.txt 155ebc0
filters/words/ascii/CMakeLists.txt 19568cd
filters/words/docx/export/CMakeLists.txt e6b27ba
filters/words/docx/import/CMakeLists.txt 375e974
filters/words/epub/CMakeLists.txt 7015188
filters/words/mobi/CMakeLists.txt 6db5b4e
filters/words/msword-odf/CMakeLists.txt 717aabd
filters/words/rtf/import/CMakeLists.txt 62f5d20
filters/words/wordperfect/import/CMakeLists.txt 4fa8a0a
filters/words/works/import/CMakeLists.txt 3ac307f
flow/part/CMakeLists.txt e17ef39
flow/plugins/dockers/CMakeLists.txt 62474e9
karbon/data/CMakeLists.txt 9d4a447
karbon/plugins/filtereffects/CMakeLists.txt 1cd6304
karbon/plugins/flattenpath/CMakeLists.txt 624cdbd
karbon/plugins/refinepath/CMakeLists.txt 0f28b9b
karbon/plugins/roundcorners/CMakeLists.txt feacf7b
karbon/plugins/tools/CMakeLists.txt a3210e2
karbon/plugins/whirlpinch/CMakeLists.txt a454cd0
karbon/ui/KarbonView.cpp c57479c
krita/plugins/extensions/dockers/flipbook/CMakeLists.txt e5099eb
krita/plugins/extensions/dockers/historydocker/CMakeLists.txt 513dae3
Diff: https://git.reviewboard.kde.org/r/119033/diff/
Testing
-------
Thanks,
Boudewijn Rempt
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20140708/dce103c9/attachment.htm>
More information about the calligra-devel
mailing list