On Friday 07 February 2014 09:38:41 Kevin Krammer wrote:
> On Friday, 2014-02-07, 08:53:54, Martin Gräßlin wrote:
> > Hi,
> >
> > I found some HAVE_X11 not defined warnings in KIO and had a look at them.
> > One of them is in core/kprotocolmanager.cpp in the following snippet.
> >
> > // This is not the OS, but the windowing system, e.g. X11 on Unix/Linux.
> > static QString platform()
> > {
> > #if HAVE_X11
> >
> > return QL1S("X11");
> >
> > #elif defined(Q_OS_MAC)
> >
> > return QL1S("Macintosh");
> >
> > #elif defined(Q_OS_WIN)
> >
> > return QL1S("Windows");
> >
> > #else
> >
> > return QL1S("Unknown");
> >
> > #endif
> > }
> >
> > I'm wondering what to do about it. The best would be to use
> > QGuiApplication::platformName, but it's a core app. Also finding X11 in
> > CMakeLists to get the HAVE_X11 defined looks very wrong to me and not
> > future safe (Wayland).
>
> My guess is that platform() in this context means operating system, not
> windowing/display system.See the comment I pasted, it's explicitly saying it's the windowing system and not the OS... Cheers Martin
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
