Skip to content

Conversation

ginnyTheCat
Copy link

@ginnyTheCat ginnyTheCat commented Aug 15, 2025

This makes sure 10 nodes still only have 1 digit as we are going from 0-9. The same applies to 100 or 1000 etc nodes ofc. Not a huge change, just a small polish.

Before:
node counts prefixed with 0

After:
node counts not prefixed with 0

@joto
Copy link
Member

joto commented Aug 24, 2025

Now the functionality is complex enough that it should go into its own function and get a unit test.

@joto
Copy link
Member

joto commented Aug 24, 2025

And maybe we should just use snprintf to tell us how long the output will be? See second paragraph under Notes. Unless that's much more expensive than the log().

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants