Commit 055268ea authored by Adam Barth's avatar Adam Barth

Merge pull request #2965 from abarth/rm_analyze_regexp

Remove two `flutter analyze` regular expressions
parents 574b26c7 522030bd
......@@ -942,6 +942,12 @@ abstract class RenderObject extends AbstractNode implements HitTestTarget {
@override
PipelineOwner get owner => super.owner;
// Workaround for https://github.com/dart-lang/sdk/issues/25232
@override
void attach(PipelineOwner owner) {
super.attach(owner);
}
bool _needsLayout = true;
/// Whether this render object's layout information is dirty.
bool get needsLayout => _needsLayout;
......
......@@ -336,6 +336,12 @@ class RenderViewport extends RenderViewportBase with RenderObjectWithChildMixin<
return null;
}
// Workaround for https://github.com/dart-lang/sdk/issues/25232
@override
void applyPaintTransform(RenderBox child, Matrix4 transform) {
super.applyPaintTransform(child, transform);
}
@override
bool hitTestChildren(HitTestResult result, { Point position }) {
if (child != null) {
......@@ -401,6 +407,12 @@ abstract class RenderVirtualViewport<T extends ContainerBoxParentDataMixin<Rende
@override
Rect describeApproximatePaintClip(RenderObject child) => Point.origin & size;
// Workaround for https://github.com/dart-lang/sdk/issues/25232
@override
void applyPaintTransform(RenderBox child, Matrix4 transform) {
super.applyPaintTransform(child, transform);
}
@override
void debugFillDescription(List<String> description) {
super.debugFillDescription(description);
......
......@@ -346,8 +346,6 @@ class AnalyzeCommand extends FlutterCommand {
'Analyzing [${mainFile.path}]...',
new RegExp('^\\[(hint|error)\\] Unused import \\(${mainFile.path},'),
new RegExp(r'^\[.+\] .+ \(.+/\.pub-cache/.+'),
new RegExp('\\[warning\\] Missing concrete implementation of \'RenderObject\\.applyPaintTransform\''), // https://github.com/dart-lang/sdk/issues/25232
new RegExp('\\[warning\\] Missing concrete implementation of \'AbstractNode\\.attach\''), // https://github.com/dart-lang/sdk/issues/25232
new RegExp(r'[0-9]+ (error|warning|hint|lint).+found\.'),
new RegExp(r'^$'),
];
......
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