Skip to content

lib/erl_interface: add proper prototypes in ei_runner.c for GCC 15#10964

Merged
jhogberg merged 1 commit intoerlang:masterfrom
psumbera:solaris1
Apr 13, 2026
Merged

lib/erl_interface: add proper prototypes in ei_runner.c for GCC 15#10964
jhogberg merged 1 commit intoerlang:masterfrom
psumbera:solaris1

Conversation

@psumbera
Copy link
Copy Markdown
Contributor

@psumbera psumbera commented Apr 2, 2026

GCC 15 rejects calls through empty-parameter declarations like

static int readn();
static void dump();

in its default C mode, reporting errors such as "too many arguments to function". GCC 13 and GCC 14 still accept this code in default mode, but GCC 15 follows newer C2x/C23 semantics here.

Fix this by replacing the non-prototype declarations in ei_runner.c with proper prototypes matching the existing definitions.

GCC 15 rejects calls through empty-parameter declarations like

    static int readn();
    static void dump();

in its default C mode, reporting errors such as "too many arguments
to function". GCC 13 and GCC 14 still accept this code in default
mode, but GCC 15 follows newer C2x/C23 semantics here.

Fix this by replacing the non-prototype declarations in ei_runner.c
with proper prototypes matching the existing definitions.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

CT Test Results

  2 files   22 suites   5m 43s ⏱️
158 tests 153 ✅ 5 💤 0 ❌
194 runs  189 ✅ 5 💤 0 ❌

Results for commit 3d1e8a8.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 2, 2026

CLA assistant check
All committers have signed the CLA.

@rickard-green rickard-green added the team:VM Assigned to OTP team VM label Apr 7, 2026
@jhogberg jhogberg self-assigned this Apr 13, 2026
@jhogberg jhogberg merged commit 7d46347 into erlang:master Apr 13, 2026
35 of 54 checks passed
@jhogberg
Copy link
Copy Markdown
Contributor

Merged, thanks for the PR!

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

Labels

team:VM Assigned to OTP team VM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants