Skip to content

chore: pre-allocate dots arrays#32

Merged
nikku merged 1 commit intomainfrom
pre-allocate-dots
Nov 25, 2025
Merged

chore: pre-allocate dots arrays#32
nikku merged 1 commit intomainfrom
pre-allocate-dots

Conversation

@nikku
Copy link
Member

@nikku nikku commented Nov 24, 2025

Proposed Changes

In the spirit of 9050cb5, pre-allocate an additional array. The array size is fixed, as evident in the source code.

Originally proposed via #27.

Checklist

To ensure you provided everything we need to look at your PR:

  • Brief textual description of the changes present
  • Visual demo attached
  • Steps to try out present, i.e. using the @bpmn-io/sr tool
  • Related issue linked via Closes {LINK_TO_ISSUE} or Related to {LINK_TO_ISSUE}

@bpmn-io-tasks bpmn-io-tasks bot added the needs review Review pending label Nov 24, 2025
@nikku nikku requested review from a team, jarekdanielak and philippfromme and removed request for a team November 24, 2025 17:20
@nikku nikku mentioned this pull request Nov 24, 2025
4 tasks
@nikku nikku merged commit c23c50e into main Nov 25, 2025
3 of 4 checks passed
@nikku nikku deleted the pre-allocate-dots branch November 25, 2025 14:42
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Nov 25, 2025
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

Comments