broulik added inline comments.

INLINE COMMENTS

> kded.cpp:328
> +{
> +    const QStringList supportedPlatforms = 
> module.rawData().value(QStringLiteral("X-KDE-OnlyShowOnQtPlatforms")).toVariant().toString().split(QStringLiteral(";"),
>  QString::SkipEmptyParts);
> +

KPluginMetaData::readStringList(module.rawData(), 
QStringLiteral("X-KDE-OnlyShowOnQtPlatforms"));

> kded.cpp:330
> +
> +    return supportedPlatforms.count() == 0 || 
> supportedPlatforms.contains(qApp->platformName());
> +}

`isEmpty()`

> kded.h:90
> +     */
> +    bool platformSupportsModule(const KPluginMetaData &module) const;
> +    //@}

static?

> kdedmodule.desktop:111
> +[PropertyDef::X-KDE-PluginInfo-OnlySupportedPlatform]
> +Type=QString

QStringList

REPOSITORY
  R297 KDED

REVISION DETAIL
  https://phabricator.kde.org/D6899

To: mart, #frameworks, #plasma
Cc: broulik, #frameworks

Reply via email to