Skip to content

Conversation

@ogoffart
Copy link
Member

@ogoffart ogoffart commented Nov 3, 2025

This line was added by mistake in a previous commit.

But the feature can't work for C++ and the interpreter for now because they can't re-use the same binding.
It would also break for eust if there was another layer of two way binding.
That's because impl BindingCallable for TwoWayBindingWithMap doesn't (and can't) implement intercept_set_binding

@ogoffart ogoffart requested a review from tronical November 3, 2025 13:03
@ogoffart ogoffart force-pushed the olivier/properties2 branch from 3e49be2 to 6f5cb00 Compare November 3, 2025 13:04
This line was added by mistake in a previous commit.

But the feature can't work for C++ and the interpreter for now because
they can't re-use the same binding.
It would also break for eust if there was another layer of two way
binding.
That's because `impl BindingCallable for TwoWayBindingWithMap` doesn't
(and can't) implement intercept_set_binding
@ogoffart ogoffart force-pushed the olivier/properties2 branch from 6f5cb00 to 4b6c8ea Compare November 3, 2025 15:16
@ogoffart ogoffart changed the title Rust: Fiw two way binding to different structs Rust: Fix two way binding to different structs Nov 4, 2025
@ogoffart ogoffart merged commit 184102a into master Nov 5, 2025
41 checks passed
@ogoffart ogoffart deleted the olivier/properties2 branch November 5, 2025 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants