From 9fcc2f0a9475fcec1e1070bbf2e5dade03b21168 Mon Sep 17 00:00:00 2001 From: Chris Bracken Date: Thu, 20 Dec 2018 12:06:26 -0800 Subject: [PATCH] [License] Enable prefer_final_fields lint --- engine/src/flutter/tools/licenses/analysis_options.yaml | 1 + engine/src/flutter/tools/licenses/lib/filesystem.dart | 4 ++-- engine/src/flutter/tools/licenses/lib/licenses.dart | 4 ++-- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/engine/src/flutter/tools/licenses/analysis_options.yaml b/engine/src/flutter/tools/licenses/analysis_options.yaml index d3dfa75eec..9ad2263657 100644 --- a/engine/src/flutter/tools/licenses/analysis_options.yaml +++ b/engine/src/flutter/tools/licenses/analysis_options.yaml @@ -47,6 +47,7 @@ linter: - package_prefixed_library_names - prefer_asserts_in_initializer_lists - prefer_equal_for_default_values + - prefer_final_fields - prefer_final_locals - prefer_is_not_empty - prefer_single_quotes diff --git a/engine/src/flutter/tools/licenses/lib/filesystem.dart b/engine/src/flutter/tools/licenses/lib/filesystem.dart index d11d0d0608..27034b57ab 100644 --- a/engine/src/flutter/tools/licenses/lib/filesystem.dart +++ b/engine/src/flutter/tools/licenses/lib/filesystem.dart @@ -530,8 +530,8 @@ class ArchiveDirectory extends IoNode implements Directory { @override final String name; - Map _subdirectories = SplayTreeMap(); - List _files = []; + final Map _subdirectories = SplayTreeMap(); + final List _files = []; void _add(a.ArchiveFile entry, List remainingPath) { if (remainingPath.length > 1) { diff --git a/engine/src/flutter/tools/licenses/lib/licenses.dart b/engine/src/flutter/tools/licenses/lib/licenses.dart index 030dc080df..fec29c4d80 100644 --- a/engine/src/flutter/tools/licenses/lib/licenses.dart +++ b/engine/src/flutter/tools/licenses/lib/licenses.dart @@ -368,8 +368,8 @@ abstract class License implements Comparable { final LicenseType type; Iterable get licensees => _licensees; - List _licensees = []; - Set _libraries = Set(); + final List _licensees = []; + final Set _libraries = Set(); bool get isUsed => _licensees.isNotEmpty;