On Wed, 19 Aug 2026 19:20:57 GMT, Kevin Rushforth <[email protected]> wrote:
>> Alexander Matveev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> 8305842: Replace qtdemux with MP4 demux based on Media Foundation [v8]
>
> modules/javafx.media/src/main/native/gstreamer/plugins/mfwrapper/mfgstbytestream.cpp
> line 189:
>
>> 187: HRESULT CMFGSTByteStream::Close()
>> 188: {
>> 189: // Nothing to close
>
> Could there be a problem with a pending async read? If so, do you need to
> cancel it here?
Yes, we need to cancel pending async read on Close. I will fix it.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2206#discussion_r3826304531