[Differential] [Changed Subscribers] D1726: Introduce a test server

bshah (Bhushan Shah) noreply at phabricator.kde.org
Tue May 31 14:12:02 UTC 2016


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160531/fbe10511/attachment.html>


More information about the Plasma-devel mailing list