HAVE_X11 usage in KIO/core

Albert Astals Cid aacid at kde.org
Fri Feb 7 23:55:22 UTC 2014


El Divendres, 7 de febrer de 2014, a les 15:25:42, Kevin Krammer va escriure:
> On Friday, 2014-02-07, 09:51:27, Martin Gräßlin wrote:
> > On Friday 07 February 2014 09:38:41 Kevin Krammer wrote:
> > > On Friday, 2014-02-07, 08:53:54, Martin Gräßlin wrote:
> > > > 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...
> 
> Ah, didn't see that. Does it actually make sense?
> 
> If yes than this obviously has be to be done at runtime, at least for
> platforms with multiple UI systems:
> 
> #if  defined(Q_OS_MAC)
>     return QL1S("Macintosh")
> #elfi defined(Q_OS_WINDOWS)
>       return QL1S("Windows")
> #else
>     const QVariant platformName = qApp ? qApp->property("platformName") :
> QVariant();
>     if (platformName.isValid()) {
>         const QString name = platformName.toString();
>         if (!name.isEmpty())
>             return name;
>     }
> #endif
> return QL1S("Unknown");


Sincerely, just drop it.

We will change from sending 

Mozilla/5.0 (compatible; Konqueror/4.0; Linux; X11; i686; en_US) KHTML/4.0.1 
(like Gecko)

to

Mozilla/5.0 (compatible; Konqueror/4.0; Linux; i686; en_US) KHTML/4.0.1 (like 
Gecko)

will anyone ever care?

Cheers,
  Albert

> 
> Cheers,
> Kevin



More information about the Kde-frameworks-devel mailing list