-
Notifications
You must be signed in to change notification settings - Fork 81
Description
Problem:
Cosmetic/stylistic suggestion: consider renaming the ML-DSA constants to use an ML_DSA_
prefix rather than MLDSA_
. This seems more idiomatic with how Rust style is normally used.
Solution:
Add an extra underscore for all 6 constants.
Requirements / Acceptance Criteria:
The relevant standard is in the API naming guidelines, but it isn't that explicit on this particular point:
https://rust-lang.github.io/api-guidelines/naming.html#casing-conforms-to-rfc-430-c-case
My argument is that "ML" and "DSA", usually hyphen-separated in prose, should be construed as 2 words in the context of naming them.
Further context
The rustls-post-quantum crate will soon offer ML-DSA support as an option.
rustls-pki-types now exposes alg_id::{ML_DSA_44, ML_DSA_65, ML_DSA_87}
.
rustls-webpki now exposes aws_lc_rs::{ML_DSA_44, ML_DSA_65, ML_DSA_87}
.