Skip to content

Conversation

@gojimmypi
Copy link
Contributor

@gojimmypi gojimmypi commented Mar 8, 2024

Description

I was a bit overly ambitious in adding debugging breadcrumbs to core source per code review of 7304.

This PR reverts some of those changes in internal.c and ecc.c.

Fixes zd# n/a

Testing

How did you test?

Tested with Jenkins only.

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@gojimmypi gojimmypi requested a review from dgarske March 8, 2024 20:07
@gojimmypi gojimmypi self-assigned this Mar 8, 2024
@gojimmypi gojimmypi marked this pull request as draft March 8, 2024 20:07
@gojimmypi gojimmypi marked this pull request as ready for review March 8, 2024 20:23
@gojimmypi
Copy link
Contributor Author

Hi @dgarske thanks for the prompt review.

Anything else I can update from #7304 ?

@dgarske dgarske merged commit b5633f9 into wolfSSL:master Mar 8, 2024
@gojimmypi gojimmypi deleted the PR-Revert-Breadcrumbs branch October 9, 2024 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants