nits: abort() and #include (!1305) · Merge requests · Knot projects / Knot Resolver · GitLab

Snippets Groups Projects

Self sign-up has been disabled due to increased spam activity. If you want to get access, please send an email to a project owner (preferred) or at gitlab(at)nic(dot)cz. We apologize for the inconvenience.

Merged Vladimír Čunát requested to merge nits into master 2 years ago

Jun 03, 2022

drop unused #include lines · 9131caf9

Vladimír Čunát authored 2 years ago

https://clangd.llvm.org/design/include-cleaner
Though somehow I'm all the time getting false positives for
"daemon/bindings/impl.h"

9131caf9

replace some occurrences of abort() by kr_require() · 246a7ae1
Vladimír Čunát authored 2 years ago
```
It provides more information and the condition is typically
easier to read, too.
```
Verified

246a7ae1