Some cleanup and provide actual performance result

kervinck · kervinck · commit 6cfbc40ea46f · 2015-09-05T18:14:42.000+02:00
diff --git a/Makefile b/Makefile
@@ -8,7 +8,4 @@ pfkpk: pfkpk.c kpk.c kpk.h
 test:
 	./pfkpk
 
-clean:
-	rm -f *.o
-
 # vi: noexpandtab
diff --git a/README.md b/README.md
@@ -9,14 +9,21 @@ the file `LICENSE' for details.
 This implementation is quite fast because it handles the 64 possible
 black king locations superposition in all operations.
 
+```
+kpkGenerate CPU time [seconds]: 0.001579
+kpkTable size [bytes]: 32768
+kpkSelfCheck: OK
+kpkProbe 24/24: OK
+```
+
 The implementation is agnostic to board geometry.  See the file
 `kpk.h' for instructions on how to adapt to an alternative square
 indexing.
 
 Distance to conversion (DTC) is not provided, but is trivial to
 add.  Please contact the author if you need advice.
 
-Marcel van Kervinck
+Marcel
 
 ```
 /*
diff --git a/kpk.c b/kpk.c
@@ -103,7 +103,6 @@ int kpkGenerate(void)
 {
         uint64_t valid[64*32];
 
-        // Prepare won
         for (int ix=0; ix<arrayLen(kpkTable[0]); ix++) {
                 int wK = wKsquare(ix), wP = wPsquare(ix);
 
@@ -115,6 +114,7 @@ int kpkGenerate(void)
                         kpkTable[black][ix] = lost;
                 }
 
+                // Valid positions after black move, pawn capture allowed
                 valid[ix] = ~king64(bit(wK));
                 if (file(wP) != fileA) valid[ix] &= ~bit(wP+N+W);
                 if (file(wP) != fileH) valid[ix] &= ~bit(wP+N+E);

Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,6 @@ int kpkGenerate(void)`
`103`	`103`	`{`
`104`	`104`	`uint64_t valid[64*32];`
`105`	`105`
`106`		`- // Prepare won`
`107`	`106`	`for (int ix=0; ix<arrayLen(kpkTable[0]); ix++) {`
`108`	`107`	`int wK = wKsquare(ix), wP = wPsquare(ix);`
`109`	`108`
`@@ -115,6 +114,7 @@ int kpkGenerate(void)`
`115`	`114`	`kpkTable[black][ix] = lost;`
`116`	`115`	`}`
`117`	`116`
	`117`	`+ // Valid positions after black move, pawn capture allowed`
`118`	`118`	`valid[ix] = ~king64(bit(wK));`
`119`	`119`	`if (file(wP) != fileA) valid[ix] &= ~bit(wP+N+W);`
`120`	`120`	`if (file(wP) != fileH) valid[ix] &= ~bit(wP+N+E);`