Skip to content

Conversation

@jmalak
Copy link
Contributor

@jmalak jmalak commented Feb 17, 2025

Description

Fix test to be usable with C89 toolchain.

Testing

Compile it with C89 compiler, need not fail.

Checklist

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

@wolfSSL-Bot
Copy link

Can one of the admins verify this patch?

@embhorn
Copy link
Member

embhorn commented Feb 17, 2025

@jmalak is an approved contributor

@douzzer douzzer mentioned this pull request Feb 17, 2025
Copy link
Contributor

@douzzer douzzer left a comment

Choose a reason for hiding this comment

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

Hi @jmalak . Can you try #8465 to make sure it also solves your problem?

If so, we'll merge that. If not, we'll merge this.

Thanks!

@douzzer douzzer assigned jmalak and unassigned wolfSSL-Bot Feb 17, 2025
@jmalak
Copy link
Contributor Author

jmalak commented Feb 18, 2025

Hi @jmalak . Can you try #8465 to make sure it also solves your problem?

If so, we'll merge that. If not, we'll merge this.

Thanks!

Of cause your change works with C89 due to static storage class.

Jiri

@douzzer
Copy link
Contributor

douzzer commented Feb 18, 2025

Thanks @jmalak for checking!

@douzzer
Copy link
Contributor

douzzer commented Feb 18, 2025

closing in favor of #8465

@douzzer douzzer closed this Feb 18, 2025
@gojimmypi gojimmypi mentioned this pull request Jul 3, 2025
4 tasks
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.

5 participants