Switching precise check from match to is_none()

Revantus · Revantus · commit 2bdac84292bb · 2021-03-14T07:23:52.000-06:00
diff --git a/src/cargo/ops/cargo_generate_lockfile.rs b/src/cargo/ops/cargo_generate_lockfile.rs
@@ -44,13 +44,8 @@ pub fn update_lockfile(ws: &Workspace<'_>, opts: &UpdateOptions<'_>) -> CargoRes
         anyhow::bail!("you can't generate a lockfile for an empty workspace.")
     }
 
-    if opts.config.offline() {
-        match opts.precise {
-            None => {
-                anyhow::bail!("you can't update in the offline mode without precise packages");
-            }
-            Some(_) => {}
-        }
+    if opts.config.offline() && opts.precise.is_none() {
+        anyhow::bail!("you can't update in the offline mode without precise packages");
     }
 
     // Updates often require a lot of modifications to the registry, so ensure

Original file line number	Diff line number	Diff line change
`@@ -44,13 +44,8 @@ pub fn update_lockfile(ws: &Workspace<'_>, opts: &UpdateOptions<'_>) -> CargoRes`
`44`	`44`	`anyhow::bail!("you can't generate a lockfile for an empty workspace.")`
`45`	`45`	`}`
`46`	`46`
`47`		`- if opts.config.offline() {`
`48`		`- match opts.precise {`
`49`		`- None => {`
`50`		`- anyhow::bail!("you can't update in the offline mode without precise packages");`
`51`		`- }`
`52`		`- Some(_) => {}`
`53`		`- }`
	`47`	`+ if opts.config.offline() && opts.precise.is_none() {`
	`48`	`+ anyhow::bail!("you can't update in the offline mode without precise packages");`
`54`	`49`	`}`
`55`	`50`
`56`	`51`	`// Updates often require a lot of modifications to the registry, so ensure`