AudioTransportSource: hasStreamFinished returns true when stream finished

BREAKING_CHANGES.md

@@ -2,6 +2,25 @@
 
 # Version 8.0.4
 
+
+## Change
+
+The behavior of AudioTransportSource::hasStreamFinished has been updated to correctly return true when the stream has finished.
+
+**Possible Issues**
+
+This change may affect any code that relied on the previous behavior, where the method never returned true.
+
+**Workaround**
+
+Review and update any code that depends on hasStreamFinished or any registered ChangeListeners that respond to stream completion.
+
+**Rationale**
+
+The previous behavior, where hasStreamFinished never returned true, was incorrect.
+This update ensures the method works as intended.
+
+
 ## Change
 
 Support for Arm32 in Projucer has been removed for Windows targets.

modules/juce_audio_devices/sources/juce_AudioTransportSource.cpp

@@ -168,9 +168,13 @@ double AudioTransportSource::getLengthInSeconds() const
 
 bool AudioTransportSource::hasStreamFinished() const noexcept
 {
-    return positionableSource == nullptr
+    if (positionableSource == nullptr)
-        || (positionableSource->getNextReadPosition() > positionableSource->getTotalLength() + 1
+        return true;
-            && ! positionableSource->isLooping());
+
+    if (positionableSource->isLooping())
+        return false;
+
+    return positionableSource->getNextReadPosition() >= positionableSource->getTotalLength();
 }
 
 void AudioTransportSource::setNextReadPosition (int64 newPosition)