Skip to content

Conversation

@ronald-cron-arm
Copy link
Contributor

Description

Test usage of MBEDTLS_PRINTF_SIZET in mbedtls_debug_print_msg

PR checklist

  • changelog not required, just changing an unit test
  • backport not required, just changing an unit test
  • tests not required, just changing an unit test

@tom-cosgrove-arm
Copy link
Contributor

This is just a test PR to demonstrate the problem on older Windows compilers

Copy link
Contributor

@mpg mpg left a comment

Choose a reason for hiding this comment

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

LGTM

@mpg
Copy link
Contributor

mpg commented Mar 11, 2024

This is just a test PR to demonstrate the problem on older Windows compilers

Still looks good to have, I think. Once we have dropped support for older Windows compilers of course.

@ronald-cron-arm ronald-cron-arm force-pushed the debug-add-printf-sizet-test branch from 53b52b7 to 78fe0ca Compare March 28, 2025 18:35
@ronald-cron-arm
Copy link
Contributor Author

The bug that this PR was targeting with a unit test has been fixed by #10020 with some non regression tests. Thus closing this PR.

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

Labels

component-test Test framework and CI scripts component-tls DO-NOT-MERGE needs-ci Needs to pass CI tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants