https://bugs.kde.org/show_bug.cgi?id=422529
--- Comment #23 from feat...@engineer.com ---
Going over the code, my suggestion is making an overload for KConfig
constructor, which takes an enum class ConfigAssociation { kde-app, plasma } as
a fourth argument, in addition to file, mode and type (but present before type
since it won't have a default value).
move the code in the current constructor to a private function, and call it
from both the old constructor and the new suggested constructor, while also
adding [[deprecated( "This constructor is deprecated, please specify
association" )]] to the old constructor.
The new constructor would check whether file exists, if so it'd move it to the
appropriate prefix depending on the ConfigAssociation, prepend that prefix to
file, and call the private constructor mentioned above.
This would make sure code for applications doesn't break, and once it's been
adopted, the configuration is seamlessly moved to the new location.
It's a very small change in code... Shall I make a PR?
--
You are receiving this mail because:
You are watching all bug changes.
