Mingw32 compile fix
ClosedPublic

Authored by habacker on Dec 17 2017, 8:16 PM.

Details

Summary

Add missing method Solid::NetworkingPrivate::serviceOwnerChanged().

The related bug is https://bugs.kde.org/show_bug.cgi?id=382695

Test Plan

cross compiled

Diff Detail

Repository
R239 KDELibs4Support
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
habacker created this revision.Dec 17 2017, 8:16 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptDec 17 2017, 8:16 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
habacker requested review of this revision.Dec 17 2017, 8:16 PM
dfaure accepted this revision.Dec 19 2017, 11:09 PM

If this wasn't kdelibs4support, I'd insist for sharing the implementation rather than duplicating it. But oh well :-)

This revision is now accepted and ready to land.Dec 19 2017, 11:09 PM
habacker added a comment.EditedDec 20 2017, 12:02 AM

If this wasn't kdelibs4support, I'd insist for sharing the implementation rather than duplicating it.

Would be nice to share, but networking.cpp is only be used on non win32 platform, so how to share ?

if(WIN32)

set(libkdelibs4support_SRCS solid/networking_win.cpp
                            kdeui/kapplication_win.cpp)

else()

set(libkdelibs4support_SRCS solid/networking.cpp)

endif()

habacker closed this revision.Dec 20 2017, 1:16 AM

That's trivial, isn't it? Rename networking.cpp to networking_unix.cpp, and move to networking.cpp the methods that are platform-independent.

habacker edited the summary of this revision. (Show Details)Dec 20 2017, 9:18 AM

That's trivial, isn't it? Rename networking.cpp to networking_unix.cpp, and move to networking.cpp the methods that are platform-independent.

Any idea how to share the line

Q_GLOBAL_STATIC(Solid::NetworkingPrivate, globalNetworkManager)

between platform independent, unix and windows related file. globalNetworkManager is required by all three files ?

Put it in the common file, and add an accessor for the other files, for instance a static method in Solid::NetworkingPrivate.