On Tue, 14 Apr 2026 09:19:41 GMT, Marcono1234 <[email protected]> wrote:
>> Weijun Wang has updated the pull request incrementally with one additional >> commit since the last revision: >> >> usage in DerUtils > > test/lib/jdk/test/lib/hexdump/ASN1Formatter.java line 342: > >> 340: // Check if the contents are too long or not >> printable >> 341: byte[] buf = new byte[Math.min(32, len)]; >> 342: int l = in.read(buf, 0, buf.length); > > For consistency with your other changes replacing `read` with `readNBytes`, > should this one here use `readNBytes` as well? Yes, I will do it. Thanks. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30685#discussion_r3079063063
