23 May
2023
23 May
'23
8:57 p.m.
This isn't consistent with the actual ProcessInput() implementation, which refuses to accept more than one buffer.
I made some tests before, the tests show that for WMV decoder DMO, calling multiple ProcessInput() is OK. The input status keeps `DMO_INPUT_STATUSF_ACCEPT_DATA` after delivering a buffer by ProcessInput(). Should I added these tests to Wine?
Probably. As far as I understand we currently reject multiple calls to ProcessInput(). I am concerned about the potential effects of having a GetInputStatus() implementation that isn't consistent with ProcessInput(). It may be fine, though... -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2881#note_33615