<table><tr><td style="">coliveira removed a reviewer: mwolff.<br />coliveira changed the visibility from "All Users" to "Public (No Login Required)".<br />coliveira updated this revision to Diff 7239.<br />coliveira marked 4 inline comments as done.<br />coliveira added a comment.

</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D2840" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Improved style by preferring camel case identifiers, cleaned wrong comments, changed ClangFormat target to issue a WARNING instead of ERROR in case of clang-format executable not being found, and changed the API on PerProjectConfigPage to issue a signal when the selection of checks change instead of using a QStringList pointer owned by someone else.</p>

<p>I didn't get a clue on how to use KConfigSkeleton in PerProjectConfigPage because the list of checks is generated during runtime, by running clang-tidy and parsing its output.  I had a hard time trying to use this and still have the list of checks not hard-coded inside the plugin code, which I believe it's the right way to do the plugin, since the list of checks may change from one version to another of clang-tidy. Ideas are welcome.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R218 KDev Clang-Tidy Support</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D2840?vs=6885&id=7239" rel="noreferrer">https://phabricator.kde.org/D2840?vs=6885&id=7239</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2840" rel="noreferrer">https://phabricator.kde.org/D2840</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>.clang-tidy<br />

.gitignore<br />

CMakeLists.txt<br />

cmake/ClangFormatAll.cmake<br />

src/config/clangtidypreferences.cpp<br />

src/config/clangtidypreferences.h<br />

src/config/configgroup.cpp<br />

src/config/configgroup.h<br />

src/config/configgroup.h.in<br />

src/config/perprojectconfigpage.cpp<br />

src/config/perprojectconfigpage.h<br />

src/config/ui/clangtidysettings.ui<br />

src/config/ui/perprojectconfig.ui<br />

src/parsers/clangtidyparser.cpp<br />

src/parsers/clangtidyparser.h<br />

src/parsers/replacementparser.cpp<br />

src/parsers/replacementparser.h<br />

src/plugin/job.cpp<br />

src/plugin/job.h<br />

src/plugin/plugin.cpp<br />

src/plugin/plugin.h<br />

src/plugin/plugin.h.in<br />

src/qCDebug/debug.cpp<br />

tests/CMakeLists.txt<br />

tests/data/nihon_plugin.cpp<br />

tests/data/plugin.cpp<br />

tests/test_clangtidyjob.cpp<br />

tests/test_clangtidyjob.h<br />

tests/test_clangtidyparser.cpp<br />

tests/test_clangtidyparser.h<br />

tests/test_plugin.cpp<br />

tests/test_plugin.h</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>coliveira, apol, antonanikin, kfunk<br /><strong>Cc: </strong>kdevelop-devel<br /></div>