D7219: Add separate interface library KDev::Execute for execute/iexecuteplugin.h
Kevin Funk
noreply at phabricator.kde.org
Wed Aug 9 19:10:07 UTC 2017
kfunk added a comment.
In https://phabricator.kde.org/D7219#134140, @kossebau wrote:
> In https://phabricator.kde.org/D7219#134137, @kfunk wrote:
>
> > Rest LGTM,
> >
> > This is pretty similar to what we did to `IMakeBuilder` before.
>
>
> Ah, had not seen, okay, so following a pattern :) Makes me wonder if KDevPlatformExecute then should get some target export, as well right? And perhaps then be named KDevPlatformIExecute (_I_), to indicate the interface as well here?
+1 on both, yep.
REPOSITORY
R32 KDevelop
REVISION DETAIL
https://phabricator.kde.org/D7219
To: kossebau, #kdevelop
Cc: kfunk, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170809/1b8560d0/attachment.html>
More information about the KDevelop-devel
mailing list