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

Address man page review concerns #138

Merged
merged 3 commits into from Dec 6, 2019
Merged

Address man page review concerns #138

merged 3 commits into from Dec 6, 2019

Conversation

fscoto
Copy link
Contributor

@fscoto fscoto commented Dec 6, 2019

No description provided.

- Fix typo in crypto_sign_init_first_pass_custom_hash(3monocypher)
- Wipe sk in crypto_sign_init_first_pass_custom_hash(3monocypher);
  while it is just a test program, it is arguably good form to do so
  in any case, especially in documentation code that people might be
  copying and pasting.
- Note that HMACs can be safely truncated.
This is supposed to be a complete and exhaustive list of
all documented functions,
as can be seen with all incremental functions being listed
separately, rather than only the "main" page that is symlinked to.
@LoupVaillant LoupVaillant merged commit f130fc7 into LoupVaillant:master Dec 6, 2019
@fscoto fscoto deleted the master+new-man branch December 6, 2019 18:08
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.

None yet

2 participants