Re: Problems with DBI 1.617_903 on MSWin32

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