26 Jun
2024
26 Jun
'24
6:26 p.m.
On Wed Jun 26 07:26:50 2024 +0000, Zhiyi Zhang wrote:
This looks correct at first glance. Do you have an application affected by this? It seems to me changing this only reduces some time and -1 will be returned if it's the last item. Yes, I have. It fails on asserting return value of sending LVM_GETNEXTITEM message. I agree about aftereffect of this fix. Anyway, for me it doesn't make sense to handle ahead of time incorrect item (== infoPtr->nItemCount).
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/5909#note_74482