Code review

lib/src/functions/color.dart

@@ -740,7 +740,7 @@ Object /* SassString | List<Value> */ _parseChannels(
   var list = channels.asList;
   if (list.length > 3) {
     throw SassScriptException("Only 3 elements allowed, but ${list.length} "
-        "${pluralize('was', list.length, plural: 'were')} passed.");
+        "were passed.");
   } else if (list.length < 3) {
     if (list.any((value) => value.isVar) ||
         (list.isNotEmpty && _isVarSlash(list.last))) {

lib/src/visitor/async_evaluate.dart

@@ -2925,9 +2925,9 @@ class _EvaluateVisitor
 
   /// Returns the [AstNode] whose span should be used for [expression].
   ///
-  /// If [expression] is a variable reference and [_sourceMap] is `true`,
-  /// [AstNode]'s span will be the span where that variable was originally
-  /// declared. Otherwise, this will just return [expression].
+  /// If [expression] is a variable reference, [AstNode]'s span will be the span
+  /// where that variable was originally declared. Otherwise, this will just
+  /// return [expression].
   ///
   /// This returns an [AstNode] rather than a [FileSpan] so we can avoid calling
   /// [AstNode.span] if the span isn't required, since some nodes need to do

lib/src/visitor/evaluate.dart

@@ -5,7 +5,7 @@
 // DO NOT EDIT. This file was generated from async_evaluate.dart.
 // See tool/grind/synchronize.dart for details.
 //
-// Checksum: 1546b59aa219428e5e9458b8f0360192b544d073
+// Checksum: 21d03e5b2631b77c590036153fa3d9078e031136
 //
 // ignore_for_file: unused_import
 
@@ -2900,9 +2900,9 @@ class _EvaluateVisitor
 
   /// Returns the [AstNode] whose span should be used for [expression].
   ///
-  /// If [expression] is a variable reference and [_sourceMap] is `true`,
-  /// [AstNode]'s span will be the span where that variable was originally
-  /// declared. Otherwise, this will just return [expression].
+  /// If [expression] is a variable reference, [AstNode]'s span will be the span
+  /// where that variable was originally declared. Otherwise, this will just
+  /// return [expression].
   ///
   /// This returns an [AstNode] rather than a [FileSpan] so we can avoid calling
   /// [AstNode.span] if the span isn't required, since some nodes need to do

test/dart_api/value/list_test.dart

@@ -116,7 +116,7 @@ void main() {
     });
   });
 
-  test("a slash-separated list is space-separated", () {
+  test("a slash-separated list is slash-separated", () {
     expect(parseValue("list.slash(a, b, c)").separator,
         equals(ListSeparator.slash));
   });