From 37bd7b8855ab72a46e1d3ced13a58ce1bd601aa9 Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 2 Apr 2026 13:35:03 +0100 Subject: [PATCH] ax(ueps): replace placeholder name with concrete value in errTLVValueTooLarge comment AX Principle 2: comments must show HOW with real values, not placeholder names. `oversized` is a non-executable placeholder; `bytes.Repeat([]byte("x"), 256)` is the concrete call that produces the error, matching the example already used in the writeTLV body comment. Co-Authored-By: Charon --- pkg/ueps/packet.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/ueps/packet.go b/pkg/ueps/packet.go index 8f77ab6..0553602 100644 --- a/pkg/ueps/packet.go +++ b/pkg/ueps/packet.go @@ -8,7 +8,7 @@ import ( "io" ) -// writeTLV(buffer, TagPayload, oversized) → errTLVValueTooLarge +// writeTLV(buffer, TagPayload, bytes.Repeat([]byte("x"), 256)) → errTLVValueTooLarge // if err == errTLVValueTooLarge { /* value exceeded 255-byte TLV length limit */ } var errTLVValueTooLarge = sentinelError("TLV value too large for 1-byte length header")