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

[Impeller] make VerticesGeometry delegate to the DL class#37835

Merged
auto-submit[bot] merged 5 commits into
flutter-team-archive:mainfrom
jonahwilliams:texture_coordinates
Nov 22, 2022
Merged

[Impeller] make VerticesGeometry delegate to the DL class#37835
auto-submit[bot] merged 5 commits into
flutter-team-archive:mainfrom
jonahwilliams:texture_coordinates

Conversation

@jonahwilliams

@jonahwilliams jonahwilliams commented Nov 22, 2022

Copy link
Copy Markdown
Contributor

Simplify the implementation of VerticesGeometry by delegating directly to the DlVertices object. Additionally reduces copying.

TriangleFan is already tested via CanConvertTriangleFanToTriangles

@jonahwilliams jonahwilliams marked this pull request as ready for review November 22, 2022 17:57
@jonahwilliams

Copy link
Copy Markdown
Contributor Author

I wanted to add texture coordinates support, but the vertices implementation had gotten a bit over complicated - so this PR simplifies it first

@bdero bdero left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@jonahwilliams jonahwilliams added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 22, 2022
@auto-submit auto-submit Bot merged commit a800e76 into flutter-team-archive:main Nov 22, 2022
@jonahwilliams jonahwilliams deleted the texture_coordinates branch November 22, 2022 20:31
auto-submit Bot pushed a commit to flutter/flutter that referenced this pull request Nov 23, 2022
* c05e7ff [Impeller] opt float/sampler into relaxed precision for gles (flutter-team-archive/engine#37828)

* 256dc78 Reland "[web] Avoid returning int from js interop classes." (flutter-team-archive/engine#37627)

* 5ab29c9 [web] Move unicode properties to third_party (flutter-team-archive/engine#37440)

* 10da1a1 Remove setSampler from FragmentShader (flutter-team-archive/engine#37839)

* a800e76 [Impeller] make VerticesGeometry delegate to the DL class (flutter-team-archive/engine#37835)

* 1067cd2 Roll Skia from 3b2d9e4bf668 to 3bd2fe46f6d2 (2 revisions) (flutter-team-archive/engine#37848)

* 2d8e539 change cloneImageElement() to return clone every time (flutter-team-archive/engine#37811)

* cefb954 fix pixel ratio (flutter-team-archive/engine#37268)

* c6b2ced Set nested clip nodes (flutter-team-archive/engine#37850)

* c7ecca8 Roll Skia from 3bd2fe46f6d2 to c098e3c5d932 (2 revisions) (flutter-team-archive/engine#37851)

* afac22d Made platform message responses threadsafe for macos. (flutter-team-archive/engine#37607)

* a805eff fix docs analysis error on setImageSampler (flutter-team-archive/engine#37852)
shogohida pushed a commit to shogohida/flutter that referenced this pull request Dec 7, 2022
…15881)

* c05e7ff [Impeller] opt float/sampler into relaxed precision for gles (flutter-team-archive/engine#37828)

* 256dc78 Reland "[web] Avoid returning int from js interop classes." (flutter-team-archive/engine#37627)

* 5ab29c9 [web] Move unicode properties to third_party (flutter-team-archive/engine#37440)

* 10da1a1 Remove setSampler from FragmentShader (flutter-team-archive/engine#37839)

* a800e76 [Impeller] make VerticesGeometry delegate to the DL class (flutter-team-archive/engine#37835)

* 1067cd2 Roll Skia from 3b2d9e4bf668 to 3bd2fe46f6d2 (2 revisions) (flutter-team-archive/engine#37848)

* 2d8e539 change cloneImageElement() to return clone every time (flutter-team-archive/engine#37811)

* cefb954 fix pixel ratio (flutter-team-archive/engine#37268)

* c6b2ced Set nested clip nodes (flutter-team-archive/engine#37850)

* c7ecca8 Roll Skia from 3bd2fe46f6d2 to c098e3c5d932 (2 revisions) (flutter-team-archive/engine#37851)

* afac22d Made platform message responses threadsafe for macos. (flutter-team-archive/engine#37607)

* a805eff fix docs analysis error on setImageSampler (flutter-team-archive/engine#37852)
gspencergoog pushed a commit to gspencergoog/flutter that referenced this pull request Jan 19, 2023
…15881)

* c05e7ff [Impeller] opt float/sampler into relaxed precision for gles (flutter-team-archive/engine#37828)

* 256dc78 Reland "[web] Avoid returning int from js interop classes." (flutter-team-archive/engine#37627)

* 5ab29c9 [web] Move unicode properties to third_party (flutter-team-archive/engine#37440)

* 10da1a1 Remove setSampler from FragmentShader (flutter-team-archive/engine#37839)

* a800e76 [Impeller] make VerticesGeometry delegate to the DL class (flutter-team-archive/engine#37835)

* 1067cd2 Roll Skia from 3b2d9e4bf668 to 3bd2fe46f6d2 (2 revisions) (flutter-team-archive/engine#37848)

* 2d8e539 change cloneImageElement() to return clone every time (flutter-team-archive/engine#37811)

* cefb954 fix pixel ratio (flutter-team-archive/engine#37268)

* c6b2ced Set nested clip nodes (flutter-team-archive/engine#37850)

* c7ecca8 Roll Skia from 3bd2fe46f6d2 to c098e3c5d932 (2 revisions) (flutter-team-archive/engine#37851)

* afac22d Made platform message responses threadsafe for macos. (flutter-team-archive/engine#37607)

* a805eff fix docs analysis error on setImageSampler (flutter-team-archive/engine#37852)
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

Development

Successfully merging this pull request may close these issues.

3 participants