PulseAudio: Remove get_latency() caching

This commit is contained in:
Luke Hubmayer-Werner 2021-01-13 17:50:56 +10:30
parent b776cf5498
commit d90626f583
3 changed files with 14 additions and 17 deletions

View file

@ -132,7 +132,7 @@
<method name="get_output_latency" qualifiers="const"> <method name="get_output_latency" qualifiers="const">
<return type="float" /> <return type="float" />
<description> <description>
Returns the audio driver's output latency. Returns the audio driver's output latency. This can be expensive, it is not recommended to call this every frame.
</description> </description>
</method> </method>
<method name="get_speaker_mode" qualifiers="const"> <method name="get_speaker_mode" qualifiers="const">

View file

@ -114,7 +114,7 @@
Number of islands in the 3D physics engine. Number of islands in the 3D physics engine.
</constant> </constant>
<constant name="AUDIO_OUTPUT_LATENCY" value="30" enum="Monitor"> <constant name="AUDIO_OUTPUT_LATENCY" value="30" enum="Monitor">
Output latency of the [AudioServer]. Output latency of the [AudioServer]. Equivalent to calling [method AudioServer.get_output_latency], it is not recommended to call this every frame.
</constant> </constant>
<constant name="MONITOR_MAX" value="31" enum="Monitor"> <constant name="MONITOR_MAX" value="31" enum="Monitor">
Represents the size of the [enum Monitor] enum. Represents the size of the [enum Monitor] enum.

View file

@ -353,27 +353,24 @@ Error AudioDriverPulseAudio::init() {
} }
float AudioDriverPulseAudio::get_latency() { float AudioDriverPulseAudio::get_latency() {
if (latency == 0) { //only do this once since it's approximate anyway lock();
lock();
pa_usec_t palat = 0; pa_usec_t pa_lat = 0;
if (pa_stream_get_state(pa_str) == PA_STREAM_READY) { if (pa_stream_get_state(pa_str) == PA_STREAM_READY) {
int negative = 0; int negative = 0;
if (pa_stream_get_latency(pa_str, &palat, &negative) >= 0) { if (pa_stream_get_latency(pa_str, &pa_lat, &negative) >= 0) {
if (negative) { if (negative) {
palat = 0; pa_lat = 0;
}
} }
} }
if (palat > 0) {
latency = double(palat) / 1000000.0;
}
unlock();
} }
if (pa_lat > 0) {
latency = double(pa_lat) / 1000000.0;
}
unlock();
return latency; return latency;
} }