增加代码格式化与警告检查, 重新格式化全部代码

Co-authored-by: wenxuanjun <41050170+wenxuanjun@users.noreply.github.com>

Author: xiaoyi1212

.clang-format

@@ -12,7 +12,7 @@ AlignArrayOfStructures: Left
 AlignConsecutiveAssignments: Consecutive
 AlignConsecutiveBitFields: true
 AlignConsecutiveDeclarations: None
-AlignConsecutiveMacros: None
+AlignConsecutiveMacros: true
 AlignEscapedNewlines: Right
 AlignOperands: Align
 AlignTrailingComments: true
@@ -26,8 +26,8 @@ AllowShortLoopsOnASingleLine: false
 AttributeMacros:
   - __attribute__
   - __declspec
-BinPackArguments: true
-BinPackParameters: true
+BinPackArguments: false
+BinPackParameters: false
 BraceWrapping:
   AfterCaseLabel: false
   AfterControlStatement: Never

.clang-tidy

@@ -13,13 +13,14 @@ Checks: >
   -readability-implicit-bool-conversion,
   -readability-magic-numbers,
   -readability-math-missing-parentheses,
-  -readability-uppercase-literal-suffix
+  -readability-uppercase-literal-suffix,
+  -bugprone-multi-level-implicit-pointer-conversion,
+  -readability-function-cognitive-complexity,
+  -clang-analyzer-security.insecureAPI.DeprecatedOrUnsafeBufferHandling
 
 FormatStyle: file
 
 CheckOptions:
-  - key: readability-braces-around-statements.ShortStatementLines
-    value: "0"
   - key: readability-function-size.LineThreshold
     value: "120"
   - key: readability-function-size.StatementThreshold
@@ -30,3 +31,4 @@ CheckOptions:
     value: "4"
   - key: readability-function-size.VariableThreshold
     value: "30"
+...

CMakeLists.txt

@@ -97,14 +97,27 @@ endif ()
 
 find_program(CPPCHECK cppcheck)
 if (CPPCHECK)
+    set(CPPCHECK_ARCH_DEFS "")
+    if (TARGET_ARCH STREQUAL "x86_64")
+        set(CPPCHECK_ARCH_DEFS -D__x86_64__=1)
+    elseif (TARGET_ARCH STREQUAL "riscv64")
+        set(CPPCHECK_ARCH_DEFS -D__riscv=1 -D__riscv_xlen=64)
+    elseif (TARGET_ARCH STREQUAL "loongarch64")
+        set(CPPCHECK_ARCH_DEFS -D__loongarch__=1 -D__loongarch_grlen=64)
+    endif ()
     add_custom_target(cppcheck
             COMMAND ${CPPCHECK}
             --project=${CMAKE_BINARY_DIR}/compile_commands.json
             --enable=warning,style,performance,portability
+            -D__GNUC__=1
+            -DSIZE_MAX=0xFFFFFFFFFFFFFFFFUL
+            ${CPPCHECK_ARCH_DEFS}
             --suppress=missingIncludeSystem
             --suppress=*:${CMAKE_SOURCE_DIR}/module/fatfs/*
             --suppress=*:${CMAKE_SOURCE_DIR}/module/iso9660/*
             --suppress=*:${CMAKE_SOURCE_DIR}/src/term/flanterm/*
+            --suppress=*:${CMAKE_SOURCE_DIR}/src/lib/*
+            --suppress=*:${CMAKE_SOURCE_DIR}/src/include/lib/*
             --quiet
             WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}
             COMMENT "Running cppcheck..."

format.sh

@@ -1,8 +1,8 @@
 #!/usr/bin/sh
 
 find src/ libs/ \
-    -path "src/lib/acpica" -prune -o \
-    -path "src/include/lib/acpica" -prune -o \
+    -path "src/lib" -prune -o \
+    -path "src/include/lib" -prune -o \
     -type f \( -name "*.c" -o -name "*.h" \) \
     -exec clang-format -i {} +
 

module/all_include/fs_subsystem.h

@@ -135,7 +135,7 @@ struct vfs_node {           // vfs节点
     uint16_t   fsid;        // 文件系统挂载 id
     void      *handle;      // 操作文件的句柄
     uint64_t   flags;       // 文件标志
-    list_t     child;       // 子节点
+    list_t    *child;       // 子节点
     vfs_node_t root;        // 根目录
     bool       visited;     // 是否与具体文件系统同步
     bool       is_mount;    // 是否是挂载点