davidedmundson added inline comments.

INLINE COMMENTS

> kprocesstest_helper.cpp:17
>  {
> +    Q_UNUSED(argc)
>      KProcess p;

If we use argv[1]   we very much should be using argc beforehand

REPOSITORY
  R244 KCoreAddons

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

To: meven, #frameworks
Cc: davidedmundson, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, 
ngraham, bruns

Reply via email to