fix: use unsigned char cast and sizeof in LogEscape escape sequence #255

Found a small correctness issue in LogEscape():

• When char is signed (common on x86_64 Linux), negative byte values (e.g. 0xFF = -1) were being printed as ffffffff instead of ff.

• This fixes it by casting to unsigned char before %02x.

• Also changed the hardcoded 4 to sizeof(escape) so the code doesn't break silently if someone increases the buffer size later.

No performance impact, just safer and more portable.

<!--e57a25ab6845829454e8d69fc972939a-->

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

<!--021abf342d371248e50ceaed478a90ca-->

Reviews

See the guideline for information on the review process.

If your review is incorrectly listed, please copy-paste <code>&lt;!--meta-tag:bot-skip--&gt;</code> into the comment that the bot should ignore.

<!--5faf32d7da4f0f540f40219e4f7537a3-->

Code review ACK 554a481ea73322c7c233dbc7b077a3cd087a9ef7. Signed fix and sizeof cleanup both makes sense. Nice catch!