Skip to content

Commit c90a2d9

Browse files
committed
PrettyPrinter: Made includeBox private to improve encapsulation
1 parent dab0407 commit c90a2d9

File tree

1 file changed

+10
-11
lines changed

1 file changed

+10
-11
lines changed

lib/src/printers/pretty_printer.dart

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -160,8 +160,7 @@ class PrettyPrinter extends LogPrinter {
160160
final List<String> excludePaths;
161161

162162
/// Contains the parsed rules resulting from [excludeBox] and [noBoxingByDefault].
163-
late final Map<Level, bool> includeBox;
164-
163+
late final Map<Level, bool> _includeBox;
165164
String _topBorder = '';
166165
String _middleBorder = '';
167166
String _bottomBorder = '';
@@ -192,11 +191,11 @@ class PrettyPrinter extends LogPrinter {
192191
_bottomBorder = '$bottomLeftCorner$doubleDividerLine';
193192

194193
// Translate excludeBox map (constant if default) to includeBox map with all Level enum possibilities
195-
includeBox = {};
194+
_includeBox = {};
196195
for (var l in Level.values) {
197-
includeBox[l] = !noBoxingByDefault;
196+
_includeBox[l] = !noBoxingByDefault;
198197
}
199-
excludeBox.forEach((k, v) => includeBox[k] = !v);
198+
excludeBox.forEach((k, v) => _includeBox[k] = !v);
200199
}
201200

202201
@override
@@ -363,34 +362,34 @@ class PrettyPrinter extends LogPrinter {
363362
String? stacktrace,
364363
) {
365364
List<String> buffer = [];
366-
var verticalLineAtLevel = (includeBox[level]!) ? ('$verticalLine ') : '';
365+
var verticalLineAtLevel = (_includeBox[level]!) ? ('$verticalLine ') : '';
367366
var color = _getLevelColor(level);
368-
if (includeBox[level]!) buffer.add(color(_topBorder));
367+
if (_includeBox[level]!) buffer.add(color(_topBorder));
369368

370369
if (error != null) {
371370
for (var line in error.split('\n')) {
372371
buffer.add(color('$verticalLineAtLevel$line'));
373372
}
374-
if (includeBox[level]!) buffer.add(color(_middleBorder));
373+
if (_includeBox[level]!) buffer.add(color(_middleBorder));
375374
}
376375

377376
if (stacktrace != null) {
378377
for (var line in stacktrace.split('\n')) {
379378
buffer.add(color('$verticalLineAtLevel$line'));
380379
}
381-
if (includeBox[level]!) buffer.add(color(_middleBorder));
380+
if (_includeBox[level]!) buffer.add(color(_middleBorder));
382381
}
383382

384383
if (time != null) {
385384
buffer.add(color('$verticalLineAtLevel$time'));
386-
if (includeBox[level]!) buffer.add(color(_middleBorder));
385+
if (_includeBox[level]!) buffer.add(color(_middleBorder));
387386
}
388387

389388
var emoji = _getEmoji(level);
390389
for (var line in message.split('\n')) {
391390
buffer.add(color('$verticalLineAtLevel$emoji$line'));
392391
}
393-
if (includeBox[level]!) buffer.add(color(_bottomBorder));
392+
if (_includeBox[level]!) buffer.add(color(_bottomBorder));
394393

395394
return buffer;
396395
}

0 commit comments

Comments
 (0)