From 12519fb2fc252dd60755bf4090aedaca0ebc136f Mon Sep 17 00:00:00 2001 From: Yusuke Suzuki Date: Wed, 12 Nov 2025 16:21:40 -0800 Subject: [PATCH] async-fs / sync-fs should generate all possible bytes randomFileContents are using (x % 255), but it only includes [0, 255). So, 0xff is not populated. I think probably (x % 256) is the right one. --- generators/async-file-system.js | 2 +- generators/sync-file-system.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/generators/async-file-system.js b/generators/async-file-system.js index 391ecae2..13497096 100644 --- a/generators/async-file-system.js +++ b/generators/async-file-system.js @@ -43,7 +43,7 @@ async function *randomFileContents() { let result = new ArrayBuffer(numBytes); let view = new Uint8Array(result); for (let i = 0; i < numBytes; ++i) - view[i] = (i + counter) % 255; + view[i] = (i + counter) % 256; yield new DataView(result); } } diff --git a/generators/sync-file-system.js b/generators/sync-file-system.js index 126b2660..62c1159c 100644 --- a/generators/sync-file-system.js +++ b/generators/sync-file-system.js @@ -43,7 +43,7 @@ function *randomFileContents() { let result = new ArrayBuffer(numBytes); let view = new Uint8Array(result); for (let i = 0; i < numBytes; ++i) - view[i] = (i + counter) % 255; + view[i] = (i + counter) % 256; yield new DataView(result); } };