Recursive cached binding

[TysonMN]

Fixes #181

Creates a recursive binding type called CachedBinding<'model, 'msg>. The caching in the OneWayLazy and SubModelSelectedItem cases are removed and now use this cached binding instead.

All the tests pass, and there is an existing test for OneWayLazy that ensures that caching is working.

I left one comment in the code to make sure we have a conversation about it. I will create a comment in this PR on that line next.

[cmeeren]

Thanks a lot! There's a lot to take in here, but I can't see anything obviously wrong. Could you also just verify that all relevant samples still work? If they do, and all tests still pass, then I think it's good to go (after resolving the other conversations).

[TysonMN]

There's a lot to take in here, but I can't see anything obviously wrong.

I tried to separate conceptual changes by commit. Reviewing one commit at a time could help.

Could you also just verify that all relevant samples still work? If they do, and all tests still pass, then I think it's good to go (after resolving the other conversations).

Will do.

[TysonMN]

Could you also just verify that all relevant samples still work? If they do, and all tests still pass, then I think it's good to go (after resolving the other conversations).

Will do.

For good measure, I tried all the samples. Everything seemed to work.

[cmeeren]

For good measure, I tried all the samples. Everything seemed to work.

Great, thanks!

[cmeeren]

Looks good to me. Merge when you're ready :)