Skip to content

Fermat's little theorem #847

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
Merged
merged 6 commits into from
May 28, 2019
Merged

Fermat's little theorem #847

merged 6 commits into from
May 28, 2019

Conversation

Erfaniaa
Copy link
Member

No description provided.

@poyea
Copy link
Member

poyea commented May 27, 2019

LGTM. But could you provide a reference to the theorem (the corollary)?
@sarthak77
Copy link
Member

Time complexity is for binary exponentiation vs normal exponentiation so can be removed from top and can be written as a note below
@Erfaniaa
Copy link
Member Author

Should I put this URL in the code?

https://en.wikipedia.org/wiki/Fermat%27s_little_theorem
@Erfaniaa
Copy link
Member Author

Please see the new changes that I've made.
@sarthak77
Copy link
Member

Looks good
@poyea
Copy link
Member

poyea commented May 28, 2019

Thank you for your pull request!🤩
@poyea poyea merged commit fc95e7a into master May 28, 2019
@Erfaniaa Erfaniaa deleted the fermat-little-theorem branch May 28, 2019 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet