D8215: Added support for STRINGS property in CMake cache editor

Kevin Funk noreply at phabricator.kde.org
Mon Oct 9 15:34:42 UTC 2017


kfunk added a comment.


  So in general I think this code looks good (apart from the bit of code duplication for handling "STRINGS" in cmakecachemodel.cpp). But it'd need to be refactored afterwards...

REPOSITORY
  R32 KDevelop

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

To: gracicot, #kdevelop, kfunk
Cc: apol, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20171009/e4d1c146/attachment.html>


More information about the KDevelop-devel mailing list