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