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

Remove unnecessary allocation for alphabet shuffling in constructor #24

Merged

aradalvand merged 2 commits into main from remove-extra-allocation

Sep 13, 2023

Collaborator

aradalvand commented Sep 13, 2023 •

edited

Resolves #23

Thanks a lot to @LeaFrock for spotting this.

aradalvand added 2 commits

September 13, 2023 06:39


          Remove unnecessary allocation for alphabet shuffling in constructor

cc79e76


          Remove leftover comment from the previous algorithm's implementation

86a8d34

aradalvand merged commit a91b293 into main

1 check passed

aradalvand deleted the remove-extra-allocation branch

September 13, 2023 03:38

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment