dfaure accepted this revision.
dfaure added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> file_unix.cpp:677
> +    if (metaData(QStringLiteral("UnitTesting")) == QLatin1String("true")) {
> +        const QString metaData = execAction.name() + ","
> +                               + QString::number(execAction.isValid()) + ","

single-quotes for the commas (','), like in my earlier recommendation.

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

To: chinmoyr, dfaure, #frameworks
Cc: #frameworks

Reply via email to