Skip to content

Comments

Clang format 3.5#29262

Closed
bob-beck wants to merge 2 commits intoopenssl:openssl-3.5from
bob-beck:clang-format-3.5
Closed

Clang format 3.5#29262
bob-beck wants to merge 2 commits intoopenssl:openssl-3.5from
bob-beck:clang-format-3.5

Conversation

@bob-beck
Copy link
Contributor

@bob-beck bob-beck commented Nov 28, 2025

clang-format 3.5

For openssl/project#1741

Checklist
  • documentation is added or updated
  • tests are added or updated

@github-actions github-actions bot added the severity: fips change The pull request changes FIPS provider sources label Nov 28, 2025
@bob-beck bob-beck marked this pull request as ready for review December 9, 2025 09:42
@Sashan Sashan added approval: done This pull request has the required number of approvals approval: ready to merge The 24 hour grace period has passed, ready to merge branch: 3.5 Applies to openssl-3.5 labels Dec 9, 2025
openssl-machine pushed a commit that referenced this pull request Dec 9, 2025
Reviewed-by: Neil Horman <[email protected]>
Reviewed-by: Saša Nedvědický <[email protected]>
Reviewed-by: Nikola Pajkovsky <[email protected]>
(Merged from #29262)
openssl-machine pushed a commit that referenced this pull request Dec 9, 2025
Reviewed-by: Neil Horman <[email protected]>
Reviewed-by: Saša Nedvědický <[email protected]>
Reviewed-by: Nikola Pajkovsky <[email protected]>
(Merged from #29262)
@nhorman
Copy link
Contributor

nhorman commented Dec 9, 2025

merged to 3.5, thank you!

@nhorman nhorman closed this Dec 9, 2025
openssl-machine pushed a commit that referenced this pull request Jan 27, 2026
3.5.5 CHANGES.md includes the following:
 * #28760
   "Improve the CPUINFO display for RISC-V"
 * #29214
   "s390x: Check and fail on invalid malformed ECDSA signatures"
 * #29262
   "Clang format 3.5"

Co-Authored-by: Tomáš Mráz <[email protected]>
Signed-off-by: Eugene Syromiatnikov <[email protected]>

Reviewed-by: Saša Nedvědický <[email protected]>
Reviewed-by: Neil Horman <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
MergeDate: Mon Jan 26 20:03:05 2026
@rainerjung
Copy link
Contributor

This change introduced a regression (syntax error) in the generated Sparc assembler due to whitespace introdoced in %icc => % icc, %xcc => % xcc and %o7 => % o7 in file include/crypto/sparc_arch.h,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals approval: ready to merge The 24 hour grace period has passed, ready to merge branch: 3.5 Applies to openssl-3.5 severity: fips change The pull request changes FIPS provider sources

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Freeze/Unfreeze tree and land clang-format changes Dec 9.

5 participants