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.

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

Make FNL::new and FNL::with_seed const #139

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Kyllingene
Copy link

Makes the two constructors const. Nothing else can be constified because they're setters (which require &mut): the two fields that this is a limitation for are transform_type_3d and fractal_bounding, as they are private. Others require float ops, and are thus also barred (such as get_noise_2d).

If/when constant float ops are stabilized, this might warrant being revisited.

@Kyllingene
Copy link
Author

Addresses #137.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant