Fix CC_FORCE_DISABLE=0 evaluating to true

madsmtm · madsmtm · commit df024a0f79be · 2025-01-17T08:11:12.000+01:00
diff --git a/src/lib.rs b/src/lib.rs
@@ -4157,10 +4157,9 @@ fn is_disabled() -> bool {
         match std::env::var_os("CC_FORCE_DISABLE") {
             // Not set? Not disabled.
             None => false,
-            // Respect `CC_FORCE_DISABLE=0` and some simple synonyms.
-            Some(v) if &*v != "0" && &*v != "false" && &*v != "no" => false,
-            // Otherwise, we're disabled. This intentionally includes `CC_FORCE_DISABLE=""`
-            Some(_) => true,
+            // Respect `CC_FORCE_DISABLE=0` and some simple synonyms, otherwise
+            // we're disabled. This intentionally includes `CC_FORCE_DISABLE=""`
+            Some(v) => &*v != "0" && &*v != "false" && &*v != "no",
         }
     }
     match val {

Original file line number	Diff line number	Diff line change
`@@ -4157,10 +4157,9 @@ fn is_disabled() -> bool {`
`4157`	`4157`	`match std::env::var_os("CC_FORCE_DISABLE") {`
`4158`	`4158`	`// Not set? Not disabled.`
`4159`	`4159`	`None => false,`
`4160`		- // Respect `CC_FORCE_DISABLE=0` and some simple synonyms.
`4161`		`- Some(v) if &v != "0" && &v != "false" && &*v != "no" => false,`
`4162`		- // Otherwise, we're disabled. This intentionally includes `CC_FORCE_DISABLE=""`
`4163`		`- Some(_) => true,`
	`4160`	+ // Respect `CC_FORCE_DISABLE=0` and some simple synonyms, otherwise
	`4161`	+ // we're disabled. This intentionally includes `CC_FORCE_DISABLE=""`
	`4162`	`+ Some(v) => &v != "0" && &v != "false" && &*v != "no",`
`4164`	`4163`	`}`
`4165`	`4164`	`}`
`4166`	`4165`	`match val {`