Details
-
Bug
-
Resolution: Won't Do
-
P3: Somewhat important
-
None
-
6.5.0 Beta3
-
None
Description
onValueChanged()/subscribe()/addNotifier() should be marked as const member function for QProperty<T> and its variants(QObjectBindableProperty, QObjectComputedProperty, QProperyAlias, and... maybe more?).
Since watching value does not change the value itself, const member function makes sense.
Futhermore, QPropertyChangeHandler and QPropertyNotifier take reference of property to const, so it seems that missing const qualifier is unintentional.
This change won't break API compatibility, but I am not sure about ABI compatibility.