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

style: clippy changes #80

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

mrghosti3
Copy link

Changes:

  • use String:new() instead of format!("")
  • use is_empty() instead of s == ""
  • use reference iterator instead of into_iter
  • replace write! + '\n' with writeln!
  • remove mut from vars with type &mut ...
  • replace single branch match with if pattern match
  • replace Stack::to_string with Display trait
  • replace match with pattern matching in fn main
  • use reference to file name, instead of direct ownership

Changes:
- use `String:new()` instead of `format!("")`
- use `is_empty()` instead of `s == ""`
- use reference iterator instead of `into_iter`
- replace `write!` + '\n' with `writeln!`
- remove mut from vars with type `&mut ...`
- replace single branch `match` with `if` pattern match
- replace `Stack::to_string` with `Display` trait
- replace match with pattern matching in `fn main`
- use reference to file name, instead of direct ownership
@mrghosti3
Copy link
Author

This contains similar changes to #70, but only sticks to clippy changes, leaving Stack in the code.

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.

1 participant