Enforce project wide #include order

.clang-format

@@ -7,3 +7,18 @@ BasedOnStyle: LLVM
 # Disable automatic line-breaks in comments
 # as this breaks SPDX headers
 ReflowComments: false
+
+# Regroup #include directives to villas - std - other
+IncludeBlocks: Regroup
+IncludeIsMainRegex: / # disable main header heuristic
+IncludeCategories:
+  - Regex: '^<villas/'
+    Priority: 3
+    CaseSensitive: true
+  - Regex: '^<[[:lower:]_]+>$'
+    Priority: 1
+    CaseSensitive: true
+  - Regex: '^<.*>$'
+    Priority: 2
+  - Regex: '^".*"$'
+    Priority: 4

clients/opal-rt/rtlab-asyncip/models/send_receive/src/socket.c

@@ -19,7 +19,6 @@
 #define RTLAB
 #include "AsyncApi.h"
 #include "OpalPrint.h"
-
 #include "config.h"
 #include "socket.h"
 

clients/opal-rt/rtlab-asyncip/models/send_receive/src/utils.c

@@ -14,7 +14,6 @@
  * to the OpalDisplay. Otherwise stdout will be used. */
 #define RTLAB
 #include "OpalPrint.h"
-
 #include "config.h"
 #include "utils.h"
 

common/include/villas/buffer.hpp

@@ -7,9 +7,8 @@
 
 #pragma once
 
-#include <vector>
-
 #include <cstdlib>
+#include <vector>
 
 #include <jansson.h>
 

common/include/villas/cpuset.hpp

@@ -10,6 +10,7 @@
 #ifdef __linux__
 
 #include <cstdint>
+
 #include <sched.h>
 
 #include <villas/exceptions.hpp>

common/include/villas/dsp/moving_average_window.hpp

@@ -8,6 +8,7 @@
 #pragma once
 
 #include <cstddef>
+
 #include <villas/dsp/window.hpp>
 
 namespace villas {

common/include/villas/dsp/window_cosine.hpp

@@ -8,7 +8,6 @@
 #pragma once
 
 #include <cmath>
-
 #include <vector>
 
 #include <villas/dsp/window.hpp>

common/include/villas/graph/edge.hpp

@@ -8,6 +8,7 @@
 #pragma once
 
 #include <fmt/ostream.h>
+
 #include <villas/config.hpp>
 #include <villas/graph/vertex.hpp>
 

common/include/villas/graph/vertex.hpp

@@ -7,9 +7,11 @@
 
 #pragma once
 
-#include <fmt/ostream.h>
 #include <list>
 #include <sstream>
+
+#include <fmt/ostream.h>
+
 #include <villas/config.hpp>
 
 namespace villas {

common/include/villas/kernel/devices/device.hpp

@@ -10,6 +10,7 @@
 
 #include <filesystem>
 #include <optional>
+
 #include <villas/kernel/devices/driver.hpp>
 
 namespace villas {

common/include/villas/kernel/devices/driver.hpp

@@ -9,6 +9,8 @@
 
 #pragma once
 
+#include <string>
+
 namespace villas {
 namespace kernel {
 namespace devices {

common/include/villas/kernel/devices/ip_device.hpp

@@ -9,6 +9,8 @@
 #pragma once
 
 #include <filesystem>
+#include <vector>
+
 #include <villas/kernel/devices/platform_device.hpp>
 
 namespace villas {

common/include/villas/kernel/devices/linux_driver.hpp

@@ -11,6 +11,7 @@
 #include <filesystem>
 #include <fstream>
 #include <iostream>
+
 #include <villas/kernel/devices/driver.hpp>
 
 namespace villas {

common/include/villas/kernel/devices/pci_device.hpp

@@ -7,11 +7,10 @@
 
 #pragma once
 
-#include <fstream>
-#include <list>
-
 #include <cstddef>
 #include <cstdint>
+#include <fstream>
+#include <list>
 
 #include <villas/log.hpp>
 

common/include/villas/kernel/devices/platform_device.hpp

@@ -9,6 +9,7 @@
 #pragma once
 
 #include <filesystem>
+
 #include <villas/kernel/devices/device.hpp>
 #include <villas/kernel/devices/driver.hpp>
 

common/include/villas/list.hpp

@@ -14,8 +14,9 @@
 #pragma once
 
 #include <cstring>
-#include <pthread.h>
 #include <string>
+
+#include <pthread.h>
 #include <sys/types.h>
 
 #include <villas/common.hpp>

common/include/villas/log.hpp

@@ -11,12 +11,11 @@
 #include <list>
 #include <string>
 
+#include <jansson.h>
 #include <spdlog/sinks/dist_sink.h>
 #include <spdlog/sinks/stdout_color_sinks.h>
 #include <spdlog/spdlog.h>
 
-#include <jansson.h>
-
 namespace villas {
 
 // Forward declarations

common/include/villas/memory.hpp

@@ -8,6 +8,7 @@
 #pragma once
 
 #include <string>
+
 #include <unistd.h>
 
 #include <villas/log.hpp>

common/include/villas/memory_manager.hpp

@@ -8,11 +8,13 @@
 #pragma once
 
 #include <cstdint>
-#include <fmt/ostream.h>
 #include <map>
 #include <stdexcept>
 #include <string>
+
+#include <fmt/ostream.h>
 #include <unistd.h>
+
 #include <villas/config.hpp>
 #include <villas/graph/directed.hpp>
 #include <villas/log.hpp>

common/include/villas/plugin.hpp

@@ -8,11 +8,13 @@
 
 #pragma once
 
-#include <fmt/ostream.h>
 #include <iostream>
-#include <jansson.h>
 #include <list>
 #include <string>
+
+#include <fmt/ostream.h>
+#include <jansson.h>
+
 #include <villas/common.hpp>
 #include <villas/config.hpp>
 #include <villas/log.hpp>

common/include/villas/popen.hpp

@@ -7,17 +7,16 @@
 
 #pragma once
 
-#include <signal.h>
-
-#include <ext/stdio_filebuf.h>
-
 #include <istream>
 #include <map>
 #include <memory>
 #include <ostream>
 #include <string>
 #include <vector>
 
+#include <ext/stdio_filebuf.h>
+#include <signal.h>
+
 namespace villas {
 namespace utils {
 

common/include/villas/task.hpp

@@ -9,7 +9,6 @@
 
 #include <cstdint>
 #include <cstdio>
-
 #include <ctime>
 
 namespace villas {

common/include/villas/timing.hpp

@@ -9,7 +9,6 @@
 
 #include <cstdint>
 #include <cstdio>
-
 #include <ctime>
 
 // Compare two timestamps. Return zero if they are equal.

common/include/villas/utils.hpp

@@ -8,20 +8,19 @@
 
 #pragma once
 
+#include <cassert>
+#include <cstdint>
+#include <cstdlib>
 #include <filesystem>
 #include <list>
 #include <string>
 #include <vector>
 
-#include <cassert>
-#include <cstdint>
-#include <cstdlib>
+#include <openssl/sha.h>
 #include <sched.h>
 #include <signal.h>
 #include <sys/types.h>
 
-#include <openssl/sha.h>
-
 #include <villas/config.hpp>
 
 #ifdef __GNUC__

common/lib/common.cpp

@@ -5,10 +5,10 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-#include <villas/common.hpp>
-
 #include <cstdlib>
 
+#include <villas/common.hpp>
+
 std::string stateToString(enum State s) {
   switch (s) {
   case State::DESTROYED:

common/lib/compat.cpp

@@ -6,6 +6,7 @@
  */
 
 #include <cstring>
+
 #include <jansson.h>
 #include <unistd.h>
 

common/lib/dsp/pid.cpp

@@ -7,6 +7,7 @@
 
 #include <cmath>
 #include <iostream>
+
 #include <villas/dsp/pid.hpp>
 
 using namespace std;

common/lib/kernel/devices/device_connection.cpp

@@ -6,10 +6,9 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-#include <villas/kernel/devices/device_connection.hpp>
-
 #include <memory>
 
+#include <villas/kernel/devices/device_connection.hpp>
 #include <villas/kernel/devices/linux_driver.hpp>
 #include <villas/memory_manager.hpp>
 

common/lib/kernel/devices/linux_driver.cpp

@@ -6,9 +6,8 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 
-#include <villas/kernel/devices/linux_driver.hpp>
-
 #include <villas/kernel/devices/device.hpp>
+#include <villas/kernel/devices/linux_driver.hpp>
 #include <villas/utils.hpp>
 
 using villas::kernel::devices::Device, villas::kernel::devices::LinuxDriver;

common/lib/kernel/devices/pci_device.cpp

@@ -8,6 +8,7 @@
  */
 
 #include <cstring>
+
 #include <dirent.h>
 #include <fcntl.h>
 #include <libgen.h>

common/lib/kernel/kernel.cpp

@@ -9,21 +9,19 @@
 #include <cstdio>
 #include <cstring>
 #include <ctime>
-#include <fcntl.h>
-#include <sys/utsname.h>
-#include <unistd.h>
 
+#include <fcntl.h>
 #include <sys/types.h>
+#include <sys/utsname.h>
 #include <sys/wait.h>
+#include <unistd.h>
 
 #include <villas/config.hpp>
+#include <villas/exceptions.hpp>
 #include <villas/kernel/kernel.hpp>
 #include <villas/log.hpp>
 #include <villas/utils.hpp>
 
-#include <villas/exceptions.hpp>
-#include <villas/kernel/kernel.hpp>
-
 using namespace villas;
 using namespace villas::utils;
 

common/lib/kernel/rt.cpp

@@ -11,11 +11,10 @@
 #include <villas/config.hpp>
 #include <villas/cpuset.hpp>
 #include <villas/exceptions.hpp>
-#include <villas/log.hpp>
-#include <villas/utils.hpp>
-
 #include <villas/kernel/kernel.hpp>
 #include <villas/kernel/rt.hpp>
+#include <villas/log.hpp>
+#include <villas/utils.hpp>
 
 #ifdef __linux__
 using villas::utils::CpuSet;

common/lib/kernel/vfio_container.cpp

@@ -18,14 +18,13 @@
 
 #include <algorithm>
 #include <array>
+#include <cstdint>
+#include <cstdlib>
+#include <cstring>
 #include <limits>
 #include <sstream>
 #include <string>
 
-#include <cstdlib>
-#include <cstring>
-
-#include <cstdint>
 #include <fcntl.h>
 #include <linux/pci_regs.h>
 #include <sys/eventfd.h>