inf-clojure 2022-05-28

@jasonkdarby i left a comment on the switch-to-repl PR about the way CIDER uses (buffer-list) rather than trying to keep a variable up to date

I do really like the idea of not keeping the extra variable around but I don't understand the workflow. I don't personally use it, I just cycle through other-window (with M-o, I'm not entirely insane), so I've never used the similar functionality in CIDER

the workflow is basically the same as you wanted to provide in your PR?

ie, C-c C-z takes you to the repl buffer when you're in a clojure mode buffer. If you are in a repl buffer, C-c C-z takes you to the last inf-clojure-minor-mode buffer (aka clojure source code buffer)

which is what your PR does except it uses a variable to keep track of it and requires you to use the shortcut instead of "just working"ā„¢

Ok, yeah I understood it as "`C-c C-z` takes you to the repl buffer when you're in a clojure mode buffer. If you are in a repl buffer, C-c C-z takes you to the last inf-clojure-minor-mode buffer you invoked C-c C-z from."

I'll give it a shot with your suggestion

right. so just drop the requirement "you invoked C-c C-z from" since that could lead to bewildering results and requires you to remember how you got to the repl

just grab the first buffer from (buffer-list) that has inf-clojure-minor-mode active and that's the target buffer

šŸ‘ 1

lol can you imagine what it's like to work with me

i'm sure you're great. you are improving tooling šŸ™‚ sounds like a great coworker

1

FYI I think I fixed it. To anybody interested, this fix is part of JasonKDarby/inf-clojure:master, along with my other PR.

@jasonkdarby i'll use these two changesets on tuesday and get them merged if all good

2

i’m applying the patches now. sorry about that

@dpsutton did you get a chance to use them?

hey @dpsutton any progress on this issue? https://github.com/nrepl/nrepl/issues/272 I'm also using nrepl + tty & inf-clojure