Steffen Mueller wrote:
Hi,
your change to the xsp files of AUI break the xsubpp call for some
reason.
I thought I did a full recompile before committing. Blame old age...
By infinite amounts of clueless editing, I found that the
following patch works. I assume it's just the missing semicolon in
AuiManager.xsp and guess you'll figure it out at a glance anyway.
After you kindly pointed it out, it indeed took only a glance :-)
Thanks!
Mattia
Index: ext/aui/XS/AuiManager.xsp
===================================================================
--- ext/aui/XS/AuiManager.xsp (Revision 2515)
+++ ext/aui/XS/AuiManager.xsp (Arbeitskopie)
@@ -208,9 +208,7 @@
%}
~wxAuiManager()
- %code%{ wxPli_thread_sv_unregister( aTHX_ "Wx::AuiManager",
THIS, ST(0) );
- delete THIS;
- %}
+ %code%{ wxPli_thread_sv_unregister( aTHX_ "Wx::AuiManager",
THIS, ST(0) ); delete THIS; %};
void UnInit();
Fixed.
Index: ext/aui/XS/AuiPaneInfo.xsp
===================================================================
--- ext/aui/XS/AuiPaneInfo.xsp (Revision 2515)
+++ ext/aui/XS/AuiPaneInfo.xsp (Arbeitskopie)
@@ -26,10 +26,7 @@
%}
~wxAuiPaneInfo()
- %code%{ wxPli_thread_sv_unregister( aTHX_ "Wx::AuiPaneInfo",
THIS, ST(0) );
- if( wxPli_object_is_deleteable( aTHX_ ST(0) ) )
- delete THIS;
- %};
+ %code%{ wxPli_thread_sv_unregister( aTHX_ "Wx::AuiPaneInfo",
THIS, ST(0) ); if( wxPli_object_is_deleteable( aTHX_ ST(0) ) ) delete
THIS; %};
void SafeSet( const wxAuiPaneInfo& source );
Is the hunk above really necessary?
