From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Thu, 5 Jan 2012 12:51:37 +0000 (+0000)
Subject: Initialize wxListItem::m_itemId to -1, not 0.
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/1e40bd4c74cd63318b41adebd5afe63e56b54a5e?ds=inline

Initialize wxListItem::m_itemId to -1, not 0.

Currently we have two redundant pieces of data in wxListEvent: m_itemIndex and
m_item.m_itemId. The former is initialized to -1 by default which makes sense
as it means "no item". But the latter was 0 by default which not only didn't
make sense but also made it inconsistent with m_itemIndex for the events which
really didn't have any associated item.

Fix this by initializing m_itemId to -1 as well.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@70270 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/include/wx/listbase.h b/include/wx/listbase.h
index 287ce790d6..237c593f06 100644
--- a/include/wx/listbase.h
+++ b/include/wx/listbase.h
@@ -354,7 +354,7 @@ protected:
     void Init()
     {
         m_mask = 0;
-        m_itemId = 0;
+        m_itemId = -1;
         m_col = 0;
         m_state = 0;
         m_stateMask = 0;