Skip to content

Significantly reduce string creation - #26

Merged
FourierTransformer merged 2 commits into
mainfrom
fewer-strings
Jan 28, 2026
Merged

Significantly reduce string creation#26
FourierTransformer merged 2 commits into
mainfrom
fewer-strings

Conversation

@FourierTransformer

@FourierTransformer FourierTransformer commented Jan 27, 2026

Copy link
Copy Markdown
Owner

Changes

  • swapped to string.format style logging
  • added a small static string builder using varargs (tadd!)
    • I like the interface, if I wasn't going for perf (which maybe I shouldn't) it would be more class-style
  • Tried to rename the actual classes to CamelCase (including filename), still trying to figure out how best to name the types.
    • Turns out this was only ThreadPool, renamed other things to snake_case since they were "static" modules. Even logging returned logger classes, but is just a logging module as imported.

closes #25

@FourierTransformer FourierTransformer changed the title Significantly reduce allocations Significantly reduce string creation Jan 28, 2026
@FourierTransformer
FourierTransformer marked this pull request as ready for review January 28, 2026 02:08
@FourierTransformer
FourierTransformer merged commit f51ee16 into main Jan 28, 2026
29 checks passed
@FourierTransformer
FourierTransformer deleted the fewer-strings branch January 28, 2026 02:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Generate fewer intermediary strings

1 participant