Skip to content

Conversation

@radik878
Copy link

The FFT domain template referenced non-existent identifiers “FinerGenerator” and “GeneratorSqrt”.
These names do not exist anywhere in the codebase and originated from outdated comments in the
template, causing the incorrect text to be propagated to all generated fft/domain.go files.

This change updates the template to accurately describe that we compute a 2^k-th root of unity
and store it in Generator, and other values are derived from it (e.g., GeneratorInv).

Rationale:

  • Eliminates misleading documentation that can confuse readers and contributors.
  • Ensures future code generation produces correct comments across all curves/fields.
  • No functional changes, code behavior and APIs remain unaffected.

@ivokub ivokub added type: typo Typo fixes src: community Community originating PRs and issues labels Oct 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

src: community Community originating PRs and issues type: typo Typo fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants