Skip to content

libShake: fix gcc 15 compilation due incorrect listElementDelete defi…#19143

Merged
LibretroAdmin merged 1 commit into
libretro:masterfrom
webosbrew:libshake-fix
Jun 24, 2026
Merged

libShake: fix gcc 15 compilation due incorrect listElementDelete defi…#19143
LibretroAdmin merged 1 commit into
libretro:masterfrom
webosbrew:libshake-fix

Conversation

@cscd98

@cscd98 cscd98 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Guidelines

  1. Rebase before opening a pull request
  2. If you are sending several unrelated fixes or features, use a branch and a separate pull request for each
  3. If possible try squashing everything in a single commit. This is particularly beneficial in the case of feature merges since it allows easy bisecting when a problem arises
  4. RetroArch codebase follows C89 coding rules for portability across many old platforms check using C89_BUILD=1

Description

Compiling RetroArch under GCC 15 will fail with:

deps/libShake/src/common/helpers.c:19:14: error: conflicting types for ‘listElementDelete’; have ‘ListElement *(ListElement *, ListElement *, void (*)(void *))’
   19 | ListElement *listElementDelete(ListElement *head, ListElement *toDelNode, void(*itemDel)(void *item))
      |              ^~~~~~~~~~~~~~~~~
In file included from deps/libShake/src/common/helpers.c:1:
deps/libShake/src/common/helpers.h:11:14: note: previous declaration of ‘listElementDelete’ with type ‘ListElement *(ListElement *, ListElement *, void (*)(void))’
   11 | ListElement *listElementDelete(ListElement *head, ListElement *toDelNode, void(*itemDel)());

This fixes that.

Same PR made to libShake.

Related Issues

Related Pull Requests

Reviewers

@LibretroAdmin LibretroAdmin merged commit 7de9e48 into libretro:master Jun 24, 2026
43 checks passed
agr541 pushed a commit to agr541/RetroArch that referenced this pull request Jun 26, 2026
agr541 pushed a commit to agr541/RetroArch that referenced this pull request Jun 26, 2026
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