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.

Sign up for GitHub

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

Jump to bottom

Curves: add missing parametric constructors, remove _assure_has_components #267

Merged
merged 2 commits into from
Feb 9, 2021
Merged

Curves: add missing parametric constructors, remove _assure_has_components #267

merged 2 commits into from
Feb 9, 2021

Commits on Feb 3, 2021

  1. Curves: add missing parametric constructors, simplify 

    For parametric types, one should provide parametric constructors
(and then perhaps, as in this case, non-parametric ones for convenience).

This allows us to merge some duplicate code.
    @fingolfin
    fingolfin committed Feb 3, 2021
    Configuration menu
    Copy the full SHA
    63c2af0 View commit details
    Browse the repository at this point in the history

  2. Curves: remove _assure_has_components 

    Instead, just never access C.components directly, instead
use curve_components(C)
    @fingolfin
    fingolfin committed Feb 3, 2021
    Configuration menu
    Copy the full SHA
    b02ccc0 View commit details
    Browse the repository at this point in the history