On Mon Jun 24 09:28:32 2024 +0000, Rémi Bernon wrote:
Is this really necessary? I don't feel very excited with the idea of adding more concurrency here, and I don't think it's supposed to work like that, especially for file-based media sources. You are working on the network media source right? It may be better to have a separate implementation for it, especially as it might be better to use Win32 HTTP/network APIs for compatibility purposes. So, I would prefer to leave this change aside for now.
I was just working with the `uridecodebin`-based media source in Proton, not writing a new network media source.
Adding concurrency here works around a Gstreamer bug, which essentially requires that all (enabled) streams are constantly drained (or none are) to make forward progress. I am trying to write a minimal reproducer for it.
I assumed that having truly asynchronous sample requests would be a desirable thing anyway, but if that is not the case I can leave it out for now.