Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use uint8_t not u_int8_t #1287

Merged
merged 1 commit into from
Nov 7, 2023
Merged

Use uint8_t not u_int8_t #1287

merged 1 commit into from
Nov 7, 2023

Conversation

justsmth
Copy link
Contributor

@justsmth justsmth commented Nov 7, 2023

Issues:

N/A

Description of changes:

  • Replace use of non-standard integer value u_int8_t with uint8_t.

Call-outs:

N/A

Testing:

Build succeeds locally.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

@justsmth justsmth requested a review from a team as a code owner November 7, 2023 17:26
@justsmth justsmth enabled auto-merge (squash) November 7, 2023 18:27
@justsmth justsmth merged commit c3cbc46 into aws:main Nov 7, 2023
12 checks passed
@skmcgrail skmcgrail mentioned this pull request Nov 8, 2023
samuel40791765 pushed a commit to samuel40791765/aws-lc that referenced this pull request Nov 10, 2023
(cherry picked from commit c3cbc46)
samuel40791765 added a commit that referenced this pull request Nov 14, 2023
(cherry picked from commit c3cbc46)

Co-authored-by: Justin W Smith <103147162+justsmth@users.noreply.github.com>
@justsmth justsmth deleted the use-uint8_t branch December 7, 2023 17:25
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