search

bastisawesome / guessinggame_ttv

A Twitch bot to play a word guessing game
BSD 3-Clause "New" or "Revised" License
0 stars 1 forks source link

Remove unnecessary properties #27

Open bastisawesome opened 1 year ago

bastisawesome commented 1 year ago

Most notably in the Game module, properties are used to enforce read-only variables. This has a noticeable performance overhead for a feature that is not being used correctly. Instead, change the property calls to the "private" variables.

bastisawesome commented 2 weeks ago

Moving this out of the 1.1.0 release. Reason: this assumes a performance penalty that may be a micro-optimisation. A much better solution would be to actually profile the code and see if this is a worthwhile endeavour. Issue will remain open just in case.