comctl32[1/8]: header: don't check lpItem for NULL in GetItemT
mikolaj at zalewski.pl
Sun May 14 02:51:09 CDT 2006
>You will need to split this patch into two. Checking for NULL has nothing
>to do with callback. Also I'm not sure why are you removing NULL check?
>Any explanations, tests?
This patch is already split into eight parts - I hope thay are small
enough. That's true the first patch doesn't have much to do with
callbacks but without it the other patches would have offsets during
patching. I've removed the checks because lpItem can't be NULL (lpItem =
&infoPtr->items[nItem]) and I thought it's useless to have conditions
that are always true. From the CVS history it seems that in the past
lpItem could be NULL for out-of-range indexes but after some time
GetItemT was changed to return FALSE in such cases but the checks remained.
More information about the wine-devel