Skip to content

Conversation

@wbl
Copy link
Collaborator

@wbl wbl commented Sep 25, 2024

Based on quictls/openssl#173

Checklist

Thank you for your contribution. Please answer the following:

  • I acknowledge that I am authorized to submit this code under
    the terms of the Apache License
  • All new public APIs are documented
  • All new public functions have tests
  • All new public functions have fuzzing tests

@wbl wbl requested a review from richsalz September 25, 2024 18:17
@wbl wbl force-pushed the wbl/memory-saving-patch branch from bf7a55c to 84b0a13 Compare September 25, 2024 19:56
@wbl
Copy link
Collaborator Author

wbl commented Sep 25, 2024

Looks like the windows error is preexisting, so will rebase after we fix that.

@wbl wbl force-pushed the wbl/memory-saving-patch branch 2 times, most recently from c5a55ab to e057147 Compare September 27, 2024 16:02
@wbl wbl force-pushed the wbl/memory-saving-patch branch from 3f9d701 to cfe9220 Compare September 27, 2024 18:24
@wbl
Copy link
Collaborator Author

wbl commented Sep 30, 2024

Checks are clean, so just needs approval to merge

@richsalz
Copy link
Member

richsalz commented Oct 1, 2024

"All checks have passed". WOO HOO, WOOT!

Is there an open issue for documenting the C99 requirement?

@wbl
Copy link
Collaborator Author

wbl commented Oct 1, 2024

Issue #25 covers the C change.

@wbl wbl merged commit 87d52dd into main Oct 1, 2024
@richsalz richsalz mentioned this pull request Oct 24, 2024
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.

3 participants