What changed, and why it matters
This commit only fixes spelling mistakes in code comments and documentation strings. No actual program logic, math formulas, or executable code was changed. It cannot affect security in any way.
No security action needed. This is a cosmetic/documentation cleanup patch.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff changes four comment lines: correcting ‘Weierstrauss’ to ‘Weierstrass’ and ‘represenation’ to ‘representation’ in C/C++ header comments. There are no changes to functions, data structures, FFI bindings, cryptographic algorithms, or build configuration.
Changed components
src/fcmp_pp/fcmp_pp_crypto.hsrc/fcmp_pp/fcmp_pp_rust/fcmp++.hInspect captured patch +4 / −4
### src/fcmp_pp/fcmp_pp_crypto.h
@@ -75,7 +75,7 @@ bool get_valid_torsion_cleared_point_vartime(const crypto::ec_point &point, cryp
/*
point_to_ed_derivatives converts an Ed25519 point to Ed25519 derivatives used for converting to
-Weierstrauss coords, as per https://www.ietf.org/archive/id/draft-ietf-lwig-curve-representations-02.pdf E.2.
+Weierstrass coords, as per https://www.ietf.org/archive/id/draft-ietf-lwig-curve-representations-02.pdf E.2.
We expect that a point passed in this function has been validated to be in the main subgroup with no torsion,
and does not equal identity. The `torsion_free_point` param is expected to be the output of
@@ -90,7 +90,7 @@ point_to_ed_derivatives can have torsion, otherwise this function has undefined
bool ed_derivatives_to_wei_x_y(const EdDerivatives &ed_derivatives, crypto::ec_coord &wei_x, crypto::ec_coord &wei_y);
/*
-point_to_wei_x_y takes a torsion free point as input, and coverts to Weierstrauss coordinates.
+point_to_wei_x_y takes a torsion free point as input, and converts to Weierstrass coordinates.
We expect that a point passed in this function has been validated to be in the main subgroup with no torsion,
and does not equal identity. The `torsion_free_point` param is expected to be the output of
### src/fcmp_pp/fcmp_pp_rust/fcmp++.h
@@ -44,7 +44,7 @@
/// A constant-time implementation of the Ed25519 field.
/// This type is expected to be opaque to the C/C++ side, meaning only the Rust side should read/write
-/// its internal represenation. We're using a modified crypto-bigint crate for this type so that we
+/// its internal representation. We're using a modified crypto-bigint crate for this type so that we
/// can work with points and scalars across the FFI without tons of byte repr conversions.
struct SeleneScalar {
uintptr_t _0[32 / sizeof(uintptr_t)];
@@ -54,7 +54,7 @@ FFI_STATIC_ASSERT(alignof(struct SeleneScalar) == sizeof(uintptr_t), "SeleneScal
/// The field novel to Helios/Selene.
/// This type is expected to be opaque to the C/C++ side, meaning only the Rust side should read/write
-/// its internal represenation. We're using a modified crypto-bigint crate for this type so that we
+/// its internal representation. We're using a modified crypto-bigint crate for this type so that we
/// can work with points and scalars across the FFI without tons of byte repr conversions.
struct HeliosScalar {
uintptr_t _0[32 / sizeof(uintptr_t)];Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.