The Wayback Machine - https://web.archive.org/web/20201211085931/https://github.com/TheAlgorithms/Go/pull/247
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

Update iterBinarySearch and add test cases #247

Open
wants to merge 7 commits into
base: master
from

Conversation

@Crazy3lf
Copy link
Contributor

@Crazy3lf Crazy3lf commented Oct 25, 2020

This is an update to my previous PR #245 .
I am not sure what name the test cases should be so I just use the same name for those cases.

Crazy3lf added 6 commits Oct 25, 2020
- highIndex should not subtract by 1
- equation now fixed to avoid overflow errors as stated in #123
@Crazy3lf Crazy3lf requested a review from Mystigan as a code owner Oct 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
1 participant
You can’t perform that action at this time.