Skip to content
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

Function selection (C-c M-f) doesn't seem to work correctly #1078

Closed
mbuczko opened this issue Apr 20, 2015 · 4 comments
Closed

Function selection (C-c M-f) doesn't seem to work correctly #1078

mbuczko opened this issue Apr 20, 2015 · 4 comments

Comments

@mbuczko
Copy link

mbuczko commented Apr 20, 2015

steps to reproduce:

  • try to change namespace (eg. in repl via C-c M-n)
  • select function via C-c M-f (all functions from selected namespace are listed correctly)
  • when confirmed an exception is thrown: 'Unable to resolve symbol: xxxxx in this context' and namespace gets swtiched to the initial one (boot.user in my case)

CIDER 0.9.0snapshot (package: 20150419.1202)
@expez
Copy link
Member

expez commented Apr 22, 2015

I can't reproduce this.

Is it reproduceable across repl restart? Across projects? Does it only happen when you use boot or with lein too?

@bbatsov
Copy link
Member

bbatsov commented Apr 22, 2015

I was thinking we should probably delete this functionality. Because of this command we keep around a lot of legacy inlined Clojure code.

@expez
Copy link
Member

expez commented Apr 22, 2015

yes, it isn't very useful either. All it does is provide a completing read of available functions. company-mode kind of does that once you start typing anyway...

@bbatsov
Copy link
Member

bbatsov commented Apr 22, 2015

OK, let's kill it. Could you do this?

expez added a commit that referenced this issue Apr 22, 2015
@expez
[Fix #1078] Remove cider-load-fn-into-repl-buffer
0add3b7 
This function isn't terrible useful, as it only provides a completing read of
all the vars available in the namespace to insert into the repl.

Furthermore, company-mode already provides the completing read if you just start
typing.
@expez expez closed this as completed in d0d2a66 Apr 22, 2015
bbatsov added a commit that referenced this issue Apr 22, 2015
@bbatsov
Merge pull request #1082 from clojure-emacs/remove-cider-load-fn-into…
934463c 
…-repl-buffer

[Fix #1078] Remove cider-load-fn-into-repl-buffer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bbatsov @mbuczko @expez