EmptyEpsilon icon indicating copy to clipboard operation
EmptyEpsilon copied to clipboard

Configuration path should have single source of truth

Open jstsmthrgk opened this issue 4 months ago • 6 comments

Back when I made #2047 I only changed one of apparently four (I realized it because of #1437) places where the configuration path is being decided. It would make sense to just have one single place where the selection logic is run.

My suggestions would be to either:

  • have a static function in PreferencesManager that computes the configuration path and returns it, or
  • compute it in main.cpp and somehow store it in PreferencesManager (either as a preference or with its own getter/setter)

If I get told which way is prefered, I will implement it and make a pull request. (Also if wished I could directly incorporate #1437 in it.)

jstsmthrgk avatar Oct 05 '24 14:10 jstsmthrgk