D9271: Fixed memory leak in new_argv

[Nathan Henry]

nathanhenry added inline comments.

INLINE COMMENTS

> davidedmundson wrote in Viewer.cpp:140-143
> that can't be right.

More info?

> davidedmundson wrote in kcm.cpp:83-85
> 1. this doesn't leak
> 
> 2. what's the point of making a manager?

How doesn't this leak when excluding my change?

REPOSITORY
  R119 Plasma Desktop

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

To: nathanhenry, davidedmundson
Cc: anthonyfieroni, davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171212/1e53d4de/attachment.html>