From f6b4e597ce05e33cfc34229e8fb76c3df6206cbf Mon Sep 17 00:00:00 2001
From: Paul Gardner-Stephen <paul@servalproject.org>
Date: Wed, 24 Apr 2019 09:50:57 +0930
Subject: [PATCH] fix bug in similarity phase duplicate rejection

---
 src/similarity.c | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/similarity.c b/src/similarity.c
index 0f58591a..41749940 100644
--- a/src/similarity.c
+++ b/src/similarity.c
@@ -85,6 +85,14 @@ int main(int argc,char **argv)
       
       for(k=0;((i+k)<s1)&&((j+k)<s2);k++)
 	if (f1[i+k]!=f2[j+k]) break;
+
+      // When we find a match, we should then mask off this region,
+      // so that we don't find all the sub-sets of the matches.
+      // e.g. matching BASIC at $100 will result in a match of ASIC at $101
+      // and SIC at $102 etc.  These are all the same match, so should be
+      // suppressed.
+      phase_mask[phase]=k;
+      
       // Ignore matches that are all the same byte
       for(l=1;l<k;l++) if (f1[i+l]!=f1[i]) break;
       if (l<k) {
@@ -123,13 +131,6 @@ int main(int argc,char **argv)
 	  
 	}
       }
-
-      // When we find a match, we should then mask off this region,
-      // so that we don't find all the sub-sets of the matches.
-      // e.g. matching BASIC at $100 will result in a match of ASIC at $101
-      // and SIC at $102 etc.  These are all the same match, so should be
-      // suppressed.
-      phase_mask[phase]=k;
     }
   }