prefer_collection_literals doesn't know about the new typedefs #2795

Hixie · 2021-07-23T06:33:11Z

typedef Foo = Map<Map<List<Map<List<int>, List<int>>>, int>, int>;

void main() {
  Foo(); // "Use collection literals when possible"
}

There is a basic premise in this rule which we cannot satisfy exactly: we need to disallow `LinkedHashSet()` unless the context type requires the developer to use `LinkedHashSet`. But the context type is long gone when the lint rule is run. This CL adds some logic to try to attempt figuring out the context type in the cases where users have filed bugs, but it will never be super accurate. Fixes dart-lang/linter#4736 Fixes dart-lang/linter#3057 Fixes dart-lang/linter#1649 Fixes dart-lang/linter#2795 Change-Id: I3e6c6de81084dca2825488c89830ab3e7ea63186 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/323680 Reviewed-by: Phil Quitslund <pquitslund@google.com> Reviewed-by: Konstantin Shcheglov <scheglov@google.com> Commit-Queue: Samuel Rawlins <srawlins@google.com>

…pe more" This reverts commit cd8a337. Reason for revert: lint starts barking at the wrong tree: b/298917960 Original change's description: > linter: Refactor prefer_collection_literals to use context type more > > There is a basic premise in this rule which we cannot satisfy exactly: > we need to disallow `LinkedHashSet()` unless the context type requires > the developer to use `LinkedHashSet`. But the context type is long > gone when the lint rule is run. > > This CL adds some logic to try to attempt figuring out the context > type in the cases where users have filed bugs, but it will never be > super accurate. > > Fixes dart-lang/linter#4736 > Fixes dart-lang/linter#3057 > Fixes dart-lang/linter#1649 > Fixes dart-lang/linter#2795 > > Change-Id: I3e6c6de81084dca2825488c89830ab3e7ea63186 > Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/323680 > Reviewed-by: Phil Quitslund <pquitslund@google.com> > Reviewed-by: Konstantin Shcheglov <scheglov@google.com> > Commit-Queue: Samuel Rawlins <srawlins@google.com> Change-Id: I980053dd51ffd4447721e0ad7436b07ca704b554 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/324021 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Reviewed-by: Alexander Thomas <athom@google.com> Commit-Queue: Ilya Yanok <yanok@google.com>

…ntext type more"" This reverts commit cbdae14. In addition, Fix prefer_collection_literals for methods with expression bodies Original description: linter: Refactor prefer_collection_literals to use context type more There is a basic premise in this rule which we cannot satisfy exactly: we need to disallow `LinkedHashSet()` unless the context type requires the developer to use `LinkedHashSet`. But the context type is long gone when the lint rule is run. This CL adds some logic to try to attempt figuring out the context type in the cases where users have filed bugs, but it will never be super accurate. Fixes dart-lang/linter#4736 Fixes dart-lang/linter#3057 Fixes dart-lang/linter#1649 Fixes dart-lang/linter#2795 Change-Id: I958ba69a56866c18523ce6cbf62645ef8e028f6b Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/324260 Commit-Queue: Samuel Rawlins <srawlins@google.com> Reviewed-by: Konstantin Shcheglov <scheglov@google.com>

pq added set-recommended Affects a rule in the recommended Dart rule set false-negative labels Jul 28, 2021

pq added the new-language-feature label Aug 6, 2021

pq added the P2 A bug or feature request we're likely to work on label Nov 14, 2022

srawlins self-assigned this Aug 31, 2023

srawlins closed this as completed Sep 11, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

prefer_collection_literals doesn't know about the new typedefs #2795

prefer_collection_literals doesn't know about the new typedefs #2795

Hixie commented Jul 23, 2021

prefer_collection_literals doesn't know about the new typedefs #2795

prefer_collection_literals doesn't know about the new typedefs #2795

Comments

Hixie commented Jul 23, 2021