First approach to for a toggable printableStringConsumer

Author: 434b

packages/core/FuzzedDataProvider.ts

@@ -404,28 +404,15 @@ export class FuzzedDataProvider {
 		return result;
 	}
 
-	/**
-	 * Wrapping subtraction function that returns a number within a set limit
-	 * @param min is a number representing the minuend
-	 * @param sub is a number representing the subtrahend
-	 * @param lim sets the upper boundary
-	 * @returns a number n with the constraint 0 <= n < lim, as long as lim >=2
-	 */
-	wrappingSub(min: number, sub: number, lim: number): number {
-		if (lim < 2) {
-			throw new Error("Upper boundary should be greater than or equal to 2");
-		}
-		return (min - sub + lim) % lim;
-	}
-
 	/**
 	 * A simple conversion helper that yields the ASCII representaion of a character.
 	 * @param char is a single character from a string represented as a number
 	 * @returns a 'string' from a single character that was represented by a number
 	 */
 	toAscii(char: number): number {
 		if (char < 32 || char > 126) {
-			char = (this.wrappingSub(char, 32, 255) % 95) + 32;
+			const moduloInRange = (char - 32 + 256) % 256;
+			return (moduloInRange % 95) + 32;
 		}
 		return char;
 	}
@@ -436,6 +423,8 @@ export class FuzzedDataProvider {
 	 * (non-smart) but printable strings, IFF the entropy of the input buffer is
 	 * high enough. Limitations are that there are collisions such that, e.g.:
 	 * the conversion of values: `0x1`, `0x42`, and `0xa2` all yield the character `B`.
+	 * Elements in `buf` that are already in ASCII printable range are not undergoing
+	 * any conversion.
 	 * @param buf - Buffer that contains arbitrary values
 	 * @param min - lower bound at which processing of the provided `Buffer` shall begin
 	 * @param max - upper bound, analogous to the lower bound