Increased accuracy when getting current audio time using OpenAL SOFT on native

[ninjamuffin99]

First time writing some bindings stuff! so feel free to tell me where I might need to clean up things. (i will clean up random whitespace / imports changes in the morning, bwah!)

wrote some bindings for al_get_sourcedv_soft and broadly followed along side the get_sourcefv stuff.

Using openal soft extension, can get a more precise time of the current playing audio.

Using ninjamuffin99/Lime-Time as sample / benchmark.

(the graphs here are sorta "right to left", and loop back to the bottom every 1000ms.) stats.update(FlxG.sound.music.time % 1000);

Before (60FPS)

The timing of the audio 'plateaus' a bit for at least a frame, which may not seem like much, however this is an issue if you were to attempt to get the current playing time from BETWEEN a frame (say, on a keyboard input via an event listener) or on higher framerates (my demo is at 60fps)

After (60FPS)

The values here are ALWAYS being incremented, due to higher precision. No 'plateau' effect.

[ninjamuffin99]

cleaned up the whitespacing, and removed those unneeded public vars 🤝

[mcagabe19]

openal soft gives null object referance at iOS

[SomeGuyWhoLovesCoding]

openal soft gives null object referance at iOS

Ikr? LMAO 🤣

[MAJigsaw77]

Maybe a define needs to be added as openal soft is not on ios or lime should add support for openal soft on ios.

[mcagabe19]

Maybe a define needs to be added as openal soft is not on ios or lime should add support for openal soft on ios.

OpenAL SOFT is not supported for static_link or for iOS itself so lime needed add support for it

[joshtynjala]

OpenAL SOFT is not supported for static_link or for iOS itself so lime needed add support for it

OpenAL SOFT is LGPL, so as I understand it, it must be dynamically linked.

[mcagabe19]

OpenAL SOFT is LGPL, so as I understand it, it must be dynamically linked.

Makes sense, but still OpenAL SOFT needs iOS fixes/patches

[ACrazyTown]

Any updates on this?

[MAJigsaw77]

Any updates on this?

iOS is currently broken

[ACrazyTown]

Any updates on this?

iOS is currently broken

Yeah I saw, I asked to see if there was any progress on fixing it and getting this ready to merge

[Dimensionscape]

We don't use OpenAL Soft when static linking. Instead we use MojoAL.

See: https://github.com/openfl/lime/blob/ca52db81efa10b1bd8c76e43c714faa9dbbc4c94/project/Build.xml#L26

From what I understand, apple devices have the proprietary licensed OpenAL embedded in the system api's(as opposed to OpenAL Soft) and we use that for those targets here: https://github.com/openfl/lime/blob/ca52db81efa10b1bd8c76e43c714faa9dbbc4c94/project/src/media/openal/OpenALBindings.cpp#L2

Any changes need to take into account for our different OpenAL backends.

[ninjamuffin99]

hmmm I'll have to do lil tests to see how accurate mojoAL is then. It could be that the previous implemtation might work fine on iOS if it's accurate enough

[Dimensionscape]

It feels a little dirty having a function that only works for a specific AL backend, but it sounds like it should be something we can manage uniformly. ✌

[ACrazyTown]

Just wondering, would setCurrentTime also have to account for these changes? I see it references AL.BYTE_OFFSET like the old audio time method did so I'm not quite sure