Skip to content

Improve Rust layout scoping #6811

@romanz

Description

@romanz

Following #6780, #6794 will address the specific instance of the GC "false positive" for confirm_with_info - by explicitly dropping the inner Rust layout object via a context manager (introduced in #6792).

Other instances (when we invoked __del__() to drop homescreen/keyboard inner Rust layout objects) will be also using the context manager above for explicit scoping.

It should be possible to switch the rest of layout-related code to use the same approach (properly scoping layouts' allocation / de-allocation).

We can also introduce #6792 (comment), but in some cases it would probably require some more refactoring (e.g. for with_info & ProgressLayout use-cases).

Metadata

Metadata

Assignees

Labels

codeCode improvements

Type

Projects

Status

🏃‍♀️ In progress

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions