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

Side by Side Diff: lib/src/asset/asset_set.dart

Issue 1947773002: Fix all strong-mode warnings. (Closed) Base URL: git@github.com:dart-lang/barback@master
Patch Set: Code review changes Created 4 years, 7 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 unified diff | Download patch
« no previous file with comments | « lib/src/asset/asset_node.dart ('k') | lib/src/asset/internal_asset.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library barback.asset.asset_set; 5 library barback.asset.asset_set;
6 6
7 import 'dart:collection'; 7 import 'dart:collection';
8 8
9 import 'asset.dart'; 9 import 'asset.dart';
10 import 'asset_id.dart'; 10 import 'asset_id.dart';
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 _assets[asset.id] = asset; 47 _assets[asset.id] = asset;
48 return asset; 48 return asset;
49 } 49 }
50 50
51 /// Adds [assets] to the set. 51 /// Adds [assets] to the set.
52 void addAll(Iterable<Asset> assets) { 52 void addAll(Iterable<Asset> assets) {
53 assets.forEach(add); 53 assets.forEach(add);
54 } 54 }
55 55
56 /// Returns `true` if the set contains [asset]. 56 /// Returns `true` if the set contains [asset].
57 bool contains(Asset asset) { 57 bool contains(Object asset) => asset is Asset && _assets[asset.id] == asset;
58 var other = _assets[asset.id];
59 return other == asset;
60 }
61 58
62 /// Returns `true` if the set contains an [Asset] with [id]. 59 /// Returns `true` if the set contains an [Asset] with [id].
63 bool containsId(AssetId id) { 60 bool containsId(AssetId id) {
64 return _assets.containsKey(id); 61 return _assets.containsKey(id);
65 } 62 }
66 63
67 /// If the set contains an [Asset] with [id], removes and returns it. 64 /// If the set contains an [Asset] with [id], removes and returns it.
68 Asset removeId(AssetId id) => _assets.remove(id); 65 Asset removeId(AssetId id) => _assets.remove(id);
69 66
70 /// Removes all assets from the set. 67 /// Removes all assets from the set.
71 void clear() { 68 void clear() {
72 _assets.clear(); 69 _assets.clear();
73 } 70 }
74 71
75 String toString() => _assets.toString(); 72 String toString() => _assets.toString();
76 } 73 }
OLDNEW
« no previous file with comments | « lib/src/asset/asset_node.dart ('k') | lib/src/asset/internal_asset.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698