Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

[Impeller] Remove pendown trick for solid strokes#36611

Merged
auto-submit[bot] merged 1 commit intoflutter:mainfrom
bdero:bdero/remove-stroke-pendown
Oct 5, 2022
Merged

[Impeller] Remove pendown trick for solid strokes#36611
auto-submit[bot] merged 1 commit intoflutter:mainfrom
bdero:bdero/remove-stroke-pendown

Conversation

@bdero
Copy link
Contributor

@bdero bdero commented Oct 5, 2022

Remove special "pen down" attribute from the solid generator algorithm. The triangle created between contours already has zero area. We already rely on zero area triangles getting discarded by the rasterizer for the overdraw prevention technique we use.

Part of preparing solid strokes for the deferred geometry refactor. See also https://flutter.dev/go/impeller-geometry#heading=h.fymtgh4bhdl2 for a detailed explanation.

Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App e: impeller

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants