Review Request: Export some of the previously hidden symbols in Solid::Networking and Solid::PowerManagement
Kevin Ottens
ervin at kde.org
Tue Jun 15 21:40:38 BST 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4324/#review6128
-----------------------------------------------------------
Ship it!
Feel free to commit now.
- Kevin
On 2010-06-15 20:35:23, Arno Rehn wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4324/
> -----------------------------------------------------------
>
> (Updated 2010-06-15 20:35:23)
>
>
> Review request for kdelibs.
>
>
> Summary
> -------
>
> This patch will export some of the previously hidden symbols. This is required for the Solid language bindings to come out pretty, because they partly rely on Qt's meta-object system for signal introspection.
>
> Changing the constructor access for the Notifier classes to 'protected' would normally be source incompatible. But the symbol was previously hidden, so no app could have linked against it anyway.
>
>
> Diffs
> -----
>
> /trunk/KDE/kdelibs/solid/solid/networking.h 1137325
> /trunk/KDE/kdelibs/solid/solid/networking.cpp 1137325
> /trunk/KDE/kdelibs/solid/solid/powermanagement.h 1137325
> /trunk/KDE/kdelibs/solid/solid/powermanagement.cpp 1137325
>
> Diff: http://reviewboard.kde.org/r/4324/diff
>
>
> Testing
> -------
>
> Compiles fine, bindings generation now works as expected.
>
>
> Thanks,
>
> Arno
>
>
More information about the kde-core-devel
mailing list