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

Reply via email to