Skip to content

Quick sort in Rust #3055

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 1 commit into from
May 31, 2020
Merged

Quick sort in Rust #3055

merged 1 commit into from
May 31, 2020

Conversation

m-code12
Copy link
Contributor

Fixes #2131

Checklist:

  • 4 space indentation.
  • Coding conventions are followed.
  • Input is taken dynamically.
  • Sample Input / Output is added at the end of file.
  • Logic Documentation (Comments).
  • File names are correct.

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

Changes proposed in this pull request:

  • Added Quick Sort in Rust

Languages Used:

  • Rust

Files Added:

  • Quick_Sort.rs

We're happy to help you get this ready -- don't be afraid to ask for help.

Thanks!

Copy link
Collaborator

@singh-shreya6 singh-shreya6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Squash your commits.

fn main() {
println!("Enter the numbers to be sorted (separated by space) : ");
let mut array: Vec<u32> = read_values :: <u32>().unwrap();
quicksort(&mut array, 0, array.len()-1);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Space around -

@singh-shreya6
Copy link
Collaborator

Squashing not done properly.

@m-code12
Copy link
Contributor Author

I am getting errors, but I am trying

singh-shreya6
singh-shreya6 previously approved these changes May 31, 2020
Copy link
Collaborator

@singh-shreya6 singh-shreya6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is okay. Since there is not much time, I am merging it as the code is correct.

@m-code12
Copy link
Contributor Author

sorry I did not see your comment

@singh-shreya6
Copy link
Collaborator

It is okay, since you have done it. I had not merged it yet.

@m-code12
Copy link
Contributor Author

i have squashed commits

Copy link
Collaborator

@singh-shreya6 singh-shreya6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@singh-shreya6 singh-shreya6 merged commit b378ed9 into jainaman224:master May 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Quick Sort
2 participants