Skip to content

Forward pContext to SetAssertionFailureFunction - #595

Open
wmamrak wants to merge 1 commit into
electronicarts:masterfrom
wmamrak:fix-assert
Open

wmamrak wants to merge 1 commit into
electronicarts:masterfrom
wmamrak:fix-assert

Conversation

@wmamrak

@wmamrak wmamrak commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

No description provided.

jmgilbert2 added a commit to jmgilbert2/EASTL that referenced this pull request Oct 21, 2026
…upstream PR electronicarts#595)

The overload taking an EASTL_AssertionFailureFunction set the function
but dropped pContext, so the user's function got a null context. It now
goes through the EASTL_AssertionFailureFunctionEx overload, which stores
the context. GCC reported the dropped parameter with -Wunused-parameter.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
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.

1 participant