Fix consistence in memelem functions

Signed-off-by: Mattias Andrée <maandree@kth.se>

1 files changed, 42 insertions, 42 deletions

diff --git a/memelem_inv.c b/memelem_inv.c
index 5c3153d..dace2d4 100644
--- a/memelem_inv.c
+++ b/memelem_inv.c
@@ -4,18 +4,18 @@
 
 
 void *
-libsimple_memelem_inv(const void *hay_, size_t hayn, const void *sub_, size_t subn)
+libsimple_memelem_inv(const void *hay_, const void *sub_, size_t width, size_t n)
 {
-	switch (subn) {
+	switch (width) {
 	case 0:
 		return NULL;
 	case 1:
-		return libsimple_memchr_inv(hay_, *(char *)sub_, hayn);
+		return libsimple_memchr_inv(hay_, *(char *)sub_, n);
 	case 2:
 		{
 			uint16_t *hay = (void *)hay_;
 			uint16_t sub = *(uint16_t *)sub_;
-			for (; hayn--; hay++)
+			for (; n--; hay++)
 				if (*hay != sub)
 					return hay;
 			break;
@@ -24,7 +24,7 @@ libsimple_memelem_inv(const void *hay_, size_t hayn, const void *sub_, size_t su
 		{
 			uint32_t *hay = (void *)hay_;
 			uint32_t sub = *(uint32_t *)sub_;
-			for (; hayn--; hay++)
+			for (; n--; hay++)
 				if (*hay != sub)
 					return hay;
 			break;
@@ -33,7 +33,7 @@ libsimple_memelem_inv(const void *hay_, size_t hayn, const void *sub_, size_t su
 		{
 			uint64_t *hay = (void *)hay_;
 			uint64_t sub = *(uint64_t *)sub_;
-			for (; hayn--; hay++)
+			for (; n--; hay++)
 				if (*hay != sub)
 					return hay;
 			break;
@@ -43,8 +43,8 @@ libsimple_memelem_inv(const void *hay_, size_t hayn, const void *sub_, size_t su
 			char *hay = (void *)hay_;
 			const char *sub = sub_;
 			size_t i;
-			for (; hayn--; hay += subn) {
-				for (i = 0; i < subn; i++)
+			for (; n--; hay += width) {
+				for (i = 0; i < width; i++)
 					if (hay[i] != sub[i])
 						return hay;
 			}
@@ -62,48 +62,48 @@ libsimple_memelem_inv(const void *hay_, size_t hayn, const void *sub_, size_t su
 int
 main(void)
 {
-	assert(!strcmpnul(libsimple_memelem_inv("xxoxx", 5, "x", 0), NULL));
-	assert(!strcmpnul(libsimple_memelem_inv("xxXxx", 5, "x", 0), NULL));
-	assert(!strcmpnul(libsimple_memelem_inv("XXoxx", 5, "x", 0), NULL));
-	assert(!strcmpnul(libsimple_memelem_inv("zzzzz", 5, "z", 0), NULL));
-	assert(!strcmpnul(libsimple_memelem_inv("zzzzx", 4, "z", 0), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("xxoxx", "x", 0, 5), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("xxXxx", "x", 0, 5), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("XXoxx", "x", 0, 5), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("zzzzz", "z", 0, 5), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("zzzzx", "z", 0, 4), NULL));
 
-	assert(!strcmpnul(libsimple_memelem_inv("xxoxx", 5, "x", 1), "oxx"));
-	assert(!strcmpnul(libsimple_memelem_inv("xxXxx", 5, "x", 1), "Xxx"));
-	assert(!strcmpnul(libsimple_memelem_inv("XXoxx", 5, "x", 1), "XXoxx"));
-	assert(!strcmpnul(libsimple_memelem_inv("zzzzz", 5, "z", 1), NULL));
-	assert(!strcmpnul(libsimple_memelem_inv("zzzzx", 4, "z", 1), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("xxoxx", "x", 1, 5), "oxx"));
+	assert(!strcmpnul(libsimple_memelem_inv("xxXxx", "x", 1, 5), "Xxx"));
+	assert(!strcmpnul(libsimple_memelem_inv("XXoxx", "x", 1, 5), "XXoxx"));
+	assert(!strcmpnul(libsimple_memelem_inv("zzzzz", "z", 1, 5), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("zzzzx", "z", 1, 4), NULL));
 
-	assert(!strcmpnul(libsimple_memelem_inv("-x-x-o-x-x", 5, "-x", 2), "-o-x-x"));
-	assert(!strcmpnul(libsimple_memelem_inv("-x-x-X-x-x", 5, "-x", 2), "-X-x-x"));
-	assert(!strcmpnul(libsimple_memelem_inv("-X-X-o-x-x", 5, "-x", 2), "-X-X-o-x-x"));
-	assert(!strcmpnul(libsimple_memelem_inv("-z-z-z-z-z", 5, "-z", 2), NULL));
-	assert(!strcmpnul(libsimple_memelem_inv("-z-z-z-z-x", 4, "-z", 2), NULL));
-	assert(!strcmpnul(libsimple_memelem_inv("---z-z-z-x", 5, "--", 2), "-z-z-z-x"));
+	assert(!strcmpnul(libsimple_memelem_inv("-x-x-o-x-x", "-x", 2, 5), "-o-x-x"));
+	assert(!strcmpnul(libsimple_memelem_inv("-x-x-X-x-x", "-x", 2, 5), "-X-x-x"));
+	assert(!strcmpnul(libsimple_memelem_inv("-X-X-o-x-x", "-x", 2, 5), "-X-X-o-x-x"));
+	assert(!strcmpnul(libsimple_memelem_inv("-z-z-z-z-z", "-z", 2, 5), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("-z-z-z-z-x", "-z", 2, 4), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("---z-z-z-x", "--", 2, 5), "-z-z-z-x"));
 
-	assert(!strcmpnul(libsimple_memelem_inv("--x--x--o--x--x", 5, "--x", 3), "--o--x--x"));
-	assert(!strcmpnul(libsimple_memelem_inv("--x--x--X--x--x", 5, "--x", 3), "--X--x--x"));
-	assert(!strcmpnul(libsimple_memelem_inv("--X--X--o--x--x", 5, "--x", 3), "--X--X--o--x--x"));
-	assert(!strcmpnul(libsimple_memelem_inv("--z--z--z--z--z", 5, "--z", 3), NULL));
-	assert(!strcmpnul(libsimple_memelem_inv("--z--z--z--z--x", 4, "--z", 3), NULL));
-	assert(!strcmpnul(libsimple_memelem_inv("-----z--z--z--x", 5, "---", 3), "--z--z--z--x"));
+	assert(!strcmpnul(libsimple_memelem_inv("--x--x--o--x--x", "--x", 3, 5), "--o--x--x"));
+	assert(!strcmpnul(libsimple_memelem_inv("--x--x--X--x--x", "--x", 3, 5), "--X--x--x"));
+	assert(!strcmpnul(libsimple_memelem_inv("--X--X--o--x--x", "--x", 3, 5), "--X--X--o--x--x"));
+	assert(!strcmpnul(libsimple_memelem_inv("--z--z--z--z--z", "--z", 3, 5), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("--z--z--z--z--x", "--z", 3, 4), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("-----z--z--z--x", "---", 3, 5), "--z--z--z--x"));
 
-	assert(!strcmpnul(libsimple_memelem_inv("---x---x---o---x---x", 5, "---x", 4), "---o---x---x"));
-	assert(!strcmpnul(libsimple_memelem_inv("---x---x---X---x---x", 5, "---x", 4), "---X---x---x"));
-	assert(!strcmpnul(libsimple_memelem_inv("---X---X---o---x---x", 5, "---x", 4), "---X---X---o---x---x"));
-	assert(!strcmpnul(libsimple_memelem_inv("---z---z---z---z---z", 5, "---z", 4), NULL));
-	assert(!strcmpnul(libsimple_memelem_inv("---z---z---z---z---x", 4, "---z", 4), NULL));
-	assert(!strcmpnul(libsimple_memelem_inv("-------z---z---z---x", 5, "----", 4), "---z---z---z---x"));
+	assert(!strcmpnul(libsimple_memelem_inv("---x---x---o---x---x", "---x", 4, 5), "---o---x---x"));
+	assert(!strcmpnul(libsimple_memelem_inv("---x---x---X---x---x", "---x", 4, 5), "---X---x---x"));
+	assert(!strcmpnul(libsimple_memelem_inv("---X---X---o---x---x", "---x", 4, 5), "---X---X---o---x---x"));
+	assert(!strcmpnul(libsimple_memelem_inv("---z---z---z---z---z", "---z", 4, 5), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("---z---z---z---z---x", "---z", 4, 4), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("-------z---z---z---x", "----", 4, 5), "---z---z---z---x"));
 
-	assert(!strcmpnul(libsimple_memelem_inv("-------x-------x-------o-------x-------x", 5, "-------x", 8),
+	assert(!strcmpnul(libsimple_memelem_inv("-------x-------x-------o-------x-------x", "-------x", 8, 5),
 	                  "-------o-------x-------x"));
-	assert(!strcmpnul(libsimple_memelem_inv("-------x-------x-------X-------x-------x", 5, "-------x", 8),
+	assert(!strcmpnul(libsimple_memelem_inv("-------x-------x-------X-------x-------x", "-------x", 8, 5),
 	                  "-------X-------x-------x"));
-	assert(!strcmpnul(libsimple_memelem_inv("-------X-------X-------o-------x-------x", 5, "-------x", 8),
+	assert(!strcmpnul(libsimple_memelem_inv("-------X-------X-------o-------x-------x", "-------x", 8, 5),
 	                  "-------X-------X-------o-------x-------x"));
-	assert(!strcmpnul(libsimple_memelem_inv("-------z-------z-------z-------z-------z", 5, "-------z", 8), NULL));
-	assert(!strcmpnul(libsimple_memelem_inv("-------z-------z-------z-------z-------x", 4, "-------z", 8), NULL));
-	assert(!strcmpnul(libsimple_memelem_inv("---------------z-------z-------z-------x", 5, "--------", 8),
+	assert(!strcmpnul(libsimple_memelem_inv("-------z-------z-------z-------z-------z", "-------z", 8, 5), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("-------z-------z-------z-------z-------x", "-------z", 8, 4), NULL));
+	assert(!strcmpnul(libsimple_memelem_inv("---------------z-------z-------z-------x", "--------", 8, 5),
 	                  "-------z-------z-------z-------x"));
 
 	return 0;