You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The Setting class in logitech_receiver/settings contains dozens of subclasses, which are expected to contain specific properties. However, that's not ensured via implementation.
A dependency graph for the logitech_receiver module can be created via pyreverse
The existing problems are
Complex code with many hard connections, "big ball of mud"
Hard to maintain with several levels of subclasses
No documentation of available/expected properties nor enforced
Maybe data classes to specify properties e.g. for RegisterDpi works better.
Information
Solaar version: 1.1.13
The text was updated successfully, but these errors were encountered:
The Setting class in logitech_receiver/settings contains dozens of subclasses, which are expected to contain specific properties. However, that's not ensured via implementation.
A dependency graph for the logitech_receiver module can be created via pyreverse
The existing problems are
Information
The text was updated successfully, but these errors were encountered: