D4883: Add new constructor for KDevelop::DetectedProblem class

Kevin Funk noreply at phabricator.kde.org
Sun Mar 12 22:31:20 UTC 2017


kfunk accepted this revision.
kfunk added a comment.


  LGTM.
  
  @mwolff: ACK?

INLINE COMMENTS

> problem.cpp:80
>      {
>      case Unknown: s = i18n("Unknown"); break;
>      case Disk: s = i18n("Disk"); break;

Side note: We should just do `return ...` in each case here...

REPOSITORY
  R33 KDevPlatform

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

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


More information about the KDevelop-devel mailing list