Skip to content

Remove asserts from Android embedding #31928

@dnfield

Description

@dnfield

/cc @matthew-carroll @mklim

We've discussed this before - the Android linter says these are unreliable. I'm now working on a bug that appears to be suffering from this (#31139). We have code that asserts the transform we're about to pass isn't null, but this makes me suspect there are many things we're asserting that are never triggered.

We should follow the linter suggestion and use if (BuildConfig.DEBUG) instead and make sure we're setting that configuration properly for debug versions of the engine.

Metadata

Metadata

Assignees

Labels

a: debuggingDebugging, breakpoints, expression evaluationengineflutter/engine related. See also e: labels.platform-androidAndroid applications specifically

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions