develooper Front page | perl.libwin32 | Postings from February 2012

Re: [rt.cpan.org #74578] Win32::API::Struct alignment wrong

Thread Previous | Thread Next
From:
Cosimo Streppone via RT
Date:
February 10, 2012 03:43
Subject:
Re: [rt.cpan.org #74578] Win32::API::Struct alignment wrong
Message ID:
rt-3.8.HEAD-17364-1328874216-663.74578-15-0@rt.cpan.org
Fri Feb 10 06:43:35 2012: Request 74578 was acted upon.
Transaction: Correspondence added by cosimo@streppone.it
       Queue: Win32-API
     Subject: Re: [rt.cpan.org #74578] Win32::API::Struct alignment wrong 
   Broken in: 0.64
    Severity: Normal
       Owner: Nobody
  Requestors: DOUGW@cpan.org
      Status: open
 Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=74578 >


On Sun, 05 Feb 2012 20:10:05 +0100, Douglas Wilson via RT  
<bug-Win32-API@rt.cpan.org> wrote:

> On Fri Feb 03 20:51:26 2012, rurban@x-ray.at wrote:
>> Patch looks good. Aldo wanted to have that repeat bit fixed for a long
>> time. Thanks.
>>
>> Now if you can repro the w64 align problem and send the output with
>> debugging info, please?
>
> Like I said before, I don't think 64-bit had anything to do with this
> bug, as I was using a 32-bit perl and dll. I've been reading up on Win32
> alignment and finally completely understand what needs to be done. My
> test structure was suppose to start off with a char[10] and I thought it
> was just a typo in the docs...now I see that it should have "xx" after
> it in the pack/unpack. Attached is an updated patch, which correctly
> fixes the repeat logic, and also I believe fixes the alignment of the
> elements in the Struct (on closer look, the repeat logic is why the
> alignments were wrong). It looks good just from calling getPack and
> getUnpack. I'll test it with the actual dll on Monday.

Thanks everyone involved, Douglas and Reini.

I'm going to release this patch to CPAN
during the weekend.

-- 
Cosimo


Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About