Skip to content

added findTheDuplicateNumber #75

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

Merged
merged 3 commits into from
Oct 20, 2015
Merged

added findTheDuplicateNumber #75

merged 3 commits into from
Oct 20, 2015

Conversation

Vally79
Copy link
Collaborator

@Vally79 Vally79 commented Oct 19, 2015

No description provided.

haoel added a commit that referenced this pull request Oct 20, 2015
added findTheDuplicateNumber
@haoel haoel merged commit a847507 into haoel:master Oct 20, 2015
@haoel
Copy link
Owner

haoel commented Oct 23, 2015

@Vally79

Actually, this pull request didn't meet the requirement - the array is read-only. So, you cannot just simply use the sort method to solve this problem. So, I manually rollback your solution, and correct it. (The correction is 8c3448a )

Anyway, thanks for this pull request, and please keep doing this.

Hao

@Vally79
Copy link
Collaborator Author

Vally79 commented Oct 23, 2015

@haoel
You're right, didn't pay attention to the requirements. Thanks!

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.

2 participants