search

digitaltrails / ddcutil-service

A Dbus ddcutil server for control of DDC Monitors/VDUs
GNU General Public License v2.0
11 stars 1 forks source link

consider adding set_vcp_from(expected_existing_value, new_value) #20

Open digitaltrails opened 6 months ago

digitaltrails commented 6 months ago

Looking to reduce the service calls due to applications polling and checking VCP values. I should consider adding a method like set_vcp_from(expected_existing_value, new_value), this would only set the new value if the existing value met the prerequisite - maybe raise a signal if it doesn't.

digitaltrails commented 6 months ago

I'm not sure this is needed, looking at my own code GetMultipleVcp is often used after a series of SetVcp calls, that's already very efficient. Current experience also indicates that speed isn't a significant issues.

I'm probably going to park this until a demonstrated need arises.