Use "(ns $repl-ns)" in dummy file when ns form not found

[vspinu]

Useful in scratch buffers with no ns.

[bbatsov]

I can see the usefulness of this, but I'm worried that it won't be very obvious to the users how we're determining the ns to use. Perhaps we should have commands about setting the scratch buffer ns and displaying it (or something similar).

[vspinu]

How about a message "Using XXX namespace"?

[vspinu]

Perhaps we should have commands about setting the scratch buffer ns and displaying it (or something similar).

How about C-c M-n to behave in scratch buffers like in the repl and interactively ask for the namespace.

[bbatsov]

Might be a bit annoying to get this message for each evaluation.

[bbatsov]

Yes, that's a good idea.

[bbatsov]

We should also display the current namespace in the mode line.

[vspinu]

I have added the stripping of :reload/:reload-all and made cider-repl-set-ns ask interactively for the ns in buffers with no ns form.

To wrap up, the semantics of the interactive evaluation is that all evaluation pins on current ns form. If no such form found, use REPL's current ns instead. I think all other tools should better have the same semantics. nrepl-buffer-ns should eventually go away.

[bbatsov]

To wrap up, the semantics of the interactive evaluation is that all evaluation pins on current ns form. If no such form found, use REPL's current ns instead. I think all other tools should better have the same semantics. nrepl-buffer-ns should eventually go away.

Yes, nrepl-buffer-ns should go away. It's legacy from the oldest days of the project.

[vspinu]

I am finished with this one. All changes in the first commit are aimed towards improving interactive eval functionality in various non-standard contexts (*cider-scratch* buffer, not yet saved clojure buffers, cider-read-eval). See the commit message for more.

[bbatsov]

👍