From b162aee497ef9be3730f59bb0c9d974171bfebee Mon Sep 17 00:00:00 2001 From: NRK Date: Thu, 16 Jun 2022 13:39:03 +0600 Subject: sort and group includes * includes are sorted alphabetically * their grouping and layout is the following: - nsxiv.h will be the first include - followed by any internal headers (e.g "commands.h" "config.h") - followed by system headers ( etc) - followed by third party headers (X.h libwebp etc) * also add `llvm-include-order` check to clang-tidy so that it can catch unsorted includes during CI. --- .woodpecker/clang-tidy-checks | 1 + autoreload.c | 2 +- commands.c | 8 ++++---- main.c | 19 ++++++++++--------- nsxiv.h | 3 ++- options.c | 2 +- util.c | 4 ++-- window.c | 5 +++-- 8 files changed, 24 insertions(+), 20 deletions(-) diff --git a/.woodpecker/clang-tidy-checks b/.woodpecker/clang-tidy-checks index 67394d4..3c6c62e 100644 --- a/.woodpecker/clang-tidy-checks +++ b/.woodpecker/clang-tidy-checks @@ -1,6 +1,7 @@ # checks clang-analyzer-*,clang-diagnostic-*,bugprone-*,performance-*,modernize-* misc-*,readability-*,android-cloexec-*,cert-* +llvm-include-order # silence -misc-unused-parameters diff --git a/autoreload.c b/autoreload.c index 47751d5..c4b5665 100644 --- a/autoreload.c +++ b/autoreload.c @@ -23,8 +23,8 @@ #include #include #include -#include #include +#include static union { char d[4096]; /* aligned buffer */ diff --git a/commands.c b/commands.c index 8a73462..8741a26 100644 --- a/commands.c +++ b/commands.c @@ -19,13 +19,13 @@ #include "nsxiv.h" -#include +#include "commands.h" + #include +#include #include -#include #include - -#include "commands.h" +#include extern img_t img; extern tns_t tns; diff --git a/main.c b/main.c index 10f7440..a6c4187 100644 --- a/main.c +++ b/main.c @@ -22,22 +22,23 @@ #include "commands.h" #include "config.h" -#include -#include -#include -#include -#include -#include #include +#include #include -#include #include +#include +#include +#include +#include +#include #include +#include #include #include -#include -#include +#include + #include +#include #define MODMASK(mask) ((mask) & USED_MODMASK) #define BAR_SEP " " diff --git a/nsxiv.h b/nsxiv.h index eb06328..7d81fff 100644 --- a/nsxiv.h +++ b/nsxiv.h @@ -20,8 +20,9 @@ #ifndef NSXIV_H #define NSXIV_H -#include #include +#include + #include #include diff --git a/options.c b/options.c index 8fade87..69b45b9 100644 --- a/options.c +++ b/options.c @@ -20,8 +20,8 @@ #include "nsxiv.h" #include "version.h" -#include #include +#include #include #include diff --git a/util.c b/util.c index 31ab4b8..65c39e5 100644 --- a/util.c +++ b/util.c @@ -20,13 +20,13 @@ #include "nsxiv.h" #include -#include +#include #include #include +#include #include #include #include -#include const char *progname; diff --git a/window.c b/window.c index 3cfa188..f472474 100644 --- a/window.c +++ b/window.c @@ -22,13 +22,14 @@ #include "config.h" #include "icon/data.h" +#include #include #include -#include #include -#include + #include #include +#include #if HAVE_LIBFONTS #include "utf8.h" -- cgit v1.2.3-54-g00ecf