From: Nicholas Clark <nick@ccl4.org>
Date: Mon, 27 Jun 2011 19:13:39 +0000 (+0200)
Subject: Tidy code in pp_study and Perl_screaminstr()
X-Git-Tag: accepted/trunk/20130322.191538~3484
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=378b4d0f82057e5af983d31c5b48b7f10f4758b3;p=platform%2Fupstream%2Fperl.git

Tidy code in pp_study and Perl_screaminstr()

In pp_study eliminate the variable pos, which duplicates len. ch should be U8,
not I32.

In Perl_screaminstr(), move the declarations of s and x to their point of use,
convert a for loop to a while loop, and avoid incrementing and decrementing s.
found is a boolean.
---

diff --git a/pp.c b/pp.c
index 229e1fa..12ab913 100644
--- a/pp.c
+++ b/pp.c
@@ -707,7 +707,6 @@ PP(pp_study)
 {
     dVAR; dSP; dPOPss;
     register unsigned char *s;
-    register I32 pos;
     register I32 ch;
     register I32 *sfirst;
     register I32 *snext;
@@ -729,9 +728,8 @@ PP(pp_study)
 	*/
 	RETPUSHNO;
     }
-    pos = len;
 
-    Newx(sfirst, 256 + pos, I32);
+    Newx(sfirst, 256 + len, I32);
 
     if (!sfirst)
 	DIE(aTHX_ "do_study: out of memory");
@@ -746,13 +744,13 @@ PP(pp_study)
     for (ch = 256; ch; --ch)
 	*snext++ = -1;
 
-    while (--pos >= 0) {
-	register const I32 ch = s[pos];
+    while (len-- > 0) {
+	const U8 ch = s[len];
 	if (sfirst[ch] >= 0)
-	    snext[pos] = sfirst[ch];
+	    snext[len] = sfirst[ch];
 	else
-	    snext[pos] = -1;
-	sfirst[ch] = pos;
+	    snext[len] = -1;
+	sfirst[ch] = (I32)len;
     }
 
     RETPUSHYES;
diff --git a/util.c b/util.c
index 4d03933..929c776 100644
--- a/util.c
+++ b/util.c
@@ -860,7 +860,7 @@ Perl_screaminstr(pTHX_ SV *bigstr, SV *littlestr, I32 start_shift, I32 end_shift
     register const unsigned char *little;
     register I32 stop_pos;
     register const unsigned char *littleend;
-    I32 found = 0;
+    bool found = FALSE;
     const MAGIC * mg;
     I32 *screamfirst;
     I32 *screamnext;
@@ -916,19 +916,19 @@ Perl_screaminstr(pTHX_ SV *bigstr, SV *littlestr, I32 start_shift, I32 end_shift
     }
     big -= previous;
     do {
-	register const unsigned char *s, *x;
 	if (pos >= stop_pos) break;
 	if (big[pos] == first) {
-	    for (x=big+pos+1,s=little; s < littleend; /**/ ) {
-		if (*s++ != *x++) {
-		    s--;
+	    const unsigned char *s = little;
+	    const unsigned char *x = big + pos + 1;
+	    while (s < littleend) {
+		if (*s != *x++)
 		    break;
-		}
+		++s;
 	    }
 	    if (s == littleend) {
 		*old_posp = pos;
 		if (!last) return (char *)(big+pos);
-		found = 1;
+		found = TRUE;
 	    }
 	}
 	pos = screamnext[pos];