Use string concatenation for header building

Open avivkeller opened this issue 1 year ago • 2 comments

This PR uses string concatenation to build the header, rather than repeated redefinitions with +=

Apr 08 '24 23:04 avivkeller

Can't believe no one thought to change this before

Apr 08 '24 23:04 WellHiIGuess

Can't believe no one thought to change this before

Well, for readability reasons, it may have been preferable in the past, but every time += is used, it is a redundant re-assignment of a variable, so by using '...' + '...', I am lowering the re-assignment count.

Technically, template literals could also be used, but for readability reasons, I chose against them

Apr 08 '24 23:04 avivkeller