clang-format src/libAtomVM/*.c and src/libAtomVM/*.h

Run recent clang-format in order to apply other formatting rules that
are not IndentPPDirectives.

Also upgrade to clang-format 20.

Signed-off-by: Davide Bettio <[email protected]>

Author: bettio

.github/workflows/check-formatting.yaml

@@ -36,11 +36,8 @@ jobs:
 
     - name: "Install run-clang-format"
       run: |
-        wget -q -O - https://apt.llvm.org/llvm-snapshot.gpg.key | sudo apt-key add -
-        echo "deb http://apt.llvm.org/focal/ llvm-toolchain-focal-16 main" | sudo tee -a /etc/apt/sources.list
-        echo "deb-src http://apt.llvm.org/focal/ llvm-toolchain-focal-16 main" | sudo tee -a /etc/apt/sources.list
         sudo apt-get update
-        sudo apt-get install -y clang-format-16
+        sudo apt-get install -y clang-format-20
         curl -sSfL https://raw.githubusercontent.com/Sarcasm/run-clang-format/master/run-clang-format.py -o run-clang-format
         chmod +x run-clang-format
 

src/libAtomVM/atom_table.c

@@ -427,7 +427,7 @@ static inline int read_encoded_len(const uint8_t **len_bytes)
 }
 
 // -1 is not a valid atom index as we're limited to 2^20
-#define ATOM_TABLE_NOT_FOUND_MARKER ((atom_index_t) -1)
+#define ATOM_TABLE_NOT_FOUND_MARKER ((atom_index_t) - 1)
 
 enum AtomTableEnsureAtomResult atom_table_ensure_atoms(struct AtomTable *table, const void *atoms, size_t count,
     atom_index_t *translate_table, enum EnsureAtomsOpt opt)

src/libAtomVM/bitstring.c

@@ -26,7 +26,14 @@
 
 static inline uint64_t from_le64(uint64_t value)
 {
-    return ((((value) &0xFF) << 56) | (((value) &0xFF00) << 40) | (((value) &0xFF0000) << 24) | (((value) &0xFF000000) << 8) | (((value) &0xFF00000000) >> 8) | (((value) &0xFF0000000000) >> 24) | (((value) &0xFF000000000000) >> 40) | (((value) &0xFF00000000000000) >> 56));
+    return ((((value) & 0xFF) << 56)
+        | (((value) & 0xFF00) << 40)
+        | (((value) & 0xFF0000) << 24)
+        | (((value) & 0xFF000000) << 8)
+        | (((value) & 0xFF00000000) >> 8)
+        | (((value) & 0xFF0000000000) >> 24)
+        | (((value) & 0xFF000000000000) >> 40)
+        | (((value) & 0xFF00000000000000) >> 56));
 }
 
 bool bitstring_extract_any_integer(const uint8_t *src, size_t offset, avm_int_t n,

src/libAtomVM/nifs.c

@@ -4121,7 +4121,9 @@ static term nif_erlang_fun_info_2(Context *ctx, int argc, term argv[])
             RAISE_ERROR(BADARG_ATOM);
     }
 
-    if (UNLIKELY(memory_ensure_free_with_roots(ctx, TUPLE_SIZE(2), 2, (term[]){ key, value }, MEMORY_CAN_SHRINK) != MEMORY_GC_OK)) {
+    if (UNLIKELY(memory_ensure_free_with_roots(
+                     ctx, TUPLE_SIZE(2), 2, (term[]) { key, value }, MEMORY_CAN_SHRINK)
+            != MEMORY_GC_OK)) {
         RAISE_ERROR(OUT_OF_MEMORY_ATOM);
     }
     term fun_info_tuple = term_alloc_tuple(2, &ctx->heap);

src/libAtomVM/term.h

@@ -1807,7 +1807,7 @@ static inline bool term_is_nomatch_binary_pos_len(BinaryPosLen pos_len)
 
 static inline BinaryPosLen term_nomatch_binary_pos_len(void)
 {
-    return (BinaryPosLen){ .pos = -1, .len = -1 };
+    return (BinaryPosLen) { .pos = -1, .len = -1 };
 }
 
 /**

src/libAtomVM/utils.h

@@ -67,8 +67,15 @@ extern "C" {
             *((uint16_t *) (ptr)) = __builtin_bswap16(val)
 
 #    else
-#        define READ_64_UNALIGNED(ptr) \
-            ((((uint64_t) ((uint8_t *) (ptr))[0]) << 56) | (((uint64_t) ((uint8_t *) (ptr))[1]) << 48) | (((uint64_t) ((uint8_t *) (ptr))[2]) << 40) | (((uint64_t) ((uint8_t *) (ptr))[3]) << 32) | (((uint64_t) ((uint8_t *) (ptr))[4]) << 24) | (((uint64_t) ((uint8_t *) (ptr))[5]) << 16) | (((uint64_t) ((uint8_t *) (ptr))[6]) << 8) | (((uint64_t) ((uint8_t *) (ptr))[7])))
+#        define READ_64_UNALIGNED(ptr)                        \
+            ((((uint64_t) ((uint8_t *) (ptr))[0]) << 56)      \
+                | (((uint64_t) ((uint8_t *) (ptr))[1]) << 48) \
+                | (((uint64_t) ((uint8_t *) (ptr))[2]) << 40) \
+                | (((uint64_t) ((uint8_t *) (ptr))[3]) << 32) \
+                | (((uint64_t) ((uint8_t *) (ptr))[4]) << 24) \
+                | (((uint64_t) ((uint8_t *) (ptr))[5]) << 16) \
+                | (((uint64_t) ((uint8_t *) (ptr))[6]) << 8)  \
+                | (((uint64_t) ((uint8_t *) (ptr))[7])))
 
 #        define WRITE_64_UNALIGNED(ptr, val)                              \
             {                                                             \
@@ -82,8 +89,9 @@ extern "C" {
                 ((uint8_t *) (ptr))[7] = ((uint64_t) val) & 0xff;         \
             }
 
-#        define READ_32_UNALIGNED(ptr) \
-            ((((uint8_t *) (ptr))[0] << 24) | (((uint8_t *) (ptr))[1] << 16) | (((uint8_t *) (ptr))[2] << 8) | ((uint8_t *) (ptr))[3])
+#        define READ_32_UNALIGNED(ptr)                                       \
+            ((((uint8_t *) (ptr))[0] << 24) | (((uint8_t *) (ptr))[1] << 16) \
+                | (((uint8_t *) (ptr))[2] << 8) | ((uint8_t *) (ptr))[3])
 
 #        define WRITE_32_UNALIGNED(ptr, val)                              \
             {                                                             \
@@ -106,7 +114,9 @@ extern "C" {
 #    ifdef __GNUC__
 #        define ENDIAN_SWAP_32(value) __builtin_bswap32(value)
 #    else
-#        define ENDIAN_SWAP_32(value) ((((value) & 0xFF) << 24) | (((value) & 0xFF00) << 8) | (((value) & 0xFF0000) >> 8) | (((value) & 0xFF000000) >> 24))
+#        define ENDIAN_SWAP_32(value)                                                           \
+            ((((value) & 0xFF) << 24) | (((value) & 0xFF00) << 8) | (((value) & 0xFF0000) >> 8) \
+                | (((value) & 0xFF000000) >> 24))
 #    endif
 
 #    ifdef __GNUC__
@@ -593,7 +603,7 @@ static inline bool uint64_does_overflow_int64(uint64_t u64, bool is_negative)
  */
 static inline uint32_t int32_safe_unsigned_abs(int32_t i32)
 {
-    return (i32 < 0) ? ((uint32_t) - (i32 + 1)) + 1 : (uint32_t) i32;
+    return (i32 < 0) ? ((uint32_t) -(i32 + 1)) + 1 : (uint32_t) i32;
 }
 
 /**
@@ -611,7 +621,7 @@ static inline uint32_t int32_safe_unsigned_abs(int32_t i32)
  */
 static inline uint64_t int64_safe_unsigned_abs(int64_t i64)
 {
-    return (i64 < 0) ? ((uint64_t) - (i64 + 1)) + 1 : (uint64_t) i64;
+    return (i64 < 0) ? ((uint64_t) -(i64 + 1)) + 1 : (uint64_t) i64;
 }
 
 /**

src/platforms/.clang-format

@@ -1,2 +1,7 @@
+# Copyright 2025 Davide Bettio <[email protected]>
+#
+# SPDX-License-Identifier: CC0-1.0
+
+---
 BasedOnStyle: InheritParentConfig
 IndentPPDirectives: None