clang-format 18 (#469)

see: awslabs/aws-c-common#1113

Author: graebm

.github/workflows/clang-format.yml

@@ -5,14 +5,12 @@ on: [push]
 jobs:
   clang-format:
 
-    runs-on: ubuntu-20.04 # latest
+    runs-on: ubuntu-24.04 # latest
 
     steps:
     - name: Checkout Sources
-      uses: actions/checkout@v1
+      uses: actions/checkout@v4
 
     - name: clang-format lint
-      uses: DoozyX/clang-format-lint-action@v0.3.1
-      with:
-        # List of extensions to check
-        extensions: c,h
+      run: |
+        ./format-check.py

format-check.py

@@ -0,0 +1,47 @@
+#!/usr/bin/env python3
+import argparse
+import os
+from pathlib import Path
+import re
+from subprocess import list2cmdline, run
+from tempfile import NamedTemporaryFile
+
+CLANG_FORMAT_VERSION = '18.1.6'
+
+INCLUDE_REGEX = re.compile(
+    r'^(include|source|tests|verification)/.*\.(c|h|inl)$')
+EXCLUDE_REGEX = re.compile(r'^$')
+
+arg_parser = argparse.ArgumentParser(description="Check with clang-format")
+arg_parser.add_argument('-i', '--inplace-edit', action='store_true',
+                        help="Edit files inplace")
+args = arg_parser.parse_args()
+
+os.chdir(Path(__file__).parent)
+
+# create file containing list of all files to format
+filepaths_file = NamedTemporaryFile(delete=False)
+for dirpath, dirnames, filenames in os.walk('.'):
+    for filename in filenames:
+        # our regexes expect filepath to use forward slash
+        filepath = Path(dirpath, filename).as_posix()
+        if not INCLUDE_REGEX.match(filepath):
+            continue
+        if EXCLUDE_REGEX.match(filepath):
+            continue
+
+        filepaths_file.write(f"{filepath}\n".encode())
+filepaths_file.close()
+
+# use pipx to run clang-format from PyPI
+# this is a simple way to run the same clang-format version regardless of OS
+cmd = ['pipx', 'run', f'clang-format=={CLANG_FORMAT_VERSION}',
+       f'--files={filepaths_file.name}']
+if args.inplace_edit:
+    cmd += ['-i']
+else:
+    cmd += ['--Werror', '--dry-run']
+
+print(f"{Path.cwd()}$ {list2cmdline(cmd)}")
+if run(cmd).returncode:
+    exit(1)

format-check.sh

@@ -455,7 +455,10 @@ struct aws_http2_setting {
  * Initializes aws_http_client_connection_options with default values.
  */
 #define AWS_HTTP_CLIENT_CONNECTION_OPTIONS_INIT                                                                        \
-    { .self_size = sizeof(struct aws_http_client_connection_options), .initial_window_size = SIZE_MAX, }
+    {                                                                                                                  \
+        .self_size = sizeof(struct aws_http_client_connection_options),                                                \
+        .initial_window_size = SIZE_MAX,                                                                               \
+    }
 
 AWS_EXTERN_C_BEGIN
 

include/aws/http/connection.h

@@ -50,7 +50,9 @@ struct aws_h2err {
 };
 
 #define AWS_H2ERR_SUCCESS                                                                                              \
-    (struct aws_h2err) { .h2_code = 0, .aws_code = 0 }
+    (struct aws_h2err) {                                                                                               \
+        .h2_code = 0, .aws_code = 0                                                                                    \
+    }
 
 #define AWS_H2_PAYLOAD_MAX (0x00FFFFFF)       /* must fit in 3 bytes */
 #define AWS_H2_WINDOW_UPDATE_MAX (0x7FFFFFFF) /* cannot use high bit */

include/aws/http/private/h2_frames.h

@@ -161,10 +161,10 @@ typedef struct aws_string *(aws_http_proxy_negotiation_get_token_sync_fn)(void *
  * Synchronous (for now) callback function to fetch a token used in modifying CONNECT request.  Includes a (byte string)
  * context intended to be used as part of a challenge-response flow.
  */
-typedef struct aws_string *(aws_http_proxy_negotiation_get_challenge_token_sync_fn)(
-    void *user_data,
-    const struct aws_byte_cursor *challenge_context,
-    int *out_error_code);
+typedef struct aws_string *(
+    aws_http_proxy_negotiation_get_challenge_token_sync_fn)(void *user_data,
+                                                            const struct aws_byte_cursor *challenge_context,
+                                                            int *out_error_code);
 
 /**
  * Proxy negotiation logic must call this function to indicate an unsuccessful outcome
@@ -307,9 +307,9 @@ struct aws_http_proxy_negotiator {
 
 /*********************************************************************************************/
 
-typedef struct aws_http_proxy_negotiator *(aws_http_proxy_strategy_create_negotiator_fn)(
-    struct aws_http_proxy_strategy *proxy_strategy,
-    struct aws_allocator *allocator);
+typedef struct aws_http_proxy_negotiator *(
+    aws_http_proxy_strategy_create_negotiator_fn)(struct aws_http_proxy_strategy *proxy_strategy,
+                                                  struct aws_allocator *allocator);
 
 struct aws_http_proxy_strategy_vtable {
     aws_http_proxy_strategy_create_negotiator_fn *create_negotiator;

include/aws/http/proxy.h

@@ -501,7 +501,9 @@ struct aws_http2_stream_write_data_options {
 };
 
 #define AWS_HTTP_REQUEST_HANDLER_OPTIONS_INIT                                                                          \
-    { .self_size = sizeof(struct aws_http_request_handler_options), }
+    {                                                                                                                  \
+        .self_size = sizeof(struct aws_http_request_handler_options),                                                  \
+    }
 
 AWS_EXTERN_C_BEGIN
 

include/aws/http/request_response.h

@@ -112,15 +112,18 @@ struct aws_http_server_options {
  * Initializes aws_http_server_options with default values.
  */
 #define AWS_HTTP_SERVER_OPTIONS_INIT                                                                                   \
-    { .self_size = sizeof(struct aws_http_server_options), .initial_window_size = SIZE_MAX, }
+    {                                                                                                                  \
+        .self_size = sizeof(struct aws_http_server_options),                                                           \
+        .initial_window_size = SIZE_MAX,                                                                               \
+    }
 
 /**
  * Invoked at the start of an incoming request.
  * To process the request, the user must create a request handler stream and return it to the connection.
  * If NULL is returned, the request will not be processed and the last error will be reported as the reason for failure.
  */
-typedef struct aws_http_stream *(
-    aws_http_on_incoming_request_fn)(struct aws_http_connection *connection, void *user_data);
+typedef struct aws_http_stream *(aws_http_on_incoming_request_fn)(struct aws_http_connection *connection,
+                                                                  void *user_data);
 
 typedef void(aws_http_on_server_connection_shutdown_fn)(
     struct aws_http_connection *connection,
@@ -163,7 +166,9 @@ struct aws_http_server_connection_options {
  * Initializes aws_http_server_connection_options with default values.
  */
 #define AWS_HTTP_SERVER_CONNECTION_OPTIONS_INIT                                                                        \
-    { .self_size = sizeof(struct aws_http_server_connection_options), }
+    {                                                                                                                  \
+        .self_size = sizeof(struct aws_http_server_connection_options),                                                \
+    }
 
 AWS_EXTERN_C_BEGIN
 

include/aws/http/server.h

@@ -12,7 +12,7 @@
 
 #include <ctype.h>
 
-#define AWS_DEFINE_ERROR_INFO_HTTP(CODE, STR) [(CODE)-0x0800] = AWS_DEFINE_ERROR_INFO(CODE, STR, "aws-c-http")
+#define AWS_DEFINE_ERROR_INFO_HTTP(CODE, STR) [(CODE) - 0x0800] = AWS_DEFINE_ERROR_INFO(CODE, STR, "aws-c-http")
 
 /* clang-format off */
 static struct aws_error_info s_errors[] = {

source/http.c

@@ -1318,11 +1318,11 @@ static int s_can_parse_as_chunked_encoding(
             }
         }
         long chunk_size = strtol((char *)chunk_ascii_hex, 0, 16);
-        long total_chunk_size_with_overhead = (long)
-            (match_cursor.ptr -  request_cursor.ptr /* size of the chunk in ascii hex */
-                + crlf_cursor.len                       /* size of the crlf */
-                + chunk_size                            /* size of the payload */
-                + crlf_cursor.len);                     /* size of the chunk terminating crlf */
+        long total_chunk_size_with_overhead =
+            (long)(match_cursor.ptr - request_cursor.ptr /* size of the chunk in ascii hex */
+                   + crlf_cursor.len                     /* size of the crlf */
+                   + chunk_size                          /* size of the payload */
+                   + crlf_cursor.len);                   /* size of the chunk terminating crlf */
 
         /* 0 length chunk signals end of stream. Check for the terminatino string and exit with success */
         if (0 == chunk_size) {