This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Optimise the logic in isPRINT(), possible following Jarkko's bugfix
authorNicholas Clark <nick@ccl4.org>
Sat, 6 Oct 2007 10:27:17 +0000 (10:27 +0000)
committerNicholas Clark <nick@ccl4.org>
Sat, 6 Oct 2007 10:27:17 +0000 (10:27 +0000)
of change 10855. (to the implementation added in change 18)
Nothing that a decent compiler optimiser would have missed.

p4raw-id: //depot/perl@32041

handy.h

diff --git a/handy.h b/handy.h
index c0cd4c8..7bb4a63 100644 (file)
--- a/handy.h
+++ b/handy.h
@@ -443,7 +443,7 @@ Converts the specified character to lowercase.
 #   define isASCII(c)  ((c) <= 127)
 #   define isCNTRL(c)  ((c) < ' ' || (c) == 127)
 #   define isGRAPH(c)  (isALNUM(c) || isPUNCT(c))
 #   define isASCII(c)  ((c) <= 127)
 #   define isCNTRL(c)  ((c) < ' ' || (c) == 127)
 #   define isGRAPH(c)  (isALNUM(c) || isPUNCT(c))
-#   define isPRINT(c)  (((c) > 32 && (c) < 127) || (c) == ' ')
+#   define isPRINT(c)  (((c) >= 32 && (c) < 127))
 #   define isPUNCT(c)  (((c) >= 33 && (c) <= 47) || ((c) >= 58 && (c) <= 64)  || ((c) >= 91 && (c) <= 96) || ((c) >= 123 && (c) <= 126))
 #   define isXDIGIT(c)  (isDIGIT(c) || ((c) >= 'a' && (c) <= 'f') || ((c) >= 'A' && (c) <= 'F'))
 #   define toUPPER(c)  (isLOWER(c) ? (c) - ('a' - 'A') : (c))
 #   define isPUNCT(c)  (((c) >= 33 && (c) <= 47) || ((c) >= 58 && (c) <= 64)  || ((c) >= 91 && (c) <= 96) || ((c) >= 123 && (c) <= 126))
 #   define isXDIGIT(c)  (isDIGIT(c) || ((c) >= 'a' && (c) <= 'f') || ((c) >= 'A' && (c) <= 'F'))
 #   define toUPPER(c)  (isLOWER(c) ? (c) - ('a' - 'A') : (c))