Merge pull request #65 from christianrauch/workflow_dispatch

update runner and workflow dispatch

.github/workflows/cmake.yml

@@ -5,6 +5,7 @@ on:
     branches: [ master ]
   pull_request:
     branches: [ master ]
+  workflow_dispatch:
 
 env:
   BUILD_TYPE: Release
@@ -17,14 +18,14 @@ jobs:
       fail-fast: false
       matrix:
         config:
-          - { name: "Linux", os: ubuntu-latest, test: ON }
-          - { name: "Windows", os: windows-latest, test: OFF }
-          - { name: "macOS", os: macos-latest, test: OFF }
+          - { name: "Linux", os: ubuntu-24.04, test: ON }
+          - { name: "Windows", os: windows-2025, test: OFF }
+          - { name: "macOS", os: macos-15, test: OFF }
 
     steps:
     - name: install Linux dependencies
       if: runner.os == 'Linux'
-      run: sudo apt install -y --no-install-recommends ninja-build libasio-dev clang-format-12 libgtest-dev
+      run: sudo apt install -y --no-install-recommends ninja-build libasio-dev clang-format-18 libgtest-dev
 
     - name: install Microsoft Visual C++ (Windows)
       uses: ilammy/msvc-dev-cmd@v1
@@ -43,7 +44,7 @@ jobs:
       if: runner.os == 'macOS'
       run: brew install ninja asio
 
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v4
 
     - name: Clang-Format Style Check
       if: runner.os == 'Linux'

CMakeLists.txt

@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 3.5)
+cmake_minimum_required(VERSION 3.16)
 project(msp VERSION 3.4.0 LANGUAGES CXX)
 
 set(PROJECT_DESCRIPTION "Implementation of the MultiWii Serial Protocol (MSP) for MultiWii and Cleanflight flight controller")

format_source_files.sh

@@ -1,2 +1,2 @@
 #!/bin/sh
-find . -regex '.*\.\(cpp\|hpp\)' -exec clang-format-12 -style=file -i {} \;
+find . -regex '.*\.\(cpp\|hpp\)' -exec clang-format-18 -style=file -i {} \;

inc/msp/ByteVector.hpp

@@ -272,7 +272,7 @@ class ByteVector : public std::vector<uint8_t> {
         if(count > unpacking_remaining()) return false;
         bool rc = true;
         val.clear();
-        int8_t tmp;
+        int8_t tmp = {};
         for(size_t i = 0; i < count; ++i) {
             rc &= unpack(tmp);
             val += tmp;

inc/msp/Value.hpp

@@ -1,6 +1,7 @@
 #ifndef VALUE_HPP
 #define VALUE_HPP
 
+#include <cstdint>
 #include <iostream>
 #include <utility>
 

inc/msp/msp_msg.hpp

@@ -3320,7 +3320,7 @@ struct ActiveBoxes : public Message {
                     if(box_conf() & (1 << (iaux * 3 + ip)))
                         aux_sp[iaux].insert(SwitchPosition(ip));
                 }  // each position (L,M,H)
-            }      // each aux switch
+            }  // each aux switch
             box_pattern.push_back(aux_sp);
         }  // each box
         return rc;