On Mon Oct 2 17:10:50 2023 +0000, Rémi Bernon wrote:
Maybe better to keep the `FIXME` and change them to "semi-stub", because this doesn't really do anything (unless you have a plan to make them do something useful shortly).
Aren't these methods just properties that get and set the values that are passed to them? Are you referring to the media control? I think that's implemented in the MediaPlayer class. This class seems to mainly house the properties of the media so that MediaPlayer can populate and update the UI. It looks like function that handles updating the UI is `ISystemMediaTransportControlsDisplayUpdater::Update()`, which Roon calls.
Also, this class can be used with custom media players just to keep track of the properties.
One last thing, there's a lot of properties that Roon calls, I don't think it's particularly helpful to keep them as `FIXME`, but it's your call.