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.

Reply via email to