From 7f982a1b3ab8f8a0ca4c1b6eefcfdcd82fe66e9a Mon Sep 17 00:00:00 2001
From: Natalie Weizenbaum <nweiz@google.com>
Date: Fri, 21 May 2021 16:23:53 -0700
Subject: [PATCH] Only emit each warning once per source location (#1322)

---
 CHANGELOG.md                        |  4 ++++
 lib/src/visitor/async_evaluate.dart | 20 ++++++++++++++------
 lib/src/visitor/evaluate.dart       | 22 +++++++++++++++-------
 pubspec.yaml                        |  2 +-
 4 files changed, 34 insertions(+), 14 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 60aab93c..104c83a8 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 1.33.1
+
+* Don't emit the same warning in the same location multiple times.
+
 ## 1.33.0
 
 * Deprecate the use of `/` for division. The new `math.div()` function should be
diff --git a/lib/src/visitor/async_evaluate.dart b/lib/src/visitor/async_evaluate.dart
index 4f60ef35..063127df 100644
--- a/lib/src/visitor/async_evaluate.dart
+++ b/lib/src/visitor/async_evaluate.dart
@@ -148,6 +148,13 @@ class _EvaluateVisitor
   /// The logger to use to print warnings.
   final Logger _logger;
 
+  /// A set of message/location pairs for warnings that have been emitted via
+  /// [_warn].
+  ///
+  /// We only want to emit one warning per location, to avoid blowing up users'
+  /// consoles with redundant warnings.
+  final _warningsEmitted = <Tuple2<String, SourceSpan>>{};
+
   /// Whether to track source map information.
   final bool _sourceMap;
 
@@ -1936,7 +1943,7 @@ class _EvaluateVisitor
     }
 
     if (node.isGlobal && !_environment.globalVariableExists(node.name)) {
-      _logger.warn(
+      _warn(
           _environment.atRoot
               ? "As of Dart Sass 2.0.0, !global assignments won't be able to\n"
                   "declare new variables. Since this assignment is at the root "
@@ -1946,8 +1953,7 @@ class _EvaluateVisitor
                   "declare new variables. Consider adding "
                   "`${node.originalName}: null` at the root of the\n"
                   "stylesheet.",
-          span: node.span,
-          trace: _stackTrace(node.span),
+          node.span,
           deprecation: true);
     }
 
@@ -3087,9 +3093,11 @@ class _EvaluateVisitor
   }
 
   /// Emits a warning with the given [message] about the given [span].
-  void _warn(String message, FileSpan span, {bool deprecation = false}) =>
-      _logger.warn(message,
-          span: span, trace: _stackTrace(span), deprecation: deprecation);
+  void _warn(String message, FileSpan span, {bool deprecation = false}) {
+    if (!_warningsEmitted.add(Tuple2(message, span))) return;
+    _logger.warn(message,
+        span: span, trace: _stackTrace(span), deprecation: deprecation);
+  }
 
   /// Returns a [SassRuntimeException] with the given [message].
   ///
diff --git a/lib/src/visitor/evaluate.dart b/lib/src/visitor/evaluate.dart
index c478d374..f50a2cd4 100644
--- a/lib/src/visitor/evaluate.dart
+++ b/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: 1fc6b9e6018eba3ac520464abb56e686f4cb9886
+// Checksum: 7eb518e3fd9269a2117e8f4a4b4149ca05e3ec25
 //
 // ignore_for_file: unused_import
 
@@ -156,6 +156,13 @@ class _EvaluateVisitor
   /// The logger to use to print warnings.
   final Logger _logger;
 
+  /// A set of message/location pairs for warnings that have been emitted via
+  /// [_warn].
+  ///
+  /// We only want to emit one warning per location, to avoid blowing up users'
+  /// consoles with redundant warnings.
+  final _warningsEmitted = <Tuple2<String, SourceSpan>>{};
+
   /// Whether to track source map information.
   final bool _sourceMap;
 
@@ -1928,7 +1935,7 @@ class _EvaluateVisitor
     }
 
     if (node.isGlobal && !_environment.globalVariableExists(node.name)) {
-      _logger.warn(
+      _warn(
           _environment.atRoot
               ? "As of Dart Sass 2.0.0, !global assignments won't be able to\n"
                   "declare new variables. Since this assignment is at the root "
@@ -1938,8 +1945,7 @@ class _EvaluateVisitor
                   "declare new variables. Consider adding "
                   "`${node.originalName}: null` at the root of the\n"
                   "stylesheet.",
-          span: node.span,
-          trace: _stackTrace(node.span),
+          node.span,
           deprecation: true);
     }
 
@@ -3058,9 +3064,11 @@ class _EvaluateVisitor
   }
 
   /// Emits a warning with the given [message] about the given [span].
-  void _warn(String message, FileSpan span, {bool deprecation = false}) =>
-      _logger.warn(message,
-          span: span, trace: _stackTrace(span), deprecation: deprecation);
+  void _warn(String message, FileSpan span, {bool deprecation = false}) {
+    if (!_warningsEmitted.add(Tuple2(message, span))) return;
+    _logger.warn(message,
+        span: span, trace: _stackTrace(span), deprecation: deprecation);
+  }
 
   /// Returns a [SassRuntimeException] with the given [message].
   ///
diff --git a/pubspec.yaml b/pubspec.yaml
index 3b801fa3..9e68a06f 100644
--- a/pubspec.yaml
+++ b/pubspec.yaml
@@ -1,5 +1,5 @@
 name: sass
-version: 1.33.0
+version: 1.33.1-dev
 description: A Sass implementation in Dart.
 author: Sass Team
 homepage: https://github.com/sass/dart-sass