Code review

Author: nex3

lib/src/ast/node.dart

@@ -15,9 +15,9 @@ abstract class AstNode {
   /// Returns an [AstNode] that doesn't have any data and whose span is
   /// generated by [callback].
   ///
-  /// Anumber of APIs take [AstNode]s instead of spans because computing spans
+  /// A number of APIs take [AstNode]s instead of spans because computing spans
   /// eagerly can be expensive. This allows arbitrary spans to be passed to
-  /// those callbacks while sitll being lazily computed.
+  /// those callbacks while still being lazily computed.
   factory AstNode.fake(FileSpan Function() callback) = _FakeAstNode;
 
   AstNode();

lib/src/async_environment.dart

@@ -235,7 +235,7 @@ class AsyncEnvironment {
   /// If [namespace] is passed, the module is made available under that
   /// namespace.
   ///
-  /// Throws a [SassException] if there's already a module with the given
+  /// Throws a [SassScriptException] if there's already a module with the given
   /// [namespace], or if [namespace] is `null` and [module] defines a variable
   /// with the same name as a variable defined in this environment.
   void addModule(Module module, AstNode nodeWithSpan, {String namespace}) {

lib/src/callable.dart

@@ -65,7 +65,7 @@ export 'callable/user_defined.dart';
 ///   access a string's length in code points.
 @sealed
 abstract class Callable extends AsyncCallable {
-  @Deprecated('Use `new Callable.function` instead.')
+  @Deprecated('Use `Callable.function` instead.')
   factory Callable(String name, String arguments,
           ext.Value callback(List<ext.Value> arguments)) =>
       Callable.function(name, arguments, callback);

lib/src/callable/async.dart

@@ -23,7 +23,7 @@ abstract class AsyncCallable {
   /// The callable's name.
   String get name;
 
-  @Deprecated('Use `new AsyncCallable.function` instead.')
+  @Deprecated('Use `AsyncCallable.function` instead.')
   factory AsyncCallable(String name, String arguments,
           FutureOr<ext.Value> callback(List<ext.Value> arguments)) =>
       AsyncCallable.function(name, arguments, callback);

lib/src/functions/color.dart

@@ -776,7 +776,8 @@ SassColor _transparentize(List<Value> arguments) {
 /// Like [fuzzyRound], but returns `null` if [number] is `null`.
 int _fuzzyRoundOrNull(num number) => number == null ? null : fuzzyRound(number);
 
-/// Like [new BuiltInCallable.function], but always sets the URL to `sass:math`.
+/// Like [new BuiltInCallable.function], but always sets the URL to
+/// `sass:color`.
 BuiltInCallable _function(
         String name, String arguments, Value callback(List<Value> arguments)) =>
     BuiltInCallable.function(name, arguments, callback, url: "sass:color");

lib/src/functions/list.dart

@@ -131,7 +131,7 @@ final _separator = _function(
 final _isBracketed = _function("is-bracketed", r"$list",
     (arguments) => SassBoolean(arguments[0].hasBrackets));
 
-/// Like [new BuiltInCallable.function], but always sets the URL to `sass:math`.
+/// Like [new BuiltInCallable.function], but always sets the URL to `sass:list`.
 BuiltInCallable _function(
         String name, String arguments, Value callback(List<Value> arguments)) =>
     BuiltInCallable.function(name, arguments, callback, url: "sass:list");

lib/src/functions/map.dart

@@ -73,7 +73,7 @@ final _hasKey = _function("has-key", r"$map, $key", (arguments) {
   return SassBoolean(map.contents.containsKey(key));
 });
 
-/// Like [new BuiltInCallable.function], but always sets the URL to `sass:math`.
+/// Like [new BuiltInCallable.function], but always sets the URL to `sass:map`.
 BuiltInCallable _function(
         String name, String arguments, Value callback(List<Value> arguments)) =>
     BuiltInCallable.function(name, arguments, callback, url: "sass:map");

lib/src/functions/meta.dart

@@ -58,7 +58,7 @@ final global = UnmodifiableListView([
   })
 ]);
 
-/// Like [new BuiltInCallable.function], but always sets the URL to `sass:math`.
+/// Like [new BuiltInCallable.function], but always sets the URL to `sass:meta`.
 BuiltInCallable _function(
         String name, String arguments, Value callback(List<Value> arguments)) =>
     BuiltInCallable.function(name, arguments, callback, url: "sass:meta");

lib/src/functions/selector.dart

@@ -139,7 +139,8 @@ CompoundSelector _prependParent(CompoundSelector compound) {
   }
 }
 
-/// Like [new BuiltInCallable.function], but always sets the URL to `sass:math`.
+/// Like [new BuiltInCallable.function], but always sets the URL to
+/// `sass:selector`.
 BuiltInCallable _function(
         String name, String arguments, Value callback(List<Value> arguments)) =>
     BuiltInCallable.function(name, arguments, callback, url: "sass:selector");

lib/src/functions/string.dart

@@ -168,7 +168,8 @@ int _codepointForIndex(int index, int lengthInCodepoints,
   return result;
 }
 
-/// Like [new BuiltInCallable.function], but always sets the URL to `sass:math`.
+/// Like [new BuiltInCallable.function], but always sets the URL to
+/// `sass:string`.
 BuiltInCallable _function(
         String name, String arguments, Value callback(List<Value> arguments)) =>
     BuiltInCallable.function(name, arguments, callback, url: "sass:string");