Alexandre Ardhuin
Alexandre Ardhuin
> This is another one that will likely require a bunch of internal cleanup before landing in the SDK. @pq is there a lot of internal cleanup to do?
> I understand the change correctly, this just addresses false-positives? Yes it should relax the original version. So it shouldn't add new diagnostics.
@pq that's what I tried to say : fewer lints
> We'll need to do an internal dry-run Did you have a chance to run it internally?
As #3379 has been merged can we close this issue?
Currious : is `List.of(iterableOfE)` better than `iterableOfE.toList()` ? If you look at https://api.dart.dev/stable/2.8.4/dart-core/Iterable/toList.html you can see that the implementation relies on `List.from`. Should it be changed to use `List.of` ?
> Does the same advice extend to `Set.of`? And `Map.of`?
AFAICT this works as intended. `SizedBox` can only be used in a limited number of cases to be sure it will act the same as `Container`. If you look at...
With the following snippet you will have a different output if you replace `Container` by `SizedBox` ```dart import 'package:flutter/material.dart'; void main() => runApp(MyApp()); class MyApp extends StatelessWidget { @override Widget...