From 5e784d588d349d87bfd40523917f645aec267d2a Mon Sep 17 00:00:00 2001 From: Karl Williamson Date: Mon, 21 Apr 2014 12:11:40 -0600 Subject: [PATCH] dquote_static.c: Clarify code We output the message when there is a printable ASCII character, so use that as the test instead of complementing the test for a control. This makes the code somewhat clearer. --- dquote_static.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/dquote_static.c b/dquote_static.c index 4c82c77923..1f74ca5777 100644 --- a/dquote_static.c +++ b/dquote_static.c @@ -63,9 +63,7 @@ S_grok_bslash_c(pTHX_ const char source, const bool output_warning) } result = toCTRL(source); - if (output_warning && ! isCNTRL_L1(result)) { - /* We use isCNTRL_L1 above and not simply isCNTRL, because on EBCDIC - * machines, things like \cT map into a C1 control. */ + if (output_warning && isPRINT_A(result)) { U8 clearer[3]; U8 i = 0; if (! isWORDCHAR(result)) {