pp.c: Call subroutine instead of repeat code

Now that toLOWER_utf8() has the intelligence to skip going out to
swashes for Latin1 code points, it's not so critical to bypass calling
it for these (for speed).  It simplifies things not to have the
intelligence repeated.  There is the additional overhead of two function
calls (minus the branches saved), but these could be avoided if it comes
down to it by making them in-line.

1 files changed, 0 insertions, 19 deletions

diff --git a/pp.c b/pp.c
index 5d5b6aee58..5ce73ee1fc 100644
--- a/pp.c
+++ b/pp.c
@@ -4111,24 +4111,6 @@ PP(pp_lc)
 	U8 tmpbuf[UTF8_MAXBYTES_CASE+1];
 
 	while (s < send) {
-	    if (UTF8_IS_INVARIANT(*s)) {
-
-		/* Invariant characters use the standard mappings compiled in.
-		 */
-		*d++ = toLOWER(*s);
-		s++;
-	    }
-	    else if (UTF8_IS_DOWNGRADEABLE_START(*s)) {
-
-		/* As do the ones in the Latin1 range */
-		U8 lower = toLOWER_LATIN1(TWO_BYTE_UTF8_TO_UNI(*s, *(s+1)));
-		CAT_UNI_TO_UTF8_TWO_BYTE(d, lower);
-		s += 2;
-	    }
-	    else {
-		/* Here, is utf8 not in Latin-1 range, have to go out and get
-		 * the mappings from the tables. */
-
 		const STRLEN u = UTF8SKIP(s);
 		STRLEN ulen;
 
@@ -4159,7 +4141,6 @@ PP(pp_lc)
 		Copy(tmpbuf, d, ulen, U8);
 		d += ulen;
 		s += u;
-	    }
 	}   /* End of looping through the source string */
 	SvUTF8_on(dest);
 	*d = '\0';