Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow having bindings referencing const Properties #29

Closed
lemirep opened this issue Mar 31, 2022 · 2 comments
Closed

Allow having bindings referencing const Properties #29

lemirep opened this issue Mar 31, 2022 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@lemirep
Copy link
Contributor

lemirep commented Mar 31, 2022

At the moment I cannot use a const Property in a binding.

const Property<float> someValue = makeBinding(someExpression);
Property<float> boundValue = makeBinding(someValue); // This does not compile

A const Property means it cannot be assigned to again but its value still might change (if the value of the const property is a binding). Therefore, being able to use a const Property in a binding would make sense.

@LeonMatthesKDAB LeonMatthesKDAB added the bug Something isn't working label Apr 1, 2022
@LeonMatthesKDAB
Copy link
Contributor

Hm, this is an issue with the makeBinding function and node operators.
These functions currently take a reference to a Property, even though that could be a const reference.

@LeonMatthesKDAB
Copy link
Contributor

Closed as of #72

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants