Misterious "ListView_SetItem failed" now with assertion (msw)

[nitram.cero]

Where do I begin?...

ClearAll() is confusing. I used it as if it were a "DeleteAllItems()".

Then when I wanted to SetItem() with a column > 0 on a new item, it failed (as ClearAll also counter-intuitively removes the columns).

Windows' GetLastError() returned nothing at all. It gave me a headache.

I think ClearAll() should be deprecated to give place to ClearItems()/ClearColumns().
Because "clear" in programming usually means a complete removal of data, not the structure that holds it.
It's similar to say that to "clear a grid control" would have to make the columns and rows equal to 1.
It's confusing.

---

On to the fix:

I added an assertion in wxMSW's "wxConvertToMSWListItem" to check if the info parameter column is in a valid range acording to ctrl parameter's column count.

I attached the patch (made from SVN 60049)

Also commented out a "item.cchTextMax = 0;" before the SetItem call that was absurd. If needed for any reason, it should be at wxConvertToMSWListItem() instead of being in SetItem function like a hack.

Regards
-Martín

[nitram.cero]

the fix

[VZ]

(In [60055]) remove "item.cchTextMax=0" line from SetItem() as it doesn't seem to make any sense (see #10687)

[VZ]

(In [60056]) added assert checking the column index validity (see #10687)

[vadz]

I agree that ClearAll() may be confusing but it's probably not confusing enough to remove it. And after all chances of not noticing that it doesn't do what you expect are rather low.

Thanks for the assert and other fixes!

[nitram.cero]

Hehe, no problem.

That was probably my anger talking, because of wasting half an hour tracing a bug on a damn list item!
Probably now that the assertion raises it's less common to get that error.
(I found about 4 topics asking about the "ListView_SetItem() failed" error on wx lists and wxpython lists, so go figure)

Regards
-Martín

[rk]

The new assert added in r60056 makes it impossible to use wxListCtrl in wxLC_LIST mode. In this mode you are not required to add any columns. When I try to insert an item now the assert will wrongly detect this as an error. You can see this faulty behavior in the listctrl sample by switching it to the "List View" mode. So either the documentation is now wrong in telling me that in wxLC_LIST mode the columns are created automatically or the assert should only be valid in report mode.

[nitram.cero]

Probably GetColumnCount() is returning 0 instead of 1 on non-report listviews.

I can't check it up now, I'm filled with work... is either that or the column not being added as you say.

Sorry for de inconvenience, as it's my patch :p

[rk]

GetColumnCount() is in fact returning 0 in list mode. So maybe the assert is not the problem after all but only revealed a problem in the ListCtrl itself.

[VZ]

(In [60277]) correct the assert added by r60056 to not trigger in non-report modes (see #10687)

[vadz]

Thanks for noticing this, rk, should be fixed by the above commit.