2.7.2 ok?

[Francesco Montorsi]

Julian Smart ha scritto:
> Vadim Zeitlin wrote:
>> On Tue, 31 Oct 2006 13:15:52 +0100 Francesco Montorsi 
>> <f18m_cpp217828 at yahoo.it> wrote:
>>
>> FM> this is because wxCollapsiblePane is being used and thus its 
>> wxWindow is FM> used as parent of a sizer.
>> FM> FM> I attached a fix for this in the wxCollapsiblePane patch - 
>> maybe I FM> forgot to include it in the patch or it wasn't committed. 
>> However I FM> attach it again.
>>
>>  I don't believe it was included in the patch.
>>
>> FM> Index: src/xrc/xh_sizer.cpp
>> FM> ===================================================================
>> FM> RCS file: /pack/cvsroots/wxwidgets/wxWidgets/src/xrc/xh_sizer.cpp,v
>> FM> retrieving revision 1.33
>> FM> diff -bu2 -r1.33 src/xrc/xh_sizer.cpp
>> FM> --- src/xrc/xh_sizer.cpp
>> FM> +++ src/xrc/xh_sizer.cpp
>> FM> @@ -189,7 +189,8 @@
>> FM>                   (m_parentAsWindow->IsKindOf(CLASSINFO(wxPanel)) ||
>> FM>                    m_parentAsWindow->IsKindOf(CLASSINFO(wxFrame)) ||
>> FM> +                  m_parentAsWindow->IsKindOf(CLASSINFO(wxWindow)) ||
>> FM>                    m_parentAsWindow->IsKindOf(CLASSINFO(wxDialog)))
>> FM>                      ), NULL,
>> FM> -                wxT("Incorrect use of sizer: parent is not 
>> 'wxDialog', 'wxFrame' or 'wxPanel'."));
>> FM> +                wxT("Incorrect use of sizer: parent is not 
>> 'wxDialog', 'wxFrame', 'wxWindow' or 'wxPanel'."));
>>   
> These checks are bizarre, what is the point of them? Almost any window 
> class could have a sizer associated with it.
I agree... it looked very strange to me, too.

Francesco