comctl32: cchTextMax needs to be set for LVS_OWNERDATA as it uses LVN_GETDISPINFO notifications
bunglehead at gmail.com
Sat Jan 2 08:12:19 CST 2010
On 1/2/2010 13:52, Jeff Latimer wrote:
> On 02/01/10 21:34, Nikolay Sivov wrote:
>> On 1/2/2010 12:48, Jeff Latimer wrote:
>>> dlls/comctl32/listview.c | 1 +
>>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>> + dispInfo.item.cchTextMax = textlenT(pszText, isW)+2;
>> What does this '2' mean?
> +1 adds the trailing null and +2 allows for the original text to be
> greater than the edited text.
I'm ok with +1 probably, but what about comparison? Why can't it be
greater without this change?
> I thought that one extra byte would allow the test to complete in all
Don't know about tests, but it looks like you're hiding a real problem
with this change.
More information about the wine-devel