Expose desktopFileName in KWindowInfo

Review Request #129327 - Created Nov. 4, 2016 and submitted

Information
Martin Flöser
kwindowsystem
master
b62af52...
Reviewers
kdeframeworks
hein
This change introduced a new method in KWindowInfo:
QByteArray KWindowInfo::desktopFileName() const

It returns the desktop file name of the application if known on the
given platform. So far only provided on X11 through
NETWinInfo::desktopFileName.

  
Eike Hein
Martin Flöser
Review request changed

Status: Closed (submitted)

Change Summary:

commit ebba68eebf90f6536dabc7d448aa1be524d7bf28
Author: Martin Gräßlin mgraesslin@kde.org
Date: Fri Nov 4 10:49:48 2016 +0100

Expose desktopFileName in KWindowInfo

This change introduced a new method in KWindowInfo:
QByteArray KWindowInfo::desktopFileName() const

It returns the desktop file name of the application if known on the
given platform. So far only provided on X11 through
NETWinInfo::desktopFileName.
Aleix Pol Gonzalez

It's weird that it's called desktopFileName but it doesn't offer a file name? I think it should be renamed or fixed to provide the path. In the end it's an API that then requires the user to do a rather big look-up.

  1. The naming follows the naming in: NETWinInfo, KAboutData and QGuiApplication. Naming it differently would be wrong IMHO.

Loading...