[openssl/openssl] b947bb: Add a WebKit clang-format file

0 views
Skip to first unread message

Bob Beck

unread,
Dec 4, 2025, 1:26:36 PM (2 days ago) Dec 4
to openssl...@openssl.org
Branch: refs/heads/openssl-3.6
Home: https://github.com/openssl/openssl
Commit: b947bb2092d48d66a17255e0a927114d77be9cd7
https://github.com/openssl/openssl/commit/b947bb2092d48d66a17255e0a927114d77be9cd7
Author: Bob Beck <be...@openssl.org>
Date: 2025-12-04 (Thu, 04 Dec 2025)

Changed paths:
A .clang-format

Log Message:
-----------
Add a WebKit clang-format file

Reviewed-by: Neil Horman <nho...@openssl.org>
Reviewed-by: Tomas Mraz <to...@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/29243)


Commit: f771ce0d3368f312ec676ae4855ee9f571445d33
https://github.com/openssl/openssl/commit/f771ce0d3368f312ec676ae4855ee9f571445d33
Author: Bob Beck <be...@openssl.org>
Date: 2025-12-04 (Thu, 04 Dec 2025)

Changed paths:
M apps/progs.pl

Log Message:
-----------
Fix apps/progs.pl to be slightly less fragile

In particular fix the regex magic to be tolerant of different ways
of formatting a main program.

My past life had forgotten this magic 14 years ago when we converted
it to just a table of commands in the forks.

https://www.youtube.com/watch?v=mWbbjvYmN8A

Reviewed-by: Neil Horman <nho...@openssl.org>
Reviewed-by: Tomas Mraz <to...@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/29243)


Commit: e78dfac23fd9551156693268a3e947ae1cfc82af
https://github.com/openssl/openssl/commit/e78dfac23fd9551156693268a3e947ae1cfc82af
Author: Bob Beck <be...@openssl.org>
Date: 2025-12-04 (Thu, 04 Dec 2025)

Changed paths:
M test/errtest.c

Log Message:
-----------
fix errtest to be less sensitive to line wrapping changes

(in it's final form it will work with either compiler
because it's currently one line, but was tripped up before
by the #ifdef, so redid it to be consistent with the
other changes previously in this stack)

While I am here correct the test to test for all possible
return values of ERR_get_error_all, without the #ifdefs

Reviewed-by: Neil Horman <nho...@openssl.org>
Reviewed-by: Tomas Mraz <to...@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/29243)


Commit: 2175c018843eb9dfb7abb1d0efa4730e9cc00d14
https://github.com/openssl/openssl/commit/2175c018843eb9dfb7abb1d0efa4730e9cc00d14
Author: Bob Beck <be...@openssl.org>
Date: 2025-12-04 (Thu, 04 Dec 2025)

Changed paths:
M test/cmp_ctx_test.c

Log Message:
-----------
Fix cmp_ctx_test.c to be less sensitive to line wrapping.

Similar to the previous errtest.c fix this also is not broken
by any reformatting today, but this change makes this follow
the same pattern as the other things that test OPENSSL_LINE
after the fact so we maintain the same paradigm everywhere.

Reviewed-by: Neil Horman <nho...@openssl.org>
Reviewed-by: Tomas Mraz <to...@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/29243)


Commit: e77c4c31227781fd9f92df0f00f41b37857bab3a
https://github.com/openssl/openssl/commit/e77c4c31227781fd9f92df0f00f41b37857bab3a
Author: Bob Beck <be...@openssl.org>
Date: 2025-12-04 (Thu, 04 Dec 2025)

Changed paths:
M include/crypto/sparc_arch.h

Log Message:
-----------
Disable clang format around multi-line macros of sparc assembly

They are not asm, but spit out stuff that is not C

Clang-format gets confused and does bad things with them.

Reviewed-by: Neil Horman <nho...@openssl.org>
Reviewed-by: Tomas Mraz <to...@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/29243)


Commit: 5bdd5f2fd63c53d85e658a23fb530fe3f3fb7a6a
https://github.com/openssl/openssl/commit/5bdd5f2fd63c53d85e658a23fb530fe3f3fb7a6a
Author: Bob Beck <be...@openssl.org>
Date: 2025-12-04 (Thu, 04 Dec 2025)

Changed paths:
M test/mem_alloc_test.c

Log Message:
-----------
Disable clang-format around line-wrap sensitive lines in malloc_test.c

If OPENSSL_LINE ends up on a different line than the following call here,
this test breaks.

We should perhaps reconsider if testing the reporting of OPENSSL_LINE
is what we want in a malloc test, but that's for another time than now.

Reviewed-by: Neil Horman <nho...@openssl.org>
Reviewed-by: Tomas Mraz <to...@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/29243)


Commit: cbe83e4af9e4cac61a2813754afd279c9f915173
https://github.com/openssl/openssl/commit/cbe83e4af9e4cac61a2813754afd279c9f915173
Author: Bob Beck <be...@openssl.org>
Date: 2025-12-04 (Thu, 04 Dec 2025)

Changed paths:
M crypto/LPdir_win32.c
M crypto/LPdir_wince.c
M crypto/bn/asm/x86_64-gcc.c
M crypto/des/cbc_enc.c
M crypto/des/des_enc.c
M crypto/o_dir.c
M engines/e_afalg.c
M engines/e_capi.c
M engines/e_dasync.c
M engines/e_loader_attic.c
M engines/e_ossltest.c
M engines/e_padlock.c
M providers/implementations/macs/blake2b_mac.c
M providers/implementations/macs/blake2s_mac.c
M test/mem_alloc_custom_fns_test.c
M test/radix/quic_radix.c

Log Message:
-----------
Disable clang format around .c includes

we assume these to be order sensitive and not self contained, so
as per our new style we disable clang format around them.

we should consider renaming to .inc, or doing away with some
of these and just putting the code inline, but that's for
later consideration.

Reviewed-by: Neil Horman <nho...@openssl.org>
Reviewed-by: Tomas Mraz <to...@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/29243)


Commit: 46e1c56b8e1be60af282bd44cac5c74f27d99cf7
https://github.com/openssl/openssl/commit/46e1c56b8e1be60af282bd44cac5c74f27d99cf7
Author: Bob Beck <be...@openssl.org>
Date: 2025-12-04 (Thu, 04 Dec 2025)

Changed paths:
M apps/progs.pl

Log Message:
-----------
nits in progs.pl

Reviewed-by: Neil Horman <nho...@openssl.org>
Reviewed-by: Tomas Mraz <to...@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/29243)


Compare: https://github.com/openssl/openssl/compare/057f22c13436...46e1c56b8e1b

To unsubscribe from these emails, change your notification settings at https://github.com/openssl/openssl/settings/notifications
Reply all
Reply to author
Forward
0 new messages