On Wed, 19 Aug 2026 19:09:21 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 209: > >> 207: SafeRelease(&m_pAsyncResult); >> 208: >> 209: return S_OK; > > Shouldn't this return `m_readResult`? Yes. I will fix it. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2206#discussion_r3826477756
