--
v3: winedmo: Introduce a winedmo_stream callback interface for I/O.
winedmo: Use the stream context as a buffer for larger reads.
winedmo: Use the stream context to track stream position.
winedmo: Allocate a client-side stream context with the demuxers.
winedmo: Implement FFmpeg seek and read with user callbacks.
https://gitlab.winehq.org/wine/wine/-/merge_requests/6480
--
v2: winedmo: Introduce a winedmo_stream callback interface for I/O.
winedmo: Use the stream context as a buffer for larger reads.
winedmo: Use the stream context to track stream position.
winedmo: Allocate a client-side stream context with the demuxers.
winedmo: Implement FFmpeg seek and read with user callbacks.
mfsrcsnk: Create a winedmo_demuxer object on the media sources.
https://gitlab.winehq.org/wine/wine/-/merge_requests/6480
On Wed Sep 11 16:44:51 2024 +0000, Jacek Caban wrote:
> You could just return 0 here, which would allow you to simplify the code
> some more. Then you don't need the assignment above. And without that
> assignment, we could remove `blit_dc != *hdc` check from error handling path.
I think we could remove `blit_bitmap != *bitmap` from error handle too. We don't assign `*bitmap` to `blit_bitmap` before the last `goto err`.
And we could remove `if(blit_dc)` check too. We return FALSE if `NtGdiCreateCompatibleDC` fails and `blit_dc` doesn't change before the last `goto err`.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/6453#note_81990