From 8fad4952c578d2c72a08a552dfe48c1bbd64deaf Mon Sep 17 00:00:00 2001 From: Tony Cook Date: Fri, 2 Aug 2013 11:27:10 +1000 Subject: [PATCH] prevent a precedence warning on clang Dumper.xs:245:29: warning: '&&' within '||' [-Wlogical-op-parentheses] ((k >= 7 && k <= 10 || k == 12 || k == 13 || k == 27) ~~~~~~~^~~~~~~~~~ ~~ Dumper.xs:245:29: note: place parentheses around the '&&' expression to silence this warning ((k >= 7 && k <= 10 || k == 12 || k == 13 || k == 27) --- dist/Data-Dumper/Dumper.xs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dist/Data-Dumper/Dumper.xs b/dist/Data-Dumper/Dumper.xs index 20ed7cf..3fdc71f 100644 --- a/dist/Data-Dumper/Dumper.xs +++ b/dist/Data-Dumper/Dumper.xs @@ -242,7 +242,7 @@ esc_q_utf8(pTHX_ SV* sv, const char *src, STRLEN slen, I32 do_utf8, I32 useqq) #ifndef EBCDIC } else if (useqq && /* we can't use the short form like '\0' if followed by a digit */ - ((k >= 7 && k <= 10 || k == 12 || k == 13 || k == 27) + (((k >= 7 && k <= 10) || k == 12 || k == 13 || k == 27) || (k < 8 && (next < '0' || next > '9')))) { grow += 2; } else if (useqq && k <= 31 && (next < '0' || next > '9')) { -- 2.7.4