https://bugs.kde.org/show_bug.cgi?id=410851
Michael Pyne <mp...@kde.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Latest Commit| |https://invent.kde.org/fram | |eworks/kcoreaddons/commit/d | |aec110942baed6c7e8bcb7684e6 | |5411fe8acbe8 Status|REPORTED |RESOLVED Resolution|--- |FIXED --- Comment #1 from Michael Pyne <mp...@kde.org> --- Git commit daec110942baed6c7e8bcb7684e65411fe8acbe8 by Michael Pyne, on behalf of Adriaan de Groot. Committed on 19/02/2021 at 02:29. Pushed by mpyne into branch 'master'. kpluginfactory: Use the user-provided base factory in our factory macro. The existing macro always inherits from KPluginFactory -- making it hard to use when sub-classing KPluginFactory in order to extend it. The parameter baseFactory to the macro isn't used. Use baseFactory as the base class for the plugin. -- Commentary by committer: This closes out merge request !1. Due to the length of time between when the MR was authored and when we could finally land it, this also includes other commits added as part of the MR to: * Use @-style Doxygen comments consistently * Adapt other Doxygen API documentation to our style guide * Some other miscellaneous editorial improvements On top of those changes, I have ensured that the other changes to this header made in the meantime now conform to the consistent API documentation (i.e. other \ to @ conversions), and to update the @since version to be accurate again. M +148 -117 src/lib/plugin/kpluginfactory.h https://invent.kde.org/frameworks/kcoreaddons/commit/daec110942baed6c7e8bcb7684e65411fe8acbe8 -- You are receiving this mail because: You are watching all bug changes.