comctl32: implement LISTVIEW_ApproximateViewRect for icon and small icon.
Nikolay Sivov
bunglehead at gmail.com
Fri Oct 2 14:01:47 CDT 2009
James Mckenzie wrote:
> Nikolay Sivov wrote at Oct 2, 2009 11:37 AM (PNT)
>
>> Aric Stewart wrote:
>>
>>> ---
>>> dlls/comctl32/listview.c | 39 ++++++++++++++++++++++++++++++++++-----
>>> 1 files changed, 34 insertions(+), 5 deletions(-)
>>>
>>>
>> Hi, Aric.
>>
>>
>> Also this is unsafe:
>> ---
>> + row_count = wWidth / (rcBox.right - rcBox.left);
>> ---
>>
>
> Is this preceeded with a zero check? Otherwise you could end up with a run-time divide by zero error or worse, a negative row count. Also, if the box were small and the item wide, you could end up with a condition where the box might be larger than the screen height.
>
> James McKenzie
>
Hi, James. Wrongly addressed actually :).
It's better to reuse coord. calculation code here as I mentioned in
reply. Tests needed anyway....
More information about the wine-devel
mailing list