devkitPro / SDL

Simple Directmedia Layer
https://libsdl.org
zlib License
15 stars 12 forks source link

Switch Audio: Added Audio Out usage and touchscreen code rework #23

Closed Wohlstand closed 1 year ago

Wohlstand commented 1 year ago

Replacement for #18

Audio Render is a second layer that converts the audio stream to output. Audio Out is the direct output of the audio stream without conversions.

Description

Existing Issue(s)

1) Audio Render API plays with less stability, and in some cases, may turn into silence because of overloads. Audio Out is the direct output of the audio stream without any conversions. 2) Touchscreen API worked incorrectly

Demo

This is a simple program that draws a white screen and plays the music. When you put fingers, it should draw a tiny red dot at the place of a finger.

Wohlstand commented 1 year ago

I see. Anyway, AUDREN has an unique ability to supply ANY type of audio stream: different sample rates, different sample formats, etc.

So, if somebody want's to submit the sound stream without an ability to convert it to the stream format, the AUDREN will be useful.

Wohlstand commented 1 year ago

Anyway, I'll get rid of AudioRender soon. Right now I'm out of home, will get back soon.

Wohlstand commented 1 year ago

Okay, I completely removed the AudioRender support as you had been requested.

Wohlstand commented 1 year ago

When you will merge this into upstream?

WinterMute commented 1 year ago

When you will merge this into upstream?

4 days ago.

Wohlstand commented 1 year ago

When you will merge this into upstream?

4 days ago.

Ah! Now I see. Usually it's need to merge via "merge" button at GitHub itself to gracefully confirm the fact of the merge. Manual merges by local pushes were suggested to be done if you going to apply some own tweaks on the fly.

WinterMute commented 1 year ago

Normally I'd prefer that the audio & touchscreen changes be done in separate PRs but I just squashed the audio changes manually this time.

Wohlstand commented 1 year ago

In next time, I'll do the separate PR for each feature. Thanks for the notice.

Wohlstand commented 1 year ago

Tested out the thing after recent dkp-pacman sync: now it works just fine!