<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Feb 7, 2014 at 9:51 AM, Martin Gräßlin <span dir="ltr"><<a href="mailto:mgraesslin@kde.org" target="_blank">mgraesslin@kde.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Friday 07 February 2014 09:38:41 Kevin Krammer wrote:<br>
> On Friday, 2014-02-07, 08:53:54, Martin Gräßlin wrote:<br>
> > Hi,<br>
> ><br>
> > I found some HAVE_X11 not defined warnings in KIO and had a look at them.<br>
> > One of them is in core/kprotocolmanager.cpp in the following snippet.<br>
> ><br>
> > // This is not the OS, but the windowing system, e.g. X11 on Unix/Linux.<br>
> > static QString platform()<br>
> > {<br>
> > #if HAVE_X11<br>
> ><br>
> >     return QL1S("X11");<br>
> ><br>
> > #elif defined(Q_OS_MAC)<br>
> ><br>
> >     return QL1S("Macintosh");<br>
> ><br>
> > #elif defined(Q_OS_WIN)<br>
> ><br>
> >     return QL1S("Windows");<br>
> ><br>
> > #else<br>
> ><br>
> >     return QL1S("Unknown");<br>
> ><br>
> > #endif<br>
> > }<br>
> ><br>
> > I'm wondering what to do about it. The best would be to use<br>
> > QGuiApplication::platformName, but it's a core app. Also finding X11 in<br>
> > CMakeLists to get the HAVE_X11 defined looks very wrong to me and not<br>
> > future safe (Wayland).<br>
><br>
> My guess is that platform() in this context means operating system, not<br>
> windowing/display system.<br>
<br>
</div>See the comment I pasted, it's explicitly saying it's the windowing system and<br>
not the OS...<br>
<br>
Cheers<br>
<span class="HOEnZb"><font color="#888888">Martin</font></span><br>_______________________________________________<br>
Kde-frameworks-devel mailing list<br>
<a href="mailto:Kde-frameworks-devel@kde.org">Kde-frameworks-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a><br>
<br></blockquote></div><br></div><div class="gmail_extra">Sounds to me like we want to use QGuiApplication::platformName() there, the only problem is that the strings differ.</div><div class="gmail_extra"><br></div><div class="gmail_extra">

Maybe we should just deprecate this function and advise users to use ::platformName().</div><div class="gmail_extra"><br></div><div class="gmail_extra">Aleix</div></div>