Skip to content

Conversation

@Perksey
Copy link
Member

@Perksey Perksey commented Sep 30, 2025

Uses the OpenAL Soft XML registry and headers.

TODO:

  • Fix *Direct not having data types trimmed
  • OpenAL Soft native builds
  • Example
  • Single enum groups still have al/alc prefix
  • Where's the EFX at?

@github-actions
Copy link
Contributor

github-actions bot commented Sep 30, 2025

File Coverage
All files 3%

Minimum allowed coverage is 0%

Generated by 🐒 cobertura-action against 52817a4

@github-actions
Copy link
Contributor

github-actions bot commented Oct 13, 2025

All native library builds modified in this PR shall attempt to be built by CI.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 14, 2025

All native library builds modified in this PR shall attempt to be built by CI.

OutOfMemory = unchecked((uint)0xA005),
StackOverflowEXT = unchecked((uint)0x19CD),
StackUnderflowEXT = unchecked((uint)0x19CE),
CNoError = unchecked((uint)0),
Copy link
Member Author

Choose a reason for hiding this comment

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

?

Copy link
Member Author

Choose a reason for hiding this comment

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

I've put in a PR to fix this: kcat/openal-soft#1200

EventTypeBufferCompleted = unchecked((uint)0x19A4),
EventTypeSourceStateChanged = unchecked((uint)0x19A5),
EventTypeDisconnected = unchecked((uint)0x19A6),
CEventTypeDefaultDeviceChanged = unchecked((uint)0x19D6),
Copy link
Member Author

Choose a reason for hiding this comment

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

?

Copy link
Member Author

Choose a reason for hiding this comment

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

I've put in a PR to fix this: kcat/openal-soft#1200

FormatRear8 = unchecked((uint)0x1207),
FormatRear16 = unchecked((uint)0x1208),
FormatRear32 = unchecked((uint)0x1209),
Format51Chn8 = unchecked((uint)0x120A),
Copy link
Member Author

Choose a reason for hiding this comment

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

Did we want to change the trimmer so that we trim even if the name would start with a number after trimming (and then add an X prefix as we've done elsewhere)?

@github-actions
Copy link
Contributor

github-actions bot commented Nov 11, 2025

Some of the native library builds modified in this PR were not referenced in the PR description. Please ensure that the PR description contains /build-native OpenAL-Soft SDL. These libraries won't be rebuilt without this being specified. If you believe this is in error, then please write a comment explaining why and ignore this suggestion. This comment will be automatically updated if rectified.

@Perksey
Copy link
Member Author

Perksey commented Nov 11, 2025

Ignore the bot, I have committed new CI-built binaries previously :)

@Perksey
Copy link
Member Author

Perksey commented Nov 11, 2025

Ready for review. Note the outstanding question.

@Perksey Perksey marked this pull request as ready for review November 11, 2025 18:59
@Perksey Perksey requested a review from a team as a code owner November 11, 2025 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

3 participants