Skip to content

Parallel drawing#2825

Merged
ZehMatt merged 3 commits intoOpenLoco:masterfrom
ZehMatt:parallel-drawing
Dec 27, 2025
Merged

Parallel drawing#2825
ZehMatt merged 3 commits intoOpenLoco:masterfrom
ZehMatt:parallel-drawing

Conversation

@ZehMatt
Copy link
Copy Markdown
Contributor

@ZehMatt ZehMatt commented Jan 14, 2025

WIP, also depends on #2824 since its racing some font variables.

Needs to be rebased, last commit is the actual change.

Copy link
Copy Markdown
Member

@AaronVanGeffen AaronVanGeffen left a comment

Choose a reason for hiding this comment

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

Even though this is a remarkably small patch, the performance improvement is clearly noticeable. I've not noticed any artefacts either. Nice work!

@AaronVanGeffen AaronVanGeffen added this to the v24.12+ milestone Jan 19, 2025
@duncanspumpkin duncanspumpkin modified the milestones: v24.12+, v25.01+ Jan 24, 2025
@AaronVanGeffen AaronVanGeffen modified the milestones: v25.02, v25.02+ Feb 28, 2025
@AaronVanGeffen AaronVanGeffen removed this from the v25.02+ milestone Mar 23, 2025
@LeftofZen
Copy link
Copy Markdown
Contributor

@ZehMatt should we merge this? It's dependency (#2824) has been merged

@ZehMatt ZehMatt closed this Dec 25, 2025
@ZehMatt ZehMatt deleted the parallel-drawing branch December 25, 2025 20:57
@ZehMatt ZehMatt restored the parallel-drawing branch December 25, 2025 20:57
@ZehMatt ZehMatt reopened this Dec 25, 2025
@ZehMatt ZehMatt marked this pull request as ready for review December 25, 2025 20:59
@AaronVanGeffen AaronVanGeffen added the changelog Requires a changelog entry label Dec 25, 2025
Copy link
Copy Markdown
Member

@AaronVanGeffen AaronVanGeffen left a comment

Choose a reason for hiding this comment

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

Speeds things up considerably. We'll probably need a vsync limiter at this rate! 😄

Could you add a changelog entry? With that, it's good to go, as far as I'm concerned.

@ZehMatt ZehMatt added this to the v25.12+ milestone Dec 27, 2025
@ZehMatt ZehMatt merged commit 8b85022 into OpenLoco:master Dec 27, 2025
14 checks passed
@ZehMatt ZehMatt deleted the parallel-drawing branch December 27, 2025 20:47
@AaronVanGeffen AaronVanGeffen mentioned this pull request Dec 28, 2025
LaPingvino added a commit to LaPingvino/OpenLoco that referenced this pull request Dec 30, 2025
LaPingvino added a commit to LaPingvino/OpenLoco that referenced this pull request Jan 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog Requires a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants