assert with wxDataViewCtrl

[Trigve]

I've got assertion in wx\scrlowin.h

virtual wxSize GetSizeAvailableForScrollTarget(const wxSize& size)
    {
        wxASSERT_MSG( m_targetWindow == m_win, "must be overridden" );

        return size;
    }

when using wxDataViewCtrl. I'm attaching a patch

[roebling]

I hope I have fixed this taking the header window area into account.

[Trigve]

Looks like there is another crash, because m_headerArea is unintialized when wxDataViewCtrl::GetSizeAvailableForScrollTarget(...) is called:

wxSize newsize = size;
    if (!HasFlag(wxDV_NO_HEADER))
       newsize.y -= m_headerArea->GetSize().y; // this line, m_headerArea = 0xcdcdcdcd
    
    return newsize;

stack:

 	wxmsw290ud_core_vc_custom.dll!wxWindowBase::GetSize()  Line 327 + 0xd bytes	C++
 	wxmsw290ud_adv_vc_custom.dll!wxDataViewCtrl::GetSizeAvailableForScrollTarget(const wxSize & size={...})  Line 4164 + 0x15 bytes	C++
 	wxmsw290ud_core_vc_custom.dll!wxScrollHelper::AdjustScrollbars()  Line 740 + 0x54 bytes	C++
 	wxmsw290ud_adv_vc_custom.dll!wxDataViewCtrl::OnSize(wxSizeEvent & __formal={...})  Line 4180 + 0x19 bytes	C++

[roebling]

I'm not quite sure from where OnSize is called before the constructor has finished, but I've corrected this, too.

[troelsk]

Make it null

[troelsk]

Unfortunately m_headerArea is uninitialized. New attachment.

[roebling]

Right.