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

Conversation

@chinmaygarde
Copy link
Contributor

Currently @lyceel has a workaround where they get the transform, create a new one, and call SetTransform. They would like to avoid this.

Fixes flutter/flutter#156604

Currently @lyceel has a workaround where they get the transform, create
a new one, and call SetTransform. They would like to avoid this.

Fixes flutter/flutter#156604
@chinmaygarde chinmaygarde added the e: libimpeller The standalone Impeller library with a single-header API. label Oct 23, 2024
@chinmaygarde chinmaygarde self-assigned this Oct 23, 2024
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

@chinmaygarde chinmaygarde added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 23, 2024
@auto-submit auto-submit bot merged commit 5565cbc into flutter:main Oct 23, 2024
@chinmaygarde chinmaygarde deleted the appendtransform branch October 23, 2024 23:16
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 24, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Oct 24, 2024
flutter/engine@5d7caf7...0b56cb8

2024-10-24 31859944+LongCatIsLooong@users.noreply.github.com Bare-bones iOS FKA implementation (flutter/engine#55964)
2024-10-23 chinmaygarde@google.com [Impeller] libImpeller: Allow appending to the transformation stack. (flutter/engine#56072)
2024-10-23 magder@google.com Add standalone 'Mac clangd' builder to replace 'Linux mac_clangd' orchestrator (flutter/engine#56014)
2024-10-23 skia-flutter-autoroll@skia.org Roll Dart SDK from 75c42f30af7a to dd06a1e3002c (2 revisions) (flutter/engine#56070)
2024-10-23 skia-flutter-autoroll@skia.org Roll Skia from 42f9070e6625 to 53c9663c3b83 (1 revision) (flutter/engine#56069)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC codefu@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
M97Chahboun pushed a commit to M97Chahboun/flutter that referenced this pull request Oct 30, 2024
…7486)

flutter/engine@5d7caf7...0b56cb8

2024-10-24 31859944+LongCatIsLooong@users.noreply.github.com Bare-bones iOS FKA implementation (flutter/engine#55964)
2024-10-23 chinmaygarde@google.com [Impeller] libImpeller: Allow appending to the transformation stack. (flutter/engine#56072)
2024-10-23 magder@google.com Add standalone 'Mac clangd' builder to replace 'Linux mac_clangd' orchestrator (flutter/engine#56014)
2024-10-23 skia-flutter-autoroll@skia.org Roll Dart SDK from 75c42f30af7a to dd06a1e3002c (2 revisions) (flutter/engine#56070)
2024-10-23 skia-flutter-autoroll@skia.org Roll Skia from 42f9070e6625 to 53c9663c3b83 (1 revision) (flutter/engine#56069)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC codefu@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
nick9822 pushed a commit to nick9822/flutter that referenced this pull request Dec 18, 2024
…lutter/engine#56072)

Currently @lyceel has a workaround where they get the transform, create a new one, and call SetTransform. They would like to avoid this.

Fixes flutter#156604
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 e: libimpeller The standalone Impeller library with a single-header API.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Impeller] libImpeller: Expose the utility method to concatenate a transformation on the stack.

2 participants