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

[Patch] Suggestion: Allow to turn off WASAPI build without post-configure patching of SDL_config.h #2589

Closed
SDLBugzilla opened this issue Feb 11, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@SDLBugzilla
Copy link
Collaborator

SDLBugzilla commented Feb 11, 2021

This bug report was migrated from our old Bugzilla tracker.

These attachments are available in the static archive:

Reported in version: HG 2.1
Reported for operating system, platform: Windows (All), All

Comments on the original bug report:

On 2017-09-05 22:10:04 +0000, Vitaly Novichkov wrote:

Hello!

Because is no way to turn off using of laggy and glitchy WASAPI, I did that by post-configure patching of SDL_config.h when using MSVC or MinGW-w64 with Autotools or CMake. (in case of Vanilla MinGW (mingw.org) it is not a problem because WASAPI is absence)

I want to have a ./configure and CMake flags to have the ability to turn off WASAPI (or any other audio API on Windows) rather patching of SDL_config.h file.

On 2018-01-26 05:26:40 +0000, Vitaly Novichkov wrote:

Created attachment 3153
Added option ENABLE_WASAPI_SUPPORT into CMakeLists.txt

This option is enabled by default. Disabling of this option will turn off WASAPI build even it is available in the system.

On 2018-01-27 00:00:42 +0000, Ryan C. Gordon wrote:

We should have this cmake option in any case, but is WASAPI still laggy and glitchy in the latest SDL revision? We made some fixes to it right before Christmas.

--ryan.

On 2018-01-27 07:38:01 +0000, Vitaly Novichkov wrote:

In most case it is no more glitchy, but:

  • on Windows 7 (on Windows 10 is NO SOUND, I already reported that)
  • until you will pass through it an output of powerful codec like libADLMIDI produces a choppy output.
  • when the built application is running on Wine with turned on "Windows 7" mode (with "Windows XP" mode the playback is fine as it doesn't use WASAPI).

The libADLMIDI itself to reproduce the issue with WASAPI:
https://github.com/Wohlstand/libADLMIDI
Built it with "-DCMAKE_BUILD_TYPE=Debug -DWITH_MIDIPLAY=ON" to build a library and the demo player and also in the debug mode. The static-only assembly is needed.
To use the player is easy: just pass through its command line argument any MIDI-file you have around of your hands, even use one of M$'s from %WinDir%/media folder.

On 2018-01-30 12:56:46 +0000, Olav Sørensen wrote:

It still is glitchy on some machines. You get symptoms that sounds like a buffer underrun issue, while making the buffer size higher changes nothing. Does the audio buffer size code handling work correctly at all?

On 2018-07-01 12:27:05 +0000, Ozkan Sezer wrote:

Created attachment 3266
configurable wasapi patch

I have this attached patch, will apply it shortly.

On 2018-07-01 14:04:04 +0000, Ozkan Sezer wrote:

Should be fixed by https://hg.libsdl.org/SDL/rev/e604fe493d45

On 2018-07-01 14:17:00 +0000, Vitaly Novichkov wrote:

Finally!

Thanks a lot! :3

@SDLBugzilla SDLBugzilla added the enhancement New feature or request label Feb 11, 2021
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

No branches or pull requests

1 participant