Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix reference-counting in set_event_filter #308

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cjwatson
Copy link

This previously resulted in the following test failure with Perl 5.38.2 on Debian unstable (https://bugs.debian.org/1064761):

t/core_events.t ................. 1/? Can't use an undefined value as a subroutine reference during global destruction.
t/core_events.t ................. Dubious, test returned 22 (wstat 5632, 0x1600)
All 691 subtests passed
        (less 1 skipped subtest: 690 okay)

@ppisar
Copy link

ppisar commented Apr 2, 2024

This issue happens for me when upgrading SDL2 from 2.28.5 to 2.30.1. I build Perl SDL against sdl12-compat-1.2.68 which is a wrapper around SDL2. I confirm that this patch fixes my test failure.

src/Core/Events.xs Outdated Show resolved Hide resolved
This previously resulted in the following test failure with Perl 5.38.2
on Debian unstable (https://bugs.debian.org/1064761):

    t/core_events.t ................. 1/? Can't use an undefined value as a subroutine reference during global destruction.
    t/core_events.t ................. Dubious, test returned 22 (wstat 5632, 0x1600)
    All 691 subtests passed
            (less 1 skipped subtest: 690 okay)
bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Apr 17, 2024
https://build.opensuse.org/request/show/1168524
by user dstoecker + dimstar_suse
- Add 308.patch: Fix reference-counting in set_event_filter
  (gh#PerlGameDev/SDL#308).
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.

None yet

2 participants