Add field for setting default CMake executable in builder's configuration page
ClosedPublic

Authored by arrowd on Sep 30 2015, 6:29 AM.

Details

Summary

Add field for setting default CMake executable in builder's configuration page

Diff Detail

Repository
R32 KDevelop
Lint
Lint Skipped
Unit
Unit Tests Skipped
arrowd updated this revision to Diff 936.Sep 30 2015, 6:29 AM
arrowd retitled this revision from to Add field for setting default CMake executable in builder's configuration page.
arrowd updated this object.
arrowd edited the test plan for this revision. (Show Details)
arrowd added a reviewer: KDevelop.
arrowd set the repository for this revision to R32 KDevelop.
arrowd added a project: KDevelop.
Restricted Application added a subscriber: kdevelop-devel. ยท View Herald TranscriptSep 30 2015, 6:29 AM
mwolff accepted this revision.Oct 8 2015, 9:26 AM
mwolff added a reviewer: mwolff.
mwolff added a subscriber: mwolff.

lgtm, but using a lambda is even better I think

projectbuilders/cmakebuilder/cmakebuilderpreferences.cpp
54

I suggest using a lambda here

This revision is now accepted and ready to land.Oct 8 2015, 9:26 AM