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.

Sign up for GitHub

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

Jump to bottom

deftype protocol method, mutation not visible #817

Closed

borkdude opened this issue Oct 20, 2022 · 0 comments

Collaborator

borkdude commented Oct 20, 2022

Repro in pure SCI:

user=> (sci/eval-string "(defprotocol ICounter (inc! [_])) (deftype Cnt [^:unsynchronized-mutable n] ICounter (inc! [_] (let [old-n n new-n-set (set! n (inc n)) new-n n] [old-n new-n-set new-n]))) (inc! (->Cnt 1))")
[1 {n 2} 1]

Expected result: [1 2 2].

Originally posted by @borkdude in babashka/babashka#1401 (comment)

The text was updated successfully, but these errors were encountered:

borkdude added a commit that referenced this issue


          Fix #817: mutation should be visible in protocol method

b29e94a

borkdude closed this as completed in

cf600e6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment