wrap match in Ok rather than all branches

This commit is contained in:
ConnorSkees 2020-04-26 18:51:08 -04:00
parent cfa734e412
commit f4f4fe2e03

View File

@ -25,24 +25,24 @@ pub(crate) fn register(f: &mut HashMap<String, Builtin>) {
Builtin::new(|mut args, scope, super_selector| { Builtin::new(|mut args, scope, super_selector| {
args.max_args(1)?; args.max_args(1)?;
match arg!(args, scope, super_selector, 0, "feature") { match arg!(args, scope, super_selector, 0, "feature") {
Value::Ident(s, _) => match s.as_str() { Value::Ident(s, _) => Ok(match s.as_str() {
// A local variable will shadow a global variable unless // A local variable will shadow a global variable unless
// `!global` is used. // `!global` is used.
"global-variable-shadowing" => Ok(Value::True), "global-variable-shadowing" => Value::True,
// the @extend rule will affect selectors nested in pseudo-classes // the @extend rule will affect selectors nested in pseudo-classes
// like :not() // like :not()
"extend-selector-pseudoclass" => Ok(Value::False), "extend-selector-pseudoclass" => Value::False,
// Full support for unit arithmetic using units defined in the // Full support for unit arithmetic using units defined in the
// [Values and Units Level 3][] spec. // [Values and Units Level 3][] spec.
"units-level-3" => Ok(Value::True), "units-level-3" => Value::True,
// The Sass `@error` directive is supported. // The Sass `@error` directive is supported.
"at-error" => Ok(Value::True), "at-error" => Value::True,
// The "Custom Properties Level 1" spec is supported. This means // The "Custom Properties Level 1" spec is supported. This means
// that custom properties are parsed statically, with only // that custom properties are parsed statically, with only
// interpolation treated as SassScript. // interpolation treated as SassScript.
"custom-property" => Ok(Value::False), "custom-property" => Value::False,
_ => Ok(Value::False), _ => Value::False,
}, }),
v => Err(( v => Err((
format!( format!(
"$feature: {} is not a string.", "$feature: {} is not a string.",