Skip to content

When SDL is runnig on EGL, pass the EGLDisplay and EGLSurface to GLCa… - #21

Open
vpxyz wants to merge 1 commit into
thorvg:mainfrom
vpxyz:fix-egl
Open

vpxyz wants to merge 1 commit into
thorvg:mainfrom
vpxyz:fix-egl

Conversation

@vpxyz

@vpxyz vpxyz commented Oct 20, 2026

Copy link
Copy Markdown
Collaborator

Ciao, this patch is for #11 issue.
Thanks

issue: #11

@vpxyz
vpxyz requested a review from hermet as a code owner October 20, 2026 12:50
@hermet hermet added the enhancement Improve features label Oct 20, 2026
@hermet
hermet requested a lite review from Copilot October 21, 2026 15:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing. Review the Copilot code review billing settings.

…nvas target.

issue: thorvg#11

Co-authored-by: Hermet Park <hermetpark@gmail.com>

@vpxyz vpxyz left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I have a concern about this change: SDL_EGL_GetWindowSurface and SDL_EGL_GetCurrentDisplay are both not thread-safe, so calling them in resize() might be a problem. What do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improve features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants