refactor: use iter workaround for contains() gap

calebcartwright · calebcartwright · commit 7aa69e5bc87f · 2021-09-16T22:01:50.000-05:00
diff --git a/src/cargo-fmt/main.rs b/src/cargo-fmt/main.rs
@@ -109,9 +109,9 @@ fn execute() -> i32 {
     let strategy = CargoFmtStrategy::from_opts(&opts);
     let mut rustfmt_args = opts.rustfmt_options;
     if opts.check {
-        let check_flag = String::from("--check");
-        if !rustfmt_args.contains(&check_flag) {
-            rustfmt_args.push(check_flag);
+        let check_flag = "--check";
+        if !rustfmt_args.iter().any(|o| o == check_flag) {
+            rustfmt_args.push(check_flag.to_owned());
         }
     }
     if let Some(message_format) = opts.message_format {

Original file line number	Diff line number	Diff line change
`@@ -109,9 +109,9 @@ fn execute() -> i32 {`
`109`	`109`	`let strategy = CargoFmtStrategy::from_opts(&opts);`
`110`	`110`	`let mut rustfmt_args = opts.rustfmt_options;`
`111`	`111`	`if opts.check {`
`112`		`- let check_flag = String::from("--check");`
`113`		`- if !rustfmt_args.contains(&check_flag) {`
`114`		`- rustfmt_args.push(check_flag);`
	`112`	`+ let check_flag = "--check";`
	`113`	`+ if !rustfmt_args.iter().any(\|o\| o == check_flag) {`
	`114`	`+ rustfmt_args.push(check_flag.to_owned());`
`115`	`115`	`}`
`116`	`116`	`}`
`117`	`117`	`if let Some(message_format) = opts.message_format {`