Skip to content

Conversation

@ApoloApps
Copy link
Contributor

@ApoloApps ApoloApps commented Dec 16, 2025

This is part 2 of this pr

What has changed?

  1. Added more extension functions that prevent intermediate object allocations from a lot of the Skiko api surface
  2. Migrated EnumClass.values() to entries in a lot of classes (maybe some have been missed)

Next
Try (important keyword here :), cause if not wrong there are Skia and Compose diffs on how they're treated) doing less copies of Matrix

Release Notes

Added API that can accept primitive values instead of Rect, RRect, Offset to avoid additional object allocation

@ApoloApps
Copy link
Contributor Author

@MatkovIvan @igordmn would be nice to have it before Xmas holidays :)

@igordmn
Copy link
Collaborator

igordmn commented Dec 19, 2025

@ApoloApps, we can review it only in ~3 weeks

Happy holidays!

@igordmn igordmn requested review from MatkovIvan and igordmn January 12, 2026 14:54
@igordmn
Copy link
Collaborator

igordmn commented Jan 12, 2026

Ignore the CI failures, we currently have issues running external PRs.

@igordmn igordmn self-requested a review January 27, 2026 20:44
Copy link
Collaborator

@igordmn igordmn left a comment

Choose a reason for hiding this comment

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

(requested changes)

ApoloApps and others added 3 commits January 27, 2026 23:41
…se the destructured function overloads (clipRRect, make*)
…s the removal of the deprecation annotations and the revert of the modified associated tests
@ApoloApps ApoloApps requested a review from igordmn January 28, 2026 13:24
@igordmn igordmn merged commit 3af0752 into JetBrains:master Jan 28, 2026
13 checks passed
@igordmn
Copy link
Collaborator

igordmn commented Jan 28, 2026

Thank you! Merged

@ApoloApps ApoloApps deleted the skikoAvoidIntermediateObjects branch January 29, 2026 01:23
MatkovIvan added a commit to JetBrains/compose-multiplatform-core that referenced this pull request Jan 29, 2026
Fixes [CMP-9705](https://youtrack.jetbrains.com/issue/CMP-9705) Update
skiko to 0.9.43

## What's Changed in `skiko`

- JetBrains/skiko#1149
- JetBrains/skiko#1140
- JetBrains/skiko#1052

**Full Changelog**:
http://github.com/JetBrains/skiko/compare/v0.9.41...v0.9.43

(no customer-facing issues/notes from Compose POV)

## Release Notes
N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants