Skip to content

fix: Bepu, illegal write when performing overlap queries - #3275

Merged
xen2 merged 1 commit into
stride3d:masterfrom
Eideren:bepu_overlap
Jul 28, 2026
Merged

xen2 merged 1 commit into
stride3d:masterfrom
Eideren:bepu_overlap

Conversation

@Eideren

@Eideren Eideren commented Jul 17, 2026 •

Copy link
Copy Markdown
Collaborator

PR Details

A line of the initial overlap implementation was left in, creating an illegal write when feeding the collision batcher. Removed the offending line and took care of a todo.

Related Issue

None logged, reported through discord

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My change requires a change to the documentation.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have built and run the editor to try this change out.

@github-actions

Copy link
Copy Markdown
Contributor

🤖 Draft PR — automatic CI is skipped to save runner minutes.

  • Mark the PR ready for review to run the full automatic CI — or add a ci-run-on-draft label to run it now without leaving draft.
  • Or arm a specific opt-in suite: ci-enduser, ci-editor, ci-ios, ci-android.

@Eideren
Eideren marked this pull request as ready for review July 17, 2026 14:41
@xen2
xen2 merged commit 2474314 into stride3d:master Jul 28, 2026
82 of 86 checks passed
@Eideren
Eideren deleted the bepu_overlap branch July 28, 2026 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants