On 08/10/2012 07:24 AM, Paul Marquess wrote:
> Steve Hay [mailto:firstname.lastname@example.org]
>> On 10 August 2012 01:07, Karl Williamson <email@example.com> wrote:
>>> Commit 9d90bd60fec830834e56dfd4cc1aaf499c322831 broke this.
>>> The attached patch adds the missing casts that are required in C++ mode.
> Thanks Karl.
> I assume there isn't a casting issue in Bzip.xs where I made a very similar change recently?
The patch I wrote causes blead to compile again under C++. Since there
was no failure in Bzip2.xs, I didn't look at it until now. In looking
at it now, it appears to me to have the correct casts, and more
importantly, the compiler thinks it's ok.
>>> But I'm not sure what to do with it, since cpan is upstream
>> Sorry about that. I'm Cc'ing Paul Marquess and hoping that he might be
>> able to roll a new release incorporating your patch in time for 5.17.3
>> on 20th?...
> That won't be a problem. I'll try to get it onto CPAN over the weekend.