Unverified Commit 33b81ed9 authored by Phil Quitslund's avatar Phil Quitslund Committed by GitHub

fixes for incoming linter (#113794)

* fixes for incomming linter

* + todo

* todo format fix

* justify ignore

* --
parent a440c468
...@@ -483,7 +483,7 @@ class _MenuAnchorState extends State<MenuAnchor> { ...@@ -483,7 +483,7 @@ class _MenuAnchorState extends State<MenuAnchor> {
// close it first. // close it first.
_close(); _close();
} }
assert(_debugMenuInfo('Opening ${this} at ${position ?? Offset.zero} with alignment offset ${widget.alignmentOffset ?? Offset.zero}')); assert(_debugMenuInfo('Opening $this at ${position ?? Offset.zero} with alignment offset ${widget.alignmentOffset ?? Offset.zero}'));
_parent?._closeChildren(); // Close all siblings. _parent?._closeChildren(); // Close all siblings.
assert(_overlayEntry == null); assert(_overlayEntry == null);
...@@ -553,6 +553,8 @@ class _MenuAnchorState extends State<MenuAnchor> { ...@@ -553,6 +553,8 @@ class _MenuAnchorState extends State<MenuAnchor> {
} }
void _closeChildren({bool inDispose = false}) { void _closeChildren({bool inDispose = false}) {
// TODO(pquitslund): remove when interpolated `$this` parsing is fixed (https://github.com/dart-lang/sdk/issues/50263)
// ignore: unnecessary_brace_in_string_interps
assert(_debugMenuInfo('Closing children of ${this}${inDispose ? ' (dispose)' : ''}')); assert(_debugMenuInfo('Closing children of ${this}${inDispose ? ' (dispose)' : ''}'));
for (final _MenuAnchorState child in List<_MenuAnchorState>.from(_anchorChildren)) { for (final _MenuAnchorState child in List<_MenuAnchorState>.from(_anchorChildren)) {
child._close(inDispose: inDispose); child._close(inDispose: inDispose);
......
...@@ -269,9 +269,6 @@ class LayoutBuilder extends ConstrainedLayoutBuilder<BoxConstraints> { ...@@ -269,9 +269,6 @@ class LayoutBuilder extends ConstrainedLayoutBuilder<BoxConstraints> {
required super.builder, required super.builder,
}) : assert(builder != null); }) : assert(builder != null);
@override
LayoutWidgetBuilder get builder => super.builder;
@override @override
RenderObject createRenderObject(BuildContext context) => _RenderLayoutBuilder(); RenderObject createRenderObject(BuildContext context) => _RenderLayoutBuilder();
} }
......
...@@ -32,12 +32,6 @@ class SliverLayoutBuilder extends ConstrainedLayoutBuilder<SliverConstraints> { ...@@ -32,12 +32,6 @@ class SliverLayoutBuilder extends ConstrainedLayoutBuilder<SliverConstraints> {
required super.builder, required super.builder,
}); });
/// Called at layout time to construct the widget tree.
///
/// The builder must return a non-null sliver widget.
@override
SliverLayoutWidgetBuilder get builder => super.builder;
@override @override
RenderObject createRenderObject(BuildContext context) => _RenderSliverLayoutBuilder(); RenderObject createRenderObject(BuildContext context) => _RenderSliverLayoutBuilder();
} }
......
...@@ -34,6 +34,7 @@ Declarer get _declarer { ...@@ -34,6 +34,7 @@ Declarer get _declarer {
Future<void>(() { Future<void>(() {
Invoker.guard<Future<void>>(() async { Invoker.guard<Future<void>>(() async {
final _Reporter reporter = _Reporter(color: false); // disable color when run directly. final _Reporter reporter = _Reporter(color: false); // disable color when run directly.
// ignore: recursive_getters, this self-call is safe since it will just fetch the declarer instance
final Group group = _declarer.build(); final Group group = _declarer.build();
final Suite suite = Suite(group, SuitePlatform(Runtime.vm)); final Suite suite = Suite(group, SuitePlatform(Runtime.vm));
await _runGroup(suite, group, <Group>[], reporter); await _runGroup(suite, group, <Group>[], reporter);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment