Skip to content
This repository has been archived by the owner on Jul 26, 2023. It is now read-only.

Add BCrypt types and test for CCM block chaining mode #74

Merged
merged 1 commit into from
Nov 23, 2015
Merged

Conversation

AArnott
Copy link
Collaborator

@AArnott AArnott commented Nov 23, 2015

No description provided.

/// The Microsoft algorithm providers for Advanced Encryption Standard (AES)
/// require a nonce for the Counter with CBC-MAC (CCM) and Galois/Counter Mode (GCM)
/// chaining modes, and will return an error if none is present.
/// If a nonce is not used, this member must be set to NULL.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't know what we should do about documentation in cases like that, I replaced NULL with <see cref="IntPtr.Zero" /> in a few places but that's tedious.

That's just a detail but might be interesting to unify on it at some point (Maybe a big XML doc cleanup before we reach 1.0).

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ya, in all my xml doc comment fixups (adding see cref's etc.) I haven't bothered fixing references to NULL. Sometimes it will be IntPtr.Zero. Other times it will be null. While the doc would be cleaner, I figure unless there's an automated way to fix it up, it's probably not worth our time since readers should be able to figure out what it means.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think there is a good way to make it automated but it could be made assisted, also the point is moot if we decide to do pointer only or to codegen the IntPtr variants.

AArnott added a commit that referenced this pull request Nov 23, 2015
Add BCrypt types and test for CCM block chaining mode
@AArnott AArnott merged commit 5a3cf60 into master Nov 23, 2015
@AArnott AArnott deleted the BCryptWork branch November 23, 2015 20:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants