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

Add function comments where missing #48

Merged
merged 4 commits into from
Aug 31, 2022
Merged

Conversation

sirkrypt0
Copy link
Contributor

This adds a lot of function comments whereever they were missing,
mostly on static functions.

This adds a lot of function comments whereever they were missing,
mostly on static functions.
@sirkrypt0 sirkrypt0 force-pushed the feature/function-comments branch from 48ae7b6 to c0b7c3d Compare August 31, 2022 13:59
@frcroth frcroth force-pushed the feature/function-comments branch from 4f20082 to cbac029 Compare August 31, 2022 14:06
@felix-gohla felix-gohla added the documentation Improvements or additions to documentation label Aug 31, 2022
@felix-gohla
Copy link
Contributor

felix-gohla commented Aug 31, 2022

Could you please add a license header to crypto.h?

include/internal.h Outdated Show resolved Hide resolved
include/internal.h Outdated Show resolved Hide resolved
src/assertion.c Outdated Show resolved Hide resolved
src/assertion.c Outdated Show resolved Hide resolved
@frcroth frcroth self-assigned this Aug 31, 2022
@frcroth frcroth requested a review from felix-gohla August 31, 2022 16:24
@sirkrypt0 sirkrypt0 merged commit b62260a into main Aug 31, 2022
@sirkrypt0 sirkrypt0 deleted the feature/function-comments branch August 31, 2022 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Development

Successfully merging this pull request may close these issues.

3 participants