Skip Menu |
 

This queue is for tickets about the Unicode-Normalize CPAN distribution.

Report information
The Basics
Id: 109577
Status: resolved
Priority: 0/
Queue: Unicode-Normalize

People
Owner: Nobody in particular
Requestors: davem [...] iabyn.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: silence Normalize.xs compiler warning in v1.23
Date: Tue, 24 Nov 2015 14:29:51 +0000
To: bug-Unicode-Normalize [...] rt.cpan.org
From: Dave Mitchell <davem [...] iabyn.com>
Download (untitled) / with headers
text/plain 952b
In blead (and possibly elsewhere), Normalize.xs emits a compiler warning due the 'ix' var, which is made available for XS subs that declare aliases: In file included from Normalize.xs:14:0: Normalize.c: In function ‘XS_Unicode__Normalize_isComp2nd’: ../../XSUB.h:202:20: warning: unused variable ‘ix’ [-Wunused-variable] #define dXSI32 I32 ix = XSANY.any_i32 ^ Normalize.c:1104:5: note: in expansion of macro ‘dXSI32’ dXSI32; ^ The following diff seems to fix it: diff --git a/cpan/Unicode-Normalize/Normalize.xs b/cpan/Unicode-Normalize/Normalize.xs index 36e20b0..bd9ce2d 100644 --- a/cpan/Unicode-Normalize/Normalize.xs +++ b/cpan/Unicode-Normalize/Normalize.xs @@ -810,6 +810,8 @@ isComp2nd(uv) ALIAS: isNFC_MAYBE = 1 isNFKC_MAYBE = 2 + INIT: + PERL_UNUSED_VAR(ix); -- My Dad used to say 'always fight fire with fire', which is probably why he got thrown out of the fire brigade.
Thanks for the patch. This is now in v1.24


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.