tracing icon indicating copy to clipboard operation
tracing copied to clipboard

subscriber: skip padding when skipping `log.*` fields in `DefaultVisitor`

Open Porges opened this issue 1 year ago • 1 comments

Motivation

Closes #2979.

The current behaviour of DefaultVisitor is that it will write padding even if it is going to skip writing a value, which results in extraneous padding being added when values are skipped by the tracing-log integration.

Solution

With this change, DefaultVisitor will only insert padding if it is actually going to write a value.

Porges avatar May 23 '24 00:05 Porges

same here

taikulawo avatar Jul 18 '24 05:07 taikulawo

Hi @hds, it's been a while since tracing-log 2.0. Would you mind publishing a new version?

xxchan avatar Jan 24 '25 07:01 xxchan