On 22/02/12 16:37, Dave Mitchell wrote:
On Wed, Feb 22, 2012 at 02:50:46PM +0000, Martin J. Evans wrote:
On 22/02/12 11:09, Dave Mitchell wrote:
It didn't:

It is possible the following is the relevant code after preprocessing:

Hi, it turns out that it's a different section of code, but looking at the
right bit of code, the BOOT: is expanded in DBI.c into a whole bunch of
code, so a declaration can't immediately follow. Hopefully (I keep saying
that, don't I...) the following diff (to apply instead of my previous
diff) will fix it.

diff --git a/DBI.xs b/DBI.xs
index 144cfe5..bf12d3d 100644
--- a/DBI.xs
+++ b/DBI.xs
@@ -4290,10 +4290,13 @@ PROTOTYPES: DISABLE

 BOOT:
+    {
+        MY_CXT_INIT;
+        (void)MY_CXT; /* avoid 'unused variable' warning */
+    }
+
     (void)cv;
     (void)items; /* avoid 'unused variable' warning */
-    MY_CXT_INIT;
-    (void)MY_CXT; /* avoid 'unused variable' warning */
     dbi_bootinit(NULL);

Works - thanks.

And thanks for working on this for me!

Not a problem - I use DBI every day and will benefit from the changes you made.

Martin
--
Martin J. Evans
Easysoft Limited
http://www.easysoft.com

Reply via email to