<div>rakuco added inline comments.</div><br /><div><strong>INLINE COMMENTS</strong><div><div>CMakeLists.txt:25-30 I think you need to merge these two `add_feature_info()` calls: as-is, either one or the other will always be shown in the "disabled features" list, and I guess you only want this to happen if neither call/header is found. Perhaps something like this:<br />
<br />
```<br />
check_include_file(...)<br />
check_symbol_exists(...)<br />
# ...<br />
if (HAVE_PR_SET_DUMPABLE OR HAVE_PROC_TRACE_CTL)<br />
  set(CAN_DISABLE_PTRACE TRUE)<br />
endif ()<br />
add_feature_info("prctl/procctl tracing control"<br />
                 CAN_DISABLE_PTRACE<br />
                 "Required for disallowing ptrace on greeter and kcheckpass processes.")<br />
```<br />
</div><div>greeter/main.cpp:67 There's an extra space before the `if` here and in a few other places.</div><div>greeter/main.cpp:140-141 The indentation here looks wrong.</div><div>kcheckpass/kcheckpass.c:339-340 The indentation looks wrong here too.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKSCREENLOCKER KScreenLocker</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1216" rel="noreferrer">https://phabricator.kde.org/D1216</a></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>tcberner, graesslin, rakuco<br /><strong>Cc: </strong>plasma-devel<br /></div>