Skip to content

Comments

Clang format 3.0#29256

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

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

Conversation

@bob-beck
Copy link
Contributor

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

clang-format 3.0

For openssl/project#1741
!--
Thank you for your pull request. Please review these requirements:

Contributors guide: https://github.com/openssl/openssl/blob/master/CONTRIBUTING.md

Other than that, provide a description above this comment if there isn't one already

If this fixes a GitHub issue, make sure to have a line saying 'Fixes #XXXX' (without quotes) in the commit message.
-->

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

@bob-beck bob-beck marked this pull request as ready for review December 2, 2025 16:46
Copy link
Contributor

@jogme jogme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The fuzzer is broken, but it is not caused with the changes. Issue tracked at openssl/project#1757

@jogme jogme 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 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: Norbert Pocs <[email protected]>
Reviewed-by: Saša Nedvědický <[email protected]>
(Merged from #29256)
openssl-machine pushed a commit that referenced this pull request Dec 9, 2025
Reviewed-by: Neil Horman <[email protected]>
Reviewed-by: Norbert Pocs <[email protected]>
Reviewed-by: Saša Nedvědický <[email protected]>
(Merged from #29256)
@nhorman
Copy link
Contributor

nhorman commented Dec 9, 2025

merged to 3.0, thank you!

@nhorman nhorman closed this Dec 9, 2025
openssl-machine pushed a commit that referenced this pull request Jan 27, 2026
3.0.19 CHANGES.md includes the following:
 * #29214
   "s390x: Check and fail on invalid malformed ECDSA signatures"
 * #29256
   "Clang format 3.0"

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:05:45 2026
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

4 participants