Sitelet https://web.archive.org/web/20201103184609/https://github.com/TheAlgorithms/Java/pull/1279
Skip to content

/ Java

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

Jump to bottom

I find a problem #1279

Merged
merged 2 commits into from Apr 17, 2020
Merged

I find a problem #1279

merged 2 commits into from Apr 17, 2020

Conversation

@littleFoot1
Copy link
Contributor

@littleFoot1 littleFoot1 commented Apr 16, 2020

The test output is [4, 23, 23, 78, 78, 78, 231, 231, 231], I think it has a problem，
After changing，
Test data output [1, 4, 6, 9, 12, 23, 54, 78, 231]
littleFoot1 added 2 commits Apr 16, 2020
@littleFoot1
change Sorts/ShellSort.java
338fced
@littleFoot1
Update ShellSort.java
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
3ef9fe7
@littleFoot1

This comment has been minimized.

Sign in to view
Copy link
Contributor Author

@littleFoot1 littleFoot1 commented on 338fced Apr 16, 2020

The test output is [4, 23, 23, 78, 78, 78, 231, 231, 231], I think it has a problem，
After changing，
Test data output [1, 4, 6, 9, 12, 23, 54, 78, 231]
@StepfenShawn
StepfenShawn approved these changes Apr 17, 2020
View changes
Copy link
Member

@StepfenShawn StepfenShawn left a comment

Thank you for reporting the problem and solving it!
@StepfenShawn StepfenShawn merged commit 0389d31 into TheAlgorithms:master Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@StepfenShawn StepfenShawn

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
@littleFoot1 @StepfenShawn
You can’t perform that action at this time.