Skip to content

Update openal-soft to 1.25.2#3154

Merged
halx99 merged 4 commits into
devfrom
oalsoft-1.25.2
May 13, 2026
Merged

Update openal-soft to 1.25.2#3154
halx99 merged 4 commits into
devfrom
oalsoft-1.25.2

Conversation

@halx99
Copy link
Copy Markdown
Collaborator

@halx99 halx99 commented May 12, 2026

Describe your changes

Issue ticket number and link

Checklist before requesting a review

For each PR

  • Add Copyright if it missed:
    - "Copyright (c) 2019-present Axmol Engine contributors (see AUTHORS.md)."

  • I have performed a self-review of my code.

    Optional:

    • I have checked readme and add important infos to this PR.
    • I have added/adapted some tests too.

For core/new feature PR

  • I have checked readme and add important infos to this PR.
  • I have added thorough tests.

Axmol 3.x ------------------------------------------------------------

For each 3.x PR

  • Check the '#include "axmol.h"' and replace it with the needed headers.

@halx99 halx99 added this to the 3.0.0 milestone May 12, 2026
@halx99 halx99 added the enhancement New feature or request label May 12, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates OpenAL Soft to version 1.25.2, introduces support for C++20 modules, adds an SDL3 backend, and enhances SIMD support (NEON/SSE). It also includes significant refactoring for improved type safety and performance, such as the introduction of strict numeric types and non-blocking attributes. The reviewer identified a critical preprocessor error in 3rdparty/openal/alc/export_list.h regarding dangling #else directives, which requires immediate attention.

Comment thread 3rdparty/openal/alc/export_list.h
@halx99 halx99 merged commit bd2b61d into dev May 13, 2026
28 checks passed
@halx99 halx99 deleted the oalsoft-1.25.2 branch May 13, 2026 02:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant