[Differential] [Requested Changes To] D3196: Fix highlighting for problems with incomplete finalLocation

kfunk (Kevin Funk) noreply at phabricator.kde.org
Wed Nov 2 09:51:53 UTC 2016


kfunk requested changes to this revision.
kfunk added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> problem.cpp:115
> +{
> +    m_finalLocationMode = mode;
> +}

Here: Look at `Problem::finalLocation` and `Problem::setFinalLocation` for inspiration.

> problem.h:241
> +
> +    IProblem::FinalLocationMode m_finalLocationMode;
>  };

This needs to be in `ProblemData` instead. Sorry for not telling you in my earlier review ;)

REPOSITORY
  rKDEVPLATFORM KDevPlatform

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: antonanikin, #kdevelop, kfunk
Cc: kfunk, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20161102/2f7a592f/attachment.html>


More information about the KDevelop-devel mailing list