bshah added inline comments. INLINE COMMENTS
> main.cpp:30-33 > + const auto arguments = parser.positionalArguments(); > + if (arguments.size() != 1) { > + return 1; > + } It might be possible that one want to use executable with command line option... lets not restrict it to 1? and have >= 1? REPOSITORY rKWAYLAND KWayland REVISION DETAIL https://phabricator.kde.org/D1726 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: graesslin, Plasma Cc: bshah, plasma-devel, sebas
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel