Re: [rt.cpan.org #81416] build broken on macosx - perl 5.14 - wx2.9.4

[Mark Dootson via RT]

Sat Nov 24 17:06:24 2012: Request 81416 was acted upon.
Transaction: Correspondence added by mark.doot...@znix.com
       Queue: Wx
     Subject: Re: [rt.cpan.org #81416] build broken on macosx - perl 5.14 - 
wx2.9.4
   Broken in: 0.9914
    Severity: (no value)
       Owner: Nobody
  Requestors: aduit...@cpan.org
      Status: new
 Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=81416 >


Hi,

On 24/11/2012 21:21, athanasios douitsis via RT wrote:

> This has to do with wxwidgets and not your module, right ? If so, please
> close the bug report straight away.
>

Err - no, Wx is completely broken on Mac for wxWidgets 2.9.4.  I think 
the attached patch should work - though I haven't tested it yet.

I'm just puzzled how I've managed to build my own Mac Wx versions.

Thanks for the report

Mark


--- cpp/app.h   2012-09-26 15:23:47.349000000 +0100
+++ cpp/app.h   2012-11-24 21:53:04.922000000 +0000
@@ -70,7 +70,7 @@
         {
             AV* files;
             files = wxPli_stringarray_2_av( aTHX_ fileNames );
-            wxPliCCback( aTHX, &m_callback, G_DISCARD|G_SCALAR,
+            wxPliCCback( aTHX_ &m_callback, G_DISCARD|G_SCALAR,
                         "S", sv_2mortal( newRV_noinc( (SV*)files ) ) );
         } else
             wxApp::MacOpenFiles( fileNames );
@@ -81,7 +81,7 @@
         dTHX;
         if( wxPliFCback( aTHX_ &m_callback, "MacOpenFile" ) )
         {
-            wxPliCCback( aTHX, &m_callback, G_DISCARD|G_SCALAR, "P", &fileName 
);
+            wxPliCCback( aTHX_ &m_callback, G_DISCARD|G_SCALAR, "P", &fileName 
);
         } else
             wxApp::MacOpenFile( fileName );
     }
@@ -91,7 +91,7 @@
         dTHX;
         if( wxPliFCback( aTHX_ &m_callback, "MacOpenURL" ) )
         {
-            wxPliCCback( aTHX, &m_callback, G_DISCARD|G_SCALAR, "P", &url );
+            wxPliCCback( aTHX_ &m_callback, G_DISCARD|G_SCALAR, "P", &url );
         } else
             wxApp::MacOpenURL( url );
     }
@@ -101,7 +101,7 @@
         dTHX;
         if( wxPliFCback( aTHX_ &m_callback, "MacPrintFile" ) )
         {
-            wxPliCCback( aTHX, &m_callback, G_DISCARD|G_SCALAR, "P", &fileName 
);
+            wxPliCCback( aTHX_ &m_callback, G_DISCARD|G_SCALAR, "P", &fileName 
);
         } else
             wxApp::MacPrintFile( fileName );
     }    
@@ -111,7 +111,7 @@
         dTHX;
         if( wxPliFCback( aTHX_ &m_callback, "MacNewFile" ) )
         {
-            wxPliCCback( aTHX, &m_callback, G_DISCARD|G_SCALAR, NULL );
+            wxPliCCback( aTHX_ &m_callback, G_DISCARD|G_SCALAR, NULL );
         } else
             wxApp::MacNewFile();
     }
@@ -121,7 +121,7 @@
         dTHX;
         if( wxPliFCback( aTHX_ &m_callback, "MacReopenApp" ) )
         {
-            wxPliCCback( aTHX, &m_callback, G_DISCARD|G_SCALAR, NULL );
+            wxPliCCback( aTHX_ &m_callback, G_DISCARD|G_SCALAR, NULL );
         } else
             wxApp::MacReopenApp();
     }