Skip to content
This repository was archived by the owner on Sep 7, 2025. It is now read-only.

made changes#43

Closed
Illusion0-0 wants to merge 1 commit intoAllAlgorithms:masterfrom
Illusion0-0:master
Closed

made changes#43
Illusion0-0 wants to merge 1 commit intoAllAlgorithms:masterfrom
Illusion0-0:master

Conversation

@Illusion0-0
Copy link

this change is more better than the original code(it is simple ,easy and basic code for binary search in c++)

this change is more better than the original code(it is simple ,easy and basic code for binary search in c++)
@abranhe abranhe added bug Something isn't working Bugs on Linux and removed bug Something isn't working labels Oct 4, 2018
@abranhe
Copy link
Member

abranhe commented Oct 4, 2018

Hi, @TARUNYO thanks for opening a pull request making changes to the Binary Search implementation.

In the pull request, you made you are using

#include <conio.h>

which is specific to the Windows console environment. This library limits you to work in a common Unix machine. Also the style used is very poor.

I personally implemented this algorithm

so I think you should review once to be be merged. You can open a new pull request requesting new changed because this one will be closed.

@abranhe abranhe closed this Oct 4, 2018
@Illusion0-0
Copy link
Author

Illusion0-0 commented Oct 6, 2018 via email

@Illusion0-0
Copy link
Author

Illusion0-0 commented Oct 6, 2018 via email

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants