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>
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