Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(17)

Unified Diff: lib/src/nesting_level.dart

Issue 1255643002: New, simpler and faster line splitter. (Closed) Base URL: https://github.com/dart-lang/dart_style.git@master
Patch Set: Optimize nesting. Reformat. Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « lib/src/nesting_builder.dart ('k') | lib/src/rule/rule.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/src/nesting_level.dart
diff --git a/lib/src/nesting_level.dart b/lib/src/nesting_level.dart
new file mode 100644
index 0000000000000000000000000000000000000000..4e749836ae31170b1812a4cf3a34319c6d9b520a
--- /dev/null
+++ b/lib/src/nesting_level.dart
@@ -0,0 +1,78 @@
+// Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+library dart_style.src.nesting_level;
+
+import 'fast_hash.dart';
+
+/// A single level of expression nesting.
+///
+/// When a line is split in the middle of an expression, this tracks the
+/// context of where in the expression that split occurs. It ensures that the
+/// [LineSplitter] obeys the expression nesting when deciding what column to
+/// start lines at when split inside an expression.
+///
+/// Each instance of this represents a single level of expression nesting. If we
+/// split at to chunks with different levels of nesting, the splitter ensures
+/// they each get assigned to different columns.
+///
+/// In addition, each level has an indent. This is the number of spaces it is
+/// indented relative to the outer expression. It's almost always
+/// [Indent.expression], but cascades are special magic snowflakes and use
+/// [Indent.cascade].
+class NestingLevel extends FastHash {
+ /// The nesting level surrounding this one, or `null` if this is represents
+ /// top level code in a block.
+ NestingLevel get parent => _parent;
+ NestingLevel _parent;
+
+ /// The number of characters that this nesting level is indented relative to
+ /// the containing level.
+ ///
+ /// Normally, this is [Indent.expression], but cascades use [Indent.cascade].
+ final int indent;
+
+ /// The total number of characters of indentation from this level and all of
+ /// its parents, after determining which nesting levels are actually used.
+ ///
+ /// This is only valid during line splitting.
+ int get totalUsedIndent => _totalUsedIndent;
+ int _totalUsedIndent;
+
+ bool get isNested => _parent != null;
+
+ NestingLevel() : indent = 0;
+
+ NestingLevel._(this._parent, this.indent);
+
+ /// Creates a new deeper level of nesting indented [spaces] more characters
+ /// that the outer level.
+ NestingLevel nest(int spaces) => new NestingLevel._(this, spaces);
+
+ /// Clears the previously calculated total indent of this nesting level.
+ void clearTotalUsedIndent() {
+ _totalUsedIndent = null;
+ if (_parent != null) _parent.clearTotalUsedIndent();
+ }
+
+ /// Calculates the total amount of indentation from this nesting level and
+ /// all of its parents assuming only [usedNesting] levels are in use.
+ void refreshTotalUsedIndent(Set<NestingLevel> usedNesting) {
+ if (_totalUsedIndent != null) return;
+
+ _totalUsedIndent = 0;
+
+ if (_parent != null) {
+ _parent.refreshTotalUsedIndent(usedNesting);
+ _totalUsedIndent += _parent.totalUsedIndent;
+ }
+
+ if (usedNesting.contains(this)) _totalUsedIndent += indent;
+ }
+
+ String toString() {
+ if (_parent == null) return indent.toString();
+ return "$parent:$indent";
+ }
+}
« no previous file with comments | « lib/src/nesting_builder.dart ('k') | lib/src/rule/rule.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698