dart-lang / native / 14708923181

push

github

[native_assets_cli] Replace extension types with classes (#2251)

This PR replaces all extension types with classes.

This PR renames all classes that mostly belong to another class to start with the name of that class.

I'm not entirely satisfied with how this looks, but I think it's a step in the right direction.

The Dart docs can be inspected by `pkgs/native_assets_cli$ dart doc .`.

Open question:

* Should we _not_ export all those helper classes? That would prevent users using these types in their code. But this would be considered a leaked symbol. So this probably doesn't work.
* Should we not generate dartdocs for those helper classes. They usually simply have an `add`, `addAll` or `operator []`. The interesting doc comments are on the accessors that return those classes.

Context:

* Work before splitting up the package: https://github.com/dart-lang/native/issues/999

11 of 20 new or added lines in 3 files covered. (55.0%)

8067 of 10367 relevant lines covered (77.81%)

9.82 hits per line