Oct. 19, 2023
8:24 p.m.
On Mon Oct 16 23:45:56 2023 +0000, Zebediah Figura wrote:
Does the application actually use IAMStreamSelect::Info()? If so we should have tests for it. Also, ideally this'd be a separate patch. If not I'd just drop this hunk. Does "it calls the function, but doesn't really do anything with the result" count? https://github.com/krkrz/krkr2/blob/dec49af97e174d31059c3ccd7efc700ba3c6b788...
It would be satisfied if IStreamSelect::Count was a semi-stub that just returns zero, but I suspect that's not the best idea. But tests, yes, absolutely. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/3938#note_49346